memorix 1.2.1 → 1.2.2

package/src/cli/commands/hooks-status.ts

@@ -1,71 +1,71 @@
-/**
- * CLI Command: memorix hooks status
- *
- * Show hook installation status for all agents.
+/**
+ * CLI Command: memorix hooks status
+ *
+ * Show hook installation status for all agents.
  * Status marks:
  *   [OK]  - installed and verified (config-based agent, file read directly)
  *   [??]  - installed but unverified (plugin-based agent, runtime load not confirmed)
  *   [!!]  - installed but outdated
  *   [ ]   - not installed
- */
-
-import { defineCommand } from 'citty';
-
-export default defineCommand({
-  meta: {
-    name: 'status',
-    description: 'Show hook installation status for all agents',
-  },
-  run: async () => {
-    const { getHookStatus } = await import('../../hooks/installers/index.js');
-    const { AGENT_SUPPORT_TIER } = await import('../../hooks/types.js');
-    const os = await import('node:os');
-    let cwd: string;
-    try { cwd = process.cwd(); } catch { cwd = os.homedir(); }
-
-    const statuses = await getHookStatus(cwd);
-
-    console.log('\nMemorix Hooks Status');
+ */
+
+import { defineCommand } from 'citty';
+
+export default defineCommand({
+  meta: {
+    name: 'status',
+    description: 'Show hook installation status for all agents',
+  },
+  run: async () => {
+    const { getHookStatus } = await import('../../hooks/installers/index.js');
+    const { AGENT_SUPPORT_TIER } = await import('../../hooks/types.js');
+    const os = await import('node:os');
+    let cwd: string;
+    try { cwd = process.cwd(); } catch { cwd = os.homedir(); }
+
+    const statuses = await getHookStatus(cwd);
+
+    console.log('\nMemorix Hooks Status');
     console.log('-'.repeat(50));
-
-    let hasOutdated = false;
-    let hasUnverified = false;
-    let hasRuntimeIssue = false;
-    for (const { agent, installed, outdated, verified, runtimeReady, configPath } of statuses) {
-      const icon = installed
-        ? (!runtimeReady ? '[!!]' : (outdated ? '[!!]' : (verified ? '[OK]' : '[??]')))
-        : '[ ]';
-      const label = agent.charAt(0).toUpperCase() + agent.slice(1);
-      const tier = AGENT_SUPPORT_TIER[agent];
+
+    let hasOutdated = false;
+    let hasUnverified = false;
+    let hasRuntimeIssue = false;
+    for (const { agent, installed, outdated, verified, runtimeReady, configPath } of statuses) {
+      const icon = installed
+        ? (!runtimeReady ? '[!!]' : (outdated ? '[!!]' : (verified ? '[OK]' : '[??]')))
+        : '[ ]';
+      const label = agent.charAt(0).toUpperCase() + agent.slice(1);
+      const tier = AGENT_SUPPORT_TIER[agent];
       const tierLabel = tier === 'core' ? '[core]' : tier === 'extended' ? '[extended]' : '[community]';
-      const suffix = !runtimeReady && installed
-        ? ' (installed but runtime NOT ready — missing pwsh on Windows)'
-        : outdated
-          ? ' (outdated - re-run `memorix hooks install`)'
-          : (!verified && installed ? ' (plugin file installed, runtime load unverified)' : '');
-      console.log(`${icon} ${tierLabel} ${label.padEnd(12)} ${installed ? configPath + suffix : '(not installed)'}`);
-      if (outdated) hasOutdated = true;
-      if (!verified && installed) hasUnverified = true;
-      if (!runtimeReady && installed) hasRuntimeIssue = true;
-    }
-
+      const suffix = !runtimeReady && installed
+        ? ' (installed but runtime NOT ready — missing pwsh on Windows)'
+        : outdated
+          ? ' (outdated - re-run `memorix hooks install`)'
+          : (!verified && installed ? ' (plugin file installed, runtime load unverified)' : '');
+      console.log(`${icon} ${tierLabel} ${label.padEnd(12)} ${installed ? configPath + suffix : '(not installed)'}`);
+      if (outdated) hasOutdated = true;
+      if (!verified && installed) hasUnverified = true;
+      if (!runtimeReady && installed) hasRuntimeIssue = true;
+    }
+
     console.log('\nSupport tiers: [core] core | [extended] extended | [community] community');
-
-    if (hasOutdated) {
-      console.log('\n[warn] Outdated hooks detected. Run `memorix hooks install` to update.');
-    }
-    if (hasUnverified) {
-      console.log('\n[info] [??] = plugin file installed but runtime load unverified.');
-      console.log('       For OpenCode: verify by running `opencode --log-level DEBUG` and checking');
-      console.log('       for [memorix-plugin] messages in the log output.');
-    }
-    if (hasRuntimeIssue) {
-      console.log('\n[warn] [!!] = installed but runtime will fail.');
-      console.log('       Copilot on Windows requires PowerShell v6+ (pwsh.exe) for hook execution.');
-      console.log('       Install pwsh: https://learn.microsoft.com/powershell/scripting/install/installing-powershell');
-      console.log('       Without pwsh, Copilot hooks will fail at runtime with "spawn pwsh.exe ENOENT".');
-    }
-
-    console.log('\nRun `memorix hooks install` to set up hooks for detected agents.');
-  },
-});
+
+    if (hasOutdated) {
+      console.log('\n[warn] Outdated hooks detected. Run `memorix hooks install` to update.');
+    }
+    if (hasUnverified) {
+      console.log('\n[info] [??] = plugin file installed but runtime load unverified.');
+      console.log('       For OpenCode: verify by running `opencode --log-level DEBUG` and checking');
+      console.log('       for [memorix-plugin] messages in the log output.');
+    }
+    if (hasRuntimeIssue) {
+      console.log('\n[warn] [!!] = installed but runtime will fail.');
+      console.log('       Copilot on Windows requires PowerShell v6+ (pwsh.exe) for hook execution.');
+      console.log('       Install pwsh: https://learn.microsoft.com/powershell/scripting/install/installing-powershell');
+      console.log('       Without pwsh, Copilot hooks will fail at runtime with "spawn pwsh.exe ENOENT".');
+    }
+
+    console.log('\nRun `memorix hooks install` to set up hooks for detected agents.');
+  },
+});

