Skip to content

Implement Proof-of-Concept Thresholds - #140

Open
ShadowBearVR wants to merge 3 commits into
chipsalliance:mainfrom
ShadowBearVR:poc-thresholds
Open

ShadowBearVR wants to merge 3 commits into
chipsalliance:mainfrom
ShadowBearVR:poc-thresholds

Conversation

@ShadowBearVR

@ShadowBearVR ShadowBearVR commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

While scanning all identified potential vulnerabilities is always going to be supported, this may not be the best default behavior in terms of both time and token usage.

This PR introduces:

  1. Configurable PoC Threshold (minPocSeverity): Adds the ability to limit PoC synthesis to a severity threshold (defaulting to Medium and higher), configurable per-project and per-job in Nix job specs.
  2. Refined Worktree Verification: Loosens verification gate regexes to recognize passing exploit assertions and inline Rust harness test executions (otherwise all PoCs are shown as "attempted" only).
  3. Review History Optimization: Skips re-evaluating findings in reviewer passes if they already have an existing review lifecycle recorded (useful for re-processing older vulnerabilities under an expanded pipeline).

Add minPocSeverity threshold (default: Medium) to skip PoC synthesis
and secondary review passes on lower-severity findings.
Support passing exploit assertions and inline Rust harness tests
in the isolated worktree verification sandbox.
Avoid redundant reviewer agent invocations for findings that
have already completed an initial_review phase.
@ShadowBearVR
ShadowBearVR marked this pull request as ready for review September 25, 2026 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant