diff --git a/plan-ceo-review/SKILL.md b/plan-ceo-review/SKILL.md index 80d76e9ef..db14a24e5 100644 --- a/plan-ceo-review/SKILL.md +++ b/plan-ceo-review/SKILL.md @@ -538,9 +538,9 @@ Sanitize every query before it leaves the machine: strip hostnames, IPs, file pa ## PRE-REVIEW SYSTEM AUDIT (before Step 0) Before anything else, audit the system for review context. Run: ``` -git log --oneline -30 # Recent history -git diff --stat # What's already changed -git stash list # Any stashed work +git log --oneline -30 # Recent history +git diff --stat # What's already changed +git stash list # Any stashed work grep -r "TODO\|FIXME\|HACK\|XXX" -l --exclude-dir=node_modules --exclude-dir=vendor --exclude-dir=.git . | head -30 git log --since=30.days --name-only --format="" | sort | uniq -c | sort -rn | head -20 # Recently touched files ``` @@ -1024,7 +1024,8 @@ Follow the preamble's session rules; `CONDUCTOR_SESSION: true` changes transport added capability → SELECTIVE EXPANSION; fix/refactor → HOLD SCOPE. In the Recommendation's `because` clause, connect a concrete plan fact or constraint to this mode's actual benefit or tradeoff, not just its count/category. -3. Resolve that recommendation. When `QUESTION_TUNING: true`, first check `question_id=plan-ceo-review-mode` through the preamble. +3. Resolve that recommendation. When `QUESTION_TUNING: true`, first check `question_id=plan-ceo-review-mode` through the preamble's + `gstack-question-preference --check`. A check that exits 0 with `AUTO_DECIDE` selects the recommendation; go to the automatic handoff in step 4. When tuning is false, omit the lookup. Without that successful check, offer all four modes in one AskUserQuestion, @@ -1032,7 +1033,7 @@ Follow the preamble's session rules; `CONDUCTOR_SESSION: true` changes transport wins. When `QUESTION_TUNING: true`, include ``. These modes differ in kind, not coverage; do NOT score completeness. -4. **Mode handoff:** After selection, send brief chat before tools or further questions: the mode's application and rationale; every governing approved row's ID, answer reference and accepted scope. Keep rows separate. +4. **Mode handoff:** After selection, send brief chat before tools or further questions: the mode's application and rationale; every governing approved row's ID, answer reference and accepted scope. Keep rows separate. Begin with the exact matching line below: - `plan-ceo-review-mode: AUTO_DECIDE`: `Auto-decided review mode → (your preference). Change with /plan-tune. Approved decisions: . .` - Other selections: `Mode: ; approved decisions: . .` diff --git a/plan-ceo-review/SKILL.md.tmpl b/plan-ceo-review/SKILL.md.tmpl index 73cc1cfb5..46b1c6d6e 100644 --- a/plan-ceo-review/SKILL.md.tmpl +++ b/plan-ceo-review/SKILL.md.tmpl @@ -99,9 +99,9 @@ Never skip Step 0, system audit, error/rescue map or failure modes. ## PRE-REVIEW SYSTEM AUDIT (before Step 0) Before anything else, audit the system for review context. Run: ``` -git log --oneline -30 # Recent history -git diff --stat # What's already changed -git stash list # Any stashed work +git log --oneline -30 # Recent history +git diff --stat # What's already changed +git stash list # Any stashed work grep -r "TODO\|FIXME\|HACK\|XXX" -l --exclude-dir=node_modules --exclude-dir=vendor --exclude-dir=.git . | head -30 git log --since=30.days --name-only --format="" | sort | uniq -c | sort -rn | head -20 # Recently touched files ``` @@ -407,7 +407,8 @@ Follow the preamble's session rules; `CONDUCTOR_SESSION: true` changes transport added capability → SELECTIVE EXPANSION; fix/refactor → HOLD SCOPE. In the Recommendation's `because` clause, connect a concrete plan fact or constraint to this mode's actual benefit or tradeoff, not just its count/category. -3. Resolve that recommendation. When `QUESTION_TUNING: true`, first check `question_id=plan-ceo-review-mode` through the preamble. +3. Resolve that recommendation. When `QUESTION_TUNING: true`, first check `question_id=plan-ceo-review-mode` through the preamble's + `gstack-question-preference --check`. A check that exits 0 with `AUTO_DECIDE` selects the recommendation; go to the automatic handoff in step 4. When tuning is false, omit the lookup. Without that successful check, offer all four modes in one AskUserQuestion, @@ -415,7 +416,7 @@ Follow the preamble's session rules; `CONDUCTOR_SESSION: true` changes transport wins. When `QUESTION_TUNING: true`, include ``. These modes differ in kind, not coverage; do NOT score completeness. -4. **Mode handoff:** After selection, send brief chat before tools or further questions: the mode's application and rationale; every governing approved row's ID, answer reference and accepted scope. Keep rows separate. +4. **Mode handoff:** After selection, send brief chat before tools or further questions: the mode's application and rationale; every governing approved row's ID, answer reference and accepted scope. Keep rows separate. Begin with the exact matching line below: - `plan-ceo-review-mode: AUTO_DECIDE`: `Auto-decided review mode → (your preference). Change with /plan-tune. Approved decisions: . .` - Other selections: `Mode: ; approved decisions: . .`