@loomcli/loom 0.6.0 → 0.8.0

package/README.md

@@ -2,7 +2,7 @@
 
 The Loom CLI toolchain. Its public surface is `@loomcli/loom/build`, which exports `packet()`.
 
-`packet()` is a `Bun.build` plugin. A Loom application keeps a `loom.packet.json` that reads `{ "build": "development" }` at its package root and passes it to its `Application` as `packet`. While Bun bundles the application, `packet()` writes `distributed` into the packet the bundle carries, so an operator sees one generic message for a defect, and the source file keeps reading `development`. The plugin also carries the data files core's Unicode tables read at run time, so a bundled application needs it to start.
+`packet()` is a `Bun.build` plugin. A Loom application keeps a `loom.packet.json` that reads `{ "build": "development" }` at its package root and passes it to its `Application` as `packet`. While Bun bundles the application, `packet()` writes `distributed` into the packet the bundle carries, so an operator sees one generic message for a defect, and the source file keeps reading `development`. An application bundled without it, by any bundler, starts and reads the source packet, `development`, so it is a development build.
 
 ## Install
 

package/dist/build.d.ts

@@ -8,7 +8,7 @@ interface PacketBuilder {
         filter: RegExp;
     }, callback: (args: PacketLoad) => Promise<{
         contents: string;
-        loader: 'js' | 'json';
+        loader: 'json';
     }>): unknown;
 }
 /**
@@ -24,9 +24,8 @@ interface PacketPlugin {
  * The `Bun.build` plugin that makes a bundle a distributed build. While Bun bundles, it answers the
  * import of every `loom.packet.json` with the file's members and `build` set to `distributed`, and
  * it never writes to the source tree, so the source run keeps reading `development`. A compiled
- * binary built through `Bun.build` with this plugin is distributed too. It also carries the data
- * files core's Unicode tables read beside their own modules into the bundle, without which neither
- * a bundle nor a compiled binary can start.
+ * binary built through `Bun.build` with this plugin is distributed too. It answers no other module,
+ * so the bundle holds every other module as Bun bundles it.
  */
 export declare function packet(): PacketPlugin;
 export type { PacketBuilder, PacketLoad, PacketPlugin };

package/dist/build.js

@@ -1,30 +1,6 @@
 import { readFile } from 'node:fs/promises';
 /** Every module whose file name is the packet's, wherever the entry imports it from. */
 const packetFile = /(?:^|[\\/])loom\.packet\.json$/u;
-/** The generated modules of core's Unicode tables, which read their data files at run time. */
-const unicodeTables = /[\\/]@rockorager[\\/]uucode[\\/]dist[\\/]src[\\/]generated[\\/][\w-]+\.js$/u;
-/** One table module's read of the data file beside it. */
-const tableRead = /JSON\.parse\(readFileSync\(new URL\("\.\/(?<file>[\w-]+\.json)", import\.meta\.url\), "utf8"\)\)/gu;
-/**
- * A table module with each data file it reads beside itself imported as a JSON module instead, so
- * the bundle carries the data. A bundle, and a compiled binary most of all, holds no file beside
- * the module that the original read could find. A table module with no read of that shape fails
- * the build, because a bundle built from it would fail at run time instead.
- */
-async function inlinedTables(path) {
-    const source = await readFile(path, 'utf8');
-    const imports = [];
-    // The one capture is the data file's name, which the callback receives after the whole match.
-    const contents = source.replaceAll(tableRead, (_read, file) => {
-        const name = `table${String(imports.length)}`;
-        imports.push(`import ${name} from './${file}' with { type: 'json' };`);
-        return name;
-    });
-    if (imports.length === 0) {
-        throw new Error(`packet() found no data file read in the Unicode table module ${path}. Update @loomcli/loom to a version that supports the installed @rockorager/uucode.`);
-    }
-    return [...imports, contents].join('\n');
-}
 /** Whether a parsed JSON value is a plain object, whose members the writer keeps. */
 function isRecord(value) {
     return typeof value === 'object' && value !== null && !Array.isArray(value);
@@ -45,9 +21,8 @@ async function distributedPacket(path) {
  * The `Bun.build` plugin that makes a bundle a distributed build. While Bun bundles, it answers the
  * import of every `loom.packet.json` with the file's members and `build` set to `distributed`, and
  * it never writes to the source tree, so the source run keeps reading `development`. A compiled
- * binary built through `Bun.build` with this plugin is distributed too. It also carries the data
- * files core's Unicode tables read beside their own modules into the bundle, without which neither
- * a bundle nor a compiled binary can start.
+ * binary built through `Bun.build` with this plugin is distributed too. It answers no other module,
+ * so the bundle holds every other module as Bun bundles it.
  */
 export function packet() {
     return {
@@ -57,10 +32,6 @@ export function packet() {
                 contents: await distributedPacket(path),
                 loader: 'json',
             }));
-            build.onLoad({ filter: unicodeTables }, async ({ path }) => ({
-                contents: await inlinedTables(path),
-                loader: 'js',
-            }));
         },
     };
 }

