@codemcp/workflows 5.0.0 → 5.1.0

package/test/e2e/commit-plugin-integration.test.ts

@@ -1,222 +0,0 @@
-/**
- * Integration test for CommitPlugin end-to-end behavior
- */
-
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { execSync } from 'node:child_process';
-import { mkdirSync, rmSync, writeFileSync } from 'node:fs';
-import { resolve } from 'node:path';
-import { initializeServerComponents } from '../../src/server-config.js';
-
-describe('CommitPlugin Integration', () => {
-  const testDir = resolve(__dirname, 'test-commit-plugin');
-  let originalEnv: Record<string, string | undefined>;
-
-  beforeEach(() => {
-    // Save original environment
-    originalEnv = {
-      COMMIT_BEHAVIOR: process.env.COMMIT_BEHAVIOR,
-      COMMIT_MESSAGE_TEMPLATE: process.env.COMMIT_MESSAGE_TEMPLATE,
-    };
-
-    // Clean up any existing test directory
-    try {
-      rmSync(testDir, { recursive: true, force: true });
-    } catch {
-      // Ignore if directory doesn't exist
-    }
-
-    // Create test git repository
-    mkdirSync(testDir, { recursive: true });
-    execSync('git init', { cwd: testDir });
-    execSync('git config user.name "Test User"', { cwd: testDir });
-    execSync('git config user.email "test@example.com"', { cwd: testDir });
-
-    // Create initial commit
-    writeFileSync(resolve(testDir, 'README.md'), '# Test Project\n');
-    execSync('git add .', { cwd: testDir });
-    execSync('git commit -m "Initial commit"', { cwd: testDir });
-  });
-
-  afterEach(() => {
-    // Restore original environment
-    for (const [key, value] of Object.entries(originalEnv)) {
-      if (value === undefined) {
-        delete process.env[key];
-      } else {
-        process.env[key] = value;
-      }
-    }
-
-    // Clean up test directory
-    try {
-      rmSync(testDir, { recursive: true, force: true });
-    } catch {
-      // Ignore cleanup errors
-    }
-  });
-
-  it('should register CommitPlugin when COMMIT_BEHAVIOR is set', async () => {
-    // Arrange
-    process.env.COMMIT_BEHAVIOR = 'step';
-
-    // Act
-    const components = await initializeServerComponents({
-      projectPath: testDir,
-    });
-
-    // Assert
-    expect(components.context.pluginRegistry).toBeDefined();
-    const plugins = components.context.pluginRegistry.getEnabledPlugins();
-    const commitPlugin = plugins.find(p => p.getName() === 'CommitPlugin');
-    expect(commitPlugin).toBeDefined();
-    expect(commitPlugin?.getSequence()).toBe(50);
-  });
-
-  it('should not register CommitPlugin when COMMIT_BEHAVIOR is not set', async () => {
-    // Arrange - no COMMIT_BEHAVIOR set
-
-    // Act
-    const components = await initializeServerComponents({
-      projectPath: testDir,
-    });
-
-    // Assert
-    const plugins = components.context.pluginRegistry.getEnabledPlugins();
-    const commitPlugin = plugins.find(p => p.getName() === 'CommitPlugin');
-    expect(commitPlugin).toBeUndefined();
-  });
-
-  it('should add final commit task to plan file when COMMIT_BEHAVIOR=end', async () => {
-    // Arrange
-    process.env.COMMIT_BEHAVIOR = 'end';
-    // Don't set COMMIT_MESSAGE_TEMPLATE to use default message
-
-    // Act
-    const components = await initializeServerComponents({
-      projectPath: testDir,
-    });
-
-    // Create a mock plan file content
-    const mockPlanContent = `# Test Plan
-
-## Explore
-### Tasks
-- [ ] Research the problem
-
-## Code  
-### Tasks
-- [ ] Implement solution
-
-## Commit
-### Tasks
-- [ ] Review implementation
-`;
-
-    // Execute the afterPlanFileCreated hook
-    const plugins = components.context.pluginRegistry.getEnabledPlugins();
-    const commitPlugin = plugins.find(p => p.getName() === 'CommitPlugin');
-    const hooks = commitPlugin?.getHooks();
-
-    if (hooks?.afterPlanFileCreated) {
-      const mockContext = {
-        conversationId: 'test',
-        planFilePath: resolve(testDir, 'plan.md'),
-        currentPhase: 'explore',
-        workflow: 'epcc',
-        projectPath: testDir,
-        gitBranch: 'main',
-      };
-
-      const updatedContent = await hooks.afterPlanFileCreated(
-        mockContext,
-        resolve(testDir, 'plan.md'),
-        mockPlanContent
-      );
-
-      // Assert
-      expect(updatedContent).toContain('Create a conventional commit');
-      expect(updatedContent).toContain(
-        'summarize the intentions and key decisions'
-      );
-    }
-  });
-
-  it('should add squash commit task for step/phase modes', async () => {
-    // Arrange
-    process.env.COMMIT_BEHAVIOR = 'step';
-
-    // Act
-    const components = await initializeServerComponents({
-      projectPath: testDir,
-    });
-
-    const mockPlanContent = `## Commit
-### Tasks
-- [ ] Review implementation
-`;
-
-    const plugins = components.context.pluginRegistry.getEnabledPlugins();
-    const commitPlugin = plugins.find(p => p.getName() === 'CommitPlugin');
-    const hooks = commitPlugin?.getHooks();
-
-    if (hooks?.afterPlanFileCreated) {
-      const mockContext = {
-        conversationId: 'test',
-        planFilePath: resolve(testDir, 'plan.md'),
-        currentPhase: 'explore',
-        workflow: 'epcc',
-        projectPath: testDir,
-        gitBranch: 'main',
-      };
-
-      const updatedContent = await hooks.afterPlanFileCreated(
-        mockContext,
-        resolve(testDir, 'plan.md'),
-        mockPlanContent
-      );
-
-      // Assert
-      expect(updatedContent).toContain('Squash WIP commits:');
-      expect(updatedContent).toContain('git reset --soft');
-    }
-  });
-
-  it('should create WIP commit on phase transition', async () => {
-    // Arrange
-    process.env.COMMIT_BEHAVIOR = 'phase';
-
-    // Create some changes
-    writeFileSync(resolve(testDir, 'test.txt'), 'test content');
-
-    // Act
-    const components = await initializeServerComponents({
-      projectPath: testDir,
-    });
-
-    const plugins = components.context.pluginRegistry.getEnabledPlugins();
-    const commitPlugin = plugins.find(p => p.getName() === 'CommitPlugin');
-    const hooks = commitPlugin?.getHooks();
-
-    if (hooks?.beforePhaseTransition) {
-      const mockContext = {
-        conversationId: 'test',
-        planFilePath: resolve(testDir, 'plan.md'),
-        currentPhase: 'explore',
-        workflow: 'epcc',
-        projectPath: testDir,
-        gitBranch: 'main',
-        targetPhase: 'plan',
-      };
-
-      await hooks.beforePhaseTransition(mockContext, 'explore', 'plan');
-
-      // Assert - check git log for WIP commit
-      const gitLog = execSync('git log --oneline', {
-        cwd: testDir,
-        encoding: 'utf-8',
-      });
-      expect(gitLog).toContain('WIP: transition to plan');
-    }
-  });
-});

