newpr 1.0.19 → 1.0.21

package/src/stack/partition.ts

@@ -13,6 +13,7 @@ interface FileSummaryInput {
 export interface PartitionInput {
 	groups: FileGroup[];
 	changed_files: string[];
+	group_order_hint?: string[];
 }
 
 export interface AmbiguityReport {
@@ -58,54 +59,71 @@ export function buildStackPartitionPrompt(
 	groups: FileGroup[],
 	fileSummaries: FileSummaryInput[],
 	commits: PrCommit[],
+	groupOrderHint?: string[],
 ): { system: string; user: string } {
+	const summaryByPath = new Map(fileSummaries.map((f) => [f.path, f.summary]));
+
 	const groupDescriptions = groups
-		.map((g) => `- "${g.name}" (${g.type}): ${g.description}`)
+		.map((g) => {
+			const canonicalFiles = g.files.slice(0, 8);
+			const fileHints = canonicalFiles.length > 0
+				? `\n    Representative files: ${canonicalFiles.join(", ")}${g.files.length > 8 ? ` (+${g.files.length - 8} more)` : ""}`
+				: "";
+			return `- "${g.name}" (${g.type}): ${g.description}${fileHints}`;
+		})
 		.join("\n");
 
+	const buildFileEntry = (path: string, extra = ""): string => {
+		const summary = summaryByPath.get(path);
+		const summaryNote = summary ? ` — ${summary}` : "";
+		return `- ${path}${summaryNote}${extra}`;
+	};
+
 	const ambiguousSection = ambiguous.length > 0
-		? `\n\nAmbiguous files (appear in multiple groups):\n${ambiguous
-				.map((a) => `- ${a.path} → in groups: ${a.groups.join(", ")}`)
+		? `\n\nAmbiguous files (appear in multiple groups — pick the BEST ONE):\n${ambiguous
+				.map((a) => buildFileEntry(a.path, ` → candidate groups: ${a.groups.join(", ")}`))
 				.join("\n")}`
 		: "";
 
 	const unassignedSection = unassigned.length > 0
-		? `\n\nUnassigned files (not in any group):\n${unassigned.map((f) => `- ${f}`).join("\n")}`
+		? `\n\nUnassigned files (assign to the most relevant group — prefer an EXISTING group over creating Shared Foundation):\n${unassigned.map((f) => buildFileEntry(f)).join("\n")}`
 		: "";
 
-	const fileSummarySection = fileSummaries.length > 0
-		? `\n\nFile summaries:\n${fileSummaries.map((f) => `- ${f.path}: ${f.summary}`).join("\n")}`
+	const commitSection = commits.length > 0
+		? `\n\nCommit history (use to understand intent of each change):\n${commits.map((c) => `- ${c.sha.substring(0, 7)} ${c.message}`).join("\n")}`
 		: "";
 
-	const commitSection = commits.length > 0
-		? `\n\nCommit history:\n${commits.map((c) => `- ${c.sha.substring(0, 7)} ${c.message}`).join("\n")}`
+	const orderHintSection = groupOrderHint && groupOrderHint.length > 1
+		? `\n\nSuggested PR stack order (foundation → integration, for context only — use to judge which group a file logically "enables"):\n${groupOrderHint.map((g, i) => `${i + 1}. ${g}`).join("\n")}`
 		: "";
 
 	return {
-		system: `You are a code organization expert. Your task is to assign each file to EXACTLY ONE group for PR stacking.
+		system: `You are a senior engineer helping organize a pull request into a reviewable stack.
+
+Your task: assign each ambiguous or unassigned file to EXACTLY ONE group.
 
 Rules:
-1. Each file must be assigned to exactly one group
-2. Do not change files that are already exclusively assigned
-3. For ambiguous files, choose the group where the file's changes are most relevant
-4. For unassigned files, assign them to the most appropriate existing group
-5. You may create a "Shared Foundation" group ONLY if files truly don't fit any existing group
-6. Respond ONLY with a JSON object
-
-Response format:
+1. Assign every file to exactly one group — no file may be skipped.
+2. Use file path structure, file summary, and commit messages to judge relevance.
+3. An unassigned file that touches shared utilities (e.g. schema types, constants, index re-exports) belongs to the group that INTRODUCES or PRIMARILY USES those utilities in this PR.
+4. Prefer existing groups. Only create a "Shared Foundation" group if a file is genuinely orthogonal to ALL existing groups AND is depended on by multiple groups.
+5. Do NOT dump hard-to-classify files into Shared Foundation — that creates an oversized catch-all PR that defeats the purpose of stacking.
+6. When in doubt, pick the group whose file paths are most similar (same directory prefix, same feature area).
+
+Response format (JSON only):
 {
   "assignments": [
-    { "path": "file.ts", "group": "group-name", "reason": "brief reason" }
+    { "path": "file.ts", "group": "exact-group-name", "reason": "one sentence" }
   ],
   "shared_foundation": null
 }
 
-If creating a Shared Foundation group:
+Only include shared_foundation if truly necessary:
 {
   "assignments": [...],
-  "shared_foundation": { "name": "Shared Foundation", "description": "Common infrastructure changes", "files": [...] }
+  "shared_foundation": { "name": "Shared Foundation", "description": "why this is shared", "files": ["path1", "path2"] }
 }`,
-		user: `Groups:\n${groupDescriptions}${ambiguousSection}${unassignedSection}${fileSummarySection}${commitSection}\n\nAssign each ambiguous/unassigned file to exactly one group.`,
+		user: `Groups:\n${groupDescriptions}${ambiguousSection}${unassignedSection}${commitSection}${orderHintSection}\n\nAssign every listed file to exactly one group. Prefer existing groups over Shared Foundation.`,
 	};
 }
 
@@ -115,6 +133,7 @@ export async function partitionGroups(
 	changedFiles: string[],
 	fileSummaries: FileSummaryInput[],
 	commits: PrCommit[],
+	groupOrderHint?: string[],
 ): Promise<PartitionResult> {
 	const report = detectAmbiguousPaths({ groups, changed_files: changedFiles });
 
@@ -133,6 +152,7 @@ export async function partitionGroups(
 		groups,
 		fileSummaries,
 		commits,
+		groupOrderHint,
 	);
 
 	const response = await client.complete(prompt.system, prompt.user);

package/src/stack/symbol-flow.ts

@@ -0,0 +1,229 @@
+import { parse } from "meriyah";
+
+const ANALYZABLE_EXTENSIONS = new Set([
+	".ts", ".tsx", ".js", ".jsx", ".mjs", ".cjs",
+]);
+
+export interface NamedImport {
+	from: string;
+	names: string[];
+}
+
+export interface FileSymbols {
+	path: string;
+	exports: string[];
+	imports: NamedImport[];
+}
+
+type AstNode = { type: string;[key: string]: unknown };
+type AstProgram = { type: "Program"; body: AstNode[] };
+
+function safeParseAst(source: string): AstProgram | null {
+	try {
+		return parse(source, { module: true, jsx: true, next: true, raw: false, ranges: false }) as unknown as AstProgram;
+	} catch {
+	}
+	try {
+		return parse(source, { module: true, jsx: false, next: true, raw: false, ranges: false }) as unknown as AstProgram;
+	} catch {
+	}
+	return null;
+}
+
+function getId(node: AstNode): string | null {
+	const id = node["id"] as { name: string } | undefined;
+	return id?.name ?? null;
+}
+
+function extractFromAst(
+	ast: AstProgram,
+	resolveSpecifier: (s: string) => string | null,
+): { exports: string[]; imports: NamedImport[] } {
+	const exports: string[] = [];
+	const importsMap = new Map<string, string[]>();
+
+	const addImport = (from: string | null, names: string[]) => {
+		if (!from) return;
+		const arr = importsMap.get(from) ?? [];
+		for (const n of names) arr.push(n);
+		importsMap.set(from, arr);
+	};
+
+	for (const node of ast.body) {
+		if (node.type === "ExportNamedDeclaration") {
+			const decl = node["declaration"] as AstNode | null;
+			if (decl) {
+				const name = getId(decl);
+				if (name) exports.push(name);
+				if (decl.type === "VariableDeclaration") {
+					for (const d of (decl["declarations"] as AstNode[])) {
+						const id = d["id"] as AstNode;
+						if (id.type === "Identifier") exports.push(id["name"] as string);
+					}
+				}
+			}
+			const specifiers = node["specifiers"] as Array<{ exported: { name: string } }>;
+			for (const s of specifiers ?? []) exports.push(s.exported.name);
+
+			const src = (node["source"] as { value: string } | null)?.value;
+			if (src) {
+				const from = resolveSpecifier(src);
+				const names = (specifiers ?? []).map((s) => s.exported.name);
+				addImport(from, names);
+			}
+		} else if (node.type === "ExportDefaultDeclaration") {
+			exports.push("default");
+		} else if (node.type === "ExportAllDeclaration") {
+			const src = (node["source"] as { value: string } | null)?.value;
+			if (src) addImport(resolveSpecifier(src), ["*"]);
+		} else if (node.type === "ImportDeclaration") {
+			const src = (node["source"] as { value: string }).value;
+			const from = resolveSpecifier(src);
+			const names = (node["specifiers"] as AstNode[]).map((s) => {
+				if (s.type === "ImportSpecifier") return (s["imported"] as { name: string }).name;
+				if (s.type === "ImportDefaultSpecifier") return "default";
+				if (s.type === "ImportNamespaceSpecifier") return "*";
+				return "";
+			}).filter(Boolean);
+			addImport(from, names);
+		}
+	}
+
+	const imports: NamedImport[] = [];
+	for (const [from, names] of importsMap) {
+		imports.push({ from, names: [...new Set(names)] });
+	}
+	return { exports: [...new Set(exports)], imports };
+}
+
+const EXPORT_RE = /\bexport\s+(?:default\s+)?(?:(?:async\s+)?function\s*\*?\s*(\w+)|class\s+(\w+)|const\s+(\w+)|let\s+(\w+)|var\s+(\w+)|type\s+(\w+)|interface\s+(\w+)|enum\s+(\w+))/g;
+const NAMED_EXPORT_RE = /\bexport\s*\{([^}]+)\}/g;
+const IMPORT_FROM_RE = /\bimport\s+(?:type\s+)?(?:\*\s+as\s+\w+|(?:\w+\s*,\s*)?\{([^}]*)\}|\w+)\s+from\s+['"]([^'"]+)['"]/g;
+const EXPORT_FROM_RE = /\bexport\s+(?:type\s+)?\{([^}]*)\}\s+from\s+['"]([^'"]+)['"]/g;
+
+function extractFallback(
+	source: string,
+	resolveSpecifier: (s: string) => string | null,
+): { exports: string[]; imports: NamedImport[] } {
+	const exports: string[] = [];
+	const importsMap = new Map<string, string[]>();
+
+	let m: RegExpExecArray | null;
+
+	EXPORT_RE.lastIndex = 0;
+	while ((m = EXPORT_RE.exec(source)) !== null) {
+		const name = m[1] ?? m[2] ?? m[3] ?? m[4] ?? m[5] ?? m[6] ?? m[7] ?? m[8];
+		if (name) exports.push(name);
+	}
+
+	NAMED_EXPORT_RE.lastIndex = 0;
+	while ((m = NAMED_EXPORT_RE.exec(source)) !== null) {
+		for (const spec of m[1]!.split(",")) {
+			const name = spec.trim().split(/\s+as\s+/).pop()?.trim();
+			if (name) exports.push(name);
+		}
+	}
+
+	const addFromSpec = (rawNames: string | null, rawSpecifier: string | undefined) => {
+		if (!rawSpecifier) return;
+		const from = resolveSpecifier(rawSpecifier);
+		if (!from) return;
+		const names = rawNames
+			? rawNames.split(",").map((s) => s.trim().split(/\s+as\s+/)[0]?.trim() ?? "").filter(Boolean)
+			: [];
+		const arr = importsMap.get(from) ?? [];
+		arr.push(...names);
+		importsMap.set(from, arr);
+	};
+
+	IMPORT_FROM_RE.lastIndex = 0;
+	while ((m = IMPORT_FROM_RE.exec(source)) !== null) addFromSpec(m[1] ?? null, m[2]);
+
+	EXPORT_FROM_RE.lastIndex = 0;
+	while ((m = EXPORT_FROM_RE.exec(source)) !== null) addFromSpec(m[1] ?? null, m[2]);
+
+	const imports: NamedImport[] = [];
+	for (const [from, names] of importsMap) {
+		imports.push({ from, names: [...new Set(names)] });
+	}
+	return { exports: [...new Set(exports)], imports };
+}
+
+function fileExt(f: string): string {
+	const dot = f.lastIndexOf(".");
+	return dot >= 0 ? f.slice(dot) : "";
+}
+
+function resolveRelative(fromFile: string, specifier: string): string {
+	const parts = fromFile.split("/");
+	parts.pop();
+	for (const seg of specifier.split("/")) {
+		if (seg === "..") parts.pop();
+		else if (seg !== ".") parts.push(seg);
+	}
+	return parts.join("/");
+}
+
+function resolveToFile(candidate: string, fileSet: Set<string>): string | null {
+	if (fileSet.has(candidate)) return candidate;
+	for (const ext of ANALYZABLE_EXTENSIONS) {
+		if (fileSet.has(`${candidate}${ext}`)) return `${candidate}${ext}`;
+	}
+	for (const idx of ["index.ts", "index.tsx", "index.js"]) {
+		if (fileSet.has(`${candidate}/${idx}`)) return `${candidate}/${idx}`;
+	}
+	return null;
+}
+
+export async function extractSymbols(
+	repoPath: string,
+	headSha: string,
+	filePaths: string[],
+): Promise<Map<string, FileSymbols>> {
+	const fileSet = new Set(filePaths);
+	const analyzable = filePaths.filter((f) => ANALYZABLE_EXTENSIONS.has(fileExt(f)));
+	const result = new Map<string, FileSymbols>();
+
+	await Promise.all(analyzable.map(async (filePath) => {
+		const r = await Bun.$`git -C ${repoPath} show ${headSha}:${filePath}`.quiet().nothrow();
+		if (r.exitCode !== 0) return;
+
+		const source = r.stdout.toString();
+
+		const resolveSpecifier = (specifier: string): string | null => {
+			if (!specifier.startsWith("./") && !specifier.startsWith("../")) return null;
+			const candidate = resolveRelative(filePath, specifier);
+			return resolveToFile(candidate, fileSet);
+		};
+
+		let symbols: { exports: string[]; imports: NamedImport[] } | null = null;
+		const ast = safeParseAst(source);
+		if (ast) {
+			try {
+				symbols = extractFromAst(ast, resolveSpecifier);
+			} catch {
+			}
+		}
+		if (!symbols) symbols = extractFallback(source, resolveSpecifier);
+
+		result.set(filePath, {
+			path: filePath,
+			exports: symbols.exports,
+			imports: symbols.imports,
+		});
+	}));
+
+	return result;
+}
+
+export function buildSymbolIndex(symbolMap: Map<string, FileSymbols>): Map<string, string[]> {
+	const exportedBy = new Map<string, string[]>();
+	for (const [file, info] of symbolMap) {
+		for (const sym of info.exports) {
+			const arr = exportedBy.get(sym) ?? [];
+			arr.push(file);
+			exportedBy.set(sym, arr);
+		}
+	}
+	return exportedBy;
+}

