@ipul-labs/forge 1.16.8 → 1.17.1

package/server/forge/core/ForgeAgent.test.js

@@ -1,179 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
-
-// We test the module-level helpers and ForgeAgent internals by importing the
-// module. The class constructor requires a project, so we mock the provider.
-// For the pure functions (buildCommitSubject, FILE_CAPS), we test them through
-// a thin wrapper or by constructing a ForgeAgent with mocked providers.
-
-// ─── Module-level helpers ────────────────────────────────────────────────────
-
-// buildCommitSubject and FILE_CAPS are not exported, so we test them through
-// ForgeAgent's public surface or by importing the module and inspecting.
-// Since they're internal, we test via the class methods that use them.
-
-import * as configStore from "./configStore.js";
-
-// Mock configStore.getPrimaryProject for ForgeAgent constructor
-vi.spyOn(configStore, "getPrimaryProject").mockReturnValue("group/web-app");
-vi.spyOn(configStore, "get").mockReturnValue({
-	repos: [{ id: "group/web-app", label: "web-app" }],
-	defaults: { promptRules: "" },
-	connections: {},
-});
-
-// Mock the provider factory
-vi.mock("./providers/index.js", () => ({
-	createProvider: () => ({
-		getIssue: vi.fn(),
-		getFileTree: vi.fn().mockResolvedValue([]),
-		getFileContent: vi.fn(),
-		createBranch: vi.fn(),
-		commitFiles: vi.fn(),
-		createMR: vi.fn(),
-		listAllForgeMRs: vi.fn().mockResolvedValue([]),
-		getMRPipelines: vi.fn().mockResolvedValue([]),
-	}),
-}));
-
-// Dynamic import after mocks are set up
-const { ForgeAgent } = await import("./ForgeAgent.js");
-
-describe("ForgeAgent", () => {
-	describe("_capSelections", () => {
-		let agent;
-
-		beforeEach(() => {
-			agent = new ForgeAgent({ project: "group/web-app" });
-		});
-
-		it("applies fix caps (12 total, 8 per repo)", () => {
-			const entries = Array.from({ length: 15 }, (_, i) => ({
-				repo: "group/web-app",
-				path: `src/file${i}.ts`,
-			}));
-
-			const capped = agent._capSelections(entries, "fix");
-			expect(capped).toHaveLength(8);
-		});
-
-		it("applies feature caps (18 total, 15 per repo)", () => {
-			const entries = Array.from({ length: 20 }, (_, i) => ({
-				repo: "group/web-app",
-				path: `src/file${i}.ts`,
-			}));
-
-			const capped = agent._capSelections(entries, "feature");
-			expect(capped).toHaveLength(15);
-		});
-
-		it("enforces total cap across multiple repos for features", () => {
-			const entries = [
-				...Array.from({ length: 12 }, (_, i) => ({
-					repo: "group/web-app",
-					path: `src/file${i}.ts`,
-				})),
-				...Array.from({ length: 12 }, (_, i) => ({
-					repo: "group/api",
-					path: `internal/file${i}.go`,
-				})),
-			];
-
-			const capped = agent._capSelections(entries, "feature");
-			expect(capped).toHaveLength(18);
-		});
-
-		it("defaults to fix caps for unknown kinds", () => {
-			const entries = Array.from({ length: 15 }, (_, i) => ({
-				repo: "group/web-app",
-				path: `src/file${i}.ts`,
-			}));
-
-			const capped = agent._capSelections(entries, "unknown");
-			expect(capped).toHaveLength(8);
-		});
-
-		it("refactor gets same caps as feature", () => {
-			const entries = Array.from({ length: 20 }, (_, i) => ({
-				repo: "group/web-app",
-				path: `src/file${i}.ts`,
-			}));
-
-			const capped = agent._capSelections(entries, "refactor");
-			expect(capped).toHaveLength(15);
-		});
-	});
-
-	describe("_normalizeFileSelections", () => {
-		let agent;
-
-		beforeEach(() => {
-			agent = new ForgeAgent({ project: "group/web-app" });
-		});
-
-		it("normalizes string entries to primary repo", () => {
-			const result = agent._normalizeFileSelections(["src/App.tsx"]);
-			expect(result).toEqual([{ repo: "group/web-app", path: "src/App.tsx" }]);
-		});
-
-		it("passes through {repo, path} objects", () => {
-			const result = agent._normalizeFileSelections([
-				{ repo: "group/api", path: "main.go" },
-			]);
-			expect(result).toEqual([{ repo: "group/api", path: "main.go" }]);
-		});
-
-		it("defaults repo when only path is present", () => {
-			const result = agent._normalizeFileSelections([{ path: "src/a.ts" }]);
-			expect(result).toEqual([{ repo: "group/web-app", path: "src/a.ts" }]);
-		});
-
-		it("filters out null and invalid entries", () => {
-			const result = agent._normalizeFileSelections([null, undefined, {}, "src/a.ts"]);
-			expect(result).toHaveLength(1);
-			expect(result[0].path).toBe("src/a.ts");
-		});
-	});
-
-	describe("_validateSelections", () => {
-		let agent;
-
-		beforeEach(() => {
-			agent = new ForgeAgent({ project: "group/web-app" });
-		});
-
-		it("keeps files that exist in the tree", () => {
-			const entries = [{ repo: "group/web-app", path: "src/App.tsx" }];
-			const trees = [{ repo: "group/web-app", files: ["src/App.tsx", "src/index.ts"] }];
-			const { valid, skipped } = agent._validateSelections(entries, trees);
-			expect(valid).toHaveLength(1);
-			expect(skipped).toHaveLength(0);
-		});
-
-		it("skips files not in the tree", () => {
-			const entries = [{ repo: "group/web-app", path: "src/Missing.tsx" }];
-			const trees = [{ repo: "group/web-app", files: ["src/App.tsx"] }];
-			const { valid, skipped } = agent._validateSelections(entries, trees);
-			expect(valid).toHaveLength(0);
-			expect(skipped).toHaveLength(1);
-			expect(skipped[0]).toContain("not in tree");
-		});
-
-		it("skips files from unknown repos", () => {
-			const entries = [{ repo: "group/unknown", path: "src/a.ts" }];
-			const trees = [{ repo: "group/web-app", files: ["src/a.ts"] }];
-			const { valid, skipped } = agent._validateSelections(entries, trees);
-			expect(valid).toHaveLength(0);
-			expect(skipped[0]).toContain("unknown repo");
-		});
-
-		it("deduplicates entries", () => {
-			const entries = [
-				{ repo: "group/web-app", path: "src/App.tsx" },
-				{ repo: "group/web-app", path: "src/App.tsx" },
-			];
-			const trees = [{ repo: "group/web-app", files: ["src/App.tsx"] }];
-			const { valid } = agent._validateSelections(entries, trees);
-			expect(valid).toHaveLength(1);
-		});
-	});
-});

