pi-review 1.0.0

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -0,0 +1,61 @@
+# pi-review
+
+Run a strict maintainer review in a new [pi](https://github.com/badlogic/pi-mono) coding agent branch.
+
+## Preview
+
+![Example pi-review output showing prioritized findings and recommendations](assets/review-output.png)
+
+## What it does
+
+Adds a `/review` command that starts a new branch from the current conversation and asks pi to review the available work. The review includes user and assistant conversation messages from the current branch, with thinking and tool calls removed.
+
+The reviewer focuses on concrete, high-confidence issues in correctness, security, performance, operability, and maintainability. If nothing material stands out, it reports `looks good`.
+
+## Installation
+
+```bash
+pi install npm:pi-review
+```
+
+Or try it temporarily:
+
+```bash
+pi -e npm:pi-review
+```
+
+## Usage
+
+```text
+/review
+```
+
+Add optional focus text:
+
+```text
+/review focus on release safety and backward compatibility
+```
+
+## How it works
+
+1. Waits for the current agent turn to finish if needed
+2. Extracts user and assistant text from the active branch
+3. Switches thinking level to `high` for the review turn
+4. Creates a new branch from the current conversation
+5. Sends a maintainer-style review prompt with optional focus text
+6. Restores your previous thinking level when the review turn ends
+
+## Review output
+
+Findings are sorted by priority:
+
+- `[P0]` severe breakage, data loss, or security issue
+- `[P1]` likely user-facing breakage or major regression
+- `[P2]` limited-scope correctness, performance, or maintainability issue
+- `[P3]` minor but real issue
+
+Each finding includes location, summary, affected behavior/invariant/code path, and a specific recommendation.
+
+## License
+
+MIT

package/dist/index.d.ts

@@ -0,0 +1,3 @@
+import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+export default function reviewExtension(pi: ExtensionAPI): void;
+//# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,+BAA+B,CAAC;AA8ClE,MAAM,CAAC,OAAO,UAAU,eAAe,CAAC,EAAE,EAAE,YAAY,QA8CvD"}

package/dist/index.js

@@ -0,0 +1,77 @@
+import { sendMessageInNewBranch } from "./lib/child-session.js";
+import { extractConversation, formatConversation } from "./lib/conversation-context.js";
+const REVIEW_THINKING_LEVEL = "high";
+const REVIEW_INSTRUCTION = `Review the available work and context.
+Put your strict maintainer hat on.
+Find concrete, high-confidence, material issues introduced by the work or revealed by the additional context.
+Focus on correctness, security, performance, operability, and maintainability.
+Do not speculate; point to the affected behavior, invariant, or code path.
+Prefer issues the author would likely fix before merge.
+Assume existing interfaces and behavior should remain backward compatible unless the user or project instructions explicitly say otherwise.
+If nothing material stands out, say \`looks good\`; otherwise return numbered sections for findings, sorted by priority.
+Use [P0] for certain severe breakage, data loss, or security issues; [P1] for likely user-facing breakage or major regressions; [P2] for limited-scope correctness, performance, or maintainability issues; [P3] for minor but real issues.
+For each finding, include a [P0]-[P3] tag, location, a concise summary, a concise explanation of the affected behavior, invariant, or code path, and \`Recommendation:\` with the top specific, actionable fix, stated concisely.`;
+function buildReviewInstruction(args) {
+    const focusText = args.trim();
+    if (!focusText) {
+        return REVIEW_INSTRUCTION;
+    }
+    return [REVIEW_INSTRUCTION, "Additional review context:", focusText].join("\n\n");
+}
+function buildReviewMessage(args, conversationXml) {
+    const reviewInstruction = buildReviewInstruction(args);
+    if (!conversationXml) {
+        return reviewInstruction;
+    }
+    return [
+        "Conversation context copied from the current branch (user + assistant messages only; thinking and tool calls removed):",
+        "",
+        "````xml",
+        conversationXml,
+        "````",
+        "",
+        reviewInstruction,
+    ].join("\n");
+}
+export default function reviewExtension(pi) {
+    let originalThinkingLevel;
+    function restoreThinkingLevel() {
+        if (!originalThinkingLevel)
+            return;
+        pi.setThinkingLevel(originalThinkingLevel);
+        originalThinkingLevel = undefined;
+    }
+    pi.on("agent_end", () => {
+        restoreThinkingLevel();
+    });
+    pi.registerCommand("review", {
+        description: "Review current work in new branch (optional focus text)",
+        handler: async (args, ctx) => {
+            if (!ctx.isIdle()) {
+                await ctx.waitForIdle();
+            }
+            const branch = ctx.sessionManager.getBranch();
+            const extractedConversation = extractConversation(branch);
+            const conversationXml = extractedConversation.length === 0 ? undefined : formatConversation(extractedConversation);
+            const reviewMessage = buildReviewMessage(args, conversationXml);
+            const currentThinkingLevel = pi.getThinkingLevel();
+            if (currentThinkingLevel !== REVIEW_THINKING_LEVEL) {
+                originalThinkingLevel = currentThinkingLevel;
+                pi.setThinkingLevel(REVIEW_THINKING_LEVEL);
+            }
+            let started = false;
+            try {
+                started = await sendMessageInNewBranch(pi, ctx, branch, reviewMessage, "review");
+            }
+            finally {
+                if (!started)
+                    restoreThinkingLevel();
+            }
+            if (!started)
+                return;
+            if (ctx.hasUI) {
+                ctx.ui.setEditorText("");
+            }
+        },
+    });
+}

package/dist/lib/child-session.d.ts

@@ -0,0 +1,3 @@
+import type { ExtensionAPI, ExtensionCommandContext, SessionEntry } from "@mariozechner/pi-coding-agent";
+export declare function sendMessageInNewBranch(pi: ExtensionAPI, ctx: ExtensionCommandContext, branch: SessionEntry[], message: string, purpose: string): Promise<boolean>;
+//# sourceMappingURL=child-session.d.ts.map

package/dist/lib/child-session.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"child-session.d.ts","sourceRoot":"","sources":["../../src/lib/child-session.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,YAAY,EACZ,uBAAuB,EACvB,YAAY,EACb,MAAM,+BAA+B,CAAC;AAgBvC,wBAAsB,sBAAsB,CAC1C,EAAE,EAAE,YAAY,EAChB,GAAG,EAAE,uBAAuB,EAC5B,MAAM,EAAE,YAAY,EAAE,EACtB,OAAO,EAAE,MAAM,EACf,OAAO,EAAE,MAAM,GACd,OAAO,CAAC,OAAO,CAAC,CAclB"}

package/dist/lib/child-session.js

@@ -0,0 +1,26 @@
+function getEmptyBranchTargetId(branch) {
+    for (const entry of branch) {
+        if (entry.type === "custom_message") {
+            return entry.id;
+        }
+        if (entry.type === "message" && entry.message.role === "user") {
+            return entry.id;
+        }
+    }
+    return undefined;
+}
+export async function sendMessageInNewBranch(pi, ctx, branch, message, purpose) {
+    const targetId = getEmptyBranchTargetId(branch);
+    if (targetId) {
+        const result = await ctx.navigateTree(targetId, { summarize: false });
+        if (result.cancelled) {
+            if (ctx.hasUI)
+                ctx.ui.notify(`New ${purpose} branch cancelled`, "info");
+            return false;
+        }
+    }
+    pi.sendUserMessage(message);
+    if (ctx.hasUI)
+        ctx.ui.notify(`Started ${purpose} branch`, "info");
+    return true;
+}

package/dist/lib/conversation-context.d.ts

@@ -0,0 +1,8 @@
+import type { SessionEntry } from "@mariozechner/pi-coding-agent";
+export type ExtractedMessage = {
+    role: "user" | "assistant";
+    text: string;
+};
+export declare function extractConversation(branch: SessionEntry[]): ExtractedMessage[];
+export declare function formatConversation(messages: ExtractedMessage[]): string;
+//# sourceMappingURL=conversation-context.d.ts.map

package/dist/lib/conversation-context.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"conversation-context.d.ts","sourceRoot":"","sources":["../../src/lib/conversation-context.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,+BAA+B,CAAC;AAElE,MAAM,MAAM,gBAAgB,GAAG;IAC7B,IAAI,EAAE,MAAM,GAAG,WAAW,CAAC;IAC3B,IAAI,EAAE,MAAM,CAAC;CACd,CAAC;AAkCF,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,YAAY,EAAE,GAAG,gBAAgB,EAAE,CAkB9E;AAED,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,gBAAgB,EAAE,GAAG,MAAM,CAOvE"}

package/dist/lib/conversation-context.js

@@ -0,0 +1,45 @@
+function isTextBlock(block) {
+    return (!!block &&
+        typeof block === "object" &&
+        "type" in block &&
+        block.type === "text" &&
+        "text" in block &&
+        typeof block.text === "string");
+}
+function extractText(content, allowString) {
+    if (allowString && typeof content === "string") {
+        return content.trim();
+    }
+    if (!Array.isArray(content)) {
+        return "";
+    }
+    return content
+        .filter(isTextBlock)
+        .map((block) => block.text.trim())
+        .filter(Boolean)
+        .join("\n\n");
+}
+export function extractConversation(branch) {
+    const messages = branch
+        .filter((entry) => entry.type === "message")
+        .map((entry) => entry.message);
+    return messages.flatMap((message) => {
+        if (message.role === "user") {
+            const text = extractText(message.content, true);
+            return text ? [{ role: "user", text }] : [];
+        }
+        if (message.role === "assistant") {
+            const text = extractText(message.content, false);
+            return text ? [{ role: "assistant", text }] : [];
+        }
+        return [];
+    });
+}
+export function formatConversation(messages) {
+    return messages
+        .map((message, index) => {
+        const tag = message.role;
+        return `<${tag} index="${index + 1}">\n${message.text}\n</${tag}>`;
+    })
+        .join("\n\n");
+}

package/package.json

@@ -0,0 +1,49 @@
+{
+  "name": "pi-review",
+  "version": "1.0.0",
+  "description": "Review current pi work in a new branch with conversation context",
+  "type": "module",
+  "keywords": [
+    "pi-package",
+    "pi-extension",
+    "review",
+    "code-review"
+  ],
+  "license": "MIT",
+  "author": "Anton Kuzmenko <hotk@hey.com>",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/default-anton/pi-review.git"
+  },
+  "main": "./dist/index.js",
+  "types": "./dist/index.d.ts",
+  "files": [
+    "assets/",
+    "dist",
+    "src/",
+    "README.md",
+    "LICENSE"
+  ],
+  "scripts": {
+    "build": "tsc",
+    "clean": "rm -rf dist",
+    "check": "tsc --noEmit",
+    "prepublishOnly": "npm run build",
+    "release:verify-tag": "node scripts/verify-release-tag.mjs",
+    "release:notes": "node scripts/changelog-release-notes.mjs",
+    "release:gate": "npm run check && npm run build"
+  },
+  "pi": {
+    "extensions": [
+      "./src/index.ts"
+    ]
+  },
+  "peerDependencies": {
+    "@mariozechner/pi-coding-agent": "^0.69.0"
+  },
+  "devDependencies": {
+    "@mariozechner/pi-coding-agent": "^0.69.0",
+    "@types/node": "^25.0.10",
+    "typescript": "^5.0.0"
+  }
+}

