critique 0.1.103 → 0.1.105

package/AGENTS.md

@@ -17,8 +17,8 @@ gh pr view 536 -R anomalyco/opentui --json commits --jq '.commits[-1].oid[:40]'
 then use it in package.json:
 
 ```
-https://pkg.pr.new/anomalyco/opentui/@opentui/core@<hash>
-https://pkg.pr.new/anomalyco/opentui/@opentui/react@<hash>
+https://pkg.pr.new/anomalyco/opentui/@opentuah/core@<hash>
+https://pkg.pr.new/anomalyco/opentui/@opentuah/react@<hash>
 ```
 
 YOU MUST ALWAYS use the commit hash 40 characters long when changing the pkg.pr.new url! not the pr number! 
@@ -134,10 +134,7 @@ npx opensrc <owner>/<repo>      # GitHub repo (e.g., npx opensrc vercel/ai)
 
 ## opentui fork
 
-we are using an opentui fork with name opentuah with my personal PRs merged
-
-"@opentui/core": "npm:@opentuah/core@^0.1.80",
-"@opentui/react": "npm:@opentuah/react@^0.1.80",
+we are using an opentui fork published as `@opentuah`. imports use `@opentuah/core` and `@opentuah/react` directly (no npm alias remapping).
 
 To find my opentui folder with that fork see kimaki projects via kimaki cli, the one named opentui.
 

package/CHANGELOG.md

