@mrclrchtr/supi-review 1.7.0 → 1.8.1

package/README.md

@@ -1,3 +1,5 @@
+![SuPi](assets/logo.png)
+
 # @mrclrchtr/supi-review
 
 Adds an interactive `/supi-review` command to the [pi coding agent](https://github.com/earendil-works/pi) for session-aware code review.
@@ -8,7 +10,7 @@ Adds an interactive `/supi-review` command to the [pi coding agent](https://gith
 pi install npm:@mrclrchtr/supi-review
 ```
 
-This is a **beta** package. It is not bundled in `@mrclrchtr/supi`.
+This is a **beta** package. Install individually.
 
 For local development:
 
@@ -16,8 +18,6 @@ For local development:
 pi install ./packages/supi-review
 ```
 
-After editing the source, run `/reload`.
-
 ## What you get
 
 After install, pi gets one command:
@@ -27,7 +27,15 @@ After install, pi gets one command:
 The reviewer runs in managed child agent sessions:
 
 - a **brief synthesizer** creates a structured review brief from the active session branch
-- a **read-only reviewer** inspects the selected snapshot and submits structured findings
+- a **read-only reviewer** inspects the selected snapshot (without receiving bulk inline diffs) and submits structured findings
+
+![Review target selection](https://raw.githubusercontent.com/mrclrchtr/supi/main/screenshots/supi-review-1.png)
+
+![Review brief preview](https://raw.githubusercontent.com/mrclrchtr/supi/main/screenshots/supi-review-2.png)
+
+![Review result](https://raw.githubusercontent.com/mrclrchtr/supi/main/screenshots/supi-review-3.png)
+
+![Review progress](https://raw.githubusercontent.com/mrclrchtr/supi/main/screenshots/supi-review-4.png)
 
 ## Review flow
 
@@ -38,8 +46,8 @@ The reviewer runs in managed child agent sessions:
 3. optionally add a short note
 4. resolve the snapshot
 5. synthesize a review brief from the current session history
-6. preview the synthesized brief + prompt coverage
-7. run the review with a live progress widget
+6. preview the synthesized brief + compact prompt preview
+7. the reviewer fetches per-file diffs on demand via snapshot-aware tools; live progress widget shows activity
 8. show the structured result as a custom message
 9. if findings exist, hand off to the main agent so it can ask what to do next
 
@@ -58,7 +66,8 @@ The generated review prompt is **not** just a static diff wrapper.
 Before the actual review starts, the package:
 
 - resolves the **active session branch into the current LLM-visible context**
-- extracts high-signal user/assistant/custom/compaction context from that resolved view
+- **serializes** that resolved context into a compaction-style transcript
+- feeds the serialized transcript (plus snapshot + optional note) to a dedicated brief synthesizer
 - synthesizes a structured brief with:
   - summary
   - intended outcome
@@ -69,7 +78,9 @@ Before the actual review starts, the package:
 
 The synthesizer also receives a bounded diff excerpt from the snapshot so it can reason about actual code changes, not just filenames.
 
-That synthesized brief is then combined with the git snapshot into the final reviewer prompt.
+That synthesized brief is then combined with the git snapshot into a compact reviewer prompt. The prompt contains the brief, file manifest, and per-file overview, but no large inline diffs. Instead, the reviewer session gets snapshot-aware tools (`read_snapshot_diff`, `read_snapshot_file`) to fetch exact per-file diffs and before/after file contents on demand.
+
+The session-transcript approach mirrors how Pi summarizes context for compaction: the entire resolved conversation is rendered in a readable label format and sent to the model as a whole, rather than relying on heuristic excerpt ranking.
 
 ## Model selection
 
@@ -104,9 +115,10 @@ When a successful review contains findings, `supi-review` also injects an agent-
 - `src/review.ts` — command orchestration and interactive flow
 - `src/model.ts` — explicit model selection helpers
 - `src/git.ts` — git snapshot resolution
-- `src/history/collect.ts` — active-branch evidence extraction
+- `src/history/collect.ts` — compaction-style session-context serialization
 - `src/history/synthesize.ts` — brief synthesis orchestration
 - `src/target/packet.ts` — final reviewer packet builder
 - `src/tool/brief-runner.ts` — brief synthesis child session
-- `src/tool/review-runner.ts` — read-only reviewer child session
+- `src/tool/review-runner.ts` — read-only reviewer child session with snapshot-aware tools
+- `src/tool/snapshot-tools.ts` — per-file diff and before/after content tools scoped to the selected snapshot
 - `src/ui/renderer.ts` — structured result rendering

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrclrchtr/supi-review",
-  "version": "1.7.0",
+  "version": "1.8.1",
   "description": "SuPi Review extension — structured code review via /supi-review command",
   "license": "MIT",
   "repository": {
@@ -42,7 +42,8 @@
   "pi": {
     "extensions": [
       "./src/extension.ts"
-    ]
+    ],
+    "image": "https://raw.githubusercontent.com/mrclrchtr/supi/main/packages/supi-review/assets/logo.png"
   },
   "main": "src/api.ts",
   "exports": {

package/src/git.ts

@@ -1,10 +1,11 @@
+// biome-ignore lint/nursery/noExcessiveLinesPerFile: many tightly-coupled git helpers; splitting would create cross-ref overhead
 import { execFile } from "node:child_process";
-import { basename } from "node:path";
+import { readFile } from "node:fs/promises";
+import { basename, join } from "node:path";
 import { promisify } from "node:util";
-import type { DiffStats, ReviewSnapshot } from "./types.ts";
+import type { DiffStats, ReviewSnapshot, ReviewTargetSpec } from "./types.ts";
 
 const execFileAsync = promisify(execFile);
-
 const GIT_TIMEOUT_MS = 30_000;
 
 function scrubGitEnv(env: NodeJS.ProcessEnv): NodeJS.ProcessEnv {
@@ -237,11 +238,7 @@ export async function resolveWorkingTreeSnapshot(
     getUncommittedDiff(repoPath),
     getUncommittedFileNames(repoPath),
   ]);
-
-  if (!diffText.trim() && changedFiles.length === 0) {
-    return undefined;
-  }
-
+  if (!diffText.trim() && changedFiles.length === 0) return undefined;
   return {
     target: { kind: "working-tree" },
     title: "Working tree changes",
@@ -257,19 +254,12 @@ export async function resolveBranchSnapshot(
   base: string,
 ): Promise<ReviewSnapshot | undefined> {
   const baseSha = await getMergeBase(repoPath, base);
-  if (!baseSha) {
-    return undefined;
-  }
-
+  if (!baseSha) return undefined;
   const [diffText, changedFiles] = await Promise.all([
     getDiff(repoPath, baseSha),
     getDiffFileNames(repoPath, baseSha),
   ]);
-
-  if (!diffText.trim() && changedFiles.length === 0) {
-    return undefined;
-  }
-
+  if (!diffText.trim() && changedFiles.length === 0) return undefined;
   return {
     target: { kind: "branch", base },
     title: `Changes vs ${base}`,
@@ -288,11 +278,7 @@ export async function resolveCommitSnapshot(
     getCommitShow(repoPath, sha),
     getCommitFileNames(repoPath, sha),
   ]);
-
-  if (!diffText.trim() && changedFiles.length === 0) {
-    return undefined;
-  }
-
+  if (!diffText.trim() && changedFiles.length === 0) return undefined;
   return {
     target: { kind: "commit", sha },
     title: `Commit ${sha.slice(0, 7)}`,
@@ -302,6 +288,122 @@ export async function resolveCommitSnapshot(
   };
 }
 
+/** Get the per-file diff for a single changed file in the snapshot. */
+export async function getSnapshotFileDiff(
+  repoPath: string,
+  snapshot: ReviewSnapshot,
+  file: string,
+): Promise<string> {
+  const { target } = snapshot;
+  switch (target.kind) {
+    case "working-tree": {
+      const { stdout } = await execFileAsync(
+        "git",
+        ["diff", "HEAD", "--", file],
+        gitExecOptions(repoPath),
+      );
+      return stdout;
+    }
+    case "branch": {
+      const baseSha = await getMergeBase(repoPath, target.base);
+      if (!baseSha) return "";
+      const { stdout } = await execFileAsync(
+        "git",
+        ["diff", baseSha, "HEAD", "--", file],
+        gitExecOptions(repoPath),
+      );
+      return stdout;
+    }
+    case "commit": {
+      const { stdout } = await execFileAsync(
+        "git",
+        ["show", target.sha, "--", file],
+        gitExecOptions(repoPath),
+      );
+      return stdout;
+    }
+  }
+}
+
+/** Run `git show <ref>:<file>` and return the blob content. */
+async function showGitBlob(repoPath: string, ref: string, file: string): Promise<string> {
+  const { stdout } = await execFileAsync(
+    "git",
+    ["show", `${ref}:${file}`],
+    gitExecOptions(repoPath),
+  );
+  return stdout;
+}
+
+async function resolveWorkingTreeContent(
+  repoPath: string,
+  file: string,
+  side: "before" | "after",
+): Promise<string | undefined> {
+  if (side === "before") {
+    try {
+      return await showGitBlob(repoPath, "HEAD", file);
+    } catch {
+      return undefined;
+    }
+  }
+  try {
+    return await readFile(join(repoPath, file), "utf-8");
+  } catch {
+    return undefined;
+  }
+}
+
+async function resolveBranchContent(
+  repoPath: string,
+  target: ReviewTargetSpec & { kind: "branch" },
+  file: string,
+  side: "before" | "after",
+): Promise<string | undefined> {
+  const baseSha = await getMergeBase(repoPath, target.base);
+  if (!baseSha) return undefined;
+  const ref = side === "before" ? baseSha : "HEAD";
+  try {
+    return await showGitBlob(repoPath, ref, file);
+  } catch (err) {
+    if (side === "before") return undefined;
+    throw err;
+  }
+}
+
+async function resolveCommitContent(
+  repoPath: string,
+  target: ReviewTargetSpec & { kind: "commit" },
+  file: string,
+  side: "before" | "after",
+): Promise<string | undefined> {
+  const ref = side === "before" ? `${target.sha}^` : target.sha;
+  try {
+    return await showGitBlob(repoPath, ref, file);
+  } catch (err) {
+    if (side === "before") return undefined;
+    throw err;
+  }
+}
+
+/** Get before or after content for a single changed file in the snapshot. Returns undefined when legitimately unavailable; propagates unexpected errors. */
+export async function getSnapshotFileContent(
+  repoPath: string,
+  snapshot: ReviewSnapshot,
+  file: string,
+  side: "before" | "after",
+): Promise<string | undefined> {
+  const { target } = snapshot;
+  switch (target.kind) {
+    case "working-tree":
+      return resolveWorkingTreeContent(repoPath, file, side);
+    case "branch":
+      return resolveBranchContent(repoPath, target, file, side);
+    case "commit":
+      return resolveCommitContent(repoPath, target, file, side);
+  }
+}
+
 /** Convenience label for one changed file, used in synthesized prompts/UI. */
 export function formatChangedFileLabel(file: string): string {
   return basename(file) === file ? file : `${basename(file)} (${file})`;

package/src/history/collect.ts

@@ -1,114 +1,134 @@
 import type { SessionContext } from "@earendil-works/pi-coding-agent";
-import type { HistoryEvidence, ReviewSnapshot } from "../types.ts";
-
-const INTENT_PATTERN =
-  /\b(fix|refactor|rename|preserve|avoid|should|must|ensure|intended|goal|risk|concern|regression|security|correctness|performance|api|breaking|review)\b/i;
 
 type ResolvedSessionMessage = SessionContext["messages"][number];
 
+const DEFAULT_MAX_CHARS = 8_000;
+
+export interface SerializeSessionContextOptions {
+  maxChars?: number;
+}
+
 /**
- * Extract the highest-signal evidence from the resolved LLM-visible session context.
+ * Serialize the resolved LLM-visible session context into a compaction-style
+ * readable transcript for the brief synthesizer.
  *
- * The collector intentionally favors user intent, assistant plans, compaction
- * summaries, and custom extension messages over raw tool chatter.
+ * This mirrors the overall approach Pi uses for compaction:
+ * - messages are labeled by role and rendered in chronological order
+ * - compaction and branch summaries appear with their own labels
+ * - the output is bounded so the synthesizer stays predictable on long sessions
+ * - no heuristic ranking or scoring is applied
  */
-export function collectHistoryEvidence(
+export function serializeSessionContext(
   messages: ResolvedSessionMessage[],
-  snapshot: ReviewSnapshot,
-  note?: string,
-): HistoryEvidence[] {
-  const scoringContext = {
-    changedPathTokens: collectPathTokens(snapshot.changedFiles),
-    noteTokens: collectFreeformTokens(note),
-    total: messages.length,
-  };
-
-  const scored = messages
-    .map((message) => toEvidence(message))
+  options?: SerializeSessionContextOptions,
+): string {
+  const maxChars = options?.maxChars ?? DEFAULT_MAX_CHARS;
+  if (messages.length === 0) return "";
+
+  const entries = messages
+    .map((message) => serializeEntry(message))
     .filter(
-      (evidence): evidence is Omit<HistoryEvidence, "score" | "reason"> => evidence !== undefined,
-    )
-    .map((evidence, index) => scoreEvidence(evidence, index, scoringContext))
-    .sort((a, b) => b.score - a.score || b.text.length - a.text.length);
+      (entry): entry is { label: string; text: string; isSummary: boolean } => entry !== undefined,
+    );
+
+  if (entries.length === 0) return "";
+
+  // Separate summary entries (compaction/branch summaries) from regular entries
+  const summaryEntries = entries.filter((e) => e.isSummary);
+  const regularEntries = entries.filter((e) => !e.isSummary);
+
+  // Estimate the total size if we keep everything
+  const allLines = entries.map(formatEntry);
+  const totalSize = measureLines(allLines);
+
+  if (totalSize <= maxChars) {
+    return allLines.join("\n");
+  }
+
+  // Build the output: always include summaries, then as many recent regular entries as fit
+  const summaryLines = summaryEntries.map(formatEntry);
+
+  // Walk regular entries from newest to oldest, keeping as many as fit
+  const keptRegular: typeof regularEntries = [];
+  let size = measureLines(summaryLines);
+
+  for (let i = regularEntries.length - 1; i >= 0; i--) {
+    const line = formatEntry(regularEntries[i]);
+    const lineLen = line.length + 1; // +1 for newline
+    if (keptRegular.length > 0 && size + lineLen > maxChars) {
+      break;
+    }
+    keptRegular.unshift(regularEntries[i]);
+    size += lineLen;
+  }
+
+  // Build final output: summaries first, then kept recent entries
+  const keptLines = [...summaryLines, ...keptRegular.map(formatEntry)];
 
-  return capEvidence(scored, 10, 4_500);
+  // If still nothing fits with summaries alone, truncate the summaries
+  if (keptLines.length > 0) {
+    const output = keptLines.join("\n");
+    if (output.length <= maxChars) return output;
+  }
+
+  // Last resort: truncate the full output
+  return keepFirstLines(keptLines, maxChars);
+}
+
+function formatEntry(entry: { label: string; text: string }): string {
+  return `[${entry.label}]\n${entry.text}`;
 }
 
-function toEvidence(
+function measureLines(lines: string[]): number {
+  return lines.reduce((acc, line) => acc + line.length + 1, 0); // +1 for newline
+}
+
+function keepFirstLines(lines: string[], maxChars: number): string {
+  const kept: string[] = [];
+  let size = 0;
+  for (const line of lines) {
+    const lineLen = line.length + 1;
+    if (size + lineLen > maxChars) break;
+    kept.push(line);
+    size += lineLen;
+  }
+  return kept.join("\n");
+}
+
+function serializeEntry(
   message: ResolvedSessionMessage,
-): Omit<HistoryEvidence, "score" | "reason"> | undefined {
+): { label: string; text: string; isSummary: boolean } | undefined {
   switch (message.role) {
     case "user":
     case "assistant": {
+      const text = normalizeText(extractMessageText(message.content));
+      if (!text) return undefined;
       return {
-        kind: message.role,
-        text: normalizeText(extractMessageText(message.content)),
+        label: message.role === "user" ? "User" : "Assistant",
+        text,
+        isSummary: false,
       };
     }
     case "custom": {
-      return {
-        kind: "custom",
-        text: normalizeText(extractMessageText(message.content)),
-      };
+      const text = normalizeText(extractMessageText(message.content));
+      if (!text) return undefined;
+      return { label: "Custom", text, isSummary: false };
     }
     case "compactionSummary": {
-      return {
-        kind: "compaction",
-        text: normalizeText(message.summary),
-      };
+      const text = normalizeText(message.summary);
+      if (!text) return undefined;
+      return { label: "Compaction summary", text, isSummary: true };
     }
     case "branchSummary": {
-      return {
-        kind: "branch-summary",
-        text: normalizeText(message.summary),
-      };
+      const text = normalizeText(message.summary);
+      if (!text) return undefined;
+      return { label: "Branch summary", text, isSummary: true };
     }
     default:
       return undefined;
   }
 }
 
-function scoreEvidence(
-  evidence: Omit<HistoryEvidence, "score" | "reason">,
-  index: number,
-  context: { total: number; changedPathTokens: string[]; noteTokens: string[] },
-): HistoryEvidence {
-  const recencyBoost = context.total > 0 ? (index / context.total) * 3 : 0;
-  const pathMatches = countTokenMatches(evidence.text, context.changedPathTokens);
-  const noteMatches = countTokenMatches(evidence.text, context.noteTokens);
-  const intentBoost = INTENT_PATTERN.test(evidence.text) ? 4 : 0;
-  const shortTextPenalty = evidence.text.length < 24 ? -2 : 0;
-  const longTextPenalty = evidence.text.length > 900 ? -1 : 0;
-  const score =
-    baseKindScore(evidence.kind) +
-    recencyBoost +
-    pathMatches * 3 +
-    noteMatches * 2 +
-    intentBoost +
-    shortTextPenalty +
-    longTextPenalty;
-
-  return {
-    ...evidence,
-    score,
-    reason: buildReason(pathMatches, noteMatches, intentBoost),
-  };
-}
-
-function buildReason(pathMatches: number, noteMatches: number, intentBoost: number): string {
-  const reasons: string[] = [];
-  if (pathMatches > 0) {
-    reasons.push(`mentions ${pathMatches} changed-path token${pathMatches === 1 ? "" : "s"}`);
-  }
-  if (noteMatches > 0) {
-    reasons.push(`matches ${noteMatches} note token${noteMatches === 1 ? "" : "s"}`);
-  }
-  if (intentBoost > 0) {
-    reasons.push("contains intent/constraint language");
-  }
-  return reasons.join(", ") || "high-signal session context";
-}
-
 function extractMessageText(content: unknown): string {
   if (typeof content === "string") {
     return content;
@@ -131,80 +151,3 @@ function extractMessageText(content: unknown): string {
 function normalizeText(text: string): string {
   return text.replace(/\s+/g, " ").trim();
 }
-
-function collectPathTokens(paths: string[]): string[] {
-  const tokens = new Set<string>();
-
-  for (const path of paths) {
-    for (const part of path.split(/[\\/]/)) {
-      addToken(tokens, part);
-      for (const inner of part.split(/[-_.]/)) {
-        addToken(tokens, inner);
-      }
-    }
-    addToken(tokens, path);
-  }
-
-  return Array.from(tokens);
-}
-
-function collectFreeformTokens(text: string | undefined): string[] {
-  if (!text) return [];
-
-  const tokens = new Set<string>();
-  for (const part of text.split(/\W+/)) {
-    addToken(tokens, part);
-  }
-  return Array.from(tokens);
-}
-
-function addToken(tokens: Set<string>, token: string): void {
-  const normalized = token.trim().toLowerCase();
-  if (normalized.length < 3) return;
-  tokens.add(normalized);
-}
-
-function countTokenMatches(text: string, tokens: string[]): number {
-  if (tokens.length === 0 || text.length === 0) return 0;
-  const lower = text.toLowerCase();
-  return tokens.reduce((count, token) => count + (lower.includes(token) ? 1 : 0), 0);
-}
-
-function baseKindScore(kind: HistoryEvidence["kind"]): number {
-  switch (kind) {
-    case "user":
-      return 8;
-    case "assistant":
-      return 5;
-    case "compaction":
-    case "branch-summary":
-      return 6;
-    case "custom":
-      return 4;
-  }
-}
-
-function capEvidence(
-  evidence: HistoryEvidence[],
-  maxItems: number,
-  maxChars: number,
-): HistoryEvidence[] {
-  const kept: HistoryEvidence[] = [];
-  let used = 0;
-
-  for (const item of evidence) {
-    if (kept.length >= maxItems) break;
-    const text = truncate(item.text, 600);
-    const nextUsed = used + text.length;
-    if (kept.length > 0 && nextUsed > maxChars) break;
-    kept.push({ ...item, text });
-    used = nextUsed;
-  }
-
-  return kept;
-}
-
-function truncate(text: string, maxChars: number): string {
-  if (text.length <= maxChars) return text;
-  return `${text.slice(0, maxChars)}…`;
-}

package/src/history/synthesize.ts

@@ -1,7 +1,7 @@
 import type { ModelRegistry } from "@earendil-works/pi-coding-agent";
 import { runBriefSynthesis } from "../tool/brief-runner.ts";
 import type { BriefSynthesisRunResult, ReviewProgress } from "../tool/runner-types.ts";
-import type { HistoryEvidence, ReviewModelSelection, ReviewSnapshot } from "../types.ts";
+import type { ReviewModelSelection, ReviewSnapshot } from "../types.ts";
 
 const DIFF_EXCERPT_CHAR_BUDGET = 12_000;
 
@@ -10,20 +10,21 @@ export interface SynthesizeReviewBriefOptions {
   modelRegistry: ModelRegistry;
   cwd: string;
   snapshot: ReviewSnapshot;
-  evidence: HistoryEvidence[];
+  serializedContext: string;
   note?: string;
   signal?: AbortSignal;
   onProgress?: (progress: ReviewProgress) => void;
 }
 
-/** Synthesize a structured review brief from the current snapshot and session evidence. */
+/** Synthesize a structured review brief from the current snapshot and session context. */
 export function synthesizeReviewBrief(
   options: SynthesizeReviewBriefOptions,
 ): Promise<BriefSynthesisRunResult> {
-  const { model, modelRegistry, cwd, snapshot, evidence, note, signal, onProgress } = options;
+  const { model, modelRegistry, cwd, snapshot, serializedContext, note, signal, onProgress } =
+    options;
 
   return runBriefSynthesis({
-    prompt: buildBriefSynthesisPrompt(snapshot, evidence, note),
+    prompt: buildBriefSynthesisPrompt(snapshot, serializedContext, note),
     model: model.model,
     modelRegistry,
     cwd,
@@ -34,7 +35,7 @@ export function synthesizeReviewBrief(
 
 export function buildBriefSynthesisPrompt(
   snapshot: ReviewSnapshot,
-  evidence: HistoryEvidence[],
+  serializedContext: string,
   note?: string,
 ): string {
   const diffExcerpt = buildDiffExcerpt(snapshot.diffText);
@@ -65,18 +66,11 @@ export function buildBriefSynthesisPrompt(
     parts.push("", "## User note", note.trim());
   }
 
-  parts.push("", "## Session evidence");
-  if (evidence.length === 0) {
-    parts.push(
-      "- No strong session evidence was extracted. Derive a minimal brief from the snapshot only.",
-    );
+  parts.push("", "## Serialized session context");
+  if (serializedContext.trim()) {
+    parts.push(serializedContext.trim());
   } else {
-    parts.push(
-      ...evidence.flatMap((item, index) => [
-        `${index + 1}. [${item.kind}] ${item.reason}`,
-        `   ${item.text}`,
-      ]),
-    );
+    parts.push("No session context was available. Derive the brief from the snapshot only.");
   }
 
   parts.push(
@@ -87,7 +81,9 @@ export function buildBriefSynthesisPrompt(
     "- intendedOutcome: what the session seems to be trying to achieve",
     "- constraints: invariants or requirements to preserve",
     "- focusAreas: what the reviewer should inspect carefully",
-    "- riskyFiles: changed files that seem especially important or risky",
+    "- riskyFiles: changed files that seem especially important or risky.",
+    "  Prioritize files touching auth, data handling, core logic, error handling, or public APIs.",
+    "  Include only files from the changed-files list above.",
     "- unresolvedQuestions: ambiguities or concerns that remain unclear",
   );
 

package/src/review.ts

@@ -1,6 +1,6 @@
 import { buildSessionContext, type ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { resolveBranchSnapshot, resolveCommitSnapshot, resolveWorkingTreeSnapshot } from "./git.ts";
-import { collectHistoryEvidence } from "./history/collect.ts";
+import { serializeSessionContext } from "./history/collect.ts";
 import { synthesizeReviewBrief } from "./history/synthesize.ts";
 import { buildReviewPacket } from "./target/packet.ts";
 import { runReviewer } from "./tool/review-runner.ts";
@@ -46,7 +46,7 @@ async function handleInteractive(ctx: CommandContext, pi: ExtensionAPI): Promise
     ctx.sessionManager.getEntries(),
     ctx.sessionManager.getLeafId(),
   );
-  const evidence = collectHistoryEvidence(sessionContext.messages, snapshot, normalizedNote);
+  const serializedContext = serializeSessionContext(sessionContext.messages);
   const synthesis = await runBriefWithLoader({
     snapshot,
     modelId: model.canonicalId,
@@ -58,7 +58,7 @@ async function handleInteractive(ctx: CommandContext, pi: ExtensionAPI): Promise
         modelRegistry: ctx.modelRegistry,
         cwd: ctx.cwd,
         snapshot,
-        evidence,
+        serializedContext,
         note: normalizedNote,
         signal,
         onProgress,
@@ -73,7 +73,6 @@ async function handleInteractive(ctx: CommandContext, pi: ExtensionAPI): Promise
   const brief = {
     ...synthesis.brief,
     note: normalizedNote,
-    evidenceCount: evidence.length,
   };
 
   const packet = buildReviewPacket(snapshot, brief, model);
@@ -289,7 +288,7 @@ function buildReviewFollowUpInstruction(
   }
 
   const contradictionNote =
-    overall_correctness === "patch is correct" && findings.length > 0
+    overall_correctness.toLowerCase().includes("is correct") && findings.length > 0
       ? "The reviewer marked the patch as correct but found issues — verify the verdict before acting."
       : undefined;
 

package/src/target/packet.ts

@@ -12,17 +12,20 @@ export interface DiffSection {
   deletions: number;
 }
 
-/** Build the final prompt packet for the reviewer child session. */
+/** Build a compact review packet for the reviewer child session.
+ *
+ * The packet contains only the session-derived brief, target metadata, and a
+ * changed-file overview. No large inline diffs are included. The reviewer uses
+ * read_snapshot_diff and read_snapshot_file tools to inspect diffs on demand.
+ */
 export function buildReviewPacket(
   snapshot: ReviewSnapshot,
   brief: SynthesizedReviewBrief,
   model: ReviewModelSelection,
 ): ReviewPacket {
-  const charBudget = getPacketCharBudget(model);
   const { preamble, sections } = splitDiffSections(snapshot.diffText);
-  const orderedSections = prioritizeSections(sections, brief, snapshot.changedFiles);
 
-  const baseParts: string[] = [
+  const parts: string[] = [
     "# Review Task",
     "",
     "## Session-derived intent",
@@ -49,63 +52,29 @@ export function buildReviewPacket(
     "",
     "## Changed files manifest",
     ...snapshot.changedFiles.map((file) => `- ${file}`),
+    "",
+    buildFileOverviewTable(snapshot.changedFiles, sections),
   ];
 
-  baseParts.push("", buildFileOverviewTable(snapshot.changedFiles, sections));
-
   if (preamble.trim()) {
-    baseParts.push("", "## Snapshot notes", truncate(preamble.trim(), 1_500));
+    parts.push("", "## Snapshot notes", truncate(preamble.trim(), 1_500));
   }
 
-  let prompt = baseParts.join("\n");
-  let remaining = charBudget - prompt.length;
-  const includedFiles: string[] = [];
-  const diffBlocks: string[] = [];
-
-  for (const section of orderedSections) {
-    if (remaining <= 2_000) break;
-    const fenced = [`### ${section.file}`, "", "```diff", section.text, "```"].join("\n");
-
-    if (fenced.length <= remaining) {
-      diffBlocks.push(fenced);
-      includedFiles.push(section.file);
-      remaining -= fenced.length + 2;
-      continue;
-    }
-
-    if (includedFiles.length === 0) {
-      const excerptBudget = Math.max(1_000, remaining - 200);
-      const excerpt = truncate(section.text, excerptBudget);
-      const partialBlock = [`### ${section.file}`, "", "```diff", excerpt, "```"].join("\n");
-      diffBlocks.push(partialBlock);
-      includedFiles.push(section.file);
-      remaining -= partialBlock.length + 2;
-    }
-    break;
-  }
-
-  const omittedFiles = snapshot.changedFiles.filter((file) => !includedFiles.includes(file));
-
-  prompt = [
-    prompt,
-    "",
-    "## Included diffs",
-    diffBlocks.length > 0
-      ? diffBlocks.join("\n\n")
-      : "No inline diff sections fit in the prompt budget. Use the changed-file manifest to inspect the files directly.",
+  parts.push(
     "",
-    "## Omitted files",
-    ...toBullets(omittedFiles, "- None"),
+    "## On-demand snapshot inspection",
+    "Use read_snapshot_diff <file> to see the exact diff for any changed file.",
+    "Use read_snapshot_file <file> before|after to inspect file contents on either side of the change.",
+    "These tools are scoped to the snapshot's changed-files list — request a file from the manifest above.",
     "",
-    "Review the included diff carefully. Use read/grep/find/ls to inspect surrounding code before submitting findings.",
-  ].join("\n");
-
-  return {
-    prompt,
-    includedFiles,
-    omittedFiles,
-    charBudget,
-  };
+    "Combine snapshot inspection with read/grep/find/ls for broader codebase context.",
+  );
+
+  const includedFiles = snapshot.changedFiles.filter((f) => !classifySkipCategory(f));
+  const omittedFiles = snapshot.changedFiles.filter((f) => !!classifySkipCategory(f));
+  const charBudget = getPacketCharBudget(model);
+
+  return { prompt: parts.join("\n"), includedFiles, omittedFiles, charBudget };
 }
 
 /** Derive a conservative prompt budget from the selected model's context window. */
@@ -193,43 +162,6 @@ function parseDiffFile(line: string): string | undefined {
   return next === "/dev/null" ? match[1] : next;
 }
 
-function prioritizeSections(
-  sections: DiffSection[],
-  brief: SynthesizedReviewBrief,
-  changedFiles: string[],
-): DiffSection[] {
-  const riskyTokens = brief.riskyFiles.flatMap(toPathTokens);
-  const order = new Map(changedFiles.map((file, index) => [file, index]));
-
-  return [...sections].sort((a, b) => {
-    const delta = sectionScore(b, riskyTokens, order) - sectionScore(a, riskyTokens, order);
-    if (delta !== 0) return delta;
-    return a.file.localeCompare(b.file);
-  });
-}
-
-function sectionScore(
-  section: DiffSection,
-  riskyTokens: string[],
-  order: Map<string, number>,
-): number {
-  const lowerFile = section.file.toLowerCase();
-  const riskyScore = riskyTokens.reduce(
-    (score, token) => score + (lowerFile.includes(token) ? 20 : 0),
-    0,
-  );
-  const orderScore = order.has(section.file) ? Math.max(0, 10 - (order.get(section.file) ?? 0)) : 0;
-  return riskyScore + orderScore;
-}
-
-function toPathTokens(path: string): string[] {
-  return path
-    .toLowerCase()
-    .split(/[\\/._-]/)
-    .map((part) => part.trim())
-    .filter((part) => part.length >= 3);
-}
-
 /** Categorize a file path for skip-list annotation, or undefined if it should be reviewed. */
 export function classifySkipCategory(file: string): string | undefined {
   const lockfiles = new Set([

package/src/tool/brief-runner.ts

@@ -27,8 +27,7 @@ function createSubmitBriefTool(resultHolder: {
     parameters: reviewBriefSchema,
     execute: async (_toolCallId, args) => {
       resultHolder.value = {
-        ...(args as Omit<SynthesizedReviewBrief, "evidenceCount" | "note">),
-        evidenceCount: 0,
+        ...(args as Omit<SynthesizedReviewBrief, "note">),
       };
       return {
         content: [{ type: "text" as const, text: "Review brief submitted successfully." }],

package/src/tool/review-runner.ts

@@ -11,6 +11,7 @@ import {
 import type { ReviewOutputEvent, ReviewResult } from "../types.ts";
 import type { ReviewInvocation, ReviewProgress } from "./runner-types.ts";
 import { reviewOutputSchema } from "./schemas.ts";
+import { createSnapshotDiffTool, createSnapshotFileTool } from "./snapshot-tools.ts";
 
 const DEFAULT_TIMEOUT_MS = 20 * 60 * 1_000;
 const GRACE_TURNS = 3;
@@ -25,6 +26,8 @@ function toolNameToActivity(name: string, phase: "start" | "end"): string {
     find: "finding files",
     ls: "listing files",
     submit_review: "submitting review",
+    read_snapshot_diff: "reading diff",
+    read_snapshot_file: "reading file",
   };
   return map[name] ?? name;
 }
@@ -52,19 +55,52 @@ function createSubmitReviewTool(resultHolder: {
 /** Build the reviewer system prompt used by the read-only child session. */
 export function buildReviewerSystemPrompt(): string {
   return [
-    "You are a rigorous code reviewer.",
-    "The review task already includes session-derived intent and a concrete code snapshot.",
-    "Use the prompt packet as the primary brief, then inspect files with the available read-only tools before drawing conclusions.",
+    "You are a rigorous code reviewer. Your task already includes session-derived intent",
+    "and a concrete list of changed files. Use the prompt packet as the primary brief,",
+    "then inspect code with the available read-only tools before drawing conclusions.",
     "",
     "--- Guardrails ---",
     "- You have read-only tools only. Do NOT modify files or propose running write/edit/bash commands.",
-    "- Verify any suspected issue against the diff and surrounding code before reporting it.",
-    "- If the patch is fully correct, set overall_correctness to 'patch is correct' with high confidence.",
     "",
-    "--- Finding quality ---",
-    "- Title: concise and specific.",
-    "- Body: explain the issue, why it matters, and a concrete fix direction.",
-    "- code_location: 1-based inclusive line range.",
+    "--- Depth ---",
+    "- Read the full changed file, not just the diff. The diff shows what changed;",
+    "  surrounding code shows whether it still makes sense.",
+    "- For high-risk files, also read immediate callers and callees (use grep / find / read",
+    "  to trace references). Without surrounding context you miss broken call sites,",
+    "  stale comments, and silent convention violations.",
+    "",
+    "--- Convention awareness ---",
+    "- Before flagging a style or convention issue, read CLAUDE.md, AGENTS.md, and",
+    "  sibling files in the same directory.",
+    '- "This doesn\'t match the codebase style" only counts when you can point to',
+    "  the real convention in the codebase.",
+    "",
+    "--- What counts as a finding ---",
+    "Report only issues that meet ALL of these criteria:",
+    "1. It meaningfully impacts correctness, security, performance, or maintainability.",
+    "2. It was introduced by this change — pre-existing issues are out of scope",
+    "   unless the change makes them worse.",
+    "3. It is discrete and actionable — the author can fix it in one focused pass.",
+    "4. It does not require assuming unstated intent or speculative downstream effects.",
+    "5. It does not demand a level of rigor not present in the rest of the codebase.",
+    "6. The author would likely fix it if they were made aware of it.",
+    "7. It is not clearly an intentional change by the original author.",
+    "",
+    "--- Do not flag ---",
+    "- Trivial style issues unless they obscure meaning or violate documented standards.",
+    "- Pre-existing bugs unrelated to this change.",
+    '- Things that "might" break without an identified concrete code path.',
+    "- Hypothetical issues without a concrete scenario.",
+    "- Speculative downstream effects — identify the specific affected code.",
+    "",
+    "--- Review checklist ---",
+    "Check for:",
+    "- Logic bugs — wrong condition, off-by-one, missing null/undefined check, race condition.",
+    "- Security — injection, authz bypass, secret exposure.",
+    "- Convention violations — only when you can cite the convention.",
+    "- Missing or weak tests — new behavior without test coverage.",
+    "- Dead or unreachable code introduced by this change.",
+    "- Breaking changes — removed exports, changed signatures, config format changes.",
     "",
     "--- Finding calibration ---",
     "Priority:",
@@ -76,22 +112,36 @@ export function buildReviewerSystemPrompt(): string {
     "  0.8-1.0: you verified the issue by reading surrounding code or grepping the codebase",
     "  0.5-0.8: suspected from the diff, plausible but not fully verified",
     "  <0.5: do not report — too uncertain; either verify further or drop it",
-    "overall_correctness:",
-    "  'patch is correct': no issues found, or only info-level suggestions",
-    "  'mostly correct': minor issues present, safe to merge with fixes",
-    "  'patch is incorrect': major/critical issues that should block merge",
+    "",
+    "--- Verdict ---",
+    "PATCH IS CORRECT: no P2 (major) or P3 (critical) findings. Info/minor suggestions",
+    "  (P0/P1) that are non-blocking do not prevent this verdict.",
+    "PATCH HAS ISSUES: one or more P2 or P3 findings exist. These should block merge.",
+    "Explain the verdict in overall_explanation.",
+    "",
+    "--- Finding format ---",
+    '- Title: concise and specific imperative (e.g. "Guard null token path").',
+    "- Body: what's wrong, why it matters, and a concrete fix direction. One paragraph.",
+    "- code_location: 1-based inclusive line range.",
     "",
     "--- Tool strategy ---",
+    "- Start by fetching the diff for each changed file using read_snapshot_diff.",
+    "- Use read_snapshot_file <file> before|after to inspect file contents on either side of the change.",
     "- Use read to inspect full files when the inline diff lacks context.",
     "- Use grep to verify patterns across the codebase.",
     "- Use find to locate related files quickly.",
     "- Use ls when you need a quick directory overview.",
     "",
+    "--- Large diffs ---",
+    "- If the diff spans many files, prioritize high-risk files (core logic, auth, data handling).",
+    "- Note in overall_explanation which files you reviewed deeply vs. skimmed.",
+    "",
     "--- Skipped files ---",
     "- Skip reviewing: lockfiles, generated/bundled code (dist/, .next/, __generated__/),",
     "  vendored dependencies, changelogs, snapshot files, minified bundles, and binary files.",
     "- Focus on application source and test code.",
     "",
+    "--- Output ---",
     "Do NOT output JSON directly — call submit_review with the structured result.",
   ].join("\n");
 }
@@ -99,6 +149,8 @@ export function buildReviewerSystemPrompt(): string {
 async function createReviewerSession(
   invocation: ReviewInvocation,
   submitReviewTool: ReturnType<typeof defineTool>,
+  snapshotDiffTool: ReturnType<typeof defineTool>,
+  snapshotFileTool: ReturnType<typeof defineTool>,
 ): Promise<AgentSession> {
   const resourceLoader = new DefaultResourceLoader({
     cwd: invocation.cwd,
@@ -117,8 +169,16 @@ async function createReviewerSession(
     model: invocation.model.model,
     modelRegistry: invocation.modelRegistry,
     thinkingLevel: clampThinkingLevel(invocation.model.model, "xhigh"),
-    tools: ["read", "grep", "find", "ls", "submit_review"],
-    customTools: [submitReviewTool],
+    tools: [
+      "read",
+      "grep",
+      "find",
+      "ls",
+      "submit_review",
+      "read_snapshot_diff",
+      "read_snapshot_file",
+    ],
+    customTools: [submitReviewTool, snapshotDiffTool, snapshotFileTool],
     resourceLoader,
     sessionManager: SessionManager.inMemory(invocation.cwd),
   });
@@ -313,10 +373,17 @@ export async function runReviewer(invocation: ReviewInvocation): Promise<ReviewR
 
   const resultHolder: { value: ReviewOutputEvent | undefined } = { value: undefined };
   const submitReviewTool = createSubmitReviewTool(resultHolder);
+  const snapshotDiffTool = createSnapshotDiffTool(invocation.cwd, invocation.snapshot);
+  const snapshotFileTool = createSnapshotFileTool(invocation.cwd, invocation.snapshot);
 
   let session: AgentSession;
   try {
-    session = await createReviewerSession(invocation, submitReviewTool);
+    session = await createReviewerSession(
+      invocation,
+      submitReviewTool,
+      snapshotDiffTool,
+      snapshotFileTool,
+    );
   } catch (error) {
     return {
       kind: "failed",

package/src/tool/runner-types.ts

@@ -20,12 +20,6 @@ export type BriefSynthesisRunResult =
   | { kind: "canceled" }
   | { kind: "timeout"; timeoutMs: number };
 
-export interface BriefSynthesisInput {
-  snapshot: ReviewSnapshot;
-  evidence: Array<{ kind: string; reason: string; text: string }>;
-  note?: string;
-}
-
 export interface BriefSynthesisInvocation {
   prompt: string;
   // biome-ignore lint/suspicious/noExplicitAny: Model<any> is pi's canonical type

package/src/tool/snapshot-tools.ts

@@ -0,0 +1,116 @@
+import { defineTool } from "@earendil-works/pi-coding-agent";
+import { Type } from "typebox";
+import { getSnapshotFileContent, getSnapshotFileDiff } from "../git.ts";
+import type { ReviewSnapshot } from "../types.ts";
+
+/** Create the read_snapshot_diff custom tool for the reviewer child session. */
+export function createSnapshotDiffTool(
+  cwd: string,
+  snapshot: ReviewSnapshot,
+): ReturnType<typeof defineTool> {
+  return defineTool({
+    name: "read_snapshot_diff",
+    label: "Read Snapshot Diff",
+    description:
+      "Read the exact diff for a single changed file in the selected review snapshot. " +
+      "The file must be in the snapshot's changed-files list.",
+    parameters: Type.Object({
+      file: Type.String(),
+    }),
+    execute: async (_toolCallId, args) => {
+      const file = (args as { file: string }).file;
+      if (!snapshot.changedFiles.includes(file)) {
+        return {
+          content: [
+            {
+              type: "text" as const,
+              text:
+                `Error: "${file}" is not in the snapshot's changed-files list. ` +
+                "Use the changed-file manifest from the review task to pick a valid file.",
+            },
+          ],
+          details: null,
+        };
+      }
+
+      const diff = await getSnapshotFileDiff(cwd, snapshot, file);
+      if (!diff.trim()) {
+        return {
+          content: [
+            {
+              type: "text" as const,
+              text:
+                `No diff available for "${file}" in this snapshot. ` +
+                "The file may be untracked; use read_snapshot_file to inspect its current content.",
+            },
+          ],
+          details: null,
+        };
+      }
+
+      return {
+        content: [{ type: "text" as const, text: diff }],
+        details: null,
+      };
+    },
+  });
+}
+
+/** Create the read_snapshot_file custom tool for the reviewer child session. */
+export function createSnapshotFileTool(
+  cwd: string,
+  snapshot: ReviewSnapshot,
+): ReturnType<typeof defineTool> {
+  return defineTool({
+    name: "read_snapshot_file",
+    label: "Read Snapshot File",
+    description:
+      "Read the before or after content of a single changed file in the selected review snapshot. " +
+      '"before" shows the file before the change (HEAD for working tree, base for branch, parent for commit). ' +
+      '"after" shows the file after the change. The file must be in the snapshot\'s changed-files list.',
+    parameters: Type.Object({
+      file: Type.String(),
+      side: Type.Union([Type.Literal("before"), Type.Literal("after")]),
+    }),
+    execute: async (_toolCallId, args) => {
+      const { file, side } = args as { file: string; side: "before" | "after" };
+
+      if (!snapshot.changedFiles.includes(file)) {
+        return {
+          content: [
+            {
+              type: "text" as const,
+              text:
+                `Error: "${file}" is not in the snapshot's changed-files list. ` +
+                "Use the changed-file manifest from the review task to pick a valid file.",
+            },
+          ],
+          details: null,
+        };
+      }
+
+      const content = await getSnapshotFileContent(cwd, snapshot, file, side);
+
+      if (content === undefined) {
+        const hint =
+          side === "before"
+            ? "The file may be newly added or renamed (use read_snapshot_diff to check for renames)."
+            : "The file may have been deleted.";
+        return {
+          content: [
+            {
+              type: "text" as const,
+              text: `Content for "${file}" (${side}) is not available. ${hint}`,
+            },
+          ],
+          details: null,
+        };
+      }
+
+      return {
+        content: [{ type: "text" as const, text: content }],
+        details: null,
+      };
+    },
+  });
+}

package/src/types.ts

@@ -51,14 +51,6 @@ export interface ReviewSnapshot {
   stats: DiffStats;
 }
 
-/** One evidence item extracted from the active session branch. */
-export interface HistoryEvidence {
-  kind: "user" | "assistant" | "custom" | "compaction" | "branch-summary";
-  text: string;
-  score: number;
-  reason: string;
-}
-
 /** Model picked explicitly for the current review run. */
 export interface ReviewModelSelection {
   canonicalId: string;
@@ -80,7 +72,6 @@ export interface SynthesizedReviewBrief {
   riskyFiles: string[];
   unresolvedQuestions: string[];
   note?: string;
-  evidenceCount: number;
 }
 
 /** Final prompt packet passed to the reviewer child session. */

package/src/ui/flow.ts

@@ -1,3 +1,8 @@
+import { spawn } from "node:child_process";
+import { unlinkSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+
 import { DynamicBorder, type ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { Container, type SelectItem, SelectList, Spacer, Text } from "@earendil-works/pi-tui";
 import { getLocalBranches, getRecentCommits } from "../git.ts";
@@ -135,23 +140,62 @@ export async function collectReviewNote(ctx: ExtensionContext): Promise<string |
 
 /** Show the synthesized brief, the actual reviewer prompt preview, and ask for approval. */
 export function previewReviewPlan(ctx: ExtensionContext, plan: ReviewPlan): Promise<boolean> {
-  return ctx.ui.custom<boolean>((_tui, theme, _kb, done) => {
+  return ctx.ui.custom<boolean>((tui, theme, _kb, done) => {
     const container = buildReviewPlanContainer(theme, plan);
+    const pagerState = { running: false };
+
+    function handleInput(data: string) {
+      if (pagerState.running) return;
+      if (data === "\r" || data === "\n" || data === "y" || data === "Y") {
+        done(true);
+      } else if (data === "\x1b" || data === "n" || data === "N") {
+        done(false);
+      } else if (data === "v" || data === "V") {
+        openPromptInPager(plan.packet.prompt, tui, pagerState);
+      }
+    }
 
     return {
       render: (width) => container.render(width),
       invalidate: () => container.invalidate(),
-      handleInput: (data) => {
-        if (data === "\r" || data === "\n" || data === "y" || data === "Y") {
-          done(true);
-        } else if (data === "\x1b" || data === "n" || data === "N") {
-          done(false);
-        }
-      },
+      handleInput,
     };
   });
 }
 
+/** Open the full reviewer prompt in the user's pager for inspection. */
+function openPromptInPager(
+  prompt: string,
+  tui: { requestRender: () => void },
+  pagerState: { running: boolean },
+): void {
+  const tmpFile = join(tmpdir(), `supi-review-prompt-${Date.now()}.txt`);
+  try {
+    writeFileSync(tmpFile, prompt, "utf-8");
+  } catch {
+    return;
+  }
+
+  pagerState.running = true;
+
+  let cleanedUp = false;
+  const cleanup = () => {
+    if (cleanedUp) return;
+    cleanedUp = true;
+    pagerState.running = false;
+    try {
+      unlinkSync(tmpFile);
+    } catch {
+      // best-effort cleanup
+    }
+    tui.requestRender();
+  };
+
+  const pager = spawn("less", ["-R", tmpFile], { stdio: "inherit" });
+  pager.on("exit", cleanup);
+  pager.on("error", cleanup);
+}
+
 /** Build the review plan preview container with all styled sections. */
 function buildReviewPlanContainer(theme: ReviewTheme, plan: ReviewPlan): Container {
   const { model, snapshot, brief, packet } = plan;
@@ -236,25 +280,16 @@ function buildReviewPlanContainer(theme: ReviewTheme, plan: ReviewPlan): Contain
   container.addChild(new Text(previewText, 1, 0));
   container.addChild(new Spacer(1));
 
-  // ── File coverage line ──
-  container.addChild(
-    new Text(
-      theme.fg(
-        "dim",
-        `  Included diffs: ${packet.includedFiles.length} file${packet.includedFiles.length === 1 ? "" : "s"}` +
-          `  \u2022  Omitted: ${packet.omittedFiles.length} file${packet.omittedFiles.length === 1 ? "" : "s"}` +
-          `  \u2022  Budget: ${(packet.charBudget / 1000).toFixed(0)}K chars`,
-      ),
-      1,
-      0,
-    ),
-  );
+  // ── Snapshot access line ──
+  const accessLine = `  Diffs: on-demand via read_snapshot_diff  \u2022  Files: ${snapshot.changedFiles.length} changed`;
+
+  container.addChild(new Text(theme.fg("dim", accessLine), 1, 0));
   container.addChild(new Spacer(1));
 
   // ── Confirm / Cancel hints ──
   container.addChild(
     new Text(
-      `  ${dim("Enter")} ${theme.fg("success", "Run review")}  ${dim("\u2022")}  ${dim("Esc")} ${theme.fg("muted", "Cancel")}  ${dim("\u2022 y/n")}`,
+      `  ${dim("Enter")} ${theme.fg("success", "Run review")}  ${dim("\u2022")}  ${dim("Esc")} ${theme.fg("muted", "Cancel")}  ${dim("\u2022 y/n")}  ${dim("\u2022")}  ${dim("v")} ${theme.fg("accent", "view full prompt")}`,
       1,
       0,
     ),