@zephyrdeng/pi-review 0.18.2 → 0.18.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zephyrdeng/pi-review",
-  "version": "0.18.2",
+  "version": "0.18.3",
   "description": "Isolated AI-powered code and plan reviews from the command line",
   "type": "module",
   "author": "ZephyrDeng <zephyrTang@aliyun.com>",

package/dist/extensions/review.d.ts

@@ -1,2 +0,0 @@
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-export default function piReviewExtension(pi: ExtensionAPI): void;

package/dist/extensions/review.js

@@ -1,22 +0,0 @@
-import { buildRvOrchestrationPrompt, parseRvArgs, RV_COMPLETIONS } from "./rv-prompts.js";
-export default function piReviewExtension(pi) {
-    pi.registerCommand("rv", {
-        description: "Delegate pi-review to the agent (English prompts). Usage: /rv [--mode plan|challenge] [--model id] [--keep-session] @file-or-text | models",
-        getArgumentCompletions: (prefix) => {
-            const filtered = RV_COMPLETIONS.filter((item) => item.value.startsWith(prefix));
-            return filtered.length
-                ? filtered.map(({ value, hint }) => ({ value, label: value, description: hint }))
-                : null;
-        },
-        handler: async (rawArgs, ctx) => {
-            const trimmed = rawArgs.trim();
-            const parsed = parseRvArgs(trimmed);
-            const prompt = buildRvOrchestrationPrompt(parsed);
-            if (!trimmed && !parsed.modelsOnly) {
-                ctx.ui.notify("/rv needs a target or `models`. Try: /rv @path or /rv models", "warning");
-                return;
-            }
-            pi.sendUserMessage(prompt);
-        },
-    });
-}

package/dist/extensions/rv-prompts.d.ts

@@ -1,19 +0,0 @@
-/**
- * English orchestration prompts for /rv — parent Pi agent runs pi-review per skill.
- * Child review sessions use presets in resources/review-presets.json (also English).
- */
-export type RvParsed = {
-    mode: "code" | "plan" | "challenge";
-    model?: string;
-    keepSession: boolean;
-    continueHandle?: string;
-    modelsOnly: boolean;
-    target: string;
-};
-export declare function parseRvArgs(raw: string): RvParsed;
-export declare function buildRvOrchestrationPrompt(parsed: RvParsed): string;
-/** Short labels for TUI completions (English). */
-export declare const RV_COMPLETIONS: {
-    value: string;
-    hint: string;
-}[];

package/dist/extensions/rv-prompts.js

