@gavin-hu/my-pi-agent 0.19.0 → 0.21.0

package/extensions/file-browse/view/markdown.ts

@@ -38,9 +38,9 @@ import {
 	sanitizeHtmlBlock,
 	sanitizeInlineTag,
 } from "./raw-html.ts";
-import { resolveImage, resolveLink } from "./urls.ts";
+import { emitAnchor, emitImage, emitImageLink } from "./emit.ts";
+import { type LinkBase, resolveImage, resolveLink } from "./urls.ts";
 
-const HTTP_SCHEME = /^https?:/i;
 /** A GFM extended autolink: `http(s)://` or `www.` followed by a domain. */
 const AUTOLINK = /^(?:https?:\/\/|www\.)[^\s<>]*/i;
 
@@ -65,12 +65,9 @@ interface LinkDefinition {
 	title?: string;
 }
 
-/**
- * Per-document render state: the resolution base, the reference definitions,
- * and the raw-HTML balancing state.
- */
+/** Per-document render state: where links resolve, the definitions, and the raw-HTML sink. */
 interface Context {
-	baseDir: string;
+	link: LinkBase;
 	defs: Map<string, LinkDefinition>;
 	/** Emit sanitized raw HTML; `false` escapes it as text. */
 	html: boolean;
@@ -82,6 +79,8 @@ interface Context {
 export interface MarkdownOptions {
 	/** Emit sanitized raw HTML (the default) or escape it (`false`). */
 	html?: boolean;
+	/** The prefix local links are written under, e.g. `/files`; `""` for the origin's root. */
+	base?: string;
 }
 
 /**
@@ -92,7 +91,7 @@ export interface MarkdownOptions {
 export function renderMarkdown(source: string, baseDir = "", options: MarkdownOptions = {}): string {
 	const html = options.html ?? true;
 	const { defs, body } = collectDefinitions(source.split(/\r\n|\r|\n/), html);
-	const context = createContext(baseDir, defs, html);
+	const context = createContext({ dir: baseDir, base: options.base ?? "" }, defs, html);
 	const rendered = renderBlocks(body, context);
 	return html ? rendered + closeHtmlSink(context.sink) : rendered;
 }
@@ -110,10 +109,10 @@ function headingSlug(text: string): string {
 }
 
 /** Build the per-document render state, suffixing repeated heading slugs. */
-function createContext(baseDir: string, defs: Map<string, LinkDefinition>, html: boolean): Context {
+function createContext(link: LinkBase, defs: Map<string, LinkDefinition>, html: boolean): Context {
 	const seen = new Map<string, number>();
 	return {
-		baseDir,
+		link,
 		defs,
 		html,
 		sink: createHtmlSink(),
@@ -338,7 +337,7 @@ function renderHtmlBlock(lines: string[], start: number, block: HtmlBlockStart,
 		if (isBlank(lines[i])) break;
 		collected.push(lines[i]);
 	}
-	return [sanitizeHtmlBlock(collected.join("\n"), ctx.baseDir, ctx.sink), i];
+	return [sanitizeHtmlBlock(collected.join("\n"), ctx.link, ctx.sink), i];
 }
 
 interface Fence {
@@ -641,22 +640,19 @@ function parseReferenceLink(text: string, start: number, ctx: Context): ParsedLi
 
 function renderLink(parsed: ParsedLink, ctx: Context): string {
 	const label = renderInline(parsed.label, ctx);
-	const target = resolveLink(parsed.dest, ctx.baseDir);
+	const target = resolveLink(parsed.dest, ctx.link);
 	if (!target) return label;
-	const title = parsed.title ? ` title="${escapeHtml(parsed.title)}"` : "";
-	const external = target.external ? ` target="_blank" rel="noopener noreferrer"` : "";
-	return `<a href="${escapeHtml(target.href)}"${title}${external}>${label}</a>`;
+	// The anchor's shape is `emit.ts`'s, the same one the raw-HTML sanitizer emits: a link
+	// written in Markdown and the same link written as HTML are the same link.
+	return emitAnchor({ href: target.href, content: label, title: parsed.title, external: target.external });
 }
 
 function renderImage(parsed: ParsedLink, ctx: Context): string {
 	const alt = escapeHtml(parsed.label);
-	const target = resolveImage(parsed.dest, ctx.baseDir);
+	const target = resolveImage(parsed.dest, ctx.link);
 	if (!target) return alt;
-	if (!target.image) {
-		return `<a href="${escapeHtml(target.href)}" target="_blank" rel="noopener noreferrer">${alt || escapeHtml(parsed.dest)}</a>`;
-	}
-	const title = parsed.title ? ` title="${escapeHtml(parsed.title)}"` : "";
-	return `<img class="md-img" src="${escapeHtml(target.href)}" alt="${alt}" loading="lazy" decoding="async"${title}>`;
+	if (!target.image) return emitImageLink(target.href, parsed.label || parsed.dest);
+	return emitImage({ href: target.href, alt, title: parsed.title });
 }
 
 function matchRun(text: string, start: number, char: string): number {
@@ -812,13 +808,15 @@ function renderInline(text: string, ctx: Context): string {
 			const match = /^<((?:https?|mailto):[^>\s]+)>/i.exec(text.slice(i));
 			if (match) {
 				const href = match[1];
-				const external = HTTP_SCHEME.test(href) ? ` target="_blank" rel="noopener noreferrer"` : "";
-				out += `<a href="${escapeHtml(href)}"${external}>${escapeHtml(href)}</a>`;
+				// Resolved rather than pattern-matched: `urls.ts` owns which schemes open in a
+				// new tab, and a `<mailto:>` link is external but does not.
+				const external = resolveLink(href, ctx.link)?.external ?? false;
+				out += emitAnchor({ href, content: escapeHtml(href), external });
 				i += match[0].length;
 				continue;
 			}
 			if (ctx.html) {
-				const tag = sanitizeInlineTag(text, i, ctx.baseDir, tags);
+				const tag = sanitizeInlineTag(text, i, ctx.link, tags);
 				if (tag) {
 					out += tag.html;
 					i = tag.end;
@@ -836,8 +834,7 @@ function renderInline(text: string, ctx: Context): string {
 				// Require a domain, so a bare scheme is not a link.
 				if (dots > 0 && /\.[^./]/.test(label.slice(dots))) {
 					const href = /^www\./i.test(label) ? `http://${label}` : label;
-					const external = HTTP_SCHEME.test(href) ? ` target="_blank" rel="noopener noreferrer"` : "";
-					out += `<a href="${escapeHtml(href)}"${external}>${escapeHtml(label)}</a>`;
+					out += emitAnchor({ href, content: escapeHtml(label), external: true });
 					i += label.length;
 					continue;
 				}

package/extensions/file-browse/view/palette.ts

@@ -122,7 +122,7 @@ export const PALETTE_CLIENT_JS = `	// Quick open: a fuzzy path palette backed by
 
 	function runSearch() {
 		var query = paletteInput.value.trim();
-		fetch("/api/files?q=" + encodeURIComponent(query))
+		fetch(pageBase() + "/api/files?q=" + encodeURIComponent(query))
 			.then(function (response) {
 				if (!response.ok) throw new Error("search failed");
 				return response.json();
@@ -212,7 +212,7 @@ export const PALETTE_CLIENT_JS = `	// Quick open: a fuzzy path palette backed by
 
 	function openActive() {
 		var result = paletteResults[paletteActive];
-		if (result) window.location.assign("/view/" + encodeSegments(result.path));
+		if (result) window.location.assign(pageBase() + "/view/" + encodeSegments(result.path));
 	}
 	var quickOpenButton = document.getElementById("quick-open");
 	if (quickOpenButton) quickOpenButton.addEventListener("click", openPalette);

package/extensions/file-browse/view/raw-html.ts

@@ -26,7 +26,8 @@
  */
 
 import { decodeCodePoint, escapeHtml } from "../../../lib/escape.ts";
-import { resolveImage, resolveLink } from "./urls.ts";
+import { emitImage, emitImageLink, EXTERNAL_ATTRS } from "./emit.ts";
+import { type LinkBase, resolveImage, resolveLink } from "./urls.ts";
 
 /** Elements re-emitted with sanitized attributes. */
 const BLOCK_ELEMENTS = new Set([
@@ -139,6 +140,12 @@ const BLOCK_OPEN = /^ {0,3}<(\/?)([a-zA-Z][a-zA-Z0-9-]*)/;
 const COMMENT_OPEN = /^ {0,3}<!--/;
 const DECLARATION_OPEN = /^ {0,3}<[!?]/;
 /** The five XML references, plus numeric ones, are the only refs decoded in a value. */
+/**
+ * The five references an attribute value may carry, decoded.
+ *
+ * Not the Markdown renderer's table: a tag's attribute is not running text, and decoding
+ * `&nbsp;` or `&mdash;` inside one would invent characters the author did not write.
+ */
 const ATTRIBUTE_ENTITIES: Record<string, string> = { amp: "&", lt: "<", gt: ">", quot: '"', apos: "'" };
 const ATTRIBUTE_ENTITY = /&(?:#([0-9]{1,7})|#[xX]([0-9a-fA-F]{1,6})|([a-zA-Z]+));/g;
 
@@ -198,7 +205,7 @@ export function closesSuppressed(line: string, name: string): boolean {
 }
 
 /** Sanitize one HTML block, emitting text and allowlisted tags into `sink`. */
-export function sanitizeHtmlBlock(source: string, baseDir: string, sink: HtmlSink): string {
+export function sanitizeHtmlBlock(source: string, linkBase: LinkBase, sink: HtmlSink): string {
 	let out = "";
 	let i = 0;
 	while (i < source.length) {
@@ -228,7 +235,7 @@ export function sanitizeHtmlBlock(source: string, baseDir: string, sink: HtmlSin
 			out += closeTo(sink.stack, sink.stack.lastIndexOf(tag.name));
 			continue;
 		}
-		out += tag.name === "img" ? sanitizeImage(tag, baseDir) : sanitizeOpenTag(tag, baseDir);
+		out += tag.name === "img" ? sanitizeImage(tag, linkBase) : sanitizeOpenTag(tag, linkBase);
 		if (tag.name !== "img" && !VOID.has(tag.name) && !tag.selfClosing) sink.stack.push(tag.name);
 	}
 	return out;
@@ -243,7 +250,7 @@ export function sanitizeHtmlBlock(source: string, baseDir: string, sink: HtmlSin
 export function sanitizeInlineTag(
 	source: string,
 	start: number,
-	baseDir: string,
+	linkBase: LinkBase,
 	stack: string[],
 ): { html: string; end: number } | undefined {
 	const tag = parseRawTag(source, start);
@@ -253,7 +260,7 @@ export function sanitizeInlineTag(
 	if (SUPPRESSED.has(tag.name)) return { html: "", end: skipSuppressed(source, tag.next, tag) };
 	if (!ALLOWED.has(tag.name) || !PHRASING.has(tag.name)) return undefined;
 	if (tag.end) return { html: closeTo(stack, stack.lastIndexOf(tag.name)), end: tag.next };
-	const html = tag.name === "img" ? sanitizeImage(tag, baseDir) : sanitizeOpenTag(tag, baseDir);
+	const html = tag.name === "img" ? sanitizeImage(tag, linkBase) : sanitizeOpenTag(tag, linkBase);
 	if (tag.name !== "img" && !VOID.has(tag.name) && !tag.selfClosing) stack.push(tag.name);
 	return { html, end: tag.next };
 }
@@ -395,7 +402,7 @@ function skipSuppressed(source: string, from: number, tag: RawTag): number {
 }
 
 /** Rebuild an allowlisted start tag from its parsed name and allowed attributes. */
-function sanitizeOpenTag(tag: RawTag, baseDir: string): string {
+function sanitizeOpenTag(tag: RawTag, linkBase: LinkBase): string {
 	const out: string[] = [];
 	const id = attribute(tag, "id");
 	if (id !== undefined && id !== "") out.push(` id="${escapeHtml(id)}"`);
@@ -404,10 +411,10 @@ function sanitizeOpenTag(tag: RawTag, baseDir: string): string {
 	switch (tag.name) {
 		case "a": {
 			const href = attribute(tag, "href");
-			const target = href === undefined ? undefined : resolveLink(href, baseDir);
+			const target = href === undefined ? undefined : resolveLink(href, linkBase);
 			if (target !== undefined) {
 				out.push(` href="${escapeHtml(target.href)}"`);
-				if (target.external) out.push(' target="_blank" rel="noopener noreferrer"');
+				if (target.external) out.push(EXTERNAL_ATTRS);
 			}
 			break;
 		}
@@ -451,21 +458,19 @@ function sanitizeOpenTag(tag: RawTag, baseDir: string): string {
  * (the page CSP forbids remote images), and an unresolvable one leaves the alt
  * text, so a broken badge is still readable.
  */
-function sanitizeImage(tag: RawTag, baseDir: string): string {
+function sanitizeImage(tag: RawTag, linkBase: LinkBase): string {
 	const alt = attribute(tag, "alt") ?? "";
 	const src = attribute(tag, "src");
-	const target = src === undefined ? undefined : resolveImage(src, baseDir);
+	const target = src === undefined ? undefined : resolveImage(src, linkBase);
 	if (target === undefined) return alt === "" ? "" : escapeHtml(alt);
-	if (!target.image) {
-		const label = alt === "" ? target.href : alt;
-		return `<a href="${escapeHtml(target.href)}" target="_blank" rel="noopener noreferrer">${escapeHtml(label)}</a>`;
-	}
-	const out: string[] = [` class="md-img" src="${escapeHtml(target.href)}" alt="${escapeHtml(alt)}"`];
-	const width = numeric(attribute(tag, "width"));
-	if (width !== undefined) out.push(` width="${width}"`);
-	const height = numeric(attribute(tag, "height"));
-	if (height !== undefined) out.push(` height="${height}"`);
-	const title = attribute(tag, "title");
-	if (title !== undefined) out.push(` title="${escapeHtml(title)}"`);
-	return `<img${out.join("")} loading="lazy" decoding="async">`;
+	if (!target.image) return emitImageLink(target.href, alt === "" ? target.href : alt);
+	// The element is `emit.ts`'s, so an `<img>` written as HTML and the same one written as
+	// Markdown are the same element, down to the lazy decoding.
+	return emitImage({
+		href: target.href,
+		alt,
+		title: attribute(tag, "title"),
+		width: numeric(attribute(tag, "width")),
+		height: numeric(attribute(tag, "height")),
+	});
 }

package/extensions/file-browse/view/tree.ts

@@ -189,7 +189,7 @@ export const TREE_CLIENT_JS = `	var tree = document.querySelector("nav.bf-tree")
 		li.dataset.loading = "true";
 		twisty.textContent = "\\u2026";
 		if (link) link.setAttribute("aria-busy", "true");
-		fetch("/api/tree?path=" + encodeURIComponent(path))
+		fetch(pageBase() + "/api/tree?path=" + encodeURIComponent(path))
 			.then(function (response) {
 				if (!response.ok) throw new Error("load failed");
 				return response.json();

package/extensions/file-browse/view/urls.ts

@@ -26,6 +26,20 @@ export interface ResolvedLink {
 	external: boolean;
 }
 
+/**
+ * Where a destination resolves from, and what it is written under.
+ *
+ * `dir` is the root-relative directory the destination is relative to; `base` is the
+ * prefix the page answers under, so a page served at `/files` writes
+ * `/files/view/<path>` and one holding the origin's root writes `/view/<path>`. The two
+ * are one value because a caller that knows one almost always knows the other, and a
+ * second parameter is a second thing to thread through ten functions.
+ */
+export interface LinkBase {
+	dir: string;
+	base: string;
+}
+
 /** An image destination the page may emit; `image: false` means link to it instead. */
 export interface ResolvedImage {
 	href: string;
@@ -53,15 +67,15 @@ function resolveLocalPath(href: string, baseDir: string): { rel: string; suffix:
 }
 
 /** Resolve a link destination, or `undefined` when the scheme or path is not allowed. */
-export function resolveLink(dest: string, baseDir: string): ResolvedLink | undefined {
+export function resolveLink(dest: string, linkBase: LinkBase): ResolvedLink | undefined {
 	const value = dest.trim();
 	if (value === "") return undefined;
 	if (value.startsWith("#")) return { href: value, external: false };
 	if (SAFE_LINK_SCHEME.test(value)) return { href: value, external: HTTP_SCHEME.test(value) };
 	if (HAS_SCHEME.test(value)) return undefined;
-	const local = resolveLocalPath(value, baseDir);
+	const local = resolveLocalPath(value, linkBase.dir);
 	if (!local) return undefined;
-	return { href: `/view/${encodePath(local.rel)}${local.suffix}`, external: false };
+	return { href: `${linkBase.base}/view/${encodePath(local.rel)}${local.suffix}`, external: false };
 }
 
 /**
@@ -69,12 +83,12 @@ export function resolveLink(dest: string, baseDir: string): ResolvedLink | undef
  * URL resolves with `image: false` because the page CSP forbids remote images, so
  * the caller renders a link instead of an `<img>`.
  */
-export function resolveImage(dest: string, baseDir: string): ResolvedImage | undefined {
+export function resolveImage(dest: string, linkBase: LinkBase): ResolvedImage | undefined {
 	const value = dest.trim();
 	if (value === "") return undefined;
 	if (HTTP_SCHEME.test(value)) return { href: value, image: false };
 	if (HAS_SCHEME.test(value)) return undefined;
-	const local = resolveLocalPath(value, baseDir);
+	const local = resolveLocalPath(value, linkBase.dir);
 	if (!local) return undefined;
-	return { href: `/raw/${encodePath(local.rel)}${local.suffix}`, image: true };
+	return { href: `${linkBase.base}/raw/${encodePath(local.rel)}${local.suffix}`, image: true };
 }

package/extensions/goal/tui.ts

@@ -10,6 +10,7 @@
 
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { truncateToWidth, type Component } from "@earendil-works/pi-tui";
+import { RenderCache } from "../../lib/tui/render-cache.ts";
 import { goalHeader, goalLine } from "./format.ts";
 import type { AchievedStyle, Goal } from "./types.ts";
 
@@ -47,6 +48,7 @@ export class GoalWidget implements Component {
 export class GoalHeader implements Component {
 	private goal: Goal;
 	private theme: Theme;
+	private readonly cache = new RenderCache();
 
 	constructor(goal: Goal, theme: Theme) {
 		this.goal = goal;
@@ -57,13 +59,21 @@ export class GoalHeader implements Component {
 	update(goal: Goal, theme: Theme): void {
 		this.goal = goal;
 		this.theme = theme;
+		this.cache.invalidate();
 	}
 
-	invalidate(): void {}
+	invalidate(): void {
+		this.cache.invalidate();
+	}
 
 	render(width: number): string[] {
+		const w = Math.max(1, width);
+		return this.cache.lines(w, undefined, () => this.build(w));
+	}
+
+	private build(w: number): string[] {
 		// The leading blank line separates the header from the call line, matching
 		// the built-ins' header/output gap.
-		return ["", truncateToWidth(goalHeader(this.goal, this.theme), Math.max(1, width), "…")];
+		return ["", truncateToWidth(goalHeader(this.goal, this.theme), w, "…")];
 	}
 }

package/extensions/job/runtime.ts

@@ -29,6 +29,7 @@ import {
 	type KillTreeFn,
 	type SpawnedProcess,
 	type SpawnFn,
+	startTokenOf,
 	type StdinSink,
 } from "../../lib/spawn.ts";
 import { loadConfig, type JobsConfig } from "./config.ts";
@@ -38,7 +39,6 @@ import { defaultInterval } from "../../lib/timer.ts";
 import { clipLabel } from "../../lib/tui/columns.ts";
 import { compareJobs, formatLogs } from "./format.ts";
 import { readLogTail } from "./logs.ts";
-import { defaultStartToken, type LivenessFn, type StartTokenFn } from "./process.ts";
 import {
 	loadRegistries,
 	mergeRecords,
@@ -114,9 +114,9 @@ export interface SendResult {
 
 export interface RuntimeOptions {
 	spawn?: SpawnFn;
-	liveness?: LivenessFn;
+	liveness?: typeof isPidAlive;
 	killTree?: KillTreeFn;
-	startToken?: StartTokenFn;
+	startToken?: typeof startTokenOf;
 	now?: () => number;
 	/** Platform seam: whether this is Windows. Defaults to the running platform. */
 	isWindows?: () => boolean;
@@ -183,7 +183,7 @@ export function createJobsRuntime(options: RuntimeOptions = {}): JobsRuntime {
 	const spawn = options.spawn ?? defaultSpawn;
 	const liveness = options.liveness ?? isPidAlive;
 	const killTree = options.killTree ?? defaultKillTree;
-	const startToken = options.startToken ?? defaultStartToken;
+	const startToken = options.startToken ?? startTokenOf;
 	const now = options.now ?? (() => Date.now());
 	const isWindows = options.isWindows ?? (() => process.platform === "win32");
 

package/extensions/job/tools.ts

@@ -11,7 +11,15 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { errorResult, textResult } from "../../lib/tool-result.ts";
 import { reuseText, toolDetails } from "../../lib/tui/chrome.ts";
-import { callLine, detailsOrLine, elapsedSuffix, rejectionLine, syncElapsedTimer } from "../../lib/tui/tool.ts";
+import {
+	callLine,
+	detailsOrLine,
+	elapsedSuffix,
+	isTailResult,
+	rejectionLine,
+	syncElapsedTimer,
+	TailResult,
+} from "../../lib/tui/tool.ts";
 import { SEPARATORS } from "../../lib/tui/glyphs.ts";
 import { paint } from "../../lib/tui/tokens.ts";
 import {
@@ -26,8 +34,8 @@ import {
 import type { JobsRuntime } from "./runtime.ts";
 import { JobParams, normalizeCall, type JobArgs, type JobCall } from "./schema.ts";
 import { toRecord } from "./store.ts";
-import { JobResult, type JobResultInput } from "./tui.ts";
-import type { JobDetails } from "./types.ts";
+import { jobTail, type JobResultInput } from "./tui.ts";
+import type { JobDetails, JobRecord } from "./types.ts";
 
 export const TOOL_NAME = "job";
 
@@ -224,8 +232,10 @@ export function registerTools(pi: ExtensionAPI, runtime: JobsRuntime): void {
 				}
 				const shown = options.expanded ? all : all.slice(Math.max(0, all.length - RESULT_ROWS));
 				const input: JobResultInput = { kind: "logs", lines: shown, earlier: all.length - shown.length };
-				const view = context.lastComponent instanceof JobResult ? context.lastComponent : new JobResult(input, theme);
-				view.setInput(input, theme);
+				const previous = context.lastComponent;
+				const tail = jobTail(input);
+				const view = isTailResult<JobRecord>(previous) ? previous : new TailResult<JobRecord>(tail, theme);
+				view.setInput(tail, theme);
 				return view;
 			}
 
@@ -242,8 +252,10 @@ export function registerTools(pi: ExtensionAPI, runtime: JobsRuntime): void {
 					more: ordered.length - shown.length,
 					sessionId: details.sessionId,
 				};
-				const view = context.lastComponent instanceof JobResult ? context.lastComponent : new JobResult(input, theme);
-				view.setInput(input, theme);
+				const previous = context.lastComponent;
+				const tail = jobTail(input);
+				const view = isTailResult<JobRecord>(previous) ? previous : new TailResult<JobRecord>(tail, theme);
+				view.setInput(tail, theme);
 				return view;
 			}
 

package/extensions/job/tui.ts

@@ -15,13 +15,12 @@
  */
 
 import type { Theme } from "@earendil-works/pi-coding-agent";
-import type { Component } from "@earendil-works/pi-tui";
 import { formatDuration, sanitizeUntrustedLine } from "../../lib/format.ts";
 import type { ScreenHint, ViewportRowsSource } from "../../lib/tui/chrome.ts";
 import { DockListScreen, type DetailContents } from "../../lib/tui/dock-screen.ts";
 import { SEPARATORS } from "../../lib/tui/glyphs.ts";
 import { statusRow, type StatusRow } from "../../lib/tui/rows.ts";
-import { TailResult, type TailResultInput } from "../../lib/tui/tool.ts";
+import type { TailResultInput } from "../../lib/tui/tool.ts";
 import { paint } from "../../lib/tui/tokens.ts";
 import {
 	compareJobs,
@@ -62,7 +61,7 @@ export type JobResultInput =
 	| { kind: "logs"; lines: string[]; earlier: number };
 
 /** Map the job vocabulary onto the shared transcript tail block. */
-function jobTail(input: JobResultInput): TailResultInput<JobRecord> {
+export function jobTail(input: JobResultInput): TailResultInput<JobRecord> {
 	if (input.kind === "logs") {
 		return { kind: "lines", lines: input.lines, earlier: input.earlier, earlierLabel: "lines" };
 	}
@@ -77,31 +76,6 @@ function jobTail(input: JobResultInput): TailResultInput<JobRecord> {
 	};
 }
 
-/**
- * Transcript result block for the `job` tool: the shared tail block over the
- * `/jobs`-style row rail or the themed log tail. Reused across renders.
- */
-export class JobResult implements Component {
-	private readonly view: TailResult<JobRecord>;
-
-	constructor(input: JobResultInput, theme: Theme) {
-		this.view = new TailResult(jobTail(input), theme);
-	}
-
-	/** Update in place so the transcript can reuse this component across renders. */
-	setInput(input: JobResultInput, theme: Theme): void {
-		this.view.setInput(jobTail(input), theme);
-	}
-
-	invalidate(): void {
-		this.view.invalidate();
-	}
-
-	render(width: number): string[] {
-		return this.view.render(width);
-	}
-}
-
 export interface JobListCallbacks {
 	/** This session's id: a record with another id belongs to a peer session. */
 	sessionId(): string;

package/extensions/plan/commands.ts

@@ -19,7 +19,7 @@ import { PlanIndexComponent, formatPlanIndex } from "./plan-index.ts";
 import { runBrowserReview } from "./review.ts";
 import type { PlanRuntime } from "./runtime.ts";
 import type { PlanSummary } from "./plans.ts";
-import { createPlanWebReview, type PlanWebReview } from "./web.ts";
+import { createPlanWebReview, NO_SERVER_DEPS, type PlanWebReview } from "./web.ts";
 
 const ENABLED_NOTICE = `Plan mode enabled — ${READ_ONLY_SUMMARY}.`;
 const DISABLED_NOTICE = "Plan mode disabled — full access restored.";
@@ -219,7 +219,7 @@ async function openPlanIndex(
 export function registerCommands(
 	pi: ExtensionAPI,
 	runtime: PlanRuntime,
-	web: PlanWebReview = createPlanWebReview(runtime.plans),
+	web: PlanWebReview = createPlanWebReview(runtime.plans, NO_SERVER_DEPS),
 ): void {
 	pi.registerCommand("plan", {
 		description: