@aj-archipelago/cortex 1.4.27 → 1.4.28

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aj-archipelago/cortex",
-  "version": "1.4.27",
+  "version": "1.4.28",
   "description": "Cortex is a GraphQL API for AI. It provides a simple, extensible interface for using AI services from OpenAI, Azure and others.",
   "private": false,
   "repository": {

package/pathways/system/entity/tools/sys_tool_codingagent.js

@@ -81,19 +81,23 @@ export default {
 
     executePathway: async ({args, resolver}) => {
         try {
-            const { codingTask, userMessage, inputFiles, codingTaskKeywords } = args;
+            const { codingTask, userMessage, inputFiles, codingTaskKeywords, contextId } = args;
+
+            // pathwayResolver.executePathway should have already extracted contextId from agentContext,
+            // but validate it's present as a safety check
+            if (!contextId) {
+                throw new Error("contextId is required. It should be provided via agentContext or contextId parameter.");
+            }
 
             let taskSuffix = "";
-            if (inputFiles) {
+            if (inputFiles && Array.isArray(inputFiles) && inputFiles.length > 0) {
                 if (!args.agentContext || !Array.isArray(args.agentContext) || args.agentContext.length === 0) {
                     throw new Error("agentContext is required when using the 'inputFiles' parameter. Use ListFileCollection or SearchFileCollection to find available files.");
                 }
                 
                 // Resolve file parameters to URLs
                 // inputFiles is an array of strings (file hashes or filenames)
-                const fileReferences = Array.isArray(inputFiles) 
-                    ? inputFiles.map(ref => String(ref).trim()).filter(ref => ref.length > 0)
-                    : [];
+                const fileReferences = inputFiles.map(ref => String(ref).trim()).filter(ref => ref.length > 0);
                 
                 const resolvedUrls = [];
                 const failedFiles = [];
@@ -142,8 +146,15 @@ export default {
             const statusMessage = "⚠️ **Task Status**: The coding task has been started and is now running in the background. Don't make up any information about the task or task results - just say that it has been started and is running. The user will be able to see the progress and results of the task, but you will not receive the response. No further action is required from you or the user.";         
             return statusMessage;
         } catch (error) {
+            const errorMessage = error instanceof Error ? error.message : String(error);
             logger.error(`Error in coding agent tool: ${error instanceof Error ? error.stack || error.message : JSON.stringify(error)}`);
-            throw error;
+            
+            // Return error as JSON string instead of throwing, so it can be properly detected by the tool execution layer
+            resolver.tool = JSON.stringify({ toolUsed: "coding" });
+            return JSON.stringify({ 
+                error: errorMessage,
+                recoveryMessage: "The coding task could not be started. Please check that all required parameters are provided and try again."
+            });
         }
     }
 }; 

package/pathways/system/entity/tools/sys_tool_store_memory.js

@@ -58,6 +58,15 @@ export default {
             });
         }
 
+        // pathwayResolver.executePathway should have already extracted contextId from agentContext,
+        // but validate it's present as a safety check
+        const { contextId, contextKey } = args;
+        if (!contextId) {
+            return JSON.stringify({ 
+                error: 'contextId is required. It should be provided via agentContext or contextId parameter.' 
+            });
+        }
+
         // Validate memories array
         if (!args.memories || !Array.isArray(args.memories) || args.memories.length === 0) {
             return JSON.stringify({ error: 'memories must be a non-empty array' });
@@ -102,9 +111,9 @@ export default {
         for (const [section, memoryLines] of Object.entries(memoriesBySection)) {
             // Read current memory for the section
             let currentMemory = await callPathway('sys_read_memory', {
-                contextId: args.contextId,
+                contextId: contextId,
                 section: section,
-                contextKey: args.contextKey
+                contextKey: contextKey
             });
 
             // Combine existing memory with new memories
@@ -114,10 +123,10 @@ export default {
 
             // Save directly to memory
             const result = await callPathway('sys_save_memory', {
-                contextId: args.contextId,
+                contextId: contextId,
                 section: section,
                 aiMemory: updatedMemory,
-                contextKey: args.contextKey
+                contextKey: contextKey
             });
             
             results[section] = result;