From 41c1c57933a1a2a56b0e82034c1994e877880bd0 Mon Sep 17 00:00:00 2001 From: Kevin Veen-Birkenbach Date: Tue, 3 Nov 2026 11:32:19 +0100 Subject: [PATCH] feat(kvb-git-commit): show worktree, branch, confidence and break risk before every commit kvb-git-commit-no-verify gains a step between the audits and git commit: the message right before the commit call names the worktree's absolute path, the branch, the fix confidence (kvb-check-confidence) and the break risk (kvb-check-break-risk). kvb-git-commit-next names the block in its hand-over step. Operator order 2026-11-03, after a commit in the #1324 worktree was rejected for omitting the path. Also replaces the stale advocate name with kvb-check-legal. Co-Authored-By: Claude Opus 5.5 --- skills/kvb-git-commit-next/SKILL.md | 5 +++-- skills/kvb-git-commit-no-verify/SKILL.md | 14 +++++++++++--- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/skills/kvb-git-commit-next/SKILL.md b/skills/kvb-git-commit-next/SKILL.md index 72f7b7e..f3693f1 100644 --- a/skills/kvb-git-commit-next/SKILL.md +++ b/skills/kvb-git-commit-next/SKILL.md @@ -18,8 +18,9 @@ description: > operator's decision; do not commit. 3. Nothing similar found, or the operator picked the staged version: run the kvb-git-commit-no-verify skill on the currently staged changes (kvb-code-no-comments - audit and kvb-check-legal check first, then `git commit --no-verify`, exact - staging index only). + audit and kvb-check-legal check first, then its four-line block of + worktree path, branch, fix confidence and break risk, then + `git commit --no-verify`, exact staging index only). 4. Group the remaining uncommitted changes into logical bundles (one concern per bundle: same failure class, same mechanism, or same role) and stage the next one; never mix concerns to save a commit. Staging diff --git a/skills/kvb-git-commit-no-verify/SKILL.md b/skills/kvb-git-commit-no-verify/SKILL.md index c4af29d..b8bbe15 100644 --- a/skills/kvb-git-commit-no-verify/SKILL.md +++ b/skills/kvb-git-commit-no-verify/SKILL.md @@ -7,9 +7,17 @@ description: > --- 1. Run the kvb-code-no-comments audit over the staged and unstaged changes first, - then the advocate check over the staging index. An advocate blocker stops + then the kvb-check-legal check over the staging index. A legal blocker stops here. -2. Commit exactly the staging index with `git commit --no-verify` and a +2. Show four lines in the message right before the `git commit` call, never + only after it: + - **Worktree**: absolute path from `git rev-parse --show-toplevel`. + - **Branch**: `git branch --show-current`, or `detached at `. + - **Fix confidence** of the staged bundle, from the kvb-check-confidence + skill. + - **Break risk** of the staged bundle, derived through the + kvb-check-break-risk skill. +3. Commit exactly the staging index with `git commit --no-verify` and a Conventional-Commits message derived from the staged diff; never add unstaged files, never push. -3. Report the created commit hash and what remains uncommitted. +4. Report the created commit hash and what remains uncommitted.