package/server/forge/core/branchName.js

@@ -1,41 +0,0 @@
-/**
- * server/forge/core/branchName.js
- *
- * Shared branch-name derivation so the validator's local workspace checkout
- * and the provider's remote branch end up on the same name — "follow the code
- * change intention, no timestamps".
- */
-
-const CONV_PREFIX = /^(?:feat|fix|chore|docs|style|refactor|perf|test|build|ci|revert)(?:\([^)]+\))?!?:\s*/i;
-
-const MAX_SLUG_LEN = 32;
-
-export function slugifyIntent(text) {
-	const stripped = String(text || "")
-		.replace(CONV_PREFIX, "")
-		.toLowerCase()
-		.normalize("NFKD")
-		.replace(/[^\w\s-]/g, "")
-		.trim()
-		.replace(/[\s_]+/g, "-")
-		.replace(/-+/g, "-")
-		.replace(/^-+|-+$/g, "");
-	if (stripped.length <= MAX_SLUG_LEN) return stripped;
-	// Truncate on a word boundary so the branch name reads like a phrase,
-	// not a mid-word cutoff ("...div-attri"). Fall back to hard cut if the
-	// first word is already too long.
-	const hardCut = stripped.slice(0, MAX_SLUG_LEN);
-	const lastDash = hardCut.lastIndexOf("-");
-	const soft = lastDash > 0 ? hardCut.slice(0, lastDash) : hardCut;
-	return soft.replace(/-+$/g, "");
-}
-
-/**
- * Branch name derives from the *issue title* rather than the model's mrTitle,
- * so the branch stays stable across refines / CI-fix loops (which rewrite
- * mrTitle to describe the latest edit instead of the overall change body).
- */
-export function branchNameFor(issueIid, analysis) {
-	const slug = slugifyIntent(analysis?.issue?.title || analysis?.mrTitle || "");
-	return slug ? `forge/issue-${issueIid}-${slug}` : `forge/issue-${issueIid}`;
-}

