@codemcp/workflows 4.10.2 → 4.10.3

package/src/components/beads/beads-instruction-generator.ts

@@ -101,20 +101,13 @@ export class BeadsInstructionGenerator implements IInstructionGenerator {
     baseInstructions: string,
     context: InstructionContext
   ): Promise<string> {
-    const {
-      phase,
-      conversationContext,
-      transitionReason,
-      isModeled,
-      planFileExists,
-    } = context;
+    const { planFileExists } = context;
 
     // Generate beads-specific task management guidance
-    const beadsTaskGuidance = await this.generateBeadsTaskGuidance(context);
+    const beadsTaskGuidance = await this.generateBeadsCLIGuidance(context);
 
     // Beads-optimized instruction structure
-    const enhanced = `You are in the ${phase} phase.
-${baseInstructions}
+    let enhanced = `${baseInstructions}
 
 **Plan File Guidance:**
 Use the plan file as memory for the current objective
@@ -122,92 +115,67 @@ Use the plan file as memory for the current objective
 - Add relevant notes to help maintain context
 - Do NOT enter tasks in the plan file, use beads CLI exclusively for task management
 
-**🔧 BD CLI Task Management:**
 ${beadsTaskGuidance}`;
 
-    // Add project context
-    const enhancedWithContext =
-      enhanced +
-      `\n\n**Project Context:**
-- Project: ${conversationContext.projectPath}
-- Branch: ${conversationContext.gitBranch}
-- Current Phase: ${phase}`;
-
-    // Add transition context if this is a modeled transition
-    let final = enhancedWithContext;
-    if (isModeled && transitionReason) {
-      final += `\n\n**Phase Context:**
-- ${transitionReason}`;
-    }
-
     // Add plan file creation note if needed
     if (!planFileExists) {
-      final +=
+      enhanced +=
         '\n\n**Note**: Plan file will be created when you first update it.';
     }
 
     // Add beads-specific reminders
-    final += `\n\n**Important Reminders:**
+    enhanced += `\n\n**Important Reminders:**
 - Use ONLY bd CLI tool for task management - do not use your own task management tools
 - Call whats_next() after the next user message to maintain the development workflow`;
 
-    return final;
+    return enhanced;
   }
 
   /**
    * Generate beads-specific task management guidance
    */
-  private async generateBeadsTaskGuidance(
+  private async generateBeadsCLIGuidance(
     context: InstructionContext
   ): Promise<string> {
-    const { phase, instructionSource } = context;
+    const { instructionSource } = context;
 
-    // For explicit phase transitions (proceed_to_phase), provide minimal guidance
-    // The detailed guidance is better suited for whats_next which analyzes context
-    if (instructionSource === 'proceed_to_phase') {
-      return `- Use bd CLI tool exclusively for task management
-- Do not use your own task management tools`;
-    }
+    // For whats_next, provide detailed guidance
+    if (instructionSource === 'whats_next') {
+      let additionalInstructions = `**bd Task Management:**
+      `;
 
-    // For implicit transitions (whats_next), provide detailed guidance
-    // Extract phase task ID from plan file (this would need to be implemented)
-    const phaseTaskId = await this.extractPhaseTaskId(context);
+      const phaseTaskId = await this.extractPhaseTaskId(context);
 
-    if (!phaseTaskId) {
-      return `- Use bd CLI tool exclusively
+      if (!phaseTaskId) {
+        return (
+          additionalInstructions +
+          `- Use bd CLI tool exclusively
 - **Start by listing ready tasks**: \`bd list --parent <phase-task-id> --status open\`
 - **Create new tasks**: \`bd create 'Task title' --parent <phase-task-id> -p <priority>\`
 - **Update status when working**: \`bd update <task-id> --status in_progress\`
 - **Complete tasks**: \`bd close <task-id>\`
 - **Focus on ready tasks first** - let beads handle dependencies
-- Add new tasks as they are identified during your work with the user`;
-    }
-
-    return `
-You are currently in the ${this.capitalizePhase(phase)} phase. All work items should be created as children of ${phaseTaskId}.
+- Add new tasks as they are identified during your work with the user`
+        );
+      }
 
-**Focus on ${this.capitalizePhase(phase)} Phase Tasks** (subtasks of \`${phaseTaskId}\`):
+      return (
+        additionalInstructions +
+        `
+**Focus on subtasks of \`${phaseTaskId}\`**:
 • \`bd list --parent ${phaseTaskId} --status open\` - List ready work items
 • \`bd update <task-id> --status in_progress\` - Start working on a specific task
 • \`bd close <task-id>\` - Mark task complete when finished
 
 **New Tasks for Current Phase**:
 • \`bd create 'Task description' --parent ${phaseTaskId} -p <priority>\` - Create work item under current phase
-• \`bd dep add <task-id> <depends-on-id>\` - Define dependencies for a task:
-
-
-**Essential bd Commands**:
-• \`bd list --parent ${phaseTaskId} --status open\` - List ready work items
-• \`bd update <task-id> --status in_progress\` - Start working
-• \`bd close <task-id>\` - Complete work item
-• \`bd show ${phaseTaskId}\` - View phase and its work items
+• \`bd dep add <task-id> <depends-on-id>\` - Define dependencies for a task:`
+      );
+    }
 
