@pcircle/memesh 4.5.0 → 4.5.1

package/dist/transports/cli/cli.js

@@ -6,11 +6,11 @@ import path from 'path';
 import { fileURLToPath } from 'url';
 import { openDatabase, closeDatabase, getDatabase, reindexFts, allowVectorIndexRebuild } from '../../db.js';
 import { remember, recallWithConflicts, forget, exportMemories, importMemories, learn, reindex, setPinned } from '../../core/operations.js';
-import { verifyAgentWork } from '../../core/verifier.js';
 import { readConfig, writeConfig, maskApiKey, detectCapabilities } from '../../core/config.js';
 import { MAX_LANGUAGE_LENGTH, languageValueError } from '../../core/output-language.js';
-import { getDbPath } from '../../core/paths.js';
+import { getDbPath, redactSecrets, redactUserPaths } from '../../core/paths.js';
 import { flushPendingEmbeddings, canRefillVectorIndex } from '../../core/embedder.js';
+import { NAMESPACES } from '../../core/types.js';
 async function withDatabase(fn) {
     openDatabase();
     try {
@@ -20,6 +20,12 @@ async function withDatabase(fn) {
         closeDatabase();
     }
 }
+function requireOneOf(value, allowed, flag) {
+    if (value === undefined || allowed.includes(value))
+        return;
+    console.error(`Error: ${flag} "${value}" is not valid. Use one of: ${allowed.join(', ')}.`);
+    process.exit(1);
+}
 const packageJsonPath = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '../../../package.json');
 const packageRoot = path.dirname(packageJsonPath);
 const pkg = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
@@ -38,9 +44,12 @@ program
     .option('--type <type>', 'Entity type')
     .option('--obs <observations...>', 'Observations (space-separated)')
     .option('--tags <tags...>', 'Tags (space-separated)')
-    .option('--namespace <namespace>', 'Namespace: personal, team, or global (default: personal)')
+    .option('--namespace <namespace>', 'Namespace: personal, team, or global. On a NEW memory this places it (default personal); on one that already exists it MOVES it out of the scope it is in — omit the flag to leave it alone.')
+    .option('--supersedes <name...>', 'This memory replaces the named one — ARCHIVES it immediately (recoverable; nothing is deleted)')
+    .option('--contradicts <name...>', 'This memory cannot both be true with the named one — both surface as a conflict on every recall')
     .option('--json', 'Output as JSON')
     .action(async (text, opts) => {
+    requireOneOf(opts.namespace, NAMESPACES, '--namespace');
     if (text && !opts.name && !opts.type) {
         const slug = String(text)
             .toLowerCase()
@@ -68,6 +77,10 @@ program
             '  memesh remember "Use OAuth 2.0 with PKCE"');
         process.exit(1);
     }
+    const relations = [
+        ...(opts.supersedes ?? []).map(to => ({ to, type: 'supersedes' })),
+        ...(opts.contradicts ?? []).map(to => ({ to, type: 'contradicts' })),
+    ];
     await withDatabase(async () => {
         const result = remember({
             name: opts.name,
@@ -75,13 +88,31 @@ program
             observations: opts.obs,
             tags: opts.tags,
             namespace: opts.namespace,
+            relations: relations.length > 0 ? relations : undefined,
+            sourceHost: 'cli',
         });
         if (opts.json) {
             console.log(JSON.stringify(result));
         }
         else {
             console.log(`✅ Stored "${result.name}" (${result.observations} observations, ${result.tags} tags)`);
+            if (result.movedFromNamespace) {
+                console.log(`   moved: ${result.movedFromNamespace} → ${opts.namespace} (was in ${result.movedFromNamespace}; re-run with --namespace ${result.movedFromNamespace} to put it back)`);
+            }
+            if (result.superseded?.length) {
+                console.log(`   archived as superseded: ${result.superseded.join(', ')}`);
+            }
+            const contradicted = (result.relationsCreated ?? [])
+                .filter(r => r.type === 'contradicts')
+                .map(r => r.to);
+            if (contradicted.length > 0) {
+                console.log(`   conflicts stated: ${contradicted.join(', ')}`);
+            }
+            for (const err of result.relationErrors ?? [])
+                console.error(`   ⚠️  ${err}`);
         }
+        if (result.relationErrors?.length)
+            process.exitCode = 1;
         await flushPendingEmbeddings();
     });
 });
