@claude-flow/cli 3.32.35 → 3.32.37

package/dist/src/init/statusline-generator.js

@@ -53,6 +53,16 @@ function getInstalledCliVersionLocal() {
         return '0.0.0';
     }
 }
+function compareVersionsLocal(a, b) {
+    const pa = a.split(/[.-]/).map(part => Number.parseInt(part, 10) || 0);
+    const pb = b.split(/[.-]/).map(part => Number.parseInt(part, 10) || 0);
+    for (let i = 0; i < Math.max(pa.length, pb.length); i++) {
+        const delta = (pa[i] ?? 0) - (pb[i] ?? 0);
+        if (delta !== 0)
+            return delta;
+    }
+    return 0;
+}
 /**
  * Generate optimized statusline script
  * Output format:
@@ -91,10 +101,12 @@ export function generateStatuslineScript(options) {
     // getInstalledCliVersionLocal() above — reuse the pattern for the
     // same reason it exists there.
     let helperContent = null;
+    let helperPackageRoot = '';
     try {
         const esmRequire = createRequire(import.meta.url);
         const pkgJsonPath = esmRequire.resolve('@claude-flow/cli/package.json');
-        const helperPath = path.join(path.dirname(pkgJsonPath), '.claude', 'helpers', 'statusline.cjs');
+        helperPackageRoot = path.dirname(pkgJsonPath);
+        const helperPath = path.join(helperPackageRoot, '.claude', 'helpers', 'statusline.cjs');
         helperContent = fs.readFileSync(helperPath, 'utf-8');
     }
     catch {
@@ -105,6 +117,7 @@ export function generateStatuslineScript(options) {
                 if (pkg && pkg.name === '@claude-flow/cli') {
                     const candidate = path.join(dir, '.claude', 'helpers', 'statusline.cjs');
                     if (fs.existsSync(candidate)) {
+                        helperPackageRoot = dir;
                         helperContent = fs.readFileSync(candidate, 'utf-8');
                         break;
                     }
@@ -128,15 +141,15 @@ export function generateStatuslineScript(options) {
     // whatever the helper hard-codes) ships. Add a paired test in
     // statusline-cost-display.test.ts before changing either token.
     helperContent = helperContent.replace(/maxAgents: \d+,/, `maxAgents: ${maxAgents},`);
+    helperContent = helperContent.replace(/const BAKED_INSTALL_ROOT = "[^"]*";/, `const BAKED_INSTALL_ROOT = ${JSON.stringify(helperPackageRoot)};`);
     // Only overwrite the helper's baked version if OURS resolves higher.
     // Otherwise the substitution could DOWNGRADE (test environments where
     // esmRequire.resolve happens to hit an older node_modules install would
-    // clobber a fresh committed helper). Naive lexicographic compare — works
-    // for canonical semver strings with same-width digit parts, which is
-    // reliable at this stage of the version space.
+    // clobber a fresh committed helper). Compare numeric components:
+    // lexicographic comparison freezes 3.32.24 below 3.32.8 (#2811).
     const helperVerMatch = helperContent.match(/let ver = "([^"]+)";/);
     const helperVer = helperVerMatch ? helperVerMatch[1] : '';
-    if (!helperVer || bakedVersion > helperVer) {
+    if (!helperVer || compareVersionsLocal(bakedVersion, helperVer) > 0) {
         helperContent = helperContent.replace(/let ver = "[^"]+";/, `let ver = ${JSON.stringify(bakedVersion)};`);
     }
     return helperContent;

package/dist/src/mcp-server.d.ts

@@ -48,6 +48,31 @@ export interface MCPServerStatus {
         metrics?: Record<string, number>;
     };
 }
+export declare function parseMcpToolSelection(value: string | undefined): string[] | 'all';
+/**
+ * Apply the existing `--tools` contract to advertised schemas. A selector can
+ * be an exact tool name, a category, or a namespace prefix (`memory` matches
+ * `memory_store`). Execution remains registered internally; only the fixed
+ * per-request schema catalogue is reduced.
+ */
+export declare function filterAdvertisedMcpTools<T extends {
+    name: string;
+    category?: string;
+}>(tools: T[], selection: string[] | 'all'): T[];
+export interface McpSchemaOverhead {
+    toolCount: number;
+    bytes: number;
+    estimatedTokens: number;
+    contextWindowTokens?: number;
+    ratio?: number;
+    risk: 'normal' | 'high';
+}
+/** Conservative JSON-size estimate for the fixed tools/list catalogue. */
+export declare function assessMcpSchemaOverhead(tools: Array<{
+    name: string;
+    description?: string;
+    inputSchema?: unknown;
+}>, contextWindowTokens?: number): McpSchemaOverhead;
 /**
  * MCP Server Manager
  *

package/dist/src/mcp-server.js

@@ -42,6 +42,54 @@ const DEFAULT_OPTIONS = {
     daemonize: false,
     timeout: 30000,
 };
+export function parseMcpToolSelection(value) {
+    if (!value || value.trim().toLowerCase() === 'all')
+        return 'all';
+    const selectors = value.split(',').map((item) => item.trim()).filter(Boolean);
+    return selectors.length > 0 ? selectors : 'all';
+}
+/**
+ * Apply the existing `--tools` contract to advertised schemas. A selector can
+ * be an exact tool name, a category, or a namespace prefix (`memory` matches
+ * `memory_store`). Execution remains registered internally; only the fixed
+ * per-request schema catalogue is reduced.
+ */
+export function filterAdvertisedMcpTools(tools, selection) {
+    if (selection === 'all')
+        return tools;
+    const selectors = new Set(selection.map((item) => item.toLowerCase()));
+    return tools.filter((tool) => {
+        const name = tool.name.toLowerCase();
+        const category = tool.category?.toLowerCase();
+        return selectors.has(name)
+            || (category !== undefined && selectors.has(category))
+            || Array.from(selectors).some((selector) => name.startsWith(`${selector}_`));
+    });
+}
+/** Conservative JSON-size estimate for the fixed tools/list catalogue. */
+export function assessMcpSchemaOverhead(tools, contextWindowTokens) {
+    const catalogue = tools.map((tool) => ({
+        name: tool.name,
+        description: tool.description,
+        inputSchema: tool.inputSchema,
+    }));
+    const bytes = Buffer.byteLength(JSON.stringify(catalogue), 'utf8');
+    const estimatedTokens = Math.ceil(bytes / 4);
+    const validWindow = Number.isFinite(contextWindowTokens) && Number(contextWindowTokens) > 0
+        ? Number(contextWindowTokens)
+        : undefined;
+    const ratio = validWindow ? estimatedTokens / validWindow : undefined;
+    return {
+        toolCount: tools.length,
+        bytes,
+        estimatedTokens,
+        ...(validWindow === undefined ? {} : { contextWindowTokens: validWindow }),
+        ...(ratio === undefined ? {} : { ratio }),
+        risk: (ratio !== undefined ? ratio >= 0.2 : estimatedTokens >= 8_000)
+            ? 'high'
+            : 'normal',
+    };
+}
 /**
  * MCP Server Manager
  *
@@ -55,7 +103,14 @@ export class MCPServerManager extends EventEmitter {
     healthCheckInterval;
     constructor(options = {}) {
         super();
-        this.options = { ...DEFAULT_OPTIONS, ...options };
+        // `options.tools`, populated by the `mcp start --tools` CLI flag, is
+        // spread last below and therefore takes precedence over this env fallback.
+        const environmentTools = parseMcpToolSelection(process.env.CLAUDE_FLOW_MCP_TOOLS);
+        this.options = {
+            ...DEFAULT_OPTIONS,
+            ...(environmentTools === 'all' ? {} : { tools: environmentTools }),
+            ...options,
+        };
     }
     /**
      * Start the MCP server
@@ -420,7 +475,7 @@ export class MCPServerManager extends EventEmitter {
                         },
                     };
                 case 'tools/list':
-                    const tools = listMCPTools();
+                    const tools = filterAdvertisedMcpTools(listMCPTools(), this.options.tools);
                     return {
                         jsonrpc: '2.0',
                         id: message.id,

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

@@ -50,13 +50,16 @@ async function getRealEmbeddingFunction() {
     }
     return realEmbeddingFn;
 }
-// Generate real ONNX embedding (falls back to deterministic hash if ONNX unavailable)
 async function generateRealEmbedding(text, dimension) {
     const realFn = await getRealEmbeddingFunction();
     if (realFn) {
         try {
             const result = await realFn(text);
-            return result.embedding;
+            return {
+                embedding: result.embedding,
+                backend: result.backend ?? 'onnx',
+                model: result.model,
+            };
         }
         catch {
             // Fall through to fallback
@@ -75,7 +78,11 @@ async function generateRealEmbedding(text, dimension) {
     }
     // L2 normalize
     const norm = Math.sqrt(embedding.reduce((sum, x) => sum + x * x, 0));
-    return embedding.map(x => x / norm);
+    return {
+        embedding: embedding.map(x => x / norm),
+        backend: 'mock',
+        model: 'hash-fallback',
+    };
 }
 // Convert Euclidean embedding to Poincaré ball
 function toPoincare(euclidean, curvature) {
@@ -238,7 +245,8 @@ export const embeddingsTools = [
             }
             const useHyperbolic = input.hyperbolic === true && config.hyperbolic.enabled;
             // Generate real ONNX embedding
-            const embedding = await generateRealEmbedding(text, config.dimension);
+            const generated = await generateRealEmbedding(text, config.dimension);
+            const embedding = generated.embedding;
             let result;
             let geometry;
             if (useHyperbolic) {
@@ -254,6 +262,8 @@ export const embeddingsTools = [
                 embedding: result,
                 metadata: {
                     model: config.model,
+                    embeddingBackend: generated.backend,
+                    semanticGrounded: generated.backend === 'onnx',
                     dimension: config.dimension,
                     geometry,
                     curvature: useHyperbolic ? config.hyperbolic.curvature : null,
@@ -309,10 +319,13 @@ export const embeddingsTools = [
                     return { success: false, error: v.error };
             }
             // Generate real ONNX embeddings for both texts
-            const [emb1, emb2] = await Promise.all([
+            const [generated1, generated2] = await Promise.all([
                 generateRealEmbedding(text1, config.dimension),
                 generateRealEmbedding(text2, config.dimension)
             ]);
+            const emb1 = generated1.embedding;
+            const emb2 = generated2.embedding;
+            const embeddingBackend = generated1.backend === 'onnx' && generated2.backend === 'onnx' ? 'onnx' : 'mock';
             let similarity;
             let distance;
             switch (metric) {
@@ -341,14 +354,21 @@ export const embeddingsTools = [
                 similarity,
                 distance,
                 metric,
+                embeddingBackend,
+                semanticGrounded: embeddingBackend === 'onnx',
                 texts: {
                     text1: { length: text1.length, preview: text1.slice(0, 50) },
                     text2: { length: text2.length, preview: text2.slice(0, 50) },
                 },
-                interpretation: similarity > 0.8 ? 'very similar' :
-                    similarity > 0.6 ? 'similar' :
-                        similarity > 0.4 ? 'somewhat similar' :
-                            similarity > 0.2 ? 'different' : 'very different',
+                interpretation: embeddingBackend === 'onnx'
+                    ? similarity > 0.8 ? 'very similar'
+                        : similarity > 0.6 ? 'similar'
+                            : similarity > 0.4 ? 'somewhat similar'
+                                : similarity > 0.2 ? 'different' : 'very different'
+                    : null,
+                warning: embeddingBackend === 'mock'
+                    ? 'Hash fallback scores are deterministic but not semantically meaningful.'
+                    : undefined,
             };
         },
     },
@@ -426,6 +446,8 @@ export const embeddingsTools = [
                     })),
                     metadata: {
                         model: config.model,
+                        embeddingBackend: queryEmbedding.backend,
+                        semanticGrounded: queryEmbedding.backend === 'onnx',
                         topK,
                         threshold,
                         namespace: namespace || 'all',
@@ -433,6 +455,9 @@ export const embeddingsTools = [
                         indexType: config.hyperbolic.enabled ? 'HNSW (hyperbolic)' : 'HNSW (euclidean)',
                         resultCount: searchResult.results.length
                     },
+                    warning: queryEmbedding.backend === 'mock'
+                        ? 'Results use a hash fallback and are not semantically ranked.'
+                        : undefined,
                 };
             }
             catch {
@@ -444,12 +469,17 @@ export const embeddingsTools = [
                     results: [],
                     metadata: {
                         model: config.model,
+                        embeddingBackend: queryEmbedding.backend,
+                        semanticGrounded: queryEmbedding.backend === 'onnx',
                         topK,
                         threshold,
                         namespace: namespace || 'all',
                         searchTime: `${searchTime}ms`,
                         indexType: config.hyperbolic.enabled ? 'HNSW (hyperbolic)' : 'HNSW (euclidean)',
                     },
+                    warning: queryEmbedding.backend === 'mock'
+                        ? 'Hash fallback is active; semantic search is unavailable.'
+                        : undefined,
                     message: 'No embeddings indexed yet. Use memory store to add documents.',
                 };
             }
@@ -795,6 +825,8 @@ export const embeddingsTools = [
             }
             catch { /* not installed */ }
             const ruvectorEnabled = config.neural.ruvector?.enabled ?? false;
