@north-light/crouter 0.3.271 → 0.3.272

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/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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@north-light/crouter",
-  "version": "0.3.271",
+  "version": "0.3.272",
   "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.272",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@north-light/crouter",
-      "version": "0.3.271",
+      "version": "0.3.272",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {