opc-agent 1.2.0 → 1.2.1

package/tests/streaming.test.ts

@@ -0,0 +1,109 @@
+import { describe, it, expect, vi } from 'vitest';
+import { StreamingManager, StreamableResponse } from '../src/core/streaming';
+import type { StreamChunk } from '../src/core/streaming';
+
+describe('StreamableResponse', () => {
+  it('should collect chunks and build text', () => {
+    const stream = new StreamableResponse('test-1');
+    stream.push({ id: '0', type: 'text', data: 'Hello ', timestamp: Date.now() });
+    stream.push({ id: '1', type: 'text', data: 'World', timestamp: Date.now() });
+    expect(stream.getText()).toBe('Hello World');
+    expect(stream.length).toBe(2);
+  });
+
+  it('should emit chunk events', () => {
+    const stream = new StreamableResponse('test-2');
+    const received: StreamChunk[] = [];
+    stream.on('chunk', (c: StreamChunk) => received.push(c));
+    stream.push({ id: '0', type: 'text', data: 'hi', timestamp: Date.now() });
+    expect(received.length).toBe(1);
+  });
+
+  it('should emit end event', () => {
+    const stream = new StreamableResponse('test-3');
+    let ended = false;
+    stream.on('end', () => { ended = true; });
+    stream.end();
+    expect(ended).toBe(true);
+    expect(stream.isEnded).toBe(true);
+  });
+
+  it('should apply backpressure at highWaterMark', () => {
+    const stream = new StreamableResponse('test-4', { highWaterMark: 2 });
+    const chunk = (): StreamChunk => ({ id: 'c', type: 'text', data: 'x', timestamp: Date.now() });
+    stream.push(chunk()); // 1 — ok
+    const ok = stream.push(chunk()); // 2 — triggers backpressure
+    expect(ok).toBe(false);
+    expect(stream.isPaused).toBe(true);
+  });
+
+  it('should flush buffer on resume', () => {
+    const stream = new StreamableResponse('test-5', { highWaterMark: 1 });
+    const received: StreamChunk[] = [];
+    stream.on('chunk', (c: StreamChunk) => received.push(c));
+    stream.push({ id: '0', type: 'text', data: 'a', timestamp: Date.now() });
+    // Now paused — next chunk goes to buffer
+    stream.push({ id: '1', type: 'text', data: 'b', timestamp: Date.now() });
+    expect(received.length).toBe(1);
+    stream.resume();
+    expect(received.length).toBe(2);
+    expect(stream.isPaused).toBe(false);
+  });
+
+  it('should reject pushes after end', () => {
+    const stream = new StreamableResponse('test-6');
+    stream.end();
+    const ok = stream.push({ id: '0', type: 'text', data: 'late', timestamp: Date.now() });
+    expect(ok).toBe(false);
+    expect(stream.length).toBe(0);
+  });
+});
+
+describe('StreamingManager', () => {
+  it('should create and manage streams', () => {
+    const mgr = new StreamingManager();
+    const stream = mgr.createStream();
+    expect(stream.id).toMatch(/^stream_/);
+    expect(mgr.activeCount).toBe(1);
+  });
+
+  it('should write chunks and end stream', () => {
+    const mgr = new StreamingManager();
+    const stream = mgr.createStream();
+    mgr.writeChunk(stream.id, 'hello');
+    mgr.writeChunk(stream.id, ' world');
+    mgr.endStream(stream.id);
+    expect(stream.getText()).toBe('hello world');
+    expect(stream.isEnded).toBe(true);
+  });
+
+  it('should return false for unknown stream writes', () => {
+    const mgr = new StreamingManager();
+    expect(mgr.writeChunk('nonexistent', 'data')).toBe(false);
+  });
+
+  it('should format SSE correctly', () => {
+    const chunk: StreamChunk = { id: 'c1', type: 'text', data: 'hi', timestamp: 123 };
+    const sse = StreamingManager.formatSSE(chunk);
+    expect(sse).toContain('event: text');
+    expect(sse).toContain('id: c1');
+    expect(sse).toContain('"data":"hi"');
+  });
+
+  it('should pipe to SSE response', () => {
+    const mgr = new StreamingManager();
+    const stream = mgr.createStream();
+    const written: string[] = [];
+    const mockRes = {
+      write: (d: string) => { written.push(d); return true; },
+      end: vi.fn(),
+      setHeader: vi.fn(),
+    };
+    StreamingManager.pipeSSE(stream, mockRes, { heartbeatInterval: 100_000 });
+    mgr.writeChunk(stream.id, 'data');
+    mgr.endStream(stream.id);
+    expect(written.length).toBeGreaterThan(0);
+    expect(mockRes.end).toHaveBeenCalled();
+    expect(mockRes.setHeader).toHaveBeenCalledWith('Content-Type', 'text/event-stream');
+  });
+});