+            const backendProbe = await generateRealEmbedding('ruflo embedding backend probe', config.dimension);
+            const semanticGrounded = backendProbe.backend === 'onnx';
             return {
                 success: true,
                 initialized: true,
@@ -822,11 +854,20 @@ export const embeddingsTools = [
                     models: config.modelPath,
                 },
                 initializedAt: config.initialized,
+                embeddingBackend: backendProbe.backend,
+                semanticGrounded,
+                warning: semanticGrounded
+                    ? undefined
+                    : 'Hash fallback is active. Similarity scores are deterministic but not semantically meaningful.',
                 capabilities: {
                     onnxModels: ['Xenova/all-MiniLM-L6-v2', 'Xenova/all-mpnet-base-v2'],
                     geometries: ['euclidean', 'poincare'],
                     normalizations: ['L2', 'L1', 'minmax', 'zscore'],
-                    features: ['semantic search', 'hyperbolic projection', 'neural substrate'],
+                    features: [
+                        ...(semanticGrounded ? ['semantic search'] : []),
+                        'hyperbolic projection',
+                        'neural substrate',
+                    ],
                 },
             };
         },

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

@@ -439,6 +439,20 @@ function getIntelligenceStatsFromMemory() {
         const category = e.metadata?.category || 'general';
         categories[category] = (categories[category] || 0) + 1;
     });