@@ -1,132 +0,0 @@
-/**
- * English orchestration prompts for /rv — parent Pi agent runs pi-review per skill.
- * Child review sessions use presets in resources/review-presets.json (also English).
- */
-export function parseRvArgs(raw) {
-    const tokens = raw.match(/(?:[^\s"']+|"[^"]*"|'[^']*')+/g) ?? [];
-    const cleaned = tokens.map((t) => t.replace(/^['"]|['"]$/g, ""));
-    let mode = "code";
-    let model;
-    let keepSession = false;
-    let continueHandle;
-    let modelsOnly = false;
-    const rest = [];
-    for (let i = 0; i < cleaned.length; i++) {
-        const t = cleaned[i];
-        if (t === "models" && rest.length === 0 && i === cleaned.length - 1) {
-            modelsOnly = true;
-            continue;
-        }
-        if (t === "--keep-session") {
-            keepSession = true;
-            continue;
-        }
-        if (t === "--mode" && cleaned[i + 1]) {
-            const m = cleaned[++i];
-            if (m === "plan" || m === "challenge" || m === "code")
-                mode = m;
-            continue;
-        }
-        if (t === "--model" && cleaned[i + 1]) {
-            model = cleaned[++i];
-            continue;
-        }
-        if (t === "--continue" && cleaned[i + 1]) {
-            continueHandle = cleaned[++i];
-            continue;
-        }
-        rest.push(t);
-    }
-    return {
-        mode,
-        model,
-        keepSession,
-        continueHandle,
-        modelsOnly,
-        target: rest.join(" ").trim(),
-    };
-}
-function baseRules() {
-    return [
-        "Follow the pi-review skill.",
-        "Run pi-review in an isolated child session; return only the review conclusion.",
-        "Do not edit, patch, commit, or implement findings unless the user asks separately.",
-        "Preserve the final PI_REVIEW_META line from pi-review output.",
-    ].join(" ");
-}
-export function buildRvOrchestrationPrompt(parsed) {
-    if (parsed.modelsOnly) {
-        return [
-            "Run the pi-review model catalog step.",
-            baseRules(),
-            "Execute: pi-review models",
-            "Present a short English summary: how many providers, suggest 2–3 IDs suited for code review vs plan/challenge.",
-            "Do not start a review until the user supplies a target (@path or text).",
-        ].join("\n\n");
-    }
-    if (parsed.continueHandle) {
-        return [
-            "Continue a prior pi-review session.",
-            baseRules(),
-            `Execute: pi-review --continue ${parsed.continueHandle} -- ${parsed.target || "follow up on the previous review"}`,
-            "Answer the follow-up in English; keep the same review-only scope.",
-        ].join("\n\n");
-    }
-    if (!parsed.target) {
-        return [
-            "The /rv command needs a review target.",
-            "Tell the user (in English): usage is /rv [--mode code|plan|challenge] [--model provider/model] [--keep-session] -- @files-or-brief",
-            "Examples: /rv @src/foo.ts | /rv --mode challenge @docs/design.md | /rv models",
-        ].join("\n\n");
-    }
-    const modeBlock = {
-        code: [
-            "Mode: code (default).",
-            "Focus: correctness, regressions, security, concurrency, API contracts, edge cases, tests.",
-            "Use read-only inspection via pi-review; no file mutations.",
-        ].join(" "),
-        plan: [
-            "Mode: plan.",
-            "Review architecture, design, or strategy through engineering, product, security, QA, ops, and DX lenses.",
-            "Identify fatal flaws, trade-offs, sequencing risks, and minimal changes to de-risk the plan.",
-        ].join(" "),
-        challenge: [
-            "Mode: challenge (adversarial).",
-            "Pressure-test assumptions, boundaries, dependencies, failure modes, migration, and missing evidence.",
-            "One-shot conclusion; do not run a live Q&A interview.",
-        ].join(" "),
-    };
-    const cliParts = ["pi-review"];
-    if (parsed.mode !== "code")
-        cliParts.push(`--mode ${parsed.mode}`);
-    if (parsed.model)
-        cliParts.push(`--model ${parsed.model}`);
-    if (parsed.keepSession)
-        cliParts.push("--keep-session");
-    cliParts.push("--", parsed.target);
-    const modelStep = parsed.model
-        ? `Use model exactly as given: ${parsed.model} (must appear in pi-review models output).`
-        : "Run pi-review models first if needed; pick a suitable listed ID or omit --model for Pi default.";
-    return [
-        "Run pi-review for the target below.",
-        baseRules(),
-        modeBlock[parsed.mode],
-        modelStep,
-        parsed.keepSession
-            ? "Use --keep-session so the user can follow up with /rv --continue <handle>."
-            : "",
-        `Execute:\n${cliParts.join(" ")}`,
-    ]
-        .filter(Boolean)
-        .join("\n\n");
-}
-/** Short labels for TUI completions (English). */
-export const RV_COMPLETIONS = [
-    { value: "@", hint: "file path after @ (e.g. @src/foo.ts)" },
-    { value: "models", hint: "list pi-review models only" },
-    { value: "--mode code", hint: "code / diff / MR review" },
-    { value: "--mode plan", hint: "architecture or plan review" },
-    { value: "--mode challenge", hint: "adversarial plan review" },
-    { value: "--model ", hint: "provider/model from pi-review models" },
-    { value: "--keep-session", hint: "persist session for follow-up" },
-];