package/src/index.ts

@@ -0,0 +1,93 @@
+import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+
+import { sendMessageInNewBranch } from "./lib/child-session.js";
+import { extractConversation, formatConversation } from "./lib/conversation-context.js";
+
+const REVIEW_THINKING_LEVEL = "high";
+
+type ThinkingLevel = ReturnType<ExtensionAPI["getThinkingLevel"]>;
+
+const REVIEW_INSTRUCTION = `Review the available work and context.
+Put your strict maintainer hat on.
+Find concrete, high-confidence, material issues introduced by the work or revealed by the additional context.
+Focus on correctness, security, performance, operability, and maintainability.
+Do not speculate; point to the affected behavior, invariant, or code path.
+Prefer issues the author would likely fix before merge.
+Assume existing interfaces and behavior should remain backward compatible unless the user or project instructions explicitly say otherwise.
+If nothing material stands out, say \`looks good\`; otherwise return numbered sections for findings, sorted by priority.
+Use [P0] for certain severe breakage, data loss, or security issues; [P1] for likely user-facing breakage or major regressions; [P2] for limited-scope correctness, performance, or maintainability issues; [P3] for minor but real issues.
+For each finding, include a [P0]-[P3] tag, location, a concise summary, a concise explanation of the affected behavior, invariant, or code path, and \`Recommendation:\` with the top specific, actionable fix, stated concisely.`;
+
+function buildReviewInstruction(args: string): string {
+  const focusText = args.trim();
+  if (!focusText) {
+    return REVIEW_INSTRUCTION;
+  }
+
+  return [REVIEW_INSTRUCTION, "Additional review context:", focusText].join("\n\n");
+}
+
+function buildReviewMessage(args: string, conversationXml?: string): string {
+  const reviewInstruction = buildReviewInstruction(args);
+  if (!conversationXml) {
+    return reviewInstruction;
+  }
+
+  return [
+    "Conversation context copied from the current branch (user + assistant messages only; thinking and tool calls removed):",
+    "",
+    "````xml",
+    conversationXml,
+    "````",
+    "",
+    reviewInstruction,
+  ].join("\n");
+}
+
+export default function reviewExtension(pi: ExtensionAPI) {
+  let originalThinkingLevel: ThinkingLevel | undefined;
+
+  function restoreThinkingLevel(): void {
+    if (!originalThinkingLevel) return;
+
+    pi.setThinkingLevel(originalThinkingLevel);
+    originalThinkingLevel = undefined;
+  }
+
+  pi.on("agent_end", () => {
+    restoreThinkingLevel();
+  });
+
+  pi.registerCommand("review", {
+    description: "Review current work in new branch (optional focus text)",
+    handler: async (args, ctx) => {
+      if (!ctx.isIdle()) {
+        await ctx.waitForIdle();
+      }
+
+      const branch = ctx.sessionManager.getBranch();
+      const extractedConversation = extractConversation(branch);
+      const conversationXml =
+        extractedConversation.length === 0 ? undefined : formatConversation(extractedConversation);
+      const reviewMessage = buildReviewMessage(args, conversationXml);
+
+      const currentThinkingLevel = pi.getThinkingLevel();
+      if (currentThinkingLevel !== REVIEW_THINKING_LEVEL) {
+        originalThinkingLevel = currentThinkingLevel;
+        pi.setThinkingLevel(REVIEW_THINKING_LEVEL);
+      }
+
+      let started = false;
+      try {
+        started = await sendMessageInNewBranch(pi, ctx, branch, reviewMessage, "review");
+      } finally {
+        if (!started) restoreThinkingLevel();
+      }
+      if (!started) return;
+
+      if (ctx.hasUI) {
+        ctx.ui.setEditorText("");
+      }
+    },
+  });
+}

