@aj-archipelago/cortex 1.4.22 → 1.4.23

package/tests/integration/graphql/async/stream/vendors/claude_streaming.test.js

@@ -21,8 +21,8 @@ test.after.always('cleanup', async () => {
 test('Claude vendor streaming over subscriptions emits OAI-style deltas', async (t) => {
   const response = await testServer.executeOperation({
     query: `
-      query($text: String!, $chatHistory: [MultiMessage]!, $stream: Boolean, $aiStyle: String) {
-        sys_entity_agent(text: $text, chatHistory: $chatHistory, stream: $stream, aiStyle: $aiStyle) {
+      query($text: String!, $chatHistory: [MultiMessage]!, $stream: Boolean) {
+        sys_entity_agent(text: $text, chatHistory: $chatHistory, stream: $stream) {
           result
         }
       }
@@ -30,8 +30,7 @@ test('Claude vendor streaming over subscriptions emits OAI-style deltas', async
     variables: {
       text: 'Say hi',
       chatHistory: [{ role: 'user', content: ['Say hi'] }],
-      stream: true,
-      aiStyle: 'Anthropic'
+      stream: true
     }
   });
 

package/tests/integration/graphql/async/stream/vendors/gemini_streaming.test.js

@@ -21,8 +21,8 @@ test.after.always('cleanup', async () => {
 test('Gemini vendor streaming over subscriptions emits OAI-style deltas', async (t) => {
   const response = await testServer.executeOperation({
     query: `
-      query($text: String!, $chatHistory: [MultiMessage]!, $stream: Boolean, $aiStyle: String) {
-        sys_entity_agent(text: $text, chatHistory: $chatHistory, stream: $stream, aiStyle: $aiStyle) {
+      query($text: String!, $chatHistory: [MultiMessage]!, $stream: Boolean) {
+        sys_entity_agent(text: $text, chatHistory: $chatHistory, stream: $stream) {
           result
         }
       }
@@ -30,8 +30,7 @@ test('Gemini vendor streaming over subscriptions emits OAI-style deltas', async
     variables: {
       text: 'Say hi',
       chatHistory: [{ role: 'user', content: ['Say hi'] }],
-      stream: true,
-      aiStyle: 'Google'
+      stream: true
     }
   });
 

package/tests/integration/graphql/async/stream/vendors/grok_streaming.test.js

@@ -21,8 +21,8 @@ test.after.always('cleanup', async () => {
 test('XAI Grok vendor streaming over subscriptions emits OAI-style deltas', async (t) => {
   const response = await testServer.executeOperation({
     query: `
-      query($text: String!, $chatHistory: [MultiMessage]!, $stream: Boolean, $aiStyle: String) {
-        sys_entity_agent(text: $text, chatHistory: $chatHistory, stream: $stream, aiStyle: $aiStyle) {
+      query($text: String!, $chatHistory: [MultiMessage]!, $stream: Boolean) {
+        sys_entity_agent(text: $text, chatHistory: $chatHistory, stream: $stream) {
           result
         }
       }
@@ -30,8 +30,7 @@ test('XAI Grok vendor streaming over subscriptions emits OAI-style deltas', asyn
     variables: {
       text: 'Say hi',
       chatHistory: [{ role: 'user', content: ['Say hi'] }],
-      stream: true,
-      aiStyle: 'XAI'
+      stream: true
     }
   });
 

package/tests/integration/graphql/async/stream/vendors/openai_streaming.test.js

@@ -21,8 +21,8 @@ test.after.always('cleanup', async () => {
 test('OpenAI vendor streaming over subscriptions emits OAI-style deltas', async (t) => {
   const response = await testServer.executeOperation({
     query: `
-      query($text: String!, $chatHistory: [MultiMessage]!, $stream: Boolean, $aiStyle: String) {
-        sys_entity_agent(text: $text, chatHistory: $chatHistory, stream: $stream, aiStyle: $aiStyle) {
+      query($text: String!, $chatHistory: [MultiMessage]!, $stream: Boolean) {
+        sys_entity_agent(text: $text, chatHistory: $chatHistory, stream: $stream) {
           result
         }
       }
@@ -30,8 +30,7 @@ test('OpenAI vendor streaming over subscriptions emits OAI-style deltas', async
     variables: {
       text: 'Say hi',
       chatHistory: [{ role: 'user', content: ['Say hi'] }],
-      stream: true,
-      aiStyle: 'OpenAI'
+      stream: true
     }
   });
 
@@ -63,7 +62,8 @@ test('OpenAI vendor streaming over subscriptions emits OAI-style deltas', async
     .filter(Boolean);
 
   if (models.length > 0) {
-    t.truthy(models.find(m => /gpt-5-chat/.test(m)));
+    // Model could be gpt-4.1, gpt-4, gpt-5, etc. depending on config
+    t.truthy(models.find(m => /gpt-4|gpt-5/.test(m)));
   }
 });
 

package/tests/unit/core/fileCollection.test.js

@@ -500,42 +500,103 @@ test('addFileToCollection should preserve original displayFilename for converted
     }
 });
 
-test('syncFilesToCollection should determine MIME type from URL, not displayFilename', async t => {
-    const { syncFilesToCollection } = await import('../../../lib/fileUtils.js');
+// Note: Tests that require Redis (adding files to collection) are in integration tests
+// These unit tests only test behavior that doesn't require Redis
+
+test('syncAndStripFilesFromChatHistory should leave all files when no contextId', async t => {
+    const { syncAndStripFilesFromChatHistory } = await import('../../../lib/fileUtils.js');
     
-    // Simulate chat history with converted file
-    const contextId = `test-sync-converted-${Date.now()}`;
     const chatHistory = [
         {
             role: 'user',
             content: [
                 {
-                    type: 'file',
-                    url: 'https://example.com/converted-report.md', // Converted to markdown
-                    gcs: 'gs://bucket/converted-report.md',
-                    hash: 'hash123'
+                    type: 'image_url',
+                    image_url: { url: 'https://example.com/image.jpg' },
+                    hash: 'somehash'
                 }
             ]
         }
     ];
     
-    try {
-        await syncFilesToCollection(chatHistory, contextId, null);
-        
-        const { loadFileCollection } = await import('../../../lib/fileUtils.js');
-        const collection = await loadFileCollection(contextId, null, false);
-        t.is(collection.length, 1);
-        
-        // MIME type should be from URL (.md), not from any displayFilename
-        t.is(collection[0].mimeType, 'text/markdown', 'MIME type should be determined from URL');
-        t.is(collection[0].url, 'https://example.com/converted-report.md');
-    } finally {
-        // Cleanup
-        const { getRedisClient } = await import('../../../lib/fileUtils.js');
-        const redisClient = await getRedisClient();
-        if (redisClient) {
-            await redisClient.del(`FileStoreMap:ctx:${contextId}`);
+    // No contextId - should leave files in place
+    const { chatHistory: processedHistory } = await syncAndStripFilesFromChatHistory(chatHistory, null, null);
+    
+    t.is(processedHistory[0].content[0].type, 'image_url');
+    t.is(processedHistory[0].content[0].image_url.url, 'https://example.com/image.jpg');
+});
+
+test('syncAndStripFilesFromChatHistory should leave files when collection is empty', async t => {
+    const { syncAndStripFilesFromChatHistory } = await import('../../../lib/fileUtils.js');
+    
+    // Use a unique contextId that won't have any files
+    const contextId = `test-empty-${Date.now()}`;
+    
+    const chatHistory = [
+        {
+            role: 'user',
+            content: [
+                {
+                    type: 'image_url',
+                    image_url: { url: 'https://example.com/image.jpg' },
+                    hash: 'somehash'
+                }
+            ]
         }
-    }
+    ];
+    
+    // Empty collection - files should stay in place (not stripped)
+    const { chatHistory: processedHistory } = await syncAndStripFilesFromChatHistory(chatHistory, contextId, null);
+    
+    t.is(processedHistory[0].content[0].type, 'image_url');
+    t.is(processedHistory[0].content[0].image_url.url, 'https://example.com/image.jpg');
+});
+
+test('syncAndStripFilesFromChatHistory should handle empty chat history', async t => {
+    const { syncAndStripFilesFromChatHistory } = await import('../../../lib/fileUtils.js');
+    
+    const { chatHistory: result1 } = await syncAndStripFilesFromChatHistory([], 'context', null);
+    t.deepEqual(result1, []);
+    
+    const { chatHistory: result2 } = await syncAndStripFilesFromChatHistory(null, 'context', null);
+    t.deepEqual(result2, []);
 });
 
+test('syncAndStripFilesFromChatHistory should preserve non-file content', async t => {
+    const { syncAndStripFilesFromChatHistory } = await import('../../../lib/fileUtils.js');
+    
+    const contextId = `test-preserve-${Date.now()}`;
+    
+    const chatHistory = [
+        {
+            role: 'user',
+            content: [
+                { type: 'text', text: 'Hello world' },
+                {
+                    type: 'image_url',
+                    image_url: { url: 'https://example.com/image.jpg' },
+                    hash: 'somehash'
+                }
+            ]
+        },
+        {
+            role: 'assistant',
+            content: 'I see an image'
+        }
+    ];
+    
+    const { chatHistory: processedHistory } = await syncAndStripFilesFromChatHistory(chatHistory, contextId, null);
+    
+    // Text content should be preserved
+    t.is(processedHistory[0].content[0].type, 'text');
+    t.is(processedHistory[0].content[0].text, 'Hello world');
+    
+    // Image not in collection should be preserved
+    t.is(processedHistory[0].content[1].type, 'image_url');
+    
+    // Assistant message should be preserved
+    t.is(processedHistory[1].role, 'assistant');
+    t.is(processedHistory[1].content, 'I see an image');
+});
+
+

package/pathways/system/workspaces/run_workspace_research_agent.js

@@ -1,27 +0,0 @@
-import { callPathway } from '../../../lib/pathwayTools.js';
-
-export default {
-    // The main prompt function that takes the input text and asks to generate a summary.
-    prompt: [],
-
-    inputParameters: {
-        model: "oai-gpt41",
-        aiStyle: "OpenAI",
-        chatHistory: [{role: '', content: []}],
-    },
-    timeout: 600,
-
-    executePathway: async ({args, _runAllPrompts, resolver}) => {
-        // chatHistory is always passed in complete
-        const response = await callPathway('sys_entity_agent', {  
-            ...args, 
-            chatHistory: args.chatHistory || [],
-            stream: false, 
-            useMemory: false,
-            researchMode: true
-        }, resolver);
-
-        return response;
-    }
-}
-