From fbd91aeef41bd538e581e700fa44488894799d07 Mon Sep 17 00:00:00 2001 From: Affaan Mustafa Date: Mon, 30 Mar 2026 01:49:45 -0400 Subject: [PATCH] fix: unblock shared CI regressions --- commands/learn-eval.md | 16 ++++++++-------- commands/multi-plan.md | 14 +++++++------- tests/scripts/codex-hooks.test.js | 6 ++++-- 3 files changed, 19 insertions(+), 17 deletions(-) diff --git a/commands/learn-eval.md b/commands/learn-eval.md index c6788559..23b3695c 100644 --- a/commands/learn-eval.md +++ b/commands/learn-eval.md @@ -75,17 +75,17 @@ origin: auto-extracted **Guideline dimensions** (informing the verdict, not scored): - - **Specificity & Actionability**: Contains code examples or commands that are immediately usable - - **Scope Fit**: Name, trigger conditions, and content are aligned and focused on a single pattern - - **Uniqueness**: Provides value not covered by existing skills (informed by checklist results) - - **Reusability**: Realistic trigger scenarios exist in future sessions +- **Specificity & Actionability**: Contains code examples or commands that are immediately usable +- **Scope Fit**: Name, trigger conditions, and content are aligned and focused on a single pattern +- **Uniqueness**: Provides value not covered by existing skills (informed by checklist results) +- **Reusability**: Realistic trigger scenarios exist in future sessions 6. **Verdict-specific confirmation flow** - - **Improve then Save**: Present the required improvements + revised draft + updated checklist/verdict after one re-evaluation; if the revised verdict is **Save**, save after user confirmation, otherwise follow the new verdict - - **Save**: Present save path + checklist results + 1-line verdict rationale + full draft → save after user confirmation - - **Absorb into [X]**: Present target path + additions (diff format) + checklist results + verdict rationale → append after user confirmation - - **Drop**: Show checklist results + reasoning only (no confirmation needed) +- **Improve then Save**: Present the required improvements + revised draft + updated checklist/verdict after one re-evaluation; if the revised verdict is **Save**, save after user confirmation, otherwise follow the new verdict +- **Save**: Present save path + checklist results + 1-line verdict rationale + full draft → save after user confirmation +- **Absorb into [X]**: Present target path + additions (diff format) + checklist results + verdict rationale → append after user confirmation +- **Drop**: Show checklist results + reasoning only (no confirmation needed) 7. Save / Absorb to the determined location diff --git a/commands/multi-plan.md b/commands/multi-plan.md index 4d7941a4..a899059f 100644 --- a/commands/multi-plan.md +++ b/commands/multi-plan.md @@ -203,17 +203,17 @@ Synthesize both analyses, generate **Step-by-step Implementation Plan**: 2. Save plan to `.claude/plan/.md` (extract feature name from requirement, e.g., `user-auth`, `payment-module`) 3. Output prompt in **bold text** (MUST use actual saved file path): - --- +--- **Plan generated and saved to `.claude/plan/actual-feature-name.md`** **Please review the plan above. You can:** - - **Modify plan**: Tell me what needs adjustment, I'll update the plan - - **Execute plan**: Copy the following command to a new session +- **Modify plan**: Tell me what needs adjustment, I'll update the plan +- **Execute plan**: Copy the following command to a new session - ``` - /ccg:execute .claude/plan/actual-feature-name.md - ``` - --- +``` +/ccg:execute .claude/plan/actual-feature-name.md +``` +--- **NOTE**: The `actual-feature-name.md` above MUST be replaced with the actual saved filename! diff --git a/tests/scripts/codex-hooks.test.js b/tests/scripts/codex-hooks.test.js index 27978c29..2ed9e689 100644 --- a/tests/scripts/codex-hooks.test.js +++ b/tests/scripts/codex-hooks.test.js @@ -116,8 +116,10 @@ if ( fs.mkdirSync(codexDir, { recursive: true }); fs.writeFileSync(configPath, config); - const syncResult = runBash(syncScript, [], { HOME: homeDir, CODEX_HOME: codexDir }); - assert.strictEqual(syncResult.status, 0, syncResult.stderr || syncResult.stdout); + const syncResult = runBash(syncScript, ['--update-mcp'], { HOME: homeDir, CODEX_HOME: codexDir }); + assert.strictEqual(syncResult.status, 0, `${syncResult.stdout}\n${syncResult.stderr}`); + const syncedConfig = fs.readFileSync(configPath, 'utf8'); + assert.match(syncedConfig, /^\[mcp_servers\.context7\]$/m); const checkResult = runBash(checkScript, [], { HOME: homeDir, CODEX_HOME: codexDir }); assert.strictEqual(checkResult.status, 0, checkResult.stderr || checkResult.stdout);