package/src/cli/commands/identity.ts

@@ -0,0 +1,116 @@
+import { defineCommand } from 'citty';
+import { AGENT_TYPE_ROLE_MAP } from '../../team/team-store.js';
+import { clearCliIdentity, loadCliIdentity, saveCliIdentity } from '../identity.js';
+import { emitError, emitResult, getCliProjectContext, parseCsvList } from './operator-shared.js';
+
+function requiredText(value: unknown, label: string): string {
+  const text = typeof value === 'string' ? value.trim() : '';
+  if (!text) throw new Error(`${label} is required`);
+  return text;
+}
+
+export default defineCommand({
+  meta: {
+    name: 'identity',
+    description: 'Select the explicit CLI actor used for private or team-scoped operations',
+  },
+  args: {
+    agentId: { type: 'string', description: 'Existing active coordination agent ID' },
+    agentType: { type: 'string', description: 'Agent type when joining (for example codex)' },
+    instanceId: { type: 'string', description: 'Stable instance identity when joining' },
+    name: { type: 'string', description: 'Optional display name when joining' },
+    role: { type: 'string', description: 'Optional coordination role when joining' },
+    capabilities: { type: 'string', description: 'Comma-separated capability list when joining' },
+    json: { type: 'boolean', description: 'Emit machine-readable JSON output' },
+  },
+  run: async ({ args }) => {
+    const action = ((args._ as string[])?.[0] || 'status').toLowerCase();
+    const asJson = !!args.json;
+
+    try {
+      const { project, dataDir, teamStore, identity, reader, identityWarning } = await getCliProjectContext();
+
+      switch (action) {
+        case 'status': {
+          const saved = await loadCliIdentity(dataDir, project.id);
+          const agent = saved ? teamStore.getAgent(saved.agentId) : undefined;
+          emitResult(
+            {
+              project,
+              identity,
+              savedIdentity: saved,
+              agent: agent ?? null,
+              reader,
+              ...(identityWarning ? { warning: identityWarning } : {}),
+            },
+            identity
+              ? `CLI identity: ${agent?.name ?? identity.agentId} (${identity.agentId})`
+              : identityWarning ?? 'CLI identity: project-scoped only. Use "memorix identity join" or "memorix identity use --agent-id <id>" for private/team operations.',
+            asJson,
+          );
+          return;
+        }
+
+        case 'join': {
+          const agentType = requiredText(args.agentType, 'agentType');
+          const agent = teamStore.registerAgent({
+            projectId: project.id,
+            agentType,
+            instanceId: args.instanceId as string | undefined,
+            name: args.name as string | undefined,
+            role: (args.role as string | undefined) || AGENT_TYPE_ROLE_MAP[agentType] || 'engineer',
+            capabilities: parseCsvList(args.capabilities as string | undefined),
+          });
+          const nextIdentity = {
+            agentId: agent.agent_id,
+            projectId: project.id,
+            activatedAt: new Date().toISOString(),
+          };
+          await saveCliIdentity(dataDir, nextIdentity);
+          emitResult(
+            { project, identity: nextIdentity, agent },
+            `CLI identity activated: ${agent.name} (${agent.agent_id})`,
+            asJson,
+          );
+          return;
+        }
+
+        case 'use': {
+          const agentId = requiredText(args.agentId, 'agentId');
+          const agent = teamStore.getAgent(agentId);
+          if (!agent || agent.project_id !== project.id || agent.status !== 'active') {
+            emitError('agentId must identify an active coordination member of this project.', asJson);
+            return;
+          }
+          const nextIdentity = {
+            agentId: agent.agent_id,
+            projectId: project.id,
+            activatedAt: new Date().toISOString(),
+          };
+          await saveCliIdentity(dataDir, nextIdentity);
+          emitResult(
+            { project, identity: nextIdentity, agent },
+            `CLI identity activated: ${agent.name} (${agent.agent_id})`,
+            asJson,
+          );
+          return;
+        }
+
+        case 'clear': {
+          await clearCliIdentity(dataDir);
+          emitResult(
+            { project, cleared: true },
+            'CLI identity cleared. Subsequent commands use project-scoped access only.',
+            asJson,
+          );
+          return;
+        }
+
+        default:
+          emitError(`unknown identity action "${action}". Use "status", "join", "use", or "clear".`, asJson);
+      }
+    } catch (error) {
+      emitError(error instanceof Error ? error.message : String(error), asJson);
+    }
+  },
+});