package/test/e2e/core-functionality.test.ts

@@ -1,167 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { createTempProjectWithDefaultStateMachine } from '../utils/temp-files';
-import {
-  DirectServerInterface,
-  createSuiteIsolatedE2EScenario,
-  assertToolSuccess,
-  initializeDevelopment,
-} from '../utils/e2e-test-setup';
-
-vi.unmock('fs');
-vi.unmock('fs/promises');
-
-/**
- * Core Functionality Tests
- *
- * Tests the essential server operations including:
- * - Server initialization and basic tool operations
- * - Resource access (plan and state resources)
- * - Basic conversation management
- * - Error handling and graceful failures
- */
-describe('Core Functionality', () => {
-  let client: DirectServerInterface;
-  let cleanup: () => Promise<void>;
-
-  beforeEach(async () => {
-    const scenario = await createSuiteIsolatedE2EScenario({
-      suiteName: 'core-functionality',
-      tempProjectFactory: createTempProjectWithDefaultStateMachine,
-    });
-    client = scenario.client;
-    cleanup = scenario.cleanup;
-
-    // Initialize development with default workflow before each test
-    await initializeDevelopment(client);
-  });
-
-  afterEach(async () => {
-    if (cleanup) {
-      await cleanup();
-    }
-  });
-
-  describe('Server Initialization', () => {
-    it('should initialize server and provide tools', async () => {
-      const tools = await client.listTools();
-      expect(tools.tools).toBeTruthy();
-      expect(tools.tools).toHaveLength(2);
-      expect(tools.tools.map(t => t.name)).toContain('whats_next');
-      expect(tools.tools.map(t => t.name)).toContain('proceed_to_phase');
-    });
-
-    it('should provide resources', async () => {
-      const resources = await client.listResources();
-      expect(resources.resources).toBeTruthy();
-      expect(resources.resources).toHaveLength(3);
-      expect(resources.resources.map(r => r.uri)).toContain('plan://current');
-      expect(resources.resources.map(r => r.uri)).toContain('state://current');
-      expect(resources.resources.map(r => r.uri)).toContain('system-prompt://');
-    });
-  });
-
-  describe('Basic Tool Operations', () => {
-    it('should handle whats_next tool calls', async () => {
-      const result = await client.callTool('whats_next', {
-        user_input: 'implement authentication',
-      });
-      const response = assertToolSuccess(result);
-
-      expect(response.phase).toBeTruthy();
-      expect(response.instructions).toBeTruthy();
-      expect(response.plan_file_path).toBeTruthy();
-    });
-
-    it('should handle proceed_to_phase tool calls', async () => {
-      // First establish a conversation
-      await client.callTool('whats_next', { user_input: 'start project' });
-
-      const result = await client.callTool('proceed_to_phase', {
-        target_phase: 'design',
-        reason: 'requirements complete',
-        review_state: 'not-required',
-      });
-      const response = assertToolSuccess(result);
-
-      expect(response.phase).toBe('design');
-      expect(response.instructions).toBeTruthy();
-    });
-  });
-
-  describe('Resource Access', () => {
-    it('should provide plan resource as markdown', async () => {
-      // Initialize conversation to create plan file
-      await client.callTool('whats_next', { user_input: 'test project' });
-
-      const planResource = await client.readResource('plan://current');
-      expect(planResource.contents).toHaveLength(1);
-      expect(planResource.contents[0].mimeType).toBe('text/markdown');
-      expect(planResource.contents[0].text).toContain('# Development Plan');
-    });
-
-    it('should provide state resource as JSON', async () => {
-      // Initialize conversation
-      await client.callTool('whats_next', { user_input: 'test project' });
-
-      const stateResource = await client.readResource('state://current');
-      expect(stateResource.contents).toHaveLength(1);
-      expect(stateResource.contents[0].mimeType).toBe('application/json');
-
-      const stateData = JSON.parse(stateResource.contents[0].text);
-      expect(stateData.conversationId).toBeTruthy();
-      expect(stateData.currentPhase).toBeTruthy();
-    });
-  });
-
-  describe('Error Handling', () => {
-    it('should handle invalid tool parameters gracefully', async () => {
-      const result = await client.callTool('proceed_to_phase', {
-        target_phase: 'invalid_phase',
-        reason: 'test',
-        review_state: 'not-required',
-      });
-
-      // Should not throw, but may return error or fallback behavior
-      expect(result).toBeTruthy();
-    });
-
-    it('should handle missing parameters gracefully', async () => {
-      const result = await client.callTool('whats_next', {});
-      const response = assertToolSuccess(result);
-
-      // Should still work with empty parameters
-      expect(response.phase).toBeTruthy();
-      expect(response.instructions).toBeTruthy();
-    });
-
-    it('should handle database errors gracefully', async () => {
-      // This test would require mocking database failures
-      // For now, verify basic resilience
-      const result = await client.callTool('whats_next', {
-        user_input: 'test resilience',
-      });
-
-      expect(assertToolSuccess(result)).toBeTruthy();
-    });
-  });
-
-  describe('Basic Conversation Management', () => {
-    it('should create new conversations', async () => {
-      await client.callTool('whats_next', {
-        user_input: 'new feature request',
-      });
-    });
-
-    it('should maintain conversation state across calls', async () => {
-      const first = await client.callTool('whats_next', {
-        user_input: 'start project',
-      });
-      assertToolSuccess(first);
-
-      const second = await client.callTool('whats_next', {
-        user_input: 'continue project',
-      });
-      assertToolSuccess(second);
-    });
-  });
-});