memoir-cli 3.16.2 → 3.16.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "memoir-cli",
-  "version": "3.16.2",
+  "version": "3.16.3",
   "mcpName": "io.github.camgitt/memoir",
   "description": "Portable, project-scoped memory and session handoffs for coding agents. Readable files, MCP recall, and optional user-passphrase encrypted backups.",
   "main": "src/index.js",

package/src/mcp.js

@@ -483,15 +483,28 @@ server.tool(
 
 server.tool(
   'memoir_session',
-  'Show the current session state — goals, next actions, open questions, recent decisions, recent sessions across machines. Use this to catch up at the start of a session, or when you need to orient yourself on what was decided.',
-  {},
-  async () => {
-    const state = sessionView(await readSession());
+  'Show the current session state — goals, next actions, open questions, recent decisions, recent sessions across machines. Use this to catch up at the start of a session, or when you need to orient yourself on what was decided. Pass project to narrow it to one project, or tag to narrow it to items whose text carries a [tag] prefix.',
+  {
+    project: z.string().optional().describe('Project directory or identity. Defaults to the current working project.'),
+    tag: z.string().optional().describe('Show only items whose text starts with a bracketed tag, e.g. "gathered" matches "[gathered — CAM] …". Case-insensitive.'),
+  },
+  async ({ project, tag }) => {
+    const state = sessionView(await readSession(), project ? { project } : {});
     const machine = await getMachineId();
-    const goals = state.current.goals.map(g => `- ${g.text}`).join('\n') || '(none)';
-    const nexts = state.current.next_actions.map(n => `- [ ] ${n.text}`).join('\n') || '(none)';
-    const questions = state.current.open_questions.map(q => `- ${q.text}`).join('\n') || '(none)';
-    const decisions = state.current.decisions.filter(d => visibleMemory(d)).slice(0, 5).map(d => {
+    // Untagged items are visible in every project by design; a [tag] prefix in
+    // the text is the only project signal many sessions actually carry.
+    const tagged = items => {
+      if (!tag) return items;
+      const want = tag.toLowerCase().replace(/^\[|\]$/g, '');
+      return items.filter(i => {
+        const m = String(i?.text || '').match(/^\s*\[([^\]]+)\]/);
+        return m && m[1].toLowerCase().includes(want);
+      });
+    };
+    const goals = tagged(state.current.goals).map(g => `- ${g.text}`).join('\n') || '(none)';
+    const nexts = tagged(state.current.next_actions).map(n => `- [ ] ${n.text}`).join('\n') || '(none)';
+    const questions = tagged(state.current.open_questions).map(q => `- ${q.text}`).join('\n') || '(none)';
+    const decisions = tagged(state.current.decisions.filter(d => visibleMemory(d, project ? { project } : {}))).slice(0, 5).map(d => {
       let line = `- ${d.text}`;
       if (d.why) line += ` — *${d.why}*`;
       return line;

package/src/memory/scope.js

@@ -5,6 +5,14 @@ import crypto from 'crypto';
 import { execFileSync } from 'child_process';
 
 const cache = new Map();
+
+// Records written before the home directory got an explicit key carry the hash
+// of the empty string. Treat that as an alias for home so they stay visible.
+const LEGACY_HOME_ID = 'local:' + crypto.createHash('sha256').update('').digest('hex').slice(0, 32);
+export function canonicalIdentity(id) {
+  return id === LEGACY_HOME_ID ? projectIdentity(os.homedir()) : id;
+}
+
 export function projectIdentity(project = process.env.MEMOIR_PROJECT_ROOT || process.cwd()) {
   if (project === 'shared') return 'shared';
   if (/^(git|local):[a-f0-9]{32}$/.test(project)) return project;
@@ -14,7 +22,11 @@ export function projectIdentity(project = process.env.MEMOIR_PROJECT_ROOT || pro
   if (old && Date.now() - old.at < 60_000) return old.id;
   let home = os.homedir();
   try { home = fs.realpathSync(home); } catch {}
-  let key = path.relative(home, absolute).replace(/\\/g, '/');
+  // The home directory itself is a legitimate working root for people who run
+  // their agent from ~. path.relative(home, home) is '', and hashing '' gives
+  // every such user the same degenerate id (sha256 of the empty string), so
+  // name it explicitly instead.
+  let key = path.relative(home, absolute).replace(/\\/g, '/') || '~';
   let kind = 'local';
   try {
     const remote = execFileSync('git', ['config', '--get', 'remote.origin.url'], { cwd: absolute, encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'], timeout: 2000 }).trim();
@@ -44,9 +56,9 @@ export function memoryVisibility({ project = process.env.MEMOIR_PROJECT_ROOT ||
       if (item.claudeProjectKey !== currentKey && item.claudeProjectKey !== sharedKey) return false;
     }
     if (allProjects || !item.project || item.project === 'shared') return true;
-    activeId ??= projectIdentity(project);
+    activeId ??= canonicalIdentity(projectIdentity(project));
     const key = String(item.project);
-    if (!identities.has(key)) identities.set(key, projectIdentity(key));
+    if (!identities.has(key)) identities.set(key, canonicalIdentity(projectIdentity(key)));
     return identities.get(key) === activeId;
   };
 }