From 4217fcfe35094bd6ee30cbc63122baed57ec9ea5 Mon Sep 17 00:00:00 2001 From: Bastien Chanot Date: Wed, 15 Jul 2026 21:24:48 +0200 Subject: [PATCH] fix(model-routing): keep commit grouping on the sonnet subagent (edit re-dispatches; /feat routes to /commit-change) --- skills/commit-change/SKILL.md | 7 ++++--- skills/feat/SKILL.md | 6 ++++-- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/skills/commit-change/SKILL.md b/skills/commit-change/SKILL.md index 38b7dfb..9801a68 100644 --- a/skills/commit-change/SKILL.md +++ b/skills/commit-change/SKILL.md @@ -75,9 +75,10 @@ AskUserQuestion: - `all` → every step in the plan is approved as-is. - `` (e.g. `1,3`) → only those steps are approved; the rest stay uncommitted for a later run. -- `edit ` → collect the corrected message/grouping for step N from the - user, redraw the plan (this dispatcher owns the text, no re-dispatch - needed), show it again and re-ask. +- `edit ` → re-dispatch `commit-changer` with `MODE: propose` and the + user's correction for step N folded into the prompt, so all grouping / + message judgment stays on the sonnet subagent (never redrawn inline on + the session model); show the redrawn plan and re-ask. - `skip` → exit cleanly, no commits created, no `MODE: apply` dispatch. ## STEP 3 — Gate 2: capitalize approval diff --git a/skills/feat/SKILL.md b/skills/feat/SKILL.md index 273f8c8..6ef54a9 100644 --- a/skills/feat/SKILL.md +++ b/skills/feat/SKILL.md @@ -173,8 +173,10 @@ feat(): ``` If the feature touched multiple concerns (e.g., feature + config + -test), consider splitting into 2-3 atomic commits — load -`$HOME/.claude/agents/commit-changer.md` and follow its grouping logic. +test), consider splitting into 2-3 atomic commits grouped by logical +unit — or run `/commit-change` on the pending work (it dispatches the +sonnet commit-changer; never inline-load the bare agent, it is now a +propose/apply executor). Print summary: ```