Repository navigation
Release 0.16.0: Stop hooks for Codex and Copilot CLI, doctor finds silent hooks, regression preset - #53
Merged
Conversation
- decideCommand and decideTool build their verdict in one place - integrations: one owner each for the package name and the hook's timeout and Stop format - glob gates share one empty-glob rule - sync writes linked instruction files through one function - scaffold: shared evidence bodies and the preamble marker are named once - command matching and deps: restated regexes and sections read their named constant - git: drop the unused gitAvailable; tests go through matchesGlob - one rule for quoting a failing command's output in a gate reason No change to CLI output, exit codes, JSON, installed files or stored hashes.
…lent hooks, regression preset - install codex --stop and install copilot-cli --stop gate the end of the agent's turn - copilot-cli install target: .github/hooks/skillgate.json, fail-closed PreToolUse - doctor reports hooks that are registered but switched off, and hooks or workflows pinned to another Skillgate version - install github-actions upgrades its own workflow (package pin, action pins, Node 22) - init --preset regression: tests kept, no new skips, CI and instruction files unchanged - check --format github writes a job summary - <command> --help and help <command> print usage without running the command - a value-taking flag without a value is a usage error
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added
skillgate install codex --stopandskillgate install copilot-cli --stopregister a Stop hook, so a Codex or Copilot CLI agent cannot end its turn while gates fail. Both accept the same--format claude-stopanswer as Claude Code (decision: "block"with the failing gates as the reason).copilot-cliinstall target:.github/hooks/skillgate.jsonwith a fail-closedPreToolUsehook onBash(plusgatedTools), read by GitHub Copilot CLI and its cloud agent.doctor copilot-cliverifies it, andinstall allincludes it.doctorreports a hook that is registered but switched off:disableAllHooksin.claude/settings.jsonor.claude/settings.local.json, or[features] hooks = falsein.codex/config.toml.doctorreports a hook, pre-commit entry or workflow pinned to a different Skillgate version than the one running, and names theinstallcommand that fixes it.skillgate init --preset regressionadds gates for three things: no test case or test file is removed, no new skipped or focused test appears, and CI workflows and agent instruction files are unchanged versus the base ref.check --format githubalso writes a result table to$GITHUB_STEP_SUMMARY.skillgate <command> --helpandskillgate help <command>print that command's usage and flags.Changed
skillgate install github-actionsnow upgrades an installed Skillgate workflow in place. It updates the package pin and the pinned checkout and setup-node actions, and moves a Node 18 or 20 line to Node 22. Other edits stay. New workflows use setup-node v7.1.0 and Node 22.Fixed
skillgate <command> --helpno longer runs the command. Before,scaffold --helpwrote evidence templates,zk-keygen --helpwrote a key pair, andcheck --helpran every gate.--cwdlast,--receipt --json) is now a usage error with exit 2. Before,--cwdcrashed with a stack trace and--receipt --jsonwrote a receipt named--json. A bare--templateor--thresholdis now an error too, where it used to fall back to the generic template or a NaN threshold.Verification
npm ci,npm run build,npm teston Node 20 and 22: 257/257 pass.npm run test:coverage(Node 22): lines 95.47%, branches 86.59%, functions 98.16%, above all thresholds.node dist/src/cli.js check: all 4 gates pass.fhe-metrics help,go test ./...andgo vet ./...incontrib/fhe-metricspass.test/release-016.test.tscovers each item throughinstallIntegration,doctorand the CLI. It includes a Copilot-formatPreToolUsepayload being blocked and the regression preset catching a deleted test, a new.skip, an edited workflow and an editedAGENTS.md.--version, subcommand help, exported declarations, schema hashes,explainover a command corpus, installed hook files,syncandscaffoldoutput are all byte-identical to 0.15.0.Review notes
42a4cf9(master). There are two commits. The first, Tidy-up: 8 behavior-preserving refactors, can be reviewed on its own. The second holds the 0.16.0 features.src/cli.ts,src/integrations.tsand, in the tidy-up only,src/core.ts,src/link.ts,src/scaffold.ts,src/command.ts,src/deps.tsandsrc/git.ts(the unusedgitAvailableis removed;git.tsis not a package export). Also docs, CHANGELOG and tests..,./core,./zk), receipts and snapshots, the ZK and fhe-metrics paths,isolate/.