cadet-agent 0.25.0 → 0.27.0

package/README.md

@@ -9,7 +9,7 @@ Cadet-Agent is **not a one-shot code generator**. It won't spit out a finished g
   - `cadet-agent.md` is the thin global directive: identity, non-negotiable rules, workflow routing, hard-gate protocol, and skill dispatch.
   - `Harness.md` is the canonical harness contract: budgets, evidence-backed gates, retries, context tiers, tool routing, privacy, and escalation.
   - `harness.schema.json` and `state.schema.json` are the machine-readable schemas for harness records and session state.
-  - `skills/` contains scoped workflow-phase skills (Requirements, Architecture, Spike, StoryBreakdown, TDD, Debugging, CodeReview, Resume, MCPSetup, AgentReviewer).
+  - `skills/` contains scoped workflow-phase skills (PlanningReview, Requirements, Architecture, Spike, StoryBreakdown, TDD, Debugging, CodeReview, Resume, MCPSetup, AgentReviewer).
   - `templates/` contains runtime templates for planning artifacts.
 - `.cadet/harness.json` holds repository-local budget/policy overrides (preserved by sync).
 - `.cadet/runs/` holds sanitized run ledgers (preserved by sync; no secrets or raw prompts by default).
@@ -23,16 +23,18 @@ Cadet-Agent is **not a one-shot code generator**. It won't spit out a finished g
 - `.agents/skills/` contains Deep Code (and cross-client) skill adapters.
 - These IDE folders hold thin integration shims; the core framework logic still lives in `.cadet/agent/core/`.
 - `package-agent.ps1` builds the distributable `cadet-agent.zip` package.
-- `publish-npm.ps1` publishes the CLI to npm using a token from `~/.npm_token`.
+- `bump-version.ps1` bumps the version, updates version-bearing files, commits, tags, and pushes. It runs `npm run lint` first and refuses to release if the link check fails.
+- `npm run lint` checks all markdown links (the same offline check CI runs); `npm run verify` runs tests + lint.
 
 ## Cross-IDE Support
 
-Cadet-Agent provides full workflow parity across five IDEs. The same 9 skills + reviewer are available in each:
+Cadet-Agent provides full workflow parity across five IDEs. The same 10 skills + reviewer are available in each:
 
 | Feature | GitHub Copilot | Cursor | Continue | Claude Code | Deep Code |
 |---|---|---|---|---|---|
 | Auto-load rules | Agent definition | `alwaysApply` rule | Project rule | Project skill | Project skill (`.agents/skills/`) |
 | Skill dispatch | `/cadet-<skill>` prompts | Natural language | `/cadet-<skill>` commands | `/cadet-<skill>` skills | `/skills` menu (`/`) |
+| Planning Review | ✅ | ✅ | ✅ | ✅ | ✅ |
 | Requirements | ✅ | ✅ | ✅ | ✅ | ✅ |
 | Architecture | ✅ | ✅ | ✅ | ✅ | ✅ |
 | Spike | ✅ | ✅ | ✅ | ✅ | ✅ |
@@ -111,6 +113,7 @@ flowchart TD
     REPORT["Report current phase,<br/>epics, stories & gates"]
     CR["🔍 Context Resolution<br/>classify change size,<br/>calibrate learner,<br/>detect policy"]
     REQ["📋 Requirements<br/>Given/When/Then criteria<br/>assumption audit"]
+    PLANREV["🧩 Planning Review<br/>interview one question<br/>at a time · decision tree"]
     ARCH["🏗️ Architecture<br/>technical design,<br/>ADR decisions"]
     SPIKE["🧪 Spikes<br/>resolve unverified<br/>assumptions"]
     BREAKDOWN["📐 Story Breakdown<br/>epics → testable stories"]
@@ -125,8 +128,11 @@ flowchart TD
     RESUME -->|"yes"| REPORT --> CR
 
     CR -->|"large change"| REQ
+    CR -->|"plan fuzzy or contested"| PLANREV
     CR -->|"small / no_test_required"| IMPL
 
+    PLANREV -->|"shared understanding reached"| REQ
+
     REQ --> ARCH
     ARCH -->|"unverified assumptions"| SPIKE
     ARCH -->|"all assumptions resolved"| BREAKDOWN
@@ -143,6 +149,7 @@ flowchart TD
     style START fill:#4a9,stroke:#333,color:#fff
     style CLOSED fill:#4a9,stroke:#333,color:#fff
     style RESUME fill:#e8a840,stroke:#333,color:#000
+    style PLANREV fill:#8a7ae8,stroke:#333,color:#fff
     style REVIEW fill:#e87440,stroke:#333,color:#fff
     style VALIDATE fill:#e87440,stroke:#333,color:#fff
 ```
@@ -206,6 +213,11 @@ Cadet Agent will classify the change, check `.cadet/state.json` for blocking gat
 
 **Skill mode:** For a specific workflow phase, use the matching slash command so the skill becomes the primary instruction context:
 
+```text
+/cadet-planning-review
+clarify the plan for a co-op loot system before I write the design
+```
+
 ```text
 /cadet-requirements
 create a requirements doc for a kart handling prototype

package/package.json

@@ -1,13 +1,15 @@
 {
   "name": "cadet-agent",
-  "version": "0.25.0",
+  "version": "0.27.0",
   "description": "Cross-IDE agent framework for Unity/C# game-development — one-command install",
   "type": "module",
   "bin": {
     "cadet-agent": "bin/cli.mjs"
   },
   "scripts": {
-    "test": "node --test test/*.test.mjs"
+    "test": "node --test test/*.test.mjs",
+    "lint": "lychee --offline --include-fragments \"**/*.md\"",
+    "verify": "npm test && npm run lint"
   },
   "files": [
     "bin/",