@lovelybunch/api 1.0.73 → 1.0.74

package/dist/routes/api/v1/ai/route.js

@@ -3,8 +3,9 @@ import { join, resolve as pathResolve, basename } from 'path';
 import { existsSync, readFileSync, promises as fs, createReadStream } from 'fs';
 import { fileURLToPath } from 'url';
 import readline from 'readline';
-import { getLogsDir } from '@lovelybunch/core';
-import { proposalsReadOnlyTool, knowledgeTool, normalizeKnowledgeMetadata, eventsTool, projectContextTool, architectureContextTool, roleContextTool } from '@lovelybunch/mcp';
+import { ZodError } from 'zod';
+import { getLogsDir, listProposals, getProposal, createProposal, updateProposal, deleteProposal, } from '@lovelybunch/core';
+import { proposalsFullTool, knowledgeTool, normalizeKnowledgeMetadata, eventsTool, projectContextTool, architectureContextTool, roleContextTool } from '@lovelybunch/mcp';
 import matter from 'gray-matter';
 import Fuse from 'fuse.js';
 import { FileStorageAdapter } from '../../../../lib/storage/file-storage.js';
@@ -53,11 +54,11 @@ export async function POST(c) {
             ? `${baseSystem}\n\nThe following persona is authoritative and overrides general guidance above. You must strictly follow it.\n\n${agentPersona}`
             : baseSystem;
         // Prepare tools for function calling
-        // Note: proposals is read-only, knowledge/project/architecture are read+write, events is read-only
+        // Note: proposals/knowledge/project/architecture are read+write, events is read-only
         const tools = enableTools ? [
             {
                 type: "function",
-                function: proposalsReadOnlyTool
+                function: proposalsFullTool
             },
             {
                 type: "function",
@@ -239,17 +240,26 @@ async function executeToolCalls(toolCalls) {
             let functionArgs;
             if (toolCall.function?.arguments) {
                 try {
-                    functionArgs = JSON.parse(toolCall.function.arguments);
+                    // Try to sanitize common JSON issues before parsing
+                    let argsStr = toolCall.function.arguments;
+                    // Remove trailing commas before ] or }
+                    argsStr = argsStr.replace(/,(\s*[}\]])/g, '$1');
+                    // Fix common LLM JSON issues:
+                    // Remove control characters (except \n, \r, \t which are valid in JSON strings)
+                    argsStr = argsStr.replace(/[\x00-\x08\x0B\x0C\x0E-\x1F]/g, '');
+                    functionArgs = JSON.parse(argsStr);
                 }
                 catch (parseError) {
                     // LLM may generate invalid JSON with unescaped characters in long content
-                    // Return a helpful error message
                     const errorMsg = parseError instanceof Error ? parseError.message : 'Unknown parse error';
+                    const hint = errorMsg.includes('position')
+                        ? 'The content may contain unescaped quotes or special characters. Try updating with shorter/simpler content, or update one field at a time.'
+                        : 'Please retry with simpler content.';
                     return {
                         tool_call_id: toolCall.id,
                         content: JSON.stringify({
                             success: false,
-                            error: `Invalid tool arguments: ${errorMsg}. Please retry with simpler content.`
+                            error: `Invalid tool arguments: ${errorMsg}. ${hint}`
                         })
                     };
                 }
@@ -299,38 +309,89 @@ async function executeToolCalls(toolCalls) {
     });
     return Promise.all(resultPromises);
 }
-// Proposals tool is READ-ONLY - only list and get operations are supported
-async function executeProposalsToolDirect(args, storage) {
-    const { operation, id, filters } = args;
+// Proposals tool - full CRUD operations using @lovelybunch/core
+async function executeProposalsToolDirect(args, _storage) {
+    const { operation, id, filters, proposal, updates } = args;
     try {
         switch (operation) {
             case 'list': {
-                const proposals = await storage.listCPs(filters || {});
+                const proposals = await listProposals(filters || {});
                 return {
                     success: true,
                     data: proposals,
-                    message: `Found ${proposals.length} proposals`
+                    count: proposals.length,
+                    message: `Found ${proposals.length} tasks`
                 };
             }
             case 'get': {
                 if (!id) {
-                    return { success: false, error: 'Proposal ID is required for get operation' };
+                    return { success: false, error: 'Task ID is required for get operation' };
                 }
-                const proposal = await storage.getCP(id);
+                const result = await getProposal(id);
+                if (!result) {
+                    return { success: false, error: 'Task not found' };
+                }
+                return {
+                    success: true,
+                    data: result,
+                    message: `Retrieved task ${id}`
+                };
+            }
+            case 'create': {
                 if (!proposal) {
-                    return { success: false, error: 'Proposal not found' };
+                    return { success: false, error: 'Task data is required for create operation' };
                 }
+                const created = await createProposal(proposal);
                 return {
                     success: true,
-                    data: proposal,
-                    message: `Retrieved proposal ${id}`
+                    data: created,
+                    message: `Created task ${created.id}`
+                };
+            }
+            case 'update': {
+                if (!id) {
+                    return { success: false, error: 'Task ID is required for update operation' };
+                }
+                const updateData = updates || proposal;
+                if (!updateData) {
+                    return { success: false, error: 'Update data is required for update operation' };
+                }
+                const updated = await updateProposal(id, updateData);
+                return {
+                    success: true,
+                    data: updated,
+                    message: `Updated task ${id}`
+                };
+            }
+            case 'delete': {
+                if (!id) {
+                    return { success: false, error: 'Task ID is required for delete operation' };
+                }
+                const deleted = await deleteProposal(id);
+                if (!deleted) {
+                    return { success: false, error: 'Task not found' };
+                }
+                return {
+                    success: true,
+                    message: `Deleted task ${id}`
                 };
             }
             default:
-                return { success: false, error: `Proposals are read-only. Only 'list' and 'get' operations are supported. To create proposals, use a coding agent (Claude Code, Cursor, etc.) or the Proposals UI.` };
+                return { success: false, error: `Unknown operation: ${operation}. Supported operations: list, get, create, update, delete` };
         }
     }
     catch (error) {
+        // Handle Zod validation errors specially
+        if (error instanceof ZodError) {
+            return {
+                success: false,
+                error: 'Validation failed',
+                details: error.issues.map(e => ({
+                    path: e.path.join('.'),
+                    message: e.message
+                }))
+            };
+        }
         console.error('Error executing proposals tool:', error);
         return { success: false, error: error.message || 'Tool execution failed' };
     }

package/dist/routes/api/v1/ai/tools.js

@@ -3,8 +3,9 @@ import { homedir } from 'os';
 import { join, resolve as pathResolve, basename } from 'path';
 import { existsSync, readFileSync, promises as fs, createReadStream } from 'fs';
 import readline from 'readline';
-import { getLogsDir } from '@lovelybunch/core';
-import { proposalsReadOnlyTool, knowledgeTool, normalizeKnowledgeMetadata, eventsTool, projectContextTool, architectureContextTool, roleContextTool } from '@lovelybunch/mcp';
+import { ZodError } from 'zod';
+import { getLogsDir, listProposals, getProposal, createProposal, updateProposal, deleteProposal, } from '@lovelybunch/core';
+import { proposalsFullTool, knowledgeTool, normalizeKnowledgeMetadata, eventsTool, projectContextTool, architectureContextTool, roleContextTool } from '@lovelybunch/mcp';
 import matter from 'gray-matter';
 import { FileStorageAdapter } from '../../../../lib/storage/file-storage.js';
 // Function to get global config API key as fallback
@@ -93,11 +94,11 @@ export async function POST(c) {
             ...toolResultMessages
         ];
         // Prepare tools for potential additional tool calls
-        // Note: proposals is read-only, knowledge/project/architecture are read+write, events is read-only
+        // Note: proposals/knowledge/project/architecture are read+write, events is read-only
         const tools = [
             {
                 type: "function",
-                function: proposalsReadOnlyTool
+                function: proposalsFullTool
             },
             {
                 type: "function",
@@ -242,6 +243,9 @@ async function executeToolCalls(toolCalls) {
                     let argsStr = toolCall.function.arguments;
                     // Remove trailing commas before ] or }
                     argsStr = argsStr.replace(/,(\s*[}\]])/g, '$1');
+                    // Fix common LLM JSON issues:
+                    // 1. Remove control characters (except \n, \r, \t which are valid in JSON strings)
+                    argsStr = argsStr.replace(/[\x00-\x08\x0B\x0C\x0E-\x1F]/g, '');
                     // Try parsing the sanitized string
                     functionArgs = JSON.parse(argsStr);
                 }
@@ -255,11 +259,16 @@ async function executeToolCalls(toolCalls) {
                     arguments: toolCall.function?.arguments?.substring(0, 500),
                     error: parseError instanceof Error ? parseError.message : 'Unknown parse error'
                 });
+                // Provide a more helpful error message
+                const errorMsg = parseError instanceof Error ? parseError.message : 'JSON parse error';
+                const hint = errorMsg.includes('position')
+                    ? 'The content may contain unescaped quotes or special characters. Try updating with shorter/simpler content, or update one field at a time.'
+                    : 'Please retry with simpler content.';
                 return {
                     tool_call_id: toolCall.id,
                     content: JSON.stringify({
                         success: false,
-                        error: `Invalid tool arguments: ${parseError instanceof Error ? parseError.message : 'JSON parse error'}. Please retry with simpler content.`
+                        error: `Invalid tool arguments: ${errorMsg}. ${hint}`
                     })
                 };
             }
@@ -304,38 +313,89 @@ async function executeToolCalls(toolCalls) {
     });
     return Promise.all(resultPromises);
 }