package/server/forge/core/claudeClient.js

@@ -1,378 +0,0 @@
-/**
- * server/forge/core/claudeClient.js
- *
- * Shells out to the `claude` CLI (Claude Code) instead of the Anthropic SDK.
- * Uses your Claude Code subscription — no separate API credits needed.
- *
- * Requires: `claude` CLI installed and authenticated (`claude auth login`)
- *
- * Quota headers are NOT available via the CLI — see `quotaClient.js` for a
- * user-triggered probe that hits the Messages API once to read them.
- */
-
-import { spawn } from "child_process";
-import {
-	SELECT_FILES_SCHEMA,
-	PROPOSE_FIX_SCHEMA,
-	CI_FIX_SCHEMA,
-	REFINE_OR_ASK_SCHEMA,
-	RESOLUTION_CHECK_SCHEMA,
-	buildIdentifyFilesPrompt,
-	buildProposeFixPrompt,
-	buildRefineOrAskPrompt,
-	buildRefineResumePrompt,
-	buildCheckResolutionPrompt,
-	classifyIssue,
-} from "./prompts.js";
-
-// FORGE_MODEL env var overrides the default. Set to a cheaper model to reduce cost.
-// e.g. FORGE_MODEL=claude-haiku-4-5-20251001
-const MODEL = process.env.FORGE_MODEL ?? "claude-sonnet-4-6";
-const FALLBACK_MODEL = process.env.FORGE_FALLBACK_MODEL ?? "claude-haiku-4-5-20251001";
-const CLAUDE_BIN = process.env.CLAUDE_BIN ?? "claude";
-
-// ─── Internal helpers ─────────────────────────────────────────────────────────
-
-function extractJSON(text) {
-	const fenced = text.match(/```(?:json)?\s*([\s\S]*?)```/);
-	if (fenced) return JSON.parse(fenced[1].trim());
-	const start = text.indexOf("{");
-	const end = text.lastIndexOf("}");
-	if (start !== -1 && end !== -1) return JSON.parse(text.slice(start, end + 1));
-	throw new Error("No JSON found in response");
-}
-
-function shortenPath(p, cwd) {
-	if (!p || typeof p !== "string") return p;
-	if (cwd && p.startsWith(cwd + "/")) return p.slice(cwd.length + 1);
-	if (cwd && p === cwd) return ".";
-	return p;
-}
-
-/**
- * Translate a stream-json `tool_use` block into a user-friendly one-line log.
- * Returns null for tools we don't want to surface (filters out noise).
- */
-function formatToolUse(block, cwd) {
-	const name = block?.name;
-	const input = block?.input ?? {};
-	switch (name) {
-		case "Read": {
-			const path = shortenPath(input.file_path, cwd);
-			return path ? `Reading ${path}` : null;
-		}
-		case "Glob": {
-			if (!input.pattern) return null;
-			const where = input.path ? ` in ${shortenPath(input.path, cwd)}` : "";
-			return `Globbing ${input.pattern}${where}`;
-		}
-		case "Grep": {
-			if (!input.pattern) return null;
-			const where = input.path ? ` in ${shortenPath(input.path, cwd)}` : "";
-			return `Grepping ${JSON.stringify(input.pattern)}${where}`;
-		}
-		default:
-			return null;
-	}
-}
-
-async function runClaude(prompt, jsonSchema, systemPrompt, model, options = {}) {
-	const { cwd, allowedTools, onEvent, sessionId, resumeSession, signal } = options;
-	if (signal?.aborted) {
-		const err = new Error("Claude CLI call cancelled before start.");
-		err.name = "AbortError";
-		throw err;
-	}
-	const streaming = typeof onEvent === "function";
-	const args = [
-		"--print",
-		"--output-format", streaming ? "stream-json" : "json",
-		...(streaming ? ["--verbose"] : []),
-		"--model", model ?? MODEL,
-		"--fallback-model", FALLBACK_MODEL,
-		// Kill every tool by default: without this, Claude sometimes picks up
-		// TodoWrite/Task/WebSearch and spends turns on them, returning
-		// stop_reason=end_turn with an empty result. When the caller explicitly
-		// passes allowedTools (e.g. refine with repo access), allow only those.
-		// NOTE: we can't use `--bare` — it disables keychain/OAuth reads, which
-		// breaks the Claude Code subscription auth path.
-		...(allowedTools?.length
-			? ["--allowed-tools", allowedTools.join(",")]
-			: ["--tools", ""]),
-		// Skip user/project/local settings (hooks, agents, CLAUDE.md cascade,
-		// auto-mode, memory). Loading those was causing extra turns that came
-		// back with stop_reason=end_turn and result="". Verified: same prompt
-		// goes from failing at turns=2/3 → passing at turns=1 with this flag.
-		"--setting-sources", "",
-	];
-
-	// Session continuity for multi-turn refine:
-	//   resumeSession=true → --resume <uuid> (replays the conversation so Claude
-	//     keeps its prior reasoning, tool-call history, and prompt cache).
-	//   sessionId without resume → --session-id <uuid> (first turn; fixes the
-	//     UUID so subsequent turns can resume by the id we own).
-	if (resumeSession && sessionId) {
-		args.push("--resume", sessionId);
-	} else if (sessionId) {
-		args.push("--session-id", sessionId);
-	}
-
-	// NOTE: we deliberately do NOT pass --json-schema anymore. The CLI's schema
-	// validator rejects the model's first turn and retries on a second turn; if
-	// that also fails, we get stop_reason=end_turn with result="". The prompts
-	// already tell Claude to emit raw JSON and `extractJSON` is lenient enough
-	// to unwrap a stray fence or preamble. We validate shape on our side instead.
-	void jsonSchema;
-	// On resume, the system prompt is baked into the session from the first
-	// turn — re-passing it would stack a second system message.
-	if (systemPrompt && !resumeSession) args.push("--system-prompt", systemPrompt);
-
-	return new Promise((resolve, reject) => {
-		// Strip ANTHROPIC_API_KEY so the CLI uses the OAuth session
-		// (Claude Code subscription) rather than falling back to API credits.
-		const { ANTHROPIC_API_KEY: _removed, ...cleanEnv } = process.env;
-		const proc = spawn(CLAUDE_BIN, args, {
-			env: { ...cleanEnv, FORCE_COLOR: "0" },
-			stdio: ["pipe", "pipe", "pipe"],
-			...(cwd ? { cwd } : {}),
-		});
-
-		let stdout = "";
-		let stderr = "";
-		let streamBuffer = "";
-		let streamedResult = null;
-
-		const timer = setTimeout(() => {
-			proc.kill();
-			reject(new Error("claude CLI timed out after 5 minutes"));
-		}, 300_000);
-
-		// External abort (e.g. user cancelled the analysis). Kill the subprocess
-		// so we stop billing the subscription / token window for work the user
-		// has already discarded.
-		const onAbort = () => {
-			try { proc.kill(); } catch { /* ignore */ }
-			clearTimeout(timer);
-			const err = new Error("Claude CLI call cancelled.");
-			err.name = "AbortError";
-			reject(err);
-		};
-		signal?.addEventListener("abort", onAbort, { once: true });
-		proc.on("close", () => { signal?.removeEventListener("abort", onAbort); });
-
-		proc.stdout.on("data", (d) => {
-			stdout += d;
-			if (!streaming) return;
-			streamBuffer += d.toString("utf8");
-			let nl;
-			while ((nl = streamBuffer.indexOf("\n")) !== -1) {
-				const line = streamBuffer.slice(0, nl).trim();
-				streamBuffer = streamBuffer.slice(nl + 1);
-				if (!line) continue;
-				let evt;
-				try { evt = JSON.parse(line); } catch { continue; }
-				if (evt.type === "assistant") {
-					const blocks = evt.message?.content ?? [];
-					for (const block of blocks) {
-						if (block.type !== "tool_use") continue;
-						const msg = formatToolUse(block, cwd);
-						if (msg) {
-							try { onEvent({ type: "log", message: msg }); } catch { /* swallow */ }
-						}
-					}
-				} else if (evt.type === "result") {
-					streamedResult = evt;
-				}
-			}
-		});
-		proc.stderr.on("data", (d) => { stderr += d; });
-
-		proc.on("close", (code) => {
-			clearTimeout(timer);
-			if (code !== 0) {
-				try {
-					const wrapper = streamedResult ?? JSON.parse(stdout.trim());
-					const msg = wrapper.result || wrapper.error || stderr.trim() || stdout.trim();
-					reject(new Error(`claude CLI error: ${msg}`));
-				} catch {
-					reject(new Error(`claude CLI error: ${stderr.trim() || stdout.trim()}`));
-				}
-				return;
-			}
-			let wrapper;
-			if (streaming) {
-				if (!streamedResult) {
-					reject(new Error(`claude CLI stream ended without a result event (${stdout.length} bytes). First 200 chars: ${stdout.slice(0, 200)}`));
-					return;
-				}
-				wrapper = streamedResult;
-			} else {
-				try {
-					wrapper = JSON.parse(stdout.trim());
-				} catch {
-					reject(new Error(`claude CLI returned non-JSON output (${stdout.length} bytes). First 200 chars: ${stdout.slice(0, 200)}`));
-					return;
-				}
-			}
-
-			const result = wrapper.result ?? "";
-			if (!result.trim()) {
-				// Claude succeeded but emitted no content. Usually means the model
-				// got stuck in a tool-use loop or was blocked by --disallowed-tools.
-				// Surface a clean, actionable message instead of dumping the wrapper.
-				const stopReason = wrapper.stop_reason ?? "unknown";
-				const turns = wrapper.num_turns ?? 0;
-				const hint = stopReason === "tool_use"
-					? "model tried to use a blocked tool — retry or allow the tool."
-					: "model produced empty output — retry or switch model.";
-				reject(new Error(`Claude returned no content (stop_reason: ${stopReason}, turns: ${turns}). ${hint}`));
-				return;
-			}
-
-			let data;
-			try {
-				data = extractJSON(result);
-			} catch (err) {
-				reject(new Error(`Claude response was not valid JSON — ${err.message}. First 200 chars of result: ${result.slice(0, 200)}`));
-				return;
-			}
-
-			// The CLI echoes the session id it actually used. If it doesn't match
-			// the one we passed (CLI version drift, resume silently forked to a
-			// new session, etc.), warn — the caller should persist the CLI's id
-			// going forward or we'll resume the wrong conversation next turn.
-			const wrapperSessionId = wrapper.session_id ?? null;
-			if (sessionId && wrapperSessionId && wrapperSessionId !== sessionId) {
-				console.warn(
-					`[claudeClient] session_id mismatch: passed ${sessionId}, CLI returned ${wrapperSessionId}. Using CLI value.`,
-				);
-			}
-
-			resolve({
-				data,
-				cost: wrapper.total_cost_usd ?? 0,
-				serviceTier: wrapper.usage?.service_tier ?? "unknown",
-				inputTokens: wrapper.usage?.input_tokens ?? 0,
-				outputTokens: wrapper.usage?.output_tokens ?? 0,
-				cacheCreationTokens: wrapper.usage?.cache_creation_input_tokens ?? 0,
-				cacheReadTokens: wrapper.usage?.cache_read_input_tokens ?? 0,
-				sessionId: wrapperSessionId ?? sessionId ?? null,
-			});
-		});
-
-		proc.stdin.write(prompt, "utf8");
-		proc.stdin.end();
-	});
-}
-
-// ─── Public API ───────────────────────────────────────────────────────────────
-
-export async function checkResolution(issue, diffs, model, opts = {}) {
-	const { data, cost, serviceTier, inputTokens, outputTokens, cacheCreationTokens, cacheReadTokens } = await runClaude(
-		buildCheckResolutionPrompt(issue, diffs),
-		RESOLUTION_CHECK_SCHEMA,
-		"You are Forge, a code review assistant. Assess whether code changes actually fix the reported issue. Be concise and direct.",
-		model,
-		{ signal: opts.signal },
-	);
-	return { ...data, cost, serviceTier, inputTokens, outputTokens, cacheCreationTokens, cacheReadTokens };
-}
-
-export async function identifyRelevantFiles(issue, fileTree, model, opts = {}) {
-	const { data, cost, serviceTier, inputTokens, outputTokens, cacheCreationTokens, cacheReadTokens } = await runClaude(
-		buildIdentifyFilesPrompt(issue, fileTree),
-		SELECT_FILES_SCHEMA,
-		"You are Forge, a code review assistant. You only output raw JSON — never markdown or prose.",
-		model,
-		{ signal: opts.signal },
-	);
-	return { ...data, cost, serviceTier, inputTokens, outputTokens, cacheCreationTokens, cacheReadTokens };
-}
-
-export async function refineOrAsk(issue, fileContents, options = {}, model, onEvent, opts = {}) {
-	const { repoCwd, sessionId, resumeSession } = options;
-	const hasRepoAccess = Boolean(repoCwd);
-	const accessLine = hasRepoAccess
-		? `You ARE running inside a checkout of the repo at the current working directory, with the user's proposed changes already overlaid on the working tree. You can use Read, Glob, and Grep to inspect any file in the repo. Use them when you need context beyond the files listed in the prompt.`
-		: `The prompt lists the file paths in the repo and the full content of the files currently in scope. Treat those as authoritative — do NOT claim you lack repo access. If you need a file that isn't attached, name the path and ask the user to share it.`;
-
-	// When resuming, the system prompt + issue/files/previousChanges are already
-	// in Claude's context from the initial turn — send only what's new.
-	const prompt = resumeSession
-		? buildRefineResumePrompt(options)
-		: buildRefineOrAskPrompt(issue, fileContents, options);
-
-	const runOptions = hasRepoAccess
-		? { cwd: repoCwd, allowedTools: ["Read", "Glob", "Grep"], onEvent }
-		: { onEvent };
-	if (sessionId) runOptions.sessionId = sessionId;
-	if (resumeSession) runOptions.resumeSession = true;
-	if (opts.signal) runOptions.signal = opts.signal;
-
-	const { data, cost, serviceTier, inputTokens, outputTokens, cacheCreationTokens, cacheReadTokens, sessionId: returnedSessionId } = await runClaude(
-		prompt,
-		REFINE_OR_ASK_SCHEMA,
-		`You are Forge, an AI code assistant in a production React/TypeScript/Express codebase.
-${accessLine}
-You auto-classify the user's message: answer questions about state, or produce a refined code change when given a directive.
-Rules when producing changes:
-- Changes must be minimal and surgical.
-- Match the existing code style exactly (TypeScript strict, no barrel imports, Tailwind for UI).
-- For "proposed", always return the COMPLETE file content.
-- Maximum 5 files changed.
-- Never add console.* — use the existing log utility.
-- Never add comments unless the logic is non-obvious.
-Rules when answering:
-- Ground statements in the validation state, the proposed changes, and (when available) files you read from the working tree.
-- If validation hasn't been run yet, say so plainly.
-- Be concise — one to four short paragraphs at most.
-Your final message MUST be a single raw JSON object matching the schema — no prose, no code fences, no preamble.`,
-		model,
-		runOptions,
-	);
-	return { ...data, cost, serviceTier, inputTokens, outputTokens, cacheCreationTokens, cacheReadTokens, sessionId: returnedSessionId };
-}
-
-export async function proposeFix(issue, fileContents, options = {}, model, opts = {}) {
-	const schema = options.ciContext ? CI_FIX_SCHEMA : PROPOSE_FIX_SCHEMA;
-	const kind = classifyIssue(issue);
-	const isFeature = kind === "feature" || kind === "refactor";
-
-	const systemPrompt = isFeature
-		? `You are Forge, an AI code assistant in a production React/TypeScript/Express codebase.
-Rules:
-- Match the existing code style exactly (TypeScript strict, no barrel imports, Tailwind for UI).
-- For "proposed", always return the COMPLETE file content, not a diff or snippet.
-- Maximum 12 files changed. Prioritise core changes over secondary ones — if the feature touches more files than the cap, implement the most important changes and note the rest in "explanation".
-- Never add console.* — use the existing log utility.
-- Never add comments unless the logic is non-obvious.
-- Before returning, verify TypeScript correctness: no implicit any, no missing imports, no type mismatches, all imported symbols must actually be exported from their source.
-Feature discipline:
-- Follow the issue specification. Only make changes the issue asks for — do not clean up, refactor, or "improve" unrelated code.
-- When the issue says to ADD an entry to an existing list or config (nav items, routes, type unions), add the entry — do not restructure the existing code around it.
-- Base new files on the patterns and conventions visible in the relevant files.
-- Use a "feat:" conventional-commit prefix in the title, not "fix:".`
-		: `You are Forge, an AI code assistant in a production React/TypeScript/Express codebase.
-Rules:
-- Changes must be minimal and surgical.
-- Match the existing code style exactly (TypeScript strict, no barrel imports, Tailwind for UI).
-- For "proposed", always return the COMPLETE file content, not a diff or snippet.
-- Maximum 5 files changed.
-- Never add console.* — use the existing log utility.
-- Never add comments unless the logic is non-obvious.
-- Before returning, verify TypeScript correctness: no implicit any, no missing imports, no type mismatches, all imported symbols must actually be exported from their source.
-- If fixing a CI failure, address only what the logs report — do not change unrelated code.
-Root-cause discipline:
-- A fix that only improves an error message or adds a guard at the point of failure is a symptom suppressor, not a fix. Reject it and find the upstream origin of the bad input.
-- When a value is undefined/null at a check, trace one or two frames up (the caller of the function, the prop passed into the component, the handler bound to an onClick). Default function parameters are defeated when the caller passes an unexpected argument (e.g. React forwarding a MouseEvent) — look for that class of bug before patching the consumer.
-- In "explanation", name the origin of the bad value and state why the reported symptom can no longer occur after your change.`;
-
-	const { data, cost, serviceTier, inputTokens, outputTokens, cacheCreationTokens, cacheReadTokens } = await runClaude(
-		buildProposeFixPrompt(issue, fileContents, options),
-		schema,
-		systemPrompt,
-		model,
-		{ signal: opts.signal },
-	);
-	return { ...data, cost, serviceTier, inputTokens, outputTokens, cacheCreationTokens, cacheReadTokens };
-}