@empirical-gauzza/cli 1.0.25 → 1.0.27

package/lib/installers/detect.js

@@ -1,47 +1,47 @@
-'use strict';
-
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
-const { commandExists } = require('../fsUtils');
-const { installCodex } = require('./codex');
-const { installCopilot } = require('./copilot');
-const { installClaude } = require('./claude');
-
-function detectInstallTargets() {
-  const home = os.homedir();
-  const targets = [];
-
-  const codexDetected = commandExists('codex') || fs.existsSync(path.join(home, '.codex'));
-  if (codexDetected) {
-    targets.push({
-      id: 'codex',
-      label: 'Codex CLI',
-      install: installCodex
-    });
-  }
-
-  const copilotDetected = commandExists('copilot') || fs.existsSync(path.join(home, '.copilot'));
-  if (copilotDetected) {
-    targets.push({
-      id: 'copilot',
-      label: 'GitHub Copilot CLI',
-      install: installCopilot
-    });
-  }
-
-  const claudeDetected = commandExists('claude') || fs.existsSync(path.join(home, '.claude'));
-  if (claudeDetected) {
-    targets.push({
-      id: 'claude',
-      label: 'Claude CLI',
-      install: installClaude
-    });
-  }
-
-  return targets;
-}
-
-module.exports = {
-  detectInstallTargets
-};
+'use strict';
+
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+const { commandExists } = require('../fsUtils');
+const { installCodex } = require('./codex');
+const { installCopilot } = require('./copilot');
+const { installClaude } = require('./claude');
+
+function detectInstallTargets() {
+  const home = os.homedir();
+  const targets = [];
+
+  const codexDetected = commandExists('codex') || fs.existsSync(path.join(home, '.codex'));
+  if (codexDetected) {
+    targets.push({
+      id: 'codex',
+      label: 'Codex CLI',
+      install: installCodex
+    });
+  }
+
+  const copilotDetected = commandExists('copilot') || fs.existsSync(path.join(home, '.copilot'));
+  if (copilotDetected) {
+    targets.push({
+      id: 'copilot',
+      label: 'GitHub Copilot CLI',
+      install: installCopilot
+    });
+  }
+
+  const claudeDetected = commandExists('claude') || fs.existsSync(path.join(home, '.claude'));
+  if (claudeDetected) {
+    targets.push({
+      id: 'claude',
+      label: 'Claude CLI',
+      install: installClaude
+    });
+  }
+
+  return targets;
+}
+
+module.exports = {
+  detectInstallTargets
+};

package/lib/installers/skill-variants/SKILL.both.md

@@ -95,6 +95,7 @@ that only records memory when explicitly told to defeats the purpose of this ski
 Two rules that are not optional:
 
 - Record one memory per finding. A session that fixed three things records three memories, each with the fix phrase and the date in its summary. One combined summary is not recorded: it cannot be found by any question about one of its parts.
+- Tags are topics and project names, at most 6 per memory. Reuse tags that recall already returned for this project before inventing one. Never put workflow state (complete, pending, fixed, verified, in-progress), task or step numbers (task-5, us-3.2), or sentences into tags, whether via `empirical memory record --tags` or `record_graph_memory`: state belongs in data.status or in the summary, numbers belong in the summary.
 - Empirical first. A note in a local file, a scratchpad, or a host memory directory is not recorded; it is invisible to every other session, tool, and agent. Record in Empirical, then keep a local pointer if you want one.
 
 - Record durable decisions, project conventions, preferences, plans, and meaningful reflections
@@ -122,6 +123,18 @@ Two rules that are not optional:
   codebase, stop: that is work-history or scar-tissue, and filing it here makes it indistinguishable
   from something the user actually said or lived. Downstream features read these as the user's life.
 
+  **Say which one it is: pass `authorType`.** Same distinction, made explicit in the data instead
+  of left to be guessed from which transport you happened to use. `authorType: "user"` when the
+  memory is about the user — something they said, decided, prefer, or are working toward.
+  `authorType: "agent"` when it is your own work. Do not set `"agent"` merely because you are the
+  one calling: an agent writing down a real fact about the user is recording a user-authored fact,
+  and that is the whole point of this tool. On the CLI it is `--author-type user|agent`. Omit it
+  when you genuinely cannot tell, and it stays unresolved rather than being invented.
+
+  This is read back at retrieval time to keep an agent's session notes out of answers about the
+  user's own life and work, so a wrong value does not show up as a warning, it shows up as the
+  wrong memories coming back.
+
 - Append a correction or follow-up to an existing memory with `add_note_by_query` / `empirical memory note`.
 - Patch an existing memory with `update_memory_by_query` / `empirical memory update`.
 - Delete only when the user explicitly asks; use the delete tool with confirmation, or `empirical memory delete --match "<memory>" --confirm`.