package/src/lib/child-session.ts

@@ -0,0 +1,41 @@
+import type {
+  ExtensionAPI,
+  ExtensionCommandContext,
+  SessionEntry,
+} from "@mariozechner/pi-coding-agent";
+
+function getEmptyBranchTargetId(branch: SessionEntry[]): string | undefined {
+  for (const entry of branch) {
+    if (entry.type === "custom_message") {
+      return entry.id;
+    }
+
+    if (entry.type === "message" && entry.message.role === "user") {
+      return entry.id;
+    }
+  }
+
+  return undefined;
+}
+
+export async function sendMessageInNewBranch(
+  pi: ExtensionAPI,
+  ctx: ExtensionCommandContext,
+  branch: SessionEntry[],
+  message: string,
+  purpose: string,
+): Promise<boolean> {
+  const targetId = getEmptyBranchTargetId(branch);
+  if (targetId) {
+    const result = await ctx.navigateTree(targetId, { summarize: false });
+    if (result.cancelled) {
+      if (ctx.hasUI) ctx.ui.notify(`New ${purpose} branch cancelled`, "info");
+      return false;
+    }
+  }
+
+  pi.sendUserMessage(message);
+
+  if (ctx.hasUI) ctx.ui.notify(`Started ${purpose} branch`, "info");
+  return true;
+}

