feat(scanner-tuning): add suppression lifecycle model and drift evidence gates (#2724)#2754
Closed
zeroknowledge0x wants to merge 3 commits into
Closed
Conversation
…nce gates Addresses review feedback from UnitOneAI#2724: - Add Suppression Risk Tiers (Safe/Moderate/Risky/Dangerous) to distinguish evidence-backed, time-bounded, owner-approved suppressions from blanket indefinite exclusions - Add Safe Suppression Requirements with mandatory fields: named owner, expiry date, scope, evidence, and auto-reopen triggers - Add Drift Evidence Gates (6 gates): suppression age, plugin update, asset exposure change, package change, compensating control status, quarterly review - Add Suppression Inventory and Drift Gate Audit to Output Format sections - Enforce critical distinction: compensated risk != confirmed false positive - Update FP Validation Workflow disposition to require lifecycle compliance - Update Prompt Injection Safety Notice with suppression lifecycle constraints - Add pitfalls UnitOneAI#6 (indefinite suppressions) and UnitOneAI#7 (compensated risk as FP) Closes UnitOneAI#2724
…ift evidence gates
…cle model - Remove duplicated 'Indefinite (no expiry)' and 'Compensated risk misclassified as FP' lines in output - Suppression lifecycle model with risk tiers, drift gates, and evidence requirements - Enforce FP vs compensated-risk disambiguation throughout - Add drift evidence gates: plugin update, exposure change, package change, control removal - Common pitfalls UnitOneAI#6 (indefinite suppressions) and UnitOneAI#7 (compensated-as-FP) Closes UnitOneAI#2724
|
Thanks for the submission! 🙏 SecuritySkills is now issue-first: contributions need a linked issue that a maintainer has marked approved before a PR is opened. Please open an issue describing the skill, wait for the |
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
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.
Closes #2724
Summary
Adds a Suppression Lifecycle Model and Drift Evidence Gates to the scanner-tuning skill, addressing two review gaps identified in #2724:
Changes
Suppression Lifecycle Model (Step 2)
Drift Evidence Gates (Step 2)
Output Format Updates
Disposition Enforcement
Common Pitfalls