opc-agent 1.1.3 → 1.2.1

package/tests/gateway.test.ts

@@ -0,0 +1,71 @@
+import { describe, it, expect, vi } from 'vitest';
+import { ToolGateway } from '../src/tools/gateway';
+import type { ToolGatewayConfig } from '../src/tools/gateway';
+
+const baseConfig: ToolGatewayConfig = {
+  enabled: true,
+  endpoint: 'https://gateway.example.com',
+  apiKey: 'test-key',
+};
+
+describe('ToolGateway', () => {
+  it('should load default tools when connect fails', async () => {
+    vi.stubGlobal('fetch', vi.fn().mockRejectedValue(new Error('network error')));
+    const gw = new ToolGateway(baseConfig);
+    await gw.connect();
+    expect(gw.isConnected).toBe(false);
+    expect(gw.toolCount).toBe(4);
+    expect(gw.listTools().map((t) => t.name)).toContain('gateway:web-search');
+    vi.unstubAllGlobals();
+  });
+
+  it('should filter tools by enabledTools config', async () => {
+    vi.stubGlobal('fetch', vi.fn().mockRejectedValue(new Error('fail')));
+    const gw = new ToolGateway({ ...baseConfig, enabledTools: ['web-search', 'tts'] });
+    await gw.connect();
+    expect(gw.toolCount).toBe(2);
+    vi.unstubAllGlobals();
+  });
+
+  it('should parse gateway discovery response', async () => {
+    vi.stubGlobal('fetch', vi.fn().mockResolvedValue({
+      ok: true,
+      json: async () => ({
+        tools: [
+          { name: 'web-search', description: 'Search', inputSchema: {}, available: true },
+        ],
+      }),
+    }));
+    const gw = new ToolGateway(baseConfig);
+    await gw.connect();
+    expect(gw.isConnected).toBe(true);
+    expect(gw.toolCount).toBe(1);
+    vi.unstubAllGlobals();
+  });
+
+  it('should return MCPTool instances from getTools()', async () => {
+    vi.stubGlobal('fetch', vi.fn().mockRejectedValue(new Error('fail')));
+    const gw = new ToolGateway(baseConfig);
+    await gw.connect();
+    const tools = gw.getTools();
+    expect(tools.length).toBe(4);
+    expect(tools[0]).toHaveProperty('execute');
+    expect(tools[0]).toHaveProperty('name');
+    vi.unstubAllGlobals();
+  });
+
+  it('should handle invoke errors gracefully', async () => {
+    vi.stubGlobal('fetch', vi.fn().mockRejectedValue(new Error('timeout')));
+    const gw = new ToolGateway(baseConfig);
+    const result = await gw.invokeTool('web-search', { query: 'test' });
+    expect(result.isError).toBe(true);
+    expect(result.content).toContain('timeout');
+    vi.unstubAllGlobals();
+  });
+
+  it('should not discover tools when disabled', async () => {
+    const gw = new ToolGateway({ ...baseConfig, enabled: false });
+    await gw.connect();
+    expect(gw.toolCount).toBe(0);
+  });
+});

package/tests/hitl.test.ts

