feat(auditor): add accepted_warnings suppression support (REQ-357)#193
Merged
Conversation
…llback
REQ-359: Widen _req_count regex from ^###\s+REQ- to ^#{2,3}\s+REQ- so
H2-style requirement headings (## REQ-BE-001) are counted alongside H3.
REQ-358: In run_sync() YAML-first mode, fall back to parsing
REQUIREMENTS.md when no YAML requirement files exist but the Markdown
file contains >=5 REQ-* references. Also falls back for tests.
Co-Authored-By: Oz <[email protected]>
Add generic accepted_warnings suppression to the audit pipeline: - AuditResult gains suppressed field; AuditReport gains suppressed_count - _SUPPRESSION_ALIASES maps scaffold.yml alias names to check names - _apply_accepted_warnings marks matching results as suppressed+passed - run_audit reads accepted_warnings + audit_suppressions from scaffold.yml - check_ledger_health no longer manually checks audit_suppressions - run_auto_fix skips suppressed results - CLI shows ~ icon and (accepted) suffix for suppressed checks - 4 new tests covering aliases, backward compat, and properties Co-Authored-By: Oz <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
REQ-357: Add
accepted_warningssuppression supportAdds generic
accepted_warningssuppression to the audit pipeline so projects can declare known/accepted audit warnings inscaffold.ymlwithout them counting as failures.Changes
AuditResultgainssuppressed: boolfieldAuditReportgainssuppressed_countproperty;failedexcludes suppressed results_SUPPRESSION_ALIASESdict maps scaffold.yml alias names to check names_apply_accepted_warnings()marks matching results as suppressed+passedrun_audit()readsaccepted_warnings+audit_suppressionsfrom scaffold.ymlcheck_ledger_healthno longer manually checksaudit_suppressions(now generic)run_auto_fixskips suppressed results~icon and(accepted)suffix for suppressed checksConversation: https://app.warp.dev/conversation/5baa5011-18ba-48c2-9da8-a577b80a1044
Run: https://oz.warp.dev/runs/019e76b9-669d-7e3b-9d00-5c2a2a24f202
This PR was generated with Oz.