@@ -1,3 +1,20 @@
+# 0.1.105
+
+- `critique --stdin` / `critique --web`:
+  - Only show 'URL is private' notice when generating with --web (fix notice appearing in scrollback/pager output like lazygit where it makes no sense)
+- Tests:
+  - Add comprehensive integration tests for --stdin pager mode using tuistory (10 test cases covering empty diffs, multiple files, renames, binary files, narrow terminals, etc.)
+  - Rewrite lazygit pager test as real integration test that launches critique in a PTY and verifies scrollback output
+
+# 0.1.104
+
+- `critique --stdin` (pager mode):
+  - Force scrollback mode for `--stdin` pager usage to fix lazygit integration (#25)
+  - When used as a pager (e.g. `critique --stdin` in lazygit), critique now correctly outputs static colored text instead of interactive TUI escape sequences
+- `critique`:
+  - Fix crash in Bun compiled binaries where terminal TTY `columns`/`rows` report as 0, causing NaN framebuffer dimensions
+  - Replace `@opentui/*` npm alias remapping with direct `@opentuah/*` imports
+
 # 0.1.103
 
 - Syntax highlighting:

package/package.json

@@ -2,7 +2,7 @@
   "name": "critique",
   "module": "src/diff.tsx",
   "type": "module",
-  "version": "0.1.103",
+  "version": "0.1.105",
   "license": "MIT",
   "private": false,
   "bin": "./src/cli.tsx",
@@ -22,17 +22,18 @@
     "hono": "^4.7.10",
     "sharp": "^0.34.5",
     "stripe": "^20.2.0",
+    "tuistory": "^0.0.13",
     "typescript": "^5.9.3",
     "wrangler": "^4.19.1"
   },
   "dependencies": {
     "@agentclientprotocol/sdk": "^0.13.1",
     "@clack/prompts": "1.0.0-alpha.9",
-    "@opentui/core": "npm:@opentuah/core@0.1.88",
-    "@opentui/react": "npm:@opentuah/react@0.1.88",
+    "@opentuah/core": "0.1.95",
+    "@opentuah/react": "0.1.95",
     "@parcel/watcher": "^2.5.6",
-    "goke": "^6.1.3",
     "diff": "^8.0.2",
+    "goke": "^6.1.3",
     "js-yaml": "^4.1.1",
     "marked": "^17.0.1",
     "picocolors": "^1.1.1",

package/scripts/preview-review.tsx

@@ -3,12 +3,12 @@
 // Renders example hunks and review data to preview TUI appearance.
 // Run with: bun run scripts/preview-review.tsx (TUI) or --web (HTML upload).
 
-import { createCliRenderer, addDefaultParsers } from "@opentui/core"
+import { createCliRenderer, addDefaultParsers } from "@opentuah/core"
 import parsersConfig from "../parsers-config.ts"
 
 // Register custom syntax highlighting parsers
 addDefaultParsers(parsersConfig.parsers)
-import { createRoot } from "@opentui/react"
+import { createRoot } from "@opentuah/react"
 import * as React from "react"
 import { ReviewApp, ReviewAppView } from "../src/review/review-app.tsx"
 import { createHunk } from "../src/review/hunk-parser.ts"

package/src/ansi-html.ts

@@ -2,8 +2,8 @@
 // Uses opentui's test renderer to capture structured span data and generates responsive HTML documents
 // with proper font scaling to fit terminal content within viewport width.
 
-import { TextAttributes, rgbToHex, type RGBA } from "@opentui/core"
-import type { CapturedFrame, CapturedLine, CapturedSpan } from "@opentui/core"
+import { TextAttributes, rgbToHex, type RGBA } from "@opentuah/core"
+import type { CapturedFrame, CapturedLine, CapturedSpan } from "@opentuah/core"
 
 export interface ToHtmlOptions {
   /** Background color for the container */

package/src/ansi-output.test.ts

@@ -1,7 +1,7 @@
 import { describe, test, expect } from "bun:test"
 import { spanToAnsi, frameToAnsi } from "./ansi-output.ts"
-import { RGBA } from "@opentui/core"
-import type { CapturedFrame, CapturedSpan, CapturedLine } from "@opentui/core"
+import { RGBA } from "@opentuah/core"
+import type { CapturedFrame, CapturedSpan, CapturedLine } from "@opentuah/core"
 
 const themeBg = RGBA.fromValues(0, 0, 0, 1) // Black background
 

package/src/ansi-output.ts

@@ -3,8 +3,8 @@
 // Falls back gracefully: truecolor → 256 → 16 → plain text.
 
 import supportsColor from "supports-color"
-import { TextAttributes, type RGBA } from "@opentui/core"
-import type { CapturedFrame, CapturedSpan, CapturedLine } from "@opentui/core"
+import { TextAttributes, type RGBA } from "@opentuah/core"
+import type { CapturedFrame, CapturedSpan, CapturedLine } from "@opentuah/core"
 
 // Color support levels: 0=none, 1=16 colors, 2=256 colors, 3=truecolor
 type ColorLevel = 0 | 1 | 2 | 3

package/src/cli-scroll.test.tsx

@@ -3,7 +3,7 @@
 import * as React from "react"
 import { afterEach, describe, expect, it } from "bun:test"
 import { act } from "react"
-import { testRender } from "@opentui/react/test-utils"
+import { testRender } from "@opentuah/react/test-utils"
 import { App } from "./cli.tsx"
 import type { ParsedFile } from "./diff-utils.ts"
 

package/src/cli.tsx

@@ -3,6 +3,10 @@
 // Provides TUI diff viewing, AI-powered review generation, and web preview upload.
 // Commands: default (diff), review (AI analysis), web (HTML upload), pick (cherry-pick files).
 
+// Must be first import: patches process.stdout.columns/rows for Bun compiled binaries
+// where they incorrectly return 0 instead of actual terminal dimensions.
+import "./patch-terminal-dimensions.ts";
+
 import { goke, wrapJsonSchema } from "goke";
 import {
   createRoot,
@@ -11,7 +15,7 @@ import {
   useOnResize,
   useRenderer,
   useTerminalDimensions,
-} from "@opentui/react";
+} from "@opentuah/react";
 import { useCopySelection } from "./hooks/use-copy-selection.ts";
 import * as React from "react";
 import { exec, execSync } from "child_process";
@@ -22,7 +26,7 @@ import {
   ScrollBoxRenderable,
   BoxRenderable,
   addDefaultParsers,
-} from "@opentui/core";
+} from "@opentuah/core";
 import parsersConfig from "../parsers-config.ts";
 
 // Register custom syntax highlighting parsers
@@ -1826,7 +1830,7 @@ cli
       return;
     }
 