@@ -1,71 +1,71 @@
-import { describe, it, expect, beforeEach } from 'vitest';
-import { HITLManager } from '../src/core/hitl';
-
-describe('HITLManager', () => {
-  let hitl: HITLManager;
-
-  beforeEach(() => {
-    hitl = new HITLManager({
-      requireApproval: ['delete', 'deploy'],
-      defaultTimeoutMs: 500,
-      defaultAction: 'deny',
-    });
-  });
-
-  it('should check if action needs approval', () => {
-    expect(hitl.needsApproval('delete')).toBe(true);
-    expect(hitl.needsApproval('read')).toBe(false);
-  });
-
-  it('should approve via handler', async () => {
-    hitl.setHandler(async (req) => ({
-      requestId: req.id,
-      decision: 'approve',
-      respondedAt: Date.now(),
-      timedOut: false,
-    }));
-
-    const response = await hitl.requestApproval('delete', 'Delete record #123');
-    expect(response.decision).toBe('approve');
-    expect(response.timedOut).toBe(false);
-  });
-
-  it('should deny via handler', async () => {
-    hitl.setHandler(async (req) => ({
-      requestId: req.id,
-      decision: 'deny',
-      respondedAt: Date.now(),
-      timedOut: false,
-    }));
-
-    const response = await hitl.requestApproval('deploy', 'Deploy to production');
-    expect(response.decision).toBe('deny');
-  });
-
-  it('should timeout with default action', async () => {
-    // No handler, no manual response → timeout
-    const response = await hitl.requestApproval('delete', 'Test timeout');
-    expect(response.timedOut).toBe(true);
-    expect(response.decision).toBe('deny'); // default action
-  });
-
-  it('should handle manual respond', async () => {
-    const promise = hitl.requestApproval('delete', 'Manual test');
-    const pending = hitl.getPending();
-    expect(pending).toHaveLength(1);
-    
-    hitl.respond(pending[0].id, 'approve', 'admin');
-    const response = await promise;
-    expect(response.decision).toBe('approve');
-    expect(response.respondedBy).toBe('admin');
-  });
-
-  it('should return false for unknown respond', () => {
-    expect(hitl.respond('unknown-id', 'approve')).toBe(false);
-  });
-
-  it('should match wildcard approval', () => {
-    const wildcard = new HITLManager({ requireApproval: ['*'] });
-    expect(wildcard.needsApproval('anything')).toBe(true);
-  });
-});
+import { describe, it, expect, beforeEach } from 'vitest';
+import { HITLManager } from '../src/core/hitl';
+
+describe('HITLManager', () => {
+  let hitl: HITLManager;
+
+  beforeEach(() => {
+    hitl = new HITLManager({
+      requireApproval: ['delete', 'deploy'],
+      defaultTimeoutMs: 500,
+      defaultAction: 'deny',
+    });
+  });
+
+  it('should check if action needs approval', () => {
+    expect(hitl.needsApproval('delete')).toBe(true);
+    expect(hitl.needsApproval('read')).toBe(false);
+  });
+
+  it('should approve via handler', async () => {
+    hitl.setHandler(async (req) => ({
+      requestId: req.id,
+      decision: 'approve',
+      respondedAt: Date.now(),
+      timedOut: false,
+    }));
+
+    const response = await hitl.requestApproval('delete', 'Delete record #123');
+    expect(response.decision).toBe('approve');
+    expect(response.timedOut).toBe(false);
+  });
+
+  it('should deny via handler', async () => {
+    hitl.setHandler(async (req) => ({
+      requestId: req.id,
+      decision: 'deny',
+      respondedAt: Date.now(),
+      timedOut: false,
+    }));
+
+    const response = await hitl.requestApproval('deploy', 'Deploy to production');
+    expect(response.decision).toBe('deny');
+  });
+
+  it('should timeout with default action', async () => {
+    // No handler, no manual response → timeout
+    const response = await hitl.requestApproval('delete', 'Test timeout');
+    expect(response.timedOut).toBe(true);
+    expect(response.decision).toBe('deny'); // default action
+  });
+
+  it('should handle manual respond', async () => {
+    const promise = hitl.requestApproval('delete', 'Manual test');
+    const pending = hitl.getPending();
+    expect(pending).toHaveLength(1);
+    
+    hitl.respond(pending[0].id, 'approve', 'admin');
+    const response = await promise;
+    expect(response.decision).toBe('approve');
+    expect(response.respondedBy).toBe('admin');
+  });
+
+  it('should return false for unknown respond', () => {
+    expect(hitl.respond('unknown-id', 'approve')).toBe(false);
+  });
+
+  it('should match wildcard approval', () => {
+    const wildcard = new HITLManager({ requireApproval: ['*'] });
+    expect(wildcard.needsApproval('anything')).toBe(true);
+  });
+});

package/tests/i18n.test.ts