+    const successfulPatterns = patternEntries.filter(e => e.metadata?.success === true).length;
+    const failedPatterns = patternEntries.filter(e => e.metadata?.success === false).length;
+    const describedPatterns = patternEntries.filter(e => {
+        if (typeof e.metadata?.task === 'string' && e.metadata.task.trim())
+            return true;
+        try {
+            const value = typeof e.value === 'string' ? JSON.parse(e.value) : e.value;
+            return typeof value?.task === 'string' &&
+                Boolean(String(value.task).trim());
+        }
+        catch {
+            return false;
+        }
+    }).length;
     // Count routing decisions
     const routingEntries = entries.filter(e => e.key.includes('routing') ||
         e.metadata?.type === 'routing-decision');
@@ -472,6 +486,10 @@ function getIntelligenceStatsFromMemory() {
         },
         patterns: {
             learned: patternEntries.length,
+            successful: successfulPatterns,
+            failed: failedPatterns,
+            unknown: Math.max(0, patternEntries.length - successfulPatterns - failedPatterns),
+            described: describedPatterns,
             categories,
         },
         memory: {
@@ -1074,21 +1092,28 @@ export const hooksMetrics = {
                 agentCounts[o.agent] = (agentCounts[o.agent] || 0) + 1;
         }
         const topAgent = Object.entries(agentCounts).sort((a, b) => b[1] - a[1])[0]?.[0] ?? null;
-        const successful = stats.trajectories.successful;
-        const total = stats.trajectories.total;
-        const failed = Math.max(0, total - successful);
         return {
             _real: true,
             _dataSource: 'intelligence-stats + routing-outcomes',
             period,
             patterns: {
                 total: stats.patterns.learned,
-                successful,
-                failed,
+                successful: stats.patterns.successful,
+                failed: stats.patterns.failed,
+                unknown: stats.patterns.unknown,
+                described: stats.patterns.described,
+                descriptionCoverage: stats.patterns.learned > 0
+                    ? stats.patterns.described / stats.patterns.learned
+                    : null,
                 avgConfidence: stats.routing.avgConfidence || null,
             },
             agents: {
-                routingAccuracy: stats.routing.avgConfidence || null,
+                // Confidence is a model self-score, not observed routing accuracy
+                // (#2809). Keep the old key as an explicit null for compatibility
+                // while exposing the truthful additive field.
+                routingAccuracy: null,
+                averageConfidence: stats.routing.avgConfidence || null,
+                outcomeSuccessRate: successRate,
                 totalRoutes: stats.routing.decisions,
                 topAgent,
             },
@@ -1097,7 +1122,7 @@ export const hooksMetrics = {
                 successRate,
                 avgRiskScore: null,
             },
-            _note: total === 0 && totalCommands === 0
+            _note: stats.patterns.learned === 0 && totalCommands === 0
                 ? 'No metrics data collected yet. Run hooks_post-task / hooks_intelligence_trajectory-end / hooks_route to populate.'
                 : undefined,
             lastUpdated: new Date().toISOString(),
@@ -1311,6 +1336,7 @@ export const hooksPostTask = {
             const bridge = await import('../memory/memory-bridge.js');
             feedbackResult = await bridge.bridgeRecordFeedback({
                 taskId,
+                task: params.task || undefined,
                 success,
                 quality,
                 agent,
@@ -3136,7 +3162,7 @@ export const hooksIntelligenceStats = {
         catch {
             memoryStats = {
                 trajectories: { total: 0, successful: 0 },
-                patterns: { learned: 0, categories: {} },
+                patterns: { learned: 0, successful: 0, failed: 0, unknown: 0, described: 0, categories: {} },
                 memory: { indexSize: 0, totalAccessCount: 0, memorySizeBytes: 0 },
                 routing: { decisions: 0, avgConfidence: 0 },
             };

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

@@ -199,7 +199,7 @@ export const metaharnessTools = [
     },
     {
         name: 'metaharness_genome',
-        description: 'ADR-150 — 7-section categorical readiness report from `metaharness genome <path>` (repo_type / agent_topology / risk_score / mcp_surface / test_confidence / publish_readiness). Use when you need the categorical view (vs numeric score). Pair with metaharness_score for the full readiness picture — score-alone is wrong because two harnesses with the same harnessFit can have very different agent_topology and mcp_surface. ' + MCP_SUCCESS_SEMANTIC,
+        description: 'ADR-150 — 7-section categorical readiness report from `metaharness genome <path>` (repo_type / agent_topology / risk_score / mcp_surface / test_confidence / publish_readiness). Upstream needs-work/blocked exit statuses are valid verdicts and return successfully as data.verdict + data.verdictExitCode; only a missing or malformed report is an error. Use when you need the categorical view (vs numeric score). Pair with metaharness_score for the full readiness picture — score-alone is wrong because two harnesses with the same harnessFit can have very different agent_topology and mcp_surface. ' + MCP_SUCCESS_SEMANTIC,
         category: 'metaharness',
         inputSchema: {
             type: 'object',

package/dist/src/memory/memory-bridge.d.ts

@@ -275,6 +275,16 @@ export declare function getControllerRegistry(dbPath?: string): Promise<any | nu
  * operator learns the cause instead of only the symptom.
  */
 export declare function getBridgeFailureReason(): string | null;
+/**
+ * Install a pre-initialized registry for deterministic bridge tests.
+ *
+ * The CLI test runner intentionally externalizes the optional
+ * `@claude-flow/memory` package so an unbuilt workspace can still exercise
+ * fallback paths. That also makes module-level mocking of ControllerRegistry
+ * environment-dependent. This narrow seam keeps native SQL regression tests
+ * independent of package build order without changing production startup.
+ */
+export declare function __setMemoryBridgeRegistryForTests(registry: any | null): void;
 /**
  * Shutdown the bridge and release resources.
  *
@@ -322,6 +332,8 @@ export declare function bridgeSearchPatterns(options: {
  */
 export declare function bridgeRecordFeedback(options: {
     taskId: string;
+    /** Human-readable task text used as the semantic learning signal. */
+    task?: string;
     success: boolean;
     quality: number;
     agent?: string;

package/dist/src/memory/memory-bridge.js

@@ -23,6 +23,10 @@ import { createRequire } from 'node:module';
 let registryPromise = null;
 let registryInstance = null;
 let bridgeAvailable = null;
+// #2652/#2120: rows created before the status column existed receive NULL
+// during migration. They are live rows, not tombstones. Every user-facing
+// read/delete path must agree with list() about their visibility.
+const ACTIVE_MEMORY_ROW_SQL = `(status = 'active' OR status IS NULL)`;
 /**
  * Why the bridge is unavailable, when it is.
  *
@@ -994,7 +998,7 @@ export async function bridgeSearchEntries(options) {
             const stmt = ctx.db.prepare(`
         SELECT id, key, namespace, content, embedding, provenance_type
         FROM memory_entries
-        WHERE status = 'active' ${whereExtra}
+        WHERE ${ACTIVE_MEMORY_ROW_SQL} ${whereExtra}
         LIMIT 1000
       `);
             rows = filterParams.length > 0 ? stmt.all(...filterParams) : stmt.all();
@@ -1114,7 +1118,7 @@ export async function bridgeListEntries(options) {
         // the `status = 'active'` filter matched zero. Treat NULL as
         // "legacy-active" — the safe default for any entry that predates the
         // status column.
-        const statusFilter = `(status = 'active' OR status IS NULL)`;
+        const statusFilter = ACTIVE_MEMORY_ROW_SQL;
         // Count
         let total = 0;
         try {
@@ -1206,7 +1210,7 @@ export async function bridgeGetEntry(options) {
             const stmt = ctx.db.prepare(`
         SELECT id, key, namespace, content, embedding, access_count, created_at, updated_at, tags
         FROM memory_entries
-        WHERE status = 'active' AND key = ? AND namespace = ?
+        WHERE ${ACTIVE_MEMORY_ROW_SQL} AND key = ? AND namespace = ?
         LIMIT 1
       `);
             row = stmt.get(key, namespace);
@@ -1274,7 +1278,7 @@ export async function bridgeDeleteEntry(options) {
             const result = ctx.db.prepare(`
         UPDATE memory_entries
         SET status = 'deleted', updated_at = ?
-        WHERE key = ? AND namespace = ? AND status = 'active'
+        WHERE key = ? AND namespace = ? AND ${ACTIVE_MEMORY_ROW_SQL}
       `).run(Date.now(), key, namespace);
             changes = result?.changes ?? 0;
         }
@@ -1306,7 +1310,7 @@ export async function bridgeDeleteEntry(options) {
         }
         let remaining = 0;
         try {
-            const row = ctx.db.prepare(`SELECT COUNT(*) as cnt FROM memory_entries WHERE status = 'active'`).get();
+            const row = ctx.db.prepare(`SELECT COUNT(*) as cnt FROM memory_entries WHERE ${ACTIVE_MEMORY_ROW_SQL}`).get();
             remaining = row?.cnt ?? 0;
         }
         catch {
@@ -1659,6 +1663,21 @@ export async function getControllerRegistry(dbPath) {
 export function getBridgeFailureReason() {
     return bridgeFailureReason;
 }
+/**
+ * Install a pre-initialized registry for deterministic bridge tests.
+ *
+ * The CLI test runner intentionally externalizes the optional
+ * `@claude-flow/memory` package so an unbuilt workspace can still exercise
+ * fallback paths. That also makes module-level mocking of ControllerRegistry
+ * environment-dependent. This narrow seam keeps native SQL regression tests
+ * independent of package build order without changing production startup.
+ */
+export function __setMemoryBridgeRegistryForTests(registry) {
+    registryInstance = registry;
+    registryPromise = registry ? Promise.resolve(registry) : null;
+    bridgeAvailable = registry ? true : null;
+    bridgeFailureReason = null;
+}
 /**
  * Shutdown the bridge and release resources.
  *
@@ -1840,11 +1859,15 @@ export async function bridgeRecordFeedback(options) {
         try {
             const intelligence = await import('./intelligence.js');
             const verdict = options.success ? 'success' : 'failure';
+            const taskContext = options.task?.trim();
             const recorded = await intelligence.recordTrajectory([{
                     type: 'action',
-                    content: `Task ${options.taskId} completed by ${options.agent || 'unknown'} — success=${options.success}, quality=${options.quality.toFixed(3)}`,
+                    // Put the task description first so the embedding represents the
+                    // work, not just the bookkeeping suffix (#2812).
+                    content: `${taskContext ? `${taskContext} — ` : ''}Task ${options.taskId} completed by ${options.agent || 'unknown'} — success=${options.success}, quality=${options.quality.toFixed(3)}`,
                     metadata: {
                         taskId: options.taskId,
+                        task: taskContext,
                         agent: options.agent,
                         quality: options.quality,
                         duration: options.duration,

package/dist/src/memory/memory-initializer.js

@@ -153,6 +153,10 @@ export function resolveDbPath(cliFlag) {
     }
     return path.join(getMemoryRoot(), 'memory.db');
 }
+// #2652/#2120: legacy rows with NULL status predate soft-delete semantics and
+// are active. Keep fallback retrieve/delete aligned with list and the native
+// bridge instead of making a row visible to one command but not another.
+const ACTIVE_MEMORY_ROW_SQL = `(status = 'active' OR status IS NULL)`;
 // ADR-053: Lazy import of AgentDB v3 bridge
 let _bridge;
 async function getBridge() {
@@ -1608,8 +1612,7 @@ export async function repairVectorIndexes(dbPath, opts = {}) {
  */
 export async function initializeMemoryDatabase(options) {
     const { backend = 'hybrid', dbPath: customPath, force = false, verbose = false, migrate = true } = options;
-    const swarmDir = getMemoryRoot();
-    const dbPath = customPath || path.join(swarmDir, 'memory.db');
+    const dbPath = resolveDbPath(customPath);
     const dbDir = path.dirname(dbPath);
     try {
         // Create directory if needed
@@ -2857,7 +2860,7 @@ export async function listEntries(options) {
         // that predate the status column may have NULL after migration.
         // See memory-bridge.ts:bridgeListEntries for full context.
         // Get total count
-        const whereClauses = [`(status = 'active' OR status IS NULL)`];
+        const whereClauses = [ACTIVE_MEMORY_ROW_SQL];
         const whereParams = [];
         if (namespace) {
             whereClauses.push('namespace = ?');
@@ -2967,7 +2970,7 @@ export async function getEntry(options) {
         const getStmt = db.prepare(`
       SELECT id, key, namespace, content, embedding, access_count, created_at, updated_at, tags
       FROM memory_entries
-      WHERE status = 'active'
+      WHERE ${ACTIVE_MEMORY_ROW_SQL}
         AND key = ?
         AND namespace = ?
       LIMIT 1
@@ -3081,7 +3084,7 @@ export async function deleteEntry(options) {
         // Check if entry exists first
         const checkStmt = db.prepare(`
       SELECT id FROM memory_entries
-      WHERE status = 'active'
+      WHERE ${ACTIVE_MEMORY_ROW_SQL}
         AND key = ?
         AND namespace = ?
       LIMIT 1
@@ -3095,7 +3098,7 @@ export async function deleteEntry(options) {
         const checkResult = checkRows.length > 0 ? [{ values: checkRows }] : [];
         if (!checkResult[0]?.values?.[0]) {
             // Get remaining count before closing
-            const countResult = db.exec(`SELECT COUNT(*) FROM memory_entries WHERE status = 'active'`);
+            const countResult = db.exec(`SELECT COUNT(*) FROM memory_entries WHERE ${ACTIVE_MEMORY_ROW_SQL}`);
             const remainingEntries = countResult[0]?.values?.[0]?.[0] || 0;
             db.close();
             return {
@@ -3116,10 +3119,10 @@ export async function deleteEntry(options) {
           updated_at = strftime('%s', 'now') * 1000
       WHERE key = ?
         AND namespace = ?
-        AND status = 'active'
+        AND ${ACTIVE_MEMORY_ROW_SQL}
     `, [key, namespace]);
         // Get remaining count
-        const countResult = db.exec(`SELECT COUNT(*) FROM memory_entries WHERE status = 'active'`);
+        const countResult = db.exec(`SELECT COUNT(*) FROM memory_entries WHERE ${ACTIVE_MEMORY_ROW_SQL}`);
         const remainingEntries = countResult[0]?.values?.[0]?.[0] || 0;
         // Save updated database
         const data = db.export();

package/dist/src/services/memory-distillation.d.ts

@@ -20,6 +20,7 @@ export interface DistillOptions {
 export interface DistillReport {
     processed: number;
     episodes: number;
+    episodeEmbeddings: number;
     patterns: number;
     patternEmbeddings: number;
     causalEdges: number;