plan-review 0.2.0 → 1.0.2

package/dist/transport.d.ts

@@ -10,9 +10,11 @@ export interface Transport {
 export declare class HttpTransport implements Transport {
     private doc;
     private submitHandler;
+    private sessionSaveHandler;
     private server;
     sendDocument(doc: PlanDocument): void;
     onReviewSubmit(handler: (comments: ReviewComment[]) => void): void;
+    onSessionSave(handler: (comments: ReviewComment[], activeSection: string | null) => void): void;
     start(port: number): Promise<{
         url: string;
     }>;

package/dist/transport.js

@@ -3,6 +3,7 @@ import { getAssetHtml } from './server/assets.js';
 export class HttpTransport {
     doc = null;
     submitHandler = null;
+    sessionSaveHandler = null;
     server = null;
     sendDocument(doc) {
         this.doc = doc;
@@ -10,6 +11,9 @@ export class HttpTransport {
     onReviewSubmit(handler) {
         this.submitHandler = handler;
     }
+    onSessionSave(handler) {
+        this.sessionSaveHandler = handler;
+    }
     async start(port) {
         if (!this.doc)
             throw new Error('No document set');
@@ -17,6 +21,7 @@ export class HttpTransport {
             getDocument: () => this.doc,
             onSubmit: (comments) => this.submitHandler?.(comments),
             getAssetHtml: () => getAssetHtml(),
+            onSessionSave: (comments, activeSection) => this.sessionSaveHandler?.(comments, activeSection),
         });
         return startServer(this.server, port);
     }

package/dist/transport.js.map

@@ -1 +1 @@
-{"version":3,"file":"transport.js","sourceRoot":"","sources":["../src/transport.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,kBAAkB,EAAE,WAAW,EAAE,UAAU,EAAE,MAAM,oBAAoB,CAAC;AACjF,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AASlD,MAAM,OAAO,aAAa;IAChB,GAAG,GAAwB,IAAI,CAAC;IAChC,aAAa,GAAiD,IAAI,CAAC;IACnE,MAAM,GAAkB,IAAI,CAAC;IAErC,YAAY,CAAC,GAAiB;QAC5B,IAAI,CAAC,GAAG,GAAG,GAAG,CAAC;IACjB,CAAC;IAED,cAAc,CAAC,OAA4C;QACzD,IAAI,CAAC,aAAa,GAAG,OAAO,CAAC;IAC/B,CAAC;IAED,KAAK,CAAC,KAAK,CAAC,IAAY;QACtB,IAAI,CAAC,IAAI,CAAC,GAAG;YAAE,MAAM,IAAI,KAAK,CAAC,iBAAiB,CAAC,CAAC;QAElD,IAAI,CAAC,MAAM,GAAG,kBAAkB,CAAC;YAC/B,WAAW,EAAE,GAAG,EAAE,CAAC,IAAI,CAAC,GAAI;YAC5B,QAAQ,EAAE,CAAC,QAAQ,EAAE,EAAE,CAAC,IAAI,CAAC,aAAa,EAAE,CAAC,QAAQ,CAAC;YACtD,YAAY,EAAE,GAAG,EAAE,CAAC,YAAY,EAAE;SACnC,CAAC,CAAC;QAEH,OAAO,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IACxC,CAAC;IAED,KAAK,CAAC,IAAI;QACR,IAAI,IAAI,CAAC,MAAM,IAAI,IAAI,CAAC,MAAM,CAAC,SAAS,EAAE,CAAC;YACzC,MAAM,UAAU,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;YAC9B,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC;QACrB,CAAC;IACH,CAAC;CACF"}
+{"version":3,"file":"transport.js","sourceRoot":"","sources":["../src/transport.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,kBAAkB,EAAE,WAAW,EAAE,UAAU,EAAE,MAAM,oBAAoB,CAAC;AACjF,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AASlD,MAAM,OAAO,aAAa;IAChB,GAAG,GAAwB,IAAI,CAAC;IAChC,aAAa,GAAiD,IAAI,CAAC;IACnE,kBAAkB,GAA+E,IAAI,CAAC;IACtG,MAAM,GAAkB,IAAI,CAAC;IAErC,YAAY,CAAC,GAAiB;QAC5B,IAAI,CAAC,GAAG,GAAG,GAAG,CAAC;IACjB,CAAC;IAED,cAAc,CAAC,OAA4C;QACzD,IAAI,CAAC,aAAa,GAAG,OAAO,CAAC;IAC/B,CAAC;IAED,aAAa,CAAC,OAA0E;QACtF,IAAI,CAAC,kBAAkB,GAAG,OAAO,CAAC;IACpC,CAAC;IAED,KAAK,CAAC,KAAK,CAAC,IAAY;QACtB,IAAI,CAAC,IAAI,CAAC,GAAG;YAAE,MAAM,IAAI,KAAK,CAAC,iBAAiB,CAAC,CAAC;QAElD,IAAI,CAAC,MAAM,GAAG,kBAAkB,CAAC;YAC/B,WAAW,EAAE,GAAG,EAAE,CAAC,IAAI,CAAC,GAAI;YAC5B,QAAQ,EAAE,CAAC,QAAQ,EAAE,EAAE,CAAC,IAAI,CAAC,aAAa,EAAE,CAAC,QAAQ,CAAC;YACtD,YAAY,EAAE,GAAG,EAAE,CAAC,YAAY,EAAE;YAClC,aAAa,EAAE,CAAC,QAAQ,EAAE,aAAa,EAAE,EAAE,CAAC,IAAI,CAAC,kBAAkB,EAAE,CAAC,QAAQ,EAAE,aAAa,CAAC;SAC/F,CAAC,CAAC;QAEH,OAAO,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;IACxC,CAAC;IAED,KAAK,CAAC,IAAI;QACR,IAAI,IAAI,CAAC,MAAM,IAAI,IAAI,CAAC,MAAM,CAAC,SAAS,EAAE,CAAC;YACzC,MAAM,UAAU,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;YAC9B,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC;QACrB,CAAC;IACH,CAAC;CACF"}

package/dist/types.d.ts

@@ -18,16 +18,17 @@ export interface Section {
     relatedFiles?: string[];
     verification?: string;
 }
+export interface LineAnchor {
+    type: 'lines';
+    startLine: number;
+    endLine: number;
+    lineTexts: string[];
+}
 export interface ReviewComment {
     sectionId: string;
     text: string;
     timestamp: Date;
-    anchor?: {
-        type: 'section' | 'range';
-        startOffset?: number;
-        endOffset?: number;
-        selectedText?: string;
-    };
+    anchor?: LineAnchor;
 }
 export type OutputTarget = 'stdout' | 'clipboard' | 'file' | 'claude';
 export type SplitStrategy = 'heading' | 'separator' | 'auto';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "plan-review",
-  "version": "0.2.0",
+  "version": "1.0.2",
   "description": "Interactive CLI for reviewing AI-generated markdown plans",
   "type": "module",
   "main": "dist/index.js",
@@ -8,10 +8,11 @@
     "plan-review": "dist/index.js"
   },
   "files": [
-    "dist"
+    "dist",
+    "skills"
   ],
   "scripts": {
-    "build": "tsc && node scripts/build-browser.js",
+    "build": "tsc && tsc --project tsconfig.browser.json --noEmit && node scripts/build-browser.js",
     "build:browser": "node scripts/build-browser.js",
     "dev:browser": "node scripts/build-browser.js --watch",
     "dev": "tsx src/index.ts",

package/skills/plan-review/SKILL.md

@@ -0,0 +1,42 @@
+---
+name: plan-review
+description: Use when the user asks to review a plan, start plan review, or says "I want to review this plan". Triggers on plan review requests for markdown implementation plans, specs, or design docs. Builds and runs the plan-review browser UI, feeds review output back into the conversation.
+---
+
+# Plan Review
+
+Launch the plan-review browser UI for interactive review of markdown plans, then feed the structured review output back into this session.
+
+## Prerequisites
+
+The `plan-review` CLI must be installed or available at `~/desenv/personal/plan-review/`. If not built, run `npm run build` first.
+
+## Process
+
+1. **Identify the plan file.** If the user specified a file, use it. If not, look for the most recent file matching `docs/superpowers/plans/*.md` in the current working directory. If multiple candidates exist, ask which one.
+
+2. **Build if needed.** Check if `dist/index.js` exists in the plan-review project. If not:
+   ```bash
+   cd ~/desenv/personal/plan-review && npm run build
+   ```
+
+3. **Run the review.** Launch in browser mode with stdout output:
+   ```bash
+   node ~/desenv/personal/plan-review/dist/index.js <plan-file> --browser -o stdout
+   ```
+   This opens the browser review UI. The command blocks until the user submits their review, then prints structured review output to stdout.
+
+4. **Read the output.** The review output is structured markdown with the user's comments anchored to specific sections. Read it and present a summary to the user.
+
+5. **Act on feedback.** Ask the user what they want to do with the review:
+   - Address the comments (modify the plan or code)
+   - Save the review to a file
+   - Continue discussion about specific comments
+
+## Important
+
+- The `--browser` flag opens a three-panel review UI (TOC + content + comments).
+- The `-o stdout` flag ensures the review output comes back to this session.
+- The command will block until the user clicks "Submit Review" in the browser.
+- If the user has an existing session for this plan, they'll be prompted to resume.
+- Use `--fresh` flag if the user explicitly wants to start a clean review.