@codemcp/workflows 5.0.1 → 5.1.0

package/test/unit/start-development-goal-extraction.test.ts

@@ -1,226 +0,0 @@
-/**
- * Unit tests for BeadsPlugin Goal extraction functionality
- *
- * Tests the extractGoalFromPlan method that extracts meaningful goal content
- * from development plan files for use in beads integration
- */
-
-import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest';
-import { TestAccess } from '../utils/test-access.js';
-import { BeadsPlugin } from '../../src/plugin-system/beads-plugin.js';
-
-describe('BeadsPlugin - Goal Extraction', () => {
-  let plugin: BeadsPlugin;
-
-  beforeEach(() => {
-    // Mock environment variable for plugin enablement
-    vi.stubEnv('TASK_BACKEND', 'beads');
-    plugin = new BeadsPlugin({ projectPath: '/test/project' });
-  });
-
-  afterEach(() => {
-    vi.unstubAllEnvs();
-  });
-
-  describe('extractGoalFromPlan', () => {
-    it('should extract meaningful goal content', () => {
-      const planContent = `# Development Plan: Test Project
-
-## Goal
-Build a user authentication system with JWT tokens and password reset functionality.
-
-## Explore
-### Tasks
-- [ ] Analyze requirements
-`;
-
-      const result = TestAccess.callMethod(
-        plugin,
-        'extractGoalFromPlan',
-        planContent
-      );
-
-      expect(result).toBe(
-        'Build a user authentication system with JWT tokens and password reset functionality.'
-      );
-    });
-
-    it('should return undefined for placeholder goal content', () => {
-      const planContent = `# Development Plan: Test Project
-
-## Goal
-*Define what you're building or fixing - this will be updated as requirements are gathered*
-
-## Explore
-### Tasks
-- [ ] Analyze requirements
-`;
-
-      const result = TestAccess.callMethod(
-        plugin,
-        'extractGoalFromPlan',
-        planContent
-      );
-
-      expect(result).toBeUndefined();
-    });
-
-    it('should return undefined for "To be defined" content', () => {
-      const planContent = `# Development Plan: Test Project
-
-## Goal
-To be defined during exploration
-
-## Explore
-### Tasks
-- [ ] Analyze requirements
-`;
-
-      const result = TestAccess.callMethod(
-        plugin,
-        'extractGoalFromPlan',
-        planContent
-      );
-
-      expect(result).toBeUndefined();
-    });
-
-    it('should return undefined for very short content', () => {
-      const planContent = `# Development Plan: Test Project
-
-## Goal
-Fix bug
-
-## Explore
-### Tasks
-- [ ] Analyze requirements
-`;
-
-      const result = TestAccess.callMethod(
-        plugin,
-        'extractGoalFromPlan',
-        planContent
-      );
-
-      expect(result).toBeUndefined();
-    });
-
-    it('should handle multiline goal content correctly', () => {
-      const planContent = `# Development Plan: Test Project
-
-## Goal
-Implement a comprehensive logging system that captures:
-- User actions and authentication events  
-- API request/response cycles
-- System errors with stack traces
-- Performance metrics
-
-The system should support different log levels and output formats.
-
-## Explore
-### Tasks
-- [ ] Analyze requirements
-`;
-
-      const result = TestAccess.callMethod(
-        plugin,
-        'extractGoalFromPlan',
-        planContent
-      );
-
-      expect(result)
-        .toBe(`Implement a comprehensive logging system that captures:
-- User actions and authentication events  
-- API request/response cycles
-- System errors with stack traces
-- Performance metrics
-
-The system should support different log levels and output formats.`);
-    });
-
-    it('should return undefined when no Goal section exists', () => {
-      const planContent = `# Development Plan: Test Project
-
-## Explore
-### Tasks
-- [ ] Analyze requirements
-`;
-
-      const result = TestAccess.callMethod(
-        plugin,
-        'extractGoalFromPlan',
-        planContent
-      );
-
-      expect(result).toBeUndefined();
-    });
-
-    it('should return undefined for empty or null input', () => {
-      expect(
-        TestAccess.callMethod(plugin, 'extractGoalFromPlan', '')
-      ).toBeUndefined();
-
-      expect(
-        TestAccess.callMethod(plugin, 'extractGoalFromPlan', null)
-      ).toBeUndefined();
-
-      expect(
-        TestAccess.callMethod(plugin, 'extractGoalFromPlan', undefined)
-      ).toBeUndefined();
-    });
-
-    it('should stop at the next section boundary', () => {
-      const planContent = `# Development Plan: Test Project
-
-## Goal
-Build a user authentication system with secure login and registration.
-
-## Key Decisions
-- Using JWT for token-based authentication
-- Password hashing with bcrypt
-`;
-
-      const result = TestAccess.callMethod(
-        plugin,
-        'extractGoalFromPlan',
-        planContent
-      );
-
-      expect(result).toBe(
-        'Build a user authentication system with secure login and registration.'
-      );
-    });
-  });
-
-  describe('plan filename extraction', () => {
-    it('should extract filename from plan file path correctly', () => {
-      // Test the logic used in setupBeadsIntegration
-      const planFilePath = '/project/.vibe/development-plan-feature-auth.md';
-      const planFilename = planFilePath.split('/').pop();
-
-      expect(planFilename).toBe('development-plan-feature-auth.md');
-    });
-
-    it('should handle various plan file path formats', () => {
-      const testCases = [
-        {
-          path: '/Users/dev/my-project/.vibe/development-plan-main.md',
-          expected: 'development-plan-main.md',
-        },
-        {
-          path: 'development-plan-bugfix.md',
-          expected: 'development-plan-bugfix.md',
-        },
-        {
-          path: '/deep/nested/path/to/.vibe/development-plan-feature-dashboard.md',
-          expected: 'development-plan-feature-dashboard.md',
-        },
-      ];
-
-      for (const { path, expected } of testCases) {
-        const filename = path.split('/').pop();
-        expect(filename).toBe(expected);
-      }
-    });
-  });
-});

