Skip to content

fix(toolkit): merge-rule guard hook and no dated model pins (gh-#751, gh-#749) - #761

Merged
genwave-radio merged 1 commit into
mainfrom
fix/toolkit-merge-guard-model-pins-751-749
Sep 13, 2026
Merged

genwave-radio merged 1 commit into
mainfrom
fix/toolkit-merge-guard-model-pins-751-749

Conversation

@genwave-radio

Copy link
Copy Markdown
Collaborator

🔧 What

gh-#749 model pins

  • Removed model: from all nine .claude/commands/*.md → commands inherit the session model.
  • reviewer.md: model: opusmodel: inherit. builder.md keeps the sonnet alias.
  • build-loop.md dispatch text and .claude/README.md (model column, prose, diagram) updated to match.

gh-#751 merge-rule guard

  • New PreToolUse hook on Bash in .claude/settings.example.json (also applied to the personal settings.local.json).
  • Blocks with exit 2: gh pr merge, gh release create, git tag <name>, git push --tags, git push … main, git push … :main, HEAD:refs/heads/main.
  • Allows: git tag -l, git tag --list, git tag --contains, git push to any other branch, gh pr create.
  • Tested all twelve cases by piping hook JSON into the command.

Dean lifts the guard per action by running the command himself.

Closes #749
Closes #751

… gh-#749)

Every phase command pinned a previous-generation model (claude-opus-4-7,
claude-sonnet-4-6, claude-haiku-4-5). A command's model: frontmatter
switches the conversation to that model for its turn, so the session model
never ran a phase. The pins are gone: commands inherit the session model,
builder keeps the sonnet alias, reviewer is model: inherit so the gate is
never weaker than the session. README model column and prose updated.

The MERGE RULE (merge, tag, release, push to main need Dean's per-action
consent) was prompt text only. A PreToolUse hook on Bash now blocks
gh pr merge, gh release create, git tag (listing allowed), git push --tags
and pushes to main with exit 2, so every subagent hits the same wall.

Refs gh-#749, gh-#751
@genwave-radio
genwave-radio force-pushed the fix/toolkit-merge-guard-model-pins-751-749 branch from 2df9a52 to a206971 Compare September 13, 2026 16:35
@genwave-radio
genwave-radio merged commit 3451eb1 into main Sep 13, 2026
11 checks passed
@genwave-radio
genwave-radio deleted the fix/toolkit-merge-guard-model-pins-751-749 branch September 13, 2026 17:30
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

1 participant