package/tests/templates.test.ts

@@ -1,77 +1,77 @@
-import { describe, it, expect } from 'vitest';
-import { createLegalAssistantConfig, ContractReviewSkill, ComplianceCheckSkill } from '../src/templates/legal-assistant';
-import { createFinancialAdvisorConfig, BudgetAnalysisSkill, FinancialPlanningSkill } from '../src/templates/financial-advisor';
-import { createExecutiveAssistantConfig, CalendarSkill, EmailDraftSkill, MeetingPrepSkill } from '../src/templates/executive-assistant';
-import type { AgentContext, Message, MemoryStore } from '../src/core/types';
-
-const mockMemory: MemoryStore = {
-  get: async () => null, set: async () => {}, getConversation: async () => [],
-  addMessage: async () => {}, clear: async () => {},
-};
-const ctx: AgentContext = { agentName: 'test', sessionId: 's1', messages: [], memory: mockMemory, metadata: {} };
-const msg = (content: string): Message => ({ id: '1', role: 'user', content, timestamp: Date.now() });
-
-describe('Legal Assistant Template', () => {
-  it('should create valid config', () => {
-    const config = createLegalAssistantConfig();
-    expect(config.metadata.name).toBe('legal-assistant');
-    expect(config.spec.skills).toHaveLength(2);
-  });
-
-  it('ContractReviewSkill matches contract terms', async () => {
-    const skill = new ContractReviewSkill();
-    const r = await skill.execute(ctx, msg('What is force majeure?'));
-    expect(r.handled).toBe(true);
-  });
-
-  it('ComplianceCheckSkill matches GDPR', async () => {
-    const skill = new ComplianceCheckSkill();
-    const r = await skill.execute(ctx, msg('GDPR requirements'));
-    expect(r.handled).toBe(true);
-  });
-});
-
-describe('Financial Advisor Template', () => {
-  it('should create valid config', () => {
-    const config = createFinancialAdvisorConfig();
-    expect(config.metadata.name).toBe('financial-advisor');
-  });
-
-  it('BudgetAnalysisSkill matches budget queries', async () => {
-    const skill = new BudgetAnalysisSkill();
-    const r = await skill.execute(ctx, msg('Help with my budget'));
-    expect(r.handled).toBe(true);
-  });
-
-  it('FinancialPlanningSkill matches investment queries', async () => {
-    const skill = new FinancialPlanningSkill();
-    const r = await skill.execute(ctx, msg('How to invest'));
-    expect(r.handled).toBe(true);
-  });
-});
-
-describe('Executive Assistant Template', () => {
-  it('should create valid config', () => {
-    const config = createExecutiveAssistantConfig();
-    expect(config.metadata.name).toBe('executive-assistant');
-    expect(config.spec.skills).toHaveLength(3);
-  });
-
-  it('CalendarSkill matches scheduling', async () => {
-    const skill = new CalendarSkill();
-    const r = await skill.execute(ctx, msg('Schedule a meeting'));
-    expect(r.handled).toBe(true);
-  });
-
-  it('EmailDraftSkill matches email queries', async () => {
-    const skill = new EmailDraftSkill();
-    const r = await skill.execute(ctx, msg('Draft an email'));
-    expect(r.handled).toBe(true);
-  });
-
-  it('MeetingPrepSkill matches prep queries', async () => {
-    const skill = new MeetingPrepSkill();
-    const r = await skill.execute(ctx, msg('Prepare the agenda'));
-    expect(r.handled).toBe(true);
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { createLegalAssistantConfig, ContractReviewSkill, ComplianceCheckSkill } from '../src/templates/legal-assistant';
+import { createFinancialAdvisorConfig, BudgetAnalysisSkill, FinancialPlanningSkill } from '../src/templates/financial-advisor';
+import { createExecutiveAssistantConfig, CalendarSkill, EmailDraftSkill, MeetingPrepSkill } from '../src/templates/executive-assistant';
+import type { AgentContext, Message, MemoryStore } from '../src/core/types';
+
+const mockMemory: MemoryStore = {
+  get: async () => null, set: async () => {}, getConversation: async () => [],
+  addMessage: async () => {}, clear: async () => {},
+};
+const ctx: AgentContext = { agentName: 'test', sessionId: 's1', messages: [], memory: mockMemory, metadata: {} };
+const msg = (content: string): Message => ({ id: '1', role: 'user', content, timestamp: Date.now() });
+
+describe('Legal Assistant Template', () => {
+  it('should create valid config', () => {
+    const config = createLegalAssistantConfig();
+    expect(config.metadata.name).toBe('legal-assistant');
+    expect(config.spec.skills).toHaveLength(2);
+  });
+
+  it('ContractReviewSkill matches contract terms', async () => {
+    const skill = new ContractReviewSkill();
+    const r = await skill.execute(ctx, msg('What is force majeure?'));
+    expect(r.handled).toBe(true);
+  });
+
+  it('ComplianceCheckSkill matches GDPR', async () => {
+    const skill = new ComplianceCheckSkill();
+    const r = await skill.execute(ctx, msg('GDPR requirements'));
+    expect(r.handled).toBe(true);
+  });
+});
+
+describe('Financial Advisor Template', () => {
+  it('should create valid config', () => {
+    const config = createFinancialAdvisorConfig();
+    expect(config.metadata.name).toBe('financial-advisor');
+  });
+
+  it('BudgetAnalysisSkill matches budget queries', async () => {
+    const skill = new BudgetAnalysisSkill();
+    const r = await skill.execute(ctx, msg('Help with my budget'));
+    expect(r.handled).toBe(true);
+  });
+
+  it('FinancialPlanningSkill matches investment queries', async () => {
+    const skill = new FinancialPlanningSkill();
+    const r = await skill.execute(ctx, msg('How to invest'));
+    expect(r.handled).toBe(true);
+  });
+});
+
+describe('Executive Assistant Template', () => {
+  it('should create valid config', () => {
+    const config = createExecutiveAssistantConfig();
+    expect(config.metadata.name).toBe('executive-assistant');
+    expect(config.spec.skills).toHaveLength(3);
+  });
+
+  it('CalendarSkill matches scheduling', async () => {
+    const skill = new CalendarSkill();
+    const r = await skill.execute(ctx, msg('Schedule a meeting'));
+    expect(r.handled).toBe(true);
+  });
+
+  it('EmailDraftSkill matches email queries', async () => {
+    const skill = new EmailDraftSkill();
+    const r = await skill.execute(ctx, msg('Draft an email'));
+    expect(r.handled).toBe(true);
+  });
+
+  it('MeetingPrepSkill matches prep queries', async () => {
+    const skill = new MeetingPrepSkill();
+    const r = await skill.execute(ctx, msg('Prepare the agenda'));
+    expect(r.handled).toBe(true);
+  });
+});

package/tests/v070.test.ts

@@ -1,76 +1,76 @@
-import { describe, it, expect } from 'vitest';
-import { createAuthMiddleware, getActiveSessions } from '../src/core/auth';
-import { HttpSkill } from '../src/skills/http';
-import { WebhookTriggerSkill } from '../src/skills/webhook-trigger';
-import { SchedulerSkill } from '../src/skills/scheduler';
-import type { AgentContext, Message } from '../src/core/types';
-
-const mockContext: AgentContext = {
-  agentName: 'test',
-  sessionId: 'test-session',
-  messages: [],
-  memory: { get: async () => null, set: async () => {}, getConversation: async () => [], addMessage: async () => {}, clear: async () => {} },
-  metadata: {},
-};
-
-function msg(content: string): Message {
-  return { id: '1', role: 'user', content, timestamp: Date.now() };
-}
-
-describe('Auth', () => {
-  it('should create middleware', () => {
-    const mw = createAuthMiddleware({ apiKeys: ['test-key'] });
-    expect(typeof mw).toBe('function');
-  });
-
-  it('should track sessions', () => {
-    const sessions = getActiveSessions();
-    expect(Array.isArray(sessions)).toBe(true);
-  });
-});
-
-describe('HttpSkill', () => {
-  it('should not match non-http messages', async () => {
-    const skill = new HttpSkill();
-    const result = await skill.execute(mockContext, msg('hello'));
-    expect(result.handled).toBe(false);
-  });
-});
-
-describe('WebhookTriggerSkill', () => {
-  it('should report unknown webhook', async () => {
-    const skill = new WebhookTriggerSkill();
-    const result = await skill.execute(mockContext, msg('webhook test'));
-    expect(result.handled).toBe(true);
-    expect(result.response).toContain('Unknown webhook');
-  });
-
-  it('should register targets', () => {
-    const skill = new WebhookTriggerSkill();
-    skill.registerTarget({ name: 'slack', url: 'https://hooks.slack.com/test' });
-    expect(skill).toBeDefined();
-  });
-});
-
-describe('SchedulerSkill', () => {
-  it('should list empty tasks', async () => {
-    const skill = new SchedulerSkill();
-    const result = await skill.execute(mockContext, msg('schedule list'));
-    expect(result.handled).toBe(true);
-    expect(result.response).toContain('No scheduled tasks');
-  });
-
-  it('should add a task', async () => {
-    const skill = new SchedulerSkill();
-    const result = await skill.execute(mockContext, msg('schedule add backup "every 5m" run backup'));
-    expect(result.handled).toBe(true);
-    expect(result.response).toContain('Task scheduled');
-    skill.destroy();
-  });
-
-  it('should not match unrelated messages', async () => {
-    const skill = new SchedulerSkill();
-    const result = await skill.execute(mockContext, msg('hello'));
-    expect(result.handled).toBe(false);
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { createAuthMiddleware, getActiveSessions } from '../src/core/auth';
+import { HttpSkill } from '../src/skills/http';
+import { WebhookTriggerSkill } from '../src/skills/webhook-trigger';
+import { SchedulerSkill } from '../src/skills/scheduler';
+import type { AgentContext, Message } from '../src/core/types';
+
+const mockContext: AgentContext = {
+  agentName: 'test',
+  sessionId: 'test-session',
+  messages: [],
+  memory: { get: async () => null, set: async () => {}, getConversation: async () => [], addMessage: async () => {}, clear: async () => {} },
+  metadata: {},
+};
+
+function msg(content: string): Message {
+  return { id: '1', role: 'user', content, timestamp: Date.now() };
+}
+
+describe('Auth', () => {
+  it('should create middleware', () => {
+    const mw = createAuthMiddleware({ apiKeys: ['test-key'] });
+    expect(typeof mw).toBe('function');
+  });
+
+  it('should track sessions', () => {
+    const sessions = getActiveSessions();
+    expect(Array.isArray(sessions)).toBe(true);
+  });
+});
+
+describe('HttpSkill', () => {
+  it('should not match non-http messages', async () => {
+    const skill = new HttpSkill();
+    const result = await skill.execute(mockContext, msg('hello'));
+    expect(result.handled).toBe(false);
+  });
+});
+
+describe('WebhookTriggerSkill', () => {
+  it('should report unknown webhook', async () => {
+    const skill = new WebhookTriggerSkill();
+    const result = await skill.execute(mockContext, msg('webhook test'));
+    expect(result.handled).toBe(true);
+    expect(result.response).toContain('Unknown webhook');
+  });
+
+  it('should register targets', () => {
+    const skill = new WebhookTriggerSkill();
+    skill.registerTarget({ name: 'slack', url: 'https://hooks.slack.com/test' });
+    expect(skill).toBeDefined();
+  });
+});
+
+describe('SchedulerSkill', () => {
+  it('should list empty tasks', async () => {
+    const skill = new SchedulerSkill();
+    const result = await skill.execute(mockContext, msg('schedule list'));
+    expect(result.handled).toBe(true);
+    expect(result.response).toContain('No scheduled tasks');
+  });
+
+  it('should add a task', async () => {
+    const skill = new SchedulerSkill();
+    const result = await skill.execute(mockContext, msg('schedule add backup "every 5m" run backup'));
+    expect(result.handled).toBe(true);
+    expect(result.response).toContain('Task scheduled');
+    skill.destroy();
+  });
+
+  it('should not match unrelated messages', async () => {
+    const skill = new SchedulerSkill();
+    const result = await skill.execute(mockContext, msg('hello'));
+    expect(result.handled).toBe(false);
+  });
+});

package/tests/versioning.test.ts

@@ -1,75 +1,75 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { VersionManager } from '../src/core/versioning';
-import * as fs from 'fs';
-
-const TEST_STORE = '.test-versions.json';
-
-describe('VersionManager', () => {
-  let vm: VersionManager;
-
-  beforeEach(() => {
-    if (fs.existsSync(TEST_STORE)) fs.unlinkSync(TEST_STORE);
-    vm = new VersionManager(TEST_STORE);
-  });
-
-  afterEach(() => {
-    if (fs.existsSync(TEST_STORE)) fs.unlinkSync(TEST_STORE);
-  });
-
-  it('should save and list versions', () => {
-    vm.snapshot('1.0.0', 'yaml-content-1', 'Initial');
-    vm.snapshot('1.1.0', 'yaml-content-2', 'Update');
-    const versions = vm.list();
-    expect(versions).toHaveLength(2);
-    expect(versions[0].version).toBe('1.0.0');
-  });
-
-  it('should get specific version', () => {
-    vm.snapshot('1.0.0', 'content', 'Test');
-    expect(vm.get('1.0.0')).toBeDefined();
-    expect(vm.get('9.9.9')).toBeUndefined();
-  });
-
-  it('should get current version', () => {
-    vm.snapshot('1.0.0', 'v1');
-    vm.snapshot('2.0.0', 'v2');
-    expect(vm.getCurrent()?.version).toBe('2.0.0');
-  });
-
-  it('should rollback to version', () => {
-    vm.snapshot('1.0.0', 'old-yaml');
-    vm.snapshot('2.0.0', 'new-yaml');
-    const rolled = vm.rollback('1.0.0');
-    expect(rolled).toBe('old-yaml');
-  });
-
-  it('should return null for unknown rollback', () => {
-    expect(vm.rollback('9.9.9')).toBeNull();
-  });
-
-  it('should run migrations', () => {
-    vm.registerMigration({
-      fromVersion: '1.0.0',
-      toVersion: '2.0.0',
-      migrate: (oad) => ({ ...oad, upgraded: true }),
-    });
-    const result = vm.migrate({ data: 'test' }, '1.0.0', '2.0.0');
-    expect((result as any).upgraded).toBe(true);
-  });
-
-  it('should throw on missing migration path', () => {
-    expect(() => vm.migrate({}, '1.0.0', '5.0.0')).toThrow('No migration path');
-  });
-
-  it('should clear all versions', () => {
-    vm.snapshot('1.0.0', 'content');
-    vm.clear();
-    expect(vm.list()).toHaveLength(0);
-  });
-
-  it('should persist to file', () => {
-    vm.snapshot('1.0.0', 'persisted');
-    const vm2 = new VersionManager(TEST_STORE);
-    expect(vm2.list()).toHaveLength(1);
-  });
-});
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { VersionManager } from '../src/core/versioning';
+import * as fs from 'fs';
+
+const TEST_STORE = '.test-versions.json';
+
+describe('VersionManager', () => {
+  let vm: VersionManager;
+
+  beforeEach(() => {
+    if (fs.existsSync(TEST_STORE)) fs.unlinkSync(TEST_STORE);
+    vm = new VersionManager(TEST_STORE);
+  });
+
+  afterEach(() => {
+    if (fs.existsSync(TEST_STORE)) fs.unlinkSync(TEST_STORE);
+  });
+
+  it('should save and list versions', () => {
+    vm.snapshot('1.0.0', 'yaml-content-1', 'Initial');
+    vm.snapshot('1.1.0', 'yaml-content-2', 'Update');
+    const versions = vm.list();
+    expect(versions).toHaveLength(2);
+    expect(versions[0].version).toBe('1.0.0');
+  });
+
+  it('should get specific version', () => {
+    vm.snapshot('1.0.0', 'content', 'Test');
+    expect(vm.get('1.0.0')).toBeDefined();
+    expect(vm.get('9.9.9')).toBeUndefined();
+  });
+
+  it('should get current version', () => {
+    vm.snapshot('1.0.0', 'v1');
+    vm.snapshot('2.0.0', 'v2');
+    expect(vm.getCurrent()?.version).toBe('2.0.0');
+  });
+
+  it('should rollback to version', () => {
+    vm.snapshot('1.0.0', 'old-yaml');
+    vm.snapshot('2.0.0', 'new-yaml');
+    const rolled = vm.rollback('1.0.0');
+    expect(rolled).toBe('old-yaml');
+  });
+
+  it('should return null for unknown rollback', () => {
+    expect(vm.rollback('9.9.9')).toBeNull();
+  });
+
+  it('should run migrations', () => {
+    vm.registerMigration({
+      fromVersion: '1.0.0',
+      toVersion: '2.0.0',
+      migrate: (oad) => ({ ...oad, upgraded: true }),
+    });
+    const result = vm.migrate({ data: 'test' }, '1.0.0', '2.0.0');
+    expect((result as any).upgraded).toBe(true);
+  });
+
+  it('should throw on missing migration path', () => {
+    expect(() => vm.migrate({}, '1.0.0', '5.0.0')).toThrow('No migration path');
+  });
+
+  it('should clear all versions', () => {
+    vm.snapshot('1.0.0', 'content');
+    vm.clear();
+    expect(vm.list()).toHaveLength(0);
+  });
+
+  it('should persist to file', () => {
+    vm.snapshot('1.0.0', 'persisted');
+    const vm2 = new VersionManager(TEST_STORE);
+    expect(vm2.list()).toHaveLength(1);
+  });
+});