@jpkme/diff-review 0.1.12 → 0.1.14

package/lib/model-label.test.mjs

@@ -1,77 +1,77 @@
-import assert from "node:assert/strict";
-import { test } from "node:test";
-import {
-  buildAcpModelSelectOptions,
-  compactModelMeta,
-  formatModelOption,
-  humanizeModelId,
-  modelOptionLabel,
-} from "./model-label.mjs";
-
-test("humanizeModelId", () => {
-  assert.equal(humanizeModelId("claude-opus-5-5"), "Claude Opus 5.5");
-  assert.equal(humanizeModelId("claude-sonnet-5.5"), "Claude Sonnet 5.5");
-  assert.equal(humanizeModelId("gemini-3.8-flash"), "Gemini 3.8 Flash");
-  assert.equal(humanizeModelId("gpt-5.6-sol"), "GPT 5.6 Sol");
-});
-
-test("compactModelMeta", () => {
-  assert.deepEqual(compactModelMeta("context=300k,effort=medium,fast=false"), ["300k", "medium"]);
-  assert.deepEqual(compactModelMeta("thinking=true,context=300k,effort=high,fast=false"), [
-    "thinking",
-    "300k",
-    "high",
-  ]);
-  assert.deepEqual(compactModelMeta("1m"), ["1m"]);
-});
-
-test("formatModelOption compacto y amigable", () => {
-  const a = formatModelOption("claude-opus-5-5", "claude-opus-5-5[context=300k,effort=medium,fast=false]");
-  assert.equal(a.title, "Claude Opus 5.5");
-  assert.deepEqual(a.badges, ["300k", "medium"]);
-  assert.equal(a.label, "Claude Opus 5.5  ·  [ 300k, medium ]");
-  assert.equal(a.hint, "[ 300k, medium ]");
-
-  const b = formatModelOption("Claude Sonnet 5.5", "claude-sonnet-5.5");
-  assert.equal(b.title, "Claude Sonnet 5.5");
-  assert.deepEqual(b.badges, []);
-  assert.equal(b.label, "Claude Sonnet 5.5");
-
-  const c = formatModelOption("Sonnet (1M context)", "sonnet[1m]");
-  assert.equal(c.title, "Sonnet (1M context)");
-  assert.deepEqual(c.badges, []);
-});
-
-test("buildAcpModelSelectOptions fusiona aliases y formatea ACP", () => {
-  const { options } = buildAcpModelSelectOptions(
-    {
-      currentModel: "default",
-      models: [
-        { value: "default", name: "Default (recommended)" },
-        {
-          value: "claude-opus-5-5[context=300k,effort=medium,fast=false]",
-          name: "claude-opus-5-5",
-        },
-      ],
-    },
-    {
-      options: [
-        { value: "", label: "Por defecto (opus)" },
-        { value: "opus", label: "opus", hint: "el más capaz" },
-      ],
-    }
-  );
-  assert.equal(options[0].label, "Por defecto (opus)");
-  assert.ok(options.some((o) => o.value === "opus"));
-  const opusAcp = options.find((o) => o.value.startsWith("claude-opus-5-5"));
-  assert.equal(opusAcp?.title, "Claude Opus 5.5");
-  assert.deepEqual(opusAcp?.badges, ["300k", "medium"]);
-  assert.equal(opusAcp?.hint, "[ 300k, medium ]");
-});
-
-test("modelOptionLabel compat", () => {
-  assert.equal(
-    modelOptionLabel("grok-4.7", "grok-4.7[context=256k,fast=true]"),
-    "Grok 4.7  ·  [ 256k, fast ]"
-  );
-});
+import assert from "node:assert/strict";
+import { test } from "node:test";
+import {
+  buildAcpModelSelectOptions,
+  compactModelMeta,
+  formatModelOption,
+  humanizeModelId,
+  modelOptionLabel,
+} from "./model-label.mjs";
+
+test("humanizeModelId", () => {
+  assert.equal(humanizeModelId("claude-opus-5-5"), "Claude Opus 5.5");
+  assert.equal(humanizeModelId("claude-sonnet-5.5"), "Claude Sonnet 5.5");
+  assert.equal(humanizeModelId("gemini-3.8-flash"), "Gemini 3.8 Flash");
+  assert.equal(humanizeModelId("gpt-5.6-sol"), "GPT 5.6 Sol");
+});
+
+test("compactModelMeta", () => {
+  assert.deepEqual(compactModelMeta("context=300k,effort=medium,fast=false"), ["300k", "medium"]);
+  assert.deepEqual(compactModelMeta("thinking=true,context=300k,effort=high,fast=false"), [
+    "thinking",
+    "300k",
+    "high",
+  ]);
+  assert.deepEqual(compactModelMeta("1m"), ["1m"]);
+});
+
+test("formatModelOption compacto y amigable", () => {
+  const a = formatModelOption("claude-opus-5-5", "claude-opus-5-5[context=300k,effort=medium,fast=false]");
+  assert.equal(a.title, "Claude Opus 5.5");
+  assert.deepEqual(a.badges, ["300k", "medium"]);
+  assert.equal(a.label, "Claude Opus 5.5  ·  [ 300k, medium ]");
+  assert.equal(a.hint, "[ 300k, medium ]");
+
+  const b = formatModelOption("Claude Sonnet 5.5", "claude-sonnet-5.5");
+  assert.equal(b.title, "Claude Sonnet 5.5");
+  assert.deepEqual(b.badges, []);
+  assert.equal(b.label, "Claude Sonnet 5.5");
+
+  const c = formatModelOption("Sonnet (1M context)", "sonnet[1m]");
+  assert.equal(c.title, "Sonnet (1M context)");
+  assert.deepEqual(c.badges, []);
+});
+
+test("buildAcpModelSelectOptions fusiona aliases y formatea ACP", () => {
+  const { options } = buildAcpModelSelectOptions(
+    {
+      currentModel: "default",
+      models: [
+        { value: "default", name: "Default (recommended)" },
+        {
+          value: "claude-opus-5-5[context=300k,effort=medium,fast=false]",
+          name: "claude-opus-5-5",
+        },
+      ],
+    },
+    {
+      options: [
+        { value: "", label: "Por defecto (opus)" },
+        { value: "opus", label: "opus", hint: "el más capaz" },
+      ],
+    }
+  );
+  assert.equal(options[0].label, "Por defecto (opus)");
+  assert.ok(options.some((o) => o.value === "opus"));
+  const opusAcp = options.find((o) => o.value.startsWith("claude-opus-5-5"));
+  assert.equal(opusAcp?.title, "Claude Opus 5.5");
+  assert.deepEqual(opusAcp?.badges, ["300k", "medium"]);
+  assert.equal(opusAcp?.hint, "[ 300k, medium ]");
+});
+
+test("modelOptionLabel compat", () => {
+  assert.equal(
+    modelOptionLabel("grok-4.7", "grok-4.7[context=256k,fast=true]"),
+    "Grok 4.7  ·  [ 256k, fast ]"
+  );
+});