-    if (options.scrollback || !process.stdout.isTTY) {
+    if (options.scrollback || options.stdin || !process.stdout.isTTY) {
       // For scrollback, prefer terminal width over --cols default (240 is for web)
       const scrollbackCols = process.stdout.columns || parseInt(options.cols) || 120;
       await runScrollbackMode(cleanedDiff, {

package/src/components/diff-view.test.tsx

@@ -3,7 +3,7 @@
 import * as React from "react"
 import { afterEach, describe, expect, it } from "bun:test"
 import { act } from "react"
-import { testRender } from "@opentui/react/test-utils"
+import { testRender } from "@opentuah/react/test-utils"
 import { DiffView } from "./diff-view.tsx"
 import { useAppStore } from "../store.ts"
 

package/src/components/diff-view.tsx

@@ -3,7 +3,7 @@
 // Supports split and unified view modes with line numbers.
 
 import * as React from "react"
-import { SyntaxStyle } from "@opentui/core"
+import { SyntaxStyle } from "@opentuah/core"
 import { getSyntaxTheme, getResolvedTheme, rgbaToHex } from "../themes.ts"
 
 export interface DiffViewProps {

package/src/diff-utils.ts

@@ -459,7 +459,7 @@ export function processFiles<T extends ParsedFile>(
 
 /**
  * Detect filetype from filename for syntax highlighting
- * Maps to tree-sitter parsers available in @opentui/core and parsers-config.ts
+ * Maps to tree-sitter parsers available in @opentuah/core and parsers-config.ts
  */
 export function detectFiletype(filePath: string): string | undefined {
   const ext = filePath.split(".").pop()?.toLowerCase();

package/src/directory-tree.test.tsx

@@ -2,7 +2,7 @@
 // Uses opentui test renderer with captureCharFrame() for visual testing
 
 import { afterEach, describe, expect, it } from "bun:test"
-import { testRender } from "@opentui/react/test-utils"
+import { testRender } from "@opentuah/react/test-utils"
 import { buildDirectoryTree, type TreeFileInfo, type TreeNode } from "./directory-tree.ts"
 import { DirectoryTreeView } from "./components/directory-tree-view.tsx"
 

package/src/dropdown.test.tsx

@@ -1,7 +1,7 @@
 import * as React from "react"
 import { afterEach, describe, expect, it } from "bun:test"
 import { act } from "react"
-import { testRender } from "@opentui/react/test-utils"
+import { testRender } from "@opentuah/react/test-utils"
 import Dropdown, { filterDropdownOptions } from "./dropdown.tsx"
 import { getResolvedTheme } from "./themes.ts"
 

package/src/dropdown.tsx

@@ -3,8 +3,8 @@
 // Used by main diff view for file picker and theme picker overlays.
 
 import React, { useCallback, useEffect, useRef, useState, type ReactNode } from "react";
-import { useKeyboard } from "@opentui/react";
-import { TextAttributes, TextareaRenderable } from "@opentui/core";
+import { useKeyboard } from "@opentuah/react";
+import { TextAttributes, TextareaRenderable } from "@opentuah/core";
 import { type ResolvedTheme, rgbaToHex } from "./themes";
 
 export interface DropdownOption {

package/src/hooks/use-copy-selection.ts

@@ -2,7 +2,7 @@
 // Automatically copies selected text to clipboard when user releases mouse button.
 // Uses native clipboard commands (pbcopy, xclip, etc.) with OSC52 fallback.
 
-import { useRenderer } from "@opentui/react"
+import { useRenderer } from "@opentuah/react"
 import { spawn } from "child_process"
 
 /**

package/src/image.ts

@@ -2,7 +2,7 @@
 // Uses opentui-image.ts for generic frame-to-image conversion.
 // Adds theme resolution and diff/review-specific rendering.
 
-import type { CapturedFrame } from "@opentui/core"
+import type { CapturedFrame } from "@opentuah/core"
 import { getResolvedTheme, rgbaToHex } from "./themes.ts"
 import {
   renderFrameToImage,
@@ -334,12 +334,11 @@ export async function renderDiffToOgImage(
   const charWidth = fontSize * 0.6
   const cols = options.cols ?? Math.floor(contentWidth / charWidth)
 
-  // Render diff to captured frame (no notice block for OG images)
+  // Render diff to captured frame
   const frame = await renderDiffToFrame(diffContent, {
     cols,
     maxRows: 200,
     themeName,
-    showNotice: false,
   })
 
   // Convert frame to OG image

package/src/opentui-image.ts

@@ -4,8 +4,8 @@
 import { tmpdir } from "os"
 import { join } from "path"
 import fs from "fs"
-import { TextAttributes, rgbToHex, type RGBA } from "@opentui/core"
-import type { CapturedFrame, CapturedLine, CapturedSpan } from "@opentui/core"
+import { TextAttributes, rgbToHex, type RGBA } from "@opentuah/core"
+import type { CapturedFrame, CapturedLine, CapturedSpan } from "@opentuah/core"
 
 // ─────────────────────────────────────────────────────────────
 // Types

package/src/patch-terminal-dimensions.ts

@@ -0,0 +1,51 @@
+// Polyfill for terminal dimensions in Bun compiled binaries.
+// Bun's `--compile` produces binaries where process.stdout.columns/rows are 0
+// instead of the actual terminal size (even when isTTY is true).
+// This must be imported before any opentui imports, since opentui reads
+// stdout.columns at init time and uses `?? fallback` which doesn't catch 0.
+
+import { execSync } from "child_process"
+
+function getTerminalSize(): { cols: number; rows: number } | null {
+  try {
+    const cols = parseInt(
+      execSync("tput cols", {
+        encoding: "utf-8",
+        stdio: ["inherit", "pipe", "pipe"],
+      }).trim(),
+    )
+    const rows = parseInt(
+      execSync("tput lines", {
+        encoding: "utf-8",
+        stdio: ["inherit", "pipe", "pipe"],
+      }).trim(),
+    )
+    if (cols > 0 && rows > 0) return { cols, rows }
+  } catch {}
+  return null
+}
+
+if (process.stdout.isTTY && process.stdout.columns === 0) {
+  const size = getTerminalSize()
+  if (size) {
+    Object.defineProperty(process.stdout, "columns", {
+      value: size.cols,
+      writable: true,
+      configurable: true,
+    })
+    Object.defineProperty(process.stdout, "rows", {
+      value: size.rows,
+      writable: true,
+      configurable: true,
+    })
+
+    // Keep patched values updated on terminal resize
+    process.on("SIGWINCH", () => {
+      const newSize = getTerminalSize()
+      if (newSize) {
+        process.stdout.columns = newSize.cols
+        process.stdout.rows = newSize.rows
+      }
+    })
+  }
+}

package/src/review/review-app.test.tsx

@@ -1,7 +1,7 @@
 // Test for ReviewAppView rendering with example YAML data
 
 import { afterEach, describe, expect, it } from "bun:test"
-import { testRender } from "@opentui/react/test-utils"
+import { testRender } from "@opentuah/react/test-utils"
 import { ReviewAppView } from "./review-app.tsx"
 import { createHunk } from "./hunk-parser.ts"
 import type { ReviewYaml } from "./types.ts"

package/src/review/review-app.tsx

@@ -3,8 +3,8 @@
 // Supports live generation updates, theme switching, and resume from saved reviews.
 
 import * as React from "react"
-import { useKeyboard, useRenderer, useTerminalDimensions } from "@opentui/react"
-import { MacOSScrollAccel, SyntaxStyle, BoxRenderable, CodeRenderable, TextRenderable, ScrollBoxRenderable } from "@opentui/core"
+import { useKeyboard, useRenderer, useTerminalDimensions } from "@opentuah/react"
+import { MacOSScrollAccel, SyntaxStyle, BoxRenderable, CodeRenderable, TextRenderable, ScrollBoxRenderable } from "@opentuah/core"
 import { useCopySelection } from "../hooks/use-copy-selection.ts"
 import type { Token } from "marked"
 import { getResolvedTheme, getSyntaxTheme, defaultThemeName, themeNames, rgbaToHex } from "../themes.ts"

package/src/review/stream-display.test.tsx

@@ -1,7 +1,7 @@
 // Tests for StreamDisplay component
 
 import { afterEach, describe, expect, it } from "bun:test"
-import { testRender } from "@opentui/react/test-utils"
+import { testRender } from "@opentuah/react/test-utils"
 import type { SessionNotification } from "@agentclientprotocol/sdk"
 import { StreamDisplay } from "./stream-display.tsx"
 

package/src/review/stream-display.tsx

@@ -3,7 +3,7 @@
 // Shows formatted tool operations with file names and edit statistics.
 
 import * as React from "react"
-import { SyntaxStyle } from "@opentui/core"
+import { SyntaxStyle } from "@opentuah/core"
 import type { SessionNotification } from "@agentclientprotocol/sdk"
 import { formatNotifications, SYMBOLS, COLORS, isEditTool, type StreamLine } from "./acp-stream-display.ts"
 import { getSyntaxTheme, getResolvedTheme, rgbaToHex } from "../themes.ts"

package/src/stdin-pager.test.ts

@@ -0,0 +1,420 @@
+// Integration tests for --stdin pager mode (lazygit integration).
+// Reproduces https://github.com/remorses/critique/issues/25
+//
+// Uses tuistory to launch critique in a PTY (exactly like lazygit does),
+// pipes a real diff via stdin, and verifies the output is plain scrollback
+// text — not interactive TUI escape sequences.
+//
+// tuistory spawns a PTY where isTTY=true, which is exactly how lazygit
+// runs its pager (via github.com/creack/pty). This makes the test
+// realistic: it catches the original bug where --stdin + TTY incorrectly
+// entered interactive TUI mode instead of scrollback mode.
+
+import { describe, test, expect, afterAll, beforeAll } from "bun:test"
+import { launchTerminal } from "tuistory"
+import fs from "fs"
+import path from "path"
+
+const TEMP_DIR = path.join(import.meta.dir, ".test-stdin-pager-tmp")
+
+function tempFile(name: string, content: string): string {
+  const p = path.join(TEMP_DIR, name)
+  fs.writeFileSync(p, content)
+  return p
+}
+
+function launchCritique(diffPath: string, opts?: { cols?: number; rows?: number }) {
+  return launchTerminal({
+    command: "bash",
+    args: ["-c", `cat "${diffPath}" | bun run src/cli.tsx --stdin`],
+    cols: opts?.cols ?? 100,
+    rows: opts?.rows ?? 30,
+    cwd: process.cwd(),
+    env: {
+      PATH: process.env.PATH,
+      HOME: process.env.HOME,
+      TERM: "xterm-256color",
+    },
+  })
+}
+
+// -- Sample diffs --
+
+const SINGLE_FILE_DIFF = [
+  "diff --git a/src/hello.ts b/src/hello.ts",
+  "--- a/src/hello.ts",
+  "+++ b/src/hello.ts",
+  "@@ -1,3 +1,3 @@",
+  " const greeting = 'hello'",
+  "-console.log(greeting)",
+  "+console.log(greeting + ' world')",
+  " export default greeting",
+].join("\n")
+
+// Empty patch — lazygit sends this when there are no changes for a file
+const EMPTY_DIFF = ""
+
+// Diff with only context lines and no actual changes (can happen with -U999)
+const CONTEXT_ONLY_DIFF = [
+  "diff --git a/readme.md b/readme.md",
+  "--- a/readme.md",
+  "+++ b/readme.md",
+  "@@ -1,3 +1,3 @@",
+  " # My Project",
+  " ",
+  " Some description",
+].join("\n")
+
+// Multiple files in a single diff
+const MULTI_FILE_DIFF = [
+  "diff --git a/src/index.ts b/src/index.ts",
+  "--- a/src/index.ts",
+  "+++ b/src/index.ts",
+  "@@ -1,4 +1,6 @@",
+  " import { App } from './app'",
+  "+import { Logger } from './logger'",
+  " ",
+  " const app = new App()",
+  "+const logger = new Logger()",
+  " app.start()",
+  "diff --git a/src/logger.ts b/src/logger.ts",
+  "new file mode 100644",
+  "--- /dev/null",
+  "+++ b/src/logger.ts",
+  "@@ -0,0 +1,5 @@",
+  "+export class Logger {",
+  "+  log(msg: string) {",
+  "+    console.log(`[LOG] ${msg}`)",
+  "+  }",
+  "+}",
+].join("\n")
+
+// Deletion-only diff
+const DELETE_ONLY_DIFF = [
+  "diff --git a/src/deprecated.ts b/src/deprecated.ts",
+  "deleted file mode 100644",
+  "--- a/src/deprecated.ts",
+  "+++ /dev/null",
+  "@@ -1,4 +0,0 @@",
+  "-// This module is no longer needed",
+  "-export function oldHelper() {",
+  "-  return 'deprecated'",
+  "-}",
+].join("\n")
+
+// Addition-only diff (new file)
+const NEW_FILE_DIFF = [
+  "diff --git a/src/utils.ts b/src/utils.ts",
+  "new file mode 100644",
+  "--- /dev/null",
+  "+++ b/src/utils.ts",
+  "@@ -0,0 +1,7 @@",
+  "+export function clamp(value: number, min: number, max: number): number {",
+  "+  return Math.min(Math.max(value, min), max)",
+  "+}",
+  "+",
+  "+export function identity<T>(x: T): T {",
+  "+  return x",
+  "+}",
+].join("\n")
+
+// Large hunk with many changes
+const LARGE_HUNK_DIFF = [
+  "diff --git a/config.json b/config.json",
+  "--- a/config.json",
+  "+++ b/config.json",
+  "@@ -1,9 +1,11 @@",
+  " {",
+  '-  "name": "my-app",',
+  '+  "name": "my-awesome-app",',
+  '-  "version": "1.0.0",',
+  '+  "version": "2.0.0",',
+  '   "description": "A sample app",',
+  '-  "main": "index.js",',
+  '+  "main": "dist/index.js",',
+  '+  "types": "dist/index.d.ts",',
+  '   "scripts": {',
+  '-    "build": "tsc"',
+  '+    "build": "tsc --project tsconfig.build.json",',
+  '+    "test": "bun test"',
+  "   }",
+  " }",
+].join("\n")
+
+// Rename diff (common in lazygit)
+const RENAME_DIFF = [
+  "diff --git a/src/old-name.ts b/src/new-name.ts",
+  "similarity index 90%",
+  "rename from src/old-name.ts",
+  "rename to src/new-name.ts",
+  "--- a/src/old-name.ts",
+  "+++ b/src/new-name.ts",
+  "@@ -1,3 +1,3 @@",
+  "-export const name = 'old'",
+  "+export const name = 'new'",
+  " export const version = 1",
+  " export default name",
+].join("\n")
+
+// Binary file diff (lazygit shows these)
+const BINARY_DIFF = [
+  "diff --git a/logo.png b/logo.png",
+  "new file mode 100644",
+  "Binary files /dev/null and b/logo.png differ",
+].join("\n")
+
+describe("--stdin pager mode (lazygit issue #25)", () => {
+  beforeAll(() => {
+    fs.mkdirSync(TEMP_DIR, { recursive: true })
+  })
+
+  afterAll(() => {
+    try {
+      fs.rmSync(TEMP_DIR, { recursive: true })
+    } catch {}
+  })
+
+  test("single file change", async () => {
+    const diffPath = tempFile("single.diff", SINGLE_FILE_DIFF)
+    const session = await launchCritique(diffPath)
+    await session.waitForText("hello", { timeout: 15000 })
+    const trimmed = await session.text({ trimEnd: true })
+
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+       a/src/hello.ts → b/src/hello.ts +1-1
+
+       1   const greeting = 'hello'
+       2 - console.log(greeting)
+       2 + console.log(greeting + ' world')
+       3   export default greeting"
+    `)
+
+    const lines = trimmed.split("\n").filter((l) => l.trim().length > 0)
+    expect(lines.length).toBeGreaterThan(0)
+    expect(lines.length).toBeLessThan(25)
+    session.close()
+  }, 30000)
+
+  test("empty diff produces no crash", async () => {
+    const diffPath = tempFile("empty.diff", EMPTY_DIFF)
+    const session = await launchCritique(diffPath)
+
+    // Empty diff should cause critique to exit quickly.
+    // Wait a bit for it to process and exit.
+    await new Promise((r) => setTimeout(r, 5000))
+    const trimmed = await session.text({ trimEnd: true, immediate: true })
+
+    // Should either be empty or show an error message — not a TUI
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+      No changes to display"
+    `)
+
+    session.close()
+  }, 15000)
+
+  test("context-only diff (no actual changes)", async () => {
+    const diffPath = tempFile("context-only.diff", CONTEXT_ONLY_DIFF)
+    const session = await launchCritique(diffPath)
+
+    await new Promise((r) => setTimeout(r, 5000))
+    const trimmed = await session.text({ trimEnd: true, immediate: true })
+
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+       a/readme.md → b/readme.md +0-0
+
+       1 # My Project
+       2
+       3 Some description"
+    `)
+
+    session.close()
+  }, 15000)
+
+  test("multiple files in one diff", async () => {
+    const diffPath = tempFile("multi.diff", MULTI_FILE_DIFF)
+    const session = await launchCritique(diffPath)
+    await session.waitForText("Logger", { timeout: 15000 })
+    const trimmed = await session.text({ trimEnd: true })
+
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+       b/src/logger.ts +5-0
+
+       1 + export class Logger {
+       2 +   log(msg: string) {
+       3 +     console.log(\`[LOG] \${msg}\`)
+       4 +   }
+       5 + }
+
+
+       a/src/index.ts → b/src/index.ts +2-0
+
+       1   import { App } from './app'
+       2 + import { Logger } from './logger'
+       3
+       4   const app = new App()
+       5 + const logger = new Logger()
+       6   app.start()"
+    `)
+
+    // Should contain both filenames
+    expect(trimmed).toContain("index.ts")
+    expect(trimmed).toContain("logger.ts")
+
+    // Should not show the privacy notice
+    expect(trimmed).not.toContain("URL is private")
+
+    const lines = trimmed.split("\n").filter((l) => l.trim().length > 0)
+    expect(lines.length).toBeLessThan(40)
+    session.close()
+  }, 30000)
+
+  test("deleted file", async () => {
+    const diffPath = tempFile("delete.diff", DELETE_ONLY_DIFF)
+    const session = await launchCritique(diffPath)
+    await session.waitForText("deprecated", { timeout: 15000 })
+    const trimmed = await session.text({ trimEnd: true })
+
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+       a/src/deprecated.ts +0-4
+
+       1 - // This module is no longer needed
+       2 - export function oldHelper() {
+       3 -   return 'deprecated'
+       4 - }"
+    `)
+
+    expect(trimmed).toContain("deprecated")
+    expect(trimmed).not.toContain("URL is private")
+    session.close()
+  }, 30000)
+
+  test("new file", async () => {
+    const diffPath = tempFile("newfile.diff", NEW_FILE_DIFF)
+    const session = await launchCritique(diffPath)
+    await session.waitForText("clamp", { timeout: 15000 })
+    const trimmed = await session.text({ trimEnd: true })
+
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+       b/src/utils.ts +7-0
+
+       1 + export function clamp(value: number, min: number, max: number): number {
+       2 +   return Math.min(Math.max(value, min), max)
+       3 + }
+       4 +
+       5 + export function identity<T>(x: T): T {
+       6 +   return x
+       7 + }"
+    `)
+
+    expect(trimmed).toContain("clamp")
+    expect(trimmed).toContain("identity")
+    expect(trimmed).not.toContain("URL is private")
+    session.close()
+  }, 30000)
+
+  test("large hunk with many changes", async () => {
+    const diffPath = tempFile("large.diff", LARGE_HUNK_DIFF)
+    const session = await launchCritique(diffPath)
+    await session.waitForText("config.json", { timeout: 15000 })
+    const trimmed = await session.text({ trimEnd: true })
+
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+       a/config.json → b/config.json +6-4
+
+        1   {
+        2 -   "name": "my-app",
+        3 -   "version": "1.0.0",
+        2 +   "name": "my-awesome-app",
+        3 +   "version": "2.0.0",
+        4     "description": "A sample app",
+        5 -   "main": "index.js",
+        5 +   "main": "dist/index.js",
+        6 +   "types": "dist/index.d.ts",
+        7     "scripts": {
+        7 -     "build": "tsc"
+        8 +     "build": "tsc --project tsconfig.build.json",
+        9 +     "test": "bun test"
+       10     }
+       11   }"
+    `)
+
+    expect(trimmed).toContain("config.json")
+    expect(trimmed).toContain("my-awesome-app")
+    expect(trimmed).not.toContain("URL is private")
+    session.close()
+  }, 30000)
+
+  test("file rename with changes", async () => {
+    const diffPath = tempFile("rename.diff", RENAME_DIFF)
+    const session = await launchCritique(diffPath)
+    await session.waitForText("new-name", { timeout: 15000 })
+    const trimmed = await session.text({ trimEnd: true })
+
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+       src/old-name.ts → src/new-name.ts +1-1
+
+       1 - export const name = 'old'
+       1 + export const name = 'new'
+       2   export const version = 1
+       3   export default name"
+    `)
+
+    // Should show the rename
+    expect(trimmed).toContain("old-name")
+    expect(trimmed).toContain("new-name")
+    expect(trimmed).not.toContain("URL is private")
+    session.close()
+  }, 30000)
+
+  test("binary file diff", async () => {
+    const diffPath = tempFile("binary.diff", BINARY_DIFF)
+    const session = await launchCritique(diffPath)
+
+    // Binary diffs may not render content — wait for exit or timeout
+    await new Promise((r) => setTimeout(r, 5000))
+    const trimmed = await session.text({ trimEnd: true, immediate: true })
+
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+       unknown +0-0"
+    `)
+
+    expect(trimmed).not.toContain("URL is private")
+    session.close()
+  }, 15000)
+
+  test("narrow terminal (40 cols) forces unified view", async () => {
+    const diffPath = tempFile("narrow.diff", SINGLE_FILE_DIFF)
+    const session = await launchCritique(diffPath, { cols: 40 })
+    await session.waitForText("hello", { timeout: 15000 })
+    const trimmed = await session.text({ trimEnd: true })
+
+    expect(trimmed).toMatchInlineSnapshot(`
+      "
+       a/src/hello.ts → b/src/hello.ts +1-1
+
+       1   const greeting = 'hello'
+       2 - console.log(greeting)
+       2 + console.log(greeting + ' world')
+       3   export default greeting"
+    `)
+
+    expect(trimmed).toContain("hello")
+    expect(trimmed).not.toContain("URL is private")
+
+    // Every non-empty line should fit within 40 cols
+    const lines = trimmed.split("\n").filter((l) => l.trim().length > 0)
+    for (const line of lines) {
+      expect(line.length).toBeLessThanOrEqual(40)
+    }
+    session.close()
+  }, 30000)
+})

package/src/themes.ts

@@ -2,7 +2,7 @@
 // Loads JSON theme files lazily on demand, resolves color references,
 // and provides both UI colors and Tree-sitter compatible syntax styles.
 
-import { parseColor, RGBA } from "@opentui/core";
+import { parseColor, RGBA } from "@opentuah/core";
 import { fileURLToPath } from "url";
 
 // Only import the default theme statically for fast startup

package/src/web-utils.test.ts

@@ -1,8 +1,8 @@
 import { describe, test, expect, afterEach } from "bun:test"
-import { createTestRenderer } from "@opentui/core/testing"
-import { createRoot } from "@opentui/react"
+import { createTestRenderer } from "@opentuah/core/testing"
+import { createRoot } from "@opentuah/react"
 import React from "react"
-import { RGBA } from "@opentui/core"
+import { RGBA } from "@opentuah/core"
 
 describe("getSpanLines rendering", () => {
   let renderer: Awaited<ReturnType<typeof createTestRenderer>>["renderer"] | null = null

package/src/web-utils.tsx

@@ -8,7 +8,7 @@ import fs from "fs"
 import { tmpdir } from "os"
 import { join } from "path"
 import { getResolvedTheme, rgbaToHex } from "./themes.ts"
-import type { CapturedFrame, RootRenderable, CliRenderer } from "@opentui/core"
+import type { CapturedFrame, RootRenderable, CliRenderer } from "@opentuah/core"
 import type { IndexedHunk, ReviewYaml } from "./review/types.ts"
 import { loadStoredLicenseKey, loadOrCreateOwnerSecret } from "./license.ts"
 
@@ -24,7 +24,7 @@ export interface CaptureOptions {
   title?: string
   /** Wrap mode for long lines (default: "word") */
   wrapMode?: "word" | "char" | "none"
-  /** Show privacy/expiry notice block at top (default: true) */
+  /** Show privacy/expiry notice block at top (default: false, enabled for web uploads) */
   showNotice?: boolean
 }
 
@@ -118,9 +118,9 @@ export async function renderDiffToFrame(
   diffContent: string,
   options: CaptureOptions
 ): Promise<CapturedFrame> {
-  const { createTestRenderer } = await import("@opentui/core/testing")
-  const { createRoot } = await import("@opentui/react")
-  const { getTreeSitterClient } = await import("@opentui/core")
+  const { createTestRenderer } = await import("@opentuah/core/testing")
+  const { createRoot } = await import("@opentuah/react")
+  const { getTreeSitterClient } = await import("@opentuah/core")
   const React = await import("react")
   const { parsePatch, formatPatch } = await import("diff")
   
@@ -160,7 +160,7 @@ export async function renderDiffToFrame(
   const webMuted = rgbaToHex(webTheme.textMuted)
 
   const showExpiryNotice = shouldShowExpiryNotice()
-  const showNotice = options.showNotice !== false
+  const showNotice = options.showNotice === true
 
   // Create the diff view component
   // NOTE: No height: "100%" - let content determine its natural height
@@ -293,8 +293,8 @@ export async function captureToHtml(
 ): Promise<string> {
   const { frameToHtmlDocument } = await import("./ansi-html.ts")
 
-  // Render diff to captured frame
-  const frame = await renderDiffToFrame(diffContent, options)
+  // Render diff to captured frame (with notice for web uploads)
+  const frame = await renderDiffToFrame(diffContent, { ...options, showNotice: true })
 
   // Get theme colors for HTML output
   const theme = getResolvedTheme(options.themeName)
@@ -375,9 +375,9 @@ export interface ReviewRenderOptions extends CaptureOptions {
 export async function renderReviewToFrame(
   options: ReviewRenderOptions
 ): Promise<CapturedFrame> {
-  const { createTestRenderer } = await import("@opentui/core/testing")
-  const { createRoot } = await import("@opentui/react")
-  const { getTreeSitterClient } = await import("@opentui/core")
+  const { createTestRenderer } = await import("@opentuah/core/testing")
+  const { createRoot } = await import("@opentuah/react")
+  const { getTreeSitterClient } = await import("@opentuah/core")
   const React = await import("react")
   
   // Pre-initialize TreeSitter client to ensure syntax highlighting works
@@ -396,7 +396,7 @@ export async function renderReviewToFrame(
   const webText = rgbaToHex(theme.text)
   const webMuted = rgbaToHex(theme.textMuted)
   const showExpiryNotice = shouldShowExpiryNotice()
-  const showNotice = options.showNotice !== false
+  const showNotice = options.showNotice === true
 
   // Content-fitting: start small, double if clipped, shrink to fit
   let currentHeight = 100
@@ -488,8 +488,8 @@ export async function captureReviewToHtml(
 ): Promise<string> {
   const { frameToHtmlDocument } = await import("./ansi-html.ts")
 
-  // Render review to captured frame
-  const frame = await renderReviewToFrame(options)
+  // Render review to captured frame (with notice for web uploads)
+  const frame = await renderReviewToFrame({ ...options, showNotice: true })
 
   // Get theme colors for HTML output
   const theme = getResolvedTheme(options.themeName)

package/tsconfig.json

@@ -7,7 +7,7 @@
     "module": "Preserve",
     "moduleDetection": "force",
     "jsx": "react-jsx",
-    "jsxImportSource": "@opentui/react",
+    "jsxImportSource": "@opentuah/react",
     "allowJs": true,
 
     "noImplicitAny": false,