critique 0.1.134 → 0.1.135

package/dist/balance-delimiters.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"balance-delimiters.d.ts","sourceRoot":"","sources":["../src/balance-delimiters.ts"],"names":[],"mappings":"AA4DA;;;;;;GAMG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,MAAM,CActE;AAmPD;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,CAAC,EAAE,MAAM,GAAG,MAAM,CAqE5E"}
+{"version":3,"file":"balance-delimiters.d.ts","sourceRoot":"","sources":["../src/balance-delimiters.ts"],"names":[],"mappings":"AAmEA;;;;;;GAMG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,MAAM,CActE;AAwbD;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,CAAC,EAAE,MAAM,GAAG,MAAM,CA4E5E"}

package/dist/balance-delimiters.js

@@ -32,7 +32,7 @@ const htmlCommentRule = {
 const LANGUAGE_DELIMITERS = {
     typescript: [{ token: "`" }, cStyleBlockCommentRule],
     python: [{ token: '"""' }, { token: "'''" }],
-    markdown: [{ token: "```" }],
+    markdown: [{ token: "```", classifyFence: classifyMarkdownFence }],
     go: [{ token: "`" }, cStyleBlockCommentRule],
     rust: [cStyleBlockCommentRule],
     cpp: [cStyleBlockCommentRule],
@@ -201,6 +201,148 @@ function escapeDelimiterAt(lines, hunkLineIndex, column) {
         return prefix + content.slice(0, column) + "\\" + content.slice(column);
     });
 }
+// ---------------------------------------------------------------------------
+// Contextual fence classification and repair (markdown code fences)
+// ---------------------------------------------------------------------------
+/**
+ * Classify a ``` occurrence as a markdown fence opener, closer, or not a fence.
+ *
+ * Returns null if the occurrence is not a valid block-level fence (e.g. inline
+ * triple-backtick in prose, or indented more than 3 spaces).
+ * Returns "open" if followed by an info string (language tag).
+ * Returns "close" if nothing follows the backtick run (only whitespace).
+ */
+function classifyMarkdownFence(content, column) {
+    // Must be at start of line with at most 3 spaces of indentation
+    const beforeFence = content.slice(0, column);
+    if (beforeFence.length > 3 || /\S/.test(beforeFence))
+        return null;
+    // Count consecutive backticks (support 4+ backtick fences per CommonMark)
+    let fenceLen = 0;
+    for (let i = column; i < content.length && content[i] === "`"; i++)
+        fenceLen++;
+    if (fenceLen < 3)
+        return null;
+    // What comes after the backtick run?
+    const afterFence = content.slice(column + fenceLen).trim();
+    // Closing fence: nothing after backticks (only whitespace)
+    if (!afterFence)
+        return "close";
+    // Opening fence: has info string (language tag)
+    // Info string must not contain backticks (CommonMark spec)
+    if (!afterFence.includes("`"))
+        return "open";
+    return null;
+}
+/**
+ * Simulate a sequential walk through classified fences to detect boundary
+ * artifacts. Markdown code fences don't nest, so depth toggles between 0
+ * (outside) and 1 (inside).
+ *
+ * - "open" (has info string) always pushes depth to 1.
+ * - "close" (bare fence) decrements if inside, or acts as a bare opener
+ *   if already outside (a code block without a language tag).
+ *
+ * Conflicts are counted when a must-open fires while already inside (depth
+ * was already 1) or when other impossible transitions occur.
+ */
+function walkFences(fences, startDepth) {
+    let depth = startDepth;
+    let conflicts = 0;
+    for (const fence of fences) {
+        if (fence.kind === "open") {
+            if (depth > 0)
+                conflicts++;
+            depth = 1;
+        }
+        else {
+            // "close" (bare fence): close if inside, otherwise bare opener
+            if (depth > 0) {
+                depth = 0;
+            }
+            else {
+                depth = 1;
+            }
+        }
+    }
+    return { startDepth, endDepth: depth, conflicts };
+}
+/**
+ * Repair context-dependent fences (like markdown ```) using sequential
+ * open/close pairing instead of simple odd/even counting.
+ *
+ * Tries two starting states (outside vs inside a code block), picks the
+ * walk with fewer conflicts, and adds synthetic fence tokens inline:
+ * - If the hunk starts inside a block: prepend ``` to the first content
+ *   line so the boundary closer has something to close.
+ * - If the hunk ends inside a block: append ``` to the last content
+ *   line so the boundary opener is properly closed.
+ * Tokens are added inline (no new lines) to preserve patch header counts.
+ */
+function repairContextualFences(contentLines, lines, rule) {
+    if (!rule.classifyFence)
+        return [...lines];
+    const occurrences = findDelimiterOccurrences(contentLines, rule.token);
+    if (occurrences.length === 0)
+        return [...lines];
+    // Classify each occurrence as fence open, close, or not-a-fence
+    const fences = [];
+    for (const occ of occurrences) {
+        const content = contentLines[occ.contentLineIndex]?.content;
+        if (!content)
+            continue;
+        const kind = rule.classifyFence(content, occ.column);
+        if (kind) {
+            fences.push({ occurrence: occ, kind });
+        }
+    }
+    if (fences.length === 0)
+        return [...lines];
+    // Try both starting states
+    const walk0 = walkFences(fences, 0);
+    const walk1 = walkFences(fences, 1);
+    // Pick better walk: fewer conflicts → fewer repairs → content heuristic
+    let chosen;
+    if (walk0.conflicts < walk1.conflicts) {
+        chosen = walk0;
+    }
+    else if (walk1.conflicts < walk0.conflicts) {
+        chosen = walk1;
+    }
+    else {
+        const repairs0 = (walk0.startDepth > 0 ? 1 : 0) + (walk0.endDepth > 0 ? 1 : 0);
+        const repairs1 = (walk1.startDepth > 0 ? 1 : 0) + (walk1.endDepth > 0 ? 1 : 0);
+        if (repairs0 < repairs1) {
+            chosen = walk0;
+        }
+        else if (repairs1 < repairs0) {
+            chosen = walk1;
+        }
+        else {
+            // Still tied: disambiguate by content position relative to the first fence.
+            // If there's non-empty content before the first fence in the hunk, the fence
+            // is likely closing a block from before the hunk → prefer depth=1 (starts inside).
+            // This produces better tree-sitter pairing: the prepended ``` + original ```
+            // form a matched pair, while appended inline ``` doesn't close a fence.
+            const firstIdx = fences[0]?.occurrence.contentLineIndex ?? 0;
+            const hasContentBefore = contentLines.slice(0, firstIdx).some((line) => line.content.trim() !== "");
+            chosen = hasContentBefore ? walk1 : walk0;
+        }
+    }
+    let result = [...lines];
+    // Append synthetic closer to end of last content line (inline, no new lines)
+    if (chosen.endDepth > 0) {
+        result = appendClosingTokensToLastContentLine(result, rule.token, 1);
+    }
+    // Prepend synthetic opener to start of first content line (inline, no new lines).
+    // Pass the first fence's hunk line index so the search only looks at lines
+    // before the boundary closer — the opener must precede it to form a pair.
+    if (chosen.startDepth > 0) {
+        const firstFenceHunkLine = fences[0]?.occurrence.hunkLineIndex;
+        result = prependOpeningTokenToFirstContentLine(result, rule.token, firstFenceHunkLine);
+    }
+    return result;
+}
 function getRuleOpenTokens(rule) {
     return rule.openTokens ?? [rule.token];
 }
