[Feat] Add the Destructive Command Guard binary service - #1060
[Feat] Add the Destructive Command Guard binary service#1060navedmerchant wants to merge 1 commit into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
5c460bd to
baade5f
Compare
Related GitHub Issue
Closes #1056
Part of #1049. Split from #1050.
Description
Adds a self-contained service boundary for the pinned Destructive Command Guard binary. It selects platform-specific archives, installs and verifies the binary through the managed-binary infrastructure, deduplicates concurrent installation requests, and runs DCG with typed allow/deny results.
This PR does not add a setting or change command auto-approval behavior.
Stack
2 / 4 — base:
feat/managed-binary-infrastructureReview only the six files in this PR's own diff. Merge after the managed-binary PR; GitHub will retarget the surviving stack automatically or the base can be changed to
mainafter its dependency merges.Test Procedure
pnpm --dir src exec vitest run services/destructive-command-guard/__tests__/manager.spec.ts services/destructive-command-guard/__tests__/runner.spec.ts pnpm --dir src check-typesResult: 2 test files passed, 22 tests passed; type-check passed. Repository pre-push type-check also passed.
Checklist