Skip to content

fix(build): copy static rule packs into dist assets#31

Merged
jonathansantilli merged 1 commit intomainfrom
codex/fix-copy-rules-assets
Mar 23, 2026
Merged

fix(build): copy static rule packs into dist assets#31
jonathansantilli merged 1 commit intomainfrom
codex/fix-copy-rules-assets

Conversation

@jonathansantilli
Copy link
Copy Markdown
Owner

Summary

  • fix build asset copying to include bundled static rule packs
  • add a contract test that enforces src/layer2-static/rules is copied into dist/layer2-static/rules

Why

node dist/cli.js scan ... can fail at runtime when the built dist output is missing bundled rule assets. This ensures built CLI runs without manual workarounds.

Verification

  • npm test -- tests/meta/build-assets-contract.test.ts
  • npm run build
  • node dist/cli.js scan test-fixtures/workflow-audits/real-cases/RC-01-bot-conditions --workflow-audits --collect project --collect-kind workflows --collect-kind actions --collect-kind dependabot --format json --no-tui --force
    • exits with 2 (findings present, expected)
    • report contains bot-conditions

@jonathansantilli jonathansantilli merged commit 1bb5467 into main Mar 23, 2026
16 checks passed
@jonathansantilli jonathansantilli deleted the codex/fix-copy-rules-assets branch March 23, 2026 13:19
jonathansantilli added a commit that referenced this pull request Mar 24, 2026
fix(build): copy static rule packs into dist assets
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