opc-agent 1.3.2 → 2.0.0

package/tests/performance.test.ts

@@ -1,115 +1,115 @@
-import { describe, it, expect } from 'vitest';
-import { ConnectionPool, RequestBatcher, LazyLoader } from '../src/core/performance';
-
-describe('ConnectionPool', () => {
-  it('should acquire and release connections', () => {
-    const pool = new ConnectionPool(3);
-    const conn = pool.acquire('openai');
-    expect(conn.inUse).toBe(true);
-    pool.release(conn.id);
-    const stats = pool.getStats();
-    expect(stats['openai'].total).toBe(1);
-    expect(stats['openai'].inUse).toBe(0);
-  });
-
-  it('should reuse released connections', () => {
-    const pool = new ConnectionPool(2);
-    const c1 = pool.acquire('openai');
-    pool.release(c1.id);
-    const c2 = pool.acquire('openai');
-    expect(c2.id).toBe(c1.id);
-  });
-
-  it('should drain all connections', () => {
-    const pool = new ConnectionPool();
-    pool.acquire('openai');
-    pool.acquire('deepseek');
-    pool.drain();
-    expect(pool.getStats()).toEqual({});
-  });
-});
-
-describe('RequestBatcher', () => {
-  it('should batch requests', async () => {
-    const batcher = new RequestBatcher<string>(
-      async (batch) => batch.map(s => s.toUpperCase()),
-      2,
-      10,
-    );
-
-    const [r1, r2] = await Promise.all([
-      batcher.add('hello'),
-      batcher.add('world'),
-    ]);
-    expect(r1).toBe('HELLO');
-    expect(r2).toBe('WORLD');
-  });
-
-  it('should flush on timer', async () => {
-    const batcher = new RequestBatcher<number>(
-      async (batch) => batch.map(n => n * 2),
-      10,
-      20,
-    );
-
-    const result = await batcher.add(5);
-    expect(result).toBe(10);
-  });
-
-  it('should track pending count', () => {
-    const batcher = new RequestBatcher<string>(
-      async (batch) => batch,
-      100,
-      10000,
-    );
-    batcher.add('a');
-    batcher.add('b');
-    expect(batcher.pending).toBe(2);
-  });
-});
-
-describe('LazyLoader', () => {
-  it('should lazily load items', async () => {
-    const loader = new LazyLoader<string>();
-    let loadCount = 0;
-    loader.register('greeting', async () => { loadCount++; return 'hello'; });
-
-    expect(loader.isLoaded('greeting')).toBe(false);
-    const val = await loader.get('greeting');
-    expect(val).toBe('hello');
-    expect(loader.isLoaded('greeting')).toBe(true);
-
-    // Second call should use cache
-    await loader.get('greeting');
-    expect(loadCount).toBe(1);
-  });
-
-  it('should throw for unregistered items', async () => {
-    const loader = new LazyLoader();
-    await expect(loader.get('unknown')).rejects.toThrow('No loader registered');
-  });
-
-  it('should evict and reload', async () => {
-    const loader = new LazyLoader<number>();
-    let count = 0;
-    loader.register('counter', async () => ++count);
-    
-    await loader.get('counter');
-    expect(loader.loadedCount).toBe(1);
-    loader.evict('counter');
-    expect(loader.loadedCount).toBe(0);
-    await loader.get('counter');
-    expect(count).toBe(2);
-  });
-
-  it('should clear all', async () => {
-    const loader = new LazyLoader<string>();
-    loader.register('a', async () => 'a');
-    loader.register('b', async () => 'b');
-    await loader.get('a');
-    await loader.get('b');
-    loader.clear();
-    expect(loader.loadedCount).toBe(0);
-    expect(loader.registeredCount).toBe(2);
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { ConnectionPool, RequestBatcher, LazyLoader } from '../src/core/performance';
+
+describe('ConnectionPool', () => {
+  it('should acquire and release connections', () => {
+    const pool = new ConnectionPool(3);
+    const conn = pool.acquire('openai');
+    expect(conn.inUse).toBe(true);
+    pool.release(conn.id);
+    const stats = pool.getStats();
+    expect(stats['openai'].total).toBe(1);
+    expect(stats['openai'].inUse).toBe(0);
+  });
+
+  it('should reuse released connections', () => {
+    const pool = new ConnectionPool(2);
+    const c1 = pool.acquire('openai');
+    pool.release(c1.id);
+    const c2 = pool.acquire('openai');
+    expect(c2.id).toBe(c1.id);
+  });
+
+  it('should drain all connections', () => {
+    const pool = new ConnectionPool();
+    pool.acquire('openai');
+    pool.acquire('deepseek');
+    pool.drain();
+    expect(pool.getStats()).toEqual({});
+  });
+});
+
+describe('RequestBatcher', () => {
+  it('should batch requests', async () => {
+    const batcher = new RequestBatcher<string>(
+      async (batch) => batch.map(s => s.toUpperCase()),
+      2,
+      10,
+    );
+
+    const [r1, r2] = await Promise.all([
+      batcher.add('hello'),
+      batcher.add('world'),
+    ]);
+    expect(r1).toBe('HELLO');
+    expect(r2).toBe('WORLD');
+  });
+
+  it('should flush on timer', async () => {
+    const batcher = new RequestBatcher<number>(
+      async (batch) => batch.map(n => n * 2),
+      10,
+      20,
+    );
+
+    const result = await batcher.add(5);
+    expect(result).toBe(10);
+  });
+
+  it('should track pending count', () => {
+    const batcher = new RequestBatcher<string>(
+      async (batch) => batch,
+      100,
+      10000,
+    );
+    batcher.add('a');
+    batcher.add('b');
+    expect(batcher.pending).toBe(2);
+  });
+});
+
+describe('LazyLoader', () => {
+  it('should lazily load items', async () => {
+    const loader = new LazyLoader<string>();
+    let loadCount = 0;
+    loader.register('greeting', async () => { loadCount++; return 'hello'; });
+
+    expect(loader.isLoaded('greeting')).toBe(false);
+    const val = await loader.get('greeting');
+    expect(val).toBe('hello');
+    expect(loader.isLoaded('greeting')).toBe(true);
+
+    // Second call should use cache
+    await loader.get('greeting');
+    expect(loadCount).toBe(1);
+  });
+
+  it('should throw for unregistered items', async () => {
+    const loader = new LazyLoader();
+    await expect(loader.get('unknown')).rejects.toThrow('No loader registered');
+  });
+
+  it('should evict and reload', async () => {
+    const loader = new LazyLoader<number>();
+    let count = 0;
+    loader.register('counter', async () => ++count);
+    
+    await loader.get('counter');
+    expect(loader.loadedCount).toBe(1);
+    loader.evict('counter');
+    expect(loader.loadedCount).toBe(0);
+    await loader.get('counter');
+    expect(count).toBe(2);
+  });
+
+  it('should clear all', async () => {
+    const loader = new LazyLoader<string>();
+    loader.register('a', async () => 'a');
+    loader.register('b', async () => 'b');
+    await loader.get('a');
+    await loader.get('b');
+    loader.clear();
+    expect(loader.loadedCount).toBe(0);
+    expect(loader.registeredCount).toBe(2);
+  });
+});

package/tests/plugin.test.ts

@@ -1,74 +1,74 @@
-import { describe, it, expect } from 'vitest';
-import { PluginManager } from '../src/plugins';
-import type { IPlugin } from '../src/plugins';
-
-describe('Plugin System', () => {
-  it('should register and list plugins', () => {
-    const pm = new PluginManager();
-    const plugin: IPlugin = { name: 'test-plugin', version: '1.0.0', description: 'Test' };
-    pm.register(plugin);
-    expect(pm.has('test-plugin')).toBe(true);
-    expect(pm.list().length).toBe(1);
-    expect(pm.list()[0].name).toBe('test-plugin');
-  });
-
-  it('should run lifecycle hooks', async () => {
-    const pm = new PluginManager();
-    const calls: string[] = [];
-
-    pm.register({
-      name: 'hook-plugin',
-      version: '1.0.0',
-      hooks: {
-        beforeInit: async () => { calls.push('beforeInit'); },
-        afterInit: async () => { calls.push('afterInit'); },
-        beforeShutdown: async () => { calls.push('beforeShutdown'); },
-      },
-    });
-
-    await pm.runHook('beforeInit');
-    await pm.runHook('afterInit');
-    await pm.runHook('beforeShutdown');
-    expect(calls).toEqual(['beforeInit', 'afterInit', 'beforeShutdown']);
-  });
-
-  it('should collect skills from plugins', () => {
-    const pm = new PluginManager();
-    pm.register({
-      name: 'skill-plugin',
-      version: '1.0.0',
-      skills: [{
-        name: 'test-skill',
-        description: 'Test',
-        execute: async () => ({ handled: false, confidence: 0 }),
-      }],
-    });
-
-    expect(pm.getAllSkills().length).toBe(1);
-    expect(pm.getAllSkills()[0].name).toBe('test-skill');
-  });
-
-  it('should unregister plugins', () => {
-    const pm = new PluginManager();
-    pm.register({ name: 'temp', version: '1.0.0' });
-    pm.unregister('temp');
-    expect(pm.has('temp')).toBe(false);
-  });
-
-  it('should run hooks from multiple plugins in order', async () => {
-    const pm = new PluginManager();
-    const order: string[] = [];
-
-    pm.register({
-      name: 'p1', version: '1.0.0',
-      hooks: { beforeInit: async () => { order.push('p1'); } },
-    });
-    pm.register({
-      name: 'p2', version: '1.0.0',
-      hooks: { beforeInit: async () => { order.push('p2'); } },
-    });
-
-    await pm.runHook('beforeInit');
-    expect(order).toEqual(['p1', 'p2']);
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { PluginManager } from '../src/plugins';
+import type { IPlugin } from '../src/plugins';
+
+describe('Plugin System', () => {
+  it('should register and list plugins', () => {
+    const pm = new PluginManager();
+    const plugin: IPlugin = { name: 'test-plugin', version: '1.0.0', description: 'Test' };
+    pm.register(plugin);
+    expect(pm.has('test-plugin')).toBe(true);
+    expect(pm.list().length).toBe(1);
+    expect(pm.list()[0].name).toBe('test-plugin');
+  });
+
+  it('should run lifecycle hooks', async () => {
+    const pm = new PluginManager();
+    const calls: string[] = [];
+
+    pm.register({
+      name: 'hook-plugin',
+      version: '1.0.0',
+      hooks: {
+        beforeInit: async () => { calls.push('beforeInit'); },
+        afterInit: async () => { calls.push('afterInit'); },
+        beforeShutdown: async () => { calls.push('beforeShutdown'); },
+      },
+    });
+
+    await pm.runHook('beforeInit');
+    await pm.runHook('afterInit');
+    await pm.runHook('beforeShutdown');
+    expect(calls).toEqual(['beforeInit', 'afterInit', 'beforeShutdown']);
+  });
+
+  it('should collect skills from plugins', () => {
+    const pm = new PluginManager();
+    pm.register({
+      name: 'skill-plugin',
+      version: '1.0.0',
+      skills: [{
+        name: 'test-skill',
+        description: 'Test',
+        execute: async () => ({ handled: false, confidence: 0 }),
+      }],
+    });
+
+    expect(pm.getAllSkills().length).toBe(1);
+    expect(pm.getAllSkills()[0].name).toBe('test-skill');
+  });
+
+  it('should unregister plugins', () => {
+    const pm = new PluginManager();
+    pm.register({ name: 'temp', version: '1.0.0' });
+    pm.unregister('temp');
+    expect(pm.has('temp')).toBe(false);
+  });
+
+  it('should run hooks from multiple plugins in order', async () => {
+    const pm = new PluginManager();
+    const order: string[] = [];
+
+    pm.register({
+      name: 'p1', version: '1.0.0',
+      hooks: { beforeInit: async () => { order.push('p1'); } },
+    });
+    pm.register({
+      name: 'p2', version: '1.0.0',
+      hooks: { beforeInit: async () => { order.push('p2'); } },
+    });
+
+    await pm.runHook('beforeInit');
+    expect(order).toEqual(['p1', 'p2']);
+  });
+});

package/tests/room.test.ts

@@ -1,106 +1,106 @@
-import { describe, it, expect } from 'vitest';
-import { Room } from '../src/core/room';
-import { BaseAgent } from '../src/core/agent';
-
-function makeAgent(name: string): BaseAgent {
-  const agent = new BaseAgent({ name });
-  // Synchronously set to ready by calling init
-  return agent;
-}
-
-describe('Room System', () => {
-  it('should create a room with a name', () => {
-    const room = new Room('test-room');
-    expect(room.name).toBe('test-room');
-    expect(room.getAgents()).toEqual([]);
-  });
-
-  it('should add and remove agents', () => {
-    const room = new Room('office');
-    const agent = makeAgent('agent-1');
-    room.addAgent(agent);
-    expect(room.getAgents()).toEqual(['agent-1']);
-    room.removeAgent('agent-1');
-    expect(room.getAgents()).toEqual([]);
-  });
-
-  it('should emit events on join/leave', () => {
-    const room = new Room('office');
-    const events: string[] = [];
-    room.on('agent:join', (name) => events.push(`join:${name}`));
-    room.on('agent:leave', (name) => events.push(`leave:${name}`));
-
-    const agent = makeAgent('a1');
-    room.addAgent(agent);
-    room.removeAgent('a1');
-    expect(events).toEqual(['join:a1', 'leave:a1']);
-  });
-
-  it('should support topic subscriptions', () => {
-    const room = new Room('office');
-    room.subscribe('agent-1', 'alerts');
-    room.subscribe('agent-2', 'alerts');
-    expect(room.getSubscribers('alerts')).toEqual(['agent-1', 'agent-2']);
-    room.unsubscribe('agent-1', 'alerts');
-    expect(room.getSubscribers('alerts')).toEqual(['agent-2']);
-  });
-
-  it('should broadcast to all agents except sender', async () => {
-    const room = new Room('office');
-    const a1 = makeAgent('a1');
-    const a2 = makeAgent('a2');
-    await a1.init();
-    await a2.init();
-    room.addAgent(a1);
-    room.addAgent(a2);
-
-    const responses = await room.broadcast('a1', 'Hello everyone');
-    expect(responses.length).toBe(1); // only a2 responds
-    expect(responses[0].role).toBe('assistant');
-  });
-
-  it('should send direct messages', async () => {
-    const room = new Room('office');
-    const a1 = makeAgent('a1');
-    const a2 = makeAgent('a2');
-    await a1.init();
-    await a2.init();
-    room.addAgent(a1);
-    room.addAgent(a2);
-
-    const responses = await room.send({
-      from: 'a1',
-      to: 'a2',
-      message: { id: 'm1', role: 'user', content: 'Hi a2', timestamp: Date.now() },
-    });
-    expect(responses.length).toBe(1);
-  });
-
-  it('should publish to topic subscribers only', async () => {
-    const room = new Room('office');
-    const a1 = makeAgent('a1');
-    const a2 = makeAgent('a2');
-    const a3 = makeAgent('a3');
-    await a1.init();
-    await a2.init();
-    await a3.init();
-    room.addAgent(a1);
-    room.addAgent(a2);
-    room.addAgent(a3);
-
-    room.subscribe('a2', 'alerts');
-    // a3 not subscribed
-    const responses = await room.publishToTopic('a1', 'alerts', 'Alert!');
-    expect(responses.length).toBe(1);
-  });
-
-  it('should remove agent from subscriptions on leave', () => {
-    const room = new Room('office');
-    room.subscribe('a1', 'topic1');
-    room.subscribe('a1', 'topic2');
-    expect(room.getSubscribers('topic1')).toContain('a1');
-    room.removeAgent('a1');
-    expect(room.getSubscribers('topic1')).not.toContain('a1');
-    expect(room.getSubscribers('topic2')).not.toContain('a1');
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { Room } from '../src/core/room';
+import { BaseAgent } from '../src/core/agent';
+
+function makeAgent(name: string): BaseAgent {
+  const agent = new BaseAgent({ name });
+  // Synchronously set to ready by calling init
+  return agent;
+}
+
+describe('Room System', () => {
+  it('should create a room with a name', () => {
+    const room = new Room('test-room');
+    expect(room.name).toBe('test-room');
+    expect(room.getAgents()).toEqual([]);
+  });
+
+  it('should add and remove agents', () => {
+    const room = new Room('office');
+    const agent = makeAgent('agent-1');
+    room.addAgent(agent);
+    expect(room.getAgents()).toEqual(['agent-1']);
+    room.removeAgent('agent-1');
+    expect(room.getAgents()).toEqual([]);
+  });
+
+  it('should emit events on join/leave', () => {
+    const room = new Room('office');
+    const events: string[] = [];
+    room.on('agent:join', (name) => events.push(`join:${name}`));
+    room.on('agent:leave', (name) => events.push(`leave:${name}`));
+
+    const agent = makeAgent('a1');
+    room.addAgent(agent);
+    room.removeAgent('a1');
+    expect(events).toEqual(['join:a1', 'leave:a1']);
+  });
+
+  it('should support topic subscriptions', () => {
+    const room = new Room('office');
+    room.subscribe('agent-1', 'alerts');
+    room.subscribe('agent-2', 'alerts');
+    expect(room.getSubscribers('alerts')).toEqual(['agent-1', 'agent-2']);
+    room.unsubscribe('agent-1', 'alerts');
+    expect(room.getSubscribers('alerts')).toEqual(['agent-2']);
+  });
+
+  it('should broadcast to all agents except sender', async () => {
+    const room = new Room('office');
+    const a1 = makeAgent('a1');
+    const a2 = makeAgent('a2');
+    await a1.init();
+    await a2.init();
+    room.addAgent(a1);
+    room.addAgent(a2);
+
+    const responses = await room.broadcast('a1', 'Hello everyone');
+    expect(responses.length).toBe(1); // only a2 responds
+    expect(responses[0].role).toBe('assistant');
+  });
+
+  it('should send direct messages', async () => {
+    const room = new Room('office');
+    const a1 = makeAgent('a1');
+    const a2 = makeAgent('a2');
+    await a1.init();
+    await a2.init();
+    room.addAgent(a1);
+    room.addAgent(a2);
+
+    const responses = await room.send({
+      from: 'a1',
+      to: 'a2',
+      message: { id: 'm1', role: 'user', content: 'Hi a2', timestamp: Date.now() },
+    });
+    expect(responses.length).toBe(1);
+  });
+
+  it('should publish to topic subscribers only', async () => {
+    const room = new Room('office');
+    const a1 = makeAgent('a1');
+    const a2 = makeAgent('a2');
+    const a3 = makeAgent('a3');
+    await a1.init();
+    await a2.init();
+    await a3.init();
+    room.addAgent(a1);
+    room.addAgent(a2);
+    room.addAgent(a3);
+
+    room.subscribe('a2', 'alerts');
+    // a3 not subscribed
+    const responses = await room.publishToTopic('a1', 'alerts', 'Alert!');
+    expect(responses.length).toBe(1);
+  });
+
+  it('should remove agent from subscriptions on leave', () => {
+    const room = new Room('office');
+    room.subscribe('a1', 'topic1');
+    room.subscribe('a1', 'topic2');
+    expect(room.getSubscribers('topic1')).toContain('a1');
+    room.removeAgent('a1');
+    expect(room.getSubscribers('topic1')).not.toContain('a1');
+    expect(room.getSubscribers('topic2')).not.toContain('a1');
+  });
+});