lucid-mem 0.5.0 → 0.5.1

package/README.md

@@ -34,7 +34,7 @@ Memory lives in `~/agent-memory` — a real git repo you can read, diff, and poi
 
 ## Multi-repo memory
 
-One memory repo serves every project. `projects/<name>-<hash>.md` files key off the repo's origin remote, so clones share memory and same-named repos don't collide. Inside a known repo, `amr context` is project-scoped — the project's latest 25 plus linked siblings (up to 8 projects). Outside one, it shows global bullets plus the latest 25 pooled across all projects. `private` policy keeps a project out of transitive auto-attach and the pooled digest; `noglobal` keeps it out of the pooled digest only.
+One memory repo serves every project. `projects/<name>-<hash>.md` files key off the repo's origin remote, so clones share memory and same-named repos don't collide. Inside a known repo, `amr context` is project-scoped — the project's latest 25 plus linked siblings (up to 8 projects). Outside one, it shows global bullets plus the latest 25 pooled across all projects. `private` and `noglobal` policies keep a project out of the pooled digest (links still reach private projects — your link graph is your own curation).
 
 ## MCP
 

package/lib/commands.mjs

@@ -91,9 +91,9 @@ export function cmdContext(flags) {
 
   // Attach list: the current project, then its config links
   // (transitive, deduped, max 8 total) and any --also names.
-  // private blocks reachability from repos that never linked it:
-  // direct links (siblings) and explicit --also attach fine; it only
-  // refuses when reached transitively through another project's links.
+  // Links are the owner's curation — private projects attach fine
+  // through them. `private` only keeps a project out of the pooled
+  // outside digest, the one unlinked exposure path.
   const bySlug = new Map(listProjects(home).map((p) => [p.slug, p]));
   const attach = [];
   const seen = new Set();
@@ -103,17 +103,16 @@ export function cmdContext(flags) {
     attach.push({ slug, name: bySlug.get(slug)?.name || slug, linked });
   };
   if (proj) push(proj.slug, false);
-  const queue = (entry?.links || []).map((k) => ({ k, via: 'entry' }));
+  const queue = (entry?.links || []).map((k) => ({ k }));
   if (flags.also)
     queue.push(
       ...String(flags.also)
         .split(',')
-        .map((k) => ({ k, via: 'also' }))
+        .map((k) => ({ k }))
     );
-  const refused = [];
   const unknown = [];
   while (queue.length && attach.length < 8) {
-    const { k: raw, via } = queue.shift();
+    const { k: raw } = queue.shift();
     const key = String(raw).trim().toLowerCase();
     if (!key) continue;
     const slug = resolveProjectSlug(home, key);
@@ -123,13 +122,8 @@ export function cmdContext(flags) {
     }
     if (seen.has(slug)) continue;
     const e = projectEntry(cfg, { slug, name: key });
-    if (e?.private && via === 'transitive') {
-      refused.push(key);
-      continue;
-    }
     push(slug, true);
-    if (e?.links)
-      queue.push(...e.links.map((l) => ({ k: l, via: 'transitive' })));
+    if (e?.links) queue.push(...e.links.map((l) => ({ k: l })));
   }
 
   const lines = [`<agent-memory repo="${home}">`];
@@ -165,11 +159,6 @@ export function cmdContext(flags) {
       lines.push('', '## Recent project memory', recent.join('\n'));
   }
   if (indexLinks.length) lines.push('', '## Index', indexLinks.join('\n'));
-  if (refused.length)
-    lines.push(
-      '',
-      `<!-- ${refused.length} linked project(s) omitted by private policy: ${refused.join(', ')} -->`
-    );
   if (unknown.length)
     lines.push(
       '',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lucid-mem",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "lucid-mem (amr) - autonomous git-backed agent memory (Agent Memory Repo spec)",
   "type": "module",
   "bin": {

package/test/context.test.mjs

@@ -28,7 +28,7 @@ test('context in a repo shows its project memory', () => {
   assert.match(r.stdout, /app uses redis cache/);
 });
 
-test('link attaches a sibling project transitively; private blocks transitive only', () => {
+test('link attaches siblings and private transitively (links are owner curation)', () => {
   const home = tmpdir();
   const base = tmpdir();
   const repoA = initRepo(path.join(base, 'a'), 'https://github.com/acme/a.git');
@@ -45,13 +45,13 @@ test('link attaches a sibling project transitively; private blocks transitive on
   const inB = run(['context'], { amrHome: home, cwd: repoB });
   assert.match(inB.stdout, /a uses sqlite/);
 
-  // c links to b; a is private -> transitive refusal in c
+  // c links to b; a is private -> still attaches transitively, since
+  // the link chain is the owner's own curation
   run(['link', 'b'], { amrHome: home, cwd: repoC });
   run(['policy', 'a', 'private', 'true'], { amrHome: home, cwd: repoA });
   const inC = run(['context'], { amrHome: home, cwd: repoC });
   assert.match(inC.stdout, /b uses kafka/);
-  assert.match(inC.stdout, /omitted by private policy/);
-  assert.ok(!inC.stdout.includes('a uses sqlite'));
+  assert.match(inC.stdout, /a uses sqlite/);
 
   // explicit --also still attaches the private project
   const explicit = run(['context', '--also', 'a'], { amrHome: home, cwd: repoC });