8edf498 · cae3b0e
@@ -56,3 +56,15 @@
strip shows the last hunt and the count of open findings.
Left to a person on purpose: ingress routes at the edge are not in the
database, and the sentinel opens no pull itself — a finding with a mechanical
fix says what it is, and a run of the crew picks the issue up.
## Hot files (#5934, #6235)
A build is not started while a file its issue names is touched by 5 or more
open pulls (`HOT_PULLS`). A pull whose diff against the base tip touches more
than 60 files (`WIDE_PULL`) counts toward no file's heat: such a diff is
almost always the base's history on a conflicted pull, not its own work, and
five of them would make hundreds of files hot and hold the whole map. It is
named instead, with its file count, on the feed post that says what waits
(`too wide to count: #4840 (1,081 files)`), so a person can close or rebase
it. A real sweeping change, such as a rename across the tree, goes uncounted
too; it conflicts with everything anyway, and it is named.
@@ -92,4 +92,6 @@
the repository paths its title and body name (`src/http/mcp.ts`,
before the picker sorts. The tick line names the hot files (`hot=src/http/mcp.ts=61`),
and each repository's writers see one post on the feed — on the web and
through the MCP `feed` tool — saying which build waits on which file, gone
when the hold lifts.
when the hold lifts. A pull whose diff touches more than 60 files counts
toward no file's heat and is named on the feed (`too wide to count: #4840
(1,081 files)`), even when no build is held.
@@ -23,6 +23,13 @@
import { runGitCapture } from "../git/env.ts";
/** A file this many open pulls touch is hot: no new build starts on it. */
export const HOT_PULLS = 5;
/**
 * A pull whose diff touches more than this many files counts toward no file's
 * heat (#6235): such a diff carries the base's history (a conflicted pull
 * never rebased), not its own work, and would make every file hot.
 */
export const WIDE_PULL = 60;
/** The most paths read from one issue: a list longer than this is a manifest, not a plan. */
const MAX_EXPECTED = 40;
@@ -52,7 +59,7 @@
export interface HotFile {
}
/** Per repository, per open pull: the head and base it was read at, and the files. */
const touched = new Map<string, Map<string, { at: string; files: readonly string[] }>>();
const touched = new Map<string, Map<string, { at: string; number: number; files: readonly string[] }>>();
const OID = /^[0-9a-f]{40,64}$/;
@@ -71,7 +78,7 @@
export async function learnPullFiles(app: App, repo: Repo): Promise<void> {
    .map((r) => r.id);
  const pulls = joinMany(app.db, [...getIssuesByIds(app.db, ids).values()]);
  const before = touched.get(repo.id) ?? new Map();
  const after = new Map<string, { at: string; files: readonly string[] }>();
  const after = new Map<string, { at: string; number: number; files: readonly string[] }>();
  for (const pull of pulls) {
    if (!OID.test(pull.headOid)) continue;
    const dir = await pullDir(app.db, app.reposRoot, repo, pull);
@@ -89,7 +96,7 @@
export async function learnPullFiles(app: App, repo: Repo): Promise<void> {
    }
    const out = await runGitCapture(["diff", "--name-only", "-z", at], { cwd: dir });
    if (out.code !== 0) continue;
    after.set(pull.id, { at, files: out.stdout.split("\0").filter((f) => f !== "") });
    after.set(pull.id, { at, number: pull.number, files: out.stdout.split("\0").filter((f) => f !== "") });
  }
  touched.set(repo.id, after);
}
@@ -97,12 +104,35 @@
export async function learnPullFiles(app: App, repo: Repo): Promise<void> {
/** Every file of `repo` that `HOT_PULLS` or more open pulls touch, with how many. */
export function hotFiles(repoId: string): Map<string, number> {
  const counts = new Map<string, number>();
  for (const { files } of touched.get(repoId)?.values() ?? [])
  for (const { files } of touched.get(repoId)?.values() ?? []) {
    if (files.length > WIDE_PULL) continue;
    for (const file of files) counts.set(file, (counts.get(file) ?? 0) + 1);
  }
  for (const [file, n] of counts) if (n < HOT_PULLS) counts.delete(file);
  return counts;
}
/** An open pull too wide to count toward heat, with the files its diff touches. */
export interface WidePull {
  readonly number: number;
  readonly files: number;
}
/** The open pulls of `repoId` wider than `WIDE_PULL`, widest first. */
export function widePulls(repoId: string): WidePull[] {
  return [...(touched.get(repoId)?.values() ?? [])]
    .filter((p) => p.files.length > WIDE_PULL)
    .map((p) => ({ number: p.number, files: p.files.length }))
    .sort((a, b) => b.files - a.files || a.number - b.number);
}
/** `too wide to count: #4840 (1,081 files), #5460 (898 files)`, or "" when none. */
export function wideSays(wide: readonly WidePull[]): string {
  if (wide.length === 0) return "";
  const list = wide.map((w) => `#${w.number} (${w.files.toLocaleString("en-US")} files)`).join(", ");
  return `too wide to count: ${list}`;
}
/** The hot files `issue` expects to touch, hottest first. */
export function hotOverlap(
  issue: Pick<Issue, "title" | "body">,
@@ -124,11 +154,22 @@
export interface HotHold {
  readonly files: readonly HotFile[];
}
/** The pulls too wide to count as the picker last saw them, per repository, held builds or not. */
const standingWideBy = new Map<string, { at: string; wide: readonly WidePull[] }>();
/** The pulls too wide to count in `repoId` as the picker last saw them, if any. */
export function standingWide(repoId: string): { at: string; wide: readonly WidePull[] } | undefined {
  return standingWideBy.get(repoId);
}
/** What the picker last held on hot files, per repository, and when. */
const standing = new Map<string, { at: string; holds: readonly HotHold[] }>();
const standing = new Map<string, { at: string; holds: readonly HotHold[]; wide: readonly WidePull[] }>();
      
        junipercrew
        19d ago
        
      
      wide here duplicates standingWideBy; nothing reads it. Remove the field and keep standingHot returning its old shape.
    
/** Remember the builds the picker holds on hot files in `repoId` now; none clears it. */
export function holdHot(repoId: string, issues: readonly Issue[], at = new Date()): void {
  const wide = widePulls(repoId);
  if (wide.length === 0) standingWideBy.delete(repoId);
  else standingWideBy.set(repoId, { at: at.toISOString(), wide });
  if (issues.length === 0) {
    standing.delete(repoId);
    return;
@@ -137,11 +178,14 @@
export function holdHot(repoId: string, issues: readonly Issue[], at = new Date(
  standing.set(repoId, {
    at: at.toISOString(),
    holds: issues.map((i) => ({ number: i.number, title: i.title, files: hotOverlap(i, hot) })),
    wide,
  });
}
/** The builds held on hot files in `repoId` as the picker last said, if any. */
export function standingHot(repoId: string): { at: string; holds: readonly HotHold[] } | undefined {
export function standingHot(
  repoId: string,
): { at: string; holds: readonly HotHold[]; wide: readonly WidePull[] } | undefined {
  return standing.get(repoId);
}
@@ -33,7 +33,7 @@
import { chooseCandidates } from "./choose.ts";
import { writeStamp } from "../core/write_clock.ts";
import { Stamped } from "../kernel/stamped.ts";
import { QUIET_WINDOW_MS, quietRunEventIds } from "./quiet.ts";
import { hotSays, standingHot } from "../crew/hot_files.ts";
import { hotSays, standingHot, standingWide, wideSays } from "../crew/hot_files.ts";
import { ago } from "./rank.ts";
import { askLinesFor } from "../web/studio/asks.ts";
@@ -278,10 +278,18 @@
function hotCandidates(writable: readonly Repo[]): Candidate[] {
  const out: Candidate[] = [];
  for (const repo of writable) {
    const standing = standingHot(repo.id);
    if (standing === undefined) continue;
    const n = standing.holds.length;
    const first = standing.holds[0]!;
    const what = `${n === 1 ? "1 build waits" : `${n} builds wait`} on hot files: ${hotSays(first)}${n > 1 ? ` And ${n - 1} more.` : ""}`;
    const wide = standingWide(repo.id);
    if (standing === undefined && wide === undefined) continue;
    const wideSuffix = wide === undefined ? "" : wideSays(wide.wide);
    let what: string;
      
        junipercrew
        19d ago
        outdated
      
      This line is only reached when standing exists, so wide pulls are named only while some build is held. Emit the wide post when nothing is held as well. Also, this line is now long and nested; pull the wide suffix out into a variable.
    
    if (standing === undefined) {
      what = `${wideSuffix} - close or rebase.`;
    } else {
      const n = standing.holds.length;
      const more = n > 1 ? ` And ${n - 1} more.` : "";
      const held = `${n === 1 ? "1 build waits" : `${n} builds wait`} on hot files: ${hotSays(standing.holds[0]!)}${more}`;
      what = wideSuffix === "" ? held : `${held} ${wideSuffix}.`;
      
        junipercrew
        19d ago
        
      
      The held text plus the wide suffix goes through clip, which can drop the wide part on a long title. Put the wide part where clipping cannot remove it, and add a test for the held case.
    
    }
    out.push({
      id: `hot:${repo.id}`,
      kind: "map",
@@ -289,7 +297,7 @@
function hotCandidates(writable: readonly Repo[]): Candidate[] {
      app: slug(repo),
      actorId: null,
      actor: null,
      at: standing.at,
      at: (standing ?? wide!).at,
      link: `${base(repo)}/map`,
      what: clip(what),
      urgency: 0,
@@ -26,9 +26,13 @@
import {
  expectedFiles,
  HOT_PULLS,
  hotFiles,
  holdHot,
  hotOverlap,
  learnPullFiles,
  standingHot,
  WIDE_PULL,
  widePulls,
  wideSays,
} from "../src/crew/hot_files.ts";
import { readFeed } from "../src/feed/feed.ts";
import { serve, type Servers } from "../src/http/server.ts";
@@ -192,4 +196,56 @@
describe("the picker holds a build on a hot file", () => {
    expect(standingHot(repo.id)).toBeUndefined();
    expect(readFeed(app.db, alice, { limit: 50 }).items.some((i) => i.id === `hot:${repo.id}`)).toBe(false);
  });
  test("a pull wider than WIDE_PULL counts toward no file's heat and is named", async () => {
    const made = await createRepo(app, alice, { name: "wide", visibility: "private" });
    if (!made.ok) throw new Error(String(made.error));
    const repo = fresh(made.value);
    const seeded = await commitToBranch(app, alice, repo, {
      message: "Start",
      files: [{ path: "src/hot.ts", content: "export const n = 0;\n" }],
    });
    if (!seeded.ok) throw new Error(JSON.stringify(seeded.error));
    const wide = async (n: number): Promise<Pull> => {
      const files = Array.from({ length: WIDE_PULL + 1 }, (_, k) => ({
        path: k === 0 ? "src/hot.ts" : `gen/${n}/f${k}.ts`,
        content: `export const n = ${n};\n`,
      }));
      const branch = await commitToBranch(app, alice, fresh(repo), {
        branch: `wide/${n}`,
        base: "main",
        message: `Wide ${n}`,
        files,
      });
      if (!branch.ok) throw new Error(JSON.stringify(branch.error));
      const pull = await openPull(app, alice, fresh(repo), {
        title: `Wide ${n}`,
        body: "",
        baseBranch: "main",
        headBranch: `wide/${n}`,
      });
      if (!pull.ok) throw new Error(String(pull.error));
      return pull.value;
    };
    const wides: Pull[] = [];
    for (let n = 1; n <= HOT_PULLS; n++) wides.push(await wide(n));
    wides.push(await pullOn(repo, 90, "src/hot.ts"));
    await learnPullFiles(app, fresh(repo));
    expect(hotFiles(repo.id).size).toBe(0);
    expect(widePulls(repo.id)).toHaveLength(HOT_PULLS);
    // No build is held, yet the feed still names them.
    const work = issue(repo, "Add a flag to src/hot.ts");
    expect(heldNow(app, fresh(repo)).by.hot_file).toEqual([]);
    holdHot(repo.id, []);
    const said = widePulls(repo.id).map((w) => `#${w.number} (${w.files} files)`).join(", ");
    const post = readFeed(app.db, alice, { limit: 50 }).items.find((i) => i.id === `hot:${repo.id}`);
    expect(post?.what).toBe(`too wide to count: ${said} - close or rebase.`);
    expect(wideSays([{ number: 4840, files: 1081 }])).toBe("too wide to count: #4840 (1,081 files)");
    // Four ordinary pulls plus one more on the same file still hold it.
    for (let n = 91; n <= 94; n++) await pullOn(repo, n, "src/hot.ts");
    await learnPullFiles(app, fresh(repo));
    expect(hotFiles(repo.id)).toEqual(new Map([["src/hot.ts", 5]]));
    expect(heldNow(app, fresh(repo)).by.hot_file.map((i) => i.number)).toEqual([work.number]);
  });
});
		@@ -56,3 +56,15 @@ strip shows the last hunt and the count of open findings.
56	56	Left to a person on purpose: ingress routes at the edge are not in the
57	57	database, and the sentinel opens no pull itself — a finding with a mechanical
58	58	fix says what it is, and a run of the crew picks the issue up.
	59
	60	## Hot files (#5934, #6235)
	61
	62	A build is not started while a file its issue names is touched by 5 or more
	63	open pulls (`HOT_PULLS`). A pull whose diff against the base tip touches more
	64	than 60 files (`WIDE_PULL`) counts toward no file's heat: such a diff is
	65	almost always the base's history on a conflicted pull, not its own work, and
	66	five of them would make hundreds of files hot and hold the whole map. It is
	67	named instead, with its file count, on the feed post that says what waits
	68	(`too wide to count: #4840 (1,081 files)`), so a person can close or rebase
	69	it. A real sweeping change, such as a rename across the tree, goes uncounted
	70	too; it conflicts with everything anyway, and it is named.
		@@ -92,4 +92,6 @@ the repository paths its title and body name (`src/http/mcp.ts`,
92	92	before the picker sorts. The tick line names the hot files (`hot=src/http/mcp.ts=61`),
93	93	and each repository's writers see one post on the feed — on the web and
94	94	through the MCP `feed` tool — saying which build waits on which file, gone
95		when the hold lifts.
	95	when the hold lifts. A pull whose diff touches more than 60 files counts
	96	toward no file's heat and is named on the feed (`too wide to count: #4840
	97	(1,081 files)`), even when no build is held.
		@@ -23,6 +23,13 @@ import { runGitCapture } from "../git/env.ts";
23	23	/** A file this many open pulls touch is hot: no new build starts on it. */
24	24	export const HOT_PULLS = 5;
25	25
	26	/**
	27	* A pull whose diff touches more than this many files counts toward no file's
	28	* heat (#6235): such a diff carries the base's history (a conflicted pull
	29	* never rebased), not its own work, and would make every file hot.
	30	*/
	31	export const WIDE_PULL = 60;
	32
26	33	/** The most paths read from one issue: a list longer than this is a manifest, not a plan. */
27	34	const MAX_EXPECTED = 40;
28	35
		@@ -52,7 +59,7 @@ export interface HotFile {
52	59	}
53	60
54	61	/** Per repository, per open pull: the head and base it was read at, and the files. */
55		const touched = new Map<string, Map<string, { at: string; files: readonly string[] }>>();
	62	const touched = new Map<string, Map<string, { at: string; number: number; files: readonly string[] }>>();
56	63
57	64	const OID = /^[0-9a-f]{40,64}$/;
58	65
		@@ -71,7 +78,7 @@ export async function learnPullFiles(app: App, repo: Repo): Promise<void> {
71	78	.map((r) => r.id);
72	79	const pulls = joinMany(app.db, [...getIssuesByIds(app.db, ids).values()]);
73	80	const before = touched.get(repo.id) ?? new Map();
74		const after = new Map<string, { at: string; files: readonly string[] }>();
	81	const after = new Map<string, { at: string; number: number; files: readonly string[] }>();
75	82	for (const pull of pulls) {
76	83	if (!OID.test(pull.headOid)) continue;
77	84	const dir = await pullDir(app.db, app.reposRoot, repo, pull);
		@@ -89,7 +96,7 @@ export async function learnPullFiles(app: App, repo: Repo): Promise<void> {
89	96	}
90	97	const out = await runGitCapture(["diff", "--name-only", "-z", at], { cwd: dir });
91	98	if (out.code !== 0) continue;
92		after.set(pull.id, { at, files: out.stdout.split("\0").filter((f) => f !== "") });
	99	after.set(pull.id, { at, number: pull.number, files: out.stdout.split("\0").filter((f) => f !== "") });
93	100	}
94	101	touched.set(repo.id, after);
95	102	}
		@@ -97,12 +104,35 @@ export async function learnPullFiles(app: App, repo: Repo): Promise<void> {
97	104	/** Every file of `repo` that `HOT_PULLS` or more open pulls touch, with how many. */
98	105	export function hotFiles(repoId: string): Map<string, number> {
99	106	const counts = new Map<string, number>();
100		for (const { files } of touched.get(repoId)?.values() ?? [])
	107	for (const { files } of touched.get(repoId)?.values() ?? []) {
	108	if (files.length > WIDE_PULL) continue;
101	109	for (const file of files) counts.set(file, (counts.get(file) ?? 0) + 1);
	110	}
102	111	for (const [file, n] of counts) if (n < HOT_PULLS) counts.delete(file);
103	112	return counts;
104	113	}
105	114
	115	/** An open pull too wide to count toward heat, with the files its diff touches. */
	116	export interface WidePull {
	117	readonly number: number;
	118	readonly files: number;
	119	}
	120
	121	/** The open pulls of `repoId` wider than `WIDE_PULL`, widest first. */
	122	export function widePulls(repoId: string): WidePull[] {
	123	return [...(touched.get(repoId)?.values() ?? [])]
	124	.filter((p) => p.files.length > WIDE_PULL)
	125	.map((p) => ({ number: p.number, files: p.files.length }))
	126	.sort((a, b) => b.files - a.files \|\| a.number - b.number);
	127	}
	128
	129	/** `too wide to count: #4840 (1,081 files), #5460 (898 files)`, or "" when none. */
	130	export function wideSays(wide: readonly WidePull[]): string {
	131	if (wide.length === 0) return "";
	132	const list = wide.map((w) => `#${w.number} (${w.files.toLocaleString("en-US")} files)`).join(", ");
	133	return `too wide to count: ${list}`;
	134	}
	135
106	136	/** The hot files `issue` expects to touch, hottest first. */
107	137	export function hotOverlap(
108	138	issue: Pick<Issue, "title" \| "body">,
		@@ -124,11 +154,22 @@ export interface HotHold {
124	154	readonly files: readonly HotFile[];
125	155	}
126	156
	157	/** The pulls too wide to count as the picker last saw them, per repository, held builds or not. */
	158	const standingWideBy = new Map<string, { at: string; wide: readonly WidePull[] }>();
	159
	160	/** The pulls too wide to count in `repoId` as the picker last saw them, if any. */
	161	export function standingWide(repoId: string): { at: string; wide: readonly WidePull[] } \| undefined {
	162	return standingWideBy.get(repoId);
	163	}
	164
127	165	/** What the picker last held on hot files, per repository, and when. */
128		const standing = new Map<string, { at: string; holds: readonly HotHold[] }>();
	166	const standing = new Map<string, { at: string; holds: readonly HotHold[]; wide: readonly WidePull[] }>();
		junipercrew 19d ago `wide` here duplicates standingWideBy; nothing reads it. Remove the field and keep standingHot returning its old shape.
129	167
130	168	/** Remember the builds the picker holds on hot files in `repoId` now; none clears it. */
131	169	export function holdHot(repoId: string, issues: readonly Issue[], at = new Date()): void {
	170	const wide = widePulls(repoId);
	171	if (wide.length === 0) standingWideBy.delete(repoId);
	172	else standingWideBy.set(repoId, { at: at.toISOString(), wide });
132	173	if (issues.length === 0) {
133	174	standing.delete(repoId);
134	175	return;
		@@ -137,11 +178,14 @@ export function holdHot(repoId: string, issues: readonly Issue[], at = new Date(
137	178	standing.set(repoId, {
138	179	at: at.toISOString(),
139	180	holds: issues.map((i) => ({ number: i.number, title: i.title, files: hotOverlap(i, hot) })),
	181	wide,
140	182	});
141	183	}
142	184
143	185	/** The builds held on hot files in `repoId` as the picker last said, if any. */
144		export function standingHot(repoId: string): { at: string; holds: readonly HotHold[] } \| undefined {
	186	export function standingHot(
	187	repoId: string,
	188	): { at: string; holds: readonly HotHold[]; wide: readonly WidePull[] } \| undefined {
145	189	return standing.get(repoId);
146	190	}
147	191
		@@ -33,7 +33,7 @@ import { chooseCandidates } from "./choose.ts";
33	33	import { writeStamp } from "../core/write_clock.ts";
34	34	import { Stamped } from "../kernel/stamped.ts";
35	35	import { QUIET_WINDOW_MS, quietRunEventIds } from "./quiet.ts";
36		import { hotSays, standingHot } from "../crew/hot_files.ts";
	36	import { hotSays, standingHot, standingWide, wideSays } from "../crew/hot_files.ts";
37	37	import { ago } from "./rank.ts";
38	38	import { askLinesFor } from "../web/studio/asks.ts";
39	39
		@@ -278,10 +278,18 @@ function hotCandidates(writable: readonly Repo[]): Candidate[] {
278	278	const out: Candidate[] = [];
279	279	for (const repo of writable) {
280	280	const standing = standingHot(repo.id);
281		if (standing === undefined) continue;
282		const n = standing.holds.length;
283		const first = standing.holds[0]!;
284		const what = `${n === 1 ? "1 build waits" : `${n} builds wait`} on hot files: ${hotSays(first)}${n > 1 ? ` And ${n - 1} more.` : ""}`;
	281	const wide = standingWide(repo.id);
	282	if (standing === undefined && wide === undefined) continue;
	283	const wideSuffix = wide === undefined ? "" : wideSays(wide.wide);
	284	let what: string;
		junipercrew 19d ago outdated This line is only reached when standing exists, so wide pulls are named only while some build is held. Emit the wide post when nothing is held as well. Also, this line is now long and nested; pull the wide suffix out into a variable.
	285	if (standing === undefined) {
	286	what = `${wideSuffix} - close or rebase.`;
	287	} else {
	288	const n = standing.holds.length;
	289	const more = n > 1 ? ` And ${n - 1} more.` : "";
	290	const held = `${n === 1 ? "1 build waits" : `${n} builds wait`} on hot files: ${hotSays(standing.holds[0]!)}${more}`;
	291	what = wideSuffix === "" ? held : `${held} ${wideSuffix}.`;
		junipercrew 19d ago The held text plus the wide suffix goes through `clip`, which can drop the wide part on a long title. Put the wide part where clipping cannot remove it, and add a test for the held case.
	292	}
285	293	out.push({
286	294	id: `hot:${repo.id}`,
287	295	kind: "map",
		@@ -289,7 +297,7 @@ function hotCandidates(writable: readonly Repo[]): Candidate[] {
289	297	app: slug(repo),
290	298	actorId: null,
291	299	actor: null,
292		at: standing.at,
	300	at: (standing ?? wide!).at,
293	301	link: `${base(repo)}/map`,
294	302	what: clip(what),
295	303	urgency: 0,
		@@ -26,9 +26,13 @@ import {
26	26	expectedFiles,
27	27	HOT_PULLS,
28	28	hotFiles,
	29	holdHot,
29	30	hotOverlap,
30	31	learnPullFiles,
31	32	standingHot,
	33	WIDE_PULL,
	34	widePulls,
	35	wideSays,
32	36	} from "../src/crew/hot_files.ts";
33	37	import { readFeed } from "../src/feed/feed.ts";
34	38	import { serve, type Servers } from "../src/http/server.ts";
		@@ -192,4 +196,56 @@ describe("the picker holds a build on a hot file", () => {
192	196	expect(standingHot(repo.id)).toBeUndefined();
193	197	expect(readFeed(app.db, alice, { limit: 50 }).items.some((i) => i.id === `hot:${repo.id}`)).toBe(false);
194	198	});
	199
	200	test("a pull wider than WIDE_PULL counts toward no file's heat and is named", async () => {
	201	const made = await createRepo(app, alice, { name: "wide", visibility: "private" });
	202	if (!made.ok) throw new Error(String(made.error));
	203	const repo = fresh(made.value);
	204	const seeded = await commitToBranch(app, alice, repo, {
	205	message: "Start",
	206	files: [{ path: "src/hot.ts", content: "export const n = 0;\n" }],
	207	});
	208	if (!seeded.ok) throw new Error(JSON.stringify(seeded.error));
	209	const wide = async (n: number): Promise<Pull> => {
	210	const files = Array.from({ length: WIDE_PULL + 1 }, (_, k) => ({
	211	path: k === 0 ? "src/hot.ts" : `gen/${n}/f${k}.ts`,
	212	content: `export const n = ${n};\n`,
	213	}));
	214	const branch = await commitToBranch(app, alice, fresh(repo), {
	215	branch: `wide/${n}`,
	216	base: "main",
	217	message: `Wide ${n}`,
	218	files,
	219	});
	220	if (!branch.ok) throw new Error(JSON.stringify(branch.error));
	221	const pull = await openPull(app, alice, fresh(repo), {
	222	title: `Wide ${n}`,
	223	body: "",
	224	baseBranch: "main",
	225	headBranch: `wide/${n}`,
	226	});
	227	if (!pull.ok) throw new Error(String(pull.error));
	228	return pull.value;
	229	};
	230	const wides: Pull[] = [];
	231	for (let n = 1; n <= HOT_PULLS; n++) wides.push(await wide(n));
	232	wides.push(await pullOn(repo, 90, "src/hot.ts"));
	233	await learnPullFiles(app, fresh(repo));
	234	expect(hotFiles(repo.id).size).toBe(0);
	235	expect(widePulls(repo.id)).toHaveLength(HOT_PULLS);
	236	// No build is held, yet the feed still names them.
	237	const work = issue(repo, "Add a flag to src/hot.ts");
	238	expect(heldNow(app, fresh(repo)).by.hot_file).toEqual([]);
	239	holdHot(repo.id, []);
	240	const said = widePulls(repo.id).map((w) => `#${w.number} (${w.files} files)`).join(", ");
	241	const post = readFeed(app.db, alice, { limit: 50 }).items.find((i) => i.id === `hot:${repo.id}`);
	242	expect(post?.what).toBe(`too wide to count: ${said} - close or rebase.`);
	243	expect(wideSays([{ number: 4840, files: 1081 }])).toBe("too wide to count: #4840 (1,081 files)");
	244
	245	// Four ordinary pulls plus one more on the same file still hold it.
	246	for (let n = 91; n <= 94; n++) await pullOn(repo, n, "src/hot.ts");
	247	await learnPullFiles(app, fresh(repo));
	248	expect(hotFiles(repo.id)).toEqual(new Map([["src/hot.ts", 5]]));
	249	expect(heldNow(app, fresh(repo)).by.hot_file.map((i) => i.number)).toEqual([work.number]);
	250	});
195	251	});