@@ -96,6 +127,7 @@ program
     .option('--cross-project', 'Search across all project tags (ignores --tag filter)')
     .option('--json', 'Output as JSON')
     .action(async (query, opts) => {
+    requireOneOf(opts.namespace, NAMESPACES, '--namespace');
     await withDatabase(async () => {
         const { entities, conflicts } = await recallWithConflicts({
             query: query || undefined,
@@ -173,8 +205,14 @@ program
         else if (result.observation_removed) {
             console.log(`✂️  Removed observation (${result.remaining_observations} remaining)`);
         }
+        else if (opts.observation && result.entity_found) {
+            console.log(`Entity "${opts.name}" has no observation matching that text (${result.remaining_observations} observation(s) present).`);
+            console.log(`See them with: memesh recall "${opts.name}" --json`);
+            process.exitCode = 1;
+        }
         else {
             console.log(`Entity "${opts.name}" not found`);
+            process.exitCode = 1;
         }
     });
 });
@@ -219,6 +257,31 @@ program
     console.error('so it is not a like-for-like replacement for compressing one named entity.');
     process.exitCode = 1;
 });
+program
+    .command('verify')
+    .description('(retired) Removed with the agentic-orchestration experiment — see the message this prints')
+    .allowUnknownOption()
+    .action(() => {
+    console.error('`memesh verify` has been retired, along with the agentic-orchestration experiment.');
+    console.error('');
+    console.error('It recorded a verification report for background-agent work. The protocol it');
+    console.error('served was removed without ever leaving opt-in. Run your own verification');
+    console.error('(typecheck / tests / lint) and store conclusions with `memesh remember` if you');
+    console.error('want them remembered. Existing verification_record entities are untouched.');
+    process.exitCode = 1;
+});
+program
+    .command('patterns')
+    .description('(retired) Removed with the agentic-orchestration experiment — see the message this prints')
+    .allowUnknownOption()
+    .action(() => {
+    console.error('`memesh patterns` has been retired, along with the agentic-orchestration experiment.');
+    console.error('');
+    console.error('It displayed the experiment\'s local skill-usage telemetry, which is no longer');
+    console.error('written. A leftover ~/.memesh/skill-usage.jsonl is inert and safe to delete.');
+    console.error('For work-pattern insights, use the `user_patterns` MCP tool or GET /v1/patterns.');
+    process.exitCode = 1;
+});
 program
     .command('export')
     .description('Export memories as JSON. Defaults to stdout (pipe-friendly); use `-o <file>` to write directly.')
