From 502601bf2117d5205e0af591edcd6ccd2face47f Mon Sep 17 00:00:00 2001 From: Hugo Nijhuis Date: Sat, 10 Jan 2026 19:51:01 +0100 Subject: [PATCH] Address review feedback: clarify approval gate before file creation - Made step 11 explicitly a gate before file creation - Added explicit conditional ("If user declines/approves") flow - Added note at Phase 6 start: "Only execute this phase after user approval" - Added error handling notes for directory creation and file writes The step ordering was already correct (approval step 11 before file creation steps 12-13), but the flow is now more explicit about the conditional nature. Co-Authored-By: Claude Opus 4.5 --- commands/create-capability.md | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/commands/create-capability.md b/commands/create-capability.md index e4d14c0..2e9f80e 100644 --- a/commands/create-capability.md +++ b/commands/create-capability.md @@ -158,23 +158,31 @@ Create new capabilities for the architecture repository with validation and inte ``` ``` -11. **Ask for approval**: +11. **Ask for approval** (gate before file creation): - "Create these files? (y/n)" - If warnings exist: "There are warnings. Proceed anyway? (y/n)" + - **If user declines**: Stop here. Offer to adjust the generated content or cancel. + - **If user approves**: Proceed to Phase 6. ### Phase 6: Create Files +**Only execute this phase after user approval in step 11.** + 12. **Create directories** if needed: ```bash mkdir -p skills/ mkdir -p agents/ ``` + If directory creation fails, report the error and stop. + 13. **Write files** to correct locations: - `skills//SKILL.md` - `commands/.md` - `agents//AGENT.md` + If any file write fails, report which files were created and which failed. + 14. **Report success**: ``` ## Capability Created