package/lib/installers/skill-variants/SKILL.cli.md

@@ -85,6 +85,7 @@ that only records memory when explicitly told to defeats the purpose of this ski
 Two rules that are not optional:
 
 - Record one memory per finding. A session that fixed three things records three memories, each with the fix phrase and the date in its summary. One combined summary is not recorded: it cannot be found by any question about one of its parts.
+- Tags are topics and project names, at most 6 per memory. Reuse tags that recall already returned for this project before inventing one. Never put workflow state (complete, pending, fixed, verified, in-progress), task or step numbers (task-5, us-3.2), or sentences into tags passed via `empirical memory record --tags`: state belongs in data.status or in the summary, numbers belong in the summary.
 - Empirical first. A note in a local file, a scratchpad, or a host memory directory is not recorded; it is invisible to every other session, tool, and agent. Record in Empirical, then keep a local pointer if you want one.
 
 - Record durable decisions, project conventions, preferences, plans, and meaningful reflections
@@ -114,6 +115,18 @@ Two rules that are not optional:
   codebase, stop: that is work-history or scar-tissue, and filing it here makes it indistinguishable
   from something the user actually said or lived. Downstream features read these as the user's life.
 
+  **Say which one it is: pass `authorType`.** Same distinction, made explicit in the data instead
+  of left to be guessed from which transport you happened to use. `authorType: "user"` when the
+  memory is about the user — something they said, decided, prefer, or are working toward.
+  `authorType: "agent"` when it is your own work. Do not set `"agent"` merely because you are the
+  one calling: an agent writing down a real fact about the user is recording a user-authored fact,
+  and that is the whole point of this tool. On the CLI it is `--author-type user|agent`. Omit it
+  when you genuinely cannot tell, and it stays unresolved rather than being invented.
+
+  This is read back at retrieval time to keep an agent's session notes out of answers about the
+  user's own life and work, so a wrong value does not show up as a warning, it shows up as the
+  wrong memories coming back.
+
 - Append a correction or follow-up to an existing memory with `empirical memory note`.
 - Patch an existing memory with `empirical memory update`.
 - Delete only when the user explicitly asks: `empirical memory delete --match "<memory>" --confirm`.

package/lib/installers/skill-variants/SKILL.mcp.md

@@ -75,6 +75,7 @@ that only records memory when explicitly told to defeats the purpose of this ski
 Two rules that are not optional:
 
 - Record one memory per finding. A session that fixed three things records three memories, each with the fix phrase and the date in its summary. One combined summary is not recorded: it cannot be found by any question about one of its parts.
+- Tags are topics and project names, at most 6 per memory. Reuse tags that recall already returned for this project before inventing one. Never put workflow state (complete, pending, fixed, verified, in-progress), task or step numbers (task-5, us-3.2), or sentences into tags passed via `record_graph_memory`: state belongs in data.status or in the summary, numbers belong in the summary.
 - Empirical first. A note in a local file, a scratchpad, or a host memory directory is not recorded; it is invisible to every other session, tool, and agent. Record in Empirical, then keep a local pointer if you want one.
 
 - Record durable decisions, project conventions, preferences, plans, and meaningful reflections
@@ -91,6 +92,18 @@ Two rules that are not optional:
   codebase, stop: that is work-history or scar-tissue, and filing it here makes it indistinguishable
   from something the user actually said or lived. Downstream features read these as the user's life.
 