package/src/lib/conversation-context.ts

@@ -0,0 +1,67 @@
+import type { SessionEntry } from "@mariozechner/pi-coding-agent";
+
+export type ExtractedMessage = {
+  role: "user" | "assistant";
+  text: string;
+};
+
+type TextBlock = {
+  type: "text";
+  text: string;
+};
+
+function isTextBlock(block: unknown): block is TextBlock {
+  return (
+    !!block &&
+    typeof block === "object" &&
+    "type" in block &&
+    block.type === "text" &&
+    "text" in block &&
+    typeof block.text === "string"
+  );
+}
+
+function extractText(content: unknown, allowString: boolean): string {
+  if (allowString && typeof content === "string") {
+    return content.trim();
+  }
+
+  if (!Array.isArray(content)) {
+    return "";
+  }
+
+  return content
+    .filter(isTextBlock)
+    .map((block) => block.text.trim())
+    .filter(Boolean)
+    .join("\n\n");
+}
+
+export function extractConversation(branch: SessionEntry[]): ExtractedMessage[] {
+  const messages = branch
+    .filter((entry): entry is SessionEntry & { type: "message" } => entry.type === "message")
+    .map((entry) => entry.message);
+
+  return messages.flatMap((message): ExtractedMessage[] => {
+    if (message.role === "user") {
+      const text = extractText(message.content, true);
+      return text ? [{ role: "user", text }] : [];
+    }
+
+    if (message.role === "assistant") {
+      const text = extractText(message.content, false);
+      return text ? [{ role: "assistant", text }] : [];
+    }
+
+    return [];
+  });
+}
+
+export function formatConversation(messages: ExtractedMessage[]): string {
+  return messages
+    .map((message, index) => {
+      const tag = message.role;
+      return `<${tag} index="${index + 1}">\n${message.text}\n</${tag}>`;
+    })
+    .join("\n\n");
+}