@north-light/crouter 0.3.319 → 0.3.320

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.319",
+  "version": "0.3.320",
   "description": "crtr — agent runtime with memory, plugins, and marketplaces",
   "type": "module",
   "main": "dist/index.js",
@@ -68,6 +68,7 @@
     "scripts/postinstall.mjs",
     "scripts/patch-pi-tree-root-ordering.mjs",
     "scripts/install-runtime.mjs",
+    "scripts/pack-minify.mjs",
     "scripts/runtime-lock.mjs",
     "scripts/self-update.mjs",
     "runtime.lock.json"

package/runtime.lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.319",
+  "version": "0.3.320",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@north-light/crouter",
-      "version": "0.3.319",
+      "version": "0.3.320",
       "hasInstallScript": true,
       "license": "MIT",
       "workspaces": [
@@ -5206,17 +5206,17 @@
     },
     "packages/crouter-api": {
       "name": "@north-light/crouter-api",
-      "version": "0.3.319",
+      "version": "0.3.320",
       "license": "UNLICENSED"
     },
     "packages/crouter-env-docker": {
       "name": "@north-light/crouter-env-docker",
-      "version": "0.3.319",
+      "version": "0.3.320",
       "license": "UNLICENSED"
     },
     "packages/crouter-sdk": {
       "name": "@north-light/crouter-sdk",
-      "version": "0.3.319",
+      "version": "0.3.320",
       "license": "UNLICENSED",
       "dependencies": {
         "@north-light/crouter-api": "^0.3.295"

package/scripts/pack-minify.mjs

@@ -0,0 +1,108 @@
+#!/usr/bin/env node
+// Minify the packed dist, and ONLY when CRTR_PACK_MINIFY=1.
+//
+// OFF BY DEFAULT because `npm run install-runtime` builds every local
+// generation by running `npm pack` on this repo (scripts/install-runtime.mjs,
+// `pack()`). An unconditional prepack minify would therefore minify the
+// runtime a developer actually runs, and with no source maps emitted and
+// `src/` not packed, every stack trace in a live daemon log would collapse to
+// `file.js:1:<column>`. The flag keeps local generations readable and moves
+// minification to the published tarball alone.
+//
+// .github/workflows/publish.yml sets the flag in BOTH places that produce a
+// release tarball — the `publish` job and the docker job's `npm pack` step —
+// because docker/README.md's CRTR_PACKAGE_FILE path depends on those two
+// tarballs being identical.
+//
+// This script must stay listed in package.json#files. `prepack` runs it, and
+// scripts/install-runtime.mjs builds a generation by running `npm pack` on the
+// INSTALLED package — so a published tarball without this file makes both the
+// crtrd image build and `crtr sys self-update` fail on a missing module.
+//
+// `keepNames` is not optional. Function and class names survive into stack
+// traces and are what makes a daemon log from a published generation legible
+// at all, since no source maps are emitted and `src/` is not packed.
+import { cp, readFile, rename, rm, stat, writeFile } from 'node:fs/promises';
+import { globSync } from 'node:fs';
+import { dirname, join, resolve } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { transform } from 'esbuild';
+
+const ROOT = resolve(dirname(fileURLToPath(import.meta.url)), '..');
+const DIST = join(ROOT, 'dist');
+const BACKUP = join(ROOT, '.dist-unminified');
+
+async function isDirectory(path) {
+  try {
+    return (await stat(path)).isDirectory();
+  } catch {
+    return false;
+  }
+}
+
+/** Files the tarball ships as executable JavaScript.
+ *
+ *  - `builtin-pi-packages/` is copied source that pi loads on its own terms,
+ *    not part of the compiled graph.
+ *  - `clients/attach/viewer.js` is already an esbuild bundle built with
+ *    `minify: true`; re-minifying 9 MiB buys nothing and risks its banner.
+ *  - `__tests__/` is excluded from `package.json#files`, so it is not packed. */
+function packedJavaScript() {
+  return globSync('**/*.js', { cwd: DIST }).filter(
+    (rel) =>
+      !rel.startsWith('builtin-pi-packages/') &&
+      rel !== 'clients/attach/viewer.js' &&
+      !rel.split('/').includes('__tests__'),
+  );
+}
+
+async function restore() {
+  if (!(await isDirectory(BACKUP))) return false;
+  await rm(DIST, { recursive: true, force: true });
+  await rename(BACKUP, DIST);
+  return true;
+}
+
+const mode = process.argv[2];
+
+if (mode === 'prepack') {
+  // A pack that crashed before postpack leaves a minified dist beside the
+  // backup. Restore first — unconditionally, even when minification is off —
+  // so this run minifies readable input rather than minifying already-minified
+  // output and losing the readable tree for good.
+  await restore();
+  if (process.env.CRTR_PACK_MINIFY !== '1') process.exit(0);
+  if (!(await isDirectory(DIST))) throw new Error('dist/ is missing — run `npm run build` before packing');
+
+  await cp(DIST, BACKUP, { recursive: true });
+  try {
+    const files = packedJavaScript();
+    let before = 0;
+    let after = 0;
+    await Promise.all(
+      files.map(async (rel) => {
+        const path = join(DIST, rel);
+        const source = await readFile(path, 'utf8');
+        before += Buffer.byteLength(source);
+        const { code } = await transform(source, {
+          minify: true,
+          keepNames: true,
+          format: 'esm',
+          target: 'node22',
+          loader: 'js',
+        });
+        after += Buffer.byteLength(code);
+        await writeFile(path, code);
+      }),
+    );
+    const mib = (bytes) => (bytes / 1048576).toFixed(1);
+    console.log(`pack-minify: ${files.length} files, ${mib(before)} MiB -> ${mib(after)} MiB`);
+  } catch (error) {
+    await restore();
+    throw error;
+  }
+} else if (mode === 'postpack') {
+  await restore();
+} else {
+  throw new Error('usage: pack-minify.mjs <prepack|postpack>');
+}