@dylanpiercey/work 0.2.31 → 0.2.33

package/bin/work.mjs

@@ -144,6 +144,10 @@ const flagValues = (flag) =>
 // verbs of its own without a case here.
 const cli = await cliCommand(command, positional[0]);
 
+// Every command runs from here, before the module's top level gets past
+// this switch: a `const` or `let` declared below it is uninitialised for
+// all of them (smoke refuses one), so a constant lives in the function
+// that uses it.
 switch (cli ? "" : command) {
   case "":
     // `work session --help` is what an agent tries before `work help`.
@@ -928,11 +932,6 @@ async function bridge() {
   );
 }
 
-/** The two packages in this tree that carry install scripts: node-pty builds
- * its native binding when there is no prebuild for the platform, and our own
- * postinstall marks the bundled spawn-helper executable. */
-const SCRIPT_PKGS = "@dylanpiercey/work,node-pty";
-
 /** Newer npm skips install scripts on a global install unless the packages
  * are named, then spends four lines telling the reader to re-run the command
  * themselves — noise on every `work update`. Name them up front instead. Both
@@ -940,17 +939,20 @@ const SCRIPT_PKGS = "@dylanpiercey/work,node-pty";
  * exactly the bet worth not making silently. Older npm has no such config and
  * would take the flag as an unknown one, so ask npm whether it knows it. */
 function allowScriptsArgs() {
+  // The two packages in this tree that carry install scripts: node-pty
+  // builds its native binding when there is no prebuild for the platform,
+  // and our own postinstall marks the bundled spawn-helper executable.
+  const pkgs = "@dylanpiercey/work,node-pty";
+  let conf;
   try {
-    const conf = execFileSync("npm", ["config", "ls", "-l"], {
+    conf = execFileSync("npm", ["config", "ls", "-l"], {
       encoding: "utf8",
       stdio: ["ignore", "pipe", "ignore"],
     });
-    return /^allow-scripts\s*=/m.test(conf)
-      ? [`--allow-scripts=${SCRIPT_PKGS}`]
-      : [];
   } catch {
     return [];
   }
+  return /^allow-scripts\s*=/m.test(conf) ? [`--allow-scripts=${pkgs}`] : [];
 }
 
 async function update() {
@@ -1291,8 +1293,6 @@ function xmlText(value) {
     .replace(/>/g, ">");
 }
 
-const XML_ENTITIES = { amp: "&", lt: "<", gt: ">", quot: '"', apos: "'" };
-
 /** WORK_* already baked into the installed service. `work update` re-runs
  * setup from whatever shell the user happens to be in, so without this the
  * first update silently resets the port, root and data dir to defaults —
@@ -1308,12 +1308,13 @@ function installedServiceEnv() {
     return installed;
   }
   if (process.platform === "darwin") {
+    const entities = { amp: "&", lt: "<", gt: ">", quot: '"', apos: "'" };
     const re =
       /<key>([A-Za-z_][A-Za-z0-9_]*)<\/key>\s*<string>([^<]*)<\/string>/g;
     for (const [, key, value] of text.matchAll(re)) {
       installed[key] = value.replace(
         /&(amp|lt|gt|quot|apos);/g,
-        (_, name) => XML_ENTITIES[name],
+        (_, name) => entities[name],
       );
     }
     return installed;

package/dist/{_DAiB3S3K.js → _16nMvJWe.js}

@@ -1,4 +1,4 @@
-import { $ as reopenSession, A as closedSession, Bn as summaryOf, Cn as lastTypedAt, Hn as turnSettling, In as setExpecting, J as notifySessionsChanged, L as getSession, Mn as onPhaseChange, Nn as onTurnEnd, Or as getSetting, Pn as sessionWatched, Q as renameSession, Qr as httpError, R as interruptSession, U as listPanesWithin, Un as turnSince, Vn as turnPhase, Y as onSessionDispose, _n as activityOf, ai as db, et as reopenable, ii as all, j as createSession, jn as onFinished, kr as isKnownAgent, ln as steerKeys, mn as cleanSummary, nt as sendToSession, oi as get, on as interruptKeys, qn as agentLabel, tt as retireSession, vn as agentPhase, xn as hookReportedAt, z as killSession } from "./_Chg81Eud.js";
+import { $ as reopenSession, A as closedSession, Bn as summaryOf, Cn as lastTypedAt, Hn as turnSettling, In as setExpecting, J as notifySessionsChanged, L as getSession, Mn as onPhaseChange, Nn as onTurnEnd, Or as getSetting, Pn as sessionWatched, Q as renameSession, Qr as httpError, R as interruptSession, U as listPanesWithin, Un as turnSince, Vn as turnPhase, Y as onSessionDispose, _n as activityOf, ai as db, et as reopenable, ii as all, j as createSession, jn as onFinished, kr as isKnownAgent, ln as steerKeys, mn as cleanSummary, nt as sendToSession, oi as get, on as interruptKeys, qn as agentLabel, tt as retireSession, vn as agentPhase, xn as hookReportedAt, z as killSession } from "./_DTcc_6tw.js";
 //#region src/server/messages.ts
 var team = {
 	memberGroup: () => null,

package/dist/_CgcXKkfV.js

@@ -0,0 +1,2 @@
+import { J as notifySessionsChanged } from "./_DTcc_6tw.js";
+export { notifySessionsChanged };

package/dist/{_CA77e0SL.js → _D43r39ym.js}

@@ -49,7 +49,7 @@ var toastTitle = {
 	warn: "_w",
 	info: "_x"
 };
-var tocDot = "_da _ba";
+var tocDot = "_de _ba";
 //#endregion
 //#region src/styles/tokens.css.ts
 var color = {
@@ -85,98 +85,98 @@ var iconSize = {
 };
 //#endregion
 //#region src/tags/settings-agents.css.ts
-var agentRow = "_f6 _dG";
-var list$3 = "_dK _dm _dF _dI";
-var metaChip = "_dY _bz _by";
-var sectionCount = "_dj _bp _bq";
-var usageNote = "_j_ _by";
-var pop$2 = "_g8 _bN";
-var spinner = "_ha _bD _bi";
-var toolLead = "_gT _dT";
-var name$4 = "_hh _dT";
-var desc$1 = "_hi _dU";
+var agentRow = "_fA _dK";
+var list$3 = "_dO _dq _dJ _dM";
+var metaChip = "_d2 _bz _by";
+var sectionCount = "_dn _bp _bq";
+var usageNote = "_j6 _by";
+var pop$2 = "_gb _bN";
+var spinner = "_ge _bD _bi";
+var toolLead = "_fR _dX";
+var name$4 = "_f1 _dX";
+var desc$1 = "_f2 _dY";
 //#endregion
 //#region src/tags/lsp-add-dialog.css.ts
-var addDlg = "_mA _bX";
-var key$1 = "_l3 _l1";
-var addTile = "_l5 _l1";
-var field$2 = "_mf _bA";
-var keyField = "_mg _mf _bA";
-var labelField = "_mh _mf _bA";
-var detailsToggle = "_gG _6";
-var themeSeg = "_fP _bR";
-var checkList = "_hw _dk";
+var addDlg = "_kK _cJ";
+var key$1 = "_mq _mo";
+var addTile = "_ms _mo";
+var field$2 = "_mD _bA";
+var keyField = "_mE _mD _bA";
+var labelField = "_mF _mD _bA";
+var detailsToggle = "_gU _6";
+var themeSeg = "_fj _bR";
+var checkList = "_h4 _do";
 //#endregion
 //#region src/tags/prompt-dialog.css.ts
-var promptDlg = "_b1 _bX";
-var item$1 = "_b6 _D";
-var glance = "_ee _D";
+var promptDlg = "_cN _cJ";
+var item$1 = "_b8 _D";
+var glance = "_dG _D";
 //#endregion
 //#region src/tags/side-nav.css.ts
-var iconBtn = "_b9 _D";
-var settingsDot = "_b_ _bQ";
-var notifyBtn = "_ce _D";
+var iconBtn = "_bU _D";
+var settingsDot = "_bV _bQ";
+var notifyBtn = "_b0 _D";
 //#endregion
 //#region src/styles/session-pop.css.ts
-var sessionPop = "_cr _bN";
-var sessionPopBtn = "_cs _bk";
-var wsBtn = "_es _bk";
-var rowIcon = "_eu _bF _D";
-var prChip = "_kS _M _kM";
+var sessionPop = "_cn _bN";
+var sessionPopBtn = "_co _bk";
+var wsBtn = "_eS _bk";
+var rowIcon = "_eU _bF _D";
+var prChip = "_no _M _ni";
 //#endregion
 //#region src/tags/port-chip.css.ts
-var port = "_oI _M _kM";
+var port = "_oN _M _ni";
 var depth = {
-	"1": "_kt",
-	"2": "_ku",
-	"3": "_kv"
+	"1": "_kT",
+	"2": "_kU",
+	"3": "_kV"
 };
-var open = "_kB _bk";
-var action = "_kG _bI _D";
-var toggle = "_d3 _6";
-var fold = "_d4 _dH";
+var open = "_k1 _bk";
+var action = "_k6 _bI _D";
+var toggle = "_d7 _6";
+var fold = "_d8 _dL";
 var statusOf = {
-	queued: "_lr",
-	starting: "_ls",
-	working: "_lt",
-	waiting: "_lu",
-	idle: "_lv",
-	done: "_lw",
-	closed: "_lx"
+	queued: "_lE",
+	starting: "_lF",
+	working: "_lG",
+	waiting: "_lH",
+	idle: "_lI",
+	done: "_lJ",
+	closed: "_lK"
 };
-var item = "_km _bk";
-var menu = "_ni _cr _bN";
-var menuItem = "_nk _bk";
-var findBtn$1 = "_jh _ic _bG _D";
+var item = "_kr _bk";
+var menu = "_m6 _cn _bN";
+var menuItem = "_m8 _bk";
+var findBtn$1 = "_ir _ic _bG _D";
 var tool = "_ic _bG _D";
 var heldRefresh = "_ik _M";
-var pop$1 = "_nB _cr _bN";
-var fileFilter = "_oP _F";
-var findInput = "_e2 _F";
-var keybarIconBtn = "_fa _bI _D";
-var input = "_cv _F";
-var group = "_cy _6";
-var hint$1 = "_nK _bp _3";
-var sheetDir = "_oD _bP";
-var upBtn = "_iC _bG _D";
-var findBtn = "_iG _ic _bG _D";
-var goBtn = "_iH _ic _bG _D";
-var note = "_m2 _by";
+var pop$1 = "_mN _cn _bN";
+var fileFilter = "_oM _F";
+var findInput = "_ek _F";
+var keybarIconBtn = "_et _bI _D";
+var input = "_cc _F";
+var group = "_cf _6";
+var hint$1 = "_nP _bp _3";
+var sheetDir = "_n6 _bP";
+var upBtn = "_jm _bG _D";
+var findBtn = "_jq _ic _bG _D";
+var goBtn = "_jr _ic _bG _D";
+var note = "_mQ _by";
 //#endregion
 //#region src/tags/manage-workspace.css.ts
-var procCmd = "_fq _dT";
-var factCopy = "_fx _bI _D";
-var resultRow = "_mL _bk";
-var resultBranch = "_mM _dY _bz _by _bF _D";
-var urlAction = "_mN _dY _bz _by _dZ";
-var curRow = "_fJ _dN";
+var procCmd = "_hI _dX";
+var factCopy = "_hP _bI _D";
+var resultRow = "_l8 _bk";
+var resultBranch = "_l9 _d2 _bz _by _bF _D";
+var urlAction = "_l_ _d2 _bz _by _d3";
+var curRow = "_hG _dR";
 //#endregion
 //#region src/tags/join-dialog.css.ts
-var dlg = "_lR _bX";
-var hint = "_lS _by";
-var kbtn = "_cR _bH _D";
-var repeat = "_cU _cT";
-var dpadUser = "_cX _cT";
-var dpadEsc = "_cY _cT";
+var dlg = "_lb _cJ";
+var hint = "_lc _by";
+var kbtn = "_cA _bH _D";
+var repeat = "_cD _cC";
+var dpadUser = "_cG _cC";
+var dpadEsc = "_cH _cC";
 //#endregion
 export { field$2 as $, item as A, prTone as At, wsBtn as B, fileFilter as C, chipBtn as Ct, findBtn$1 as D, iconBtnXs as Dt, tool as E, iconBtnRow as Et, depth as F, settingsDot as G, sessionPopBtn as H, open as I, promptDlg as J, glance as K, port as L, fold as M, usageFill as Mt, toggle as N, usageInk as Nt, menu as O, inputLg as Ot, action as P, addTile as Q, prChip as R, keybarIconBtn as S, btnSm as St, heldRefresh as T, iconBtnLg as Tt, iconBtn as U, sessionPop as V, notifyBtn as W, themeSeg as X, checkList as Y, detailsToggle as Z, sheetDir as _, tocDot as _t, dlg as a, name$4 as at, input as b, toastTitle as bt, resultBranch as c, spinner as ct, factCopy as d, list$3 as dt, key$1 as et, procCmd as f, metaChip as ft, upBtn as g, sans as gt, goBtn as h, iconSize as ht, repeat as i, desc$1 as it, statusOf as j, tone as jt, menuItem as k, phaseIcon as kt, resultRow as l, usageNote as lt, findBtn as m, color as mt, dpadUser as n, labelField as nt, hint as o, toolLead as ot, note as p, agentRow as pt, item$1 as q, kbtn as r, addDlg as rt, curRow as s, pop$2 as st, dpadEsc as t, keyField as tt, urlAction as u, sectionCount as ut, hint$1 as v, toastKind as vt, pop$1 as w, glyphBadgeNum as wt, findInput as x, btnLg as xt, group as y, toastStack as yt, rowIcon as z };

package/dist/_DNAIlWyV.js

@@ -0,0 +1,2 @@
+import { f as teamsOf } from "./_16nMvJWe.js";
+export { teamsOf };

package/dist/{_Chg81Eud.js → _DTcc_6tw.js}

@@ -2434,6 +2434,7 @@ var WORKSPACE_PREFS = [
 	"diff-collapsed",
 	"diff-focus",
 	"diff-pos",
+	"review-view",
 	"files-recent"
 ];
 function deleteWorkspacePrefs(id) {
@@ -9464,7 +9465,7 @@ function publishChange(sessionId) {
 	const row = get("SELECT workspace_id FROM sessions WHERE id = ?", sessionId);
 	if (row?.workspace_id) {
 		const workspaceId = row.workspace_id;
-		import("./_ENzyVrzA.js").then((m) => m.notifySessionsChanged(workspaceId));
+		import("./_CgcXKkfV.js").then((m) => m.notifySessionsChanged(workspaceId));
 		return;
 	}
 	publish("sessions.changed");
@@ -12058,10 +12059,13 @@ var STATUS_QUIET_MS = 3e4;
 /** A checkout's `git status --porcelain=v2 -z --branch --untracked-files=all`,
 * the stamp's own (kept as above). `fresh` reads it again whatever the
 * last read: a review snapshot (server/snapshot) is of the disk as the
-* listing after it finds it, and the stamp then has that read too. */
-async function statusOf(dir, fresh = false) {
+* listing after it finds it, and the stamp then has that read too. One
+* begun under `within` ms ago stands whatever ran since: an edit count
+* (server/diff `editsOf`) taken straight after a snapshot is that
+* snapshot's status, not a second walk of every file. */
+async function statusOf(dir, fresh = false, within = 0) {
 	const hit = statuses.get(dir);
-	if (!fresh && hit && Date.now() - hit.at < STATUS_QUIET_MS && !activeSince(dir, hit.from)) return hit.out;
+	if (!fresh && hit && (Date.now() - hit.from < within || Date.now() - hit.at < STATUS_QUIET_MS && !activeSince(dir, hit.from))) return hit.out;
 	const from = Date.now();
 	const out = await git([
 		"status",
@@ -12140,16 +12144,25 @@ function statusPaths(out) {
 * same tree again. */
 var lastSnap = /* @__PURE__ */ new Map();
 var indexSeq = 0;
-/** The checkout at `dir` as a commit: its HEAD when nothing differs from
-* it, else a snapshot commit on top of it. Null with no HEAD to put it on
-* (an unborn branch). Costs a `git status` (fresh: the listing after it
-* must not show what it missed) and what that lists, not the size of the
-* repo: only those paths are added to the copied index. */
+/** The checkout at `dir` as a commit (`snap`): its HEAD when nothing
+* differs from it, else a snapshot commit on top of it. Null with no HEAD
+* to put it on (an unborn branch). Costs a `git status` (fresh: the
+* listing after it must not show what it missed) and what that lists, not
+* the size of the repo: only those paths are added to the copied index.
+* `clean` is that status listing nothing: the disk is HEAD exactly, which
+* a snapshot equal to HEAD does not say (it leaves out an untracked file
+* past MAX_UNTRACKED_BYTES). */
 async function snapshotWorktree(dir) {
 	const [{ oid: head }, status] = await Promise.all([headOid(dir), statusOf(dir, true).catch(() => null)]);
-	if (!head || status == null) return head || null;
+	if (!head || status == null) return {
+		snap: head || null,
+		clean: false
+	};
 	const { paths, untracked } = statusPaths(status);
-	if (!paths.length) return head;
+	if (!paths.length) return {
+		snap: head,
+		clean: true
+	};
 	const sizes = await Promise.all(paths.map((p) => fs$1.lstat(path.join(dir, p)).then((st) => ({
 		size: st.size,
 		sig: `${st.size}:${st.mtimeMs}`
@@ -12159,7 +12172,10 @@ async function snapshotWorktree(dir) {
 	}))));
 	const key = `${head}\0${status}\0${sizes.map((s) => s.sig).join(",")}`;
 	const hit = lastSnap.get(dir);
-	if (hit?.key === key) return hit.snap;
+	if (hit?.key === key) return {
+		snap: await hit.snap,
+		clean: false
+	};
 	const snap = writeSnapshot(dir, head, paths.filter((p, i) => !untracked.has(p) || sizes[i].size <= MAX_UNTRACKED_BYTES));
 	lastSnap.set(dir, {
 		key,
@@ -12168,7 +12184,10 @@ async function snapshotWorktree(dir) {
 	snap.catch(() => {
 		if (lastSnap.get(dir)?.snap === snap) lastSnap.delete(dir);
 	});
-	return snap;
+	return {
+		snap: await snap,
+		clean: false
+	};
 }
 async function writeSnapshot(dir, head, paths) {
 	const index = await git([
@@ -13308,21 +13327,18 @@ async function laneWorktrees(dir) {
 	else if (field.startsWith("branch refs/heads/") && at && path.resolve(at) !== self) found.set(field.slice(18), at);
 	return found;
 }
-/** Files changed or new in a worktree, uncommitted. Without the index
-* lock (exec's GIT_OPTIONAL_LOCKS): the agent working there may be
-* mid-commit. A worktree gone from disk has none. */
-var editCount = (dir) => git([
-	"status",
-	"--porcelain=v2",
-	"-z",
-	"--untracked-files=normal"
-], dir).then((out) => {
+/** Files changed or new in a worktree, uncommitted: its status as the
+* stamp and snapshots read it (`statusOf`), so a load that has just
+* snapshotted the worktree counts from that status rather than walk every
+* file again. A worktree gone from disk has none. */
+var editCount = (dir) => statusOf(dir, false, EDITS_FRESH_MS).then((out) => {
 	let n = 0;
 	const fields = out.split("\0");
 	for (let i = 0; i < fields.length; i++) {
-		if (!fields[i]) continue;
+		const field = fields[i];
+		if (!field || field.startsWith("# ")) continue;
 		n++;
-		if (fields[i].startsWith("2 ")) i++;
+		if (field.startsWith("2 ")) i++;
 	}
 	return n;
 }, () => 0);
@@ -13768,8 +13784,11 @@ async function prDiff(dir, repo, pr, ctx) {
 	const githubDrift = !!pr.headRefOid && head !== pr.headRefOid;
 	const one = ctx.rev ? await commitIn(dir, ctx.rev, against, head) : void 0;
 	const newHead = ctx.scope === "dirty" ? ctx.localHead || head : head;
-	const snap = ctx.rev ? void 0 : await snapOf(dir, ctx.scope === "dirty" ? null : ctx.scope, head);
-	const [files, commits] = await Promise.all([one ? collectFiles(dir, one.parent, one.sha, "committed") : ctx.rev ? Promise.resolve([]) : collectFiles(dir, against, newHead, ctx.scope), listCommits(dir, against, head)]);
+	const { snap, clean } = ctx.rev ? {
+		snap: void 0,
+		clean: false
+	} : await worktreeOf(dir, ctx.scope === "dirty" ? null : ctx.scope, head);
+	const [files, commits] = await Promise.all([one ? collectFiles(dir, one.parent, one.sha, "committed") : ctx.rev ? Promise.resolve([]) : collectFiles(dir, against, newHead, clean ? "committed" : ctx.scope), listCommits(dir, against, head)]);
 	return {
 		...base,
 		...githubDrift && { githubDrift: true },
@@ -13781,14 +13800,26 @@ async function prDiff(dir, repo, pr, ctx) {
 	};
 }
 /** What a section's files are taken from, as a commit a Viewed mark can
-* keep (`RepoDiff.snap`): its head for the committed range, a snapshot of
-* the worktree at `dir` with its edits. Taken before the listing, so a
-* mark keeps at most what was shown: a file edited in between then reads
-* as changed since, never as seen. */
-async function snapOf(dir, scope, head) {
-	if (scope === "committed") return head || void 0;
-	if (scope !== "working") return void 0;
-	return await snapshotWorktree(dir).catch(() => null) ?? void 0;
+* keep (`snap`, `RepoDiff.snap`): its head for the committed range, a
+* snapshot of the worktree at `dir` with its edits. Taken before the
+* listing, so a mark keeps at most what was shown: a file edited in
+* between then reads as changed since, never as seen. And whether that
+* worktree has nothing on disk past `head` (`clean`; always so for the
+* committed range): its change is then the committed range, kept by its
+* commits (`collectFiles`), where reading the worktree is four processes
+* each walking every file — for every topic worktree of a workspace on
+* every load, most of them clean. */
+async function worktreeOf(dir, scope, head) {
+	if (scope === "committed") return {
+		snap: head || void 0,
+		clean: true
+	};
+	if (scope !== "working") return { clean: false };
+	const taken = await snapshotWorktree(dir).catch(() => null);
+	return {
+		snap: taken?.snap ?? void 0,
+		clean: !!head && !!taken?.clean && taken.snap === head
+	};
 }
 async function localDiff(dir, ws, repo, branch, localHead, baseRef, scope, rev, id = repo.worktree, wait = false, parent, merged) {
 	const uncommitted = scope !== "committed";
@@ -13840,8 +13871,11 @@ async function localDiff(dir, ws, repo, branch, localHead, baseRef, scope, rev,
 	const contained = !!merged?.has(`${branch}@${localHead}`) || !missing && !emptyRemote && !!baseSha && await treeContainedIn(dir, baseSha, localHead, sha ? tree : void 0);
 	const against = contained ? localHead : mergeBase;
 	const one = rev ? await commitIn(dir, rev, against, localHead) : void 0;
-	const snap = rev ? void 0 : await snapOf(dir, scope, localHead);
-	const [files, commits] = await Promise.all([one ? collectFiles(dir, one.parent, one.sha, "committed") : rev ? Promise.resolve([]) : collectFiles(dir, against, scope === "committed" ? localHead : "HEAD", scope), missing ? Promise.resolve([]) : listCommits(dir, against, localHead)]);
+	const { snap, clean } = rev ? {
+		snap: void 0,
+		clean: false
+	} : await worktreeOf(dir, scope, localHead);
+	const [files, commits] = await Promise.all([one ? collectFiles(dir, one.parent, one.sha, "committed") : rev ? Promise.resolve([]) : clean ? collectFiles(dir, against, localHead, "committed") : collectFiles(dir, against, "HEAD", scope), missing ? Promise.resolve([]) : listCommits(dir, against, localHead)]);
 	return {
 		...base,
 		head: localHead,
@@ -13978,8 +14012,12 @@ async function branchDiff(dir, ws, repo, name, checkout, foreign, scope, rev, pa
 		...section,
 		files: []
 	};
-	const snap = rev ? void 0 : edits ? await snapOf(own, edits, head) : await snapOf(dir, "committed", head);
-	const [files, commits] = await Promise.all([one ? collectFiles(dir, one.parent, one.sha, "committed") : edits ? collectFiles(own, against, head, edits) : collectFiles(dir, against, head, "committed"), listCommits(dir, against, head)]);
+	const { snap, clean } = rev ? {
+		snap: void 0,
+		clean: true
+	} : edits ? await worktreeOf(own, edits, head) : await worktreeOf(dir, "committed", head);
+	const disk = clean ? null : edits;
+	const [files, commits] = await Promise.all([one ? collectFiles(dir, one.parent, one.sha, "committed") : disk ? collectFiles(own, against, head, disk) : collectFiles(dir, against, head, "committed"), listCommits(dir, against, head)]);
 	return {
 		...section,
 		...revisions(one, against, head, edits ?? "committed"),
@@ -14032,6 +14070,7 @@ function prMeta(pr) {
 * marks each file (viewed, its mark) as it goes. An empty listing is not
 * kept: it is as likely a diff that failed. */
 async function collectFiles(dir, oldRev, newRev, scope) {
+	if (scope === "committed" && oldRev === newRev && isSha(oldRev)) return [];
 	if (scope !== "committed" || !isSha(oldRev) || !isSha(newRev)) return readFiles(dir, oldRev, newRev, scope);
 	return (await memo(`files\0${oldRev}\0${newRev}`, async () => {
 		const read = await readFiles(dir, oldRev, newRev, scope);
@@ -17568,7 +17607,7 @@ function notifySessionsChanged(workspaceId) {
 async function emitSessionsChanged(workspaceId) {
 	try {
 		const states = await sessionStates();
-		const { teamsOf } = await import("./_CeTAKoPH.js");
+		const { teamsOf } = await import("./_DNAIlWyV.js");
 		publish("sessions.changed", {
 			workspaceId,
 			sessions: listSessions(workspaceId).map((s) => ({
@@ -19163,7 +19202,8 @@ ${repoLines}
   \`[${first}#12](https://github.com/…/pull/12) ([review](…))\`). A file is \`/w/${wsId}/files?file=<repo>/<path>&line=<n>\`
   (\`.share/<path>\` for a shared one) and a folder ends in \`/\`. The
   review opens on what changed since the user last marked each topic
-  reviewed: commit your follow-up and the same link shows it.
+  reviewed, or on the whole change if they left it there: commit your
+  follow-up and the same link shows it.
 - **PRs:** only when the user asks; the description is a few sentences on
   what changed and why, no verification logs or file lists.${independentPr}
 - **After a merge** (its PR's, or yours into the branch it stacks on):