@gavin-hu/my-pi-agent 0.18.0 → 0.20.0

package/extensions/file-browse/transport/server.ts

@@ -0,0 +1,110 @@
+/**
+ * The HTTP server for the browse extension.
+ *
+ * Wraps the pure `handleRequest` in `lib/http`'s loopback server: it resolves the
+ * root once, feeds the live status channel, and exposes an idempotent `close()`.
+ * The request handler is injectable so tests can avoid a socket when they only
+ * need routing.
+ *
+ * What stays here is this extension's own policy: the request handler, its
+ * request count, the catch that turns a thrown handler into a plain-text 500, the
+ * cap that refuses a page rather than opening a stream for it, and how long a
+ * server with nobody reading it is worth keeping.
+ */
+
+import { realpathSync } from "node:fs";
+import { type ServeResponse, serveLoopback, textResponse } from "../../../lib/http/index.ts";
+import { IDLE_STOP_MS, type ServeConfig } from "../config/config.ts";
+import { createFileIndex } from "../files/files.ts";
+import type { GitDiffProvider, GitStatusProvider } from "../git/git.ts";
+import { createLiveChannel, createLiveStatus } from "../live/live.ts";
+import { handleRequest, type ServeContext } from "./router.ts";
+
+export interface FileServer {
+	/** Origin URL, e.g. `http://127.0.0.1:49152/`. */
+	url: string;
+	/** The bound port. */
+	port: number;
+	/** Resolved absolute root that is actually served. */
+	root: string;
+	/** Stop listening. Safe to call more than once or before `listen` settles. */
+	close(): Promise<void>;
+}
+
+export interface FileServerOptions {
+	root: string;
+	config: ServeConfig;
+	/** Read-only git context for the page header; omitted when unavailable. */
+	git?: GitStatusProvider;
+	/** Read-only diff provider for the change and diff views; omitted when unavailable. */
+	diff?: GitDiffProvider;
+	/** Called once the last page has been gone for `idleStopMs`; omitted, the server waits for `close()`. */
+	onIdle?: () => void;
+	/** Grace before `onIdle`, in ms; defaults to `IDLE_STOP_MS`. */
+	idleStopMs?: number;
+	/** Override the request handler (tests). */
+	handler?: typeof handleRequest;
+}
+
+/** Start a read-only server rooted at `options.root` on 127.0.0.1. */
+export async function createFileServer(options: FileServerOptions): Promise<FileServer> {
+	const root = realpathSync.native(options.root);
+	const { config } = options;
+	// The status feed's two server-owned facts: how long it has been up, and how
+	// many requests it has answered. Both are read per frame, never cached.
+	let requests = 0;
+	const status = config.live
+		? createLiveStatus({
+				git: options.git,
+				diff: config.gitDiff ? options.diff : undefined,
+				startedAt: Date.now(),
+				requests: () => requests,
+			})
+		: undefined;
+	const live = status ? createLiveChannel({ status, intervalMs: config.liveIntervalMs }) : undefined;
+	const context: ServeContext = {
+		root,
+		config,
+		git: options.git,
+		diff: options.diff,
+		index: createFileIndex(root, config.maxIndexEntries),
+		live: status,
+	};
+	const handler = options.handler ?? handleRequest;
+
+	const server = await serveLoopback({
+		ports: [config.port],
+		idle: options.onIdle === undefined ? undefined : { ms: options.idleStopMs ?? IDLE_STOP_MS, onIdle: options.onIdle },
+		handler: async (request) => {
+			requests += 1;
+			let result: ServeResponse;
+			try {
+				result = await handler(context, request.method, request.url, request.headers);
+			} catch {
+				return { result: textResponse(500, "Internal error") };
+			}
+			if (!result.stream || !live || request.method.toUpperCase() === "HEAD") return { result };
+			// Checked before the head is written: a refusal must not leave a stream the
+			// client would wait on. `204` is the one status an event source does not
+			// reconnect after, so a page beyond the cap goes quiet instead of retrying.
+			if (live.atCapacity()) {
+				return { result: { status: 204, headers: { "Cache-Control": "no-store" }, body: "" } };
+			}
+			return {
+				result,
+				openSink: (sink) => live.attach({ write: (frame) => sink.write(frame), end: () => sink.end() }),
+			};
+		},
+	});
+
+	return {
+		url: `${server.origin}/`,
+		port: server.port,
+		root,
+		close: async () => {
+			// End the streams first, so a connected page does not hold the close open.
+			live?.close();
+			await server.close();
+		},
+	};
+}

