@claude-flow/cli 3.10.13 → 3.10.15

package/dist/src/mcp-tools/hooks-tools.d.ts

@@ -52,6 +52,13 @@ export declare const hooksCodemod: MCPTool;
 export declare const hooksWorkerCancel: MCPTool;
 export declare const hooksTeammateIdle: MCPTool;
 export declare const hooksTaskCompleted: MCPTool;
+/**
+ * Unified learning-stats aggregator MCP tool (#2245 → ADR-075).
+ *
+ * One honest call across the four historical stat sources — every sub-view
+ * names its store and a `consistency` block flags relationships that drift.
+ */
+export declare const hooksIntelligenceUnifiedStats: MCPTool;
 export declare const hooksTools: MCPTool[];
 export default hooksTools;
 //# sourceMappingURL=hooks-tools.d.ts.map

package/dist/src/mcp-tools/hooks-tools.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"hooks-tools.d.ts","sourceRoot":"","sources":["../../../src/mcp-tools/hooks-tools.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAKH,OAAO,EAAE,KAAK,OAAO,EAAiB,MAAM,YAAY,CAAC;AAiBzD;;;;;;;;GAQG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CASzD;AAwqBD,eAAO,MAAM,YAAY,EAAE,OAwC1B,CAAC;AAEF,eAAO,MAAM,aAAa,EAAE,OA+C3B,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,OA6C7B,CAAC;AAEF,eAAO,MAAM,gBAAgB,EAAE,OAuD9B,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,OAgLxB,CAAC;AAEF,eAAO,MAAM,YAAY,EAAE,OAoE1B,CAAC;AAEF,eAAO,MAAM,SAAS,EAAE,OA8CvB,CAAC;AAEF,eAAO,MAAM,YAAY,EAAE,OA2F1B,CAAC;AAEF,eAAO,MAAM,aAAa,EAAE,OAuK3B,CAAC;AAGF,eAAO,MAAM,YAAY,EAAE,OA4E1B,CAAC;AAGF,eAAO,MAAM,aAAa,EAAE,OAiJ3B,CAAC;AAGF,eAAO,MAAM,gBAAgB,EAAE,OAoE9B,CAAC;AAGF,eAAO,MAAM,aAAa,EAAE,OA+E3B,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAuI/B,CAAC;AAGF,eAAO,MAAM,eAAe,EAAE,OAuF7B,CAAC;AAGF,eAAO,MAAM,mBAAmB,EAAE,OAyCjC,CAAC;AAGF,eAAO,MAAM,WAAW,EAAE,OA+BzB,CAAC;AAGF,eAAO,MAAM,SAAS,EAAE,OAsCvB,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAkI/B,CAAC;AAGF,eAAO,MAAM,sBAAsB,EAAE,OAmEpC,CAAC;AAGF,eAAO,MAAM,oBAAoB,EAAE,OAyDlC,CAAC;AAEF,eAAO,MAAM,mBAAmB,EAAE,OAmEjC,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,OAuKhC,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OA4E/B,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,OAwGhC,CAAC;AAGF,eAAO,MAAM,sBAAsB,EAAE,OAgNpC,CAAC;AAGF,eAAO,MAAM,sBAAsB,EAAE,OAiGpC,CAAC;AAGF,eAAO,MAAM,0BAA0B,EAAE,OA8MxC,CAAC;AAgQF,eAAO,MAAM,eAAe,EAAE,OA8C7B,CAAC;AAGF,eAAO,MAAM,mBAAmB,EAAE,OAwIjC,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAuD/B,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAkE/B,CAAC;AAiBF,eAAO,MAAM,eAAe,EAAE,OA4C7B,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAgC/B,CAAC;AAGF,eAAO,MAAM,eAAe,EAAE,OAuB7B,CAAC;AAeF,eAAO,MAAM,YAAY,EAAE,OAmI1B,CAAC;AAqBF,eAAO,MAAM,iBAAiB,EAAE,OA0C/B,CAAC;AAOF,eAAO,MAAM,iBAAiB,EAAE,OAwB/B,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,OA4BhC,CAAC;AAGF,eAAO,MAAM,UAAU,EAAE,OAAO,EA4C/B,CAAC;AAEF,eAAe,UAAU,CAAC"}
+{"version":3,"file":"hooks-tools.d.ts","sourceRoot":"","sources":["../../../src/mcp-tools/hooks-tools.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAKH,OAAO,EAAE,KAAK,OAAO,EAAiB,MAAM,YAAY,CAAC;AAiBzD;;;;;;;;GAQG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CASzD;AAwqBD,eAAO,MAAM,YAAY,EAAE,OAwC1B,CAAC;AAEF,eAAO,MAAM,aAAa,EAAE,OA+C3B,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,OA6C7B,CAAC;AAEF,eAAO,MAAM,gBAAgB,EAAE,OAuD9B,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,OAgLxB,CAAC;AAEF,eAAO,MAAM,YAAY,EAAE,OAoE1B,CAAC;AAEF,eAAO,MAAM,SAAS,EAAE,OA8CvB,CAAC;AAEF,eAAO,MAAM,YAAY,EAAE,OA2F1B,CAAC;AAEF,eAAO,MAAM,aAAa,EAAE,OAuK3B,CAAC;AAGF,eAAO,MAAM,YAAY,EAAE,OA4E1B,CAAC;AAGF,eAAO,MAAM,aAAa,EAAE,OA4K3B,CAAC;AAGF,eAAO,MAAM,gBAAgB,EAAE,OAoE9B,CAAC;AAGF,eAAO,MAAM,aAAa,EAAE,OA+E3B,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAuI/B,CAAC;AAGF,eAAO,MAAM,eAAe,EAAE,OAuF7B,CAAC;AAGF,eAAO,MAAM,mBAAmB,EAAE,OAyCjC,CAAC;AAGF,eAAO,MAAM,WAAW,EAAE,OA+BzB,CAAC;AAGF,eAAO,MAAM,SAAS,EAAE,OAsCvB,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAkI/B,CAAC;AAGF,eAAO,MAAM,sBAAsB,EAAE,OAmEpC,CAAC;AAGF,eAAO,MAAM,oBAAoB,EAAE,OAyDlC,CAAC;AAEF,eAAO,MAAM,mBAAmB,EAAE,OAmEjC,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,OAuKhC,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OA4E/B,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,OAwGhC,CAAC;AAGF,eAAO,MAAM,sBAAsB,EAAE,OAgNpC,CAAC;AAGF,eAAO,MAAM,sBAAsB,EAAE,OAiGpC,CAAC;AAGF,eAAO,MAAM,0BAA0B,EAAE,OA8MxC,CAAC;AAgQF,eAAO,MAAM,eAAe,EAAE,OA8C7B,CAAC;AAGF,eAAO,MAAM,mBAAmB,EAAE,OAwIjC,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAuD/B,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAkE/B,CAAC;AAiBF,eAAO,MAAM,eAAe,EAAE,OA4C7B,CAAC;AAGF,eAAO,MAAM,iBAAiB,EAAE,OAgC/B,CAAC;AAGF,eAAO,MAAM,eAAe,EAAE,OAuB7B,CAAC;AAeF,eAAO,MAAM,YAAY,EAAE,OAmI1B,CAAC;AAqBF,eAAO,MAAM,iBAAiB,EAAE,OA0C/B,CAAC;AAOF,eAAO,MAAM,iBAAiB,EAAE,OAwB/B,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,OAoFhC,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,6BAA6B,EAAE,OAc3C,CAAC;AAGF,eAAO,MAAM,UAAU,EAAE,OAAO,EA6C/B,CAAC;AAEF,eAAe,UAAU,CAAC"}

package/dist/src/mcp-tools/hooks-tools.js

@@ -1531,8 +1531,13 @@ export const hooksPretrain = {
         };
         scan(repoPath, 0);
         const elapsed = Math.round(performance.now() - startTime);
-        // Store extracted patterns in AgentDB
-        let patternsStored = 0;
+        // Persist extracted patterns. Two stores get written so the user can find
+        // them where they expect:
+        //   1. memory-bridge `pretrain` namespace — one summary bundle
+        //   2. neural store — one row PER pattern so `neural_patterns list` reflects them
+        // #2245 — without (2), the dashboards reported "0 patterns" after pretrain.
+        let patternsBundled = 0;
+        let patternsIndexed = 0;
         try {
             const bridge = await import('../memory/memory-bridge.js');
             await bridge.bridgeStoreEntry({
@@ -1541,9 +1546,23 @@ export const hooksPretrain = {
                 namespace: 'pretrain',
                 tags: ['pretrain', depth],
             });
-            patternsStored = patterns.length;
+            patternsBundled = patterns.length;
         }
         catch { /* AgentDB not available */ }
+        try {
+            const neural = await import('./neural-tools.js');
+            const items = patterns.map((p) => ({
+                name: p.length > 200 ? p.slice(0, 200) : p,
+                type: 'import-pattern',
+                content: p,
+                metadata: { source: 'hooks_pretrain', depth },
+            }));
+            const result = await neural.storeNeuralPatterns(items);
+            patternsIndexed = result.stored;
+        }
+        catch { /* neural store unavailable */ }
+        // Back-compat field
+        const patternsStored = patternsBundled;
         // #1847: when the corpus contains files but no patterns were extracted
         // (typical for Markdown vaults), make the source-code-only extraction
         // contract explicit so users don't conclude the hook system is broken.
@@ -1565,13 +1584,20 @@ export const hooksPretrain = {
                 filesAnalyzed,
                 totalLines,
                 patternsExtracted: patterns.length,
-                patternsStored,
+                patternsBundled, // #2245: 1 summary row in memory-bridge `pretrain` namespace
+                patternsIndexed, // #2245: per-pattern rows in neural store — surfaced by neural_patterns list
+                patternsStored, // back-compat alias for patternsBundled
                 fileTypes: Object.entries(extCounts).sort((a, b) => b[1] - a[1]).slice(0, 15).map(([ext, count]) => ({ ext, count })),
                 // #1847: explicit extraction contract so callers can tell pretrain
                 // patterns apart from live trajectories and hook statusline state.
+                // #2245: also call out exactly which stores got written.
                 sources: {
                     extractedFrom: SUPPORTED_EXTRACTION_EXTS,
                     scope: 'pretrain-only (live trajectories + statusline are tracked separately)',
+                    stores: {
+                        'memory-bridge:pretrain': patternsBundled > 0 ? 1 : 0, // one bundle row
+                        'neural-store (neural_patterns list)': patternsIndexed,
+                    },
                 },
             },
             ...(note ? { note } : {}),
@@ -4150,7 +4176,7 @@ export const hooksTeammateIdle = {
 };
 export const hooksTaskCompleted = {
     name: 'hooks_task-completed',
-    description: 'Agent Teams hook — fired when a task is marked complete; records completion and (eventually) trains patterns + notifies the team lead. Use when native TodoWrite is wrong because the work was a persisted, agent-assigned task whose outcome should feed cross-session learning and team coordination. For an in-session checklist tick, native TodoWrite is fine. (Pattern-learning is delegated to the intelligence pipeline — this records the completion today.)',
+    description: 'Agent Teams hook — fired when a task is marked complete. Records the completion and, when `trainPatterns:true`, feeds the outcome to the SONA + EWC++ learning pipeline (the same path used by hooks_intelligence trajectory-*). Multiple ways to drive learning exist: (a) call this with trainPatterns:true for a one-step trajectory, (b) use hooks_intelligence trajectory-start/step/end for richer multi-step learning, (c) just record an episode via memory_store if no learning is needed. Each path is honest about what it persists; check the returned `learningPath` field.',
     category: 'hooks',
     inputSchema: {
         type: 'object',
@@ -4159,26 +4185,98 @@ export const hooksTaskCompleted = {
             teammateId: { type: 'string', description: 'Teammate that completed it' },
             success: { type: 'boolean', description: 'Whether the task succeeded' },
             quality: { type: 'number', description: 'Quality score 0-1' },
-            trainPatterns: { type: 'boolean', description: 'Feed the outcome to the learning pipeline' },
+            trainPatterns: { type: 'boolean', description: 'When true, runs the SONA + EWC++ trajectory pipeline on this completion so globalStats.patternsLearned reflects it. When false (default), only records the completion.' },
             notifyLead: { type: 'boolean', description: 'Notify the team lead' },
+            content: { type: 'string', description: 'Optional richer task description; used as the trajectory step content when training. Defaults to the taskId.' },
         },
         required: ['taskId'],
     },
     handler: async (input) => {
         const taskId = String(input.taskId ?? '');
-        const quality = typeof input.quality === 'number' ? input.quality : (input.success === false ? 0 : 1);
+        const success = input.success !== false;
+        const quality = typeof input.quality === 'number' ? input.quality : (success ? 1 : 0);
+        const trainPatterns = input.trainPatterns === true;
+        const teammateId = input.teammateId ? String(input.teammateId) : undefined;
+        // #2241 (OWASP ASI06 Memory/Context Poisoning) — task content is user-
+        // supplied and feeds the SONA learning model. Cap length, strip control
+        // chars, and reject obvious prompt-injection sentinels before training.
+        const rawContent = typeof input.content === 'string' && input.content.trim()
+            ? String(input.content)
+            : `Task ${taskId} completed (quality=${quality.toFixed(2)})`;
+        const content = rawContent
+            // Strip ASCII control chars except newline/tab.
+            .replace(/[\x00-\x08\x0B-\x1F\x7F]/g, '')
+            // Cap to 4 KB — way over a typical trajectory step, well under a memory bomb.
+            .slice(0, 4096);
+        let patternsLearned = 0;
+        let trajectoriesRecorded = 0;
+        let learningPath = 'recorded-only';
+        let learningError;
+        if (trainPatterns) {
+            // #2245 — actually feed the learning loop. Synthesize a one-step
+            // trajectory from {taskId, success, quality} and run it through the
+            // same SONA + EWC + globalStats++ path as hooks_intelligence trajectory-end.
+            try {
+                const intel = await import('../memory/intelligence.js');
+                const before = intel.getIntelligenceStats();
+                await intel.recordTrajectory([{
+                        type: 'result',
+                        content,
+                        metadata: { taskId, success, quality, teammateId },
+                        timestamp: Date.now(),
+                    }], success ? 'success' : 'failure');
+                const after = intel.getIntelligenceStats();
+                patternsLearned = Math.max(0, after.patternsLearned - before.patternsLearned);
+                trajectoriesRecorded = Math.max(0, after.trajectoriesRecorded - before.trajectoriesRecorded);
+                learningPath = 'trajectory-pipeline';
+            }
+            catch (err) {
+                learningError = err.message;
+                // Fall back to recorded-only — be honest about it.
+            }
+        }
+        const note = trainPatterns
+            ? (learningPath === 'trajectory-pipeline'
+                ? `Trained via SONA + EWC++ trajectory pipeline (verdict=${success ? 'success' : 'failure'}, patternsLearned=${patternsLearned}, trajectoriesRecorded=${trajectoriesRecorded}).`
+                : `trainPatterns=true but the trajectory pipeline failed (${learningError ?? 'unknown error'}). Completion recorded only.`)
+            : 'Completion recorded only. Pass trainPatterns:true (or use hooks_intelligence trajectory-* directly) to feed the learning loop.';
         return {
             success: true,
             taskId,
-            patternsLearned: 0,
+            patternsLearned,
+            trajectoriesRecorded,
+            learningPath, // 'trajectory-pipeline' | 'recorded-only'
             leadNotified: input.notifyLead === true,
-            metrics: { duration: 0, quality, learningUpdates: 0 },
-            note: 'completion recorded; pattern-learning is delegated to the intelligence pipeline (#1916 follow-up)',
+            metrics: { duration: 0, quality, learningUpdates: patternsLearned },
+            ...(learningError ? { learningError } : {}),
+            note,
         };
     },
 };
+/**
+ * Unified learning-stats aggregator MCP tool (#2245 → ADR-075).
+ *
+ * One honest call across the four historical stat sources — every sub-view
+ * names its store and a `consistency` block flags relationships that drift.
+ */
+export const hooksIntelligenceUnifiedStats = {
+    name: 'hooks_intelligence_unified-stats',
+    description: 'One honest view across the four learning stat sources: globalStats (`.claude-flow/neural/stats.json`), the in-memory SONA coordinator, memory-bridge AgentDB entries, and the neural-patterns store. Each sub-view names its source path. The `consistency` block notes cross-store drift (e.g. globalStats reports N patterns but neural_patterns is empty). Use this when one dashboard call should show "did learning happen" coherently — vs the four original aggregators which each return only their narrow slice. See ADR-075.',
+    category: 'hooks',
+    inputSchema: {
+        type: 'object',
+        properties: {
+            verbose: { type: 'boolean', description: 'Include extended breakdowns', default: true },
+        },
+    },
+    handler: async (_input) => {
+        const intel = await import('../memory/intelligence.js');
+        return intel.getUnifiedLearningStats();
+    },
+};
 // Export all hooks tools
 export const hooksTools = [
+    hooksIntelligenceUnifiedStats,
     hooksTeammateIdle,
     hooksTaskCompleted,
     hooksPreEdit,