Skip to content

Commit 7d42401

Browse files
committed
feat(plan-review): add plan completeness review plugin
New plugin with a finalize-plan skill that reviews plan files before context clear to ensure key decisions, commands, and constraints from the conversation are captured. Assisted-by: Claude:claude-opus-4-6
1 parent 0c4d493 commit 7d42401

5 files changed

Lines changed: 85 additions & 0 deletions

File tree

.claude-plugin/marketplace.json

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,11 @@
3737
"name": "sandbox-ssh-fix",
3838
"source": "./sandbox-ssh-fix",
3939
"strict": true
40+
},
41+
{
42+
"name": "plan-review",
43+
"source": "./plan-review",
44+
"strict": true
4045
}
4146
]
4247
}

README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ claude plugin marketplace add cblecker/claude-plugins
2222
| [gws](./gws) | Google Workspace CLI skills for Gmail, Calendar, Drive, Docs, Sheets, Slides, and Meet |
2323
| [rh-dataverse](./rh-dataverse) | Red Hat Dataverse MCP server |
2424
| [sandbox-ssh-fix](./sandbox-ssh-fix) | Fixes git-over-SSH in the Claude Code sandbox on macOS by replacing the broken BSD nc SOCKS5 proxy |
25+
| [plan-review](./plan-review) | Pre-flight review of plan files before context clear |
2526

2627
## License
2728

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
{
2+
"$schema": "https://json.schemastore.org/claude-plugin.json",
3+
"name": "plan-review",
4+
"version": "0.1.0",
5+
"description": "Pre-flight review of plan files before context clear",
6+
"author": {
7+
"name": "cblecker",
8+
"email": "admin@toph.ca"
9+
}
10+
}

plan-review/README.md

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
# Plan Review
2+
3+
Pre-flight review of plan files before context clear. Ensures key decisions,
4+
file names, commands, and constraints from the conversation are captured in the
5+
plan file so a fresh context can execute it.
6+
7+
## Usage
8+
9+
```text
10+
/plan-review:finalize-plan
11+
```
12+
13+
Invoke this while in plan mode, before accepting the plan and clearing context.
14+
15+
## How It Works
16+
17+
The skill runs three phases:
18+
19+
1. **Pre-flight** — confirms plan mode is active. If not, it stops immediately.
20+
2. **Review** — reads the plan file and scans the conversation for details that
21+
would be lost on context clear. It checks five categories:
22+
- Decisions the user made between alternatives
23+
- User corrections ("no, do it this way")
24+
- Specific names (files, functions, types, CLI flags, tools, MCP resources)
25+
- Exact build/test/run commands the user specified
26+
- Constraints (ordering, compatibility, "don't touch X", style preferences)
27+
3. **Report** — if the plan already covers everything, says so in one sentence.
28+
Otherwise lists gaps as brief bullets and offers to update the plan file.
29+
30+
The skill skips anything a competent agent can rediscover from the codebase
31+
itself (line numbers, import paths, boilerplate structure).
Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
---
2+
name: finalize-plan
3+
description: >-
4+
Review a plan for completeness before accepting — ensures key decisions,
5+
file names, commands, and constraints from the conversation are captured
6+
in the plan file so a fresh context can execute it.
7+
---
8+
9+
## Pre-flight: confirm plan mode
10+
11+
Check your system context for a "Plan mode is active" reminder. If you are
12+
**not** in plan mode, tell the user and stop — this skill only applies during
13+
planning.
14+
15+
## Review the plan
16+
17+
Read the plan file (path is in the plan-mode system reminder).
18+
19+
Scan the conversation for details that a **fresh context with only the plan
20+
file** would need to faithfully execute the work. Focus on:
21+
22+
- **Decisions** — choices the user made between alternatives
23+
- **User corrections** — anything the user said "no, do it this way" about
24+
- **Specific names** — files, functions, types, CLI flags, tools, or MCP
25+
resources the user referenced or you agreed to use
26+
- **Commands** — exact build/test/run commands the user specified
27+
- **Constraints** — ordering, compatibility, "don't touch X", style preferences
28+
29+
Skip anything a competent agent can rediscover from the codebase itself (line
30+
numbers, import paths, boilerplate structure, surrounding code context).
31+
32+
## Report
33+
34+
If the plan already covers everything: say so in one sentence.
35+
36+
Otherwise list what's missing — brief bullets, each naming the gap and
37+
suggesting what to add. Then offer to update the plan file with the missing
38+
details.

0 commit comments

Comments
 (0)