@tech-leads-club/harness-toolkit 0.3.1 → 0.3.2

package/bin/tlc-build.mjs

@@ -37,14 +37,37 @@ function sourcesIn(dir) {
     .map((entry) => ({ name: basename(entry.name, ".ts"), source: join(dir, entry.name) }));
 }
 
-function buildOne(source, out) {
+/**
+ * why one invocation with every entry: `--splitting` can only share a chunk between entries it sees together.
+ * Built one at a time, each bundle inlined the whole core — measured, 24 bundles of ~257 KB each where more than
+ * half of every one was byte-identical to its neighbour: 5.0 MB of `dist/` to carry 548 KB of distinct code
+ * ([/decisions/ad-098.md](/decisions/ad-098.md)).
+ *
+ * invariant: the entry names stay flat and keep `.mjs`, because the launcher resolves `dist/<entry>.mjs` and the
+ * hooks on every installed machine name the launcher. Chunks go in a subdirectory, so pruning an orphan entry
+ * never has to tell the two apart.
+ */
+const CHUNK_DIR = "chunks";
+
+function build(targets) {
   const result = spawnSync(
     "bun",
-    ["build", "--target=node", "--format=esm", `--outfile=${out}`, source],
+    [
+      "build",
+      "--target=node",
+      "--format=esm",
+      "--splitting",
+      `--outdir=${dist}`,
+      "--entry-naming",
+      "[name].mjs",
+      "--chunk-naming",
+      `${CHUNK_DIR}/[name]-[hash].mjs`,
+      ...targets.map((target) => target.source),
+    ],
     { stdio: "inherit" },
   );
   if (result.error?.code === "ENOENT") {
-    console.error("tlc-build: Bun is not on PATH, and dist/ is committed so the bundler is part of the artefact.");
+    console.error("tlc-build: Bun is not on PATH, and it is the only bundler this builds with.");
     console.error("  curl -fsSL https://bun.sh/install | bash");
     process.exit(1);
   }
@@ -61,9 +84,15 @@ const targets = [
 
 mkdirSync(dist, { recursive: true });
 console.log(`tlc-build → ${dist}`);
-for (const target of targets) {
-  buildOne(target.source, join(dist, `${target.name}.mjs`));
-}
+
+/**
+ * hazard: chunk names carry a content hash, so a rebuild after a source change writes new ones and leaves the old
+ * ones behind. Nothing references them and they ship anyway, growing the package with every build.
+ *
+ * invariant: the chunk directory is derived, so it is replaced rather than merged.
+ */
+rmSync(join(dist, CHUNK_DIR), { recursive: true, force: true });
+build(targets);
 
 // invariant: the launcher stays executable on a filesystem that tracks the bit. A no-op where it does not.
 try {