package/src/cli/commands/ingest-commit.ts

@@ -1,153 +1,153 @@
-/**
- * CLI Command: memorix ingest commit
- *
- * Extract engineering knowledge from a git commit and store as memory.
- * This is the Git→Memory direction — unique to Memorix.
- */
-
-import { defineCommand } from 'citty';
-import * as p from '@clack/prompts';
-
-export default defineCommand({
-  meta: {
-    name: 'commit',
-    description: 'Ingest a git commit as memory',
-  },
-  args: {
-    ref: {
-      type: 'string',
-      description: 'Commit ref (default: HEAD)',
-      required: false,
-    },
-    auto: {
-      type: 'boolean',
-      description: 'Non-interactive mode (used by git post-commit hook)',
-      required: false,
-    },
-    force: {
-      type: 'boolean',
-      description: 'Bypass Git noise filter and ingest anyway',
-      required: false,
-    },
-  },
-  run: async ({ args }) => {
-    const os = await import('node:os');
-    let cwd: string;
-    try { cwd = process.cwd(); } catch { cwd = os.homedir(); }
-
-    const ref = args.ref || 'HEAD';
-    const auto = !!args.auto;
-    const force = !!args.force;
-
-    if (!auto) p.intro(`Ingest commit: ${ref}`);
-
-    try {
-      const { getCommitInfo, ingestCommit } = await import('../../git/extractor.js');
-      const commit = getCommitInfo(cwd, ref);
-
-      // Noise filter: skip low-value commits (typo, format, lockfile, merge, etc.)
-      const { shouldFilterCommit } = await import('../../git/noise-filter.js');
-      const { getGitConfig } = await import('../../config.js');
-      const gitCfg = getGitConfig();
-      const filterResult = shouldFilterCommit(commit, {
-        skipMergeCommits: gitCfg.skipMergeCommits,
-        excludePatterns: gitCfg.excludePatterns,
-        noiseKeywords: gitCfg.noiseKeywords,
-      });
-      if (filterResult.skip && !force) {
-        if (auto) {
-          console.error(`[memorix] Skipped ${commit.shortHash}: ${filterResult.reason}`);
-          process.exit(0);
-        } else {
-          p.log.warn(`Commit ${commit.shortHash} filtered as noise: ${filterResult.reason}`);
-          p.outro('Use --force to override the noise filter.');
-        }
-        return;
-      }
-
-      const result = ingestCommit(commit);
-
-      // Store via memorix_store logic
-      const { initObservations, storeObservation } = await import('../../memory/observations.js');
-      const { getProjectDataDir } = await import('../../store/persistence.js');
-      const { detectProject } = await import('../../project/detector.js');
-      const { initObservationStore, getObservationStore: getStore } = await import('../../store/obs-store.js');
-
-      const project = detectProject(cwd);
-      if (!project) {
-        if (!auto) p.log.error('No .git found — not a project directory.');
-        return;
-      }
-      const dataDir = await getProjectDataDir(project.id);
-      await initObservationStore(dataDir);
-      await initObservations(dataDir);
-
-      // Dedup: skip if this commit hash was already ingested
-      const existingObs = await getStore().loadAll() as Array<{ commitHash?: string }>;
-      if (existingObs.some(o => o.commitHash === commit.hash)) {
-        if (!auto) p.log.warn(`Commit ${commit.shortHash} already ingested. Skipping.`);
-        if (auto) process.exit(0);
-        return;
-      }
-
-      if (!auto) {
-        // Interactive: show details and confirm
-        console.log('');
-        console.log(`Commit: ${commit.shortHash} — ${commit.subject}`);
-        console.log(`Author: ${commit.author} @ ${commit.date}`);
-        console.log(`Files:  ${commit.filesChanged.length} changed (+${commit.insertions}/-${commit.deletions})`);
-        console.log('');
-        console.log(`Entity: ${result.entityName}`);
-        console.log(`Type:   ${result.type}`);
-        console.log(`Title:  ${result.title}`);
-        console.log('');
-        console.log('Facts:');
-        for (const fact of result.facts) {
-          console.log(`  - ${fact}`);
-        }
-        if (result.concepts.length > 0) {
-          console.log(`Concepts: ${result.concepts.join(', ')}`);
-        }
-        console.log('');
-
-        const confirmed = await p.confirm({
-          message: 'Store this commit as memory?',
-        });
-
-        if (p.isCancel(confirmed) || !confirmed) {
-          p.outro('Ingest cancelled.');
-          return;
-        }
-      }
-
-      await storeObservation({
-        entityName: result.entityName,
-        type: result.type as any,
-        title: result.title,
-        sourceDetail: 'git-ingest',
-        narrative: result.narrative,
-        facts: result.facts,
-        concepts: result.concepts,
-        filesModified: result.filesModified,
-        projectId: project.id,
-        source: 'git',
-        commitHash: commit.hash,
-      });
-
-      if (auto) {
-        console.error(`[memorix] Git memory: ${commit.shortHash} — ${commit.subject}`);
-        process.exit(0);
-      } else {
-        p.outro(`Memory stored from commit ${commit.shortHash}.`);
-      }
-    } catch (err) {
-      if (auto) {
-        console.error(`[memorix] Git hook ingest failed: ${err}`);
-        process.exit(1);
-      } else {
-        console.error(`Failed to ingest commit: ${err}`);
-        p.outro('Ingest failed. Make sure you are in a git repository.');
-      }
-    }
-  },
-});
+/**
+ * CLI Command: memorix ingest commit
+ *
+ * Extract engineering knowledge from a git commit and store as memory.
+ * This is the Git→Memory direction — unique to Memorix.
+ */
+
+import { defineCommand } from 'citty';
+import * as p from '@clack/prompts';
+
+export default defineCommand({
+  meta: {
+    name: 'commit',
+    description: 'Ingest a git commit as memory',
+  },
+  args: {
+    ref: {
+      type: 'string',
+      description: 'Commit ref (default: HEAD)',
+      required: false,
+    },
+    auto: {
+      type: 'boolean',
+      description: 'Non-interactive mode (used by git post-commit hook)',
+      required: false,
+    },
+    force: {
+      type: 'boolean',
+      description: 'Bypass Git noise filter and ingest anyway',
+      required: false,
+    },
+  },
+  run: async ({ args }) => {
+    const os = await import('node:os');
+    let cwd: string;
+    try { cwd = process.cwd(); } catch { cwd = os.homedir(); }
+
+    const ref = args.ref || 'HEAD';
+    const auto = !!args.auto;
+    const force = !!args.force;
+
+    if (!auto) p.intro(`Ingest commit: ${ref}`);
+
+    try {
+      const { getCommitInfo, ingestCommit } = await import('../../git/extractor.js');
+      const commit = getCommitInfo(cwd, ref);
+
+      // Noise filter: skip low-value commits (typo, format, lockfile, merge, etc.)
+      const { shouldFilterCommit } = await import('../../git/noise-filter.js');
+      const { getGitConfig } = await import('../../config.js');
+      const gitCfg = getGitConfig();
+      const filterResult = shouldFilterCommit(commit, {
+        skipMergeCommits: gitCfg.skipMergeCommits,
+        excludePatterns: gitCfg.excludePatterns,
+        noiseKeywords: gitCfg.noiseKeywords,
+      });
+      if (filterResult.skip && !force) {
+        if (auto) {
+          console.error(`[memorix] Skipped ${commit.shortHash}: ${filterResult.reason}`);
+          process.exit(0);
+        } else {
+          p.log.warn(`Commit ${commit.shortHash} filtered as noise: ${filterResult.reason}`);
+          p.outro('Use --force to override the noise filter.');
+        }
+        return;
+      }
+
+      const result = ingestCommit(commit);
+
+      // Store via memorix_store logic
+      const { initObservations, storeObservation } = await import('../../memory/observations.js');
+      const { getProjectDataDir } = await import('../../store/persistence.js');
+      const { detectProject } = await import('../../project/detector.js');
+      const { initObservationStore, getObservationStore: getStore } = await import('../../store/obs-store.js');
+
+      const project = detectProject(cwd);
+      if (!project) {
+        if (!auto) p.log.error('No .git found — not a project directory.');
+        return;
+      }
+      const dataDir = await getProjectDataDir(project.id);
+      await initObservationStore(dataDir);
+      await initObservations(dataDir);
+
+      // Dedup: skip if this commit hash was already ingested
+      const existingObs = await getStore().loadAll() as Array<{ commitHash?: string }>;
+      if (existingObs.some(o => o.commitHash === commit.hash)) {
+        if (!auto) p.log.warn(`Commit ${commit.shortHash} already ingested. Skipping.`);
+        if (auto) process.exit(0);
+        return;
+      }
+
+      if (!auto) {
+        // Interactive: show details and confirm
+        console.log('');
+        console.log(`Commit: ${commit.shortHash} — ${commit.subject}`);
+        console.log(`Author: ${commit.author} @ ${commit.date}`);
+        console.log(`Files:  ${commit.filesChanged.length} changed (+${commit.insertions}/-${commit.deletions})`);
+        console.log('');
+        console.log(`Entity: ${result.entityName}`);
+        console.log(`Type:   ${result.type}`);
+        console.log(`Title:  ${result.title}`);
+        console.log('');
+        console.log('Facts:');
+        for (const fact of result.facts) {
+          console.log(`  - ${fact}`);
+        }
+        if (result.concepts.length > 0) {
+          console.log(`Concepts: ${result.concepts.join(', ')}`);
+        }
+        console.log('');
+
+        const confirmed = await p.confirm({
+          message: 'Store this commit as memory?',
+        });
+
+        if (p.isCancel(confirmed) || !confirmed) {
+          p.outro('Ingest cancelled.');
+          return;
+        }
+      }
+
+      await storeObservation({
+        entityName: result.entityName,
+        type: result.type as any,
+        title: result.title,
+        sourceDetail: 'git-ingest',
+        narrative: result.narrative,
+        facts: result.facts,
+        concepts: result.concepts,
+        filesModified: result.filesModified,
+        projectId: project.id,
+        source: 'git',
+        commitHash: commit.hash,
+      });
+
+      if (auto) {
+        console.error(`[memorix] Git memory: ${commit.shortHash} — ${commit.subject}`);
+        process.exit(0);
+      } else {
+        p.outro(`Memory stored from commit ${commit.shortHash}.`);
+      }
+    } catch (err) {
+      if (auto) {
+        console.error(`[memorix] Git hook ingest failed: ${err}`);
+        process.exit(1);
+      } else {
+        console.error(`Failed to ingest commit: ${err}`);
+        p.outro('Ingest failed. Make sure you are in a git repository.');
+      }
+    }
+  },
+});