@north-light/crouter 0.3.166 → 0.3.167

package/dist/core/__tests__/history-inbox.test.js

@@ -6,11 +6,12 @@
 // sender (`from`) is visible in both the read body and the artifact title.
 import { test, before, after, beforeEach } from 'node:test';
 import assert from 'node:assert/strict';
-import { mkdtempSync, rmSync } from 'node:fs';
+import { mkdtempSync, rmSync, writeFileSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { createNode } from '../canvas/canvas.js';
 import { closeDb } from '../canvas/db.js';
+import { contextDir } from '../canvas/paths.js';
 import { appendInbox } from '../feed/inbox.js';
 import { buildCorpus, resolveRef } from '../canvas/history.js';
 let home;
@@ -39,6 +40,15 @@ after(() => {
     rmSync(home, { recursive: true, force: true });
     delete process.env['CRTR_HOME'];
 });
+test('history corpus skips malformed frontmatter in another cwd', () => {
+    const broken = createNode(node('broken-frontmatter', '/tmp/hist-broken'));
+    const valid = createNode(node('valid-history', '/tmp/hist-valid'));
+    writeFileSync(join(contextDir(broken.node_id), 'copied-source.md'), '---\ndesc: "unterminated\n---\nsource text\n');
+    writeFileSync(join(contextDir(valid.node_id), 'result.md'), '---\ntitle: result\n---\n0a0afd613\n');
+    const corpus = buildCorpus({ allCwds: true }, { types: ['report', 'doc', 'roadmap'] });
+    assert.equal(corpus.some((a) => a.ref === `${broken.node_id}:context/copied-source.md`), false, 'invalid artifact is skipped');
+    assert.ok(corpus.some((a) => a.loadBody().includes('0a0afd613')), 'valid artifacts remain searchable');
+});
 test('default history excludes inbox; --type inbox includes it, with full sender-attributed bodies', () => {
     const cwd = '/tmp/hist-work';
     createNode(node('sender-1', cwd));

package/dist/core/__tests__/self-update.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/core/__tests__/self-update.test.js

@@ -0,0 +1,46 @@
+import assert from 'node:assert/strict';
+import { execFileSync, spawnSync } from 'node:child_process';
+import { chmodSync, mkdirSync, mkdtempSync, readFileSync, readlinkSync, rmSync, symlinkSync, writeFileSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import test from 'node:test';
+const ROOT = join(dirname(fileURLToPath(import.meta.url)), '..', '..', '..');
+const SELF_UPDATE = join(ROOT, 'scripts', 'self-update.mjs');
+test('self-update installs the downloaded release without replacing the stable global shim', () => {
+    const directory = mkdtempSync(join(tmpdir(), 'crtr-self-update-test-'));
+    try {
+        const release = join(directory, 'release');
+        const archive = join(directory, 'release.tgz');
+        const bin = join(directory, 'bin');
+        const stableShim = join(bin, 'crtr');
+        const marker = join(directory, 'selected');
+        const calls = join(directory, 'npm-calls');
+        const fakeBin = join(directory, 'fake-bin');
+        mkdirSync(join(release, 'scripts'), { recursive: true });
+        mkdirSync(bin);
+        mkdirSync(fakeBin);
+        writeFileSync(join(release, 'scripts', 'install-runtime.mjs'), `import { writeFileSync } from 'node:fs';\nwriteFileSync(process.env.CRTR_SELF_UPDATE_MARKER, 'selected');\n`);
+        execFileSync('tar', ['-czf', archive, '-C', release, '.']);
+        symlinkSync('../stable-crtr', stableShim);
+        writeFileSync(join(fakeBin, 'npm'), `#!/usr/bin/env node\nimport { copyFileSync, appendFileSync } from 'node:fs';\nimport { basename, join } from 'node:path';\nconst args = process.argv.slice(2);\nappendFileSync(process.env.CRTR_SELF_UPDATE_CALLS, args.join(' ') + '\\n');\nif (args[0] !== 'pack') process.exit(1);\nconst destination = args[args.indexOf('--pack-destination') + 1];\ncopyFileSync(process.env.CRTR_SELF_UPDATE_ARCHIVE, join(destination, basename(process.env.CRTR_SELF_UPDATE_ARCHIVE)));\nprocess.stdout.write(JSON.stringify([{ filename: basename(process.env.CRTR_SELF_UPDATE_ARCHIVE) }]));\n`);
+        chmodSync(join(fakeBin, 'npm'), 0o755);
+        const result = spawnSync(process.execPath, [SELF_UPDATE], {
+            encoding: 'utf8',
+            env: {
+                ...process.env,
+                PATH: `${fakeBin}:${process.env.PATH}`,
+                CRTR_SELF_UPDATE_ARCHIVE: archive,
+                CRTR_SELF_UPDATE_CALLS: calls,
+                CRTR_SELF_UPDATE_MARKER: marker,
+            },
+        });
+        assert.equal(result.status, 0, result.stderr);
+        assert.equal(readFileSync(marker, 'utf8'), 'selected');
+        assert.equal(readlinkSync(stableShim), '../stable-crtr');
+        assert.match(readFileSync(calls, 'utf8'), /^pack @north-light\/crouter@latest --json --pack-destination .+\/crtr-self-update-\S+\n$/);
+    }
+    finally {
+        rmSync(directory, { recursive: true, force: true });
+    }
+});

package/dist/core/auto-update.js

@@ -2,7 +2,7 @@ import { spawn } from 'node:child_process';
 import { readConfig, readState, updateState } from './config.js';
 import { nowIso } from './fs-utils.js';
 import { info } from './output.js';
-import { selfCheck, contentCheck } from './self-update.js';
+import { SELF_UPDATE_SCRIPT, selfCheck, contentCheck } from './self-update.js';
 const HOUR_MS = 60 * 60 * 1000;
 const SKIP_SUBCOMMANDS = new Set([
     'update',
@@ -27,7 +27,7 @@ function withinInterval(lastIso, intervalHours) {
     return Date.now() - last < intervalHours * HOUR_MS;
 }
 function spawnDetachedSelfUpdate() {
-    const child = spawn('npm', ['i', '-g', '@north-light/crouter@latest'], {
+    const child = spawn(process.execPath, [SELF_UPDATE_SCRIPT], {
         detached: true,
         stdio: 'ignore',
     });

package/dist/core/canvas/history.js

@@ -15,6 +15,7 @@ import { getNode, listNodes, view } from './canvas.js';
 import { contextDir, reportsDir, nodeDir, inboxPath } from './paths.js';
 import { readTextIfExists, walkFiles, pathExists } from '../fs-utils.js';
 import { parseFrontmatterGeneric } from '../frontmatter.js';
+import { warn } from '../output.js';
 /** The cwd the caller means by default: the calling node's cwd (resolved from
  *  CRTR_NODE_ID via the canvas data layer), falling back to the process cwd
  *  when not run as a node. */
@@ -242,6 +243,11 @@ function listDocFiles(nodeId) {
         return [];
     }
 }
+/** Keep one malformed artifact from breaking a cross-node corpus scan. */
+function warnInvalidFrontmatter(path, error) {
+    const message = (error instanceof Error ? error.message : String(error)).split('\n')[0];
+    warn(`invalid frontmatter in ${path}: ${message}`);
+}
 /** The node desc lives in meta.json, not the row — read it once per survivor. */
 function nodeDescOf(nodeId) {
     const n = getNode(nodeId);
@@ -281,7 +287,15 @@ export function buildCorpus(scope, corpus) {
             out.push(metaArtifact(row, desc));
         if (want.has('report')) {
             for (const f of listReportFiles(row.node_id)) {
-                const a = reportArtifact(row, desc, f);
+                const abs = join(reportsDir(row.node_id), f);
+                let a;
+                try {
+                    a = reportArtifact(row, desc, f);
+                }
+                catch (e) {
+                    warnInvalidFrontmatter(abs, e);
+                    continue;
+                }
                 if (a === null)
                     continue;
                 if (wantReportKind !== undefined && a.reportKind !== wantReportKind)
@@ -291,7 +305,14 @@ export function buildCorpus(scope, corpus) {
         }
         if (want.has('doc') || want.has('roadmap')) {
             for (const abs of listDocFiles(row.node_id)) {
-                const a = docArtifact(row, desc, abs);
+                let a;
+                try {
+                    a = docArtifact(row, desc, abs);
+                }
+                catch (e) {
+                    warnInvalidFrontmatter(abs, e);
+                    continue;
+                }
                 if (a.source === 'roadmap' && !want.has('roadmap'))
                     continue;
                 if (a.source === 'doc' && !want.has('doc'))

package/dist/core/self-update.d.ts

@@ -1,3 +1,4 @@
+export declare const SELF_UPDATE_SCRIPT: string;
 export declare function currentVersion(): string;
 export declare function selfUpdate(): void;
 /** Check whether a newer crtr version is available on npm.

package/dist/core/self-update.js

@@ -15,16 +15,18 @@ const __dirname = dirname(__filename);
 // src/core/self-update.ts → up to src/ → up to pkg root
 const PKG_ROOT = join(__dirname, '..', '..');
 const PACKAGE_JSON_PATH = join(PKG_ROOT, 'package.json');
+export const SELF_UPDATE_SCRIPT = join(PKG_ROOT, 'scripts', 'self-update.mjs');
 export function currentVersion() {
     const raw = readFileSync(PACKAGE_JSON_PATH, 'utf8');
     const parsed = JSON.parse(raw);
     return parsed.version;
 }
 export function selfUpdate() {
-    const res = spawnSync('npm', ['i', '-g', '@north-light/crouter@latest'], { stdio: 'inherit' });
-    if (res.status !== 0) {
-        throw general('npm install failed');
-    }
+    const res = spawnSync(process.execPath, [SELF_UPDATE_SCRIPT], { stdio: 'inherit' });
+    if (res.error)
+        throw general(`self-update failed: ${res.error.message}`);
+    if (res.status !== 0)
+        throw general(`self-update failed (${res.status ?? res.signal ?? 'unknown'})`);
 }
 /** Check whether a newer crtr version is available on npm.
  *  Warns to stderr if network unavailable; returns {current, latest} or null if unreachable. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.166",
+  "version": "0.3.167",
   "description": "crtr — agent runtime with memory, plugins, and marketplaces",
   "type": "module",
   "main": "dist/index.js",
@@ -70,6 +70,7 @@
     "scripts/patch-pi-tree-root-ordering.mjs",
     "scripts/install-runtime.mjs",
     "scripts/runtime-lock.mjs",
+    "scripts/self-update.mjs",
     "runtime.lock.json"
   ],
   "scripts": {
@@ -107,7 +108,7 @@
   },
   "license": "MIT",
   "dependencies": {
-    "@crouton-kit/humanloop": "0.4.21",
+    "@crouton-kit/humanloop": "0.4.22",
     "@earendil-works/pi-agent-core": "0.83.0",
     "@earendil-works/pi-ai": "0.83.0",
     "@earendil-works/pi-coding-agent": "0.83.0",

package/runtime.lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.166",
+  "version": "0.3.167",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@north-light/crouter",
-      "version": "0.3.166",
+      "version": "0.3.167",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {

package/scripts/self-update.mjs

@@ -0,0 +1,32 @@
+#!/usr/bin/env node
+import { spawnSync } from 'node:child_process';
+import { mkdirSync, mkdtempSync, rmSync } from 'node:fs';
+import { join } from 'node:path';
+import { tmpdir } from 'node:os';
+
+const PACKAGE_NAME = '@north-light/crouter';
+
+function run(command, args, options) {
+  const result = spawnSync(command, args, options);
+  if (result.error) throw new Error(`${command} ${args.join(' ')}: ${result.error.message}`);
+  if (result.status !== 0) throw new Error(`${command} ${args.join(' ')} failed (${result.status ?? result.signal ?? 'unknown'})`);
+  return result;
+}
+
+function latestArchive(directory) {
+  const result = run('npm', ['pack', `${PACKAGE_NAME}@latest`, '--json', '--pack-destination', directory], { encoding: 'utf8', stdio: ['ignore', 'pipe', 'inherit'] });
+  const packed = JSON.parse(result.stdout);
+  if (!Array.isArray(packed) || typeof packed[0]?.filename !== 'string') throw new Error('npm pack did not report an archive');
+  return join(directory, packed[0].filename);
+}
+
+const temporary = mkdtempSync(join(tmpdir(), 'crtr-self-update-'));
+try {
+  const archive = latestArchive(temporary);
+  const release = join(temporary, 'package');
+  mkdirSync(release);
+  run('tar', ['-xzf', archive, '--strip-components=1', '-C', release]);
+  run(process.execPath, [join(release, 'scripts', 'install-runtime.mjs'), release], { stdio: 'inherit' });
+} finally {
+  rmSync(temporary, { recursive: true, force: true });
+}