@doingdev/opencode-claude-manager-plugin 0.1.64 → 0.1.66

package/dist/test/report-claude-event.test.js

@@ -1,304 +0,0 @@
-/**
- * Tests for reportClaudeEvent via the real plugin onEvent chain,
- * plus integration tests for second-invocation continuity across
- * clearPluginServices() / new plugin instance.
- */
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import { mkdtemp, rm } from 'node:fs/promises';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import { ClaudeManagerPlugin } from '../src/plugin/claude-manager.plugin.js';
-import { clearPluginServices, getOrCreatePluginServices, getSessionTeam, registerParentSession, } from '../src/plugin/service-factory.js';
-import { AGENT_CTO, ENGINEER_AGENT_IDS } from '../src/plugin/agent-hierarchy.js';
-import { TeamStateStore } from '../src/state/team-state-store.js';
-import { TeamOrchestrator } from '../src/manager/team-orchestrator.js';
-const BROWSER_QA_TEST_CAPS = {
-    sessionPrompt: 'Browser QA prompt',
-    restrictWriteTools: true,
-    skipModeInstructions: true,
-    plannerEligible: false,
-    isRuntimeUnavailableResponse: (text) => text.trimStart().startsWith('PLAYWRIGHT_UNAVAILABLE:'),
-    runtimeUnavailableTitle: '❌ Playwright unavailable',
-};
-function makeContext(worktree, agentId, sessionID) {
-    const metadata = vi.fn();
-    const ctx = {
-        metadata,
-        worktree,
-        sessionID,
-        agent: agentId,
-        abort: new AbortController().signal,
-    };
-    return { metadata, ctx };
-}
-function makeDispatchResult(override = {}) {
-    const context = {
-        sessionId: 'ses-tom-1',
-        totalTurns: 1,
-        totalCostUsd: 0.01,
-        latestInputTokens: 500,
-        latestOutputTokens: 100,
-        contextWindowSize: 200_000,
-        estimatedContextPercent: 0.5,
-        warningLevel: 'ok',
-        compactionCount: 0,
-    };
-    return {
-        teamId: 'team-1',
-        engineer: 'Tom',
-        mode: 'explore',
-        sessionId: 'ses-tom-1',
-        finalText: 'done',
-        turns: 1,
-        totalCostUsd: 0.01,
-        inputTokens: 500,
-        outputTokens: 100,
-        contextWindowSize: 200_000,
-        context,
-        ...override,
-    };
-}
-async function executeClaude(plugin, ctx, args = { mode: 'explore', message: 'do work' }) {
-    const claudeTool = plugin.tool['claude'];
-    return claudeTool.execute(args, ctx);
-}
-// ── reportClaudeEvent via the plugin's onEvent chain ────────────────────────
-describe('reportClaudeEvent — via plugin onEvent chain', () => {
-    let tempRoot;
-    beforeEach(async () => {
-        tempRoot = await mkdtemp(join(tmpdir(), 'report-event-'));
-        clearPluginServices();
-    });
-    afterEach(async () => {
-        clearPluginServices();
-        if (tempRoot)
-            await rm(tempRoot, { recursive: true, force: true });
-    });
-    /**
-     * Helper: creates a plugin, sets the active CTO team, then stubs
-     * dispatchEngineer so it fires the given events before returning.
-     */
-    async function setupPlugin(events) {
-        const plugin = await ClaudeManagerPlugin({ worktree: tempRoot });
-        const chatMessage = plugin['chat.message'];
-        await chatMessage({ agent: AGENT_CTO, sessionID: 'cto-1' });
-        const services = getOrCreatePluginServices(tempRoot);
-        vi.spyOn(services.orchestrator, 'dispatchEngineer').mockImplementation(async (input) => {
-            for (const event of events) {
-                await input.onEvent?.(event);
-            }
-            return makeDispatchResult();
-        });
-        vi.spyOn(services.orchestrator, 'recordWrapperExchange').mockResolvedValue(undefined);
-        return { plugin, services };
-    }
-    it('surfaces tool name, args, and toolId from a tool_call event', async () => {
-        const event = {
-            type: 'tool_call',
-            sessionId: 'ses-1',
-            text: JSON.stringify({ name: 'read', id: 'call-abc', input: { file_path: '/foo.ts' } }),
-        };
-        const { plugin } = await setupPlugin([event]);
-        const { metadata, ctx } = makeContext(tempRoot, ENGINEER_AGENT_IDS.Tom, 'wrapper-1');
-        await executeClaude(plugin, ctx);
-        const call = metadata.mock.calls.find(([c]) => c?.title?.includes('→'))?.[0];
-        expect(call).toBeDefined();
-        expect(call.title).toBe('⚡ Tom → Reading: /foo.ts');
-        expect(call.metadata.toolName).toBe('read');
-        expect(call.metadata.toolId).toBe('call-abc');
-        expect(call.metadata.toolArgs).toEqual({ file_path: '/foo.ts' });
-        expect(call.metadata.sessionId).toBe('ses-1');
-        expect(call.metadata.workerName).toBe('Tom');
-    });
-    it('double-decodes a JSON-string input (tool input serialised twice)', async () => {
-        // The SDK adapter may serialize `input` as a JSON string inside the outer JSON on
-        // some tool calls. The handler should parse the inner string into an object.
-        const event = {
-            type: 'tool_call',
-            sessionId: 'ses-2',
-            text: JSON.stringify({
-                name: 'bash',
-                id: 'call-def',
-                input: JSON.stringify({ command: 'ls -la' }),
-            }),
-        };
-        const { plugin } = await setupPlugin([event]);
-        const { metadata, ctx } = makeContext(tempRoot, ENGINEER_AGENT_IDS.Tom, 'wrapper-2');
-        await executeClaude(plugin, ctx);
-        const call = metadata.mock.calls.find(([c]) => c?.title?.includes('→'))?.[0];
-        expect(call.title).toBe('⚡ Tom → Running: ls -la');
-        expect(call.metadata.toolArgs).toEqual({ command: 'ls -la' });
-    });
-    it('falls back to generic title and omits tool fields when event.text is not JSON', async () => {
-        const event = {
-            type: 'tool_call',
-            sessionId: 'ses-3',
-            text: 'not-json-at-all',
-        };
-        const { plugin } = await setupPlugin([event]);
-        const { metadata, ctx } = makeContext(tempRoot, ENGINEER_AGENT_IDS.Tom, 'wrapper-3');
-        await executeClaude(plugin, ctx);
-        const call = metadata.mock.calls.find(([c]) => c?.title?.includes('is using Claude Code tools'))?.[0];
-        expect(call).toBeDefined();
-        expect(call.title).toBe('⚡ Tom is using Claude Code tools');
-        expect(call.metadata).not.toHaveProperty('toolName');
-        expect(call.metadata).not.toHaveProperty('toolId');
-        expect(call.metadata).not.toHaveProperty('toolArgs');
-    });
-    it('falls back to generic title when parsed JSON has no name field', async () => {
-        const event = {
-            type: 'tool_call',
-            sessionId: 'ses-4',
-            text: JSON.stringify({ id: 'call-xyz', input: {} }),
-        };
-        const { plugin } = await setupPlugin([event]);
-        const { metadata, ctx } = makeContext(tempRoot, ENGINEER_AGENT_IDS.John, 'wrapper-4');
-        await executeClaude(plugin, ctx);
-        const call = metadata.mock.calls.find(([c]) => c?.title?.includes('is using Claude Code tools'))?.[0];
-        expect(call).toBeDefined();
-        expect(call.metadata).not.toHaveProperty('toolName');
-    });
-    it('includes toolArgs when input is an empty object', async () => {
-        const event = {
-            type: 'tool_call',
-            sessionId: 'ses-5',
-            text: JSON.stringify({ name: 'git_status', id: 'call-ghi', input: {} }),
-        };
-        const { plugin } = await setupPlugin([event]);
-        const { metadata, ctx } = makeContext(tempRoot, ENGINEER_AGENT_IDS.Maya, 'wrapper-5');
-        await executeClaude(plugin, ctx);
-        const call = metadata.mock.calls.find(([c]) => c?.title?.includes('→'))?.[0];
-        expect(call.title).toBe('⚡ Maya → git_status');
-        expect(call.metadata.toolArgs).toEqual({});
-    });
-    it('surfaces status event as visible metadata', async () => {
-        const event = {
-            type: 'status',
-            text: 'Context exhausted; resetting session and retrying once with a fresh session.',
-        };
-        const { plugin } = await setupPlugin([event]);
-        const { metadata, ctx } = makeContext(tempRoot, ENGINEER_AGENT_IDS.Sara, 'wrapper-6');
-        await executeClaude(plugin, ctx);
-        const statusCall = metadata.mock.calls.find(([c]) => c?.title?.includes('ℹ️'))?.[0];
-        expect(statusCall).toBeDefined();
-        expect(statusCall.title).toBe('ℹ️ Sara: Context exhausted; resetting session and retrying once with a fresh session.');
-        expect(statusCall.metadata.status).toBe('Context exhausted; resetting session and retrying once with a fresh session.');
-    });
-});
-// ── Second-invocation continuity ─────────────────────────────────────────────
-describe('second invocation continuity', () => {
-    let tempRoot;
-    beforeEach(async () => {
-        tempRoot = await mkdtemp(join(tmpdir(), 'continuity-'));
-        clearPluginServices();
-    });
-    afterEach(async () => {
-        clearPluginServices();
-        if (tempRoot)
-            await rm(tempRoot, { recursive: true, force: true });
-    });
-    it('wrapper memory is injected after clearPluginServices and same CTO session resumes', async () => {
-        // ── Phase 1: first task via orchestrator (no real SDK needed) ──────────
-        // Team ID = 'cto-1' (the CTO session ID that originally ran this work).
-        const store = new TeamStateStore();
-        const orchestrator = new TeamOrchestrator({ runTask: vi.fn() }, store, { appendEvents: vi.fn(async () => undefined) }, 'Base prompt', 'Synthesis prompt', { BrowserQA: BROWSER_QA_TEST_CAPS });
-        await orchestrator.recordWrapperSession(tempRoot, 'cto-1', 'Tom', 'wrapper-tom-1');
-        await orchestrator.recordWrapperExchange(tempRoot, 'cto-1', 'Tom', 'wrapper-tom-1', 'explore', 'Investigate the auth flow', 'Found two race conditions in the token refresh path.');
-        // ── Phase 2: process restart ───────────────────────────────────────────
-        clearPluginServices();
-        // ── Phase 3: same CTO session resumes, engineers run a new wrapper ──────
-        const plugin2 = await ClaudeManagerPlugin({ worktree: tempRoot });
-        const chatMessage2 = plugin2['chat.message'];
-        const systemTransform2 = plugin2['experimental.chat.system.transform'];
-        // Same CTO session ID resumes — re-registers itself as the team.
-        await chatMessage2({ agent: AGENT_CTO, sessionID: 'cto-1' });
-        expect(getSessionTeam('cto-1')).toBe('cto-1');
-        // Simulate session.created: OpenCode fires this when cto-1 spawns the new wrapper.
-        registerParentSession('wrapper-tom-2', 'cto-1');
-        // Tom's new wrapper session is registered under the same team.
-        await chatMessage2({ agent: ENGINEER_AGENT_IDS.Tom, sessionID: 'wrapper-tom-2' });
-        // Transform fires (after chat.message has registered the session mapping).
-        const output = { system: [] };
-        await systemTransform2({ sessionID: 'wrapper-tom-2', model: 'claude-sonnet-4-6' }, output);
-        expect(output.system).toHaveLength(1);
-        expect(output.system[0]).toContain('Persistent wrapper memory for Tom');
-        expect(output.system[0]).toContain('Investigate the auth flow');
-        expect(output.system[0]).toContain('Found two race conditions');
-    });
-    it('existing engineer Claude session is resumed on second invocation', async () => {
-        // ── Phase 1: pre-seed Tom with a claudeSessionId under team 'cto-1' ────
-        const store = new TeamStateStore();
-        const orchestrator = new TeamOrchestrator({ runTask: vi.fn() }, store, { appendEvents: vi.fn(async () => undefined) }, 'Base prompt', 'Synthesis prompt', { BrowserQA: BROWSER_QA_TEST_CAPS });
-        await orchestrator.getOrCreateTeam(tempRoot, 'cto-1');
-        await store.updateTeam(tempRoot, 'cto-1', (team) => ({
-            ...team,
-            engineers: team.engineers.map((e) => e.name === 'Tom' ? { ...e, claudeSessionId: 'ses-tom-persisted' } : e),
-        }));
-        // ── Phase 2: process restart ───────────────────────────────────────────
-        clearPluginServices();
-        // ── Phase 3: same CTO session resumes, engineer runs second task ───────
-        const plugin2 = await ClaudeManagerPlugin({ worktree: tempRoot });
-        const chatMessage2 = plugin2['chat.message'];
-        // Same CTO session ID — re-registers as the team so Tom can find his session.
-        await chatMessage2({ agent: AGENT_CTO, sessionID: 'cto-1' });
-        // Simulate session.created: OpenCode fires this when cto-1 spawns the new wrapper.
-        registerParentSession('wrapper-tom-2', 'cto-1');
-        await chatMessage2({ agent: ENGINEER_AGENT_IDS.Tom, sessionID: 'wrapper-tom-2' });
-        const services2 = getOrCreatePluginServices(tempRoot);
-        // Mock at the session level so dispatchEngineer runs its real logic
-        // (reads claudeSessionId, passes resumeSessionId to runTask).
-        const runTask = vi.spyOn(services2.sessions, 'runTask').mockResolvedValue({
-            sessionId: 'ses-tom-persisted',
-            events: [],
-            finalText: 'resumed result',
-            turns: 2,
-            totalCostUsd: 0.02,
-            inputTokens: 1000,
-            outputTokens: 200,
-            contextWindowSize: 200_000,
-        });
-        vi.spyOn(services2.orchestrator, 'recordWrapperExchange').mockResolvedValue(undefined);
-        const { ctx } = makeContext(tempRoot, ENGINEER_AGENT_IDS.Tom, 'wrapper-tom-2');
-        await executeClaude(plugin2, ctx);
-        // dispatchEngineer should have read claudeSessionId='ses-tom-persisted' from
-        // the team store and forwarded it as resumeSessionId.
-        expect(runTask).toHaveBeenCalledOnce();
-        expect(runTask.mock.calls[0]?.[0]).toMatchObject({
-            resumeSessionId: 'ses-tom-persisted',
-        });
-        expect(runTask.mock.calls[0]?.[0].systemPrompt).toBeUndefined(); // no system prompt when resuming
-    });
-    it('BrowserQA PLAYWRIGHT_UNAVAILABLE returns plainly and emits warning metadata', async () => {
-        const plugin = await ClaudeManagerPlugin({ worktree: tempRoot });
-        const chatMessage = plugin['chat.message'];
-        // Register BrowserQA session
-        await chatMessage({ agent: ENGINEER_AGENT_IDS.BrowserQA, sessionID: 'wrapper-browserqa-1' });
-        const services = getOrCreatePluginServices(tempRoot);
-        // Mock dispatchEngineer to return PLAYWRIGHT_UNAVAILABLE sentinel
-        const unavailableMessage = 'PLAYWRIGHT_UNAVAILABLE: Playwright is not available in this environment. Consider installing it via npm.';
-        vi.spyOn(services.orchestrator, 'dispatchEngineer').mockResolvedValueOnce(makeDispatchResult({
-            engineer: 'BrowserQA',
-            finalText: unavailableMessage,
-        }));
-        vi.spyOn(services.orchestrator, 'recordWrapperExchange').mockResolvedValue(undefined);
-        const { metadata, ctx } = makeContext(tempRoot, ENGINEER_AGENT_IDS.BrowserQA, 'wrapper-browserqa-1');
-        const result = await executeClaude(plugin, ctx, {
-            mode: 'verify',
-            message: 'Check if browser is available',
-        });
-        // Result should contain the PLAYWRIGHT_UNAVAILABLE text plainly (not wrapped in error)
-        expect(result).toContain('PLAYWRIGHT_UNAVAILABLE');
-        expect(result).toContain('Playwright is not available');
-        // Verify warning metadata was emitted
-        expect(metadata).toHaveBeenCalled();
-        const calls = metadata.mock.calls;
-        const warningCall = calls.find((call) => {
-            const arg = call[0];
-            const title = arg?.title ?? '';
-            return (title.includes('Playwright') ||
-                title.includes('unavailable') ||
-                title.toLowerCase().includes('playwright unavailable'));
-        });
-        expect(warningCall).toBeDefined();
-    });
-});