package/extensions/file-browse/{chrome.ts → view/chrome.ts}

@@ -8,10 +8,10 @@
  * it reaches the page. The page shell and the page bodies stay in `html.ts`.
  */
 
-import { escapeHtml } from "../../lib/escape.ts";
-import { GLYPHS } from "../../lib/tui/glyphs.ts";
-import type { GitStatus } from "./git.ts";
-import type { ChangeIndex } from "./live.ts";
+import { escapeHtml } from "../../../lib/escape.ts";
+import { GLYPHS } from "../../../lib/tui/glyphs.ts";
+import type { GitStatus } from "../git/git.ts";
+import type { ChangeIndex } from "../live/live.ts";
 
 /** The header's git chip: the branch, or the detached head; changes are the changes chip's job. */
 export function renderGit(git: GitStatus | undefined): string {

package/extensions/file-browse/{client.ts → view/client.ts}

@@ -12,7 +12,7 @@
  * and `bun build --no-bundle` needs no separate asset.
  */
 
-import { keysDialogScript, THEME_TOGGLE_JS } from "../../lib/web/index.ts";
+import { keysDialogScript, THEME_TOGGLE_JS } from "../../../lib/web/index.ts";
 import { LIVE_CLIENT_JS } from "./live-client.ts";
 import { PALETTE_CLIENT_JS } from "./palette.ts";
 import { TREE_CLIENT_JS } from "./tree.ts";

package/extensions/file-browse/{diff-html.ts → view/diff-html.ts}

@@ -8,9 +8,9 @@
  * hunk and never across the unchanged lines a hunk does not show.
  */
 
-import { escapeHtml } from "../../lib/escape.ts";
-import { highlightLines, type HighlightLanguage } from "../../lib/highlight.ts";
-import type { DiffLine } from "./diff.ts";
+import { escapeHtml } from "../../../lib/escape.ts";
+import { highlightLines, type HighlightLanguage } from "../../../lib/highlight.ts";
+import type { DiffLine } from "../text/diff.ts";
 
 /**
  * The inner HTML per diff line. Hunk bodies are highlighted by rebuilding the

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

@@ -0,0 +1,51 @@
+/**
+ * The elements a page emits for a resolved Markdown link or image.
+ *
+ * Two renderers reach a link: the Markdown renderer (`markdown.ts`) and the raw-HTML
+ * sanitizer (`raw-html.ts`). Both ask `urls.ts` what a destination resolves to, and both
+ * then build the same anchor and the same image — the external-rel attributes, the
+ * `md-img` class, and the lazy decoding — so those live here rather than in a fourth and
+ * fifth copy. Values are escaped here; the callers hand in what the document said.
+ */
+
+import { escapeHtml } from "../../../lib/escape.ts";
+
+/**
+ * The attributes an external link carries, for a caller that is building the element
+ * itself: the raw-HTML sanitizer rewrites an existing tag's attribute list and has no
+ * content string to hand an emitter.
+ *
+ * `noopener` because the target is another origin, and `noreferrer` because the page is a
+ * local view of the reader's own repository and should not tell anyone where they read it.
+ */
+export const EXTERNAL_ATTRS = ' target="_blank" rel="noopener noreferrer"';
+
+/** An `<a>` for one destination; `title` and the rel attributes are omitted when absent. */
+export function emitAnchor(input: { href: string; content: string; title?: string; external: boolean }): string {
+	const title = input.title === undefined ? "" : ` title="${escapeHtml(input.title)}"`;
+	return `<a href="${escapeHtml(input.href)}"${title}${input.external ? EXTERNAL_ATTRS : ""}>${input.content}</a>`;
+}
+
+/**
+ * A link standing in for an image the page may not show: an external `src` is blocked by
+ * the CSP, so the reader gets something to click, labelled by the alt text.
+ */
+export function emitImageLink(href: string, label: string): string {
+	return `<a href="${escapeHtml(href)}"${EXTERNAL_ATTRS}>${escapeHtml(label)}</a>`;
+}
+
+/** An `<img>`: block-level spacing class, alt text, and the sizes a document may pin. */
+export function emitImage(input: {
+	href: string;
+	alt: string;
+	title?: string;
+	/** A document may pin a pixel size; escaped like every other value the page emits. */
+	width?: string | number;
+	height?: string | number;
+}): string {
+	const size = `${input.width === undefined ? "" : ` width="${escapeHtml(String(input.width))}"`}${
+		input.height === undefined ? "" : ` height="${escapeHtml(String(input.height))}"`
+	}`;
+	const title = input.title === undefined ? "" : ` title="${escapeHtml(input.title)}"`;
+	return `<img class="md-img" src="${escapeHtml(input.href)}" alt="${escapeHtml(input.alt)}"${size}${title} loading="lazy" decoding="async">`;
+}

package/extensions/file-browse/{html.ts → view/html.ts}

@@ -8,18 +8,19 @@
  * the page-only `browse-*` rules from `styles.ts`.
  */
 
-import { brandLockup, docHead, type KeyHint, keysSheet, themeToggle } from "../../lib/web/index.ts";
-import { escapeHtml } from "../../lib/escape.ts";
+import { brandLockup, docHead, type KeyHint, keysSheet, tabTitle, themeToggle } from "../../../lib/web/index.ts";
+import { escapeHtml } from "../../../lib/escape.ts";
 import { renderChanges, renderGit, renderLiveChrome, renderLiveStatus, type LivePage } from "./chrome.ts";
-import { formatSize, type DirEntry, type FileView, type Listing, type TreeNode } from "./files.ts";
-import { DIFF_MODES, statusLetter, type ChangedFile, type DiffMode } from "./diff.ts";
+import { type DirEntry, type FileView, type Listing, type TreeNode } from "../files/files.ts";
+import { formatSize } from "../../../lib/format.ts";
+import { DIFF_MODES, statusLetter, type ChangedFile, type DiffMode } from "../text/diff.ts";
 import { renderDiffLines } from "./diff-html.ts";
-import type { DiffFileView, GitStatus } from "./git.ts";
-import type { ChangeIndex } from "./live.ts";
-import { detectLanguage, highlightLines, type HighlightLanguage } from "../../lib/highlight.ts";
+import type { DiffFileView, GitStatus } from "../git/git.ts";
+import type { ChangeIndex } from "../live/live.ts";
+import { detectLanguage, highlightLines, type HighlightLanguage } from "../../../lib/highlight.ts";
 import { FOLDER_ICON, iconHref, languageOf, SPRITE, type IconSpec } from "./icons.ts";
 import { renderMarkdown } from "./markdown.ts";
-import { encodePath } from "./paths.ts";
+import { encodePath } from "../files/paths.ts";
 import { BROWSE_CSS } from "./styles.ts";
 
 const MONTHS = ["Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"];
@@ -86,7 +87,6 @@ function renderLayout(layout: Layout): string {
 <html lang="en">
 <head>
 ${docHead({ title: layout.title, css: BROWSE_CSS })}
-<link rel="icon" href="data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 16 16'%3E%3Ccircle cx='8' cy='8' r='6' fill='%2358a6ff'/%3E%3C/svg%3E">
 </head>
 <body class="bf-app">
 <a class="skip" href="#main">Skip to content</a>
@@ -220,7 +220,7 @@ ${page.listing.entries.map((entry) => renderRow(entry, page.maxThumbBytes, page.
 		body += `<p class="bf-note">Showing the first ${page.listing.entries.length} of ${page.listing.total} entries.</p>`;
 
 	return renderLayout({
-		title: `browse · ${page.rel || folderName}`,
+		title: tabTitle("browse", page.rel || folderName),
 		rootLabel: page.rootLabel,
 		tree: page.tree,
 		currentRel: page.rel,
@@ -286,8 +286,9 @@ function numberedLines(page: FilePage): boolean {
 	return page.file.kind === "text" || (page.file.kind === "markdown" && page.markdownRaw === true);
 }
 
-/** The card-head controls: Markdown mode, line wrapping, and copy actions. */
-function renderFileActions(page: FilePage): string {
+/** The card-head controls: Markdown mode, line wrapping, and the copy actions,
+ * plus the download the diff head keeps in the same group. */
+function renderFileActions(page: FilePage, download: string): string {
 	const isMarkdown = page.file.kind === "markdown";
 	const rawHref = `/raw/${encodePath(page.rel)}`;
 	const viewHref = `/view/${encodePath(page.rel)}`;
@@ -324,7 +325,7 @@ function renderFileActions(page: FilePage): string {
 			`<button type="button" class="bf-btn" id="copy-reference" disabled title="Select lines in the file first">copy reference</button>`,
 		);
 	}
-	return `<span class="bf-card__actions">${groups.join("")}</span>`;
+	return `<span class="bf-card__actions">${groups.join("")}${download}</span>`;
 }
 
 /** Render a file page (text, Markdown, image, binary, or over-size). */
@@ -336,7 +337,7 @@ export function renderFilePage(page: FilePage): string {
 	// about the file; the language label, size, and date are the useful metadata.
 	const meta = `${spec.label} · ${formatSize(page.file.size)} · ${formatDate(page.file.mtimeMs)}`;
 	const download = `<a class="bf-btn" href="/raw/${encodePath(page.rel)}" download>download</a>`;
-	const head = `<div class="bf-card__head">${iconMarkup(spec)}<h1 class="bf-card__title">${escapeHtml(name)}</h1><span class="bf-card__meta">${escapeHtml(meta)}</span><span class="bf-spacer"></span>${renderFileActions(page)}${download}</div>`;
+	const head = `<div class="bf-card__head">${iconMarkup(spec)}<h1 class="bf-card__title">${escapeHtml(name)}</h1><span class="bf-card__meta">${escapeHtml(meta)}</span><span class="bf-spacer"></span>${renderFileActions(page, download)}</div>`;
 
 	const isMarkdown = page.file.kind === "markdown";
 	const rendered = isMarkdown && !page.markdownRaw;
@@ -369,7 +370,7 @@ export function renderFilePage(page: FilePage): string {
 	}
 
 	return renderLayout({
-		title: `browse · ${page.rel}`,
+		title: tabTitle("browse", page.rel),
 		rootLabel: page.rootLabel,
 		tree: page.tree,
 		currentRel: page.rel,
@@ -442,7 +443,7 @@ ${page.files.map((file) => renderChangedRow(file, page.mode)).join("\n")}
 </table>`;
 	const stat = page.stat ? `<pre class="browse-diff-stat">${escapeHtml(page.stat)}</pre>` : "";
 	return renderLayout({
-		title: "browse · changes",
+		title: tabTitle("browse", "changes"),
 		rootLabel: page.rootLabel,
 		tree: page.tree,
 		currentRel: "",
@@ -514,7 +515,7 @@ export function renderFileDiffPage(page: FileDiffPage): string {
 		if (page.diff.truncated) note = `<p class="bf-note">Showing the first ${page.diff.lines.length} diff lines.</p>`;
 	}
 	return renderLayout({
-		title: `browse · ${page.rel}`,
+		title: tabTitle("browse", page.rel),
 		rootLabel: page.rootLabel,
 		tree: page.tree,
 		currentRel: page.rel,
@@ -542,7 +543,7 @@ export interface ErrorPage {
 /** Render a status/error page with the same chrome. */
 export function renderErrorPage(page: ErrorPage): string {
 	return renderLayout({
-		title: `browse · ${page.status}`,
+		title: tabTitle("browse", String(page.status)),
 		rootLabel: page.rootLabel,
 		tree: page.tree,
 		currentRel: "",

package/extensions/file-browse/{icons.ts → view/icons.ts}

@@ -5,7 +5,7 @@
  */
 
 import { extname } from "node:path";
-import { IMAGE_EXTENSIONS } from "./files.ts";
+import { IMAGE_EXTENSIONS } from "../files/files.ts";
 
 export type IconName = "folder" | "file-code" | "file-text" | "file-image" | "file-json" | "file-archive" | "file";
 

package/extensions/file-browse/{markdown.ts → view/markdown.ts}

@@ -25,8 +25,8 @@
  * policy and raw HTML do not.
  */
 
-import { decodeCodePoint, escapeHtml } from "../../lib/escape.ts";
-import { fenceLanguage, highlightLines } from "../../lib/highlight.ts";
+import { decodeCodePoint, escapeHtml } from "../../../lib/escape.ts";
+import { fenceLanguage, highlightLines } from "../../../lib/highlight.ts";
 import {
 	closesSuppressed,
 	closeHtmlSink,
@@ -38,9 +38,9 @@ import {
 	sanitizeHtmlBlock,
 	sanitizeInlineTag,
 } from "./raw-html.ts";
+import { emitAnchor, emitImage, emitImageLink } from "./emit.ts";
 import { 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;
 
@@ -126,8 +126,8 @@ function createContext(baseDir: string, defs: Map<string, LinkDefinition>, html:
 	};
 }
 
-/** Fold a reference label the way CommonMark does: trim, collapse, case-fold. */
-function normalizeLabel(label: string): string {
+/** Fold a reference label the way CommonMark does: trim, collapse, case-fold. Not `lib/session-label.ts`, which names a session. */
+function referenceLabel(label: string): string {
 	return label.trim().replace(/\s+/g, " ").toLowerCase();
 }
 
@@ -180,7 +180,7 @@ function collectDefinitions(lines: string[], html: boolean): { defs: Map<string,
 			body.push(line);
 			continue;
 		}
-		const key = normalizeLabel(match[1]);
+		const key = referenceLabel(match[1]);
 		if (!defs.has(key)) {
 			const dest = match[2].startsWith("<") && match[2].endsWith(">") ? match[2].slice(1, -1) : match[2];
 			defs.set(key, { dest, title: match[3] });
@@ -630,11 +630,11 @@ function parseReferenceLink(text: string, start: number, ctx: Context): ParsedLi
 		const end = matchBracket(text, close + 1);
 		if (end !== -1) {
 			const raw = text.slice(close + 2, end);
-			const def = ctx.defs.get(normalizeLabel(raw === "" ? label : raw));
+			const def = ctx.defs.get(referenceLabel(raw === "" ? label : raw));
 			if (def) return { label, dest: def.dest, title: def.title, end: end + 1 };
 		}
 	}
-	const def = ctx.defs.get(normalizeLabel(label));
+	const def = ctx.defs.get(referenceLabel(label));
 	if (def) return { label, dest: def.dest, title: def.title, end: close + 1 };
 	return undefined;
 }
@@ -643,20 +643,17 @@ function renderLink(parsed: ParsedLink, ctx: Context): string {
 	const label = renderInline(parsed.label, ctx);
 	const target = resolveLink(parsed.dest, ctx.baseDir);
 	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);
 	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,8 +809,10 @@ 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.baseDir)?.external ?? false;
+				out += emitAnchor({ href, content: escapeHtml(href), external });
 				i += match[0].length;
 				continue;
 			}
@@ -836,8 +835,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/{raw-html.ts → view/raw-html.ts}

@@ -25,7 +25,8 @@
  * how `<details>` wraps Markdown.
  */
 
-import { decodeCodePoint, escapeHtml } from "../../lib/escape.ts";
+import { decodeCodePoint, escapeHtml } from "../../../lib/escape.ts";
+import { emitImage, emitImageLink, EXTERNAL_ATTRS } from "./emit.ts";
 import { resolveImage, resolveLink } from "./urls.ts";
 
 /** Elements re-emitted with sanitized attributes. */
@@ -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;
 
@@ -407,7 +414,7 @@ function sanitizeOpenTag(tag: RawTag, baseDir: string): string {
 			const target = href === undefined ? undefined : resolveLink(href, baseDir);
 			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;
 		}
@@ -456,16 +463,14 @@ function sanitizeImage(tag: RawTag, baseDir: string): string {
 	const src = attribute(tag, "src");
 	const target = src === undefined ? undefined : resolveImage(src, baseDir);
 	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/{styles.ts → view/styles.ts}

@@ -12,7 +12,7 @@ import {
 	WEB_COMPONENTS_CSS,
 	WEB_LAYOUT_CSS,
 	WEB_THEME_CSS,
-} from "../../lib/web/index.ts";
+} from "../../../lib/web/index.ts";
 
 /** The shared shell, components, and the page-only `browse-*` rules. */
 export const BROWSE_CSS = `${WEB_BASE_CSS}
@@ -37,7 +37,6 @@ ${WEB_COMPONENTS_CSS}
 .browse-dotfile .bf-table__name a{opacity:.72}
 .browse-broken{color:var(--muted);text-decoration:line-through}
 .browse-link-mark{color:var(--muted);font-size:var(--fs-xs)}
-.browse-when{display:none}
 body.palette-open{overflow:hidden}
 .browse-diff{font-family:var(--font-mono);gap:5px}
 .browse-diff:hover{color:var(--accent);text-decoration:none}
@@ -93,8 +92,24 @@ body.palette-open{overflow:hidden}
 .browse-measure .bf-md>*{max-width:var(--browse-measure);margin-inline:auto}
 /* Tables start at the prose column and take only the gutter to its right, so a
    wide window widens them without a dead band on their left. Code stays inside
-   the measure with the rest of the prose. */
-.browse-measure .bf-md>.md-table-frame{max-width:none;padding-left:var(--browse-gutter)}
+   the measure with the rest of the prose. The frame is also the container a
+   squeezed table asks about: it is the pane, not the window, that narrows one. */
+.browse-measure .bf-md>.md-table-frame{max-width:none;padding-left:var(--browse-gutter);container-type:inline-size}
+/* A table spans the reading column rather than stopping at its own content: a
+   shrink-wrapped table leaves a ragged band beside itself — 254px on a 2560px
+   screen, and 300px beside the three-column tables at 1440 — which the reader
+   cannot tell from a column that failed to draw. The card is wider than the text
+   it holds, so the measure is the ceiling: a table ends where the prose does, and
+   below the floor's threshold the minimum still wins and the wrapper scrolls. */
+.browse-measure .bf-md .md-table-frame table{width:100%;max-width:var(--browse-measure)}
+/* In a pane this narrow, auto layout gives the prose cell whatever an unbreakable
+   code span leaves — one word per line, and a four-column table read as a
+   300px-tall row. A floor on the cells makes the table wider than the pane
+   instead, so every column keeps a readable width and the wrapper scrolls them;
+   a table that still fits is left as it was. */
+@container (max-width:600px){
+.browse-measure .bf-md .md-table-frame td{min-width:14em}
+}
 /* A table wider than the pane scrolls, but an overlay scrollbar is invisible
    until it moves; a gradient on the frame marks the columns still out of view.
    It has to be the frame: the scroller clips its own content, so a pseudo-element
@@ -116,10 +131,19 @@ body.palette-open{overflow:hidden}
 .browse-side .bf-tree{flex:1 1 auto;min-height:0;overflow:auto}
 }
 @media (max-width:820px){
-.browse-root{order:9;flex:1 1 100%;direction:ltr;text-align:left}
+.browse-root{order:9;flex:1 1 100%}
 }
 @media (max-width:520px){
 .bf-toolbar .bf-theme-toggle{margin-left:auto}
+/* A phone has no room for four button rows above the file: the actions keep one
+   line and scroll instead, so the document starts where it should. A control
+   keeps its own width — shrunk, a label wraps and a segmented control clips. */
+.bf-card__actions{max-width:100%;flex-wrap:nowrap;overflow-x:auto}
+.bf-card__actions>*{flex:0 0 auto}
+/* A trailing date or one-word change state costs the column a reader is choosing
+   between: at 390px the listing gave the name 106px and every entry truncated,
+   and the diff gave a path 140px while "modified" held 130. */
+.browse-listing .bf-table__when,.browse-diff .browse-diff-word{display:none}
 }
 /* A printed page scrolls nothing: the card and its body open up, the pinned hunk
    header stops pinning, and the progress hairline belongs to the screen. */

package/extensions/file-browse/{tree.ts → view/tree.ts}

@@ -8,7 +8,7 @@
  * intent, and the one row the keyboard is on.
  */
 
-import { SIDEBAR_COLLAPSE_MAX } from "../../lib/web/index.ts";
+import { SIDEBAR_COLLAPSE_MAX } from "../../../lib/web/index.ts";
 
 export const TREE_CLIENT_JS = `	var tree = document.querySelector("nav.bf-tree");
 	var filter = document.getElementById("filter");

package/extensions/file-browse/{urls.ts → view/urls.ts}

@@ -14,7 +14,7 @@
  */
 
 import { posix } from "node:path";
-import { encodePath } from "./paths.ts";
+import { encodePath } from "../files/paths.ts";
 
 const HAS_SCHEME = /^[a-zA-Z][a-zA-Z0-9+.-]*:/;
 const SAFE_LINK_SCHEME = /^(?:https?|mailto):/i;

package/extensions/file-read/schema.ts

@@ -3,10 +3,11 @@
  */
 
 import { Type, type Static } from "typebox";
+import { READ_DOC_TOOL } from "../../lib/tool-names.ts";
 import type { FormatId } from "./formats.ts";
 
 /** The tool's registered name, shared by the definition and its renderer. */
-export const TOOL_NAME = "read_doc";
+export const TOOL_NAME = READ_DOC_TOOL;
 
 export const MIN_CHARS = 200;
 export const MAX_CHARS = 100_000;

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");