@north-light/crouter 0.3.208 → 0.3.209

package/dist/commands/memory/delete.js

@@ -1,26 +1,11 @@
-import { existsSync, readdirSync, rmSync, rmdirSync } from 'node:fs';
-import { dirname, resolve as resolvePath } from 'node:path';
+import { rmSync } from 'node:fs';
+import { resolve as resolvePath } from 'node:path';
 import { defineLeaf } from '../../core/command.js';
 import { CrtrError, notFound, usage } from '../../core/errors.js';
 import { readText } from '../../core/fs-utils.js';
 import { resolveMemoryDoc } from '../../core/memory-resolver.js';
 import { appendHistoryRecord, buildHistoryRecord, historyLogPathFor } from '../../core/memory/history.js';
-import { MEMORY_SCOPES } from './shared.js';
-/** Remove the file's now-empty parent directories up to (never including) the
- *  scope's `memory/` root, so deleting the last doc under an `area/` prefix
- *  does not orphan an empty directory. Stops at the first non-empty ancestor.
- *  `memoryRoot` is derived by walking up one dirname per name segment: a doc
- *  named `a/b` sits at `<root>/a/b.md`, so its root is two dirnames above. */
-function pruneEmptyParents(filePath, nameSegments) {
-    let memoryRoot = filePath;
-    for (let i = 0; i < nameSegments; i += 1)
-        memoryRoot = dirname(memoryRoot);
-    let dir = dirname(filePath);
-    while (dir !== memoryRoot && dir.startsWith(memoryRoot) && existsSync(dir) && readdirSync(dir).length === 0) {
-        rmdirSync(dir);
-        dir = dirname(dir);
-    }
-}
+import { MEMORY_SCOPES, pruneEmptyParents } from './shared.js';
 export const deleteLeaf = defineLeaf({
     name: 'delete',
     description: 'delete a memory document by name',

package/dist/commands/memory/edit.js

@@ -4,12 +4,11 @@ import { parseFrontmatterGeneric } from '../../core/frontmatter.js';
 import { readText, writeText } from '../../core/fs-utils.js';
 import { resolveMemoryDoc } from '../../core/memory-resolver.js';
 import { appendHistoryRecord, buildHistoryRecord, historyLogPathFor, readHistoryRecords, } from '../../core/memory/history.js';
-import { DOC_RATIONALE_CONSTRAINT, GUIDE_DOC_LINKS, GUIDE_PREDICATE_VOCABULARY, GUIDE_ROUTING_LINE, GUIDE_VISIBILITY_RUNGS, MEMORY_SCOPES, coerceAppliesTo, coerceGate, coerceReadWhen, overlayParam, literalBodySegment, serializeMemoryDocLiteral, } from './shared.js';
-/** Frontmatter fields an overlay flag cannot express removing. `kind`, the
- *  routing line, and the two visibility rungs are the document's contract (a
- *  doc without them is malformed); `origin` and `last-updated` are runtime
- *  provenance. */
-const CLEARABLE_FIELDS = ['short-form', 'gate', 'applies-to', 'read-when', 'slash', 'rationale'];
+import { DOC_RATIONALE_CONSTRAINT, GUIDE_DOC_LINKS, GUIDE_PREDICATE_VOCABULARY, GUIDE_ROUTING_LINE, GUIDE_SURFACES, MEMORY_SCOPES, coerceGate, coerceSurface, overlayParam, literalBodySegment, serializeMemoryDocLiteral, } from './shared.js';
+/** Frontmatter fields an overlay flag cannot express removing. `kind` and the
+ *  routing line are the document's contract (a doc without them is malformed);
+ *  `origin` and `last-updated` are runtime provenance. */
+const CLEARABLE_FIELDS = ['short-form', 'gate', 'surfaces', 'unlisted', 'slash', 'rationale'];
 export const editLeaf = defineLeaf({
     name: 'edit',
     description: 'revise an existing memory document, recording why',
@@ -20,11 +19,11 @@ export const editLeaf = defineLeaf({
         guide: 'The body you pipe is saved byte for byte. Nothing between stdin and disk parses, normalizes, summarizes, or rewrites it \u2014 the frontmatter fence is prepended and the text is otherwise untouched. Relaying a person\u2019s own words is therefore safe: pass them through unchanged and mark them with --verbatim.\n\n' +
             'Two different rationales exist and they never share a flag. --rationale is why THIS REVISION is happening; it is required, it lands in the revision history, and it never touches the document text. --doc-rationale replaces the document\u2019s standing `rationale` frontmatter field \u2014 the observed gap the doc exists to close.\n\n' +
             'Omitting stdin leaves the body exactly as it stands, which is how a frontmatter-only revision is made. Piping a body REPLACES the whole body; there is no partial or append mode. Emptying a document is not an edit \u2014 that intent is `crtr memory delete`.\n\n' +
-            'Omitted frontmatter flags preserve their fields. --unset is the only way to REMOVE one, and it is limited to fields a doc can legitimately live without (short-form, gate, applies-to, read-when, slash, rationale). --applies-to replaces the whole route set rather than adding to it.\n\n' +
+            'Omitted frontmatter flags preserve their fields. --unset is the only way to REMOVE one, and it is limited to fields a doc can legitimately live without (short-form, gate, surfaces, unlisted, slash, rationale). --surface replaces the whole surfaces list rather than adding to it.\n\n' +
             'An edit that changes nothing fails. A rationale attached to zero change is noise in the trail the history exists to build \u2014 if the doc already says what you meant, there is nothing to record.\n\n' +
             '`last-updated` is stamped on every successful edit; `origin` is preserved, so it keeps pointing at the conversation that created the doc rather than the last one to touch it.\n\n' +
             GUIDE_ROUTING_LINE + '\n\n' +
-            GUIDE_VISIBILITY_RUNGS + '\n\n' +
+            GUIDE_SURFACES + '\n\n' +
             GUIDE_PREDICATE_VOCABULARY + '\n\n' +
             GUIDE_DOC_LINKS + '\n\n' +
             'Create a document that does not exist yet with `crtr memory write`. Read the revisions back with `crtr memory history`.',
@@ -35,14 +34,12 @@ export const editLeaf = defineLeaf({
             overlayParam('kind'),
             overlayParam('when-and-why-to-read'),
             overlayParam('short-form'),
-            overlayParam('system-prompt-visibility'),
-            overlayParam('file-read-visibility'),
+            overlayParam('unlisted', {}, 'Presence SETS the field; remove it with `--unset unlisted`.'),
+            overlayParam('surface'),
             overlayParam('gate'),
-            overlayParam('applies-to'),
-            overlayParam('read-when'),
             overlayParam('slash', {}, 'Presence SETS the field; remove it with `--unset slash`.'),
             { kind: 'flag', name: 'doc-rationale', type: 'string', required: false, constraint: DOC_RATIONALE_CONSTRAINT },
-            { kind: 'flag', name: 'unset', type: 'enum', choices: [...CLEARABLE_FIELDS], required: false, repeatable: true, constraint: 'Remove a frontmatter field an overlay flag cannot express removing. One field per occurrence. `kind`, `when-and-why-to-read`, the two visibility rungs, `origin`, and `last-updated` are never clearable. Unsetting `applies-to` while file-read-visibility is not none fails on the same invariant `write` raises.' },
+            { kind: 'flag', name: 'unset', type: 'enum', choices: [...CLEARABLE_FIELDS], required: false, repeatable: true, constraint: 'Remove a frontmatter field an overlay flag cannot express removing. One field per occurrence. `kind`, `when-and-why-to-read`, `origin`, and `last-updated` are never clearable.' },
             { kind: 'flag', name: 'scope', type: 'enum', choices: [...MEMORY_SCOPES], required: false, constraint: 'Restrict resolution to this scope before editing. A disambiguation filter only \u2014 never a creation target, and never a way to move a doc between stores.' },
             { kind: 'stdin', name: 'body', required: false, constraint: 'Full replacement body (markdown, no frontmatter), saved byte for byte. Piped on stdin only \u2014 this leaf already claims the one positional for NAME. Absent or empty stdin leaves the existing body untouched, which is how a frontmatter-only revision is made.' },
         ],
@@ -100,31 +97,27 @@ export const editLeaf = defineLeaf({
         setIf('kind', input['kind']);
         setIf('when-and-why-to-read', input['whenAndWhyToRead']);
         setIf('short-form', input['shortForm']);
-        setIf('system-prompt-visibility', input['systemPromptVisibility']);
-        setIf('file-read-visibility', input['fileReadVisibility']);
+        if (input['unlisted'] === true)
+            frontmatter['unlisted'] = true;
+        if (input['surface'] !== undefined) {
+            frontmatter['surfaces'] = input['surface'].map(coerceSurface);
+        }
         if (input['gate'] !== undefined)
             frontmatter['gate'] = coerceGate(input['gate']);
-        if (input['appliesTo'] !== undefined)
-            frontmatter['applies-to'] = coerceAppliesTo(input['appliesTo']);
-        if (input['readWhen'] !== undefined)
-            frontmatter['read-when'] = coerceReadWhen(input['readWhen']);
         if (input['slash'] === true)
             frontmatter['slash'] = true;
         setIf('rationale', input['docRationale']);
         for (const field of input['unset'] ?? []) {
             delete frontmatter[field];
         }
-        const applies = frontmatter['applies-to'];
-        const routes = typeof applies === 'string'
-            ? [applies]
-            : Array.isArray(applies) && applies.every((value) => typeof value === 'string')
-                ? applies
-                : [];
-        if (frontmatter['file-read-visibility'] !== 'none' && routes.length === 0) {
-            throw usage(`file-read-visibility ${String(frontmatter['file-read-visibility'])} requires at least one applies-to route: pass --applies-to (use "." for project workspace/profile-open context, a real file glob for matching reads), or set --file-read-visibility none`);
-        }
-        if (doc.scope !== 'project' && routes.some((route) => route.trim() === '.')) {
-            throw usage('applies-to "." is a project-store workspace mount route; use a real file glob or choose file-read-visibility none for user, profile, and node memory');
+        // Same invariant `write` raises: a workspace-open entry fires only when
+        // cwd/profile mounts the doc's project store, so outside project scope it
+        // is a dead route.
+        const surfacesVal = frontmatter['surfaces'];
+        if (doc.scope !== 'project' &&
+            Array.isArray(surfacesVal) &&
+            surfacesVal.some((e) => e !== null && typeof e === 'object' && e['on'] === 'workspace-open')) {
+            throw usage('a workspace-open surfaces entry is a project-store mount route; user, profile, and node memory cannot carry one — drop the entry');
         }
         // LITERAL SAVE: the piped bytes reach disk untransformed, one structural
         // blank line after the fence. Omitting stdin re-emits the existing body

package/dist/commands/memory/history.js

@@ -60,7 +60,7 @@ export const historyLeaf = defineLeaf({
     help: {
         name: 'memory history',
         summary: 'list a document\u2019s revisions newest-first, with diffs or one full record on demand',
-        guide: 'Every mutation through `crtr memory write`, `edit`, and `delete` appends one self-contained record \u2014 complete document text before and after, plus who, when, and (on an edit) why. Diffs and revision numbers are derived here at read time, never stored.\n\n' +
+        guide: 'Every mutation through `crtr memory write`, `edit`, `move`, and `delete` appends one self-contained record \u2014 complete document text before and after, plus who, when, and (on an edit) why. Diffs and revision numbers are derived here at read time, never stored.\n\n' +
             'A record whose `before` differs from the previous record\u2019s `after` is the evidence of an out-of-band direct file edit; nothing prevents those, and nothing detects them beyond this.\n\n' +
             'The log is a plain JSONL file at `log_path`, one JSON object per line \u2014 an external consumer reads it directly rather than through this command.',
         params: [

package/dist/commands/memory/lint.d.ts

@@ -1,16 +1,15 @@
-/** The length rule. Caps are computed from the RAW rung strings — an invalid
- *  rung already fails the schema check, so it simply matches no cap here.
+/** Warn when a body short enough to inline still pays a routing line. */
+export declare function lintShortPreviewBody(fm: Record<string, unknown>, body: string): string | null;
+/** The length rule. Caps are computed from the parsed entries — an invalid
+ *  entry already fails the schema check, so it simply matches no cap here.
  *  `docName` is the doc's canonical resolver identity (explicit frontmatter
  *  `name`, else the normalized path-derived name) — persona layers under
  *  `kinds/` are recognized by it and never capped. */
-export declare function lintShortPreviewBody(fm: Record<string, unknown>, body: string): string | null;
 export declare function lintBodyLength(fm: Record<string, unknown>, body: string, docName: string): string | null;
 /** Schema checks for a doc living in a substrate memory dir: a memory store
  *  holds ONLY substrate docs, so a missing/invalid `kind` is an authoring
- *  error here (elsewhere it just means "not a substrate doc"). Both rungs are
- *  REQUIRED — there is no kind default to lean on, so a missing rung is an
- *  authoring error. Rung and gate values are checked RAW — the runtime parser
- *  silently falls back to the neutral floor / inert gates, which is exactly the
- *  silent tolerance this lint exists to catch at authoring time. */
+ *  error here (elsewhere it just means "not a substrate doc"). Fields are
+ *  checked RAW — the runtime parser silently drops invalid entries, which is
+ *  exactly the silent tolerance this lint exists to catch at authoring time. */
 export declare function lintSubstrateSchema(fm: Record<string, unknown> | null): string | null;
 export declare const lintLeaf: import("../../core/command.js").LeafDef;