@cjhyy/code-shell-core 0.7.1 → 0.8.0

package/dist/lsp/root-path.js

@@ -1,12 +0,0 @@
-import { fileURLToPath } from "node:url";
-/**
- * Convert a `file://` root URI to a platform filesystem path.
- *
- * The LSP manager used `rootUri.replace("file://", "")`, which only strips the
- * first occurrence and leaves percent-encoding intact, and on Windows turns
- * `file:///C:/x` into the invalid `/C:/x`. fileURLToPath does the proper
- * platform conversion (drive letters, percent-decoding, UNC).
- */
-export function rootUriToPath(rootUri) {
-    return fileURLToPath(rootUri);
-}

package/dist/lsp/servers.d.ts

@@ -1,16 +0,0 @@
-/**
- * Built-in language server configurations.
- */
-export interface LSPServerConfig {
-    name: string;
-    language: string;
-    extensions: string[];
-    command: string;
-    args: string[];
-    installHint: string;
-}
-export declare const BUILTIN_LSP_SERVERS: LSPServerConfig[];
-/**
- * Detect which LSP server to use based on file extension.
- */
-export declare function detectLSPServer(filePath: string): LSPServerConfig | undefined;

package/dist/lsp/servers.js

@@ -1,60 +0,0 @@
-/**
- * Built-in language server configurations.
- */
-export const BUILTIN_LSP_SERVERS = [
-    {
-        name: "typescript",
-        language: "typescript",
-        extensions: [".ts", ".tsx", ".js", ".jsx"],
-        command: "typescript-language-server",
-        args: ["--stdio"],
-        installHint: "npm install -g typescript-language-server typescript",
-    },
-    {
-        name: "python",
-        language: "python",
-        extensions: [".py"],
-        command: "pylsp",
-        args: [],
-        installHint: "pip install python-lsp-server",
-    },
-    {
-        name: "go",
-        language: "go",
-        extensions: [".go"],
-        command: "gopls",
-        args: ["serve"],
-        installHint: "go install golang.org/x/tools/gopls@latest",
-    },
-    {
-        name: "rust",
-        language: "rust",
-        extensions: [".rs"],
-        command: "rust-analyzer",
-        args: [],
-        installHint: "rustup component add rust-analyzer",
-    },
-    {
-        name: "json",
-        language: "json",
-        extensions: [".json"],
-        command: "vscode-json-languageserver",
-        args: ["--stdio"],
-        installHint: "npm install -g vscode-langservers-extracted",
-    },
-    {
-        name: "css",
-        language: "css",
-        extensions: [".css", ".scss", ".less"],
-        command: "vscode-css-languageserver",
-        args: ["--stdio"],
-        installHint: "npm install -g vscode-langservers-extracted",
-    },
-];
-/**
- * Detect which LSP server to use based on file extension.
- */
-export function detectLSPServer(filePath) {
-    const ext = filePath.slice(filePath.lastIndexOf("."));
-    return BUILTIN_LSP_SERVERS.find((s) => s.extensions.includes(ext));
-}

package/dist/prompt/sections/coding.md