@@ -239,6 +381,35 @@ function appendClosingTokensToLastContentLine(lines, closeToken, count) {
         return `${line} ${closingSuffix}`;
     });
 }
+function prependOpeningTokenToFirstContentLine(lines, openToken, beforeHunkLineIndex) {
+    // Prefer a blank/whitespace content line to avoid creating a fake info string
+    // (e.g. "```   handler() {" makes tree-sitter think "handler()" is a language).
+    // Only search among lines BEFORE the first fence so the synthetic opener
+    // appears before the boundary closer (they need to pair).
+    const firstContentLineIndex = lines.findIndex(isDiffContentLine);
+    if (firstContentLineIndex === -1) {
+        return [...lines];
+    }
+    const searchEnd = beforeHunkLineIndex ?? lines.length;
+    let targetIndex = firstContentLineIndex;
+    for (let i = firstContentLineIndex; i < searchEnd; i++) {
+        const line = lines[i];
+        if (!isDiffContentLine(line))
+            continue;
+        if (line.slice(1).trim() === "") {
+            targetIndex = i;
+            break;
+        }
+    }
+    return lines.map((line, index) => {
+        if (index !== targetIndex || !isDiffContentLine(line)) {
+            return line;
+        }
+        const prefix = line[0] ?? "";
+        const content = line.slice(1);
+        return `${prefix}${openToken} ${content}`;
+    });
+}
 /**
  * Balance paired delimiters in a unified diff patch for correct syntax
  * highlighting.
@@ -284,6 +455,12 @@ export function balanceDelimiters(rawDiff, filetype) {
             if (!isSymmetricRule(rule)) {
                 continue;
             }
+            // Contextual fence pairing (markdown code fences): uses sequential
+            // open/close tracking instead of simple odd/even parity.
+            if (rule.classifyFence) {
+                repairedLines = repairContextualFences(contentLines, repairedLines, rule);
+                continue;
+            }
             const occurrences = findDelimiterOccurrences(contentLines, rule.token);
             if (occurrences.length % 2 === 0) {
                 continue;

package/dist/balance-delimiters.test.js

@@ -439,7 +439,7 @@ describe("balanceDelimiters", () => {
             ]);
             expect(balanceDelimiters(patch, "markdown")).toBe(patch);
         });
-        it("escapes a leading closing code fence when count is odd", () => {
+        it("prepends synthetic opener when hunk starts inside a code block", () => {
             const patch = mdPatch([
                 " inside fenced block",
                 " ```",
@@ -448,8 +448,9 @@ describe("balanceDelimiters", () => {
             ]);
             const result = balanceDelimiters(patch, "markdown");
             const lines = result.split("\n");
-            expect(lines[3]).toBe(" inside fenced block");
-            expect(lines[4]).toBe(" \\```");
+            // Synthetic ``` opener prepended inline to first content line
+            expect(lines[3]).toBe(" ``` inside fenced block");
+            expect(lines[4]).toBe(" ```");
         });
         it("does not modify when only inline code backticks are present", () => {
             const patch = mdPatch([
@@ -459,7 +460,7 @@ describe("balanceDelimiters", () => {
             ]);
             expect(balanceDelimiters(patch, "markdown")).toBe(patch);
         });
-        it("escapes a trailing unmatched fence opener instead of duplicating it", () => {
+        it("appends synthetic closer when hunk ends with unclosed opener", () => {
             const patch = mdPatch([
                 " ```ts",
                 " const x = 1",
@@ -468,8 +469,319 @@ describe("balanceDelimiters", () => {
             ]);
             const result = balanceDelimiters(patch, "markdown");
             const lines = result.split("\n");
-            expect(lines[3]).toBe(" \\```ts");
+            // Original content untouched
+            expect(lines[3]).toBe(" ```ts");
             expect(lines[4]).toBe(" const x = 1");
+            // Synthetic ``` closer appended inline to last content line
+            expect(lines[lines.length - 1]).toBe("+new line ```");
+        });
+        it("adds synthetic fences at both boundaries when even count but first=closer last=opener (6 tokens)", () => {
+            const patch = mdPatch([
+                " ```",
+                " ",
+                " ## Section",
+                " ",
+                " ```ts",
+                " const a = 1",
+                " ```",
+                " ",
+                " ```ts",
+                " const b = 2",
+                " ```",
+                " ",
+                "+```ts",
+            ]);
+            const result = balanceDelimiters(patch, "markdown");
+            const lines = result.split("\n");
+            // Synthetic opener prepended inline to first content line
+            expect(lines[3]).toBe(" ``` ```");
+            // Synthetic closer appended inline to last content line
+            expect(lines[lines.length - 1]).toBe("+```ts ```");
+            // middle fences stay untouched
+            expect(lines[7]).toBe(" ```ts");
+            expect(lines[9]).toBe(" ```");
+            expect(lines[11]).toBe(" ```ts");
+            expect(lines[13]).toBe(" ```");
+        });
+        it("adds synthetic fences at both boundaries with 4 tokens (bare, ```ts, bare, ```ts)", () => {
+            const patch = mdPatch([
+                " inside code block",
+                " ```",
+                " ",
+                " ```ts",
+                " const x = 1",
+                " ```",
+                " ",
+                "+```ts",
+            ]);
+            const result = balanceDelimiters(patch, "markdown");
+            const lines = result.split("\n");
+            // Synthetic opener prepended inline to first content line
+            expect(lines[3]).toBe(" ``` inside code block");
+            // Synthetic closer appended inline to last content line
+            expect(lines[lines.length - 1]).toBe("+```ts ```");
+        });
+        it("returns patch unchanged when 4 tokens are fully balanced (```ts, bare, ```ts, bare)", () => {
+            const patch = mdPatch([
+                " ```ts",
+                " const a = 1",
+                " ```",
+                " ",
+                " ```ts",
+                " const b = 2",
+                " ```",
+                "+New paragraph",
+            ]);
+            expect(balanceDelimiters(patch, "markdown")).toBe(patch);
+        });
+        it("adds synthetic fences at both boundaries when 2 tokens are bare-closer then opener", () => {
+            const patch = mdPatch([
+                " inside block",
+                " ```",
+                " ",
+                "+```ts",
+            ]);
+            const result = balanceDelimiters(patch, "markdown");
+            const lines = result.split("\n");
+            // Synthetic opener prepended inline to first content line
+            expect(lines[3]).toBe(" ``` inside block");
+            // Synthetic closer appended inline to last content line
+            expect(lines[lines.length - 1]).toBe("+```ts ```");
+        });
+        it("returns patch unchanged for 2 balanced tokens (```ts then bare)", () => {
+            const patch = mdPatch([
+                " ```ts",
+                " const x = 1",
+                " ```",
+                "+New paragraph",
+            ]);
+            expect(balanceDelimiters(patch, "markdown")).toBe(patch);
+        });
+        it("returns patch unchanged for two bare fences (open + close, no language)", () => {
+            const patch = mdPatch([
+                " ```",
+                " some code",
+                " ```",
+                "+New paragraph",
+            ]);
+            expect(balanceDelimiters(patch, "markdown")).toBe(patch);
+        });
+        it("ignores inline triple backticks in prose (not at start of line)", () => {
+            const patch = mdPatch([
+                " Use the ``` delimiter for code fences",
+                "-old",
+                "+new",
+            ]);
+            expect(balanceDelimiters(patch, "markdown")).toBe(patch);
+        });
+        it("treats fences with up to 3 spaces indent as valid", () => {
+            const patch = mdPatch([
+                "    ```ts",
+                "    const x = 1",
+                "    ```",
+                "+New paragraph",
+            ]);
+            // 3 spaces + ``` = column 3, indent 3 = valid fence
+            expect(balanceDelimiters(patch, "markdown")).toBe(patch);
+        });
+        it("ignores fences indented more than 3 spaces (code indentation)", () => {
+            const patch = mdPatch([
+                "     ```ts",
+                "     const x = 1",
+                "-old",
+                "+new",
+            ]);
+            // 4+ spaces = not a fence, treated as code content → no escaping
+            expect(balanceDelimiters(patch, "markdown")).toBe(patch);
+        });
+        it("handles 4-backtick fence pair correctly", () => {
+            const patch = mdPatch([
+                " ````ts",
+                " const x = 1",
+                " ````",
+                "+New paragraph",
+            ]);
+            expect(balanceDelimiters(patch, "markdown")).toBe(patch);
+        });
+        it("recognizes closing fence with trailing spaces", () => {
+            const patch = mdPatch([
+                " ```ts",
+                " const x = 1",
+                " ```   ",
+                "+New paragraph",
+            ]);
+            expect(balanceDelimiters(patch, "markdown")).toBe(patch);
+        });
+        it("single bare fence with content on both sides treats as opener (appends closer)", () => {
+            // Ambiguous: bare ``` with content before AND after.
+            // Tie-break prefers depth=1 (prepend opener) since content exists before fence.
+            const patch = mdPatch([
+                " Intro paragraph",
+                " ```",
+                " code line",
+                "+new line",
+            ]);
+            const result = balanceDelimiters(patch, "markdown");
+            const lines = result.split("\n");
+            // Prepend opener on first content line (content before fence)
+            expect(lines[3]).toBe(" ``` Intro paragraph");
+            expect(lines[4]).toBe(" ```");
+        });
+        it("single bare fence with content only after treats as opener (appends closer)", () => {
+            const patch = mdPatch([
+                " ```",
+                " code line",
+                "-old",
+                "+new",
+            ]);
+            const result = balanceDelimiters(patch, "markdown");
+            const lines = result.split("\n");
+            // No content before fence → depth=0 → append closer
+            expect(lines[3]).toBe(" ```");
+            expect(lines[lines.length - 1]).toBe("+new ```");
+        });
+        it("prefers blank line for synthetic opener to avoid fake info string", () => {
+            const patch = mdPatch([
+                " ",
+                " inside code block",
+                " ```",
+                "-old line",
+                "+new line",
+            ]);
+            const result = balanceDelimiters(patch, "markdown");
+            const lines = result.split("\n");
+            // Blank line before fence is used for synthetic opener (no fake info string)
+            expect(lines[3]).toBe(" ``` ");
+            expect(lines[4]).toBe(" inside code block");
+            expect(lines[5]).toBe(" ```");
+        });
+        it("handles two hunks independently for markdown fences", () => {
+            const patch = [
+                "--- file.md",
+                "+++ file.md",
+                "@@ -5,4 +5,4 @@",
+                " ```ts",
+                " const x = 1",
+                " ```",
+                "+New paragraph",
+                "@@ -20,4 +20,4 @@",
+                " inside block",
+                " ```",
+                "-old line",
+                "+new line",
+            ].join("\n");
+            const result = balanceDelimiters(patch, "markdown");
+            const lines = result.split("\n");
+            // First hunk: balanced, no changes
+            expect(lines[3]).toBe(" ```ts");
+            expect(lines[5]).toBe(" ```");
+            // Second hunk: bare closer at boundary → synthetic opener prepended inline
+            const secondHunkIdx = lines.findIndex((l, i) => i > 2 && l.startsWith("@@"));
+            expect(lines[secondHunkIdx + 1]).toBe(" ``` inside block");
+            expect(lines[secondHunkIdx + 2]).toBe(" ```");
+        });
+        it("handles real README.md hunk with boundary fences (from critique.work patch)", () => {
+            // Real hunk from https://critique.work/v/daa808658ee537a745b80101ba3195ae.patch
+            // First hunk: starts inside a code block (bare ``` closer), ends with ```ts opener
+            const patch = [
+                "--- README.md",
+                "+++ README.md",
+                "@@ -741,12 +741,14 @@",
+                "     }),",
+                "   )",
+                " ```",
+                " ",
+                " ## Base Path",
+                " ",
+                "+For standalone API servers (without Vite), set the base path in the constructor:",
+                "+",
+                " ```ts",
+                " import { Spiceflow } from 'spiceflow'",
+                " ",
+                " const app = new Spiceflow({ basePath: '/api/v1' })",
+                " app.route({",
+                "   method: 'GET',",
+                "@@ -754,12 +756,47 @@",
+                "   handler() {",
+                "     return 'Hello'",
+                "   },",
+                " }) // Accessible at /api/v1/hello",
+                " ```",
+                " ",
+                "+### Base Path with Vite (RSC apps)",
+                "+",
+                "+When using Spiceflow as a full-stack RSC framework with Vite, configure the base path via Vite's `base` option instead of the constructor:",
+                "+",
+                "+```ts",
+                "+// vite.config.ts",
+                "+import { defineConfig } from 'vite'",
+                "+import { spiceflowPlugin } from 'spiceflow/vite'",
+                "+",
+                "+export default defineConfig({",
+                "+  base: '/my-app',",
+                "+  plugins: [spiceflowPlugin({ entry: 'src/main.tsx' })],",
+                "+})",
+                "+```",
+                "+",
+                "+The base path must be an absolute path starting with `/`. CDN URLs and relative paths are not supported.",
+                "+",
+                " ## Async Generators (Streaming)",
+                " ",
+                " Async generators will create a server sent event response.",
+                " ",
+                " ```ts",
+                " // server.ts",
+            ].join("\n");
+            const result = balanceDelimiters(patch, "markdown");
+            expect(result).toMatchInlineSnapshot(`
+        "--- README.md
+        +++ README.md
+        @@ -741,12 +741,14 @@
+         \`\`\`     }),
+           )
+         \`\`\`
+         
+         ## Base Path
+         
+        +For standalone API servers (without Vite), set the base path in the constructor:
+        +
+         \`\`\`ts
+         import { Spiceflow } from 'spiceflow'
+         
+         const app = new Spiceflow({ basePath: '/api/v1' })
+         app.route({
+           method: 'GET', \`\`\`
+        @@ -754,12 +756,47 @@
+         \`\`\`   handler() {
+             return 'Hello'
+           },
+         }) // Accessible at /api/v1/hello
+         \`\`\`
+         
+        +### Base Path with Vite (RSC apps)
+        +
+        +When using Spiceflow as a full-stack RSC framework with Vite, configure the base path via Vite's \`base\` option instead of the constructor:
+        +
+        +\`\`\`ts
+        +// vite.config.ts
+        +import { defineConfig } from 'vite'
+        +import { spiceflowPlugin } from 'spiceflow/vite'
+        +
+        +export default defineConfig({
+        +  base: '/my-app',
+        +  plugins: [spiceflowPlugin({ entry: 'src/main.tsx' })],
+        +})
+        +\`\`\`
+        +
+        +The base path must be an absolute path starting with \`/\`. CDN URLs and relative paths are not supported.
+        +
+         ## Async Generators (Streaming)
+         
+         Async generators will create a server sent event response.
+         
+         \`\`\`ts
+         // server.ts \`\`\`"
+      `);
         });
     });
     describe("scala", () => {

package/dist/cli.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.tsx"],"names":[],"mappings":";AAOA,OAAO,gCAAgC,CAAC;AAYxC,OAAO,KAAK,KAAK,MAAM,OAAO,CAAC;AAwB/B,OAAO,EAgBL,KAAK,UAAU,EAEhB,MAAM,iBAAiB,CAAC;AA45CzB,MAAM,WAAW,QAAQ;IACvB,WAAW,EAAE,UAAU,EAAE,CAAC;CAC3B;AAED,wBAAgB,GAAG,CAAC,EAAE,WAAW,EAAE,EAAE,QAAQ,GAAG,KAAK,CAAC,YAAY,CAkVjE"}
+{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../src/cli.tsx"],"names":[],"mappings":";AAOA,OAAO,gCAAgC,CAAC;AAYxC,OAAO,KAAK,KAAK,MAAM,OAAO,CAAC;AAwB/B,OAAO,EAiBL,KAAK,UAAU,EAEhB,MAAM,iBAAiB,CAAC;AA45CzB,MAAM,WAAW,QAAQ;IACvB,WAAW,EAAE,UAAU,EAAE,CAAC;CAC3B;AAED,wBAAgB,GAAG,CAAC,EAAE,WAAW,EAAE,EAAE,QAAQ,GAAG,KAAK,CAAC,YAAY,CAkVjE"}

package/dist/cli.js

@@ -26,7 +26,7 @@ import { debounce } from "./utils.js";
 import { DiffView, DirectoryTreeView } from "./components/index.js";
 import { logger } from "./logger.js";
 import { saveStoredLicenseKey } from "./license.js";
-import { buildGitCommand, filterParsedFilesByPatterns, getFileName, getFileStatus, getOldFileName, countChanges, getViewMode, processFiles, detectFiletype, stripSubmoduleHeaders, parseGitDiffFiles, getDirtySubmodulePaths, buildSubmoduleDiffCommand, getFilterPatterns, IGNORED_FILES, } from "./diff-utils.js";
+import { buildGitCommand, ensureGitRepo, filterParsedFilesByPatterns, getFileName, getFileStatus, getOldFileName, countChanges, getViewMode, processFiles, detectFiletype, stripSubmoduleHeaders, parseGitDiffFiles, getDirtySubmodulePaths, buildSubmoduleDiffCommand, getFilterPatterns, IGNORED_FILES, } from "./diff-utils.js";
 import packageJson from "../package.json" assert { type: "json" };
 // Lazy-load watcher only when --watch is used
 let watcherModule = null;
@@ -1364,6 +1364,7 @@ cli
     description: "Filter files by glob pattern (can be used multiple times)",
 }))
     .action(async (options) => {
+    ensureGitRepo();
     const { parseHunksWithIds, hunkToStableId, } = await import("./review/index.js");
     // Build git command - unstaged by default, staged with --staged
     const gitCommand = buildGitCommand({
@@ -1427,6 +1428,7 @@ cli
 cli
     .command("hunks add [...ids]", "Stage specific hunks by their stable ID")
     .action(async (ids) => {
+    ensureGitRepo();
     if (!ids || ids.length === 0) {
         console.error("Usage: critique hunks add <hunk-id> [<hunk-id> ...]");
         console.error("Use 'critique hunks list' to see available hunk IDs.");
@@ -1540,6 +1542,10 @@ cli
     .option("--stdin", "Read diff from stdin (for use as a pager)")
     .option("--scrollback", "Output to terminal scrollback instead of TUI (auto-enabled when non-TTY)")
     .action(async (base, head, options) => {
+    // Ensure we're inside a git repository before doing anything
+    if (!options.stdin) {
+        ensureGitRepo();
+    }
     // Apply theme if specified (zustand subscription auto-persists)
     if (options.theme && themeNames.includes(options.theme)) {
         useAppStore.setState({ themeName: options.theme });
@@ -1811,6 +1817,7 @@ cli
     .option("--json", "Output JSON to stdout (implies --web)")
     .option("--resume [id]", "Resume a previous review (shows select if no ID provided)")
     .action(async (base, head, options) => {
+    ensureGitRepo();
     try {
         // Handle resume mode
         if (options.resume !== undefined) {
@@ -1877,6 +1884,7 @@ cli
 cli
     .command("difftool <local> <remote>", "Git difftool integration")
     .action(async (local, remote) => {
+    ensureGitRepo();
     if (!process.stdout.isTTY) {
         execSync(`git diff --no-ext-diff "${local}" "${remote}"`, {
             stdio: "inherit",
@@ -1910,6 +1918,7 @@ cli
 cli
     .command("pick <branch>", "Pick files from another branch to apply to HEAD")
     .action(async (branch) => {
+    ensureGitRepo();
     try {
         const { stdout: currentBranch } = await execAsync("git branch --show-current");
         const current = currentBranch.trim();
@@ -2046,6 +2055,7 @@ cli
 }))
     .option("--title <title>", "HTML document title")
     .action(async (base, head, options) => {
+    ensureGitRepo();
     // Build git command and get diff
     const gitCommand = buildGitCommand({
         staged: options.staged,

package/dist/diff-utils.d.ts

@@ -1,3 +1,8 @@
+/**
+ * Check if the current directory is inside a git repository.
+ * If not, print a friendly error message and exit.
+ */
+export declare function ensureGitRepo(): void;
 /**
  * Strip submodule status lines from git diff output.
  * git diff --submodule=diff adds various status lines that the diff parser doesn't understand:

package/dist/diff-utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"diff-utils.d.ts","sourceRoot":"","sources":["../src/diff-utils.ts"],"names":[],"mappings":"AAOA;;;;;;;;GAQG;AACH,wBAAgB,qBAAqB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAchE;AAED;;;GAGG;AACH,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,QAAQ,GAAG,MAAM,CAAA;IACvB,IAAI,EAAE,MAAM,CAAA;IACZ,EAAE,EAAE,MAAM,CAAA;IACV,UAAU,EAAE,MAAM,CAAA;CACnB;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,MAAM,GAAG;IAC/C,aAAa,EAAE,MAAM,CAAA;IACrB,UAAU,EAAE,GAAG,CAAC,MAAM,EAAE,UAAU,CAAC,CAAA;CACpC,CA6FA;AAED;;;;;;;;GAQG;AACH,wBAAgB,iBAAiB,CAAC,CAAC,EACjC,OAAO,EAAE,MAAM,EACf,UAAU,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,CAAC,EAAE,GAChC,CAAC,CAAC,GAAG;IAAE,UAAU,CAAC,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAAC,UAAU,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,EAAE,CAiBzE;AAED,eAAO,MAAM,aAAa,UASzB,CAAC;AAEF,MAAM,WAAW,UAAU;IACzB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,KAAK,CAAC;QAAE,KAAK,EAAE,MAAM,EAAE,CAAA;KAAE,CAAC,CAAC;IAClC,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,mDAAmD;IACnD,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,uDAAuD;IACvD,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,4DAA4D;AAC5D,eAAO,MAAM,qBAAqB,IAAI,CAAC;AAEvC,MAAM,WAAW,iBAAiB;IAChC,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAC;IAC3B,iBAAiB,CAAC,EAAE,MAAM,EAAE,CAAC;CAC9B;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAC/B,OAAO,EAAE,IAAI,CAAC,iBAAiB,EAAE,QAAQ,GAAG,mBAAmB,CAAC,GAC/D,MAAM,EAAE,CAQV;AAED;;;GAGG;AACH,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,OAAO,CAsBhF;AAED;;;;GAIG;AACH,wBAAgB,2BAA2B,CAAC,CAAC,SAAS,UAAU,EAC9D,KAAK,EAAE,CAAC,EAAE,EACV,OAAO,EAAE,IAAI,CAAC,iBAAiB,EAAE,QAAQ,GAAG,mBAAmB,CAAC,GAC/D,CAAC,EAAE,CAKL;AAED;;GAEG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,iBAAiB,GAAG,MAAM,CAyDlE;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,sBAAsB,IAAI,MAAM,EAAE,CAiBjD;AAED;;;GAGG;AACH,wBAAgB,yBAAyB,CACvC,cAAc,EAAE,MAAM,EAAE,EACxB,OAAO,EAAE,IAAI,CAAC,iBAAiB,EAAE,SAAS,CAAC,GAC1C,MAAM,CAMR;AAED;;;;;;;GAOG;AACH,wBAAgB,aAAa,CAAC,IAAI,EAAE;IAClC,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB,GAAG,OAAO,GAAG,UAAU,GAAG,SAAS,GAAG,SAAS,CAW/C;AAED;;;GAGG;AACH,wBAAgB,WAAW,CAAC,IAAI,EAAE;IAChC,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB,GAAG,MAAM,CAYT;AAED;;;GAGG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE;IACnC,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB,GAAG,MAAM,GAAG,SAAS,CAQrB;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,KAAK,EAAE,KAAK,CAAC;IAAE,KAAK,EAAE,MAAM,EAAE,CAAA;CAAE,CAAC,GAAG;IAC/D,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,EAAE,MAAM,CAAC;CACnB,CAYA;AAED;;;GAGG;AACH,wBAAgB,WAAW,CACzB,SAAS,EAAE,MAAM,EACjB,SAAS,EAAE,MAAM,EACjB,IAAI,EAAE,MAAM,EACZ,cAAc,GAAE,MAAY,GAC3B,OAAO,GAAG,SAAS,CAQrB;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,CAAC,SAAS,UAAU,EAC/C,KAAK,EAAE,CAAC,EAAE,EACV,WAAW,EAAE,CAAC,IAAI,EAAE,CAAC,KAAK,MAAM,GAC/B,CAAC,CAAC,GAAG;IAAE,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC,EAAE,CAyD7B;AAED;;;GAGG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAqGnE"}
+{"version":3,"file":"diff-utils.d.ts","sourceRoot":"","sources":["../src/diff-utils.ts"],"names":[],"mappings":"AAOA;;;GAGG;AACH,wBAAgB,aAAa,IAAI,IAAI,CASpC;AAED;;;;;;;;GAQG;AACH,wBAAgB,qBAAqB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAchE;AAED;;;GAGG;AACH,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,QAAQ,GAAG,MAAM,CAAA;IACvB,IAAI,EAAE,MAAM,CAAA;IACZ,EAAE,EAAE,MAAM,CAAA;IACV,UAAU,EAAE,MAAM,CAAA;CACnB;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,MAAM,GAAG;IAC/C,aAAa,EAAE,MAAM,CAAA;IACrB,UAAU,EAAE,GAAG,CAAC,MAAM,EAAE,UAAU,CAAC,CAAA;CACpC,CA6FA;AAED;;;;;;;;GAQG;AACH,wBAAgB,iBAAiB,CAAC,CAAC,EACjC,OAAO,EAAE,MAAM,EACf,UAAU,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,CAAC,EAAE,GAChC,CAAC,CAAC,GAAG;IAAE,UAAU,CAAC,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAAC,UAAU,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,EAAE,CAiBzE;AAED,eAAO,MAAM,aAAa,UASzB,CAAC;AAEF,MAAM,WAAW,UAAU;IACzB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,KAAK,CAAC;QAAE,KAAK,EAAE,MAAM,EAAE,CAAA;KAAE,CAAC,CAAC;IAClC,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,mDAAmD;IACnD,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,uDAAuD;IACvD,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,4DAA4D;AAC5D,eAAO,MAAM,qBAAqB,IAAI,CAAC;AAEvC,MAAM,WAAW,iBAAiB;IAChC,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAAC;IAC3B,iBAAiB,CAAC,EAAE,MAAM,EAAE,CAAC;CAC9B;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAC/B,OAAO,EAAE,IAAI,CAAC,iBAAiB,EAAE,QAAQ,GAAG,mBAAmB,CAAC,GAC/D,MAAM,EAAE,CAQV;AAED;;;GAGG;AACH,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,OAAO,CAsBhF;AAED;;;;GAIG;AACH,wBAAgB,2BAA2B,CAAC,CAAC,SAAS,UAAU,EAC9D,KAAK,EAAE,CAAC,EAAE,EACV,OAAO,EAAE,IAAI,CAAC,iBAAiB,EAAE,QAAQ,GAAG,mBAAmB,CAAC,GAC/D,CAAC,EAAE,CAKL;AAED;;GAEG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,iBAAiB,GAAG,MAAM,CAyDlE;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,sBAAsB,IAAI,MAAM,EAAE,CAiBjD;AAED;;;GAGG;AACH,wBAAgB,yBAAyB,CACvC,cAAc,EAAE,MAAM,EAAE,EACxB,OAAO,EAAE,IAAI,CAAC,iBAAiB,EAAE,SAAS,CAAC,GAC1C,MAAM,CAMR;AAED;;;;;;;GAOG;AACH,wBAAgB,aAAa,CAAC,IAAI,EAAE;IAClC,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB,GAAG,OAAO,GAAG,UAAU,GAAG,SAAS,GAAG,SAAS,CAW/C;AAED;;;GAGG;AACH,wBAAgB,WAAW,CAAC,IAAI,EAAE;IAChC,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB,GAAG,MAAM,CAYT;AAED;;;GAGG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE;IACnC,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB,GAAG,MAAM,GAAG,SAAS,CAQrB;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,KAAK,EAAE,KAAK,CAAC;IAAE,KAAK,EAAE,MAAM,EAAE,CAAA;CAAE,CAAC,GAAG;IAC/D,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,EAAE,MAAM,CAAC;CACnB,CAYA;AAED;;;GAGG;AACH,wBAAgB,WAAW,CACzB,SAAS,EAAE,MAAM,EACjB,SAAS,EAAE,MAAM,EACjB,IAAI,EAAE,MAAM,EACZ,cAAc,GAAE,MAAY,GAC3B,OAAO,GAAG,SAAS,CAQrB;AAED;;GAEG;AACH,wBAAgB,YAAY,CAAC,CAAC,SAAS,UAAU,EAC/C,KAAK,EAAE,CAAC,EAAE,EACV,WAAW,EAAE,CAAC,IAAI,EAAE,CAAC,KAAK,MAAM,GAC/B,CAAC,CAAC,GAAG;IAAE,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC,EAAE,CAyD7B;AAED;;;GAGG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAqGnE"}