@monoes/monomindcli 1.10.27 → 1.10.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@monoes/monomindcli",
-  "version": "1.10.27",
+  "version": "1.10.29",
   "type": "module",
   "description": "Monomind CLI - Enterprise AI agent orchestration with 60+ specialized agents, swarm coordination, MCP server, self-learning hooks, and vector memory for Claude Code",
   "main": "dist/src/index.js",

package/scripts/understand-analyze.mjs

@@ -654,32 +654,65 @@ async function main() {
       mg.closeDb(db);
       process.exit(1);
     }
-    const updateNode = db.prepare(`UPDATE nodes SET properties = ? WHERE id = ?`);
+    // Resolve each analysis item to a DB integer node id.
+    // graph.json nodes have id='file:<path>' (synthetic); the real PK is an integer.
+    // Accept: analysis.dbId (integer), analysis.filePath, or derive path from analysis.id.
+    const lookupByPath = db.prepare(`SELECT id, properties FROM nodes WHERE file_path=? OR file_path=? LIMIT 1`);
+    const lookupById   = db.prepare(`SELECT id, properties FROM nodes WHERE id=? LIMIT 1`);
+    const updateNode   = db.prepare(`UPDATE nodes SET properties = ? WHERE id = ?`);
     let written = 0;
     const tx = db.transaction(() => {
       for (const analysis of analyses) {
-        if (!analysis || !analysis.id) continue;
-        const existing = (() => {
-          try { const row = db.prepare('SELECT properties FROM nodes WHERE id=?').get(analysis.id); return row?.properties ? JSON.parse(row.properties) : {}; } catch { return {}; }
-        })();
+        if (!analysis) continue;
+        let nodeRow = null;
+        // 1. Prefer explicit integer dbId (added by newer graph.json emitter)
+        if (analysis.dbId != null) {
+          nodeRow = lookupById.get(analysis.dbId);
+        }
+        // 2. Fall back to filePath from graph.json node
+        if (!nodeRow && analysis.filePath) {
+          const rel = analysis.filePath.startsWith('/') ? relative(projectDir, analysis.filePath) : analysis.filePath;
+          nodeRow = lookupByPath.get(analysis.filePath, rel);
+        }
+        // 3. Derive path from synthetic id ('file:<path>')
+        if (!nodeRow && analysis.id && String(analysis.id).startsWith('file:')) {
+          const derivedPath = String(analysis.id).slice(5);
+          const rel = derivedPath.startsWith('/') ? relative(projectDir, derivedPath) : derivedPath;
+          nodeRow = lookupByPath.get(derivedPath, rel);
+        }
+        if (!nodeRow) continue;
+        const existing = (() => { try { return nodeRow.properties ? JSON.parse(nodeRow.properties) : {}; } catch { return {}; } })();
         const merged = {
           ...existing,
-          ...(analysis.fileSummary   ? { summary:             analysis.fileSummary }            : {}),
-          ...(analysis.tags          ? { tags:                 analysis.tags }                   : {}),
-          ...(analysis.complexity    ? { complexity:           analysis.complexity }              : {}),
-          ...(analysis.functionSummaries ? { functionSummaries: analysis.functionSummaries }     : {}),
-          ...(analysis.classSummaries    ? { classSummaries:    analysis.classSummaries }        : {}),
+          ...(analysis.fileSummary       ? { summary:             analysis.fileSummary }            : {}),
+          ...(analysis.tags              ? { tags:                 analysis.tags }                   : {}),
+          ...(analysis.complexity        ? { complexity:           analysis.complexity }              : {}),
+          ...(analysis.functionSummaries ? { functionSummaries:    analysis.functionSummaries }     : {}),
+          ...(analysis.classSummaries    ? { classSummaries:       analysis.classSummaries }        : {}),
           ua_analyzed_at: new Date().toISOString(),
         };
-        updateNode.run(JSON.stringify(merged), analysis.id);
+        updateNode.run(JSON.stringify(merged), nodeRow.id);
         written++;
       }
     });
     tx();
     // Rebuild FTS so summaries are immediately searchable
     try { db.prepare(`INSERT INTO nodes_fts(nodes_fts) VALUES('rebuild')`).run(); } catch {}
-    mg.closeDb(db);
+
+    // Validate write-back: count nodes that now have a readable summary in properties
+    let enrichedCount = 0;
+    try {
+      const row = db.prepare(
+        `SELECT COUNT(*) AS c FROM nodes WHERE json_extract(properties, '$.summary') IS NOT NULL AND length(json_extract(properties, '$.summary')) > 5`
+      ).get();
+      enrichedCount = row ? row.c : 0;
+    } catch (_) {}
     console.log(`[understand] Imported ${written} analyses from stdin. FTS rebuilt.`);
+    console.log(`[understand] Validation: ${enrichedCount} node(s) now have LLM summaries readable via json_extract.`);
+    if (written > 0 && enrichedCount < written) {
+      console.warn(`[understand] WARNING: only ${enrichedCount} of ${written} imported analyses are queryable. Some properties may not have committed — check if properties column is TEXT.`);
+    }
+    mg.closeDb(db);
     return;
   }
 
@@ -1001,6 +1034,7 @@ function buildGraphJson(dir, fileNodes, analysisMap, layers) {
     const a = analysisMap[n.file_path] || {};
     return {
       id: 'file:' + (n.file_path || n.name),
+      dbId: n.id,          // integer PK — use this in --import-analyses-stdin payloads
       type: 'file',
       name: n.name,
       filePath: n.file_path,