opc-agent 4.1.0 → 4.1.1

package/tests/ide-bridge.test.ts

@@ -1,38 +1,38 @@
-import { describe, it, expect } from 'vitest';
-import { IDEBridge } from '../src/core/ide-bridge';
-
-describe('IDEBridge', () => {
-  it('should create with vscode config', () => {
-    const bridge = new IDEBridge({ editor: 'vscode' });
-    expect(bridge).toBeInstanceOf(IDEBridge);
-  });
-
-  it('should create with jetbrains config', () => {
-    const bridge = new IDEBridge({ editor: 'jetbrains', workspacePath: '/tmp' });
-    expect(bridge).toBeInstanceOf(IDEBridge);
-  });
-
-  it('getDiagnostics returns empty array (stub)', async () => {
-    const bridge = new IDEBridge({ editor: 'vscode' });
-    const diags = await bridge.getDiagnostics();
-    expect(diags).toEqual([]);
-  });
-
-  it('getOpenFiles returns empty array (stub)', async () => {
-    const bridge = new IDEBridge({ editor: 'zed' });
-    const files = await bridge.getOpenFiles();
-    expect(files).toEqual([]);
-  });
-
-  it('getSelection returns null (stub)', async () => {
-    const bridge = new IDEBridge({ editor: 'cursor' });
-    const sel = await bridge.getSelection();
-    expect(sel).toBeNull();
-  });
-
-  it('applyEdit throws for non-empty edits (stub)', async () => {
-    const bridge = new IDEBridge({ editor: 'vscode' });
-    await expect(bridge.applyEdit('test.ts', [{ range: { startLine: 1, startColumn: 0, endLine: 1, endColumn: 5 }, newText: 'hi' }]))
-      .rejects.toThrow('extension');
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { IDEBridge } from '../src/core/ide-bridge';
+
+describe('IDEBridge', () => {
+  it('should create with vscode config', () => {
+    const bridge = new IDEBridge({ editor: 'vscode' });
+    expect(bridge).toBeInstanceOf(IDEBridge);
+  });
+
+  it('should create with jetbrains config', () => {
+    const bridge = new IDEBridge({ editor: 'jetbrains', workspacePath: '/tmp' });
+    expect(bridge).toBeInstanceOf(IDEBridge);
+  });
+
+  it('getDiagnostics returns empty array (stub)', async () => {
+    const bridge = new IDEBridge({ editor: 'vscode' });
+    const diags = await bridge.getDiagnostics();
+    expect(diags).toEqual([]);
+  });
+
+  it('getOpenFiles returns empty array (stub)', async () => {
+    const bridge = new IDEBridge({ editor: 'zed' });
+    const files = await bridge.getOpenFiles();
+    expect(files).toEqual([]);
+  });
+
+  it('getSelection returns null (stub)', async () => {
+    const bridge = new IDEBridge({ editor: 'cursor' });
+    const sel = await bridge.getSelection();
+    expect(sel).toBeNull();
+  });
+
+  it('applyEdit throws for non-empty edits (stub)', async () => {
+    const bridge = new IDEBridge({ editor: 'vscode' });
+    await expect(bridge.applyEdit('test.ts', [{ range: { startLine: 1, startColumn: 0, endLine: 1, endColumn: 5 }, newText: 'hi' }]))
+      .rejects.toThrow('extension');
+  });
+});

package/tests/image-generator.test.ts

@@ -1,84 +1,84 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { ImageGenerator } from '../src/tools/image-generator';
-
-describe('ImageGenerator', () => {
-  describe('detectProvider', () => {
-    it('returns null when no keys configured', () => {
-      const gen = new ImageGenerator({});
-      expect(gen.detectProvider()).toBeNull();
-    });
-
-    it('detects dalle when openai key set', () => {
-      const gen = new ImageGenerator({ openaiApiKey: 'sk-test' });
-      expect(gen.detectProvider()).toBe('dalle');
-    });
-
-    it('detects stable-diffusion when SD URL set', () => {
-      const gen = new ImageGenerator({ sdApiUrl: 'http://localhost:7860' });
-      expect(gen.detectProvider()).toBe('stable-diffusion');
-    });
-
-    it('detects replicate when token set', () => {
-      const gen = new ImageGenerator({ replicateApiKey: 'r8_test' });
-      expect(gen.detectProvider()).toBe('replicate');
-    });
-
-    it('prefers dalle over others', () => {
-      const gen = new ImageGenerator({ openaiApiKey: 'sk-test', sdApiUrl: 'http://localhost:7860' });
-      expect(gen.detectProvider()).toBe('dalle');
-    });
-  });
-
-  describe('getStatus', () => {
-    it('returns provider status', () => {
-      const gen = new ImageGenerator({ openaiApiKey: 'sk-test' });
-      const status = gen.getStatus();
-      expect(status.configured).toBe(true);
-      expect(status.providers).toHaveLength(3);
-      expect(status.providers.find(p => p.name === 'dalle')?.configured).toBe(true);
-    });
-
-    it('shows unconfigured when no keys', () => {
-      const gen = new ImageGenerator({});
-      const status = gen.getStatus();
-      expect(status.configured).toBe(false);
-    });
-  });
-
-  describe('generate', () => {
-    it('returns error when no provider configured', async () => {
-      const gen = new ImageGenerator({});
-      const result = await gen.generate('a cat');
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('No image generation provider configured');
-    });
-
-    it('returns error for unknown provider', async () => {
-      const gen = new ImageGenerator({});
-      const result = await gen.generate('a cat', { provider: 'unknown' });
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('Unknown provider');
-    });
-
-    it('returns error when dalle key missing', async () => {
-      const gen = new ImageGenerator({});
-      const result = await gen.generate('a cat', { provider: 'dalle' });
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('OPENAI_API_KEY');
-    });
-
-    it('returns error when SD URL missing', async () => {
-      const gen = new ImageGenerator({});
-      const result = await gen.generate('a cat', { provider: 'stable-diffusion' });
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('SD_API_URL');
-    });
-
-    it('returns error when replicate key missing', async () => {
-      const gen = new ImageGenerator({});
-      const result = await gen.generate('a cat', { provider: 'replicate' });
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('REPLICATE_API_TOKEN');
-    });
-  });
-});
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { ImageGenerator } from '../src/tools/image-generator';
+
+describe('ImageGenerator', () => {
+  describe('detectProvider', () => {
+    it('returns null when no keys configured', () => {
+      const gen = new ImageGenerator({});
+      expect(gen.detectProvider()).toBeNull();
+    });
+
+    it('detects dalle when openai key set', () => {
+      const gen = new ImageGenerator({ openaiApiKey: 'sk-test' });
+      expect(gen.detectProvider()).toBe('dalle');
+    });
+
+    it('detects stable-diffusion when SD URL set', () => {
+      const gen = new ImageGenerator({ sdApiUrl: 'http://localhost:7860' });
+      expect(gen.detectProvider()).toBe('stable-diffusion');
+    });
+
+    it('detects replicate when token set', () => {
+      const gen = new ImageGenerator({ replicateApiKey: 'r8_test' });
+      expect(gen.detectProvider()).toBe('replicate');
+    });
+
+    it('prefers dalle over others', () => {
+      const gen = new ImageGenerator({ openaiApiKey: 'sk-test', sdApiUrl: 'http://localhost:7860' });
+      expect(gen.detectProvider()).toBe('dalle');
+    });
+  });
+
+  describe('getStatus', () => {
+    it('returns provider status', () => {
+      const gen = new ImageGenerator({ openaiApiKey: 'sk-test' });
+      const status = gen.getStatus();
+      expect(status.configured).toBe(true);
+      expect(status.providers).toHaveLength(3);
+      expect(status.providers.find(p => p.name === 'dalle')?.configured).toBe(true);
+    });
+
+    it('shows unconfigured when no keys', () => {
+      const gen = new ImageGenerator({});
+      const status = gen.getStatus();
+      expect(status.configured).toBe(false);
+    });
+  });
+
+  describe('generate', () => {
+    it('returns error when no provider configured', async () => {
+      const gen = new ImageGenerator({});
+      const result = await gen.generate('a cat');
+      expect(result.success).toBe(false);
+      expect(result.error).toContain('No image generation provider configured');
+    });
+
+    it('returns error for unknown provider', async () => {
+      const gen = new ImageGenerator({});
+      const result = await gen.generate('a cat', { provider: 'unknown' });
+      expect(result.success).toBe(false);
+      expect(result.error).toContain('Unknown provider');
+    });
+
+    it('returns error when dalle key missing', async () => {
+      const gen = new ImageGenerator({});
+      const result = await gen.generate('a cat', { provider: 'dalle' });
+      expect(result.success).toBe(false);
+      expect(result.error).toContain('OPENAI_API_KEY');
+    });
+
+    it('returns error when SD URL missing', async () => {
+      const gen = new ImageGenerator({});
+      const result = await gen.generate('a cat', { provider: 'stable-diffusion' });
+      expect(result.success).toBe(false);
+      expect(result.error).toContain('SD_API_URL');
+    });
+
+    it('returns error when replicate key missing', async () => {
+      const gen = new ImageGenerator({});
+      const result = await gen.generate('a cat', { provider: 'replicate' });
+      expect(result.success).toBe(false);
+      expect(result.error).toContain('REPLICATE_API_TOKEN');
+    });
+  });
+});

package/tests/init-role.test.ts

@@ -1,124 +1,124 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import * as fs from 'fs';
-import * as path from 'path';
-import { execSync } from 'child_process';
-import { searchRoles, getPopularRoles, getRole, getCategories } from 'agent-workstation';
-
-const CLI_PATH = path.join(__dirname, '..', 'dist', 'cli.js');
-const TMP_DIR = path.join(process.env.TEMP || require('os').tmpdir(), 'opc-test-init-role-' + process.pid);
-
-function run(args: string): string {
-  return execSync(`node "${CLI_PATH}" ${args}`, { cwd: TMP_DIR, encoding: 'utf-8', timeout: 15000 });
-}
-
-describe('opc init --role integration with agent-workstation', () => {
-  beforeEach(() => {
-    if (fs.existsSync(TMP_DIR)) fs.rmSync(TMP_DIR, { recursive: true, force: true });
-    fs.mkdirSync(TMP_DIR, { recursive: true });
-  });
-
-  afterEach(() => {
-    if (fs.existsSync(TMP_DIR)) fs.rmSync(TMP_DIR, { recursive: true, force: true });
-  });
-
-  // --- agent-workstation API tests ---
-
-  it('searchRoles returns results for "customer"', () => {
-    const results = searchRoles('customer');
-    expect(results.length).toBeGreaterThan(0);
-    expect(results[0].category).toBe('customer-service');
-  });
-
-  it('searchRoles returns results for "developer"', () => {
-    const results = searchRoles('developer');
-    expect(results.length).toBeGreaterThan(0);
-    expect(results.some((r: any) => r.role.includes('developer'))).toBe(true);
-  });
-
-  it('searchRoles returns empty for nonexistent role', () => {
-    const results = searchRoles('xyznonexistentrole12345');
-    expect(results.length).toBe(0);
-  });
-
-  it('getPopularRoles returns roles with category and role fields', () => {
-    const roles = getPopularRoles();
-    expect(roles.length).toBeGreaterThan(5);
-    for (const r of roles) {
-      expect(r).toHaveProperty('category');
-      expect(r).toHaveProperty('role');
-    }
-  });
-
-  it('getRole loads full role data with files', () => {
-    const role = getRole('customer-service', 'customer-service-rep');
-    expect(role).toBeTruthy();
-    expect(role.files).toBeTruthy();
-    expect(role.files['system-prompt.md']).toBeTruthy();
-    expect(role.files['brain-seed.md']).toBeTruthy();
-  });
-
-  it('getCategories returns category list', () => {
-    const cats = getCategories();
-    expect(cats.length).toBeGreaterThan(3);
-    expect(cats.some((c: any) => c.name === 'customer-service')).toBe(true);
-  });
-
-  // --- CLI integration tests ---
-
-  it('--list-roles shows available roles', () => {
-    const output = run('init --list-roles');
-    expect(output).toContain('Available workstation roles');
-    expect(output).toContain('customer-service');
-    expect(output).toContain('--role');
-  });
-
-  it('--role customer-service generates SOUL.md from template', () => {
-    const output = run('init test-cs-agent --role customer-service');
-    expect(output).toContain('customer-service');
-    const soulPath = path.join(TMP_DIR, 'test-cs-agent', 'SOUL.md');
-    expect(fs.existsSync(soulPath)).toBe(true);
-    const soul = fs.readFileSync(soulPath, 'utf-8');
-    expect(soul).toContain('Customer Service');
-    expect(soul.split('\n').length).toBeGreaterThan(10);
-  });
-
-  it('--role generates agent.yaml with role systemPrompt', () => {
-    run('init test-yaml-agent --role customer-service');
-    const agentYaml = fs.readFileSync(path.join(TMP_DIR, 'test-yaml-agent', 'agent.yaml'), 'utf-8');
-    expect(agentYaml).toContain('Customer Service');
-    expect(agentYaml).toContain('systemPrompt');
-    expect(agentYaml).toContain('deepbrain');
-  });
-
-  it('--role generates brain-seed when available', () => {
-    run('init test-brain-agent --role customer-service');
-    const brainPath = path.join(TMP_DIR, 'test-brain-agent', 'data', 'brain-seed.md');
-    expect(fs.existsSync(brainPath)).toBe(true);
-    const brain = fs.readFileSync(brainPath, 'utf-8');
-    expect(brain.length).toBeGreaterThan(50);
-  });
-
-  it('--role generates CONTEXT.md with role info', () => {
-    run('init test-ctx-agent --role customer-service');
-    const ctx = fs.readFileSync(path.join(TMP_DIR, 'test-ctx-agent', 'CONTEXT.md'), 'utf-8');
-    expect(ctx).toContain('Customer Service');
-  });
-
-  it('--role with unknown role gives error', () => {
-    expect(() => run('init test-bad --role xyznonexistent123')).toThrow();
-  });
-
-  it('--role with partial match "developer" works', () => {
-    const output = run('init test-dev-agent --role developer');
-    expect(output).toContain('developer');
-    expect(fs.existsSync(path.join(TMP_DIR, 'test-dev-agent', 'SOUL.md'))).toBe(true);
-  });
-
-  it('init without --role still works (generic project)', () => {
-    const output = run('init test-generic -y');
-    expect(output).toContain('Created agent project');
-    expect(output).toContain('Tip');
-    expect(output).toContain('--role');
-    expect(fs.existsSync(path.join(TMP_DIR, 'test-generic', 'SOUL.md'))).toBe(true);
-  });
-});
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import * as fs from 'fs';
+import * as path from 'path';
+import { execSync } from 'child_process';
+import { searchRoles, getPopularRoles, getRole, getCategories } from 'agent-workstation';
+
+const CLI_PATH = path.join(__dirname, '..', 'dist', 'cli.js');
+const TMP_DIR = path.join(process.env.TEMP || require('os').tmpdir(), 'opc-test-init-role-' + process.pid);
+
+function run(args: string): string {
+  return execSync(`node "${CLI_PATH}" ${args}`, { cwd: TMP_DIR, encoding: 'utf-8', timeout: 15000 });
+}
+
+describe('opc init --role integration with agent-workstation', () => {
+  beforeEach(() => {
+    if (fs.existsSync(TMP_DIR)) fs.rmSync(TMP_DIR, { recursive: true, force: true });
+    fs.mkdirSync(TMP_DIR, { recursive: true });
+  });
+
+  afterEach(() => {
+    if (fs.existsSync(TMP_DIR)) fs.rmSync(TMP_DIR, { recursive: true, force: true });
+  });
+
+  // --- agent-workstation API tests ---
+
+  it('searchRoles returns results for "customer"', () => {
+    const results = searchRoles('customer');
+    expect(results.length).toBeGreaterThan(0);
+    expect(results[0].category).toBe('customer-service');
+  });
+
+  it('searchRoles returns results for "developer"', () => {
+    const results = searchRoles('developer');
+    expect(results.length).toBeGreaterThan(0);
+    expect(results.some((r: any) => r.role.includes('developer'))).toBe(true);
+  });
+
+  it('searchRoles returns empty for nonexistent role', () => {
+    const results = searchRoles('xyznonexistentrole12345');
+    expect(results.length).toBe(0);
+  });
+
+  it('getPopularRoles returns roles with category and role fields', () => {
+    const roles = getPopularRoles();
+    expect(roles.length).toBeGreaterThan(5);
+    for (const r of roles) {
+      expect(r).toHaveProperty('category');
+      expect(r).toHaveProperty('role');
+    }
+  });
+
+  it('getRole loads full role data with files', () => {
+    const role = getRole('customer-service', 'customer-service-rep');
+    expect(role).toBeTruthy();
+    expect(role.files).toBeTruthy();
+    expect(role.files['system-prompt.md']).toBeTruthy();
+    expect(role.files['brain-seed.md']).toBeTruthy();
+  });
+
+  it('getCategories returns category list', () => {
+    const cats = getCategories();
+    expect(cats.length).toBeGreaterThan(3);
+    expect(cats.some((c: any) => c.name === 'customer-service')).toBe(true);
+  });
+
+  // --- CLI integration tests ---
+
+  it('--list-roles shows available roles', () => {
+    const output = run('init --list-roles');
+    expect(output).toContain('Available workstation roles');
+    expect(output).toContain('customer-service');
+    expect(output).toContain('--role');
+  });
+
+  it('--role customer-service generates SOUL.md from template', () => {
+    const output = run('init test-cs-agent --role customer-service');
+    expect(output).toContain('customer-service');
+    const soulPath = path.join(TMP_DIR, 'test-cs-agent', 'SOUL.md');
+    expect(fs.existsSync(soulPath)).toBe(true);
+    const soul = fs.readFileSync(soulPath, 'utf-8');
+    expect(soul).toContain('Customer Service');
+    expect(soul.split('\n').length).toBeGreaterThan(10);
+  });
+
+  it('--role generates agent.yaml with role systemPrompt', () => {
+    run('init test-yaml-agent --role customer-service');
+    const agentYaml = fs.readFileSync(path.join(TMP_DIR, 'test-yaml-agent', 'agent.yaml'), 'utf-8');
+    expect(agentYaml).toContain('Customer Service');
+    expect(agentYaml).toContain('systemPrompt');
+    expect(agentYaml).toContain('deepbrain');
+  });
+
+  it('--role generates brain-seed when available', () => {
+    run('init test-brain-agent --role customer-service');
+    const brainPath = path.join(TMP_DIR, 'test-brain-agent', 'data', 'brain-seed.md');
+    expect(fs.existsSync(brainPath)).toBe(true);
+    const brain = fs.readFileSync(brainPath, 'utf-8');
+    expect(brain.length).toBeGreaterThan(50);
+  });
+
+  it('--role generates CONTEXT.md with role info', () => {
+    run('init test-ctx-agent --role customer-service');
+    const ctx = fs.readFileSync(path.join(TMP_DIR, 'test-ctx-agent', 'CONTEXT.md'), 'utf-8');
+    expect(ctx).toContain('Customer Service');
+  });
+
+  it('--role with unknown role gives error', () => {
+    expect(() => run('init test-bad --role xyznonexistent123')).toThrow();
+  });
+
+  it('--role with partial match "developer" works', () => {
+    const output = run('init test-dev-agent --role developer');
+    expect(output).toContain('developer');
+    expect(fs.existsSync(path.join(TMP_DIR, 'test-dev-agent', 'SOUL.md'))).toBe(true);
+  });
+
+  it('init without --role still works (generic project)', () => {
+    const output = run('init test-generic -y');
+    expect(output).toContain('Created agent project');
+    expect(output).toContain('Tip');
+    expect(output).toContain('--role');
+    expect(fs.existsSync(path.join(TMP_DIR, 'test-generic', 'SOUL.md'))).toBe(true);
+  });
+});