@@ -227,6 +290,7 @@ program
     .option('--limit <n>', 'Max entities to export', '1000')
     .option('-o, --out <file>', 'Write JSON to <file> instead of stdout. Parent directory must exist.')
     .action(async (opts) => {
+    requireOneOf(opts.namespace, NAMESPACES, '--namespace');
     await withDatabase(() => {
         const result = exportMemories({
             tag: opts.tag,
@@ -235,6 +299,12 @@ program
         });
         const json = JSON.stringify(result, null, 2);
         if (opts.out) {
+            const outDir = path.dirname(path.resolve(opts.out));
+            if (!fs.existsSync(outDir)) {
+                console.error(`Error: cannot write ${opts.out} — the directory ${outDir} does not exist.`);
+                console.error(`       Create it first (mkdir -p "${outDir}"), or drop -o to write to stdout.`);
+                process.exit(1);
+            }
             fs.writeFileSync(opts.out, json + '\n');
             process.stderr.write(`✅ Exported ${result.entity_count} entities to ${opts.out}\n`);
         }
@@ -250,6 +320,8 @@ program
     .option('--namespace <ns>', 'Override namespace for all imported entities')
     .option('--merge <strategy>', 'Merge strategy: skip | overwrite | append', 'skip')
     .action(async (file, opts) => {
+    requireOneOf(opts.merge, ['skip', 'overwrite', 'append'], '--merge');
+    requireOneOf(opts.namespace, NAMESPACES, '--namespace');
     await withDatabase(() => {
         let raw;
         try {
@@ -281,11 +353,18 @@ program
             console.error(`       Try: memesh export > my-export.json && memesh import my-export.json`);
             process.exit(1);
         }
-        const result = importMemories({
-            data: data,
-            namespace: opts.namespace,
-            merge_strategy: opts.merge,
-        });
+        let result;
+        try {
+            result = importMemories({
+                data: data,
+                namespace: opts.namespace,
+                merge_strategy: opts.merge,
+            });
+        }
+        catch (err) {
+            console.error(`Error: ${err instanceof Error ? err.message : String(err)}`);
+            process.exit(1);
+        }
         console.log(`Imported: ${result.imported}, Skipped: ${result.skipped}, Appended: ${result.appended}`);
         if (result.errors.length > 0) {
             console.error(`Errors:\n  ${result.errors.join('\n  ')}`);
@@ -303,6 +382,7 @@ program
     .option('--severity <level>', 'Severity: critical|major|minor', 'minor')
     .option('--json', 'Output as JSON')
     .action(async (opts) => {
+    requireOneOf(opts.severity, ['critical', 'major', 'minor'], '--severity');
     await withDatabase(() => {
         const result = learn({
             error: opts.error,
@@ -310,6 +390,7 @@ program
             root_cause: opts.rootCause,
             prevention: opts.prevention,
             severity: opts.severity,
+            sourceHost: 'cli',
         });
         if (opts.json) {
             console.log(JSON.stringify(result));
@@ -319,50 +400,6 @@ program
         }
     });
 });
-program
-    .command('verify <workdir>')
-    .description('Record a verification report for agent work in <workdir>. ' +
-    'Exit codes: 0 pass, 1 fail, 2 unverified (nothing was checked).')
-    .requiredOption('--agent-id <id>', 'Identifier for the agent being verified')
-    .option('--base <ref>', 'Git ref to diff against (default: merge-base with origin/main)')
-    .option('--expected-files <n>', 'Number of files the agent claimed to change', (v) => parseInt(v, 10))
-    .option('--report <path>', 'Path to a JSON file with pre-computed report (typecheck/tests/lint/build)')
-    .option('--json', 'Output as JSON')
-    .action(async (workdir, opts) => {
-    await withDatabase(() => {
-        let externalReport;
-        let result;
-        try {
-            if (opts.report) {
-                const raw = fs.readFileSync(opts.report, 'utf8');
-                externalReport = JSON.parse(raw);
-            }
-            result = verifyAgentWork({
-                agent_id: opts.agentId,
-                workdir: path.resolve(workdir),
-                base: opts.base,
-                claim: opts.expectedFiles != null ? { expected_files: opts.expectedFiles } : undefined,
-                report: externalReport,
-            });
-        }
-        catch (err) {
-            console.error(err instanceof Error ? err.message : String(err));
-            console.error('Nothing was checked (exit 2 = unverified).');
-            process.exit(2);
-        }
-        if (opts.json) {
-            console.log(JSON.stringify(result, null, 2));
-        }
-        else {
-            console.log(`${result.verdict.toUpperCase()}  agent=${opts.agentId}  entity=${result.entity_name}`);
-            console.log(`  reality: ${result.reality_check.summary}`);
-            if (result.verdict === 'unverified') {
-                console.log('  nothing was checked — pass --expected-files <n> and/or --report <file> to verify something.');
-            }
-        }
-        process.exit(result.verdict === 'pass' ? 0 : result.verdict === 'fail' ? 1 : 2);
-    });
-});
 const configCmd = program.command('config').description('Manage configuration');
 configCmd
     .command('list')
@@ -392,7 +429,6 @@ const ALLOWED_KEYS = new Set([
     'embedder.model',
     'autoUpdate',
     'sessionLimit',
-    'enableAgenticOrchestration',
     'autoCapture',
     'llmFallbacks',
     'language',
@@ -516,7 +552,7 @@ configCmd
         coerced = parseInt(value, 10);
     if (canonical === 'llmFallbacks')
         coerced = JSON.parse(value);
-    if (canonical === 'enableAgenticOrchestration' || canonical === 'autoCapture') {
+    if (canonical === 'autoCapture') {
         coerced = value === 'true' || value === '1';
     }
     const config = readConfig();
@@ -524,6 +560,10 @@ configCmd
     writeConfig(config);
     const displayValue = canonical.toLowerCase().includes('key') ? maskApiKey(String(value)) : String(value);
     console.log(`✅ Set ${canonical} = ${displayValue}`);
+    if (canonical === 'llm.apiKey' && !config.llm?.provider) {
+        console.log('⚠️  No llm.provider is set, so this key is not used yet and LLM features stay off.');
+        console.log('    Set one with: memesh config set llm.provider <anthropic|openai|ollama>');
+    }
 });
 configCmd
     .command('unset')
@@ -592,10 +632,16 @@ program
     .description('Start the HTTP API server and web dashboard')
     .option('--port <port>', 'Port number', '3737')
     .option('--host <host>', 'Host to bind', '127.0.0.1')
-    .option('--allow-remote', 'Allow binding to non-loopback hosts. A bearer token is generated and REQUIRED for every /v1 request — the startup output shows where it lives and how to rotate it.')
+    .option('--allow-remote', 'Permit binding to a non-loopback host. Pair it with --host; on a non-loopback bind a bearer token is generated and REQUIRED for every /v1 request, and the startup output says where it lives. On the default loopback host this flag changes nothing.')
     .action(async (opts) => {
     const { startServer } = await import('../http/server.js');
-    startServer(opts.host, parseInt(opts.port, 10), { allowRemote: opts.allowRemote, autoUpdateCheck: true });
+    try {
+        startServer(opts.host, parseInt(opts.port, 10), { allowRemote: opts.allowRemote, autoUpdateCheck: true });
+    }
+    catch (err) {
+        console.error(`MeMesh: ${err instanceof Error ? err.message : String(err)}`);
+        process.exit(1);
+    }
 });
 program
     .command('update')
@@ -641,6 +687,12 @@ program
     .option('--prune <days>', 'Delete rows older than N days BEFORE rendering (closes v4.2.0 retention gap)', (v) => parseInt(v, 10))
     .option('--json', 'Output as JSON')
     .action(async (opts) => {
+    for (const [flag, value] of [['--window', opts.window], ['--prune', opts.prune]]) {
+        if (value !== undefined && !Number.isFinite(value)) {
+            console.error(`Error: ${flag} needs a number of days.`);
+            process.exit(1);
+        }
+    }
     await withDatabase(async () => {
         const { summariseTelemetry, pruneTelemetry } = await import('../../core/llm-telemetry.js');
         let pruneResult = null;
@@ -1011,6 +1063,11 @@ dreamCmd
         });
         console.log(`${opts.dryRun ? '[dry-run] ' : ''}Dream pass complete in ${result.durationMs}ms`);
         console.log(`  clusters scanned: ${result.clustersScanned}`);
+        if (result.clusteringMode) {
+            console.log(`  grouped by:       ${result.clusteringMode === 'semantic' ? 'meaning (embeddings)' : 'calendar week (no embeddings)'}`);
+        }
+        if (result.clusteringNote)
+            console.log(`    note: ${result.clusteringNote}`);
         console.log(`  LLM calls:        ${result.llmCalls}`);
         console.log(`  proposals created: ${result.proposalsCreated}`);
         if (result.skipped.length > 0) {
@@ -1209,6 +1266,9 @@ program
         }
         console.log(`${opts.dryRun ? '[dry-run] ' : ''}Settings: ${result.settingsPath}`);
         console.log(`${opts.dryRun ? '[dry-run] Would add ' : 'Added '}${result.added} hook entr${result.added === 1 ? 'y' : 'ies'}, ${opts.dryRun ? 'would skip ' : 'skipped '}${result.skipped} already-installed.`);
+        if (result.pruned > 0) {
+            console.log(`${opts.dryRun ? '[dry-run] Would remove ' : 'Removed '}${result.pruned} retired memesh hook entr${result.pruned === 1 ? 'y' : 'ies'} no longer shipped by this version.`);
+        }
         if (result.backupPath)
             console.log(`Backup: ${result.backupPath}`);
         if (result.conflicts.length > 0) {
@@ -1289,11 +1349,19 @@ program
             body += `\n\n---\n**System Info**\n- Version: \`${pkg.version}\`\n- Node: \`${process.version}\`\n- Platform: \`${process.platform} ${process.arch}\`\n_Diagnostics unavailable: doctor probe failed._`;
         }
     }
+    body = redactUserPaths(redactSecrets(body));
     const url = `https://github.com/PCIRCLE-AI/memesh-llm-memory/issues/new?title=${encodeURIComponent(`[${typeLabel}] `)}&body=${encodeURIComponent(body)}&labels=${encodeURIComponent(labels)}`;
     if (opts.open === false) {
         console.log(url);
         return;
     }
+    console.log('This will be pre-filled into a PUBLIC GitHub issue:');
+    console.log('---');
+    console.log(body);
+    console.log('---');
+    if (opts.diagnostics !== false) {
+        console.log('Re-run with --no-diagnostics to leave out the install ID and the doctor report.');
+    }
     const { spawn } = await import('child_process');
     const cmd = process.platform === 'darwin' ? 'open'
         : process.platform === 'win32' ? 'cmd'
@@ -1319,6 +1387,7 @@ program
     'stored embedding first. Needed only when switching embedding providers.')
     .option('--json', 'Output as JSON')
     .action(async (opts) => {
+    requireOneOf(opts.namespace, NAMESPACES, '--namespace');
     try {
         if (opts.fts) {
             if (opts.vectors) {
@@ -1403,33 +1472,6 @@ program
         throw err;
     }
 });
-program
-    .command('patterns')
-    .description('Show local skill-usage telemetry (agentic-orchestration banner injections, verify_agent_work invocations). Local-only — never uploaded.')
-    .option('--json', 'Output as JSON')
-    .action(async (opts) => {
-    const { summariseSkillUsage } = await import('../../core/skill-usage-log.js');
-    const summary = summariseSkillUsage();
-    if (opts.json) {
-        console.log(JSON.stringify(summary, null, 2));
-        return;
-    }
-    console.log(`Skill usage log: ${summary.log_path}`);
-    console.log(`  total events: ${summary.total_events}`);
-    console.log(`  log size:     ${summary.log_bytes} bytes`);
-    if (summary.first_event)
-        console.log(`  first event:  ${summary.first_event}`);
-    if (summary.last_event)
-        console.log(`  last event:   ${summary.last_event}`);
-    if (summary.total_events === 0) {
-        console.log('  (no events recorded yet — open a Claude Code session with memesh installed, or run a verification, to start collecting)');
-        return;
-    }
-    console.log('  events by name:');
-    for (const [name, count] of Object.entries(summary.events_by_name).sort((a, b) => b[1] - a[1])) {
-        console.log(`    ${count.toString().padStart(6)}  ${name}`);
-    }
-});
 program
     .command('status')
     .description('Show MeMesh status and capabilities')
@@ -1444,7 +1486,7 @@ program
     console.log(`MeMesh v${pkg.version}`);
     console.log(`Search level: ${caps.searchLevel} (${caps.searchLevel === 1 ? 'Smart Mode' : 'Core'})`);
     console.log(`Embeddings: ${caps.embeddings}`);
-    console.log(`LLM: ${caps.llm ? `${caps.llm.provider} (${caps.llm.model})` : 'not configured'}`);
+    console.log(`LLM: ${caps.llm ? `${caps.llm.provider} (${caps.llm.model ?? 'default'})` : 'not configured'}`);
     console.log(`Install method: ${installSupport.label}`);
     for (const line of formatUpdateCheckStatus(update)) {
         console.log(`\n${line}`);