package/dist/test/session-controller.test.d.ts

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

package/dist/test/session-controller.test.js

@@ -1,149 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { SessionController } from '../src/manager/session-controller.js';
-import { ContextTracker } from '../src/manager/context-tracker.js';
-function createMockAdapter(results = [{}]) {
-    let callIndex = 0;
-    return {
-        runSession: vi.fn(async () => {
-            const base = {
-                sessionId: 'ses_test',
-                events: [],
-                finalText: 'Done.',
-                turns: 1,
-                totalCostUsd: 0.01,
-                inputTokens: 10_000,
-                outputTokens: 500,
-                contextWindowSize: 200_000,
-            };
-            const overrides = results[callIndex] ?? {};
-            callIndex++;
-            return { ...base, ...overrides };
-        }),
-    };
-}
-describe('SessionController', () => {
-    it('creates a new session on first sendMessage', async () => {
-        const adapter = createMockAdapter();
-        const tracker = new ContextTracker();
-        const controller = new SessionController(adapter, tracker, 'test prompt', 'test', '/tmp');
-        expect(controller.isActive).toBe(false);
-        const result = await controller.sendMessage('hello');
-        expect(controller.isActive).toBe(true);
-        expect(controller.sessionId).toBe('ses_test');
-        expect(result.finalText).toBe('Done.');
-        // Verify system prompt was used (no resume)
-        const call = adapter.runSession.mock.calls[0][0];
-        expect(call.systemPrompt).toBe('test prompt');
-        expect(call.resumeSessionId).toBeUndefined();
-    });
-    it('sends settingSources as user-only', async () => {
-        const adapter = createMockAdapter();
-        const tracker = new ContextTracker();
-        const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp');
-        await controller.sendMessage('hello');
-        const call = adapter.runSession.mock.calls[0][0];
-        expect(call.settingSources).toEqual(['user']);
-    });
-    it('resumes session on subsequent sends', async () => {
-        const adapter = createMockAdapter([{}, {}]);
-        const tracker = new ContextTracker();
-        const controller = new SessionController(adapter, tracker, 'test prompt', 'test', '/tmp');
-        await controller.sendMessage('first');
-        await controller.sendMessage('second');
-        const secondCall = adapter.runSession.mock.calls[1][0];
-        expect(secondCall.resumeSessionId).toBe('ses_test');
-        expect(secondCall.systemPrompt).toBeUndefined();
-    });
-    it('updates context tracker on each message', async () => {
-        const adapter = createMockAdapter([{ turns: 3, totalCostUsd: 0.05, inputTokens: 50_000 }]);
-        const tracker = new ContextTracker();
-        const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp');
-        await controller.sendMessage('task');
-        const snap = controller.getContextSnapshot();
-        expect(snap.totalTurns).toBe(3);
-        expect(snap.totalCostUsd).toBe(0.05);
-        expect(snap.latestInputTokens).toBe(50_000);
-        expect(snap.estimatedContextPercent).toBe(25);
-    });
-    it('clears session and resets context', async () => {
-        const adapter = createMockAdapter();
-        const tracker = new ContextTracker();
-        const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp');
-        await controller.sendMessage('task');
-        expect(controller.isActive).toBe(true);
-        const clearedId = await controller.clearSession();
-        expect(clearedId).toBe('ses_test');
-        expect(controller.isActive).toBe(false);
-        expect(controller.sessionId).toBeNull();
-        const snap = controller.getContextSnapshot();
-        expect(snap.totalTurns).toBe(0);
-        expect(snap.sessionId).toBeNull();
-    });
-    it('sends /compact to current session', async () => {
-        const adapter = createMockAdapter([{}, {}]);
-        const tracker = new ContextTracker();
-        const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp');
-        await controller.sendMessage('task');
-        await controller.compactSession();
-        const compactCall = adapter.runSession.mock.calls[1][0];
-        expect(compactCall.prompt).toBe('/compact');
-        expect(compactCall.resumeSessionId).toBe('ses_test');
-    });
-    it('throws when compacting without active session', async () => {
-        const adapter = createMockAdapter();
-        const tracker = new ContextTracker();
-        const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp');
-        await expect(controller.compactSession()).rejects.toThrow('No active session to compact');
-    });
-    it('threads effort through to the SDK input', async () => {
-        const adapter = createMockAdapter();
-        const tracker = new ContextTracker();
-        const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp');
-        await controller.sendMessage('hard task', { effort: 'max' });
-        const call = adapter.runSession.mock.calls[0][0];
-        expect(call.effort).toBe('max');
-    });
-    describe('plan/free mode', () => {
-        const modePrefixes = {
-            plan: '[PLAN MODE] Read-only planning.',
-            free: '',
-        };
-        it('defaults to free mode with acceptEdits permissionMode', async () => {
-            const adapter = createMockAdapter();
-            const tracker = new ContextTracker();
-            const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp', modePrefixes);
-            await controller.sendMessage('do something');
-            const call = adapter.runSession.mock.calls[0][0];
-            expect(call.permissionMode).toBe('acceptEdits');
-            expect(call.prompt).toBe('do something');
-        });
-        it('sets permissionMode to plan and prepends prefix in plan mode', async () => {
-            const adapter = createMockAdapter();
-            const tracker = new ContextTracker();
-            const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp', modePrefixes);
-            await controller.sendMessage('analyze this', { mode: 'plan' });
-            const call = adapter.runSession.mock.calls[0][0];
-            expect(call.permissionMode).toBe('plan');
-            expect(call.prompt).toBe('[PLAN MODE] Read-only planning.\n\nanalyze this');
-        });
-        it('explicit free mode uses acceptEdits and no prefix', async () => {
-            const adapter = createMockAdapter();
-            const tracker = new ContextTracker();
-            const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp', modePrefixes);
-            await controller.sendMessage('build it', { mode: 'free' });
-            const call = adapter.runSession.mock.calls[0][0];
-            expect(call.permissionMode).toBe('acceptEdits');
-            expect(call.prompt).toBe('build it');
-        });
-        it('works without modePrefixes constructor arg (backward compat)', async () => {
-            const adapter = createMockAdapter();
-            const tracker = new ContextTracker();
-            const controller = new SessionController(adapter, tracker, 'test', 'test', '/tmp');
-            await controller.sendMessage('hello', { mode: 'plan' });
-            const call = adapter.runSession.mock.calls[0][0];
-            expect(call.permissionMode).toBe('plan');
-            // Empty prefix defaults — prompt should be unchanged
-            expect(call.prompt).toBe('hello');
-        });
-    });
-});

package/dist/test/session-live-tailer.test.d.ts

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