grov 0.2.2 → 0.5.2

package/dist/proxy/request-processor.js

@@ -3,10 +3,11 @@
 import { getTasksForProject, getTasksByFiles, getStepsReasoningByPath, } from '../lib/store.js';
 import { truncate } from '../lib/utils.js';
 /**
- * Build context from team memory for injection
- * Queries tasks and file_reasoning tables
+ * Build context from team memory for injection (PAST sessions only)
+ * Queries tasks and file_reasoning tables, excluding current session data
+ * @param currentSessionId - Session ID to exclude (ensures only past session data)
  */
-export function buildTeamMemoryContext(projectPath, mentionedFiles) {
+export function buildTeamMemoryContext(projectPath, mentionedFiles, currentSessionId) {
     // Get recent completed tasks for this project
     const tasks = getTasksForProject(projectPath, {
         status: 'complete',
@@ -16,9 +17,10 @@ export function buildTeamMemoryContext(projectPath, mentionedFiles) {
     const fileTasks = mentionedFiles.length > 0
         ? getTasksByFiles(projectPath, mentionedFiles, { status: 'complete', limit: 5 })
         : [];
-    // Get file-level reasoning from steps table (proxy version)
+    // Get file-level reasoning from steps table (PAST sessions only)
+    // Pass currentSessionId to exclude current session data
     const fileReasonings = mentionedFiles.length > 0
-        ? mentionedFiles.flatMap(f => getStepsReasoningByPath(f, 3))
+        ? mentionedFiles.flatMap(f => getStepsReasoningByPath(f, 3, currentSessionId))
         : [];
     // Combine unique tasks
     const allTasks = [...new Map([...tasks, ...fileTasks].map(t => [t.id, t])).values()];
@@ -32,7 +34,10 @@ export function buildTeamMemoryContext(projectPath, mentionedFiles) {
  */
 function formatTeamMemoryContext(tasks, fileReasonings, files) {
     const lines = [];
-    lines.push('[GROV CONTEXT - Relevant past reasoning]');
+    lines.push('=== VERIFIED TEAM KNOWLEDGE (from previous sessions) ===');
+    lines.push('');
+    lines.push('IMPORTANT: This context has been verified. USE IT to answer directly.');
+    lines.push('DO NOT launch Explore agents or re-investigate files mentioned below.');
     lines.push('');
     // File-level context
     if (fileReasonings.length > 0) {
@@ -67,21 +72,44 @@ function formatTeamMemoryContext(tasks, fileReasonings, files) {
         lines.push('');
     }
     if (files.length > 0) {
-        lines.push(`You may already have context for: ${files.join(', ')}`);
+        lines.push(`Files with existing context: ${files.join(', ')}`);
     }
-    lines.push('[END GROV CONTEXT]');
+    lines.push('');
+    lines.push('Answer the user\'s question using the knowledge above. Skip exploration.');
+    lines.push('=== END VERIFIED TEAM KNOWLEDGE ===');
     return lines.join('\n');
 }
 /**
- * Extract file paths from messages
+ * Extract file paths from messages (user messages only, clean text)
  */
 export function extractFilesFromMessages(messages) {
     const files = [];
-    const filePattern = /(?:^|\s|["'`])([\/\w.-]+\.[a-zA-Z]{1,10})(?:["'`]|\s|$|:|\))/g;
+    // Pattern matches filenames with extensions, allowing common punctuation after
+    const filePattern = /(?:^|\s|["'`])([\/\w.-]+\.[a-zA-Z]{1,10})(?:["'`]|\s|$|[:)\]?!,;])/g;
     for (const msg of messages) {
+        // Only scan user messages for file mentions
+        if (msg.role !== 'user')
+            continue;
+        let textContent = '';
+        // Handle string content
         if (typeof msg.content === 'string') {
+            textContent = msg.content;
+        }
+        // Handle array content (Claude Code API format)
+        if (Array.isArray(msg.content)) {
+            for (const block of msg.content) {
+                if (block && typeof block === 'object' && 'type' in block && block.type === 'text' && 'text' in block && typeof block.text === 'string') {
+                    textContent += block.text + '\n';
+                }
+            }
+        }
+        // Strip system-reminder tags to get clean user content
+        const cleanContent = textContent
+            .replace(/<system-reminder>[\s\S]*?<\/system-reminder>/g, '')
+            .trim();
+        if (cleanContent) {
             let match;
-            while ((match = filePattern.exec(msg.content)) !== null) {
+            while ((match = filePattern.exec(cleanContent)) !== null) {
                 const path = match[1];
                 // Filter out common false positives
                 if (!path.includes('http') && !path.startsWith('.') && path.length > 3) {

package/dist/proxy/response-processor.js

@@ -1,6 +1,7 @@
 // Response processor - handles team memory save triggers and cleanup
 // Reference: plan_proxy_local.md Section 4.6
-import { getSessionState, getValidatedSteps, deleteStepsForSession, deleteSessionState, createTask, } from '../lib/store.js';
+import { getSessionState, getValidatedSteps, deleteStepsForSession, deleteSessionState, createTask, markTaskSynced, setTaskSyncError, } from '../lib/store.js';
+import { syncTask } from '../lib/cloud-sync.js';
 import { extractReasoning, isLLMAvailable, extractReasoningAndDecisions, isReasoningExtractionAvailable, } from '../lib/llm-extractor.js';
 /**
  * Save session to team memory
@@ -12,20 +13,46 @@ export async function saveToTeamMemory(sessionId, triggerReason) {
         return;
     }
     const steps = getValidatedSteps(sessionId);
-    if (steps.length === 0 && triggerReason !== 'abandoned') {
+    // Allow saving if: has steps OR has final_response OR is abandoned
+    const hasFinalResponse = sessionState.final_response && sessionState.final_response.length > 100;
+    if (steps.length === 0 && !hasFinalResponse && triggerReason !== 'abandoned') {
         return; // Nothing to save
     }
     // Build task data from session state and steps
     const taskData = await buildTaskFromSession(sessionState, steps, triggerReason);
     // Create task in team memory
-    createTask(taskData);
+    const task = createTask(taskData);
+    // Fire-and-forget cloud sync; never block capture path
+    syncTask(task)
+        .then((success) => {
+        if (success) {
+            markTaskSynced(task.id);
+        }
+        else {
+            setTaskSyncError(task.id, 'Sync not enabled or team not configured');
+        }
+    })
+        .catch((err) => {
+        const message = err instanceof Error ? err.message : 'Unknown sync error';
+        setTaskSyncError(task.id, message);
+    });
 }
 /**
  * Build task data from session state and steps
  */
 async function buildTaskFromSession(sessionState, steps, triggerReason) {
-    // Aggregate files from steps
-    const filesTouched = [...new Set(steps.flatMap(s => s.files))];
+    // Aggregate files from steps (tool_use actions)
+    const stepFiles = steps.flatMap(s => s.files);
+    // Also extract file paths mentioned in reasoning text (Claude's text responses)
+    const reasoningFiles = steps
+        .filter(s => s.reasoning)
+        .flatMap(s => {
+        // Match common code file extensions
+        const matches = s.reasoning?.match(/[\w\/.-]+\.(ts|js|tsx|jsx|py|go|rs|java|css|html|md|json|yaml|yml)/g) || [];
+        // Filter out obvious non-paths (urls, version numbers)
+        return matches.filter(m => !m.includes('://') && !m.match(/^\d+\.\d+/));
+    });
+    const filesTouched = [...new Set([...stepFiles, ...reasoningFiles])];
     // Build basic reasoning trace from steps (fallback)
     const basicReasoningTrace = steps
         .filter(s => s.is_key_decision || s.action_type === 'edit' || s.action_type === 'write')
@@ -43,12 +70,16 @@ async function buildTaskFromSession(sessionState, steps, triggerReason) {
     let reasoningTrace = basicReasoningTrace;
     let decisions = [];
     let constraints = sessionState.constraints || [];
-    if (isReasoningExtractionAvailable() && steps.length > 0) {
+    if (isReasoningExtractionAvailable()) {
         try {
-            // Collect reasoning from steps
+            // Collect reasoning from steps + final response
             const stepsReasoning = steps
                 .map(s => s.reasoning)
                 .filter((r) => !!r && r.length > 10);
+            // Include final response (contains the actual analysis/conclusion)
+            if (sessionState.final_response && sessionState.final_response.length > 100) {
+                stepsReasoning.push(sessionState.final_response);
+            }
             if (stepsReasoning.length > 0) {
                 const extracted = await extractReasoningAndDecisions(stepsReasoning, sessionState.original_goal || '');
                 if (extracted.reasoning_trace.length > 0) {

package/dist/proxy/server.d.ts

@@ -1,9 +1,13 @@
 import { FastifyInstance } from 'fastify';
+export declare function setDebugMode(enabled: boolean): void;
 /**
  * Create and configure the Fastify server
  */
 export declare function createServer(): FastifyInstance;
 /**
  * Start the proxy server
+ * @param options.debug - Enable debug logging to grov-proxy.log
  */
-export declare function startServer(): Promise<FastifyInstance>;
+export declare function startServer(options?: {
+    debug?: boolean;
+}): Promise<FastifyInstance>;