Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions skills/writing-plans/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,16 @@ naming and copy rules, platform requirements — one line each, with exact
values copied verbatim from the spec. Every task's requirements implicitly
include this section.]

## Flow Diagrams

[REQUIRED when the plan spans 3+ interacting components or crosses an async
boundary (webhooks, queues, background jobs, callbacks); omit the section
for single-component work. Mermaid fenced blocks: `sequenceDiagram` for
request/response and handshake flows, `flowchart` for data forking across
paths. Label arrows with the real endpoint/event/function names the tasks
implement — the diagram doubles as the interface map between tasks, and it
is how an implementer holding one task sees where their piece sits.]

---
```

Expand Down Expand Up @@ -145,6 +155,8 @@ After writing the complete plan, look at the spec with fresh eyes and check the

**3. Type consistency:** Do the types, method signatures, and property names you used in later tasks match what you defined in earlier tasks? A function called `clearLayers()` in Task 3 but `clearFullLayers()` in Task 7 is a bug.

**4. Diagram check:** If the plan spans 3+ components or crosses an async boundary, the header has a Flow Diagrams section and every arrow label matches an endpoint/event/function a task implements. An arrow labeled with a name no task defines is the diagram equivalent of a placeholder — fix one or the other.

If you find issues, fix them inline. No need to re-review — just fix and move on. If you find a spec requirement with no task, add the task.

## Execution Handoff
Expand Down