-// Proposals tool is READ-ONLY - only list and get operations are supported
-async function executeProposalsToolDirect(args, storage) {
-    const { operation, id, filters } = args;
+// Proposals tool - full CRUD operations using @lovelybunch/core
+async function executeProposalsToolDirect(args, _storage) {
+    const { operation, id, filters, proposal, updates } = args;
     try {
         switch (operation) {
             case 'list': {
-                const proposals = await storage.listCPs(filters || {});
+                const proposals = await listProposals(filters || {});
                 return {
                     success: true,
                     data: proposals,
-                    message: `Found ${proposals.length} proposals`
+                    count: proposals.length,
+                    message: `Found ${proposals.length} tasks`
                 };
             }
             case 'get': {
                 if (!id) {
-                    return { success: false, error: 'Proposal ID is required for get operation' };
+                    return { success: false, error: 'Task ID is required for get operation' };
                 }
-                const proposal = await storage.getCP(id);
+                const result = await getProposal(id);
+                if (!result) {
+                    return { success: false, error: 'Task not found' };
+                }
+                return {
+                    success: true,
+                    data: result,
+                    message: `Retrieved task ${id}`
+                };
+            }
+            case 'create': {
                 if (!proposal) {
-                    return { success: false, error: 'Proposal not found' };
+                    return { success: false, error: 'Task data is required for create operation' };
+                }
+                const created = await createProposal(proposal);
+                return {
+                    success: true,
+                    data: created,
+                    message: `Created task ${created.id}`
+                };
+            }
+            case 'update': {
+                if (!id) {
+                    return { success: false, error: 'Task ID is required for update operation' };
+                }
+                const updateData = updates || proposal;
+                if (!updateData) {
+                    return { success: false, error: 'Update data is required for update operation' };
+                }
+                const updated = await updateProposal(id, updateData);
+                return {
+                    success: true,
+                    data: updated,
+                    message: `Updated task ${id}`
+                };
+            }
+            case 'delete': {
+                if (!id) {
+                    return { success: false, error: 'Task ID is required for delete operation' };
+                }
+                const deleted = await deleteProposal(id);
+                if (!deleted) {
+                    return { success: false, error: 'Task not found' };
                 }
                 return {
                     success: true,
-                    data: proposal,
-                    message: `Retrieved proposal ${id}`
+                    message: `Deleted task ${id}`
                 };
             }
             default:
-                return { success: false, error: `Proposals are read-only. Only 'list' and 'get' operations are supported. To create proposals, use a coding agent (Claude Code, Cursor, etc.) or the Proposals UI.` };
+                return { success: false, error: `Unknown operation: ${operation}. Supported operations: list, get, create, update, delete` };
         }
     }
     catch (error) {
+        // Handle Zod validation errors specially
+        if (error instanceof ZodError) {
+            return {
+                success: false,
+                error: 'Validation failed',
+                details: error.issues.map(e => ({
+                    path: e.path.join('.'),
+                    message: e.message
+                }))
+            };
+        }
         console.error('Error executing proposals tool:', error);
         return { success: false, error: error.message || 'Tool execution failed' };
     }