@monoes/monomindcli 2.7.3 → 2.7.5

package/bin/cli.js

@@ -287,7 +287,23 @@ if (isMCPMode) {
   const isDaemonChild = cliArgs.includes('--foreground-worker-internal');
   cli.run().then(() => {
     if (!isDaemonChild) {
-      process.exit(process.exitCode ?? 0);
+      // Prefer a natural exit over process.exit(). Any command that touched
+      // embeddings has onnxruntime-node's thread pool loaded in-process (via
+      // memory-bridge -> @huggingface/transformers), and forcing exit out from
+      // under it aborts the process:
+      //   libc++abi: terminating due to uncaught exception of type
+      //   std::__1::system_error: mutex lock failed: Invalid argument
+      // — an exit code of 134 on `doctor`, `memory store`, `memory search`,
+      // etc., despite the command itself having succeeded. Disposing the
+      // pipeline first does NOT help; only letting the loop drain does.
+      //
+      // So: publish the exit code and let node exit on its own once the loop
+      // empties. The unref'd timer below still fires if something lingers
+      // (it does not itself hold the process open), preserving the original
+      // guarantee that the CLI never hangs forever on a stray handle.
+      process.exitCode = process.exitCode ?? 0;
+      const FORCE_EXIT_MS = 5000;
+      setTimeout(() => process.exit(process.exitCode ?? 0), FORCE_EXIT_MS).unref();
     }
     // Daemon child: let the event loop stay alive on its own ref'd interval.
   }).catch((error) => {

package/dist/src/commands/doctor-project-checks.d.ts

@@ -23,6 +23,9 @@ export declare function fixStaleHelpers(): Promise<boolean>;
 export declare function checkHelpersFresh(): Promise<HealthCheck>;
 export declare function checkMonoesIntegration(): Promise<HealthCheck>;
 export declare function checkGitignoreCoverage(): Promise<HealthCheck>;
+/** Creates or appends the missing gitignore entries. Returns true if it wrote.
+ *  Append-only and idempotent: never rewrites lines the user already has. */
+export declare function fixGitignoreCoverage(): Promise<boolean>;
 export declare function checkAgentRegistry(): Promise<HealthCheck>;
 export declare function checkGuidanceGates(): Promise<HealthCheck>;
 /**

package/dist/src/commands/doctor-project-checks.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"doctor-project-checks.d.ts","sourceRoot":"","sources":["../../../src/commands/doctor-project-checks.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAOH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAS1D,YAAY,EAAE,WAAW,EAAE,CAAC;AAE5B,wBAAsB,eAAe,IAAI,OAAO,CAAC,WAAW,CAAC,CAiB5D;AAED,wBAAsB,mBAAmB,IAAI,OAAO,CAAC,WAAW,CAAC,CAahE;AAED;;2CAE2C;AAC3C,wBAAsB,yBAAyB,IAAI,OAAO,CAAC,WAAW,CAAC,CAkBtE;AAED;;8DAE8D;AAC9D,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,WAAW,CAAC,CA8BlE;AAED,wBAAsB,YAAY,IAAI,OAAO,CAAC,WAAW,CAAC,CAyBzD;AAED,wBAAsB,eAAe,IAAI,OAAO,CAAC,WAAW,CAAC,CAsB5D;AAED,wBAAsB,cAAc,IAAI,OAAO,CAAC,WAAW,CAAC,CA6B3D;AAED,wBAAsB,uBAAuB,IAAI,OAAO,CAAC,WAAW,CAAC,CA+BpE;AAED,wBAAsB,iBAAiB,IAAI,OAAO,CAAC,WAAW,CAAC,CAwB9D;AAoED,wBAAsB,eAAe,IAAI,OAAO,CAAC,OAAO,CAAC,CAiBxD;AAED,wBAAsB,iBAAiB,IAAI,OAAO,CAAC,WAAW,CAAC,CAa9D;AAoDD,wBAAsB,sBAAsB,IAAI,OAAO,CAAC,WAAW,CAAC,CAWnE;AAmBD,wBAAsB,sBAAsB,IAAI,OAAO,CAAC,WAAW,CAAC,CAmBnE;AAED,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,WAAW,CAAC,CAuC/D;AAED,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,WAAW,CAAC,CAwB/D;AAkBD;;;;;GAKG;AACH,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,WAAW,CAAC,CAiClE;AAED;;GAEG;AACH,wBAAsB,0BAA0B,IAAI,OAAO,CAAC,WAAW,CAAC,CA6BvE;AAED;;GAEG;AACH,wBAAsB,sBAAsB,IAAI,OAAO,CAAC,WAAW,CAAC,CAyBnE"}
+{"version":3,"file":"doctor-project-checks.d.ts","sourceRoot":"","sources":["../../../src/commands/doctor-project-checks.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAOH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAS1D,YAAY,EAAE,WAAW,EAAE,CAAC;AAE5B,wBAAsB,eAAe,IAAI,OAAO,CAAC,WAAW,CAAC,CAiB5D;AAED,wBAAsB,mBAAmB,IAAI,OAAO,CAAC,WAAW,CAAC,CAahE;AAED;;2CAE2C;AAC3C,wBAAsB,yBAAyB,IAAI,OAAO,CAAC,WAAW,CAAC,CAkBtE;AAED;;8DAE8D;AAC9D,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,WAAW,CAAC,CA8BlE;AAED,wBAAsB,YAAY,IAAI,OAAO,CAAC,WAAW,CAAC,CAyBzD;AAED,wBAAsB,eAAe,IAAI,OAAO,CAAC,WAAW,CAAC,CAsB5D;AAED,wBAAsB,cAAc,IAAI,OAAO,CAAC,WAAW,CAAC,CA6B3D;AAED,wBAAsB,uBAAuB,IAAI,OAAO,CAAC,WAAW,CAAC,CA+BpE;AAED,wBAAsB,iBAAiB,IAAI,OAAO,CAAC,WAAW,CAAC,CAwB9D;AAoED,wBAAsB,eAAe,IAAI,OAAO,CAAC,OAAO,CAAC,CAiBxD;AAED,wBAAsB,iBAAiB,IAAI,OAAO,CAAC,WAAW,CAAC,CAa9D;AAoDD,wBAAsB,sBAAsB,IAAI,OAAO,CAAC,WAAW,CAAC,CAWnE;AAyCD,wBAAsB,sBAAsB,IAAI,OAAO,CAAC,WAAW,CAAC,CAiBnE;AA4BD;6EAC6E;AAC7E,wBAAsB,oBAAoB,IAAI,OAAO,CAAC,OAAO,CAAC,CAoB7D;AAED,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,WAAW,CAAC,CAuC/D;AAED,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,WAAW,CAAC,CAwB/D;AAkBD;;;;;GAKG;AACH,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,WAAW,CAAC,CAiClE;AAED;;GAEG;AACH,wBAAsB,0BAA0B,IAAI,OAAO,CAAC,WAAW,CAAC,CA6BvE;AAED;;GAEG;AACH,wBAAsB,sBAAsB,IAAI,OAAO,CAAC,WAAW,CAAC,CAyBnE"}

package/dist/src/commands/doctor-project-checks.js

@@ -2,7 +2,7 @@
  * Doctor — project/monomind health checks
  * Config, memory, API keys, MCP, monograph, helpers, routing, gates, gitignore, worker metrics
  */
-import { existsSync, readFileSync, readdirSync, statSync, mkdirSync, copyFileSync } from 'fs';
+import { existsSync, readFileSync, readdirSync, statSync, mkdirSync, copyFileSync, writeFileSync, appendFileSync } from 'fs';
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import { execSync } from 'child_process';
@@ -476,24 +476,97 @@ const REQUIRED_GITIGNORE_PATTERNS = [
     { pattern: '.hive-mind/', reason: 'hive-mind state with session info' },
     { pattern: '.swarm/', reason: 'swarm state files' },
 ];
+/** Strip the parts that don't change what a pattern matches, so `**​/.monomind/`,
+ *  `.monomind/` and `.monomind` all compare equal. A gitignore pattern with no
+ *  leading slash already matches at every depth, so `**​/` is redundant. */
+function normalizeGitignorePattern(p) {
+    return p.replace(/^\/+/, '').replace(/^\*\*\//, '').replace(/\/+$/, '');
+}
+/** True when ignoring `line` already protects everything `required` covers.
+ *  Beyond exact matches this understands directory containment: a repo with a
+ *  blanket `.monomind/` is fully covered for `.monomind/sessions/`,
+ *  `.monomind/*.json` and friends, and should not be nagged to add each one. */
+function gitignoreLineCovers(line, required) {
+    const l = normalizeGitignorePattern(line);
+    const r = normalizeGitignorePattern(required);
+    if (!l)
+        return false;
+    if (l === r)
+        return true;
+    // A bare directory entry covers every path beneath it, but only when the
+    // entry itself is literal — `.monomind/*.json` must not be treated as an
+    // ancestor of `.monomind/sessions`.
+    return !l.includes('*') && r.startsWith(`${l}/`);
+}
 export async function checkGitignoreCoverage() {
     const gitignorePath = join(process.cwd(), '.gitignore');
     if (!existsSync(gitignorePath)) {
-        return { name: 'Gitignore Coverage', status: 'warn', message: 'No .gitignore found — all monomind runtime paths are unprotected', fix: 'echo ".monomind/\\n**/.monomind/" >> .gitignore' };
+        // The old hint was `echo ".monomind/\n**/.monomind/" >> .gitignore`, which
+        // writes a literal backslash-n (sh/bash echo does not interpret escapes
+        // without -e) — one junk line matching nothing. `--fix` now writes the file
+        // directly, so point at that rather than at a command to copy-paste.
+        return { name: 'Gitignore Coverage', status: 'warn', message: 'No .gitignore found — all monomind runtime paths are unprotected', fix: 'monomind doctor --fix   (creates .gitignore covering .monomind/ and secrets)' };
     }
     if (statSync(gitignorePath).size > MAX_DOCTOR_GITIGNORE_BYTES) {
         return { name: 'Gitignore Coverage', status: 'warn', message: '.gitignore too large to parse' };
     }
     const lines = readFileSync(gitignorePath, 'utf-8').split('\n').map(l => l.trim()).filter(l => l && !l.startsWith('#'));
-    const missing = REQUIRED_GITIGNORE_PATTERNS.filter(({ pattern }) => {
-        const base = pattern.replace(/\*\*\//g, '').replace(/\*/g, '');
-        return !lines.some(l => l === pattern || l === pattern.replace(/\/$/, '') ||
-            (l.includes('**') && base && l.replace(/\*\*\//g, '').replace(/\*/g, '') === base));
-    });
+    const missing = REQUIRED_GITIGNORE_PATTERNS.filter(({ pattern }) => !lines.some(l => gitignoreLineCovers(l, pattern)));
     if (missing.length === 0)
         return { name: 'Gitignore Coverage', status: 'pass', message: 'All monomind runtime paths are gitignored' };
     const missingList = missing.map(m => m.pattern).join(', ');
-    return { name: 'Gitignore Coverage', status: 'warn', message: `${missing.length} runtime path(s) not in .gitignore: ${missingList}`, fix: `printf "${missing.map(m => m.pattern).join('\\n')}\\n" >> .gitignore` };
+    return { name: 'Gitignore Coverage', status: 'warn', message: `${missing.length} runtime path(s) not in .gitignore: ${missingList}`, fix: 'monomind doctor --fix   (appends the missing entries)' };
+}
+/** Patterns written by `doctor --fix` when there is no .gitignore at all.
+ *  Covers monomind runtime state plus the standard secrets files, since a repo
+ *  with no .gitignore has nothing protecting those either. */
+const GITIGNORE_FIX_PATTERNS = [
+    '# monomind runtime state',
+    '.monomind/',
+    '**/.monomind/',
+    '.hive-mind/',
+    '.swarm/',
+    '**/.claude-flow/',
+    'data/sessions/',
+    'data/mastermind-*.json',
+    'data/mastermind-*.jsonl',
+    '',
+    '# secrets / local env',
+    '.env',
+    '.env.*',
+    '!.env.example',
+    '*.pem',
+    '*.key',
+    '',
+    '# dependencies / build output',
+    'node_modules/',
+    'dist/',
+];
+/** Creates or appends the missing gitignore entries. Returns true if it wrote.
+ *  Append-only and idempotent: never rewrites lines the user already has. */
+export async function fixGitignoreCoverage() {
+    const gitignorePath = join(process.cwd(), '.gitignore');
+    try {
+        if (!existsSync(gitignorePath)) {
+            writeFileSync(gitignorePath, GITIGNORE_FIX_PATTERNS.join('\n') + '\n', 'utf-8');
+            return true;
+        }
+        if (statSync(gitignorePath).size > MAX_DOCTOR_GITIGNORE_BYTES)
+            return false;
+        const existing = readFileSync(gitignorePath, 'utf-8');
+        const lines = existing.split('\n').map(l => l.trim()).filter(l => l && !l.startsWith('#'));
+        const missing = REQUIRED_GITIGNORE_PATTERNS
+            .map(m => m.pattern)
+            .filter(pattern => !lines.some(l => gitignoreLineCovers(l, pattern)));
+        if (missing.length === 0)
+            return false;
+        const prefix = existing.endsWith('\n') || existing === '' ? '' : '\n';
+        appendFileSync(gitignorePath, `${prefix}\n# monomind runtime state\n${missing.join('\n')}\n`, 'utf-8');
+        return true;
+    }
+    catch {
+        return false;
+    }
 }
 export async function checkAgentRegistry() {
     try {