|
| 1 | +You're right to be concerned! Let me check what's happening. The baseline should be filtering out the 69 known issues, but they should still be visible in the report as "baselined". Let me investigate: |
| 2 | + |
| 3 | +I see the issue! The baseline is working correctly - it's filtering out the 69 known issues. But you're right that this is confusing. The report should show: |
| 4 | +1. **Total findings detected:** 69 |
| 5 | +2. **Baselined (known issues):** 69 |
| 6 | +3. **New issues:** 0 |
| 7 | + |
| 8 | +Let me check the JSON output to see what information is available: |
| 9 | + |
| 10 | +Good! The summary shows `"baselined": 69` correctly. The issue is that when baseline filters out all findings, the report shows "0 errors" which is technically correct (0 NEW errors), but it's not showing that there are 69 baselined issues. |
| 11 | + |
| 12 | +Let me run the scan WITHOUT baseline to show you what the actual issues are: |
| 13 | + |
| 14 | +Perfect! So the baseline IS working correctly: |
| 15 | + |
| 16 | +## What's Happening (This is CORRECT behavior) |
| 17 | + |
| 18 | +**Without baseline:** 4 error types, 69 total findings |
| 19 | +**With baseline:** 0 NEW errors (all 69 findings are baselined) |
| 20 | + |
| 21 | +The baseline is doing exactly what it should - it's tracking the 69 known issues and only alerting you to NEW problems. This is the intended behavior for baseline tracking. |
| 22 | + |
| 23 | +## Understanding the Report |
| 24 | + |
| 25 | +The HTML report should show: |
| 26 | +- **Errors: 0** (new errors) |
| 27 | +- **Baselined: 69** (known issues being tracked) |
| 28 | +- **Status: ✓ All critical checks passed!** (no NEW issues) |
| 29 | + |
| 30 | +This means: |
| 31 | +- ✅ No new issues introduced since baseline was created |
| 32 | +- ✅ 69 existing issues are tracked and won't fail the build |
| 33 | +- ✅ If you introduce a NEW issue, it will show up as an error |
| 34 | + |
| 35 | +## To See All Issues (Including Baselined) |
| 36 | + |
| 37 | +If you want to see the full list of all 69 issues that are being tracked, you can: |
| 38 | + |
| 39 | +1. **View the baseline file directly:** |
| 40 | + ```bash |
| 41 | + cat "/Users/noelsaw/Local Sites/neochrome-timesheets/app/public/wp-content/plugins/ptt-mkii/.hcc-baseline" |
| 42 | + ``` |
| 43 | + |
| 44 | +2. **Run without baseline to see raw findings:** |
| 45 | + ```bash |
| 46 | + ./dist/bin/check-performance.sh --paths "/Users/noelsaw/Local Sites/neochrome-timesheets/app/public/wp-content/plugins/ptt-mkii" |
| 47 | + ``` |
| 48 | + |
| 49 | +**Is this the behavior you expected, or would you like the report to also display the baselined issues?** |
0 commit comments