CRITICAL FIXES (3):
1. P6.14 (Laravel Vendor Size) - Fixed unit loss in size calculation
• Was comparing "500M" → "500" incorrectly
• Now uses pattern matching for proper MB/G detection
2. P6.22 (System Load) - Fixed integer comparison bug
• Was truncating decimal in load ratio calculation
• Now uses proper floating point comparison with bc
3. P6.18 (Process Limits) - Fixed off-by-one error
• Was counting header line from ps aux
• Now subtracts 1 for actual process count
HIGH SEVERITY FIXES (3):
4. P6.17 (I/O Scheduler) - Added multi-device support
• Was hardcoded to "sda" only
• Now checks sda, sdb, nvme*, vd*, xvd* devices
5. P6.19 (Swap I/O) - Improved vmstat column handling
• Was using ambiguous column positioning
• Now captures both swap_in and swap_out with validation
6. P6.13 (Laravel Cache Driver) - Added whitespace trimming
• Was missing values with leading/trailing spaces
• Now uses xargs and tr for proper quote/space stripping
MEDIUM SEVERITY FIXES (4):
7. P6.10 (Magento Extensions) - Fixed count off-by-one
• Was including root directory in count
• Now uses mindepth=1 to exclude root
8. P6.15 (Custom Framework) - Reduced false positive threshold
• Was 20 config files (too low, many frameworks have this)
• Now 50 files (more realistic for genuinely bloated configs)
9. P6.1 (Drupal Modules) - Added database error handling
• Was silently failing if database unavailable
• Now checks function exists and validates query result
10. P6.2 (Drupal Cache) - Added case-insensitive grep
• Was missing "Redis" or "Memcache" with capital letters
• Now uses grep -ci for case-insensitive matching
STATUS:
✅ All 10 logic issues resolved
✅ Syntax validation passed
✅ Ready for testing and deployment
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
cschantz