package/dist/helpers/release-check.js

@@ -1,4 +1,5 @@
 import { isDeepStrictEqual } from 'node:util';
+import { checkDecisions } from './release-decisions.js';
 import { prepareLockfile, releaseInsertion } from './release-files.js';
 import { prepareRelease, releaseDate, requireReleaseNotes } from './release.js';
 import { currentVersion, git, readLibraries, readRegularFile } from './repository.js';
@@ -43,11 +44,13 @@ export function checkRelease(root, base, head, version, changed) {
             throw new Error(`${library.path}: release cuts change only the version field.`);
         }
     }
+    const decisions = checkDecisions(root, base, head, { date: options.date, version }, changed);
     const allowed = new Set([
         'CHANGELOG.md',
         'pnpm-lock.yaml',
         ...libraries.map((library) => library.path),
         ...release.fragments.map((fragment) => `.changes/${fragment.name}`),
+        ...decisions,
     ]);
     for (const path of changed) {
         if (!allowed.has(path)) {

package/dist/helpers/release-decisions.d.ts

@@ -0,0 +1,9 @@
+/**
+ * Admits the decision files a release cut may change: each record it marks accepted, and the index
+ * row of each one. Any other change to a record or the index throws naming the file. Returns the
+ * admitted paths.
+ */
+export declare function checkDecisions(root: string, base: string, head: string, release: {
+    date: string;
+    version: string;
+}, changed: string[]): string[];

package/dist/helpers/release-decisions.js

@@ -0,0 +1,95 @@
+import { git, readRegularFile } from './repository.js';
+const decisionsDirectory = 'docs/decisions/';
+const decisionIndex = `${decisionsDirectory}README.md`;
+const recordName = /^\d{4}-[^/]+\.md$/u;
+/**
+ * The record a release cut writes when it marks a proposed decision accepted: the status, the
+ * modified date, and a Status section that names the release, with every other line kept.
+ * Undefined when the source is not a proposed record with one modified date and one Status section.
+ */
+function acceptedRecord(source, version, date) {
+    const lines = source.split('\n');
+    const close = lines.indexOf('---', 1);
+    if (lines[0] !== '---' || close === -1) {
+        return undefined;
+    }
+    const field = (key) => lines.slice(1, close).flatMap((line, index) => (line.startsWith(`${key}:`) ? [index + 1] : []));
+    const [status, ...extraStatus] = field('status');
+    const [modified, ...extraModified] = field('modified');
+    const headings = lines.flatMap((line, index) => index > close && line === '## Status' ? [index] : []);
+    const [heading, ...extraHeadings] = headings;
+    if (status === undefined ||
+        lines[status] !== 'status: proposed' ||
+        modified === undefined ||
+        heading === undefined ||
+        extraStatus.length + extraModified.length + extraHeadings.length > 0) {
+        return undefined;
+    }
+    const next = lines.findIndex((line, index) => index > heading && /^#{1,2} /u.test(line));
+    const end = next === -1 ? lines.length : next;
+    const fields = lines.map((line, index) => {
+        if (index === status) {
+            return 'status: accepted';
+        }
+        return index === modified ? `modified: ${date}` : line;
+    });
+    return [
+        ...fields.slice(0, heading + 1),
+        '',
+        `Accepted in ${version}.`,
+        '',
+        ...fields.slice(end),
+    ].join('\n');
+}
+/**
+ * The index with each named record's status cell changed from proposed to accepted. Undefined when
+ * a named record has no single row whose first cell links it and whose last cell reads proposed.
+ */
+function acceptedIndex(source, names) {
+    const lines = source.split('\n');
+    for (const name of names) {
+        const rows = lines.flatMap((line, index) => /^\[[^\]]+\]\((?<target>[^)]+)\)$/u.exec(line.split('|')[1]?.trim() ?? '')?.groups?.target ===
+            name
+            ? [index]
+            : []);
+        const [row, ...extraRows] = rows;
+        const text = row === undefined ? undefined : lines[row];
+        const flipped = text?.replace(/\|(?<before>\s*)proposed(?<after>\s*)\|$/u, '|$1accepted$2|');
+        if (row === undefined || extraRows.length > 0 || flipped === undefined || flipped === text) {
+            return undefined;
+        }
+        lines[row] = flipped;
+    }
+    return lines.join('\n');
+}
+function present(root, ref, path) {
+    return git(root, ['ls-tree', ref, '--', path]) !== '';
+}
+/**
+ * Admits the decision files a release cut may change: each record it marks accepted, and the index
+ * row of each one. Any other change to a record or the index throws naming the file. Returns the
+ * admitted paths.
+ */
+export function checkDecisions(root, base, head, release, changed) {
+    const records = changed.filter((path) => path.startsWith(decisionsDirectory) && recordName.test(path.slice(decisionsDirectory.length)));
+    for (const path of records) {
+        const expected = present(root, base, path) && present(root, head, path)
+            ? acceptedRecord(readRegularFile(root, path, base), release.version, release.date)
+            : undefined;
+        if (expected === undefined || readRegularFile(root, path, head) !== expected) {
+            throw new Error(`${path}: a release cut changes a decision record only to mark a proposed decision accepted, with status accepted, modified ${release.date}, and a Status section that reads "Accepted in ${release.version}.". Make any other change in an ordinary PR.`);
+        }
+    }
+    if (records.length === 0 && !changed.includes(decisionIndex)) {
+        return [];
+    }
+    const expected = present(root, base, decisionIndex)
+        ? acceptedIndex(readRegularFile(root, decisionIndex, base), records.map((path) => path.slice(decisionsDirectory.length)))
+        : undefined;
+    if (expected === undefined ||
+        !present(root, head, decisionIndex) ||
+        readRegularFile(root, decisionIndex, head) !== expected) {
+        throw new Error(`${decisionIndex}: a release cut changes the decision index only to mark each decision it accepts from proposed to accepted. Make any other change in an ordinary PR.`);
+    }
+    return [...records, decisionIndex];
+}

package/dist/helpers/release-files.js

@@ -33,7 +33,7 @@ function pnpmManifestPath(resolve = (specifier) => import.meta.resolve(specifier
         return fileURLToPath(resolve('pnpm/package.json'));
     }
     catch (error) {
-        throw new Error('Preparing pnpm-lock.yaml requires pnpm, an optional peer dependency of @loomcli/loom. Install pnpm 12.3.2 beside @loomcli/loom and run the command again.', { cause: error });
+        throw new Error('Preparing pnpm-lock.yaml requires pnpm, an optional peer dependency of @loomcli/loom. Install pnpm 12.8.1 beside @loomcli/loom and run the command again.', { cause: error });
     }
 }
 function updateLockfile(root) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@loomcli/loom",
-  "version": "0.6.0",
+  "version": "0.8.0",
   "description": "The Loom CLI toolchain. Provides the packet() Bun.build plugin that marks a bundled Loom CLI application as a distributed build.",
   "license": "MIT",
   "repository": {
@@ -19,16 +19,16 @@
     }
   },
   "dependencies": {
-    "@loomcli/core": "0.6.0",
+    "@loomcli/core": "0.8.0",
     "mdast-util-from-markdown": "^2.0.3",
-    "yaml": "^2.9.0",
-    "zod": "^4.5.4"
+    "yaml": "^2.9.1",
+    "zod": "^4.6.5"
   },
   "devDependencies": {
-    "pnpm": "12.3.2"
+    "pnpm": "12.8.1"
   },
   "peerDependencies": {
-    "pnpm": "12.3.2"
+    "pnpm": "12.8.1"
   },
   "peerDependenciesMeta": {
     "pnpm": {