-**Immediate Action**: Run \`bd list --parent ${phaseTaskId} --status open\` to see ready tasks.`;
+    return '';
   }
 
-  /**
-   * Extract phase task ID from plan file (simplified implementation)
-   */
   private async extractPhaseTaskId(
     context: InstructionContext
   ): Promise<string | null> {

package/src/tool-handlers/get-tool-info.ts

@@ -73,7 +73,6 @@ interface GetToolInfoResponse {
 
   workflow_states?: {
     current_phase?: string;
-    conversation_id?: string;
     plan_file_path?: string;
   };
 }
@@ -195,7 +194,6 @@ export class GetToolInfoHandler extends BaseToolHandler<
         await context.conversationManager.getConversationContext();
       workflowState = {
         current_phase: conversationContext.currentPhase,
-        conversation_id: conversationContext.conversationId,
         plan_file_path: conversationContext.planFilePath,
       };
     } catch (error) {

package/src/tool-handlers/proceed-to-phase.ts

@@ -27,8 +27,6 @@ export interface ProceedToPhaseResult {
   instructions: string;
   plan_file_path: string;
   transition_reason: string;
-  is_modeled_transition: boolean;
-  conversation_id: string;
 }
 
 /**
@@ -173,8 +171,6 @@ export class ProceedToPhaseHandler extends ConversationRequiredToolHandler<
       instructions: finalInstructions,
       plan_file_path: conversationContext.planFilePath,
       transition_reason: transitionResult.transitionReason,
-      is_modeled_transition: transitionResult.isModeled,
-      conversation_id: conversationContext.conversationId,
     };
 
     // Log interaction

package/src/tool-handlers/resume-workflow.ts

@@ -61,7 +61,6 @@ interface WorkflowRecommendations {
  */
 export interface ResumeWorkflowResult {
   workflow_status: {
-    conversation_id: string;
     current_phase: string;
     project_path: string;
     git_branch: string;
@@ -132,7 +131,6 @@ export class ResumeWorkflowHandler extends ConversationRequiredToolHandler<
     const response: ResumeWorkflowResult = {
       // Core workflow resumption info
       workflow_status: {
-        conversation_id: conversationContext.conversationId,
         current_phase: conversationContext.currentPhase,
         project_path: conversationContext.projectPath,
         git_branch: conversationContext.gitBranch,

package/src/tool-handlers/start-development.ts

@@ -38,8 +38,6 @@ export interface StartDevelopmentResult {
   phase: string;
   instructions: string;
   plan_file_path: string;
-  conversation_id: string;
-  workflow: YamlStateMachine;
   workflowDocumentationUrl?: string;
 }
 
@@ -134,8 +132,6 @@ export class StartDevelopmentHandler extends BaseToolHandler<
         phase: 'branch-prompt',
         instructions: `You're currently on the ${currentBranch} branch. It's recommended to create a feature branch for development. Propose a branch creation by suggesting a branch command to the user call start_development again.\n\nSuggested command: \`git checkout -b ${suggestedBranchName}\`\n\nPlease create a new branch and then call start_development again to begin development.`,
         plan_file_path: '',
-        conversation_id: '',
-        workflow: {} as YamlStateMachine,
       };
 
       this.logger.debug(
@@ -270,8 +266,6 @@ export class StartDevelopmentHandler extends BaseToolHandler<
       phase: transitionResult.newPhase,
       instructions: finalInstructions,
       plan_file_path: conversationContext.planFilePath,
-      conversation_id: conversationContext.conversationId,
-      workflow: stateMachine,
       workflowDocumentationUrl,
     };
 
@@ -377,8 +371,6 @@ export class StartDevelopmentHandler extends BaseToolHandler<
         phase: 'artifact-setup',
         instructions: setupGuidance,
         plan_file_path: '',
-        conversation_id: '',
-        workflow: {} as YamlStateMachine,
       };
     } catch (error) {
       this.logger.warn(

package/src/tool-handlers/whats-next.ts

@@ -30,8 +30,6 @@ export interface WhatsNextResult {
   phase: string;
   instructions: string;
   plan_file_path: string;
-  is_modeled_transition: boolean;
-  conversation_id: string;
 }
 
 /**
@@ -189,8 +187,6 @@ export class WhatsNextHandler extends ConversationRequiredToolHandler<
       phase: transitionResult.newPhase,
       instructions: finalInstructions,
       plan_file_path: conversationContext.planFilePath,
-      is_modeled_transition: transitionResult.isModeled,
-      conversation_id: conversationContext.conversationId,
     };
 
     // Log interaction

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

@@ -153,7 +153,7 @@ function validateBeadsInstructions(instructions: string): void {
   expect(instructions.toLowerCase()).toContain('task');
 
   // Should have beads-specific guidance
-  expect(instructions).toContain('🔧 BD CLI Task Management');
+  expect(instructions).toContain('bd');
 
   // Should mention using ONLY bd CLI
   expect(instructions).toContain('Use ONLY bd CLI tool');
@@ -506,11 +506,11 @@ describe('Beads Plugin Comprehensive Integration', () => {
 
       // VALIDATE: WITH beads mentions bd CLI
       expect(instructionsWithBeads.toLowerCase()).toContain('bd');
-      expect(instructionsWithBeads).toContain('BD CLI Task Management');
+      expect(instructionsWithBeads).toContain('bd Task Management');
 
       // VALIDATE: WITHOUT beads does NOT mention bd CLI
       expect(instructionsWithout.toLowerCase()).not.toContain('bd cli');
-      expect(instructionsWithout).not.toContain('BD CLI Task Management');
+      expect(instructionsWithout).not.toContain('bd Task Management');
     });
 
     it('should maintain identical response contracts regardless of beads', async () => {
@@ -548,12 +548,10 @@ describe('Beads Plugin Comprehensive Integration', () => {
       await scenarioWithout.cleanup();
 
       // VALIDATE: Both responses have identical properties
-      expect(responseWith).toHaveProperty('conversation_id');
       expect(responseWith).toHaveProperty('phase');
       expect(responseWith).toHaveProperty('plan_file_path');
       expect(responseWith).toHaveProperty('instructions');
 
-      expect(responseWithout).toHaveProperty('conversation_id');
       expect(responseWithout).toHaveProperty('phase');
       expect(responseWithout).toHaveProperty('plan_file_path');
       expect(responseWithout).toHaveProperty('instructions');
@@ -722,7 +720,6 @@ describe('Beads Plugin Comprehensive Integration', () => {
 
       // VALIDATE: Has all required response fields
       const response = assertToolSuccess(result) as StartDevelopmentResult;
-      expect(response).toHaveProperty('conversation_id');
       expect(response).toHaveProperty('phase');
       expect(response).toHaveProperty('plan_file_path');
       expect(response).toHaveProperty('instructions');
@@ -827,9 +824,6 @@ describe('Beads Plugin Comprehensive Integration', () => {
       expect(whatsNextResponse.plan_file_path).toBe(
         startResponse.plan_file_path
       );
-      expect(whatsNextResponse.conversation_id).toBe(
-        startResponse.conversation_id
-      );
 
       // VALIDATE: Plan file is still valid
       const planContent = await fs.readFile(
@@ -1344,10 +1338,7 @@ describe('Beads Plugin Comprehensive Integration', () => {
         commit_behaviour: 'none',
       });
 
-      const startResponse = assertToolSuccess(
-        startResult
-      ) as StartDevelopmentResult;
-      const conversationId = startResponse.conversation_id;
+      assertToolSuccess(startResult) as StartDevelopmentResult;
 
       // Get whats_next
       const whatsNextResult = await client.callTool('whats_next', {
@@ -1361,10 +1352,6 @@ describe('Beads Plugin Comprehensive Integration', () => {
         whatsNextResult
       ) as WhatsNextResult;
 
-      // VALIDATE: Conversation ID preserved after hook execution
-      expect(whatsNextResponse.conversation_id).toBe(conversationId);
-
-      // VALIDATE: Phase information consistent
       expect(whatsNextResponse.phase).toBe('explore');
     });
   });
@@ -1510,11 +1497,9 @@ describe('Beads Plugin Comprehensive Integration', () => {
       ) as StartDevelopmentResult;
 
       // VALIDATE: All standard response properties present
-      expect(startResponse).toHaveProperty('conversation_id');
       expect(startResponse).toHaveProperty('phase');
       expect(startResponse).toHaveProperty('plan_file_path');
       expect(startResponse).toHaveProperty('instructions');
-      expect(startResponse).toHaveProperty('workflow');
 
       // VALIDATE: Can transition phases normally
       const transitionResult = await client.callTool('proceed_to_phase', {

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

@@ -69,7 +69,6 @@ describe('Core Functionality', () => {
 
       expect(response.phase).toBeTruthy();
       expect(response.instructions).toBeTruthy();
-      expect(response.conversation_id).toBeTruthy();
       expect(response.plan_file_path).toBeTruthy();
     });
 
@@ -148,29 +147,21 @@ describe('Core Functionality', () => {
 
   describe('Basic Conversation Management', () => {
     it('should create new conversations', async () => {
-      const result = await client.callTool('whats_next', {
+      await client.callTool('whats_next', {
         user_input: 'new feature request',
       });
-      const response = assertToolSuccess(result);
-
-      expect(response.conversation_id).toBeTruthy();
-      expect(response.conversation_id).toMatch(/^default-sm-/);
     });
 
     it('should maintain conversation state across calls', async () => {
       const first = await client.callTool('whats_next', {
         user_input: 'start project',
       });
-      const firstResponse = assertToolSuccess(first);
+      assertToolSuccess(first);
 
       const second = await client.callTool('whats_next', {
         user_input: 'continue project',
       });
-      const secondResponse = assertToolSuccess(second);
-
-      expect(firstResponse.conversation_id).toBe(
-        secondResponse.conversation_id
-      );
+      assertToolSuccess(second);
     });
   });
 });

package/test/e2e/mcp-contract.test.ts

@@ -206,7 +206,6 @@ describe('MCP Contract Validation', () => {
       const responseText = textContent!.text;
       expect(responseText).toContain('phase');
       expect(responseText).toContain('instructions');
-      expect(responseText).toContain('conversation_id');
     });
 
     it('should execute proceed_to_phase tool successfully', async () => {
@@ -334,7 +333,6 @@ describe('MCP Contract Validation', () => {
 
       // Should contain valid JSON with state information
       const stateData = JSON.parse(content.text);
-      expect(stateData.conversationId).toBeTruthy();
       expect(stateData.currentPhase).toBeTruthy();
       expect(stateData.projectPath).toBeTruthy();
     });
@@ -386,34 +384,6 @@ describe('MCP Contract Validation', () => {
       expect(Array.isArray(results[3].resources)).toBe(true);
     });
 
-    it('should maintain conversation state across multiple interactions', async () => {
-      // First interaction
-      const result1 = await client.callTool({
-        name: 'whats_next',
-        arguments: {
-          user_input: 'start new project',
-        },
-      });
-
-      const response1 = JSON.parse(result1.content[0].text);
-      const conversationId1 = response1.conversation_id;
-
-      // Second interaction in the same session
-      const result2 = await client.callTool({
-        name: 'whats_next',
-        arguments: {
-          user_input: 'continue project',
-        },
-      });
-
-      const response2 = JSON.parse(result2.content[0].text);
-      const conversationId2 = response2.conversation_id;
-
-      // Should maintain same conversation ID within the same MCP session
-      // Note: Each MCP client connection maintains its own conversation context
-      expect(conversationId1).toBe(conversationId2);
-    });
-
     it('should handle malformed requests appropriately', async () => {
       // Test with missing required parameters
       try {
@@ -534,7 +504,6 @@ describe('MCP Contract Validation', () => {
       const response = JSON.parse(result.content[0].text);
       expect(response.phase).toBeTruthy();
       expect(response.instructions).toBeTruthy();
-      expect(response.conversation_id).toBeTruthy();
     });
   });
 });