persyst-mcp 2.2.3 → 2.2.5

package/src/tools.js

@@ -48,6 +48,7 @@ import { searchHybrid, getOptimizedContext, consolidateMemories } from './search
 import { getRecentCommits } from './git.js';
 import { verifyChainIntegrity } from './attestation.js';
 import { searchCache } from './cache.js';
+import { memoryEventBus } from './events.js';
 
 // ============================================================
 // CONSTANTS
@@ -152,6 +153,9 @@ export async function addMemoryInternal({ content, importance = 1.0, agent_id, s
     // Feature 1: Invalidate search cache on write
     searchCache.invalidate();
 
+    // Broadcast to SSE subscribers (HTTP gateway + SSE clients)
+    memoryEventBus.emit('memory_added', { id, content, namespace, source: normalizedAgentId || 'manual' });
+
     // Feature 2: Contradiction Detection
     let contradictions = [];
     try {
@@ -290,7 +294,7 @@ export function registerTools(server) {
     'Search memories using hybrid keyword + semantic search with cryptographic attestation. CRITICAL: Call this tool at the start of a session or task to retrieve relevant user preferences, coding guidelines, and past decisions.',
     {
       query: z.string().describe('What to search for'),
-      limit: z.number().default(5).describe('Max results (default: 5)'),
+      limit: z.number().int().min(1).default(5).describe('Max results (default: 5)'),
       agent_id: z.string().optional().describe('Agent ID — filters results to this agent\'s namespace + shared'),
       session_id: z.string().optional().describe('Session ID')
     },
@@ -403,6 +407,9 @@ export function registerTools(server) {
         // Feature 1: Invalidate search cache on write
         searchCache.invalidate();
 
+        // Broadcast deletion to SSE subscribers
+        memoryEventBus.emit('memory_deleted', { id });
+
         return text({ success: true, id, message: `Memory #${id} deleted` });
       } catch (err) {
         return text({ error: err.message });
@@ -415,7 +422,7 @@ export function registerTools(server) {
     'get_recent_memories',
     'Get the most recently created memories, newest first. Filtered by agent namespace if agent_id is provided.',
     {
-      limit: z.number().default(10).describe('How many to return (default: 10)'),
+      limit: z.number().int().min(1).default(10).describe('How many to return (default: 10)'),
       agent_id: z.string().optional().describe('Agent ID — filters to this agent\'s namespace + shared')
     },
     async ({ limit, agent_id }) => {
@@ -434,7 +441,7 @@ export function registerTools(server) {
     'get_important_memories',
     'Get memories ranked by importance score, highest first. Filtered by agent namespace if agent_id is provided.',
     {
-      limit: z.number().default(10).describe('How many to return (default: 10)'),
+      limit: z.number().int().min(1).default(10).describe('How many to return (default: 10)'),
       agent_id: z.string().optional().describe('Agent ID — filters to this agent\'s namespace + shared')
     },
     async ({ limit, agent_id }) => {
@@ -778,12 +785,13 @@ export function registerTools(server) {
       query: z.string().describe('The search query context'),
       max_tokens: z.number().default(4000).describe('Token budget for LLM context compression (default: 4000)'),
       agent_id: z.string().optional().describe('Agent ID requesting context — filters to this agent\'s namespace + shared'),
-      session_id: z.string().optional().describe('Session ID')
+      session_id: z.string().optional().describe('Session ID'),
+      intent: z.string().optional().describe('The active task intent / category (e.g. debugging, ui_styling, database_management)')
     },
-    async ({ query, max_tokens, agent_id, session_id }) => {
+    async ({ query, max_tokens, agent_id, session_id, intent }) => {
       try {
         const namespace = agent_id || null;
-        const contextData = await getOptimizedContext(query, max_tokens, agent_id, session_id, namespace);
+        const contextData = await getOptimizedContext(query, max_tokens, agent_id, session_id, namespace, intent);
         return text(contextData);
       } catch (err) {
         return text({ error: err.message });

package/src/watcher.js

@@ -8,7 +8,7 @@
 
 import { join, resolve } from 'path';
 import { homedir } from 'os';
-import { existsSync, readFileSync, writeFileSync, readdirSync, statSync } from 'fs';
+import { existsSync, readFileSync, writeFileSync, readdirSync, statSync, openSync, readSync, closeSync } from 'fs';
 import {
   getWatchPosition,
   upsertWatchPosition,
@@ -20,6 +20,7 @@ import { generateEmbedding } from './embeddings.js';
 import { extractHeuristic } from './extractor-heuristic.js';
 import { searchHybrid } from './search.js';
 import { searchCache } from './cache.js';
+import { memoryEventBus } from './events.js';
 
 // Config path: ~/.persyst/config.json
 const CONFIG_FILE = join(homedir(), '.persyst', 'config.json');
@@ -86,10 +87,19 @@ async function processJsonlFile(filePath) {
 
     if (stat.size <= lastPos) return;
 
-    // Read only new content appended to the file
-    const fileBuffer = readFileSync(filePath);
-    const newContentBuffer = fileBuffer.subarray(lastPos, stat.size);
-    const text = newContentBuffer.toString('utf8');
+    // Read only new content appended to the file (Bug C fix)
+    const length = stat.size - lastPos;
+    let text = '';
+    if (length > 0) {
+      const newContentBuffer = Buffer.alloc(length);
+      const fd = openSync(filePath, 'r');
+      try {
+        readSync(fd, newContentBuffer, 0, length, lastPos);
+      } finally {
+        closeSync(fd);
+      }
+      text = newContentBuffer.toString('utf8');
+    }
 
     const lines = text.split('\n');
     let addedCount = 0;
@@ -116,16 +126,16 @@ async function processJsonlFile(filePath) {
 
         const facts = extractHeuristic(cleanText);
         for (const fact of facts) {
-          // Verify against exact duplicate
-          if (memoryExists(fact.content)) continue;
+          // Verify against exact duplicate (Bug A fix: check namespace 'shared')
+          if (memoryExists(fact.content, 'shared')) continue;
 
-          // Verify against semantic similarity
-          const similar = await searchHybrid(fact.content, 1);
+          // Verify against semantic similarity (Bug B fix: check namespace 'shared')
+          const similar = await searchHybrid(fact.content, 1, null, null, 'shared');
           if (similar.length > 0 && parseFloat(similar[0].similarity) >= DEDUP_THRESHOLD) {
             continue;
           }
 
-          // Insert memory with provenance
+          // Insert memory with provenance (written to 'shared' by default)
           const id = insertMemory(fact.content, fact.confidence, {
             source_type: 'agent',
             source_id: record.source === 'MODEL' ? 'antigravity-worker' : 'user-dialogue',
@@ -136,6 +146,7 @@ async function processJsonlFile(filePath) {
           insertVector(id, embedding);
           addedCount++;
           console.error(`[persyst-watcher] Auto-extracted fact: "${fact.content}" (Memory #${id})`);
+          memoryEventBus.emit('memory_added', { id, content: fact.content, namespace: 'shared', source: 'watcher-antigravity' });
         }
       }
     }
@@ -181,13 +192,16 @@ async function processJsonFile(filePath) {
       if (msg.role === 'user' || msg.role === 'assistant') {
         const facts = extractHeuristic(msg.content);
         for (const fact of facts) {
-          if (memoryExists(fact.content)) continue;
+          // Verify against exact duplicate (Bug A fix: check namespace 'shared')
+          if (memoryExists(fact.content, 'shared')) continue;
 
-          const similar = await searchHybrid(fact.content, 1);
+          // Verify against semantic similarity (Bug B fix: check namespace 'shared')
+          const similar = await searchHybrid(fact.content, 1, null, null, 'shared');
           if (similar.length > 0 && parseFloat(similar[0].similarity) >= DEDUP_THRESHOLD) {
             continue;
           }
 
+          // Insert memory with provenance (written to 'shared' by default)
           const id = insertMemory(fact.content, fact.confidence, {
             source_type: 'agent',
             source_id: msg.role === 'assistant' ? 'roo-worker' : 'user-dialogue',
@@ -198,6 +212,7 @@ async function processJsonFile(filePath) {
           insertVector(id, embedding);
           addedCount++;
           console.error(`[persyst-watcher] Auto-extracted fact: "${fact.content}" (Memory #${id})`);
+          memoryEventBus.emit('memory_added', { id, content: fact.content, namespace: 'shared', source: 'watcher-roo' });
         }
       }
     }