Skip to content

feat(vuln-scanner): verify high severity claims - #1008

Merged
aaronjmars merged 7 commits into
aeonfun:mainfrom
Svector-anu:feat/vuln-poc-gate-upstream
Sep 1, 2026
Merged

aaronjmars merged 7 commits into
aeonfun:mainfrom
Svector-anu:feat/vuln-poc-gate-upstream

Conversation

@Svector-anu

Copy link
Copy Markdown
Collaborator

what

require deterministic verification before an original code finding can be published as high or critical.

  • add foundry-fork verification pinned to an observed chain id and block
  • add command-mode verification for deterministic non-solidity reproductions
  • bind evidence to the target commit and exact finding claim
  • redact execution evidence and run the verifier with an isolated environment
  • downgrade missing or failed verification to needs-verification
  • keep dependency advisories and verified secret findings on their existing evidence-backed paths
  • add a benign, opt-in live base smoke test

why

the scanner currently allows pattern matching and manual reasoning to promote an original code finding directly to high or critical. state-dependent severity claims need reproducible evidence before disclosure.

verification

  • every command registered in .github/workflows/ci-tests.yml passed locally
  • bash scripts/tests/test_vuln_poc_gate.sh — 11/11 passed
  • bash scripts/tests/test_skill_mode.sh — passed
  • bash scripts/check-skill-categories.sh — passed
  • node scripts/validate-config.js — clean
  • catalogs regenerated — 6 packs / 78 skills
  • eyebrow v0.4.1 lock regenerated and eyebrow verify --ci passed
  • git diff --check — clean

mutation proof: removing env -i from the verifier made the poc suite exit 1 with the foundry reproduction, evidence, pinned-block, and local-verifier assertions failing. restoring it returned all 11 tests to green.

real runner proof: actions run 33396274300 exercised the opt-in smoke against base at pinned block 50696537 and recorded foundry-poc=verified plus live-fork-smoke foundry-fork verified. this proves the gate infrastructure ran; it does not claim that a real vulnerability was found.

the integrity lock regeneration also reconciles six pre-existing stale entries on current upstream main because the repository's integrity job verifies the full tree rather than changed skills only.

cultosagent added a commit to cultosagent/dogma-registry that referenced this pull request Sep 1, 2026
aaronjmars and others added 2 commits September 1, 2026 14:39
…m Aeon root

Forge exit 0 with no matching tests is not a reproduction. The SKILL
examples now cd back to GITHUB_WORKSPACE and pass the clone as --repo.

Co-Authored-By: Grok <grok@x.ai>
Content hash + finding line offsets for vuln-scanner SKILL.md. No new
reach. Gate still fail-closed on HIGH/CRITICAL without a matching forge pass.

Co-Authored-By: Grok <grok@x.ai>
@aaronjmars
aaronjmars merged commit 66711d3 into aeonfun:main Sep 1, 2026
7 checks passed
@Svector-anu
Svector-anu deleted the feat/vuln-poc-gate-upstream branch September 1, 2026 23:42
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.

2 participants