@north-light/crouter 0.3.271 → 0.3.273

package/dist/commands/canvas-browse.js

@@ -64,7 +64,7 @@ export const browseLeaf = defineLeaf({
             profileId,
             source,
             viewedNodeIds: async () => new Set((await cliClient().listFocuses()).map((focus) => focus.node_id)),
-            closeNode: (id) => cliClient().closeNode(id, { finish: true, caller_pid: process.ppid }).then((r) => ({ closed: r.closed })),
+            closeNode: (id) => cliClient().closeNode(id, process.ppid > 0 ? { finish: true, caller_pid: process.ppid } : { finish: true }).then((r) => ({ closed: r.closed })),
         });
     },
 });

package/dist/commands/cron.js

@@ -63,12 +63,11 @@ function parseRunTimeout(raw) {
     }
     return seconds;
 }
-/** The calling agent's profile, which is the scope every cron verb speaks in:
- *  a profile-scoped cron lists and cancels within its profile, a global one is
- *  canvas-home-wide. Outside a profile there is no scope to narrow to, so the
- *  caller sees the whole canvas home. */
-function scopeQuery() {
-    const p = envProfileId();
+/** The profile scope for a cron route: an explicit selector overrides the
+ *  calling agent's profile. Outside a profile there is no scope to narrow to,
+ *  so the caller sees the whole canvas home. */
+function scopeQuery(profile) {
+    const p = typeof profile === 'string' && profile.trim() !== '' ? canonicalProfileId(profile) : envProfileId();
     return p !== undefined && p.trim() !== '' ? { profile: p } : undefined;
 }
 /** Resolve the chosen-else-inherited cwd, and refuse one that is not a
@@ -270,7 +269,7 @@ const addLeaf = defineLeaf({
     render: (r) => {
         const recur = String(r['recur']);
         const cadence = recur === 'none' ? 'one-shot (row deleted after the run)' : recur;
-        return `Cron armed — ${String(r['cron_id'])}.\n\n- next fire: ${String(r['fire_at'])}\n- cadence: ${cadence}\n\nIt runs daemon-side; nothing more to do here. Inspect it with \`crtr cron show ${String(r['cron_id'])}\`; cancel any time with \`crtr cron cancel ${String(r['cron_id'])}\`.`;
+        return `Cron armed — ${String(r['cron_id'])}.\n\n- next fire: ${String(r['fire_at'])}\n- cadence: ${cadence}\n\nIt runs daemon-side; nothing more to do here. Inspect it with \`crtr cron show ${String(r['cron_id'])}\`; cancel with \`crtr cron cancel ${String(r['cron_id'])}\` from its profile, or pass \`--profile <profile>\` from another profile.`;
     },
 });
 // cron list
@@ -494,9 +493,11 @@ const cancelLeaf = defineLeaf({
         summary: 'remove one cron',
         params: [
             { kind: 'positional', name: 'cron-id', required: true, constraint: 'The cron to remove (from `cron list`, or $CRTR_CRON_ID inside a run).' },
+            { kind: 'flag', name: 'profile', type: 'string', required: false, constraint: 'Profile scope to cancel within. Defaults to the calling node\'s profile; pass the profile id or manifest name for a cron armed under another profile.' },
         ],
         output: [],
         outputKind: 'object',
+        dynamicState: () => profilesStateBlock(),
         effects: ['Deletes the cron row and its run history; kills any in-flight run. A no-op if already gone.'],
     },
     run: async (input) => {
@@ -504,7 +505,7 @@ const cancelLeaf = defineLeaf({
         const runId = process.env['CRTR_RUN_ID'];
         await cliClient()
             .cancelCron(id, {
-            ...scopeQuery(),
+            ...scopeQuery(input['profile']),
             ...(runId === undefined || runId === '' ? {} : { run_id: runId }),
         })
             .catch((err) => rethrowAsCliError(err, 'Check the id against `crtr cron list`.'));

package/dist/commands/memory/__tests__/command-selector-and-mutation-guards.test.js

@@ -283,6 +283,42 @@ test('a failing lint returns its declared rows and fails by exit code', async ()
         process.exitCode = 0;
     }
 });
+test('initializing a repository-root store does not lint its valid namespace as misplaced', async () => {
+    const fresh = join(base, 'fresh-root');
+    mkdirSync(fresh, { recursive: true });
+    const result = (await writeLeaf.run({
+        name: 'fresh-root',
+        kind: 'knowledge',
+        dir: fresh,
+        body: 'FRESH ROOT BODY',
+        whenAndWhyToRead: 'When opening this repository, read this because it explains the workspace.',
+        surface: ['{on: workspace-open, at: content}', '{on: read, match: "./**", at: content}'],
+    }));
+    assert.equal(result.findings, undefined, JSON.stringify(result.findings));
+});
+test('a targeted read suppresses malformed descendant-store warnings', async () => {
+    const noisy = join(base, 'noisy');
+    mkdirSync(storeDir(noisy), { recursive: true });
+    writeDoc(join(home, '.crouter', 'memory', 'quiet-target.md'), 'kind: knowledge', 'QUIET TARGET BODY');
+    writeDoc(join(noisy, 'unrelated', '.crouter', 'memory', 'doc.md'), 'kind: knowledge', 'UNRELATED BODY');
+    process.chdir(noisy);
+    resetScopeCache();
+    const stderr = process.stderr;
+    const originalWrite = stderr.write;
+    const writes = [];
+    stderr.write = ((chunk) => {
+        writes.push(String(chunk));
+        return true;
+    });
+    try {
+        const read = (await readLeaf.run({ name: 'quiet-target', kind: undefined, frontmatter: false, scope: undefined, dir: undefined }));
+        assert.match(read.content, /QUIET TARGET BODY/);
+    }
+    finally {
+        stderr.write = originalWrite;
+    }
+    assert.equal(writes.some((line) => line.includes('memory store not mounted')), false, writes.join(''));
+});
 test('deleted history points at the revision holding the retained text, never a read that cannot answer', async () => {
     await deleteLeaf.run({ name: 'acme/disposable', dir: root });
     const result = (await historyLeaf.run({ name: 'acme/disposable', dir: root }));

package/dist/commands/memory/read.js

@@ -4,7 +4,7 @@ import { defineLeaf } from '../../core/command.js';
 import { CrtrError } from '../../core/errors.js';
 import { memoryExtensionEffectiveCatalog, projectEffectiveMemoryExtensions } from '../../core/memory/extensions.js';
 import { docLinkNames } from '../../core/memory/doc-link-grammar.js';
-import { createMemoryDocSnapshot, listAllMemoryDocs, loadMemoryStoreView, readMemoryDocContent, resolveMemoryCandidates, resolveMemoryDocs, } from '../../core/memory-resolver.js';
+import { ambientMemoryTarget, createMemoryDocSnapshot, listAllMemoryDocs, loadMemoryStoreView, readMemoryDocContent, resolveMemoryCandidates, resolveMemoryDocs, } from '../../core/memory-resolver.js';
 import { expandShellBlocks, hasShellBlocks, makeNodeShellRunner } from '../../core/runtime/shell-expansion.js';
 import { emptyContextExposureState, exposureTarget, loadContextExposureState, registerDocumentExposure, registerExposure, saveContextExposureState, } from '../../core/substrate/injected-store.js';
 import { dirDedupKey, docsByName, isDirName, renderDirListing } from '../../core/substrate/listings.js';
@@ -121,7 +121,10 @@ export const readLeaf = defineLeaf({
                 ...(selector.scope === undefined ? {} : { scope: selector.scope }),
                 includeDescendants: true,
             };
-            listingByName = linkByName = docsByName(listAllMemoryDocs(selector.scope, false, true));
+            // A named read must not turn malformed stores elsewhere in the corpus
+            // into warnings about the requested document. `memory lint` owns that
+            // corpus health report; resolution still includes every candidate.
+            listingByName = linkByName = docsByName(listAllMemoryDocs(selector.scope, true, true));
             try {
                 set = resolveMemoryCandidates(nameRaw, resolveOpts);
             }
@@ -190,6 +193,7 @@ export const readLeaf = defineLeaf({
             subject,
             target,
             byName: listingByName,
+            memoryTarget: ambientMemoryTarget(),
             listings: 'blocks',
             explicit: true,
         });

package/dist/commands/memory/write.js

@@ -1,12 +1,12 @@
 import { envNodeId, envProfileId } from '../../shared/env.js';
-import { join } from 'node:path';
+import { dirname, join } from 'node:path';
 import { defineLeaf } from '../../core/command.js';
 import { usage } from '../../core/errors.js';
 import { docLinkNames } from '../../core/memory/doc-link-grammar.js';
 import { lintMemoryDocument, resolvableCanonicalNames } from '../../core/memory/lint.js';
 import { descendantStoreRoots } from '../../core/nested-stores.js';
 import { projectScopeRoots } from '../../core/scope.js';
-import { loadMemoryTargetView } from '../../core/memory-resolver.js';
+import { loadMemoryTargetView, openProjectMemoryStore } from '../../core/memory-resolver.js';
 import { parsedSubstrateSurfaces } from '../../core/substrate/frontmatter-validation.js';
 import { ensureDir, realpathOrSelf, walkFiles, writeText } from '../../core/fs-utils.js';
 import { associateRepository } from '../../core/memory/repository-association.js';
@@ -187,7 +187,11 @@ export const writeLeaf = defineLeaf({
         const surfaceEntries = parsedSubstrateSurfaces(frontmatter);
         const carriesReference = docLinkNames(body).length > 0 || surfaceEntries.some((entry) => entry.on === 'memory-read');
         const carriesBootEntry = surfaceEntries.some((entry) => entry.on === 'boot');
-        const lint = lintMemoryDocument(store, path, {
+        // Initializing created the root declaration after selector resolution, so
+        // validate against a freshly described store: the stale descriptor has no
+        // repository-root association yet and would reject its valid namespace.
+        const lintStore = initializing ? openProjectMemoryStore(store.ownerDir ?? dirname(dirname(memoryDir))) : store;
+        const lint = lintMemoryDocument(lintStore, path, {
             ...(carriesReference
                 ? {
                     resolvable: resolvableCanonicalNames(loadMemoryTargetView({

package/dist/commands/node/lifecycle.js

@@ -100,7 +100,7 @@ const nodeClose = defineLeaf({
             throw new InputError({ error: 'no_node', message: 'no node found in this pane to close', next: 'Pass --node <id>, or run from inside the agent\'s pane.' });
         }
         const res = await cliClient()
-            .closeNode(id, { caller_pid: process.ppid })
+            .closeNode(id, process.ppid > 0 ? { caller_pid: process.ppid } : {})
             .catch((err) => rethrowAsCliError(err, 'List nodes with `crtr node inspect list`.'));
         return { node_id: res.root, count: res.closed.length, closed_ids: res.closed, spared: res.spared };
     },

package/dist/core/command-plugins/transport/exec-invoke.js

@@ -21,7 +21,7 @@ const NEXT = 'Run `crtr pkg plugin show` or `crtr sys doctor` to inspect the plu
  *  Returns null if valid; otherwise returns a { field, receivedType } error.
  *  Transport-neutral: used by exec and HTTP transport adapters. */
 export function validateDeclaredResult(output, result) {
-    return validateFields(output, result, false);
+    return validateFields(output, result, true);
 }
 /** Core leaf declarations may explicitly require a nullable field. */
 export function validateCoreDeclaredResult(output, result) {

package/dist/core/memory-resolver.d.ts

@@ -195,7 +195,7 @@ export declare function openProjectMemoryStore(ownerDir: string): MemoryStoreDes
 /** Every store a target sees, in precedence order: node, each project store
  *  nearest-first (its enabled plugins after it), then profile, user, builtin.
  *  One real store reached twice contributes once, at its first position. */
-export declare function memoryStoresInPrecedence(target: MemoryTarget, scope?: MemoryScope, includeDescendants?: boolean): MemoryStoreDescriptor[];
+export declare function memoryStoresInPrecedence(target: MemoryTarget, scope?: MemoryScope, includeDescendants?: boolean, quiet?: boolean): MemoryStoreDescriptor[];
 /** The native (non-plugin) MOUNTED store dirs in resolution precedence.
  *  Addresses a store directly when the document itself may be absent, which is
  *  how a DELETED doc's revision log is still found: history outlives the doc,

package/dist/core/memory-resolver.js

@@ -193,7 +193,7 @@ export function openProjectMemoryStore(ownerDir) {
 /** Every store a target sees, in precedence order: node, each project store
  *  nearest-first (its enabled plugins after it), then profile, user, builtin.
  *  One real store reached twice contributes once, at its first position. */
-export function memoryStoresInPrecedence(target, scope, includeDescendants = false) {
+export function memoryStoresInPrecedence(target, scope, includeDescendants = false, quiet = false) {
     const out = [];
     const projectStores = (crtrRoot, projectMemory) => {
         out.push(projectStoreDescriptor(dirname(crtrRoot), join(crtrRoot, 'memory'), projectMemory));
@@ -211,7 +211,7 @@ export function memoryStoresInPrecedence(target, scope, includeDescendants = fal
             // nested one. They carry the neutral ceiling — no profile entry names
             // them, and descendant discovery is off for every automatic delivery path.
             if (includeDescendants) {
-                for (const root of descendantStoreRoots(annotated.map(({ root: r }) => r))) {
+                for (const root of descendantStoreRoots(annotated.map(({ root: r }) => r), quiet)) {
                     projectStores(root, NEUTRAL_PROJECT_MEMORY);
                 }
             }
@@ -438,7 +438,7 @@ function makeView(stores, docs, scope) {
  *  with its migration remedy and contributes nothing. */
 export function loadMemoryTargetView(target, opts = {}) {
     const quiet = opts.quiet ?? false;
-    const stores = memoryStoresInPrecedence(target, opts.scope, opts.includeDescendants ?? false);
+    const stores = memoryStoresInPrecedence(target, opts.scope, opts.includeDescendants ?? false, quiet);
     const docs = [];
     for (const store of stores) {
         if (store.mountStatus === 'absent')

package/dist/core/nested-stores.d.ts

@@ -9,7 +9,7 @@
  * excluding centrally gives the resolver and lint the identical call with no
  * duplicated dedupe logic.
  */
-export declare function descendantStoreRoots(ancestorRoots: string[]): string[];
+export declare function descendantStoreRoots(ancestorRoots: string[], quiet?: boolean): string[];
 /** Complete descendant-store discovery for state migrations. Unlike the
  *  interactive addressing lane, this filesystem walk has no time or depth
  *  budget, crosses embedded repositories, admits plugin-only `.crouter` roots,

package/dist/core/nested-stores.js

@@ -39,7 +39,7 @@ const cache = new Map();
  * excluding centrally gives the resolver and lint the identical call with no
  * duplicated dedupe logic.
  */
-export function descendantStoreRoots(ancestorRoots) {
+export function descendantStoreRoots(ancestorRoots, quiet = false) {
     const key = [...ancestorRoots].sort().join('\n');
     const cached = cache.get(key);
     if (cached !== undefined)
@@ -56,7 +56,9 @@ export function descendantStoreRoots(ancestorRoots) {
         owners.push(owner);
     }
     const tripBudget = (owner) => {
-        warn(`memory: nested-store discovery under ${owner} exceeded its ${TIME_BUDGET_MS}ms budget — nested stores may be missing from this listing; register deep stores explicitly (add their dirs to the selected profile's projects)`);
+        if (!quiet) {
+            warn(`memory: nested-store discovery under ${owner} exceeded its ${TIME_BUDGET_MS}ms budget — nested stores may be missing from this listing; register deep stores explicitly (add their dirs to the selected profile's projects)`);
+        }
     };
     const found = [];
     const foundSeen = new Set();

package/dist/core/runtime/broker-extension-render.js

@@ -88,7 +88,10 @@ export function buildContextBearingsFromState(state, projectContext, situational
 }
 export function buildForkBearingsFromState(state, situational = '') {
     const { node, fork_source: source } = state;
-    const bearings = ['<crtr-bearings>', identity(state), '', 'The source node’s earlier task, graph position, lifecycle, children, schedules, worktree, situational context, and context-directory path do not carry over. Shared conversation, project, and operating context remain reference material.', '', '## Your node now', `Lifecycle: ${node.lifecycle}.`];
+    const sourceMandate = source?.status === 'active' || source?.status === 'idle'
+        ? `Source node ${source.node_id} ("${fullName(source)}") is still live and still holds the mandate from the copied conversation; do not resume that work here.`
+        : 'The source node’s earlier graph position, lifecycle, children, schedules, worktree, situational context, and context-directory path do not carry over. Shared conversation, project, and operating context remain reference material.';
+    const bearings = ['<crtr-bearings>', identity(state), '', sourceMandate, '', '## Your node now', `Lifecycle: ${node.lifecycle}.`];
     const map = graphMap(state);
     bearings.push(...(map === '' ? ['Canvas: independent root with no parent or inherited graph edges.'] : ['', map]));
     bearings.push(`Context directory: \`${contextDir(node.node_id)}\` (\`$CRTR_CONTEXT_DIR\`). Use this path in place of every source-node context path above.`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.271",
+  "version": "0.3.273",
   "description": "crtr — agent runtime with memory, plugins, and marketplaces",
   "type": "module",
   "main": "dist/index.js",

package/runtime.lock.json

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