Public: anyone can read this; the crew can change it.

small: the unreached audit names the two files that are entries by design, and the allow list shrinks by two #11469

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

mason/forge:mason/11461-small-the-unreached-audit-names-the-two → main 1 commit fork

  • Merged. mason/11461-small-the-unreached-audit-names-the-two was merged into main as daaa1f6.

Review

Sign in to review this change.
3 files changed +6 −3
scripts/unreached-allow.json −2
@@ -12,8 +12,6 @@
1212 "src/flow/stale_fold.ts",
1313 "src/perf/page_budget.ts",
1414 "src/spawn/pod.ts",
15 "src/tui/gallery.ts",
1615 "src/tui/ui/qr.ts",
17 "src/web/highlight-client.ts",
1816 "src/web/studio/held_posts.ts"
1917 ]
scripts/unreached.ts +4
@@ -21,6 +21,10 @@ export const ENTRIES = [
2121 "src/chaos/run.ts",
2222 "src/release/boot.ts",
2323 "src/git/hook-entry.ts",
24 // Run by hand: `bun run src/tui/gallery.ts` prints every widget.
25 "src/tui/gallery.ts",
26 // Bundled for the browser by Bun.build in src/web/asset.ts, named there by a template path.
27 "src/web/highlight-client.ts",
2428 ];
2529
2630 /** Scripts the check and operators run, and the benches run at release, are callers too: their imports count. */
test/unreached.test.ts +2 −1
@@ -50,8 +50,9 @@ test("unreached() follows imports from the entries and scripts, and nothing else
5050 put("src/types.d.ts");
5151 put("src/core/migrations/001.ts");
5252 put("src/f.ts");
53 put("src/orphan.ts");
5354 put("scripts/tool.ts", 'const { f } = await import("../src/f.ts");\n');
54 expect(unreached(dir)).toEqual(["src/c.ts", "src/d.ts", "src/e.ts", "src/g.ts"]);
55 expect(unreached(dir)).toEqual(["src/c.ts", "src/d.ts", "src/e.ts", "src/g.ts", "src/orphan.ts"]);
5556 } finally {
5657 rmSync(dir, { recursive: true, force: true });
5758 }

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs