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

package/extensions/file-browse/{live.ts → live/live.ts}

@@ -12,10 +12,11 @@
  * pair — so the whole feed is tested without one.
  */
 
-import { defaultInterval, defaultTimer } from "../../lib/timer.ts";
-import { LIVE_MAX_CLIENTS } from "./config.ts";
-import { changeDirs } from "./diff.ts";
-import type { DiffList, GitDiffProvider, GitStatus, GitStatusProvider } from "./git.ts";
+import type { EventSink } from "../../../lib/http/index.ts";
+import { defaultInterval, type RepeatTimer } from "../../../lib/timer.ts";
+import { LIVE_MAX_CLIENTS } from "../config/config.ts";
+import { changeDirs } from "../text/diff.ts";
+import type { DiffList, GitDiffProvider, GitStatus, GitStatusProvider } from "../git/git.ts";
 import { changeRevision } from "./rev.ts";
 
 /**
@@ -141,48 +142,25 @@ export function frameOf(snapshot: LiveSnapshot): string {
 	return `event: status\ndata: ${JSON.stringify(snapshot)}\n\n`;
 }
 
-/** A connected page: the two things a server can do to its stream. */
-export interface LiveClient {
-	/** Write one frame. */
-	write(frame: string): void;
-	/** End the stream. */
-	end(): void;
-}
-
-/** Starts a repeating tick and returns its cancel. */
-export type LiveTimer = (tick: () => void, intervalMs: number) => () => void;
-
-/** Starts a one-shot delay and returns its cancel. */
-export type LiveTimeout = (run: () => void, delayMs: number) => () => void;
-
-/** The default ticker: a repeating timer that cannot hold the process open. */
-const defaultTicker: LiveTimer = (tick, intervalMs) => defaultInterval(intervalMs, tick);
-
-/** The default grace timer: one shot, and it cannot hold the process open either. */
-const defaultTimeout: LiveTimeout = (run, delayMs) => defaultTimer(delayMs, run);
+/** A connected page, as the channel writes it: the sink's two verbs. */
+export type LiveClient = EventSink;
 
 export interface LiveChannelOptions {
 	/** What each frame carries. */
 	status: LiveStatusSource;
 	/** How far apart the frames are, in ms. */
 	intervalMs: number;
-	/** How long the last page's departure is held before `onIdle` is called, in ms. */
-	idleMs: number;
 	/** How many pages may attach at once; defaults to `LIVE_MAX_CLIENTS`. */
 	maxClients?: number;
-	/** Called once the last page has been gone for `idleMs`; omitted, the channel only idles. */
-	onIdle?: () => void;
 	/** Timer seam; defaults to an unref'd interval. */
-	timer?: LiveTimer;
-	/** Grace seam; defaults to an unref'd timeout. */
-	timeout?: LiveTimeout;
+	timer?: RepeatTimer;
 }
 
 /** The fan-out for one server: pages attach, ticks publish, close ends them. */
 export interface LiveChannel {
 	/** Add a connected page, send it the current status, and return its detach. */
 	attach(client: LiveClient): () => void;
-	/** End every connected page, stop the ticker, and drop a pending idle stop. */
+	/** End every connected page, and stop the ticker. */
 	close(): void;
 	/**
 	 * Whether another page must be refused. The caller asks before it opens the
@@ -192,18 +170,18 @@ export interface LiveChannel {
 }
 
 /**
- * Build a channel for one server. The ticker runs only while at least one page
- * is attached, so an idle server does no status work at all; the idle stop is one
- * timer armed when the last page leaves and cancelled by the next page, so a
- * reload keeps the server and a closed window releases it.
+ * Build a channel for one server. The ticker runs only while at least one page is
+ * attached, so an idle server does no status work at all; the last page leaving
+ * parks it and the next page starts it again. What happens to a server nobody is
+ * reading — the idle grace and the close it leads to — is the server's policy, so
+ * the caller hands it to `lib/http`'s `serveLoopback` instead of configuring it
+ * here.
  */
 export function createLiveChannel(options: LiveChannelOptions): LiveChannel {
-	const timer = options.timer ?? defaultTicker;
-	const timeout = options.timeout ?? defaultTimeout;
+	const timer = options.timer ?? defaultInterval;
 	const maxClients = options.maxClients ?? LIVE_MAX_CLIENTS;
 	const clients = new Set<LiveClient>();
 	let cancel: (() => void) | undefined;
-	let cancelIdle: (() => void) | undefined;
 	let inFlight = false;
 
 	const stop = (): void => {
@@ -211,34 +189,9 @@ export function createLiveChannel(options: LiveChannelOptions): LiveChannel {
 		cancel = undefined;
 	};
 
-	const cancelIdleStop = (): void => {
-		cancelIdle?.();
-		cancelIdle = undefined;
-	};
-
-	const armIdleStop = (): void => {
-		if (!options.onIdle || cancelIdle) return;
-		cancelIdle = timeout(() => {
-			cancelIdle = undefined;
-			options.onIdle?.();
-		}, options.idleMs);
-	};
-
-	/** Nobody is reading a frame any more: park the ticker and start the grace. */
-	const pageLeft = (): void => {
-		stop();
-		armIdleStop();
-	};
-
 	const send = (frame: string): void => {
-		for (const client of clients) {
-			try {
-				client.write(frame);
-			} catch {
-				clients.delete(client);
-			}
-		}
-		if (clients.size === 0) pageLeft();
+		for (const client of clients) client.write(frame);
+		if (clients.size === 0) stop();
 	};
 
 	// One snapshot serves every attached page, so a tick is one git read and one
@@ -255,17 +208,15 @@ export function createLiveChannel(options: LiveChannelOptions): LiveChannel {
 
 	return {
 		attach(client) {
-			cancelIdleStop();
 			clients.add(client);
-			cancel ??= timer(() => void publish(), options.intervalMs);
+			cancel ??= timer(options.intervalMs, () => void publish());
 			void publish();
 			return () => {
 				if (!clients.delete(client)) return;
-				if (clients.size === 0) pageLeft();
+				if (clients.size === 0) stop();
 			};
 		},
 		close() {
-			cancelIdleStop();
 			stop();
 			for (const client of clients) client.end();
 			clients.clear();

package/extensions/file-browse/{router.ts → transport/router.ts}

@@ -19,14 +19,14 @@ import {
 	parseRequestUrl,
 	scriptHeaders,
 	type ServeResponse,
-} from "../../lib/http/index.ts";
-import { THEME_JS, THEME_JS_PATH } from "../../lib/web/index.ts";
-import { CLIENT_JS } from "./client.ts";
-import type { LivePage } from "./chrome.ts";
-import type { ServeConfig } from "./config.ts";
-import { isDiffMode, type DiffMode } from "./diff.ts";
-import type { DiffList, GitDiffProvider, GitStatus, GitStatusProvider } from "./git.ts";
-import { changeIndexOf, type LiveStatusSource } from "./live.ts";
+} from "../../../lib/http/index.ts";
+import { THEME_JS, THEME_JS_PATH } from "../../../lib/web/index.ts";
+import { CLIENT_JS } from "../view/client.ts";
+import type { LivePage } from "../view/chrome.ts";
+import type { ServeConfig } from "../config/config.ts";
+import { isDiffMode, type DiffMode } from "../text/diff.ts";
+import type { DiffList, GitDiffProvider, GitStatus, GitStatusProvider } from "../git/git.ts";
+import { changeIndexOf, type LiveStatusSource } from "../live/live.ts";
 import {
 	buildFileIndex,
 	buildTree,
@@ -38,13 +38,19 @@ import {
 	type DirEntry,
 	type FileIndexProvider,
 	type TreeNode,
-} from "./files.ts";
-import { FOLDER_ICON, iconHref, languageOf } from "./icons.ts";
-import { fuzzySearch } from "./fuzzy.ts";
-import { renderDirectoryPage, renderDiffPage, renderErrorPage, renderFileDiffPage, renderFilePage } from "./html.ts";
-import type { ChangeSummary } from "./html.ts";
-import { encodePath, HttpError, relativePath, resolveDecodedPath, resolveRequestPath } from "./paths.ts";
-import { listingRevision } from "./rev.ts";
+} from "../files/files.ts";
+import { FOLDER_ICON, iconHref, languageOf } from "../view/icons.ts";
+import { fuzzySearch } from "../text/fuzzy.ts";
+import {
+	renderDirectoryPage,
+	renderDiffPage,
+	renderErrorPage,
+	renderFileDiffPage,
+	renderFilePage,
+} from "../view/html.ts";
+import type { ChangeSummary } from "../view/html.ts";
+import { encodePath, HttpError, relativePath, resolveDecodedPath, resolveRequestPath } from "../files/paths.ts";
+import { listingRevision } from "../live/rev.ts";
 
 export interface ServeContext {
 	root: string;

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/{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, 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"];
@@ -286,8 +287,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 +326,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 +338,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;

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,
@@ -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;
 }

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

@@ -25,7 +25,7 @@
  * how `<details>` wraps Markdown.
  */
 
-import { decodeCodePoint, escapeHtml } from "../../lib/escape.ts";
+import { decodeCodePoint, escapeHtml } from "../../../lib/escape.ts";
 import { resolveImage, resolveLink } from "./urls.ts";
 
 /** Elements re-emitted with sanitized attributes. */

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/memory/store.ts

@@ -74,17 +74,24 @@ export function readMemoryFile(path: string): string {
 	return readFileSync(path, "utf-8");
 }
 
-/** Coarse identity of a store file, for cheap change detection. */
+/**
+ * Identity of a store file, for cheap change detection.
+ *
+ * The stamp is nanoseconds, not milliseconds: an edit that keeps the file's size
+ * is otherwise invisible for a whole millisecond, which is long enough for an
+ * editor that writes in place. `size` stays a number so the doctor's byte counts
+ * read the same.
+ */
 export interface MemoryFileStamp {
-	mtimeMs: number;
+	mtimeNs: bigint;
 	size: number;
 }
 
 /** Freshness stamp for a store file; `undefined` when it is missing or unstattable. */
 export function memoryFileStamp(path: string): MemoryFileStamp | undefined {
 	try {
-		const info = statSync(path);
-		return { mtimeMs: info.mtimeMs, size: info.size };
+		const info = statSync(path, { bigint: true });
+		return { mtimeNs: info.mtimeNs, size: Number(info.size) };
 	} catch {
 		return undefined;
 	}
@@ -93,7 +100,7 @@ export function memoryFileStamp(path: string): MemoryFileStamp | undefined {
 /** Whether two stamps describe the same file state. */
 export function sameStamp(a: MemoryFileStamp | undefined, b: MemoryFileStamp | undefined): boolean {
 	if (a === undefined || b === undefined) return a === b;
-	return a.mtimeMs === b.mtimeMs && a.size === b.size;
+	return a.mtimeNs === b.mtimeNs && a.size === b.size;
 }
 
 /** Note text of every bullet line, sanitized to one terminal-safe line. */