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 <noreply@anthropic.com>
This commit is contained in:
@@ -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)"
|
- "Create these files? (y/n)"
|
||||||
- If warnings exist: "There are warnings. Proceed anyway? (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
|
### Phase 6: Create Files
|
||||||
|
|
||||||
|
**Only execute this phase after user approval in step 11.**
|
||||||
|
|
||||||
12. **Create directories** if needed:
|
12. **Create directories** if needed:
|
||||||
```bash
|
```bash
|
||||||
mkdir -p skills/<name>
|
mkdir -p skills/<name>
|
||||||
mkdir -p agents/<name>
|
mkdir -p agents/<name>
|
||||||
```
|
```
|
||||||
|
|
||||||
|
If directory creation fails, report the error and stop.
|
||||||
|
|
||||||
13. **Write files** to correct locations:
|
13. **Write files** to correct locations:
|
||||||
- `skills/<name>/SKILL.md`
|
- `skills/<name>/SKILL.md`
|
||||||
- `commands/<name>.md`
|
- `commands/<name>.md`
|
||||||
- `agents/<name>/AGENT.md`
|
- `agents/<name>/AGENT.md`
|
||||||
|
|
||||||
|
If any file write fails, report which files were created and which failed.
|
||||||
|
|
||||||
14. **Report success**:
|
14. **Report success**:
|
||||||
```
|
```
|
||||||
## Capability Created
|
## Capability Created
|
||||||
|
|||||||
Reference in New Issue
Block a user