+  **Say which one it is: pass `authorType`.** Same distinction, made explicit in the data instead
+  of left to be guessed from which transport you happened to use. `authorType: "user"` when the
+  memory is about the user — something they said, decided, prefer, or are working toward.
+  `authorType: "agent"` when it is your own work. Do not set `"agent"` merely because you are the
+  one calling: an agent writing down a real fact about the user is recording a user-authored fact,
+  and that is the whole point of this tool. On the CLI it is `--author-type user|agent`. Omit it
+  when you genuinely cannot tell, and it stays unresolved rather than being invented.
+
+  This is read back at retrieval time to keep an agent's session notes out of answers about the
+  user's own life and work, so a wrong value does not show up as a warning, it shows up as the
+  wrong memories coming back.
+
 - Append a correction or follow-up to an existing memory with `add_note_by_query`.
 - Patch an existing memory with `update_memory_by_query`.
 - Delete only when the user explicitly asks; use `delete_memory_by_query` with confirmation.

package/lib/memoryCommands.js

@@ -47,7 +47,7 @@ const MEMORY_SUBCOMMAND_USAGE = {
   query: 'Usage: empirical memory query --match <query> [--top-k 3] [--category <cat>] [--tags <t1,t2>] [--tags-all <t1,t2>] [--status <s1,s2>] [--pretty] [--format table]',
   get: 'Usage: empirical memory get --memory-id <id> [--pretty]',
   list: 'Usage: empirical memory list [--category <cat>] [--tags <t1,t2>] [--start-date <ISO>] [--end-date <ISO>] [--limit 20] [--cursor <cursor>] [--oldest] [--pretty] [--format table]',
-  record: 'Usage: empirical memory record --category <cat> (--summary <text> | --summary - | --summary-file <path>) [--tags t1,t2] [--node-type observation] [--json-file <path>] [--no-dedupe] [--no-auto-link]',
+  record: 'Usage: empirical memory record --category <cat> (--summary <text> | --summary - | --summary-file <path>) [--tags t1,t2] [--node-type observation] [--json-file <path>] [--no-dedupe] [--no-auto-link] [--author-type user|agent]',
   note: 'Usage: empirical memory note --text <note> (--match <query> | --memory-id <id>) [--min-score 0.35]',
   update: 'Usage: empirical memory update (--match <query> | --memory-id <id>) (--patch-json <json> | --json-file <path>) [--min-score 0.35]',
   delete: 'Usage: empirical memory delete (--match <query> | --memory-id <id>) [--confirm] [--min-score 0.35]\nUsage: empirical memory delete (--tag <t> | --category <c> | --all) --confirm',
@@ -351,7 +351,7 @@ async function runMemoryRecord(args) {
     }
 
     if (!category || !summary) {
-      warn('Usage: empirical memory record --category <cat> (--summary <text> | --summary - | --summary-file <path>) [--tags t1,t2] [--node-type observation] [--json-file <path>] [--no-dedupe] [--no-auto-link]');
+      warn('Usage: empirical memory record --category <cat> (--summary <text> | --summary - | --summary-file <path>) [--tags t1,t2] [--node-type observation] [--json-file <path>] [--no-dedupe] [--no-auto-link] [--author-type user|agent]');
       warn('Valid categories: ' + MEMORY_CATEGORIES.join(', '));
       process.exit(1);
     }
@@ -395,6 +395,29 @@ async function runMemoryRecord(args) {
     payload.autoLink = false;
   }
 
+  // Authorship (2026-09-09). `source` is observable truth: this is the CLI, so say so.
+  // Until now the CLI sent nothing, the server fell back to the transport it saw ('api'),
+  // and api maps to authorType 'user' because that mapping was written for the web portal.
+  // Every agent write through this command was therefore stored as authored by the user.
+  // 'cli' resolves to authorType 'unknown', which is the honest answer when nobody says.
+  if (payload.source === undefined) payload.source = 'cli';
+
+  // --author-type lets the caller state what the transport cannot know. An agent recording
+  // its own session notes should pass 'agent'; an agent recording something the user said
+  // or decided should pass 'user'. Omitted, it stays 'unknown' rather than being guessed.
+  const authorType = parseOption(args, '--author-type', '');
+  if (authorType) {
+    if (!['user', 'agent', 'system'].includes(authorType)) {
+      warn(JSON.stringify({
+        error: true,
+        code: 'invalid_author_type',
+        message: `Invalid --author-type "${authorType}". Valid: user, agent, system`
+      }));
+      process.exit(1);
+    }
+    payload.authorType = authorType;
+  }
+
   const accessToken = requireAccessToken();
   const baseUrl = resolveApiBaseUrl();
   const recordUrl = new URL('/api/v1/memories', baseUrl).toString();