package/lib/models-cache.mjs

@@ -1,75 +1,75 @@
-/**
- * Caché en disco de las listas de modelos ACP por agente.
- * TTL 24h: los catálogos cambian de a poco (semanas); un día evita spawnear el CLI en cada review
- * y sigue fresco si aparece un modelo nuevo.
- */
-import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
-import { homedir } from "node:os";
-import { dirname, join } from "node:path";
-
-/** 24 horas. */
-export const MODELS_CACHE_TTL_MS = 24 * 60 * 60 * 1000;
-
-/** @returns {string} */
-export function modelsCachePath(home = homedir()) {
-  return join(home, ".diff-review", "models-cache.json");
-}
-
-/**
- * @param {string} agentId
- * @param {{ home?: string, now?: number, ttlMs?: number }} [opts]
- * @returns {{ models: { value: string, name: string }[], currentModel: string, fetchedAt: string } | null}
- */
-export function readModelsCache(agentId, opts = {}) {
-  const id = String(agentId || "").trim();
-  if (!id) return null;
-  const path = modelsCachePath(opts.home);
-  let root;
-  try {
-    root = JSON.parse(readFileSync(path, "utf8"));
-  } catch {
-    return null;
-  }
-  const entry = root?.agents?.[id];
-  if (!entry || !Array.isArray(entry.models)) return null;
-  const fetchedAt = Date.parse(String(entry.fetchedAt || ""));
-  if (!Number.isFinite(fetchedAt)) return null;
-  const ttl = opts.ttlMs ?? MODELS_CACHE_TTL_MS;
-  const now = opts.now ?? Date.now();
-  if (now - fetchedAt > ttl) return null;
-  return {
-    models: entry.models
-      .filter((m) => m && typeof m.value === "string")
-      .map((m) => ({ value: m.value, name: String(m.name ?? m.value) })),
-    currentModel: String(entry.currentModel ?? ""),
-    fetchedAt: new Date(fetchedAt).toISOString(),
-  };
-}
-
-/**
- * @param {string} agentId
- * @param {{ models: { value: string, name: string }[], currentModel?: string }} data
- * @param {{ home?: string, now?: number }} [opts]
- */
-export function writeModelsCache(agentId, data, opts = {}) {
-  const id = String(agentId || "").trim();
-  if (!id || !Array.isArray(data?.models) || !data.models.length) return;
-  const path = modelsCachePath(opts.home);
-  let root = { version: 1, agents: {} };
-  try {
-    if (existsSync(path)) root = { ...root, ...JSON.parse(readFileSync(path, "utf8")) };
-  } catch {
-    /* archivo nuevo */
-  }
-  if (!root.agents || typeof root.agents !== "object") root.agents = {};
-  root.agents[id] = {
-    fetchedAt: new Date(opts.now ?? Date.now()).toISOString(),
-    currentModel: String(data.currentModel ?? ""),
-    models: data.models.map((m) => ({
-      value: String(m.value),
-      name: String(m.name ?? m.value),
-    })),
-  };
-  mkdirSync(dirname(path), { recursive: true });
-  writeFileSync(path, `${JSON.stringify(root, null, 2)}\n`);
-}
+/**
+ * Caché en disco de las listas de modelos ACP por agente.
+ * TTL 24h: los catálogos cambian de a poco (semanas); un día evita spawnear el CLI en cada review
+ * y sigue fresco si aparece un modelo nuevo.
+ */
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import { homedir } from "node:os";
+import { dirname, join } from "node:path";
+
+/** 24 horas. */
+export const MODELS_CACHE_TTL_MS = 24 * 60 * 60 * 1000;
+
+/** @returns {string} */
+export function modelsCachePath(home = homedir()) {
+  return join(home, ".diff-review", "models-cache.json");
+}
+
+/**
+ * @param {string} agentId
+ * @param {{ home?: string, now?: number, ttlMs?: number }} [opts]
+ * @returns {{ models: { value: string, name: string }[], currentModel: string, fetchedAt: string } | null}
+ */
+export function readModelsCache(agentId, opts = {}) {
+  const id = String(agentId || "").trim();
+  if (!id) return null;
+  const path = modelsCachePath(opts.home);
+  let root;
+  try {
+    root = JSON.parse(readFileSync(path, "utf8"));
+  } catch {
+    return null;
+  }
+  const entry = root?.agents?.[id];
+  if (!entry || !Array.isArray(entry.models)) return null;
+  const fetchedAt = Date.parse(String(entry.fetchedAt || ""));
+  if (!Number.isFinite(fetchedAt)) return null;
+  const ttl = opts.ttlMs ?? MODELS_CACHE_TTL_MS;
+  const now = opts.now ?? Date.now();
+  if (now - fetchedAt > ttl) return null;
+  return {
+    models: entry.models
+      .filter((m) => m && typeof m.value === "string")
+      .map((m) => ({ value: m.value, name: String(m.name ?? m.value) })),
+    currentModel: String(entry.currentModel ?? ""),
+    fetchedAt: new Date(fetchedAt).toISOString(),
+  };
+}
+
+/**
+ * @param {string} agentId
+ * @param {{ models: { value: string, name: string }[], currentModel?: string }} data
+ * @param {{ home?: string, now?: number }} [opts]
+ */
+export function writeModelsCache(agentId, data, opts = {}) {
+  const id = String(agentId || "").trim();
+  if (!id || !Array.isArray(data?.models) || !data.models.length) return;
+  const path = modelsCachePath(opts.home);
+  let root = { version: 1, agents: {} };
+  try {
+    if (existsSync(path)) root = { ...root, ...JSON.parse(readFileSync(path, "utf8")) };
+  } catch {
+    /* archivo nuevo */
+  }
+  if (!root.agents || typeof root.agents !== "object") root.agents = {};
+  root.agents[id] = {
+    fetchedAt: new Date(opts.now ?? Date.now()).toISOString(),
+    currentModel: String(data.currentModel ?? ""),
+    models: data.models.map((m) => ({
+      value: String(m.value),
+      name: String(m.name ?? m.value),
+    })),
+  };
+  mkdirSync(dirname(path), { recursive: true });
+  writeFileSync(path, `${JSON.stringify(root, null, 2)}\n`);
+}