package/src/web/client/components/AnalyticsConsent.tsx

@@ -37,21 +37,21 @@ export function AnalyticsConsent({ onDone }: { onDone: () => void }) {
 							<BarChart3 className="h-5 w-5 text-blue-500" />
 						</div>
 						<div>
-							<h2 className="text-sm font-semibold">Help improve newpr</h2>
-							<p className="text-[11px] text-muted-foreground">Anonymous usage analytics</p>
-						</div>
+						<h2 className="text-base font-semibold">Help improve newpr</h2>
+						<p className="text-xs text-muted-foreground">Anonymous usage analytics</p>
 					</div>
+				</div>
 
-					<p className="text-xs text-muted-foreground leading-relaxed mb-3">
+				<p className="text-sm text-muted-foreground leading-relaxed mb-3">
 						We'd like to collect anonymous usage data to understand how newpr is used and improve the experience.
 					</p>
 
 					<div className="rounded-lg bg-muted/40 px-3.5 py-2.5 space-y-1.5 mb-4">
 						<div className="flex items-start gap-2">
 							<Shield className="h-3.5 w-3.5 text-emerald-500 mt-0.5 shrink-0" />
-							<div className="text-[11px] text-muted-foreground leading-relaxed">
-								<p className="font-medium text-foreground/80 mb-1">What we collect:</p>
-								<ul className="space-y-0.5 list-disc list-inside text-[10.5px]">
+						<div className="text-xs text-muted-foreground leading-relaxed">
+							<p className="font-medium text-foreground/80 mb-1">What we collect:</p>
+							<ul className="space-y-0.5 list-disc list-inside text-xs">
 									<li>Feature usage (which tabs, buttons, and actions you use)</li>
 									<li>Performance metrics (analysis duration, error rates)</li>
 									<li>Basic device info (browser, screen size)</li>
@@ -60,9 +60,9 @@ export function AnalyticsConsent({ onDone }: { onDone: () => void }) {
 						</div>
 						<div className="flex items-start gap-2 pt-1">
 							<Shield className="h-3.5 w-3.5 text-emerald-500 mt-0.5 shrink-0" />
-							<div className="text-[11px] text-muted-foreground leading-relaxed">
-								<p className="font-medium text-foreground/80 mb-1">What we never collect:</p>
-								<ul className="space-y-0.5 list-disc list-inside text-[10.5px]">
+						<div className="text-xs text-muted-foreground leading-relaxed">
+							<p className="font-medium text-foreground/80 mb-1">What we never collect:</p>
+							<ul className="space-y-0.5 list-disc list-inside text-xs">
 									<li>PR content, code, or commit messages</li>
 									<li>Chat messages or review comments</li>
 									<li>API keys, tokens, or personal data</li>
@@ -71,7 +71,7 @@ export function AnalyticsConsent({ onDone }: { onDone: () => void }) {
 						</div>
 					</div>
 
-					<p className="text-[10px] text-muted-foreground/50 mb-4">
+					<p className="text-xs text-muted-foreground/50 mb-4">
 						Powered by Google Analytics. You can change this anytime in Settings.
 					</p>
 				</div>
@@ -80,14 +80,14 @@ export function AnalyticsConsent({ onDone }: { onDone: () => void }) {
 					<button
 						type="button"
 						onClick={handleDecline}
-						className="flex-1 px-4 py-3 text-xs text-muted-foreground hover:bg-muted/50 transition-colors"
+						className="flex-1 px-4 py-3 text-sm text-muted-foreground hover:bg-muted/50 transition-colors"
 					>
 						Decline
 					</button>
 					<button
 						type="button"
 						onClick={handleAccept}
-						className="flex-1 px-4 py-3 text-xs font-medium bg-foreground text-background hover:opacity-90 transition-opacity"
+						className="flex-1 px-4 py-3 text-sm font-medium bg-foreground text-background hover:opacity-90 transition-opacity"
 					>
 						Accept
 					</button>

package/src/web/client/components/ChatSection.tsx

@@ -111,7 +111,7 @@ function AssistantMessage({ segments, activeToolName, isStreaming, onAnchorClick
 				const isTrailingSegment = i === segments.length - 1;
 				const isStreamingTail = isStreaming && isTrailingSegment;
 				return (
-					<div key={`text-${i}`} className="text-xs leading-relaxed">
+					<div key={`text-${i}`} className="text-sm leading-relaxed">
 						{isStreamingTail ? (
 							<Markdown streaming onAnchorClick={onAnchorClick} activeId={activeId}>{seg.content}</Markdown>
 						) : (

package/src/web/client/components/DetailPane.tsx

@@ -146,19 +146,19 @@ function FileDetail({
 				{loading && (
 					<div className="flex items-center justify-center py-16 gap-2">
 						<Loader2 className="h-3.5 w-3.5 animate-spin text-muted-foreground/40" />
-						<span className="text-xs text-muted-foreground/50">Loading diff</span>
+						<span className="text-sm text-muted-foreground/50">Loading diff</span>
 					</div>
 				)}
 				{error && (
 					<div className="flex flex-col items-center justify-center py-16 gap-3">
 						<div className="flex items-center gap-2 text-destructive">
 							<AlertCircle className="h-3.5 w-3.5" />
-							<p className="text-xs">{error}</p>
-						</div>
-						<button
-							type="button"
-							onClick={fetchPatch}
-							className="text-[11px] text-muted-foreground/50 hover:text-foreground transition-colors"
+					<p className="text-sm">{error}</p>
+					</div>
+					<button
+						type="button"
+						onClick={fetchPatch}
+						className="text-xs text-muted-foreground/50 hover:text-foreground transition-colors"
 						>
 							Retry
 						</button>
@@ -203,8 +203,8 @@ export function DetailPane({
 				<div className="shrink-0 flex items-center justify-between gap-2 px-4 h-12 border-b">
 					<div className="flex items-center gap-2 min-w-0">
 						<span className={`h-2 w-2 rounded-full shrink-0 ${TYPE_DOT[g.type] ?? TYPE_DOT.chore}`} />
-						<span className="text-xs font-medium truncate">{g.name}</span>
-						<span className="text-[10px] text-muted-foreground/30">{g.type}</span>
+					<span className="text-sm font-medium truncate">{g.name}</span>
+					<span className="text-xs text-muted-foreground/30">{g.type}</span>
 					</div>
 					{onClose && (
 						<button type="button" onClick={onClose} className="flex h-6 w-6 items-center justify-center rounded-md text-muted-foreground/40 hover:text-foreground hover:bg-accent/40 transition-colors">

package/src/web/client/components/DiffViewer.tsx

@@ -257,11 +257,11 @@ function CommentCard({
 				) : (
 					<div className="h-4 w-4 rounded-full bg-muted-foreground/20 shrink-0" />
 				)}
-				<span className="text-[11px] font-medium text-foreground/90">{comment.author}</span>
-				<span className="text-[10px] text-muted-foreground/60">{formatTimeAgo(comment.createdAt)}</span>
-				{comment.startLine != null && comment.startLine !== comment.line && (
-					<span className="text-[10px] text-muted-foreground/40 font-mono">L{comment.startLine}-{comment.line}</span>
-				)}
+			<span className="text-xs font-medium text-foreground/90">{comment.author}</span>
+			<span className="text-xs text-muted-foreground/60">{formatTimeAgo(comment.createdAt)}</span>
+			{comment.startLine != null && comment.startLine !== comment.line && (
+				<span className="text-xs text-muted-foreground/40 font-mono">L{comment.startLine}-{comment.line}</span>
+			)}
 				{comment.githubUrl && (
 					<a href={comment.githubUrl} target="_blank" rel="noopener noreferrer" className="text-muted-foreground/40 hover:text-foreground/60 transition-colors">
 						<ExternalLink className="h-2.5 w-2.5" />
@@ -318,7 +318,7 @@ function CommentCard({
 				</div>
 			</div>
 			) : (
-				<p className="text-[12px] text-foreground/80 whitespace-pre-wrap break-words leading-[1.6] pl-[22px]">{comment.body}</p>
+				<p className="text-sm text-foreground/80 whitespace-pre-wrap break-words leading-[1.6] pl-[22px]">{comment.body}</p>
 			)}
 		</div>
 	);
@@ -374,7 +374,7 @@ function CommentForm({
 					<button
 						type="button"
 						onClick={onCancel}
-						className="text-[11px] text-muted-foreground/60 hover:text-foreground/80 px-2 py-1 rounded-md transition-colors"
+						className="text-xs text-muted-foreground/60 hover:text-foreground/80 px-2 py-1 rounded-md transition-colors"
 					>
 						Cancel
 					</button>
@@ -383,7 +383,7 @@ function CommentForm({
 						onClick={handleSubmit}
 						disabled={!hasContent || submitting}
 						className={`
-							text-[11px] font-medium px-3 py-1 rounded-md transition-all
+							text-xs font-medium px-3 py-1 rounded-md transition-all
 							${hasContent && !submitting
 								? "bg-foreground text-background hover:bg-foreground/90"
 								: "bg-muted text-muted-foreground/40 cursor-not-allowed"}
@@ -391,7 +391,7 @@ function CommentForm({
 					>
 						{submitting ? "Posting..." : "Comment"}
 					</button>
-					<kbd className="hidden sm:flex items-center gap-0.5 text-[10px] text-muted-foreground/40 select-none">
+					<kbd className="hidden sm:flex items-center gap-0.5 text-xs text-muted-foreground/40 select-none">
 						{modKey}<CornerDownLeft className="h-2.5 w-2.5" />
 					</kbd>
 				</div>
@@ -481,7 +481,7 @@ function AskAiPanel({
 		<div className="px-3 py-2.5 font-sans">
 			<div className="space-y-2.5">
 				<div className="flex items-center justify-between">
-					<div className="flex items-center gap-1.5 text-[11px] text-muted-foreground/60">
+					<div className="flex items-center gap-1.5 text-xs text-muted-foreground/60">
 						<Sparkles className="h-3 w-3" />
 						<span>AI Analysis</span>
 						<span className="text-muted-foreground/30">L{startLine}{endLine !== startLine ? `-L${endLine}` : ""}</span>
@@ -494,14 +494,14 @@ function AskAiPanel({
 				{loading && !response && (
 					<div className="flex items-center gap-1.5 py-2">
 						<Loader2 className="h-3 w-3 animate-spin text-muted-foreground/40" />
-						<span className="text-[11px] text-muted-foreground/40">Analyzing...</span>
+						<span className="text-xs text-muted-foreground/40">Analyzing...</span>
 					</div>
 				)}
 
 				{response && (
-					<div className="text-xs leading-relaxed">
-						<Markdown>{response}</Markdown>
-					</div>
+				<div className="text-sm leading-relaxed">
+					<Markdown>{response}</Markdown>
+				</div>
 				)}
 
 				{!loading && (
@@ -512,13 +512,13 @@ function AskAiPanel({
 							onChange={(e) => setQuestion(e.target.value)}
 							onKeyDown={(e) => { if (e.key === "Enter" && question.trim()) ask(); }}
 							placeholder="Ask a follow-up..."
-							className="flex-1 h-7 rounded-md border bg-background px-2.5 text-[11px] placeholder:text-muted-foreground/30 focus:outline-none focus:border-foreground/20"
+							className="flex-1 h-7 rounded-md border bg-background px-2.5 text-xs placeholder:text-muted-foreground/30 focus:outline-none focus:border-foreground/20"
 						/>
 						<button
 							type="button"
 							onClick={() => ask()}
 							disabled={loading}
-							className="h-7 px-2.5 rounded-md bg-foreground text-background text-[11px] font-medium disabled:opacity-30 hover:opacity-80 transition-opacity"
+							className="h-7 px-2.5 rounded-md bg-foreground text-background text-xs font-medium disabled:opacity-30 hover:opacity-80 transition-opacity"
 						>
 							Ask
 						</button>
@@ -573,7 +573,7 @@ function InlineComments({
 							<button
 								type="button"
 								onClick={() => setShowAi(true)}
-								className="flex items-center gap-1.5 text-[11px] text-muted-foreground/40 hover:text-foreground transition-colors"
+								className="flex items-center gap-1.5 text-xs text-muted-foreground/40 hover:text-foreground transition-colors"
 							>
 								<Sparkles className="h-3 w-3" />
 								Ask AI about this code

package/src/web/client/components/ErrorScreen.tsx

@@ -82,7 +82,7 @@ export function ErrorScreen({
 
 				<div className="flex flex-col items-center gap-2 text-center">
 					<h2 className="text-lg font-semibold tracking-tight">{title}</h2>
-					<p className="text-sm text-muted-foreground leading-relaxed max-w-sm">
+					<p className="text-base text-muted-foreground leading-relaxed max-w-sm">
 						{hint}
 					</p>
 				</div>

package/src/web/client/components/InputScreen.tsx

@@ -126,7 +126,7 @@ export function InputScreen({
 							</span>
 						)}
 					</div>
-					<p className="text-sm text-muted-foreground/50">
+					<p className="text-base text-muted-foreground/50">
 						Turn PRs into navigable stories
 					</p>
 				</div>
@@ -144,7 +144,7 @@ export function InputScreen({
 								onFocus={() => setFocused(true)}
 								onBlur={() => setFocused(false)}
 								placeholder="https://github.com/owner/repo/pull/123"
-								className="flex-1 h-12 bg-transparent px-3 text-sm font-mono placeholder:text-muted-foreground/25 focus:outline-none"
+								className="flex-1 h-12 bg-transparent px-3 text-base font-mono placeholder:text-muted-foreground/25 focus:outline-none"
 								autoFocus
 							/>
 							<button

package/src/web/client/components/LoadingTimeline.tsx

@@ -125,12 +125,12 @@ export function LoadingTimeline({
 							<span className="text-sm font-semibold font-mono">newpr</span>
 						)}
 						<span className="text-muted-foreground/30">·</span>
-						<span className="text-xs text-muted-foreground/50 tabular-nums shrink-0">
+						<span className="text-sm text-muted-foreground/50 tabular-nums shrink-0">
 							{seconds < 60 ? `${seconds}s` : `${Math.floor(seconds / 60)}m ${seconds % 60}s`}
 						</span>
 					</div>
 					{prNum && (
-						<span className="text-[11px] text-muted-foreground/40 font-mono">#{prNum}</span>
+						<span className="text-xs text-muted-foreground/40 font-mono">#{prNum}</span>
 					)}
 				</div>
 
@@ -153,18 +153,18 @@ export function LoadingTimeline({
 								)}
 								<div className="min-w-0 flex-1">
 									<div className="flex items-center gap-2">
-										<span className={`text-sm font-medium ${step.done ? "text-muted-foreground" : step.active ? "text-foreground" : "text-muted-foreground/50"}`}>
-											{STAGE_LABELS[step.stage]}{progress}
+								<span className={`text-base font-medium ${step.done ? "text-muted-foreground" : step.active ? "text-foreground" : "text-muted-foreground/50"}`}>
+										{STAGE_LABELS[step.stage]}{progress}
+									</span>
+									{step.done && step.durationMs !== undefined && (
+										<span className="text-sm text-muted-foreground/60">
+											{formatDuration(step.durationMs)}
 										</span>
-										{step.done && step.durationMs !== undefined && (
-											<span className="text-xs text-muted-foreground/60">
-												{formatDuration(step.durationMs)}
-											</span>
-										)}
-									</div>
-									{step.done && completionDetail && (
-										<p className="text-xs text-muted-foreground/60 mt-0.5 truncate">{completionDetail}</p>
 									)}
+								</div>
+								{step.done && completionDetail && (
+									<p className="text-sm text-muted-foreground/60 mt-0.5 truncate">{completionDetail}</p>
+								)}
 									{step.active && recentLog.length > 0 && (
 										<div className="mt-1.5 space-y-0.5 max-h-40 overflow-y-auto">
 											{recentLog.map((line, j) => {