ralph-review 0.1.3 → 0.1.5

package/README.md

@@ -5,7 +5,7 @@
 [![Version](https://img.shields.io/github/v/tag/kenryu42/ralph-review?label=version&color=blue)](https://github.com/kenryu42/ralph-review)
 [![License: MIT](https://img.shields.io/badge/License-MIT-red.svg)](https://opensource.org/licenses/MIT)
 
-A CLI tool that orchestrates agentic review-fix cycles until your code is clean.
+Orchestrating coding agents for code review, verification and fixing via the ralph loop.
 
 ---
 
@@ -131,6 +131,10 @@ Treats review findings as untrusted input — verifies every claim against actua
 ## Installation
 
 ```bash
+# Homebrew
+brew install kenryu42/tap/ralph-review
+
+# npm
 npm install -g ralph-review
 ```
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ralph-review",
-  "version": "0.1.3",
-  "description": "A CLI tool that orchestrates agentic review-fix cycles until your code is clean.",
+  "version": "0.1.5",
+  "description": "Orchestrating coding agents for code review, verification and fixing via the ralph loop.",
   "license": "MIT",
   "type": "module",
   "author": {

package/src/commands/init.ts

@@ -754,8 +754,8 @@ export async function buildAutoInitInput(
       iterationTimeoutMinutes,
       defaultReviewType: "uncommitted",
       runSimplifierByDefault: false,
-      runWatchByDefault: DEFAULT_CONFIG.run?.watch ?? true,
-      soundNotificationsEnabled: DEFAULT_CONFIG.notifications?.sound.enabled ?? true,
+      runWatchByDefault: true,
+      soundNotificationsEnabled: true,
     },
     skippedAgents,
   };
@@ -1043,14 +1043,21 @@ export async function runInitWithRuntime(
   }
 
   const resolvedInput = requireInitInput(runtime, input);
-  const inputWithPreferences: InitInput = {
-    ...resolvedInput,
-    runWatchByDefault: await promptForRunWatch(runtime, resolvedInput.runWatchByDefault),
-    soundNotificationsEnabled: await promptForSoundNotifications(
-      runtime,
-      resolvedInput.soundNotificationsEnabled
-    ),
-  };
+  const inputWithPreferences: InitInput =
+    setupMode === "auto"
+      ? {
+          ...resolvedInput,
+          runWatchByDefault: true,
+          soundNotificationsEnabled: true,
+        }
+      : {
+          ...resolvedInput,
+          runWatchByDefault: await promptForRunWatch(runtime, resolvedInput.runWatchByDefault),
+          soundNotificationsEnabled: await promptForSoundNotifications(
+            runtime,
+            resolvedInput.soundNotificationsEnabled
+          ),
+        };
 
   const config = buildConfig(inputWithPreferences);
   runtime.prompt.log.info(`Proposed configuration:\n${formatConfigDisplay(config)}`);

package/src/commands/run.ts

@@ -618,15 +618,26 @@ export async function startReview(
     }
   }
 
-  const modeOptions = [
-    options.base !== undefined && "--base",
-    options.uncommitted && "--uncommitted",
-    options.commit !== undefined && "--commit",
-    options.custom !== undefined && "--custom",
-  ].filter(Boolean);
-
-  if (modeOptions.length > 1) {
-    runtime.prompt.log.error(`Cannot use ${modeOptions.join(" and ")} together`);
+  if (options.base !== undefined && options.commit !== undefined) {
+    runtime.prompt.log.error("Cannot use --base and --commit together");
+    runtime.process.exit(1);
+    return;
+  }
+
+  if (options.uncommitted && options.base !== undefined) {
+    runtime.prompt.log.error("Cannot use --uncommitted and --base together");
+    runtime.process.exit(1);
+    return;
+  }
+
+  if (options.uncommitted && options.commit !== undefined) {
+    runtime.prompt.log.error("Cannot use --uncommitted and --commit together");
+    runtime.process.exit(1);
+    return;
+  }
+
+  if (options.uncommitted && options.custom !== undefined) {
+    runtime.prompt.log.error("Cannot use --uncommitted and --custom together");
     runtime.process.exit(1);
     return;
   }

package/src/lib/agents/codex.ts

@@ -52,6 +52,12 @@ export const codexConfig: AgentConfig = {
 
     const baseReviewArgs = withReasoningEffort(["exec", "--json"], reasoning);
 
+    if (reviewOptions?.customInstructions) {
+      const fullPrompt = prompt ? `review ${prompt}` : "review";
+      const customArgs = withReasoningEffort(["exec", "--full-auto", "--json"], reasoning);
+      return withModel([...customArgs, fullPrompt], model);
+    }
+
     if (reviewOptions?.commitSha) {
       return withModel([...baseReviewArgs, "review", "--commit", reviewOptions.commitSha], model);
     }
@@ -60,12 +66,6 @@ export const codexConfig: AgentConfig = {
       return withModel([...baseReviewArgs, "review", "--base", reviewOptions.baseBranch], model);
     }
 
-    if (reviewOptions?.customInstructions) {
-      const fullPrompt = prompt ? `review ${prompt}` : "review";
-      const customArgs = withReasoningEffort(["exec", "--full-auto", "--json"], reasoning);
-      return withModel([...customArgs, fullPrompt], model);
-    }
-
     return withModel([...baseReviewArgs, "review", "--uncommitted"], model);
   },
   buildEnv: defaultBuildEnv,

package/src/lib/diagnostics/checks.ts

@@ -508,13 +508,7 @@ export async function runDiagnostics(
         }
       }
 
-      if (
-        !options.baseBranch &&
-        !options.commitSha &&
-        !options.customInstructions &&
-        insideGitRepo &&
-        !gitRepoError
-      ) {
+      if (!options.baseBranch && !options.commitSha && insideGitRepo && !gitRepoError) {
         let hasChanges = false;
         let hasChangesError: string | null = null;
         try {

package/src/lib/format.ts

@@ -1,23 +1,31 @@
 import type { ReviewOptions } from "@/lib/types";
 
+function formatCustomReviewType(customInstructions: string): string {
+  const instruction = customInstructions.slice(0, 40);
+  return customInstructions.length > 40 ? `custom (${instruction}...)` : `custom (${instruction})`;
+}
+
 export function formatReviewType(reviewOptions: ReviewOptions | undefined): string {
   if (!reviewOptions) return "uncommitted changes";
 
-  if (reviewOptions.customInstructions) {
-    const instruction = reviewOptions.customInstructions.slice(0, 40);
-    return reviewOptions.customInstructions.length > 40
-      ? `custom (${instruction}...)`
-      : `custom (${instruction})`;
-  }
-
   if (reviewOptions.commitSha) {
     const shortSha = reviewOptions.commitSha.slice(0, 7);
+    if (reviewOptions.customInstructions) {
+      return `commit (${shortSha}) + ${formatCustomReviewType(reviewOptions.customInstructions)}`;
+    }
     return `commit (${shortSha})`;
   }
 
   if (reviewOptions.baseBranch) {
+    if (reviewOptions.customInstructions) {
+      return `base (${reviewOptions.baseBranch}) + ${formatCustomReviewType(reviewOptions.customInstructions)}`;
+    }
     return `base (${reviewOptions.baseBranch})`;
   }
 
+  if (reviewOptions.customInstructions) {
+    return formatCustomReviewType(reviewOptions.customInstructions);
+  }
+
   return "uncommitted changes";
 }

package/src/lib/html/dashboard/script.ts

@@ -163,6 +163,48 @@ const DASHBOARD_SCRIPT_TEMPLATE = `
   const sortByPriority = (fixes) =>
     [...fixes].sort((a, b) => getPriorityRank(a.priority) - getPriorityRank(b.priority));
 
+  const formatFixRangeHunk = (lineStart, lineEnd) => {
+    const count = lineEnd - lineStart + 1;
+    if (count <= 1) {
+      return \`@@ -\${lineStart} +\${lineStart} @@\`;
+    }
+    return \`@@ -\${lineStart},\${count} +\${lineStart},\${count} @@\`;
+  };
+
+  const normalizeFixCodeLocation = (fix) => {
+    const vmLocation = fix?.codeLocation;
+    if (
+      vmLocation &&
+      typeof vmLocation.absoluteFilePath === "string" &&
+      Number.isInteger(vmLocation.lineStart) &&
+      Number.isInteger(vmLocation.lineEnd) &&
+      vmLocation.lineStart > 0 &&
+      vmLocation.lineEnd >= vmLocation.lineStart
+    ) {
+      return vmLocation;
+    }
+
+    const rawLocation = fix?.code_location;
+    const start = rawLocation?.line_range?.start;
+    const end = rawLocation?.line_range?.end;
+    if (
+      rawLocation &&
+      typeof rawLocation.absolute_file_path === "string" &&
+      Number.isInteger(start) &&
+      Number.isInteger(end) &&
+      start > 0 &&
+      end >= start
+    ) {
+      return {
+        absoluteFilePath: rawLocation.absolute_file_path,
+        lineStart: start,
+        lineEnd: end,
+      };
+    }
+
+    return null;
+  };
+
   const extractFixes = (entries) => {
     const fixes = [];
     const skipped = [];
@@ -298,15 +340,24 @@ const DASHBOARD_SCRIPT_TEMPLATE = `
           <div class="panel-title">Fixes Applied</div>
           \${fixes.length
             ? \`<ul class="fix-list">\${fixes
-                .map((fix) => \`
+                .map((fix) => {
+                  const location = normalizeFixCodeLocation(fix);
+                  const filePath = fix.file || location?.absoluteFilePath || "";
+                  const range = location
+                    ? \`<div class="fix-range mono">\${escapeHtml(formatFixRangeHunk(location.lineStart, location.lineEnd))}</div>\`
+                    : "";
+
+                  return \`
                   <li class="fix-item">
                     <div class="fix-pill \${getPriorityPillClass(fix.priority)}">\${escapeHtml(fix.priority)}</div>
                     <div>
                       <div class="fix-title">\${escapeHtml(fix.title)}</div>
-                      <div class="fix-meta muted">\${escapeHtml(fix.file || "")}</div>
+                      <div class="fix-meta muted">\${escapeHtml(filePath)}</div>
+                      \${range}
                     </div>
                   </li>
-                \`)
+                \`;
+                })
                 .join("")}</ul>\`
             : '<div class="muted">No fixes recorded for this session.</div>'}
         </div>

package/src/lib/html/dashboard/styles.ts

@@ -359,6 +359,11 @@ export const DASHBOARD_CSS = `
   .fix-pill-default { background: var(--accent); }
   .fix-title, .skip-title { font-weight: 600; font-size: 13px; }
   .fix-meta, .skip-reason { font-size: 11px; margin-top: 4px; }
+  .fix-range {
+    margin-top: 6px;
+    color: rgba(237, 242, 255, 0.84);
+    font-size: 11px;
+  }
   .muted { color: var(--muted); }
   .empty { padding: 24px; text-align: center; color: var(--muted); border: 1px dashed var(--border); border-radius: 16px; background: rgba(9, 14, 23, 0.6); }
   .empty.tiny { padding: 12px; font-size: 12px; }

package/src/lib/html/dashboard/view-model.ts

@@ -17,6 +17,11 @@ interface FixViewModel {
   priority: FixEntry["priority"];
   title: string;
   file: string;
+  codeLocation?: {
+    absoluteFilePath: string;
+    lineStart: number;
+    lineEnd: number;
+  };
 }
 
 interface SkippedViewModel {
@@ -99,16 +104,45 @@ function formatRoleDisplay(name: string, model: string, reasoning: string): stri
   return `${name} (${details.join(", ")})`;
 }
 
+function toCodeLocationViewModel(fix: FixEntry): FixViewModel["codeLocation"] {
+  const location = fix.code_location;
+  if (!location) {
+    return undefined;
+  }
+
+  const lineStart = location.line_range?.start;
+  const lineEnd = location.line_range?.end;
+  if (
+    typeof location.absolute_file_path !== "string" ||
+    !Number.isInteger(lineStart) ||
+    !Number.isInteger(lineEnd) ||
+    lineStart < 1 ||
+    lineEnd < lineStart
+  ) {
+    return undefined;
+  }
+
+  return {
+    absoluteFilePath: location.absolute_file_path,
+    lineStart,
+    lineEnd,
+  };
+}
+
 function buildSessionViewModel(session: SessionStats): SessionViewModel {
   const { fixes, skipped } = extractFixes(session.entries ?? []);
   const priorities = new Set(fixes.map((fix) => fix.priority));
   const sortedFixes = [...fixes]
     .sort((a, b) => getPriorityRank(a.priority) - getPriorityRank(b.priority))
-    .map((fix) => ({
-      priority: fix.priority,
-      title: fix.title,
-      file: fix.file ?? "",
-    }));
+    .map((fix) => {
+      const codeLocation = toCodeLocationViewModel(fix);
+      return {
+        priority: fix.priority,
+        title: fix.title,
+        file: fix.file ?? "",
+        ...(codeLocation ? { codeLocation } : {}),
+      };
+    });
 
   const sortedSkipped = [...skipped]
     .sort((a, b) => getPriorityRank(a.priority) - getPriorityRank(b.priority))

package/src/lib/html/log/page.ts

@@ -3,8 +3,10 @@ import { getPriorityPillClass } from "@/lib/html/priority";
 import { escapeHtml, formatDate, formatDuration } from "@/lib/html/shared";
 import type {
   AgentSettings,
+  CodeLocation,
   FixEntry,
   IterationEntry,
+  LineRange,
   LogEntry,
   SkippedEntry,
   SystemEntry,
@@ -20,8 +22,64 @@ function isCodeSimplified(systemEntry: SystemEntry | undefined): boolean {
   return Boolean(systemEntry.codeSimplifier || systemEntry.reviewOptions?.simplifier);
 }
 
+function isValidLineRange(lineRange: LineRange | undefined): lineRange is LineRange {
+  if (!lineRange) {
+    return false;
+  }
+
+  return (
+    Number.isInteger(lineRange.start) &&
+    Number.isInteger(lineRange.end) &&
+    lineRange.start > 0 &&
+    lineRange.end >= lineRange.start
+  );
+}
+
+function isValidCodeLocation(location: CodeLocation | null | undefined): location is CodeLocation {
+  if (!location || typeof location.absolute_file_path !== "string") {
+    return false;
+  }
+
+  return isValidLineRange(location.line_range);
+}
+
+function getFixDisplayFile(fix: FixEntry): string {
+  if (fix.file) {
+    return fix.file;
+  }
+
+  const location = fix.code_location;
+  if (isValidCodeLocation(location)) {
+    return location.absolute_file_path;
+  }
+
+  return "";
+}
+
+function formatFixRangeHunk(start: number, end: number): string {
+  const count = end - start + 1;
+  if (count <= 1) {
+    return `@@ -${start} +${start} @@`;
+  }
+  return `@@ -${start},${count} +${start},${count} @@`;
+}
+
+function renderFixRange(fix: FixEntry): string {
+  const location = fix.code_location;
+  if (!isValidCodeLocation(location)) {
+    return "";
+  }
+
+  const lineStart = location.line_range.start;
+  const lineEnd = location.line_range.end;
+
+  return `<div class="fix-range mono">${escapeHtml(formatFixRangeHunk(lineStart, lineEnd))}</div>`;
+}
+
 function renderFixEntry(fix: FixEntry): string {
-  const file = fix.file ? `<span class="muted">${escapeHtml(fix.file)}</span>` : "";
+  const filePath = getFixDisplayFile(fix);
+  const file = filePath ? `<span class="muted">${escapeHtml(filePath)}</span>` : "";
+  const range = renderFixRange(fix);
   const pillClass = getPriorityPillClass(fix.priority);
   return `
     <li class="fix-item">
@@ -29,6 +87,7 @@ function renderFixEntry(fix: FixEntry): string {
       <div>
         <div class="fix-title">${escapeHtml(fix.title)}</div>
         <div class="fix-meta">${file}</div>
+        ${range}
       </div>
     </li>
   `;

package/src/lib/html/log/styles.ts

@@ -84,9 +84,15 @@ export const LOG_CSS = `
   .fix-pill-default { background: var(--accent); }
   .fix-title { font-weight: 600; }
   .fix-meta { font-size: 12px; margin-top: 4px; }
+  .fix-range {
+    margin-top: 6px;
+    color: rgba(237, 242, 255, 0.85);
+    font-size: 12px;
+  }
   .skip-title { font-weight: 600; }
   .skip-reason { font-size: 12px; }
   .muted { color: var(--muted); }
+  .mono { font-family: "Space Grotesk", monospace; }
   .callout {
     margin-top: 12px;
     padding: 12px 16px;

package/src/lib/prompts/fixer.ts

@@ -36,6 +36,7 @@ ${reviewOutput}
    - Prefer one aggregate command; otherwise run available lint -> typecheck -> test -> build.
    - Treat warnings as blocking.
    - Iterate fix + rerun until clean.
+6) For each APPLY item, include the applied code location when available (absolute path + line range).
 
 ## Special rule: tracking-status claims
 Claims like "file is untracked/not committed/missing from git" are SKIP in this pre-commit workflow.
@@ -69,6 +70,10 @@ ${FIX_SUMMARY_START_TOKEN}
       "title": "<one-line title>",
       "priority": "<P0 | P1 | P2 | P3>",
       "file": "<path or null>",
+      "code_location": {
+        "absolute_file_path": "<absolute path>",
+        "line_range": {"start": <int>, "end": <int>}
+      },
       "claim": "<issue claim>",
       "evidence": "<file:line / behavior>",
       "fix": "<what changed>"
@@ -95,6 +100,7 @@ JSON rules:
   - fixes MUST be []
   - skipped MUST contain only SKIP items
 - Include all APPLY items in fixes.
+- For each APPLY item, include code_location when available; otherwise set code_location to null or omit it.
 - Include all SKIP items in skipped with required reason prefix.
 - Use [] when empty.
 - Priority must be exactly P0/P1/P2/P3.

package/src/lib/prompts/review.ts

@@ -19,6 +19,16 @@ const BASE_BRANCH_PROMPT_BACKUP = (branch: string) =>
 const COMMIT_PROMPT = (commitHash: string) =>
   `Review the code changes for the commit ${commitHash}. Provide prioritized, actionable findings.`;
 
+const CUSTOM_FOCUS_PROMPT = (customInstructions: string) =>
+  `Additional review focus from user instructions:\n${customInstructions}`;
+
+function withCustomFocus(instruction: string, customInstructions?: string): string {
+  if (!customInstructions) {
+    return instruction;
+  }
+  return `${instruction}\n\n${CUSTOM_FOCUS_PROMPT(customInstructions)}`;
+}
+
 export interface ReviewerPromptOptions {
   repoPath: string;
   baseBranch?: string;
@@ -26,19 +36,20 @@ export interface ReviewerPromptOptions {
   customInstructions?: string;
 }
 
-/** Priority: commitSha > baseBranch > customInstructions > uncommitted (default) */
+/** Target priority: commitSha > baseBranch > uncommitted (default), with custom focus overlay. */
 export function createReviewerPrompt(options: ReviewerPromptOptions): string {
   const { repoPath, baseBranch, commitSha, customInstructions } = options;
 
   let instruction: string;
 
   if (commitSha) {
-    instruction = COMMIT_PROMPT(commitSha);
+    instruction = withCustomFocus(COMMIT_PROMPT(commitSha), customInstructions);
   } else if (baseBranch) {
     const mergeBaseSha = mergeBaseWithHead(repoPath, baseBranch);
-    instruction = mergeBaseSha
+    const baseInstruction = mergeBaseSha
       ? BASE_BRANCH_PROMPT(baseBranch, mergeBaseSha)
       : BASE_BRANCH_PROMPT_BACKUP(baseBranch);
+    instruction = withCustomFocus(baseInstruction, customInstructions);
   } else if (customInstructions) {
     instruction = customInstructions;
   } else {

package/src/lib/tui/components/Header.tsx

@@ -80,18 +80,21 @@ export function Header({ branch, elapsed, session, projectPath, config }: Header
       flexShrink={0}
     >
       <box flexDirection="row">
-        <box flexDirection="column" width={12}>
+        <box flexDirection="column" width={20}>
           <text>
-            <span fg={TUI_COLORS.brand.logo}> {"   "}║ </span>
+            <span fg={TUI_COLORS.brand.title}>{" ██████╗ ██████╗ "}</span>
           </text>
           <text>
-            <span fg={TUI_COLORS.brand.logo}> ███████ </span>
+            <span fg={TUI_COLORS.brand.title}>{" ██╔══██╗██╔══██╗"}</span>
           </text>
           <text>
-            <span fg={TUI_COLORS.brand.logo}>▐██▃█▃██▌</span>
+            <span fg={TUI_COLORS.brand.title}>{" ██████╔╝██████╔╝"}</span>
           </text>
           <text>
-            <span fg={TUI_COLORS.brand.logo}> ██▅▅▅██ </span>
+            <span fg={TUI_COLORS.brand.title}>{" ██║  ██║██║  ██║"}</span>
+          </text>
+          <text>
+            <span fg={TUI_COLORS.brand.title}>{" ╚═╝  ╚═╝╚═╝  ╚═╝"}</span>
           </text>
         </box>
 

package/src/lib/tui/components/SessionPanel.tsx

@@ -1,5 +1,6 @@
 import { useTerminalDimensions } from "@opentui/react";
 import { useEffect, useMemo, useRef } from "react";
+import { formatReviewType } from "@/lib/format";
 import type { LockData } from "@/lib/lockfile";
 import { TUI_COLORS } from "@/lib/tui/colors";
 import type {
@@ -81,24 +82,6 @@ function toSingleLine(value: string): string {
   return value.replace(/\s+/g, " ").trim();
 }
 
-function formatReviewType(reviewOptions: ReviewOptions | undefined): string {
-  if (!reviewOptions) return "uncommitted changes";
-
-  if (reviewOptions.customInstructions) {
-    return `custom (${toSingleLine(reviewOptions.customInstructions)})`;
-  }
-
-  if (reviewOptions.commitSha) {
-    return `commit (${toSingleLine(reviewOptions.commitSha)})`;
-  }
-
-  if (reviewOptions.baseBranch) {
-    return `base (${toSingleLine(reviewOptions.baseBranch)})`;
-  }
-
-  return "uncommitted changes";
-}
-
 interface FixListProps {
   fixes: FixEntry[];
   showFiles: boolean;
@@ -566,7 +549,7 @@ export function SessionPanel({
       <box flexDirection="row" gap={1}>
         <text fg={TUI_COLORS.text.muted}>Review Type:</text>
         <text fg={TUI_COLORS.text.primary} wrapMode="none">
-          {formatReviewType(reviewOptions)}
+          {toSingleLine(formatReviewType(reviewOptions))}
         </text>
       </box>
 

package/src/lib/types/fix.ts

@@ -1,11 +1,13 @@
 import type { FixDecision, Priority } from "./domain";
 import { VALID_FIX_DECISIONS, VALID_PRIORITIES } from "./domain";
+import type { CodeLocation } from "./review";
 
 export interface FixEntry {
   id: number;
   title: string;
   priority: Priority;
   file?: string | null;
+  code_location?: CodeLocation | null;
   claim: string;
   evidence: string;
   fix: string;
@@ -25,6 +27,37 @@ export interface FixSummary {
   skipped: SkippedEntry[];
 }
 
+function isLineRange(value: unknown): value is CodeLocation["line_range"] {
+  if (typeof value !== "object" || value === null) {
+    return false;
+  }
+
+  const obj = value as Record<string, unknown>;
+
+  return (
+    typeof obj.start === "number" &&
+    Number.isInteger(obj.start) &&
+    obj.start > 0 &&
+    typeof obj.end === "number" &&
+    Number.isInteger(obj.end) &&
+    obj.end >= obj.start
+  );
+}
+
+function isCodeLocation(value: unknown): value is CodeLocation {
+  if (typeof value !== "object" || value === null) {
+    return false;
+  }
+
+  const obj = value as Record<string, unknown>;
+
+  if (typeof obj.absolute_file_path !== "string") {
+    return false;
+  }
+
+  return isLineRange(obj.line_range);
+}
+
 function isFixEntry(value: unknown): value is FixEntry {
   if (typeof value !== "object" || value === null) {
     return false;
@@ -38,6 +71,9 @@ function isFixEntry(value: unknown): value is FixEntry {
     typeof obj.priority === "string" &&
     VALID_PRIORITIES.includes(obj.priority as Priority) &&
     (obj.file === undefined || obj.file === null || typeof obj.file === "string") &&
+    (obj.code_location === undefined ||
+      obj.code_location === null ||
+      isCodeLocation(obj.code_location)) &&
     typeof obj.claim === "string" &&
     typeof obj.evidence === "string" &&
     typeof obj.fix === "string"

package/src/lib/types/index.ts

@@ -31,9 +31,11 @@ export type {
   SystemEntry,
 } from "./log";
 export {
+  type CodeLocation,
   type CodexReviewSummary,
   type Finding,
   isReviewSummary,
+  type LineRange,
   parseCodexReviewText,
   type ReviewSummary,
 } from "./review";

package/src/lib/types/review.ts

@@ -1,12 +1,12 @@
 import type { OverallCorrectness } from "./domain";
 import { VALID_OVERALL_CORRECTNESS } from "./domain";
 
-interface LineRange {
+export interface LineRange {
   start: number;
   end: number;
 }
 
-interface CodeLocation {
+export interface CodeLocation {
   absolute_file_path: string;
   line_range: LineRange;
 }