@almyty/acp-server 1.0.10 → 1.0.12

package/dist/__tests__/events.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/__tests__/events.test.js

@@ -1,81 +1 @@
-import { describe, it, expect } from 'vitest';
-import { mapStreamEvent, buildPlanFromPipeline, isTerminalEvent, extractFinalOutput } from '../events.js';
-describe('mapStreamEvent', () => {
-    it('maps node.started to tool_call with running status', () => {
-        const results = mapStreamEvent({ event: 'node.started', data: { nodeId: 'n1', nodeType: 'tool_call', name: 'get_users' } });
-        expect(results.length).toBeGreaterThan(0);
-        const tc = results.find(r => r.type === 'tool_call');
-        expect(tc).toBeDefined();
-        expect(tc.status).toBe('running');
-    });
-    it('maps node.completed to tool_call_update', () => {
-        const results = mapStreamEvent({ event: 'node.completed', data: { nodeId: 'n1', output: { users: [] } } });
-        expect(results.length).toBeGreaterThan(0);
-        const update = results.find(r => r.type === 'tool_call_update');
-        expect(update).toBeDefined();
-        expect(update.status).toBe('completed');
-    });
-    it('maps node.error to tool_call_update with error', () => {
-        const results = mapStreamEvent({ event: 'node.error', data: { nodeId: 'n1', error: 'timeout' } });
-        expect(results.length).toBeGreaterThan(0);
-        const update = results.find(r => r.type === 'tool_call_update');
-        expect(update).toBeDefined();
-        expect(update.status).toBe('error');
-    });
-    it('maps text events to agent_message_chunk', () => {
-        for (const event of ['text', 'message', 'chunk', 'token']) {
-            const results = mapStreamEvent({ event, data: { text: 'hello' } });
-            expect(results.length).toBeGreaterThan(0);
-            expect(results[0].type).toBe('agent_message_chunk');
-        }
-    });
-    it('maps thinking events to agent_thought_chunk', () => {
-        const results = mapStreamEvent({ event: 'thinking', data: { text: 'hmm' } });
-        expect(results.length).toBeGreaterThan(0);
-        expect(results[0].type).toBe('agent_thought_chunk');
-    });
-    it('returns empty array for unknown events', () => {
-        expect(mapStreamEvent({ event: 'unknown_xyz', data: {} })).toHaveLength(0);
-    });
-});
-describe('buildPlanFromPipeline', () => {
-    it('converts pipeline nodes to plan entries (filters input/output)', () => {
-        const nodes = [
-            { id: 'n1', type: 'input', data: {} },
-            { id: 'n2', type: 'llm_call', data: {} },
-            { id: 'n3', type: 'tool_call', data: { toolName: 'fetch' } },
-            { id: 'n4', type: 'output', data: {} },
-        ];
-        const plan = buildPlanFromPipeline(nodes);
-        expect(plan.type).toBe('plan');
-        // input + output filtered out, llm_call + tool_call remain
-        expect(plan.entries.length).toBe(2);
-    });
-    it('returns empty plan for empty pipeline', () => {
-        const plan = buildPlanFromPipeline([]);
-        expect(plan.entries).toHaveLength(0);
-    });
-});
-describe('isTerminalEvent', () => {
-    it('recognizes terminal events', () => {
-        expect(isTerminalEvent({ event: 'execution.completed', data: {} })).toBe(true);
-        expect(isTerminalEvent({ event: 'execution.error', data: {} })).toBe(true);
-        expect(isTerminalEvent({ event: 'run.completed', data: {} })).toBe(true);
-    });
-    it('rejects non-terminal events', () => {
-        expect(isTerminalEvent({ event: 'text', data: {} })).toBe(false);
-        expect(isTerminalEvent({ event: 'node.started', data: {} })).toBe(false);
-    });
-});
-describe('extractFinalOutput', () => {
-    it('extracts text from output', () => {
-        expect(extractFinalOutput({ event: 'execution.completed', data: { output: 'result' } })).toBe('result');
-    });
-    it('stringifies object output', () => {
-        const result = extractFinalOutput({ event: 'execution.completed', data: { output: { key: 'val' } } });
-        expect(result).toContain('key');
-    });
-    it('returns undefined for missing output', () => {
-        expect(extractFinalOutput({ event: 'execution.completed', data: {} })).toBeUndefined();
-    });
-});
+"use strict";

package/dist/agent.js

@@ -260,7 +260,6 @@ export class AlmytyAcpAgent {
                     if (event.event === 'execution.error' || event.event === 'run.error') {
                         return {
                             stopReason: 'end_turn',
-                            content: [{ type: 'text', text: collectedText.join('') || 'Execution failed.' }],
                         };
                     }
                 }
@@ -281,14 +280,12 @@ export class AlmytyAcpAgent {
                 });
                 return {
                     stopReason: 'end_turn',
-                    content: [{ type: 'text', text }],
                 };
             }
             catch (invokeErr) {
                 const errMsg = invokeErr instanceof Error ? invokeErr.message : String(invokeErr);
                 return {
                     stopReason: 'end_turn',
-                    content: [{ type: 'text', text: `Error: ${errMsg}` }],
                 };
             }
         }
@@ -420,7 +417,6 @@ export class AlmytyAcpAgent {
             const errMsg = err instanceof Error ? err.message : String(err);
             return {
                 stopReason: 'end_turn',
-                content: [{ type: 'text', text: `Error: ${errMsg}` }],
             };
         }
     }
@@ -490,9 +486,24 @@ export class AlmytyAcpAgent {
         });
     }
     sendSessionUpdate(sessionId, update) {
+        // Transform internal format to ACP spec format:
+        // - "type" -> "sessionUpdate"
+        // - text content wrapped in { type: 'text', text }
+        const specUpdate = { ...update };
+        if ('type' in specUpdate) {
+            specUpdate.sessionUpdate = specUpdate.type;
+            delete specUpdate.type;
+        }
+        // Wrap bare text in content object for message/thought chunks
+        if (specUpdate.sessionUpdate === 'agent_message_chunk' || specUpdate.sessionUpdate === 'agent_thought_chunk') {
+            if ('text' in specUpdate && !('content' in specUpdate)) {
+                specUpdate.content = { type: 'text', text: specUpdate.text };
+                delete specUpdate.text;
+            }
+        }
         this.sendNotification('session/update', {
             sessionId,
-            update,
+            update: specUpdate,
         });
     }
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@almyty/acp-server",
-  "version": "1.0.10",
+  "version": "1.0.12",
   "description": "ACP agent server for almyty — expose any almyty agent via the Agent Client Protocol",
   "type": "module",
   "main": "dist/index.js",