@@ -1,35 +0,0 @@
-# Coding assistant
-
-The default job in this preset is software engineering in the current working directory.
-
-## Reading before writing
- - Do not propose changes to code you haven't read. If a user asks about or wants you to modify a file, read it first. Understand existing code before suggesting modifications.
- - Read code before editing it, prefer modifying existing files, and avoid unrelated refactors.
-
-## Investigation has a budget — looking is not acting
- - Read, search, and investigate freely, but treat read-only exploration as a means to action, not an end in itself. After about 3 read-only tool calls (Read/Grep/Glob) on the same question, you should either (a) make a change, (b) run a command with side effects (Bash, debug log, repro), or (c) ask the user a specific question. Continuing to read more files usually means your current strategy isn't working — change strategy instead.
- - Never re-read a file (or the same line range of a file) you've already read in this conversation. The content is already in your context. If you've re-derived the same conclusion twice from the same code, the answer isn't in static reading — switch to runtime verification.
- - For bug reports where static analysis hasn't pinpointed the cause in 2-3 reads, the right next step is almost always to add a debug log line and ask the user to reproduce, or to inspect runtime state — not to read more files.
- - Don't narrate the same code path twice in your reasoning. If you've already traced "input X → parser Y → handler Z" once and concluded "this looks correct", stop. Do not re-trace it from a different angle hoping for a different answer. Trust your first analysis and move to verification.
- - These rules are enforced at runtime by an investigation guard. A second read of the same target prepends a `<system-reminder>` to that tool's result; a third read of the same target is hard-blocked with an error. Four or more consecutive read-only calls without a side-effecting action also surface a reminder, and several consecutive turns of read-only work with no text update to the user surface another. When you see one of these reminders, treat it as a signal that your current strategy isn't producing new information — switch to a side-effecting verification (Bash, debug log, repro, edit) or ask the user a specific question rather than reading more.
-
-## Code quality discipline
- - Don't add features, refactor code, or make "improvements" beyond what was asked. A bug fix doesn't need surrounding code cleaned up. A simple feature doesn't need extra configurability. Don't add docstrings, comments, or type annotations to code you didn't change. Only add comments where the logic isn't self-evident.
- - Don't add error handling, fallbacks, or validation for scenarios that can't happen. Trust internal code and framework guarantees. Only validate at system boundaries (user input, external APIs). Don't use feature flags or backwards-compatibility shims when you can just change the code.
- - Don't create helpers, utilities, or abstractions for one-time operations. Don't design for hypothetical future requirements. The right amount of complexity is what the task actually requires — no speculative abstractions, but no half-finished implementations either. Three similar lines of code is better than a premature abstraction.
- - Avoid backwards-compatibility hacks like renaming unused _vars, re-exporting types, adding // removed comments for removed code, etc. If you are certain that something is unused, you can delete it completely.
-
-## Security
- - Be careful not to introduce security vulnerabilities such as command injection, XSS, SQL injection, and other OWASP top 10 vulnerabilities. If you notice that you wrote insecure code, immediately fix it. Prioritize writing safe, secure, and correct code.
-
-## Git
- - When referencing specific functions or pieces of code include the pattern file_path:line_number to allow the user to easily navigate to the source code location.
- - When referencing GitHub issues or pull requests, use the owner/repo#123 format so they render as clickable links.
-
-## Opening URLs and local previews
- - To show the user a web page or a local dev-server preview, print the address in your reply as a bare URL or a markdown link (e.g. `http://localhost:5173` or [预览](http://localhost:5173)) — do NOT wrap it in backticks. The desktop app turns a link in your answer into a clickable link that opens inside its built-in browser panel; backtick-wrapped URLs render as plain code and are not clickable.
- - Do NOT run shell commands to open a URL (`open <url>` on macOS, `xdg-open` on Linux, `start <url>` on Windows). Those launch the user's external system browser and bypass the in-app browser panel. Start the dev server, then just tell the user the address as a link.
-
-## Coding tools
- - Use git/worktree/LSP/notebook tools when they help, but treat them as optional helpers rather than core assumptions.
- - If the user asks for help or feedback, point them to /help and the project repository.

package/dist/quota/credentials.d.ts

@@ -1,3 +0,0 @@
-import type { QuotaCredentials } from "./types.js";
-/** Resolve all tokens from their standard local locations. Never throws. */
-export declare function resolveQuotaCredentials(): Promise<QuotaCredentials>;

package/dist/quota/credentials.js

@@ -1,80 +0,0 @@
-/**
- * Default resolver for the tokens `quota/` needs.
- *
- * This is the ONE platform-specific file in the module — it reads a vendor's
- * Keychain entry / auth.json. A host that stores secrets differently (or a
- * future plugin) can bypass this and build QuotaCredentials itself; core logic
- * in index.ts only depends on the QuotaCredentials shape.
- *
- * All reads are best-effort: a missing token yields `undefined`, never a throw,
- * so a machine without Codex or without Keychain access degrades to "quota
- * unavailable for that provider" instead of failing the whole call.
- */
-import { execFile } from "node:child_process";
-import { readFile } from "node:fs/promises";
-import { homedir } from "node:os";
-import { join } from "node:path";
-function userHome() {
-    return process.env.HOME || process.env.USERPROFILE || homedir();
-}
-/** Read the Codex OAuth token + account id from ~/.codex/auth.json. */
-async function readCodexCreds() {
-    try {
-        const raw = await readFile(join(userHome(), ".codex", "auth.json"), "utf8");
-        const j = JSON.parse(raw);
-        const tokens = (j.tokens ?? {});
-        const token = typeof tokens.access_token === "string" ? tokens.access_token : undefined;
-        const acct = typeof tokens.account_id === "string" ? tokens.account_id : undefined;
-        return { codexAccessToken: token, codexAccountId: acct };
-    }
-    catch {
-        return {};
-    }
-}
-/**
- * Read the Claude Code OAuth access token.
- * macOS: Keychain generic-password "Claude Code-credentials".
- * Other platforms: fall back to a plaintext ~/.claude/.credentials.json if present.
- */
-async function readClaudeCreds() {
-    // macOS Keychain first.
-    if (process.platform === "darwin") {
-        const raw = await runSecurity();
-        const token = parseClaudeToken(raw);
-        if (token)
-            return { claudeAccessToken: token };
-    }
-    // Cross-platform fallback: some installs keep a plaintext credentials file.
-    try {
-        const raw = await readFile(join(userHome(), ".claude", ".credentials.json"), "utf8");
-        const token = parseClaudeToken(raw);
-        if (token)
-            return { claudeAccessToken: token };
-    }
-    catch {
-        /* not present */
-    }
-    return {};
-}
-function parseClaudeToken(raw) {
-    if (!raw)
-        return undefined;
-    try {
-        const j = JSON.parse(raw);
-        const oauth = (j.claudeAiOauth ?? {});
-        return typeof oauth.accessToken === "string" ? oauth.accessToken : undefined;
-    }
-    catch {
-        return undefined;
-    }
-}
-function runSecurity() {
-    return new Promise((resolve) => {
-        execFile("security", ["find-generic-password", "-s", "Claude Code-credentials", "-w"], { timeout: 4000 }, (err, stdout) => resolve(err ? null : stdout.toString().trim()));
-    });
-}
-/** Resolve all tokens from their standard local locations. Never throws. */
-export async function resolveQuotaCredentials() {
-    const [codex, claude] = await Promise.all([readCodexCreds(), readClaudeCreds()]);
-    return { ...codex, ...claude };
-}

package/dist/quota/index.d.ts

@@ -1,36 +0,0 @@
-/**
- * Query remaining quota for the external coding-agent CLIs.
- *
- * Both lookups are verified against the real backends (2026-07-01):
- *   - Codex: GET https://chatgpt.com/backend-api/wham/usage → JSON
- *            rate_limit.{primary_window,secondary_window}.{used_percent,reset_at}.
- *            Zero cost (no message sent).
- *   - Claude: POST /v1/messages (max_tokens:1) → response headers
- *            anthropic-ratelimit-unified-{5h,7d}-{utilization,reset}.
- *            Costs ~1 output token (Claude exposes quota only via response
- *            headers — there is no standalone usage endpoint).
- *
- * The `fetch` and credentials are injected so this is unit-testable offline and
- * so the host owns secret resolution (see types.ts boundary note).
- */
-import type { ProviderQuota, QuotaCredentials, QuotaResult } from "./types.js";
-type FetchLike = typeof fetch;
-export interface CheckQuotaOptions {
-    creds: QuotaCredentials;
-    /** Restrict to specific providers; defaults to both. */
-    providers?: ("claude" | "codex")[];
-    /** Injected for tests; defaults to global fetch. */
-    fetchImpl?: FetchLike;
-    /** Caller's abort signal (user Stop); composed with an internal timeout. */
-    signal?: AbortSignal;
-    timeoutMs?: number;
-}
-/** Codex: GET the usage endpoint and map its JSON to ProviderQuota. */
-export declare function queryCodexQuota(creds: QuotaCredentials, fetchImpl: FetchLike, signal: AbortSignal): Promise<ProviderQuota>;
-/** Claude: POST a 1-token probe and read the unified rate-limit headers. */
-export declare function queryClaudeQuota(creds: QuotaCredentials, fetchImpl: FetchLike, signal: AbortSignal): Promise<ProviderQuota>;
-/** Query both providers (or the subset requested), concurrently. */
-export declare function checkQuota(opts: CheckQuotaOptions): Promise<QuotaResult>;
-/** Render a QuotaResult as a compact human/agent-readable summary. */
-export declare function formatQuota(result: QuotaResult, nowSec: number): string;
-export {};

package/dist/quota/index.js

@@ -1,155 +0,0 @@
-const CODEX_USAGE_URL = "https://chatgpt.com/backend-api/wham/usage";
-const CLAUDE_MESSAGES_URL = "https://api.anthropic.com/v1/messages";
-/** Cheapest always-available model for the 1-token probe. */
-const CLAUDE_PROBE_MODEL = "claude-haiku-4-5-20251001";
-const DEFAULT_TIMEOUT_MS = 8000;
-/** Compose the caller signal with a timeout so a hung backend can't block. */
-function withTimeout(signal, timeoutMs) {
-    const timeout = AbortSignal.timeout(timeoutMs);
-    return signal ? AbortSignal.any([signal, timeout]) : timeout;
-}
-function num(v) {
-    const n = typeof v === "string" ? parseFloat(v) : typeof v === "number" ? v : NaN;
-    return Number.isFinite(n) ? n : null;
-}
-/** Codex: GET the usage endpoint and map its JSON to ProviderQuota. */
-export async function queryCodexQuota(creds, fetchImpl, signal) {
-    if (!creds.codexAccessToken) {
-        return { provider: "codex", error: "no Codex token (是否已 codex login?)" };
-    }
-    let resp;
-    try {
-        resp = await fetchImpl(CODEX_USAGE_URL, {
-            method: "GET",
-            headers: {
-                authorization: `Bearer ${creds.codexAccessToken}`,
-                ...(creds.codexAccountId ? { "chatgpt-account-id": creds.codexAccountId } : {}),
-                originator: "codex_cli_rs",
-                "user-agent": "codex_cli_rs",
-            },
-            signal,
-        });
-    }
-    catch (err) {
-        return { provider: "codex", error: `请求失败: ${err.message}` };
-    }
-    if (!resp.ok) {
-        return { provider: "codex", error: `HTTP ${resp.status}${resp.status === 401 ? " (token 可能已过期)" : ""}` };
-    }
-    let body;
-    try {
-        body = (await resp.json());
-    }
-    catch {
-        return { provider: "codex", error: "响应非 JSON" };
-    }
-    const rl = (body.rate_limit ?? {});
-    const windows = [];
-    const map = [
-        ["primary_window", "5h"],
-        ["secondary_window", "7d"],
-    ];
-    for (const [key, kind] of map) {
-        const w = rl[key];
-        if (!w)
-            continue;
-        const usedPercent = num(w.used_percent);
-        if (usedPercent == null)
-            continue;
-        windows.push({ kind, usedPercent, resetsAt: num(w.reset_at) });
-    }
-    if (windows.length === 0)
-        return { provider: "codex", error: "响应无 rate_limit 窗口" };
-    return {
-        provider: "codex",
-        windows,
-        planType: typeof body.plan_type === "string" ? body.plan_type : null,
-    };
-}
-/** Claude: POST a 1-token probe and read the unified rate-limit headers. */
-export async function queryClaudeQuota(creds, fetchImpl, signal) {
-    if (!creds.claudeAccessToken) {
-        return { provider: "claude", error: "no Claude Code token (是否已登录 Claude Code?)" };
-    }
-    let resp;
-    try {
-        resp = await fetchImpl(CLAUDE_MESSAGES_URL, {
-            method: "POST",
-            headers: {
-                authorization: `Bearer ${creds.claudeAccessToken}`,
-                "anthropic-version": "2023-06-01",
-                "anthropic-beta": "oauth-2025-04-20",
-                "content-type": "application/json",
-            },
-            body: JSON.stringify({
-                model: CLAUDE_PROBE_MODEL,
-                max_tokens: 1,
-                messages: [{ role: "user", content: "hi" }],
-            }),
-            signal,
-        });
-    }
-    catch (err) {
-        return { provider: "claude", error: `请求失败: ${err.message}` };
-    }
-    if (!resp.ok) {
-        return { provider: "claude", error: `HTTP ${resp.status}${resp.status === 401 ? " (token 可能已过期)" : ""}` };
-    }
-    const h = resp.headers;
-    const windows = [];
-    const map = [
-        ["anthropic-ratelimit-unified-5h-utilization", "anthropic-ratelimit-unified-5h-reset", "5h"],
-        ["anthropic-ratelimit-unified-7d-utilization", "anthropic-ratelimit-unified-7d-reset", "7d"],
-    ];
-    for (const [utilKey, resetKey, kind] of map) {
-        const util = num(h.get(utilKey)); // 0–1
-        if (util == null)
-            continue;
-        windows.push({ kind, usedPercent: util * 100, resetsAt: num(h.get(resetKey)) });
-    }
-    if (windows.length === 0)
-        return { provider: "claude", error: "响应头无 rate-limit 字段" };
-    return { provider: "claude", windows };
-}
-/** Query both providers (or the subset requested), concurrently. */
-export async function checkQuota(opts) {
-    const fetchImpl = opts.fetchImpl ?? fetch;
-    const providers = opts.providers ?? ["claude", "codex"];
-    const signal = withTimeout(opts.signal, opts.timeoutMs ?? DEFAULT_TIMEOUT_MS);
-    const result = {};
-    await Promise.all(providers.map(async (p) => {
-        if (p === "codex")
-            result.codex = await queryCodexQuota(opts.creds, fetchImpl, signal);
-        else
-            result.claude = await queryClaudeQuota(opts.creds, fetchImpl, signal);
-    }));
-    return result;
-}
-/** Render a QuotaResult as a compact human/agent-readable summary. */
-export function formatQuota(result, nowSec) {
-    const lines = [];
-    for (const pq of [result.claude, result.codex]) {
-        if (!pq)
-            continue;
-        const name = pq.provider === "claude" ? "Claude Code" : "Codex";
-        if (pq.error || !pq.windows) {
-            lines.push(`${name}: 查询失败 — ${pq.error ?? "unknown"}`);
-            continue;
-        }
-        const plan = pq.planType ? ` [${pq.planType}]` : "";
-        const parts = pq.windows.map((w) => {
-            const reset = w.resetsAt != null ? ` (重置 ${formatReset(w.resetsAt - nowSec)})` : "";
-            return `${w.kind} 用了 ${w.usedPercent.toFixed(0)}%${reset}`;
-        });
-        lines.push(`${name}${plan}: ${parts.join("，")}`);
-    }
-    return lines.length ? lines.join("\n") : "(无可用额度信息)";
-}
-/** "2h13m" / "45m" / "已重置" from a seconds delta. */
-function formatReset(deltaSec) {
-    if (deltaSec <= 0)
-        return "已重置";
-    const h = Math.floor(deltaSec / 3600);
-    const m = Math.floor((deltaSec % 3600) / 60);
-    return h > 0 ? `${h}h${m}m 后` : `${m}m 后`;
-}

package/dist/quota/types.d.ts

@@ -1,48 +0,0 @@
-/**
- * Quota (rate-limit / usage) types for external coding-agent CLIs.
- *
- * DESIGN / BOUNDARY NOTE (see project_core_minimal_harness_business_layer):
- * "how to read Claude Code / Codex remaining quota" is provider-specific
- * BUSINESS policy, not core harness mechanism — it pokes a vendor's Keychain
- * entry, an undocumented usage endpoint, and private response headers. It lives
- * in this self-contained `quota/` module (one interface out: `checkQuota`) on
- * purpose, so it can be lifted wholesale into a plugin later without untangling
- * it from the rest of core. Nothing outside this module should know about
- * Keychain / wham endpoints / `anthropic-ratelimit-*` header names.
- */
-/** A single rolling limit window (e.g. the 5-hour or 7-day window). */
-export interface QuotaWindow {
-    /** Which window this is. */
-    kind: "5h" | "7d";
-    /** Percent of the window's limit already used, 0–100. */
-    usedPercent: number;
-    /** Unix epoch seconds when this window resets, or null if unknown. */
-    resetsAt: number | null;
-}
-/** Quota for one provider (claude | codex). */
-export interface ProviderQuota {
-    provider: "claude" | "codex";
-    /** Present when the lookup succeeded. */
-    windows?: QuotaWindow[];
-    /** Subscription tier, when the source exposes it (e.g. "team", "pro"). */
-    planType?: string | null;
-    /** Human-readable failure reason when the lookup did not succeed. */
-    error?: string;
-}
-export interface QuotaResult {
-    claude?: ProviderQuota;
-    codex?: ProviderQuota;
-}
-/**
- * A credential the host supplies so this module can talk to a vendor backend.
- * Core never reads the Keychain / auth.json itself — the host (desktop main /
- * server) resolves the secret and hands it in. This is the seam that makes the
- * whole module portable to a plugin.
- */
-export interface QuotaCredentials {
-    /** Claude Code OAuth access token (from Keychain "Claude Code-credentials"). */
-    claudeAccessToken?: string;
-    /** Codex OAuth access token + account id (from ~/.codex/auth.json). */
-    codexAccessToken?: string;
-    codexAccountId?: string;
-}

package/dist/quota/types.js

@@ -1,13 +0,0 @@
-/**
- * Quota (rate-limit / usage) types for external coding-agent CLIs.
- *
- * DESIGN / BOUNDARY NOTE (see project_core_minimal_harness_business_layer):
- * "how to read Claude Code / Codex remaining quota" is provider-specific
- * BUSINESS policy, not core harness mechanism — it pokes a vendor's Keychain
- * entry, an undocumented usage endpoint, and private response headers. It lives
- * in this self-contained `quota/` module (one interface out: `checkQuota`) on
- * purpose, so it can be lifted wholesale into a plugin later without untangling
- * it from the rest of core. Nothing outside this module should know about
- * Keychain / wham endpoints / `anthropic-ratelimit-*` header names.
- */
-export {};

package/dist/review/review-prompt.d.ts

@@ -1,28 +0,0 @@
-/**
- * Code-review prompt builder (TODO 7.3). Produces the instruction sent to the
- * model for `/review`. Pure + testable — the TUI/desktop command gathers the
- * diff or file contents and calls this; the model does the actual reasoning.
- *
- * Findings are structured: priority P0–P3, a confidence score, and a precise
- * location, so the output is consistent and (optionally) machine-readable for
- * CI/CD via `--json`.
- */
-export type ReviewDimension = "security" | "performance" | "readability" | "correctness";
-export declare const ALL_DIMENSIONS: ReviewDimension[];
-export interface ReviewPromptOptions {
-    /** The unified diff (incremental) or full file contents to review. */
-    content: string;
-    /** Which dimensions to weigh. Defaults to all four. */
-    dimensions?: ReviewDimension[];
-    /** True when reviewing a diff (incremental); false when reviewing whole files. */
-    incremental?: boolean;
-    /** Emit a machine-readable JSON findings array (for CI/CD) instead of prose. */
-    json?: boolean;
-    /** Optional label for what's under review (e.g. a file path). */
-    label?: string;
-    /** Truncate content to this many chars (default 12000). */
-    maxChars?: number;
-}
-/** Normalize a comma/space list of dimension names; invalid names dropped. */
-export declare function parseDimensions(input: string | undefined): ReviewDimension[];
-export declare function buildReviewPrompt(opts: ReviewPromptOptions): string;

package/dist/review/review-prompt.js

@@ -1,81 +0,0 @@
-/**
- * Code-review prompt builder (TODO 7.3). Produces the instruction sent to the
- * model for `/review`. Pure + testable — the TUI/desktop command gathers the
- * diff or file contents and calls this; the model does the actual reasoning.
- *
- * Findings are structured: priority P0–P3, a confidence score, and a precise
- * location, so the output is consistent and (optionally) machine-readable for
- * CI/CD via `--json`.
- */
-export const ALL_DIMENSIONS = [
-    "correctness",
-    "security",
-    "performance",
-    "readability",
-];
-const DIMENSION_LABEL = {
-    security: "安全 (injection, authz, secret leakage, unsafe deserialization)",
-    performance: "性能 (N+1, needless allocation, blocking I/O, complexity)",
-    readability: "可读性 (naming, structure, dead code, comment accuracy)",
-    correctness: "正确性 (logic bugs, edge cases, error handling, races)",
-};
-/** Normalize a comma/space list of dimension names; invalid names dropped. */
-export function parseDimensions(input) {
-    if (!input)
-        return ALL_DIMENSIONS;
-    const wanted = input
-        .split(/[\s,]+/)
-        .map((s) => s.trim().toLowerCase())
-        .filter(Boolean);
-    const picked = ALL_DIMENSIONS.filter((d) => wanted.includes(d));
-    return picked.length > 0 ? picked : ALL_DIMENSIONS;
-}
-const PRIORITY_GUIDE = "Priority: P0 = must fix before merge (data loss, security hole, crash); " +
-    "P1 = should fix (likely bug, real risk); " +
-    "P2 = nice to fix (smell, minor inefficiency); " +
-    "P3 = optional (style, nit).";
-export function buildReviewPrompt(opts) {
-    const dims = opts.dimensions?.length ? opts.dimensions : ALL_DIMENSIONS;
-    const maxChars = opts.maxChars ?? 12_000;
-    const body = opts.content.length > maxChars
-        ? opts.content.slice(0, maxChars) + "\n…(truncated)…"
-        : opts.content;
-    const fence = opts.incremental === false ? "" : "diff";
-    const what = opts.incremental === false ? "code" : "code changes (diff)";
-    const subject = opts.label ? ` for \`${opts.label}\`` : "";
-    const dimLines = dims.map((d) => `  - ${DIMENSION_LABEL[d]}`).join("\n");
-    if (opts.json) {
-        return [
-            `Review the following ${what}${subject} across these dimensions:`,
-            dimLines,
-            ``,
-            PRIORITY_GUIDE,
-            ``,
-            `Respond with ONLY a JSON object (no prose, no markdown fence) of the form:`,
-            `{"summary": string, "findings": [{"priority": "P0"|"P1"|"P2"|"P3", ` +
-                `"dimension": ${dims.map((d) => `"${d}"`).join("|")}, ` +
-                `"confidence": number /*0..1*/, "location": string /*file:line*/, ` +
-                `"title": string, "detail": string, "suggestion": string}]}`,
-            `Order findings by priority then confidence. Omit a finding rather than guess.`,
-            ``,
-            "```" + fence,
-            body,
-            "```",
-        ].join("\n");
-    }
-    return [
-        `Review the following ${what}${subject} across these dimensions:`,
-        dimLines,
-        ``,
-        PRIORITY_GUIDE,
-        ``,
-        `For each finding, give: **[P0-P3]** a short title, the location (file:line), ` +
-            `your confidence (low/med/high), what's wrong, and a concrete fix.`,
-        `Start with a one-line summary, then list findings ordered by priority. ` +
-            `If nothing warrants a finding, say so plainly. Don't pad with nits to look thorough.`,
-        ``,
-        "```" + fence,
-        body,
-        "```",
-    ].join("\n");
-}