Skip to content

Commit 8514d07

Browse files
fix: address 4 unresolved PR review comments (round 2)
- Fix report sequence to use next available number (-002, -003, etc.) instead of hardcoded -002 - Use timestamped path when preserving prototype and directory already exists to prevent overwriting earlier preserved work - Pass constraints summary in handoff to /ce:plan since plan skill does not auto-discover validation reports - Add ce:prototype to README workflow table Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 1e92913 commit 8514d07

3 files changed

Lines changed: 5 additions & 4 deletions

File tree

plugins/compound-engineering/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ The primary entry points for engineering work, invoked as slash commands:
1919
|-------|-------------|
2020
| `/ce:ideate` | Discover high-impact project improvements through divergent ideation and adversarial filtering |
2121
| `/ce:brainstorm` | Explore requirements and approaches before planning |
22+
| `/ce:prototype` | Build throwaway prototypes to validate assumptions before planning |
2223
| `/ce:plan` | Create structured plans for any multi-step task -- software features, research workflows, events, study plans -- with automatic confidence checking |
2324
| `/ce:review` | Structured code review with tiered persona agents, confidence gating, and dedup pipeline |
2425
| `/ce:work` | Execute work items systematically |

plugins/compound-engineering/skills/ce-prototype/SKILL.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ For each goal, record:
171171

172172
Read `assets/validation-report-template.md` for the report structure.
173173

174-
Write the report to `docs/prototypes/<topic-slug>-validation-<date>.md`. If a report with the same name already exists (e.g., a second prototype round on the same day), append a sequence number: `<topic-slug>-validation-<date>-002.md`.
174+
Write the report to `docs/prototypes/<topic-slug>-validation-<date>.md`. If a report with the same name already exists (e.g., multiple prototype rounds on the same day), append the next available sequence number: `-002`, `-003`, etc.
175175

176176
Create `docs/prototypes/` if it does not exist.
177177

@@ -194,7 +194,7 @@ If anything is found, flag it to the user with a brief explanation. Let them dec
194194

195195
Delete the prototype directory under `.context/compound-engineering/ce-prototype/`. The validation report in `docs/prototypes/` persists.
196196

197-
If the user explicitly asks to keep the prototype, move it out of `.context/` (which is ephemeral scratch space subject to cleanup) to a durable location: `prototypes/<topic-slug>/` at the repo root. Note the preserved path in the validation report.
197+
If the user explicitly asks to keep the prototype, move it out of `.context/` (which is ephemeral scratch space subject to cleanup) to a durable location at the repo root: `prototypes/<topic-slug>/`. If that directory already exists (from a prior preserved prototype), use a timestamped path (`prototypes/<topic-slug>-<YYYYMMDD-HHMMSS>/`) to avoid overwriting earlier work. Note the preserved path in the validation report.
198198

199199
#### 4.3 Recommend Next Step
200200

@@ -209,7 +209,7 @@ Based on the validation results, recommend the appropriate next step:
209209

210210
Present as a question using the platform's blocking question tool.
211211

212-
**Handoff to /ce:plan:** When the user selects planning, run `/ce:plan` and pass the validation report path (e.g., `/ce:plan docs/prototypes/<topic-slug>-validation-<date>.md`). If a requirements document also exists, pass both paths. This ensures the plan has direct access to validated constraints and discovered surprises without relying on context window or user re-entry.
212+
**Handoff to /ce:plan:** When the user selects planning, run `/ce:plan` and pass both the requirements document path (if one exists) and a concise constraints summary extracted from the validation report. The summary should include: proved/disproved goal results, key constraints discovered, and the validation report path for reference. Example: `/ce:plan docs/brainstorms/2026-04-01-topic-requirements.md — Prototype validated: [goals summary]. Constraints: [key constraints]. Full report: docs/prototypes/topic-validation-2026-04-04.md`. This ensures planning starts from validated assumptions even if `/ce:plan` does not auto-discover validation reports.
213213

214214
### Phase 5 (Optional): Brainstorm Integration Suggestion
215215

plugins/compound-engineering/skills/ce-prototype/references/validation-report-schema.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ Canonical frontmatter fields for prototype validation reports in `docs/prototype
3737

3838
File: `docs/prototypes/<topic-slug>-validation-<YYYY-MM-DD>.md`
3939

40-
If a report with the same name already exists (e.g., a second prototype round on the same day), append a sequence number: `<topic-slug>-validation-<YYYY-MM-DD>-002.md`.
40+
If a report with the same name already exists (e.g., multiple prototype rounds on the same day), append the next available sequence number: `-002`, `-003`, etc.
4141

4242
Examples:
4343
- `docs/prototypes/stripe-refund-webhooks-validation-2026-04-04.md`

0 commit comments

Comments
 (0)