Skip to content

feat(core,agents): FindingCategory extensible + agent distribution prep (s78) - #166

Merged
cagodoy merged 4 commits into
mainfrom
feat/mvp-blockers-s78
Aug 3, 2026
Merged

feat(core,agents): FindingCategory extensible + agent distribution prep (s78)#166
cagodoy merged 4 commits into
mainfrom
feat/mvp-blockers-s78

Conversation

@cagodoy

@cagodoy cagodoy commented Aug 3, 2026

Copy link
Copy Markdown
Member

Summary

  • FindingCategory extensible: BaseFindingCategory | (string & {}) — AUDIT-E1..E4, 18/18 EARS green
  • Prisma: FindingCategory enum → String column (schema + prisma:sync + generate)
  • Semgrep CWE fix: CWE-89/79/22/78/94/502 → security-vulnerability (aligned with spec SGP-C3)
  • Agent packages: added gitgov.agent.categories field, semantic-release devDeps, fixed workspace:* → peerDeps pattern
  • CLI bundling: @gitgov/agent-security-audit as dependency (zero-friction install)

Test plan

  • Core: 3051 tests green (125 suites)
  • security-audit: 29/29 green
  • semgrep: 27/27 green
  • After merge: trigger "Release Agent — security-audit" workflow_dispatch
  • Verify: npm view @gitgov/agent-security-audit version returns published version

cagodoy added 4 commits August 1, 2026 16:28
…/37 green

Factories: createWaiver(), createScan(), createFix() with validation + invariants.
Fix type formalized (was inline { description: string }).
createScan computes AuditSummary from findings (AUDIT-G2 invariant).
Test builders: makeTestFinding, makeTestFix, makeTestWaiver, makeTestScan.
Exported from @gitgov/core/audit + @gitgov/core. Builders from audit/testing subpath.
3050/3050 core tests green, 0 regressions.
…ep (s78)

- FindingCategory: BaseFindingCategory | (string & {}) — AUDIT-E1..E4
- Prisma: enum → String for Finding.category column
- Semgrep CWE mapping fix: CWE-89/79/22/78/94/502 → security-vulnerability (SGP-C3)
- Agent packages: add gitgov.agent.categories field to all 4 agents
- security-audit: add semantic-release + plugins as devDeps (fix exit 127)
- CLI: add @gitgov/agent-security-audit as dependency (zero-friction bundling)
- http_llm_detector: refactor isValidCategory to isKnownCategory with BaseFindingCategory
workspace:* in dependencies would be published as literal string by
npm publish (semantic-release). Move core to peerDependencies (for npm
consumers) + devDependencies (for monorepo dev). Follows test-echo
pattern which is already published correctly.

CLI keeps workspace:* in dependencies because it uses pnpm publish
which auto-resolves workspace protocol.
@cagodoy
cagodoy merged commit 02d6950 into main Aug 3, 2026
3 of 4 checks passed
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 3.21.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant