@zephyrdeng/pi-review 0.6.0 → 0.6.2

package/extensions/rv-locale.ts

@@ -0,0 +1,81 @@
+/** Lightweight locale for /rv UI strings (no LLM). */
+
+export type RvLocale = "en" | "zh";
+
+const CJK_RE = /[\u4e00-\u9fff\u3400-\u4dbf]/;
+
+export function detectRvLocale(textSamples: string[]): RvLocale {
+  let cjk = 0;
+  let latin = 0;
+  for (const sample of textSamples) {
+    const t = sample.slice(0, 2000);
+    for (const ch of t) {
+      if (CJK_RE.test(ch)) cjk++;
+      else if (/[a-zA-Z]/.test(ch)) latin++;
+    }
+  }
+  if (cjk === 0 && latin === 0) return "en";
+  return cjk >= latin * 0.15 ? "zh" : "en";
+}
+
+export type RvUiStrings = {
+  modelsKeyword: string;
+  modelsDesc: string;
+  modeCode: string;
+  modePlan: string;
+  modeChallenge: string;
+  keepSession: string;
+  keepSessionDesc: string;
+  listModels: string;
+  listModelsDesc: string;
+  codePreset: string;
+  frontendPreset: string;
+  planPreset: string;
+  challengePreset: string;
+  suggested: string;
+  presetTier: (n: number) => string;
+  thinkingSuggested: string;
+};
+
+const UI: Record<RvLocale, RvUiStrings> = {
+  en: {
+    modelsKeyword: "models",
+    modelsDesc: "List pi-review models only",
+    modeCode: "code review",
+    modePlan: "plan review",
+    modeChallenge: "challenge review",
+    keepSession: "keep session for follow-up",
+    keepSessionDesc: "Persist review session for /rv --continue",
+    listModels: "list models",
+    listModelsDesc: "Show provider/model catalog",
+    codePreset: "Code review (preset)",
+    frontendPreset: "Frontend / multimodal review (preset)",
+    planPreset: "Plan review (preset)",
+    challengePreset: "Challenge review (preset)",
+    suggested: "Suggested",
+    presetTier: (n) => (n === 0 ? "Suggested" : `Preset #${n + 1}`),
+    thinkingSuggested: "Suggested ",
+  },
+  zh: {
+    modelsKeyword: "models",
+    modelsDesc: "仅列出 pi-review 可用模型",
+    modeCode: "代码审核",
+    modePlan: "方案审核",
+    modeChallenge: "对抗性审核",
+    keepSession: "保留会话可追问",
+    keepSessionDesc: "保留 review 会话，便于 /rv --continue",
+    listModels: "查看模型列表",
+    listModelsDesc: "列出当前可用 provider/model",
+    codePreset: "代码审核（推荐配置）",
+    frontendPreset: "前端 / 多模态审核（推荐）",
+    planPreset: "方案 / 架构审核（推荐）",
+    challengePreset: "对抗性方案审核（可追问）",
+    suggested: "推荐",
+    presetTier: (n) => (n === 0 ? "推荐" : `预设 #${n + 1}`),
+    thinkingSuggested: "推荐 ",
+  },
+};
+
+export function rvUi(locale: RvLocale): RvUiStrings {
+  return UI[locale];
+}

package/extensions/rv-model-priorities.test.ts

@@ -0,0 +1,69 @@
+import assert from "node:assert/strict";
+import { describe, it } from "node:test";
+import {
+  DEFAULT_REVIEW_MODEL_PRIORITIES,
+  matchPresetEntry,
+  rankModelsWithPresets,
+  resolveReviewProfile,
+  type PresetModelEntry,
+} from "./rv-model-priorities.js";
+import type { ModelInfo } from "./rv-completions.js";
+
+function m(provider: string, id: string, extra?: Partial<ModelInfo>): ModelInfo {
+  return {
+    provider,
+    id,
+    label: `${provider}/${id}`,
+    name: id,
+    reasoning: true,
+    contextWindow: 200_000,
+    thinkingLevels: ["high", "xhigh"],
+    ...extra,
+  };
+}
+
+describe("resolveReviewProfile", () => {
+  it("uses plan profile for plan/challenge mode", () => {
+    assert.equal(resolveReviewProfile("plan", "ts"), "plan");
+    assert.equal(resolveReviewProfile("challenge", "md"), "plan");
+  });
+
+  it("uses frontend profile for vue/svelte/css targets", () => {
+    assert.equal(resolveReviewProfile("code", "vue"), "frontend");
+    assert.equal(resolveReviewProfile("code", "css"), "frontend");
+  });
+
+  it("defaults to code profile", () => {
+    assert.equal(resolveReviewProfile("code", "ts"), "code");
+  });
+});
+
+describe("matchPresetEntry", () => {
+  const registry = [
+    m("openai", "gpt-5.5"),
+    m("zhipu", "glm-5.2-chat"),
+    m("moonshot", "kimi-k2.7-preview"),
+    m("moonshot", "kimi-k2.5"),
+    m("anthropic", "claude-opus-4-8"),
+    m("anthropic", "claude-opus-4-6"),
+    m("deepseek", "deepseek-v4-pro"),
+  ];
+
+  it("picks newest kimi when versionPrefer 2.7", () => {
+    const entry: PresetModelEntry = { idContains: "kimi", versionPrefer: "2.7" };
+    const hit = matchPresetEntry(registry, entry);
+    assert.ok(hit);
+    assert.match(hit!.id, /2\.7|k2\.7/i);
+  });
+
+  it("ranks gpt-5.5 first for code presets", () => {
+    const ordered = rankModelsWithPresets(registry, "code", DEFAULT_REVIEW_MODEL_PRIORITIES);
+    assert.equal(ordered[0].id, "gpt-5.5");
+    assert.equal(ordered[1].id, "glm-5.2-chat");
+  });
+
+  it("ranks claude-opus-4-8 before older opus for plan", () => {
+    const ordered = rankModelsWithPresets(registry, "plan", DEFAULT_REVIEW_MODEL_PRIORITIES);
+    assert.equal(ordered[0].id, "claude-opus-4-8");
+  });
+});

package/extensions/rv-model-priorities.ts

@@ -0,0 +1,138 @@
+import fs from "node:fs";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+import type { ModelInfo } from "./rv-completions.js";
+
+export type PresetModelEntry = {
+  idContains: string;
+  thinking?: string;
+  versionPrefer?: string;
+};
+
+export type ReviewModelPriorities = {
+  code: PresetModelEntry[];
+  frontend: PresetModelEntry[];
+  plan: PresetModelEntry[];
+};
+
+export const DEFAULT_REVIEW_MODEL_PRIORITIES: ReviewModelPriorities = {
+  code: [
+    { idContains: "gpt-5.5", thinking: "xhigh" },
+    { idContains: "glm-5.2", thinking: "high" },
+  ],
+  frontend: [
+    { idContains: "kimi", versionPrefer: "2.7" },
+    { idContains: "claude-sonnet", versionPrefer: "5" },
+    { idContains: "minimax-m3" },
+  ],
+  plan: [
+    { idContains: "claude-opus-4-8" },
+    { idContains: "claude-opus-4", versionPrefer: "8" },
+    { idContains: "deepseek-v4-pro" },
+    { idContains: "deepseek-v4" },
+  ],
+};
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const DEFAULT_PRIORITIES_PATH = path.resolve(__dirname, "../resources/rv-model-priorities.json");
+
+export function loadReviewModelPriorities(
+  filePath: string = process.env.PI_REVIEW_RV_PRIORITIES || DEFAULT_PRIORITIES_PATH,
+): ReviewModelPriorities {
+  try {
+    const raw = fs.readFileSync(filePath, "utf8");
+    const parsed = JSON.parse(raw) as Partial<ReviewModelPriorities>;
+    return {
+      code: parsed.code?.length ? parsed.code : DEFAULT_REVIEW_MODEL_PRIORITIES.code,
+      frontend: parsed.frontend?.length ? parsed.frontend : DEFAULT_REVIEW_MODEL_PRIORITIES.frontend,
+      plan: parsed.plan?.length ? parsed.plan : DEFAULT_REVIEW_MODEL_PRIORITIES.plan,
+    };
+  } catch {
+    return DEFAULT_REVIEW_MODEL_PRIORITIES;
+  }
+}
+
+export type ReviewProfile = "code" | "frontend" | "plan";
+
+const FRONTEND_EXTS = new Set([
+  "vue", "svelte", "css", "scss", "sass", "less", "styl", "html", "htm",
+]);
+
+export function resolveReviewProfile(mode: string, targetExt?: string): ReviewProfile {
+  if (mode === "plan" || mode === "challenge") return "plan";
+  if (targetExt && FRONTEND_EXTS.has(targetExt)) return "frontend";
+  return "code";
+}
+
+function idVersionSortKey(id: string): string {
+  return id.toLowerCase();
+}
+
+export function matchPresetEntry(
+  models: ModelInfo[],
+  entry: PresetModelEntry,
+): ModelInfo | undefined {
+  const needle = entry.idContains.toLowerCase();
+  let candidates = models.filter((m) => m.id.toLowerCase().includes(needle));
+  if (entry.versionPrefer) {
+    const vp = entry.versionPrefer.toLowerCase();
+    const withVersion = candidates.filter((m) => m.id.toLowerCase().includes(vp));
+    if (withVersion.length) candidates = withVersion;
+  }
+  if (!candidates.length) return undefined;
+  candidates.sort((a, b) =>
+    idVersionSortKey(b.id).localeCompare(idVersionSortKey(a.id), undefined, { numeric: true }),
+  );
+  return candidates[0];
+}
+
+export type ResolvedPresetModel = {
+  model: ModelInfo;
+  thinking?: string;
+  presetRank: number;
+  presetLabel: string;
+};
+
+export function resolvePresetOrderedModels(
+  models: ModelInfo[],
+  profile: ReviewProfile,
+  priorities: ReviewModelPriorities,
+): ResolvedPresetModel[] {
+  const rows = priorities[profile];
+  const seen = new Set<string>();
+  const out: ResolvedPresetModel[] = [];
+  for (let i = 0; i < rows.length; i++) {
+    const entry = rows[i];
+    const matched = matchPresetEntry(models, entry);
+    if (!matched || seen.has(matched.label)) continue;
+    seen.add(matched.label);
+    out.push({
+      model: matched,
+      thinking: entry.thinking,
+      presetRank: i,
+      presetLabel: entry.idContains,
+    });
+  }
+  return out;
+}
+
+export function rankModelsWithPresets(
+  models: ModelInfo[],
+  profile: ReviewProfile,
+  priorities: ReviewModelPriorities,
+): ModelInfo[] {
+  const preset = resolvePresetOrderedModels(models, profile, priorities);
+  const presetLabels = new Set(preset.map((p) => p.model.label));
+  const rest = models
+    .filter((m) => !presetLabels.has(m.label))
+    .sort((a, b) => a.label.localeCompare(b.label));
+  return [...preset.map((p) => p.model), ...rest];
+}
+
+export function primaryPresetForProfile(
+  models: ModelInfo[],
+  profile: ReviewProfile,
+  priorities: ReviewModelPriorities,
+): ResolvedPresetModel | undefined {
+  return resolvePresetOrderedModels(models, profile, priorities)[0];
+}

package/extensions/rv-prompts.test.ts

@@ -3,6 +3,7 @@ import { describe, it } from "node:test";
 import {
   buildPiReviewArgv,
   buildRvOrchestrationPrompt,
+  orchestrationLocaleNote,
   parseRvArgs,
   validateRvParsed,
 } from "./rv-prompts.js";
@@ -57,6 +58,14 @@ describe("parseRvArgs / validateRvParsed", () => {
     ]);
   });
 
+  it("models orchestration includes locale note and model-selection reference", () => {
+    const p = parseRvArgs("models");
+    const zh = buildRvOrchestrationPrompt(p, "zh");
+    assert.match(zh, /用中文向用户总结/);
+    assert.match(zh, /model-selection/);
+    assert.match(orchestrationLocaleNote("en"), /English/);
+  });
+
   it("orchestration prompt forbids default no-stream and progress-log in Pi", () => {
     const p = parseRvArgs("@src/foo.ts");
     const prompt = buildRvOrchestrationPrompt(p);

package/extensions/rv-prompts.ts

@@ -1,5 +1,8 @@
 /** /rv → parent-agent task text; child review uses resources/review-presets.json */
 
+import type { RvLocale } from "./rv-locale.js";
+import { mergeSemanticIntoParsed, stripSemanticPhrases } from "./rv-semantic.js";
+
 export const RV_MODES = ["code", "plan", "challenge"] as const;
 export type RvMode = (typeof RV_MODES)[number];
 
@@ -21,7 +24,8 @@ export type RvValidation =
   | { ok: false; message: string };
 
 export function parseRvArgs(raw: string): RvParsed {
-  const tokens = raw.match(/(?:[^\s"']+|"[^"]*"|'[^']*')+/g) ?? [];
+  const { remainder, apply: semanticApply } = stripSemanticPhrases(raw);
+  const tokens = remainder.match(/(?:[^\s"']+|"[^"]*"|'[^']*')+/g) ?? [];
   const cleaned = tokens.map((t) => t.replace(/^['"]|['"]$/g, ""));
 
   let mode = "code";
@@ -86,16 +90,19 @@ export function parseRvArgs(raw: string): RvParsed {
     rest.push(t);
   }
 
-  return {
-    mode,
-    model,
-    thinking,
-    keepSession,
-    noStream,
-    continueHandle,
-    modelsOnly,
-    target: rest.join(" ").trim(),
-  };
+  return mergeSemanticIntoParsed(
+    {
+      mode,
+      model,
+      thinking,
+      keepSession,
+      noStream,
+      continueHandle,
+      modelsOnly,
+      target: rest.join(" ").trim(),
+    },
+    semanticApply,
+  );
 }
 
 export function validateRvParsed(parsed: RvParsed): RvValidation {
@@ -126,6 +133,13 @@ export function buildPiReviewArgv(parsed: RvParsed, target: string): string[] {
   return parts;
 }
 
+export function orchestrationLocaleNote(locale: RvLocale = "en"): string {
+  if (locale === "zh") {
+    return "用中文向用户总结（模型目录、选型理由、review 结论）；技术 id 如 provider/model 可保留英文。";
+  }
+  return "Summarize for the user in English (catalog, model rationale, review conclusion); keep technical ids such as provider/model as listed.";
+}
+
 function piHostRules(): string {
   return [
     "Follow the pi-review skill.",
@@ -137,13 +151,15 @@ function piHostRules(): string {
   ].join(" ");
 }
 
-export function buildRvOrchestrationPrompt(parsed: RvParsed): string {
+export function buildRvOrchestrationPrompt(parsed: RvParsed, locale: RvLocale = "en"): string {
+  const localeNote = orchestrationLocaleNote(locale);
   if (parsed.modelsOnly) {
     return [
       "Run the pi-review model catalog step.",
       piHostRules(),
+      localeNote,
       "Execute: pi-review models",
-      "Summarize briefly: provider count and 2–3 model IDs that fit code review vs plan/challenge.",
+      "Summarize briefly: provider count and 2–3 model IDs that fit code vs frontend vs plan/challenge (see pi-review skill references/model-selection.md).",
       "Do not start a review until the user supplies a target (@path or text).",
     ].join("\n\n");
   }
@@ -183,8 +199,8 @@ export function buildRvOrchestrationPrompt(parsed: RvParsed): string {
   const modelStep = parsed.model
     ? `Use model exactly as given: ${parsed.model}${parsed.thinking ? ` at thinking ${parsed.thinking}` : ""} (model must appear in pi-review models output).`
     : parsed.thinking
-      ? `Run pi-review models first if needed; pick a suitable listed ID or omit --model for Pi default. Use thinking ${parsed.thinking}.`
-      : "Run pi-review models first if needed; pick a suitable listed ID or omit --model for Pi default.";
+      ? `Run pi-review models first if needed; pick using skill references/model-selection.md (code / frontend / plan) or omit --model for Pi default. Use thinking ${parsed.thinking}.`
+      : "Run pi-review models first if needed; pick using skill references/model-selection.md (code / frontend / plan) or omit --model for Pi default.";
 
   const continueNote = parsed.continueHandle
     ? "Continuing an existing review session. Optional --mode and --model match the same flags as an initial /rv run."
@@ -195,6 +211,7 @@ export function buildRvOrchestrationPrompt(parsed: RvParsed): string {
   return [
     parsed.continueHandle ? "Continue a prior pi-review session." : "Run pi-review for the target below.",
     piHostRules(),
+    localeNote,
     modeBlock[parsed.mode] ??
       `Mode: ${parsed.mode}. Follow the preset named in review-presets.json when present.`,
     modelStep,

package/extensions/rv-semantic.test.ts

@@ -0,0 +1,23 @@
+import assert from "node:assert/strict";
+import { describe, it } from "node:test";
+import { parseRvArgs } from "./rv-prompts.js";
+import { stripSemanticPhrases } from "./rv-semantic.js";
+
+describe("stripSemanticPhrases / parseRvArgs", () => {
+  it("maps 方案审核 to plan mode", () => {
+    const p = parseRvArgs("方案审核 @docs/plan.md");
+    assert.equal(p.mode, "plan");
+    assert.equal(p.target, "@docs/plan.md");
+  });
+
+  it("maps list models phrase to modelsOnly", () => {
+    const p = parseRvArgs("查看模型列表");
+    assert.equal(p.modelsOnly, true);
+  });
+
+  it("strips code review before flag parse", () => {
+    const { remainder, apply } = stripSemanticPhrases("code review @src/a.ts");
+    assert.equal(apply.mode, "code");
+    assert.match(remainder, /@src\/a\.ts/);
+  });
+});

package/extensions/rv-semantic.ts

@@ -0,0 +1,110 @@
+/**
+ * Semantic /rv phrases (localized labels, not raw CLI flags in the input box).
+ */
+
+import type { RvParsed } from "./rv-prompts.js";
+import type { RvLocale } from "./rv-locale.js";
+import { rvUi } from "./rv-locale.js";
+
+export type SemanticApply = {
+  mode?: string;
+  keepSession?: boolean;
+  modelsOnly?: boolean;
+  noStream?: boolean;
+};
+
+export type SemanticPhrase = {
+  phrases: string[];
+  apply: SemanticApply;
+  completionValue: (locale: RvLocale) => string;
+  completionDesc?: (locale: RvLocale) => string;
+};
+
+export const SEMANTIC_PHRASES: SemanticPhrase[] = [
+  {
+    phrases: ["code review", "代码审核", "代码审查", "审代码"],
+    apply: { mode: "code" },
+    completionValue: (l) => rvUi(l).modeCode,
+    completionDesc: (l) => (l === "zh" ? "聚焦 diff / 正确性 / 测试" : "Diff, correctness, tests"),
+  },
+  {
+    phrases: ["plan review", "方案审核", "方案评审", "架构审核", "审方案"],
+    apply: { mode: "plan" },
+    completionValue: (l) => rvUi(l).modePlan,
+    completionDesc: (l) => (l === "zh" ? "架构 / 设计 / 产品方案" : "Architecture and design"),
+  },
+  {
+    phrases: ["challenge review", "对抗性审核", "对抗审核", "挑战审核"],
+    apply: { mode: "challenge" },
+    completionValue: (l) => rvUi(l).modeChallenge,
+    completionDesc: (l) => (l === "zh" ? "压力测试假设与风险" : "Adversarial plan pressure-test"),
+  },
+  {
+    phrases: ["keep session for follow-up", "保留会话可追问", "保留会话", "keep session"],
+    apply: { keepSession: true },
+    completionValue: (l) => rvUi(l).keepSession,
+    completionDesc: (l) => rvUi(l).keepSessionDesc,
+  },
+  {
+    phrases: ["list models", "查看模型列表", "模型列表", "list available models"],
+    apply: { modelsOnly: true },
+    completionValue: (l) => rvUi(l).listModels,
+    completionDesc: (l) => rvUi(l).listModelsDesc,
+  },
+];
+
+export function stripSemanticPhrases(raw: string): { remainder: string; apply: SemanticApply } {
+  let remainder = raw;
+  const merged: SemanticApply = {};
+  const sorted = [...SEMANTIC_PHRASES].flatMap((row) =>
+    row.phrases.map((p) => ({ phrase: p, apply: row.apply })),
+  );
+  sorted.sort((a, b) => b.phrase.length - a.phrase.length);
+
+  for (const { phrase, apply } of sorted) {
+    const re = new RegExp(phrase.replace(/[.*+?^${}()|[\]\\]/g, "\\$&"), "gi");
+    if (!re.test(remainder)) continue;
+    remainder = remainder.replace(re, " ").replace(/\s+/g, " ").trim();
+    if (apply.mode) merged.mode = apply.mode;
+    if (apply.keepSession) merged.keepSession = true;
+    if (apply.modelsOnly) merged.modelsOnly = true;
+    if (apply.noStream) merged.noStream = true;
+  }
+  return { remainder, apply: merged };
+}
+
+export function mergeSemanticIntoParsed(parsed: RvParsed, apply: SemanticApply): RvParsed {
+  const out = { ...parsed };
+  if (apply.mode) out.mode = apply.mode;
+  if (apply.keepSession) out.keepSession = true;
+  if (apply.modelsOnly) out.modelsOnly = true;
+  if (apply.noStream) out.noStream = true;
+  return out;
+}
+
+export function semanticCompletionItems(
+  locale: RvLocale,
+  tail: string,
+  headPrefix: string,
+  head: string[],
+): { value: string; label: string; description?: string }[] {
+  const present = new Set(head.map((h) => h.toLowerCase()));
+  const q = tail.toLowerCase().trim();
+  const items: { value: string; label: string; description?: string }[] = [];
+  for (const row of SEMANTIC_PHRASES) {
+    const value = row.completionValue(locale);
+    if (present.has(value.toLowerCase())) continue;
+    const matches =
+      !q ||
+      value.toLowerCase().includes(q) ||
+      value.toLowerCase().startsWith(q) ||
+      row.phrases.some((p) => p.toLowerCase().startsWith(q) || p.toLowerCase().includes(q));
+    if (!matches) continue;
+    items.push({
+      value: `${headPrefix}${value}`,
+      label: value,
+      description: row.completionDesc?.(locale),
+    });
+  }
+  return items;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zephyrdeng/pi-review",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "Isolated AI-powered code and plan reviews from the command line",
   "type": "module",
   "author": "ZephyrDeng <zephyrTang@aliyun.com>",
@@ -67,8 +67,8 @@
     "build": "tsc",
     "check": "tsc --noEmit",
     "smoke": "node dist/cli.js --help >/dev/null",
-    "test": "npx --yes tsx --test src/child-process.test.ts src/review.meta.test.ts src/meta-footer.test.ts src/json-events.test.ts src/install-args.test.ts extensions/rv-prompts.test.ts extensions/rv-completions.test.ts",
-    "test:rv": "npx --yes tsx --test extensions/rv-prompts.test.ts extensions/rv-completions.test.ts",
+    "test": "npx --yes tsx --test src/child-process.test.ts src/review.meta.test.ts src/meta-footer.test.ts src/json-events.test.ts src/install-args.test.ts extensions/rv-prompts.test.ts extensions/rv-completions.test.ts extensions/rv-model-priorities.test.ts extensions/rv-locale.test.ts extensions/rv-semantic.test.ts",
+    "test:rv": "npx --yes tsx --test extensions/rv-prompts.test.ts extensions/rv-completions.test.ts extensions/rv-model-priorities.test.ts extensions/rv-locale.test.ts extensions/rv-semantic.test.ts",
     "prepublishOnly": "npm run build",
     "pack:dry": "npm pack --dry-run"
   },

package/resources/rv-model-priorities.json

@@ -0,0 +1,17 @@
+{
+  "code": [
+    { "idContains": "gpt-5.5", "thinking": "xhigh" },
+    { "idContains": "glm-5.2", "thinking": "high" }
+  ],
+  "frontend": [
+    { "idContains": "kimi", "versionPrefer": "2.7" },
+    { "idContains": "claude-sonnet", "versionPrefer": "5" },
+    { "idContains": "minimax-m3" }
+  ],
+  "plan": [
+    { "idContains": "claude-opus-4-8" },
+    { "idContains": "claude-opus-4", "versionPrefer": "8" },
+    { "idContains": "deepseek-v4-pro" },
+    { "idContains": "deepseek-v4" }
+  ]
+}

package/skills/pi-review/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: pi-review
-description: Use pi-review to delegate isolated code, diff, repository, architecture, or plan reviews to a fresh Pi session and return only the review conclusion. On Claude Code or Codex, default to --progress-log with a background shell run and tail the log (chat tools buffer stdout). Use when the user asks for Pi review, isolated review, code review, plan review, challenge review, review status/progress, or wants to avoid context pollution.
+description: Use pi-review to delegate isolated code, diff, repository, architecture, or plan reviews to a fresh Pi session and return only the review conclusion. Model selection presets for code, frontend, and plan review (see references/model-selection.md). On Claude Code or Codex, default to --progress-log with a background shell run and tail the log (chat tools buffer stdout). Use when the user asks for Pi review, isolated review, code review, plan review, challenge review, review status/progress, or wants to avoid context pollution.
 ---
 
 # Pi Review
@@ -26,7 +26,8 @@ When resolving this fallback, use the actual directory that contains this `SKILL
 ## Codex / Claude Code
 
 - Map skill mentions of `Bash` to your host shell tool.
-- Read **[references/codex-tools.md](./references/codex-tools.md)** (next to this `SKILL.md`) for Codex/Claude Code defaults: `--progress-log` + background run + tail.
+- Read **[references/codex-tools.md](./references/codex-tools.md)** for `--progress-log` + background run + tail.
+- Read **[references/model-selection.md](./references/model-selection.md)** when choosing `--model` after `pi-review models` (code / frontend / plan presets).
 - Do not use a blocking foreground shell alone for live progress; follow **Default workflow by host** below.
 
 ## Default workflow by host
@@ -79,15 +80,17 @@ Do not implement findings from a status check; status is informational only.
 
 ## Steps
 
-1. Prime the model catalog:
+1. Prime the model catalog and pick a model:
    ```bash
    pi-review models [search]
    ```
    Rules:
    - Run this before building the review command.
-   - If the user named a model, use that provider/model text as `[search]` and verify the exact ID from the output.
-   - If the user did not name a model, run `pi-review models` and either choose an exact suitable ID from the output or deliberately omit `--model` to use Pi's default.
-   Completion criterion: the model choice is either an exact listed `provider/model[:thinking]` or an explicit decision to use Pi's default.
+   - If the user named a model, use that text as `[search]` and verify the **exact** listed `provider/model` from the output.
+   - If the user did not name a model, follow **[references/model-selection.md](./references/model-selection.md)**: infer profile (code / frontend / plan), match the priority list against catalog ids, then set `--model` with optional `:thinking` when supported.
+   - On **Claude Code / Codex / Cursor**, state the chosen model in one line (match the user's language) before running the review command.
+   - If nothing in the priority list matches, omit `--model` (Pi default) or pick a listed reasoning model with large context—say which and why.
+   Completion criterion: exact listed `provider/model[:thinking]` or an explicit default; never invented ids.
 
 2. Choose one mode:
    - default `code` — code, diff, MR, file, or repository review.

package/skills/pi-review/references/codex-tools.md

@@ -25,6 +25,8 @@ When the **pi-review** skill applies on Codex, Claude Code, or Cursor:
 
 See the parent skill: sections *Default workflow by host* and *Streaming vs agent hosts (Claude Code / Codex)*.
 
+**Model choice:** after `pi-review models`, use **[model-selection.md](./model-selection.md)** (code / frontend / plan presets)—same guidance as Pi `/rv` completions.
+
 Example tail (milestone events only):
 
 ```bash