package/lib/models-cache.test.mjs

@@ -1,50 +1,50 @@
-import assert from "node:assert/strict";
-import { mkdtempSync, readFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { test } from "node:test";
-import {
-  MODELS_CACHE_TTL_MS,
-  modelsCachePath,
-  readModelsCache,
-  writeModelsCache,
-} from "./models-cache.mjs";
-import { sortModelSelectOptions } from "./model-label.mjs";
-
-test("sortModelSelectOptions: default y auto primero, resto lexico", () => {
-  const sorted = sortModelSelectOptions([
-    { value: "zulu", label: "Zulu" },
-    { value: "auto", label: "Auto" },
-    { value: "alpha", label: "Alpha" },
-    { value: "", label: "Por defecto" },
-    { value: "bravo", label: "Bravo" },
-  ]);
-  assert.deepEqual(
-    sorted.map((o) => o.value),
-    ["", "auto", "alpha", "bravo", "zulu"]
-  );
-});
-
-test("models cache roundtrip + TTL", () => {
-  const home = mkdtempSync(join(tmpdir(), "diff-review-mc-"));
-  const now = Date.parse("2026-01-01T12:00:00.000Z");
-  writeModelsCache(
-    "claude",
-    {
-      models: [{ value: "opus", name: "Opus" }],
-      currentModel: "default",
-    },
-    { home, now }
-  );
-  assert.ok(readFileSync(modelsCachePath(home), "utf8").includes("opus"));
-
-  const hit = readModelsCache("claude", { home, now: now + 1000 });
-  assert.equal(hit?.currentModel, "default");
-  assert.equal(hit?.models[0].value, "opus");
-
-  const expired = readModelsCache("claude", {
-    home,
-    now: now + MODELS_CACHE_TTL_MS + 1,
-  });
-  assert.equal(expired, null);
-});
+import assert from "node:assert/strict";
+import { mkdtempSync, readFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { test } from "node:test";
+import {
+  MODELS_CACHE_TTL_MS,
+  modelsCachePath,
+  readModelsCache,
+  writeModelsCache,
+} from "./models-cache.mjs";
+import { sortModelSelectOptions } from "./model-label.mjs";
+
+test("sortModelSelectOptions: default y auto primero, resto lexico", () => {
+  const sorted = sortModelSelectOptions([
+    { value: "zulu", label: "Zulu" },
+    { value: "auto", label: "Auto" },
+    { value: "alpha", label: "Alpha" },
+    { value: "", label: "Por defecto" },
+    { value: "bravo", label: "Bravo" },
+  ]);
+  assert.deepEqual(
+    sorted.map((o) => o.value),
+    ["", "auto", "alpha", "bravo", "zulu"]
+  );
+});
+
+test("models cache roundtrip + TTL", () => {
+  const home = mkdtempSync(join(tmpdir(), "diff-review-mc-"));
+  const now = Date.parse("2026-01-01T12:00:00.000Z");
+  writeModelsCache(
+    "claude",
+    {
+      models: [{ value: "opus", name: "Opus" }],
+      currentModel: "default",
+    },
+    { home, now }
+  );
+  assert.ok(readFileSync(modelsCachePath(home), "utf8").includes("opus"));
+
+  const hit = readModelsCache("claude", { home, now: now + 1000 });
+  assert.equal(hit?.currentModel, "default");
+  assert.equal(hit?.models[0].value, "opus");
+
+  const expired = readModelsCache("claude", {
+    home,
+    now: now + MODELS_CACHE_TTL_MS + 1,
+  });
+  assert.equal(expired, null);
+});

package/lib/prompt.mjs

@@ -91,7 +91,7 @@ const RULES = `- Ids: themes use \`t1\`, \`t2\`, …; findings use \`f1\`, \`f2\
 function workflowLocal(base, branch, ctx = {}) {
   if (ctx.hasDiff) {
     return `Workflow:
-1. Read the three-dot diff from the path given in "Precomputed git context" below. Do not re-run \`git diff\` unless that file is missing.
+1. The host already computed the three-dot diff and wrote it to disk (see "Precomputed git context"). Read that file with your file tools. Do not run \`git diff\`, \`git merge-base\`, or rebuild the patch yourself unless that file is missing.
 2. Read enough surrounding code (types, callers, tests) to judge behavior, not only the hunk lines.
 3. Emit the JSON file. Do not modify the repo.
 4. Validate it with the command in the output section. If it prints INVALID, follow "How to continue" and re-run until it prints OK.`;
@@ -115,7 +115,7 @@ function workflowUrl(base, branch, ctx = {}) {
   if (ctx.hasDiff) {
     return `Workflow:
 1. If the repo is not local, clone it (or use whatever access you have). Do not modify it.
-2. Read the three-dot diff from the path given in "Precomputed git context" below. Do not re-run \`git diff\` unless that file is missing.
+2. The host already computed the three-dot diff and wrote it to disk (see "Precomputed git context"). Read that file with your file tools. Do not run \`git diff\`, \`git merge-base\`, or rebuild the patch yourself unless that file is missing.
 3. Read enough surrounding code (types, callers, tests) to judge behavior, not only the hunk lines.
 4. Emit the JSON file.
 5. Validate it with the command in the output section. If it prints INVALID, follow "How to continue" and re-run until it prints OK.`;
@@ -133,7 +133,7 @@ function workflowUrl(base, branch, ctx = {}) {
 function workflowLocalAcp(ctx = {}) {
   if (ctx.hasDiff) {
     return `Workflow:
-1. Read the three-dot diff from the path given in "Precomputed git context" below. Do not re-run \`git diff\` unless that file is missing.
+1. The host already computed the three-dot diff and wrote it to disk (see "Precomputed git context"). Read that file with your file tools. Do not run \`git diff\`, \`git merge-base\`, or rebuild the patch yourself unless that file is missing.
 2. Read enough surrounding code (types, callers, tests) to judge behavior, not only the hunk lines.
 3. Write the JSON with your file Write/Edit tool (never shell redirection). Do not modify other repo files. The host validates when your turn ends — do not run the validate CLI.`;
   }
@@ -147,7 +147,7 @@ function workflowUrlAcp(ctx = {}) {
   if (ctx.hasDiff) {
     return `Workflow:
 1. If the repo is not local, clone it (or use whatever access you have). Do not modify it.
-2. Read the three-dot diff from the path given in "Precomputed git context" below. Do not re-run \`git diff\` unless that file is missing.
+2. The host already computed the three-dot diff and wrote it to disk (see "Precomputed git context"). Read that file with your file tools. Do not run \`git diff\`, \`git merge-base\`, or rebuild the patch yourself unless that file is missing.
 3. Read enough surrounding code (types, callers, tests) to judge behavior, not only the hunk lines.
 4. Write the JSON with your file Write/Edit tool (never shell redirection). The host validates when your turn ends — do not run the validate CLI.`;
   }
@@ -197,13 +197,15 @@ export function outputSchemaBlock(meta) {
  * }} ctx
  */
 function formatPrecomputedContext(ctx) {
-  const lines = ["Precomputed git context (do not rediscover):"];
+  const lines = [
+    "Precomputed git context (provided by the host — do not rediscover or recompute):",
+  ];
   if (ctx.mergeBase) lines.push(`- Merge-base SHA: \`${ctx.mergeBase}\``);
   if (ctx.branchSha) lines.push(`- Branch tip SHA: \`${ctx.branchSha}\``);
   if (ctx.baseSha) lines.push(`- Base tip SHA: \`${ctx.baseSha}\``);
   if (ctx.diffRelPath) {
     lines.push(
-      `- Diff file (relative to repo root): \`${ctx.diffRelPath}\` — read this file; do not re-run \`git diff\` unless it is missing.`
+      `- Diff file (relative to repo root): \`${ctx.diffRelPath}\` — Read this file for the full patch. Do not run \`git diff\` / \`git merge-base\` to rebuild it unless the file is missing.`
     );
   }
   const files = Array.isArray(ctx.files) ? ctx.files : [];

package/lib/review-prep.mjs

@@ -1,63 +1,63 @@
-/**
- * Precomputa contexto git para el prompt de review (diff cache + lista de archivos).
- */
-import { loadReviewConfig } from "./config.mjs";
-import { extractBranchDiff } from "./extract.mjs";
-import { outputRelPath } from "./output-name.mjs";
-import { ensureReviewDir, readDiffCache, writeDiffCache } from "./review-dir.mjs";
-
-/**
- * Cachea el three-dot diff y arma las opciones de `buildPrompt` (tips, files, ignores).
- * @param {{
- *   repo: string,
- *   branch: string,
- *   base?: string,
- *   outputFilename: string,
- *   acp?: boolean,
- * }} input
- */
-export function prepareReviewPromptOpts(input) {
-  const repo = String(input.repo || "").trim();
-  const branch = String(input.branch || "").trim();
-  const base = String(input.base || "develop").trim() || "develop";
-  const outputFilename = String(input.outputFilename || "").trim();
-  if (!repo || !branch || !outputFilename) {
-    throw new Error("prepareReviewPromptOpts requiere repo, branch y outputFilename.");
-  }
-
-  ensureReviewDir(repo);
-  writeDiffCache(repo, outputFilename, { branch, base });
-  const cached = readDiffCache(repo, outputFilename);
-
-  let extracted;
-  try {
-    extracted = extractBranchDiff({
-      repo,
-      branch,
-      base,
-      ...(cached ? { cachedDiff: cached } : {}),
-    });
-  } catch {
-    extracted = extractBranchDiff({ repo, branch, base });
-  }
-
-  const config = loadReviewConfig(repo);
-  const diffRelPath = cached
-    ? outputRelPath(outputFilename).replace(/\.json$/i, ".diff")
-    : null;
-
-  return {
-    outputFilename,
-    acp: Boolean(input.acp),
-    diffRelPath,
-    mergeBase: extracted.mergeBase || undefined,
-    branchSha: extracted.branchSha || undefined,
-    baseSha: extracted.baseSha || undefined,
-    files: (extracted.files || []).map((f) => ({
-      path: f.path,
-      changeType: f.changeType,
-    })),
-    ignored: extracted.context?.ignored || [],
-    ignoreGlobs: config.ignore,
-  };
-}
+/**
+ * Precomputa contexto git para el prompt de review (diff cache + lista de archivos).
+ */
+import { loadReviewConfig } from "./config.mjs";
+import { extractBranchDiff } from "./extract.mjs";
+import { outputRelPath } from "./output-name.mjs";
+import { ensureReviewDir, readDiffCache, writeDiffCache } from "./review-dir.mjs";
+
+/**
+ * Cachea el three-dot diff y arma las opciones de `buildPrompt` (tips, files, ignores).
+ * @param {{
+ *   repo: string,
+ *   branch: string,
+ *   base?: string,
+ *   outputFilename: string,
+ *   acp?: boolean,
+ * }} input
+ */
+export function prepareReviewPromptOpts(input) {
+  const repo = String(input.repo || "").trim();
+  const branch = String(input.branch || "").trim();
+  const base = String(input.base || "develop").trim() || "develop";
+  const outputFilename = String(input.outputFilename || "").trim();
+  if (!repo || !branch || !outputFilename) {
+    throw new Error("prepareReviewPromptOpts requiere repo, branch y outputFilename.");
+  }
+
+  ensureReviewDir(repo);
+  writeDiffCache(repo, outputFilename, { branch, base });
+  const cached = readDiffCache(repo, outputFilename);
+
+  let extracted;
+  try {
+    extracted = extractBranchDiff({
+      repo,
+      branch,
+      base,
+      ...(cached ? { cachedDiff: cached } : {}),
+    });
+  } catch {
+    extracted = extractBranchDiff({ repo, branch, base });
+  }
+
+  const config = loadReviewConfig(repo);
+  const diffRelPath = cached
+    ? outputRelPath(outputFilename).replace(/\.json$/i, ".diff")
+    : null;
+
+  return {
+    outputFilename,
+    acp: Boolean(input.acp),
+    diffRelPath,
+    mergeBase: extracted.mergeBase || undefined,
+    branchSha: extracted.branchSha || undefined,
+    baseSha: extracted.baseSha || undefined,
+    files: (extracted.files || []).map((f) => ({
+      path: f.path,
+      changeType: f.changeType,
+    })),
+    ignored: extracted.context?.ignored || [],
+    ignoreGlobs: config.ignore,
+  };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jpkme/diff-review",
-  "version": "0.1.12",
+  "version": "0.1.14",
   "description": "Local UI to prompt a coding agent for a branch review and browse the result.",
   "type": "module",
   "bin": {

package/web/build/client/_app/immutable/assets/{0.CLQiq3Ed.css → 0.BKRG1f36.css}

@@ -1 +1 @@
-:root{--bg:#0b0e14;--bg-elev:#11151d;--bg-card:#151a24;--bg-card-hover:#1a2130;--border:#242b3a;--border-soft:#1c2230;--text:#e6e9ef;--text-dim:#9aa4b8;--text-faint:#5f6b82;--accent:#6ea8fe;--accent-soft:#1c2b45;--ok:#7ee2a8;--ok-bg:#0d2a1c;--warn:#e2c67e;--danger:#ff9a9a;--danger-bg:#2a1414;--danger-border:#4a1e1e;--mono:ui-monospace, "JetBrains Mono", "Fira Code", Consolas, "SFMono-Regular", Menlo, monospace;--sans:-apple-system, "Segoe UI", Inter, Roboto, Helvetica, Arial, sans-serif;--btn-primary-bg:#1c2b45;--btn-primary-border:#3a5a8f;--btn-primary-text:#cfe0ff;--kind-feat:#7ee2a8;--kind-fix:#ff9a9a;--kind-refactor:#c9a5ff;--kind-perf:#e2c67e;--kind-test:#6ea8fe;--kind-chore:#9aa4b8;--kind-docs:#7fd6e0;--kind-infra:#f0a868;--sev-high:#ff9a9a;--sev-med:#f08a5a;--sev-low:#c4b86a;--sev-nit:#7f8aa3;--diff-add-bg:#0f2419;--diff-add-fg:#7ee2a8;--diff-del-bg:#2a1418;--diff-del-fg:#ff9a9a;--diff-hunk-bg:#141924;--diff-gutter:#46516a;--diff-mark:#2b3f63;--diff-mark-edge:#6ea8fe;--diff-add-strong:#1f6b3c;--diff-del-strong:#6d2530;--diff-blank-bg:repeating-linear-gradient(135deg, #11151d 0 6px, #151a24 6px 12px);--fix-bg:#0e1a13;--fix-border:#2f6b46;--fix-text:#7ee2a8;--publish-bg:#0e1622;--publish-border:#3a5a8f;--publish-text:#7db8ff;--scrollbar-thumb:#2a3348;--scrollbar-thumb-hover:#3a4660;--selection-bg:#33507a;--header-bg:#0b0e14eb}:root[data-theme=light]{--bg:#fff;--bg-elev:#f6f7f9;--bg-card:#eef0f3;--bg-card-hover:#e4e7ec;--border:#d7dbe2;--border-soft:#e4e7ec;--text:#16181d;--text-dim:#4b5563;--text-faint:#6b7280;--accent:#2f6fe0;--accent-soft:#dce8ff;--ok:#1f9d55;--ok-bg:#e3f8ea;--warn:#a5720a;--danger:#c92a2a;--danger-bg:#fbe4e4;--danger-border:#f3b4b4;--btn-primary-bg:#2f6fe0;--btn-primary-border:#2358b8;--btn-primary-text:#fff;--kind-feat:#1f9d55;--kind-fix:#c92a2a;--kind-refactor:#7c4dd6;--kind-perf:#a5720a;--kind-test:#2f6fe0;--kind-chore:#5b6472;--kind-docs:#0f8fa3;--kind-infra:#c1690f;--sev-high:#c92a2a;--sev-med:#c44d1a;--sev-low:#8b7d2e;--sev-nit:#6b7280;--diff-add-bg:#e6f7ec;--diff-add-fg:#1f9d55;--diff-del-bg:#fbe9e9;--diff-del-fg:#c92a2a;--diff-hunk-bg:#eef1f5;--diff-gutter:#9aa3b2;--diff-mark:#cfe0ff;--diff-mark-edge:#2f6fe0;--diff-add-strong:#a7ebbd;--diff-del-strong:#f9b8b8;--diff-blank-bg:repeating-linear-gradient(135deg, #f6f7f9 0 6px, #eef0f3 6px 12px);--fix-bg:#e6f7ec;--fix-border:#1f9d55;--fix-text:#157a41;--publish-bg:#e8f0fe;--publish-border:#2f6fe0;--publish-text:#1d4fa8;--scrollbar-thumb:#c7cbd4;--scrollbar-thumb-hover:#aab0bb;--selection-bg:#cfe0ff;--header-bg:#ffffffeb}.code .chg{border-radius:2px}.code.del .chg{background:var(--diff-del-strong)}.code.add .chg{background:var(--diff-add-strong)}.hljs-comment,.hljs-quote{color:#6b7689;font-style:italic}.hljs-keyword,.hljs-selector-tag,.hljs-literal,.hljs-doctag,.hljs-formula{color:#c9a5ff}.hljs-string,.hljs-regexp,.hljs-addition,.hljs-attribute,.hljs-meta .hljs-string{color:#a5d6a7}.hljs-number,.hljs-variable,.hljs-template-variable,.hljs-type,.hljs-selector-class,.hljs-selector-attr,.hljs-selector-pseudo{color:#f0a868}.hljs-title,.hljs-section,.hljs-title.class_,.hljs-title.function_{color:#7fd6e0}.hljs-attr,.hljs-property,.hljs-symbol,.hljs-bullet,.hljs-link{color:#6ea8fe}.hljs-built_in,.hljs-class,.hljs-params{color:#e6e9ef}.hljs-meta,.hljs-tag{color:#9aa4b8}:root[data-theme=light] .hljs-comment,:root[data-theme=light] .hljs-quote{color:#6b7280}:root[data-theme=light] .hljs-keyword,:root[data-theme=light] .hljs-selector-tag,:root[data-theme=light] .hljs-literal,:root[data-theme=light] .hljs-doctag,:root[data-theme=light] .hljs-formula{color:#7c4dd6}:root[data-theme=light] .hljs-string,:root[data-theme=light] .hljs-regexp,:root[data-theme=light] .hljs-addition,:root[data-theme=light] .hljs-attribute,:root[data-theme=light] .hljs-meta .hljs-string{color:#1f9d55}:root[data-theme=light] .hljs-number,:root[data-theme=light] .hljs-variable,:root[data-theme=light] .hljs-template-variable,:root[data-theme=light] .hljs-type,:root[data-theme=light] .hljs-selector-class,:root[data-theme=light] .hljs-selector-attr,:root[data-theme=light] .hljs-selector-pseudo{color:#b5650f}:root[data-theme=light] .hljs-title,:root[data-theme=light] .hljs-section,:root[data-theme=light] .hljs-title.class_,:root[data-theme=light] .hljs-title.function_{color:#0f8fa3}:root[data-theme=light] .hljs-attr,:root[data-theme=light] .hljs-property,:root[data-theme=light] .hljs-symbol,:root[data-theme=light] .hljs-bullet,:root[data-theme=light] .hljs-link{color:#2f6fe0}:root[data-theme=light] .hljs-built_in,:root[data-theme=light] .hljs-class,:root[data-theme=light] .hljs-params{color:#16181d}:root[data-theme=light] .hljs-meta,:root[data-theme=light] .hljs-tag{color:#5b6472}.hljs-emphasis{font-style:italic}.hljs-strong{font-weight:700}.anchor-flash{animation:1.4s ease-out anchor-flash}@keyframes anchor-flash{0%{box-shadow:0 0 0 2px var(--accent)}to{box-shadow:0 0 0 2px #0000}}*{box-sizing:border-box}html,body{background:var(--bg);color:var(--text);font-family:var(--sans);min-height:100%;margin:0;padding:0}body{-webkit-font-smoothing:antialiased}button,input,select{font:inherit;color:inherit}button{cursor:pointer}::selection{background:var(--selection-bg)}html,body,*{scrollbar-width:thin;scrollbar-color:var(--scrollbar-thumb) transparent}::-webkit-scrollbar{width:8px;height:8px}::-webkit-scrollbar-track{background:0 0}::-webkit-scrollbar-thumb{background:var(--scrollbar-thumb);background-clip:padding-box;border:2px solid #0000;border-radius:999px}::-webkit-scrollbar-thumb:hover{background:var(--scrollbar-thumb-hover);background-clip:padding-box;border:2px solid #0000}::-webkit-scrollbar-corner{background:0 0}.boot.svelte-12qhfyh{color:var(--text-dim);font-family:var(--mono);padding:48px 28px;font-size:13px}.boot.svelte-12qhfyh button:where(.svelte-12qhfyh){background:var(--btn-primary-bg);border:1px solid var(--btn-primary-border);color:var(--btn-primary-text);border-radius:8px;margin-top:12px;padding:8px 14px}
+:root{--bg:#0b0e14;--bg-elev:#11151d;--bg-card:#151a24;--bg-card-hover:#1a2130;--border:#242b3a;--border-soft:#1c2230;--text:#e6e9ef;--text-dim:#9aa4b8;--text-faint:#5f6b82;--accent:#6ea8fe;--accent-soft:#1c2b45;--ok:#7ee2a8;--ok-bg:#0d2a1c;--warn:#e2c67e;--danger:#ff9a9a;--danger-bg:#2a1414;--danger-border:#4a1e1e;--mono:ui-monospace, "JetBrains Mono", "Fira Code", Consolas, "SFMono-Regular", Menlo, monospace;--sans:-apple-system, "Segoe UI", Inter, Roboto, Helvetica, Arial, sans-serif;--btn-primary-bg:#1c2b45;--btn-primary-border:#3a5a8f;--btn-primary-text:#cfe0ff;--kind-feat:#7ee2a8;--kind-fix:#ff9a9a;--kind-refactor:#c9a5ff;--kind-perf:#e2c67e;--kind-test:#6ea8fe;--kind-chore:#9aa4b8;--kind-docs:#7fd6e0;--kind-infra:#f0a868;--sev-high:#ff9a9a;--sev-med:#f08a5a;--sev-low:#c4b86a;--sev-nit:#7f8aa3;--diff-add-bg:#0f2419;--diff-add-fg:#7ee2a8;--diff-del-bg:#2a1418;--diff-del-fg:#ff9a9a;--diff-hunk-bg:#141924;--diff-gutter:#46516a;--diff-mark:#2b3f63;--diff-mark-edge:#6ea8fe;--diff-add-strong:#1f6b3c;--diff-del-strong:#6d2530;--diff-blank-bg:repeating-linear-gradient(135deg, #11151d 0 6px, #151a24 6px 12px);--fix-bg:#0e1a13;--fix-border:#2f6b46;--fix-text:#7ee2a8;--publish-bg:#0e1622;--publish-border:#3a5a8f;--publish-text:#7db8ff;--scrollbar-thumb:#2a3348;--scrollbar-thumb-hover:#3a4660;--selection-bg:#33507a;--header-bg:#0b0e14eb}:root[data-theme=light]{--bg:#fff;--bg-elev:#f6f7f9;--bg-card:#eef0f3;--bg-card-hover:#e4e7ec;--border:#d7dbe2;--border-soft:#e4e7ec;--text:#16181d;--text-dim:#4b5563;--text-faint:#6b7280;--accent:#2f6fe0;--accent-soft:#dce8ff;--ok:#1f9d55;--ok-bg:#e3f8ea;--warn:#a5720a;--danger:#c92a2a;--danger-bg:#fbe4e4;--danger-border:#f3b4b4;--btn-primary-bg:#2f6fe0;--btn-primary-border:#2358b8;--btn-primary-text:#fff;--kind-feat:#1f9d55;--kind-fix:#c92a2a;--kind-refactor:#7c4dd6;--kind-perf:#a5720a;--kind-test:#2f6fe0;--kind-chore:#5b6472;--kind-docs:#0f8fa3;--kind-infra:#c1690f;--sev-high:#c92a2a;--sev-med:#c44d1a;--sev-low:#8b7d2e;--sev-nit:#6b7280;--diff-add-bg:#e6f7ec;--diff-add-fg:#1f9d55;--diff-del-bg:#fbe9e9;--diff-del-fg:#c92a2a;--diff-hunk-bg:#eef1f5;--diff-gutter:#9aa3b2;--diff-mark:#cfe0ff;--diff-mark-edge:#2f6fe0;--diff-add-strong:#a7ebbd;--diff-del-strong:#f9b8b8;--diff-blank-bg:repeating-linear-gradient(135deg, #f6f7f9 0 6px, #eef0f3 6px 12px);--fix-bg:#e6f7ec;--fix-border:#1f9d55;--fix-text:#157a41;--publish-bg:#e8f0fe;--publish-border:#2f6fe0;--publish-text:#1d4fa8;--scrollbar-thumb:#c7cbd4;--scrollbar-thumb-hover:#aab0bb;--selection-bg:#cfe0ff;--header-bg:#ffffffeb}.code .chg{border-radius:2px}.code.del .chg{background:var(--diff-del-strong)}.code.add .chg{background:var(--diff-add-strong)}.hljs-comment,.hljs-quote{color:#6b7689;font-style:italic}.hljs-keyword,.hljs-selector-tag,.hljs-literal,.hljs-doctag,.hljs-formula{color:#c9a5ff}.hljs-string,.hljs-regexp,.hljs-addition,.hljs-attribute,.hljs-meta .hljs-string{color:#a5d6a7}.hljs-number,.hljs-variable,.hljs-template-variable,.hljs-type,.hljs-selector-class,.hljs-selector-attr,.hljs-selector-pseudo{color:#f0a868}.hljs-title,.hljs-section,.hljs-title.class_,.hljs-title.function_{color:#7fd6e0}.hljs-attr,.hljs-property,.hljs-symbol,.hljs-bullet,.hljs-link{color:#6ea8fe}.hljs-built_in,.hljs-class,.hljs-params{color:#e6e9ef}.hljs-meta,.hljs-tag{color:#9aa4b8}:root[data-theme=light] .hljs-comment,:root[data-theme=light] .hljs-quote{color:#6b7280}:root[data-theme=light] .hljs-keyword,:root[data-theme=light] .hljs-selector-tag,:root[data-theme=light] .hljs-literal,:root[data-theme=light] .hljs-doctag,:root[data-theme=light] .hljs-formula{color:#7c4dd6}:root[data-theme=light] .hljs-string,:root[data-theme=light] .hljs-regexp,:root[data-theme=light] .hljs-addition,:root[data-theme=light] .hljs-attribute,:root[data-theme=light] .hljs-meta .hljs-string{color:#1f9d55}:root[data-theme=light] .hljs-number,:root[data-theme=light] .hljs-variable,:root[data-theme=light] .hljs-template-variable,:root[data-theme=light] .hljs-type,:root[data-theme=light] .hljs-selector-class,:root[data-theme=light] .hljs-selector-attr,:root[data-theme=light] .hljs-selector-pseudo{color:#b5650f}:root[data-theme=light] .hljs-title,:root[data-theme=light] .hljs-section,:root[data-theme=light] .hljs-title.class_,:root[data-theme=light] .hljs-title.function_{color:#0f8fa3}:root[data-theme=light] .hljs-attr,:root[data-theme=light] .hljs-property,:root[data-theme=light] .hljs-symbol,:root[data-theme=light] .hljs-bullet,:root[data-theme=light] .hljs-link{color:#2f6fe0}:root[data-theme=light] .hljs-built_in,:root[data-theme=light] .hljs-class,:root[data-theme=light] .hljs-params{color:#16181d}:root[data-theme=light] .hljs-meta,:root[data-theme=light] .hljs-tag{color:#5b6472}.hljs-emphasis{font-style:italic}.hljs-strong{font-weight:700}.anchor-flash{animation:1.4s ease-out anchor-flash}@keyframes anchor-flash{0%{box-shadow:0 0 0 2px var(--accent)}to{box-shadow:0 0 0 2px #0000}}*{box-sizing:border-box}html,body{background:var(--bg);color:var(--text);font-family:var(--sans);min-height:100%;margin:0;padding:0}body{-webkit-font-smoothing:antialiased}button,input,select{font:inherit;color:inherit}button{cursor:pointer}::selection{background:var(--selection-bg)}html,body,*{scrollbar-width:thin;scrollbar-color:var(--scrollbar-thumb) transparent}::-webkit-scrollbar{width:8px;height:8px}::-webkit-scrollbar-track{background:0 0}::-webkit-scrollbar-thumb{background:var(--scrollbar-thumb);background-clip:padding-box;border:2px solid #0000;border-radius:999px}::-webkit-scrollbar-thumb:hover{background:var(--scrollbar-thumb-hover);background-clip:padding-box;border:2px solid #0000}::-webkit-scrollbar-corner{background:0 0}.boot.svelte-12qhfyh{color:var(--text-dim);font-family:var(--mono);padding:48px 28px;font-size:13px}.boot.svelte-12qhfyh button:where(.svelte-12qhfyh){background:var(--btn-primary-bg);border:1px solid var(--btn-primary-border);color:var(--btn-primary-text);border-radius:0;margin-top:12px;padding:8px 14px}