@@ -1,41 +1,41 @@
-import { describe, it, expect } from 'vitest';
-import { t, setLocale, getLocale, detectLocale, addMessages } from '../src/i18n';
-
-describe('i18n', () => {
-  it('should return English messages by default', () => {
-    setLocale('en');
-    expect(t('agent.greeting')).toBe('Hello! How can I help you?');
-  });
-
-  it('should return Chinese messages', () => {
-    setLocale('zh-CN');
-    expect(t('agent.greeting')).toBe('你好！有什么可以帮你的？');
-    setLocale('en'); // reset
-  });
-
-  it('should interpolate params', () => {
-    setLocale('en');
-    expect(t('agent.started', { name: 'TestBot' })).toBe('Agent "TestBot" started successfully');
-  });
-
-  it('should fall back to key if not found', () => {
-    expect(t('nonexistent.key')).toBe('nonexistent.key');
-  });
-
-  it('should get and set locale', () => {
-    setLocale('zh-CN');
-    expect(getLocale()).toBe('zh-CN');
-    setLocale('en');
-    expect(getLocale()).toBe('en');
-  });
-
-  it('should add custom messages', () => {
-    addMessages('en', { 'custom.key': 'Custom value' });
-    expect(t('custom.key')).toBe('Custom value');
-  });
-
-  it('should detect locale from environment', () => {
-    const locale = detectLocale();
-    expect(['en', 'zh-CN']).toContain(locale);
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { t, setLocale, getLocale, detectLocale, addMessages } from '../src/i18n';
+
+describe('i18n', () => {
+  it('should return English messages by default', () => {
+    setLocale('en');
+    expect(t('agent.greeting')).toBe('Hello! How can I help you?');
+  });
+
+  it('should return Chinese messages', () => {
+    setLocale('zh-CN');
+    expect(t('agent.greeting')).toBe('你好！有什么可以帮你的？');
+    setLocale('en'); // reset
+  });
+
+  it('should interpolate params', () => {
+    setLocale('en');
+    expect(t('agent.started', { name: 'TestBot' })).toBe('Agent "TestBot" started successfully');
+  });
+
+  it('should fall back to key if not found', () => {
+    expect(t('nonexistent.key')).toBe('nonexistent.key');
+  });
+
+  it('should get and set locale', () => {
+    setLocale('zh-CN');
+    expect(getLocale()).toBe('zh-CN');
+    setLocale('en');
+    expect(getLocale()).toBe('en');
+  });
+
+  it('should add custom messages', () => {
+    addMessages('en', { 'custom.key': 'Custom value' });
+    expect(t('custom.key')).toBe('Custom value');
+  });
+
+  it('should detect locale from environment', () => {
+    const locale = detectLocale();
+    expect(['en', 'zh-CN']).toContain(locale);
+  });
+});

package/tests/mcp.test.ts

@@ -1,54 +1,54 @@
-import { describe, it, expect } from 'vitest';
-import { MCPToolRegistry, createMCPTool } from '../src/tools/mcp';
-
-describe('MCP Tool System', () => {
-  it('should register and list tools', () => {
-    const registry = new MCPToolRegistry();
-    const tool = createMCPTool('calculator', 'Basic calculator', {
-      type: 'object',
-      properties: { expression: { type: 'string' } },
-    }, async (input) => ({ content: `Result: ${input.expression}` }));
-
-    registry.register(tool);
-    expect(registry.has('calculator')).toBe(true);
-    expect(registry.list().length).toBe(1);
-    expect(registry.list()[0].name).toBe('calculator');
-  });
-
-  it('should execute a tool', async () => {
-    const registry = new MCPToolRegistry();
-    registry.register(createMCPTool('echo', 'Echo tool', {}, async (input) => ({
-      content: `Echo: ${input.text}`,
-    })));
-
-    const result = await registry.execute('echo', { text: 'hello' });
-    expect(result.content).toBe('Echo: hello');
-    expect(result.isError).toBeUndefined();
-  });
-
-  it('should return error for missing tool', async () => {
-    const registry = new MCPToolRegistry();
-    const result = await registry.execute('nonexistent', {});
-    expect(result.isError).toBe(true);
-    expect(result.content).toContain('not found');
-  });
-
-  it('should handle tool execution errors', async () => {
-    const registry = new MCPToolRegistry();
-    registry.register(createMCPTool('failing', 'Fails', {}, async () => {
-      throw new Error('boom');
-    }));
-
-    const result = await registry.execute('failing', {});
-    expect(result.isError).toBe(true);
-    expect(result.content).toContain('boom');
-  });
-
-  it('should unregister tools', () => {
-    const registry = new MCPToolRegistry();
-    registry.register(createMCPTool('temp', 'Temp', {}, async () => ({ content: '' })));
-    expect(registry.has('temp')).toBe(true);
-    registry.unregister('temp');
-    expect(registry.has('temp')).toBe(false);
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { MCPToolRegistry, createMCPTool } from '../src/tools/mcp';
+
+describe('MCP Tool System', () => {
+  it('should register and list tools', () => {
+    const registry = new MCPToolRegistry();
+    const tool = createMCPTool('calculator', 'Basic calculator', {
+      type: 'object',
+      properties: { expression: { type: 'string' } },
+    }, async (input) => ({ content: `Result: ${input.expression}` }));
+
+    registry.register(tool);
+    expect(registry.has('calculator')).toBe(true);
+    expect(registry.list().length).toBe(1);
+    expect(registry.list()[0].name).toBe('calculator');
+  });
+
+  it('should execute a tool', async () => {
+    const registry = new MCPToolRegistry();
+    registry.register(createMCPTool('echo', 'Echo tool', {}, async (input) => ({
+      content: `Echo: ${input.text}`,
+    })));
+
+    const result = await registry.execute('echo', { text: 'hello' });
+    expect(result.content).toBe('Echo: hello');
+    expect(result.isError).toBeUndefined();
+  });
+
+  it('should return error for missing tool', async () => {
+    const registry = new MCPToolRegistry();
+    const result = await registry.execute('nonexistent', {});
+    expect(result.isError).toBe(true);
+    expect(result.content).toContain('not found');
+  });
+
+  it('should handle tool execution errors', async () => {
+    const registry = new MCPToolRegistry();
+    registry.register(createMCPTool('failing', 'Fails', {}, async () => {
+      throw new Error('boom');
+    }));
+
+    const result = await registry.execute('failing', {});
+    expect(result.isError).toBe(true);
+    expect(result.content).toContain('boom');
+  });
+
+  it('should unregister tools', () => {
+    const registry = new MCPToolRegistry();
+    registry.register(createMCPTool('temp', 'Temp', {}, async () => ({ content: '' })));
+    expect(registry.has('temp')).toBe(true);
+    registry.unregister('temp');
+    expect(registry.has('temp')).toBe(false);
+  });
+});

package/tests/oad.test.ts

@@ -1,68 +1,68 @@
-import { describe, it, expect } from 'vitest';
-import { OADSchema } from '../src/schema/oad';
-
-describe('OAD Schema', () => {
-  it('should parse a valid OAD document', () => {
-    const doc = OADSchema.parse({
-      apiVersion: 'opc/v1',
-      kind: 'Agent',
-      metadata: { name: 'test-agent' },
-      spec: {},
-    });
-    expect(doc.metadata.name).toBe('test-agent');
-    expect(doc.metadata.version).toBe('1.0.0');
-    expect(doc.spec.model).toBe('deepseek-chat');
-  });
-
-  it('should reject invalid apiVersion', () => {
-    expect(() =>
-      OADSchema.parse({
-        apiVersion: 'v2',
-        kind: 'Agent',
-        metadata: { name: 'test' },
-        spec: {},
-      })
-    ).toThrow();
-  });
-
-  it('should parse full document with all fields', () => {
-    const doc = OADSchema.parse({
-      apiVersion: 'opc/v1',
-      kind: 'Agent',
-      metadata: {
-        name: 'full-agent',
-        version: '2.0.0',
-        description: 'A full agent',
-        author: 'Test',
-        license: 'MIT',
-        marketplace: { certified: true, category: 'support' },
-      },
-      spec: {
-        provider: { default: 'openai', allowed: ['openai'] },
-        model: 'gpt-4',
-        systemPrompt: 'You are helpful.',
-        skills: [{ name: 'faq', description: 'FAQ skill' }],
-        channels: [{ type: 'web', port: 8080 }],
-        memory: { shortTerm: true, longTerm: true },
-        dtv: {
-          trust: { level: 'certified' },
-          value: { metrics: ['response_time'] },
-        },
-      },
-    });
-    expect(doc.metadata.version).toBe('2.0.0');
-    expect(doc.spec.channels[0].port).toBe(8080);
-    expect(doc.spec.dtv?.trust?.level).toBe('certified');
-  });
-
-  it('should reject missing metadata.name', () => {
-    expect(() =>
-      OADSchema.parse({
-        apiVersion: 'opc/v1',
-        kind: 'Agent',
-        metadata: {},
-        spec: {},
-      })
-    ).toThrow();
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { OADSchema } from '../src/schema/oad';
+
+describe('OAD Schema', () => {
+  it('should parse a valid OAD document', () => {
+    const doc = OADSchema.parse({
+      apiVersion: 'opc/v1',
+      kind: 'Agent',
+      metadata: { name: 'test-agent' },
+      spec: {},
+    });
+    expect(doc.metadata.name).toBe('test-agent');
+    expect(doc.metadata.version).toBe('1.0.0');
+    expect(doc.spec.model).toBe('deepseek-chat');
+  });
+
+  it('should reject invalid apiVersion', () => {
+    expect(() =>
+      OADSchema.parse({
+        apiVersion: 'v2',
+        kind: 'Agent',
+        metadata: { name: 'test' },
+        spec: {},
+      })
+    ).toThrow();
+  });
+
+  it('should parse full document with all fields', () => {
+    const doc = OADSchema.parse({
+      apiVersion: 'opc/v1',
+      kind: 'Agent',
+      metadata: {
+        name: 'full-agent',
+        version: '2.0.0',
+        description: 'A full agent',
+        author: 'Test',
+        license: 'MIT',
+        marketplace: { certified: true, category: 'support' },
+      },
+      spec: {
+        provider: { default: 'openai', allowed: ['openai'] },
+        model: 'gpt-4',
+        systemPrompt: 'You are helpful.',
+        skills: [{ name: 'faq', description: 'FAQ skill' }],
+        channels: [{ type: 'web', port: 8080 }],
+        memory: { shortTerm: true, longTerm: true },
+        dtv: {
+          trust: { level: 'certified' },
+          value: { metrics: ['response_time'] },
+        },
+      },
+    });
+    expect(doc.metadata.version).toBe('2.0.0');
+    expect(doc.spec.channels[0].port).toBe(8080);
+    expect(doc.spec.dtv?.trust?.level).toBe('certified');
+  });
+
+  it('should reject missing metadata.name', () => {
+    expect(() =>
+      OADSchema.parse({
+        apiVersion: 'opc/v1',
+        kind: 'Agent',
+        metadata: {},
+        spec: {},
+      })
+    ).toThrow();
+  });
+});