package/test/unit/system-prompt-resource.test.ts

@@ -1,102 +0,0 @@
-/**
- * System Prompt Resource Tests
- *
- * Tests for the system-prompt resource handler to ensure it properly
- * exposes the system prompt through the MCP protocol.
- */
-
-import { describe, it, expect } from 'vitest';
-import { SystemPromptResourceHandler } from '../../src/resource-handlers/system-prompt.js';
-import type { ServerContext } from '../../src/types.js';
-
-describe('System Prompt Resource', () => {
-  it('should expose system prompt as MCP resource', async () => {
-    const handler = new SystemPromptResourceHandler();
-
-    // Call the handler directly
-    const result = await handler.handle(
-      new URL('system-prompt://'),
-      {} as ServerContext
-    );
-
-    // Verify the safeExecute wrapper structure
-    expect(result).toBeDefined();
-    expect(result.success).toBe(true);
-    expect(result.data).toBeDefined();
-
-    const data = result.data!;
-    expect(data.uri).toBe('system-prompt://');
-    expect(data.mimeType).toBe('text/plain');
-    expect(data.text).toBeDefined();
-    expect(typeof data.text).toBe('string');
-
-    // Verify content contains expected system prompt elements (streamlined version)
-    expect(data.text).toContain(
-      'You are an AI assistant that helps users develop software features'
-    );
-    expect(data.text).toContain('responsible-vibe-mcp');
-    expect(data.text).toContain('whats_next()');
-    expect(data.text).toContain('instructions');
-    expect(data.text).toContain('development plan');
-
-    // Verify it's concise but not empty (streamlined prompt is ~400 chars)
-    expect(data.text.length).toBeGreaterThan(200);
-    expect(data.text.length).toBeLessThan(1000);
-  });
-
-  it('should be workflow-independent and consistent', async () => {
-    const handler = new SystemPromptResourceHandler();
-
-    // Get system prompt multiple times
-    const result1 = await handler.handle(
-      new URL('system-prompt://'),
-      {} as ServerContext
-    );
-    const result2 = await handler.handle(
-      new URL('system-prompt://'),
-      {} as ServerContext
-    );
-    const result3 = await handler.handle(
-      new URL('system-prompt://'),
-      {} as ServerContext
-    );
-
-    // All should be successful
-    expect(result1.success).toBe(true);
-    expect(result2.success).toBe(true);
-    expect(result3.success).toBe(true);
-
-    // All should be identical
-    expect(result1.data!.text).toBe(result2.data!.text);
-    expect(result2.data!.text).toBe(result3.data!.text);
-
-    // Verify the prompt contains standard elements (streamlined version)
-    expect(result1.data!.text).toContain('You are an AI assistant');
-    expect(result1.data!.text).toContain('whats_next()');
-    expect(result1.data!.text).toContain('development');
-    expect(result1.data!.text).toContain('instructions');
-  });
-
-  it('should use streamlined system prompt', async () => {
-    const handler = new SystemPromptResourceHandler();
-
-    const result = await handler.handle(
-      new URL('system-prompt://'),
-      {} as ServerContext
-    );
-
-    expect(result.success).toBe(true);
-
-    // The streamlined system prompt should be concise and focused
-    // It relies on tool responses for detailed phase instructions
-    expect(result.data!.text).toContain(
-      'You are an AI assistant that helps users develop software features'
-    );
-    expect(result.data!.text).toContain('whats_next()');
-    expect(result.data!.text).toContain('instructions');
-    expect(result.data!.text).toContain('development plan');
-
-    // Streamlined prompt should be concise (~400 chars vs old 2000+)
-    expect(result.data!.text.length).toBeLessThan(1000);
-  });
-});

package/test/unit/tool-handlers/no-idea.test.ts

@@ -1,40 +0,0 @@
-/**
- * Tests for NoIdeaHandler
- */
-
-import { describe, it, expect } from 'vitest';
-import { NoIdeaHandler } from '../../../src/tool-handlers/no-idea.js';
-import { MockContextFactory } from '../../utils/test-helpers.js';
-
-describe('NoIdeaHandler', () => {
-  const handler = new NoIdeaHandler();
-  const mockContext = MockContextFactory.createBasicContext('/tmp/test');
-
-  it('should return instructions with key terms', async () => {
-    const result = await handler.handle({}, mockContext);
-
-    expect(result.success).toBe(true);
-    const instructions = result.data?.instructions || '';
-    expect(instructions.toLowerCase()).toContain('you have no');
-    expect(instructions.toLowerCase()).toContain('admit');
-    expect(instructions.toLowerCase()).toContain('clarify');
-  });
-
-  it('should include provided context', async () => {
-    const result = await handler.handle(
-      { context: 'quantum physics' },
-      mockContext
-    );
-
-    expect(result.success).toBe(true);
-    const instructions = result.data?.instructions || '';
-    expect(instructions).toContain('quantum physics');
-  });
-
-  it('should handle empty context', async () => {
-    const result = await handler.handle({ context: '' }, mockContext);
-
-    expect(result.success).toBe(true);
-    expect(result.data?.instructions).toBeDefined();
-  });
-});