@codemcp/workflows 4.10.1 → 4.10.3

package/test/e2e/state-management.test.ts

@@ -285,7 +285,7 @@ states:
       const first = await client.callTool('whats_next', {
         user_input: 'start project',
       });
-      const firstResponse = assertToolSuccess(first);
+      assertToolSuccess(first);
 
       const proceedResult = await client.callTool('proceed_to_phase', {
         target_phase: 'design',
@@ -296,9 +296,6 @@ states:
 
       // Verify the proceed_to_phase result reflects the transition
       expect(proceedResponse.phase).toBe('design');
-      expect(firstResponse.conversation_id).toBe(
-        proceedResponse.conversation_id
-      );
     });
 
     it('should maintain conversation state consistency', async () => {
@@ -310,7 +307,6 @@ states:
       const stateResource = await client.readResource('state://current');
       const stateData = JSON.parse(stateResource.contents[0].text);
 
-      expect(stateData.conversationId).toBe(response.conversation_id);
       expect(stateData.currentPhase).toBe(response.phase);
     });
 

package/test/e2e/workflow-integration.test.ts

@@ -111,7 +111,6 @@ describe('Workflow Integration', () => {
       const stateResource = await client.readResource('state://current');
       const stateData = JSON.parse(stateResource.contents[0].text);
       expect(stateData.currentPhase).toBe('finalize');
-      expect(stateData.conversationId).toBe(reqResponse.conversation_id);
     });
 
     it('should handle iterative development with phase revisiting', async () => {
@@ -197,18 +196,14 @@ describe('Workflow Integration', () => {
         user_input: 'project 1 feature',
         context: 'first project',
       });
-      const p1Response = assertToolSuccess(project1);
-      const p1ConversationId = p1Response.conversation_id;
+      assertToolSuccess(project1);
 
       // Continue with same project
       const project1Continue = await client.callTool('whats_next', {
         user_input: 'continue project 1',
         context: 'same project context',
       });
-      const p1ContinueResponse = assertToolSuccess(project1Continue);
-
-      // Should maintain same conversation
-      expect(p1ContinueResponse.conversation_id).toBe(p1ConversationId);
+      assertToolSuccess(project1Continue);
     });
 
     it('should maintain separate plan files for different contexts', async () => {
@@ -276,7 +271,6 @@ describe('Workflow Integration', () => {
       const response = assertToolSuccess(result);
       expect(response.phase).toBeTruthy();
       expect(response.instructions).toBeTruthy();
-      expect(response.conversation_id).toBeTruthy();
     });
 
     it('should handle rapid development iterations', async () => {
@@ -306,7 +300,6 @@ describe('Workflow Integration', () => {
       });
       const impl2Response = assertToolSuccess(impl2);
       expect(impl2Response.phase).toBe('implementation');
-      expect(impl2Response.conversation_id).toBeTruthy();
     });
   });
 
@@ -348,7 +341,6 @@ describe('Workflow Integration', () => {
 
       // Even if there are file system issues, basic functionality should work
       expect(response.phase).toBeTruthy();
-      expect(response.conversation_id).toBeTruthy();
     });
 
     it('should maintain functionality under stress', async () => {
@@ -366,7 +358,6 @@ describe('Workflow Integration', () => {
       for (const result of results) {
         const response = assertToolSuccess(result);
         expect(response.phase).toBeTruthy();
-        expect(response.conversation_id).toBeTruthy();
       }
 
       // Final state should be consistent

package/test/unit/beads-instruction-generator.test.ts

@@ -36,28 +36,45 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       transitionReason: 'test transition',
       isModeled: false,
       planFileExists: true,
+      instructionSource: 'whats_next', // Default to whats_next for tests expecting BD guidance
     };
   });
 
   describe('Beads-Specific Content Must Be Present', () => {
-    it('should generate complete beads task management header structure', async () => {
+    it('should generate complete beads task management header structure for whats_next', async () => {
       const result = await beadsInstructionGenerator.generateInstructions(
         'Work on design tasks.',
-        mockInstructionContext
+        { ...mockInstructionContext, instructionSource: 'whats_next' }
       );
 
-      // Verify ALL beads-specific elements are present
-      expect(result.instructions).toContain('🔧 BD CLI Task Management:');
+      // Verify beads-specific elements are present for whats_next
+      expect(result.instructions).toContain('bd Task Management:');
       expect(result.instructions).toContain('Use bd CLI tool exclusively');
     });
 
-    it('should contain core beads CLI commands with proper formatting', async () => {
+    it('should not generate BD CLI guidance for proceed_to_phase', async () => {
+      const result = await beadsInstructionGenerator.generateInstructions(
+        'Work on design tasks.',
+        { ...mockInstructionContext, instructionSource: 'proceed_to_phase' }
+      );
+
+      // Verify BD CLI guidance is NOT present for proceed_to_phase (your optimization!)
+      expect(result.instructions).not.toContain('bd:');
+      expect(result.instructions).not.toContain('bd Task Management:');
+      expect(result.instructions).not.toContain('bd list --parent');
+      // But should still contain basic beads reminders
+      expect(result.instructions).toContain(
+        'Use ONLY bd CLI tool for task management'
+      );
+    });
+
+    it('should contain core beads CLI commands with proper formatting for whats_next', async () => {
       const result = await beadsInstructionGenerator.generateInstructions(
         'Work on tasks.',
-        mockInstructionContext
+        { ...mockInstructionContext, instructionSource: 'whats_next' }
       );
 
-      // Verify specific beads CLI commands are present
+      // Verify specific beads CLI commands are present for whats_next
       expect(result.instructions).toContain(
         'bd list --parent <phase-task-id> --status open'
       );
@@ -73,7 +90,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
     it('should contain beads-specific task management prohibition', async () => {
       const result = await beadsInstructionGenerator.generateInstructions(
         'Work on tasks.',
-        mockInstructionContext
+        { ...mockInstructionContext, instructionSource: 'whats_next' }
       );
 
       // Verify beads-specific prohibition
@@ -82,6 +99,34 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       );
     });
 
+    it('should contain beads-specific reminders section', async () => {
+      const result = await beadsInstructionGenerator.generateInstructions(
+        'Work on tasks.',
+        mockInstructionContext // Works for any instructionSource
+      );
+
+      // Verify beads-specific reminders (always present)
+      expect(result.instructions).toContain(
+        'Use ONLY bd CLI tool for task management - do not use your own task management tools'
+      );
+      expect(result.instructions).toContain(
+        'Call whats_next() after the next user message to maintain the development workflow'
+      );
+    });
+
+    it('should contain beads plan file guidance', async () => {
+      const result = await beadsInstructionGenerator.generateInstructions(
+        'Work on tasks.',
+        mockInstructionContext // Works for any instructionSource
+      );
+
+      // Verify beads plan file guidance (always present)
+      expect(result.instructions).toContain('Plan File Guidance:');
+      expect(result.instructions).toContain(
+        'Do NOT enter tasks in the plan file, use beads CLI exclusively'
+      );
+    });
+
     it('should contain beads-specific reminders section', async () => {
       const result = await beadsInstructionGenerator.generateInstructions(
         'Work on tasks.',
@@ -109,16 +154,47 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       );
     });
 
-    it('should use proper beads terminology and structure', async () => {
+    it('should use proper beads terminology and structure for whats_next', async () => {
       const result = await beadsInstructionGenerator.generateInstructions(
         'Work on tasks.',
-        mockInstructionContext
+        { ...mockInstructionContext, instructionSource: 'whats_next' as const }
       );
 
-      // Verify beads terminology
+      // Verify beads terminology (only present in whats_next detailed guidance)
       expect(result.instructions).toContain('ready tasks');
       expect(result.instructions).toContain('phase-task-id');
-      expect(result.instructions).toContain('Current Phase:');
+      // Removed 'Current Phase' expectation - no longer part of minimal output
+    });
+
+    it('should not contain beads CLI terminology for proceed_to_phase', async () => {
+      const result = await beadsInstructionGenerator.generateInstructions(
+        'Work on tasks.',
+        {
+          ...mockInstructionContext,
+          instructionSource: 'proceed_to_phase' as const,
+        }
+      );
+
+      // Verify beads CLI terminology is NOT present for proceed_to_phase (your optimization!)
+      expect(result.instructions).not.toContain('ready tasks');
+      expect(result.instructions).not.toContain('phase-task-id');
+      expect(result.instructions).not.toContain('Current Phase'); // Removed from minimal output
+      // But should still contain basic beads reminders
+      expect(result.instructions).toContain(
+        'Use ONLY bd CLI tool for task management'
+      );
+    });
+
+    it('should not contain beads CLI terminology for proceed_to_phase', async () => {
+      const result = await beadsInstructionGenerator.generateInstructions(
+        'Work on tasks.',
+        { ...mockInstructionContext, instructionSource: 'proceed_to_phase' }
+      );
+
+      // Verify beads CLI terminology is NOT present for proceed_to_phase
+      expect(result.instructions).not.toContain('ready tasks');
+      expect(result.instructions).not.toContain('phase-task-id');
+      // Removed 'Current Phase:' expectation - no longer part of minimal output for proceed_to_phase
     });
   });
 
@@ -203,28 +279,58 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       const phases = ['explore', 'plan', 'code', 'commit'];
 
       for (const phase of phases) {
-        const context = { ...mockInstructionContext, phase };
+        const context = {
+          ...mockInstructionContext,
+          phase,
+          instructionSource: 'whats_next' as const,
+        };
         const result = await beadsInstructionGenerator.generateInstructions(
           `${phase} instructions`,
           context
         );
 
-        // All phases should have consistent beads structure
+        // All phases should have consistent beads structure when instructionSource is whats_next
         expect(
           result.instructions,
-          `Phase ${phase} should have BD CLI header`
-        ).toContain('🔧 BD CLI Task Management:');
+          `Phase ${phase} should have bd Task Management header for whats_next`
+        ).toContain('bd Task Management:');
         expect(
           result.instructions,
-          `Phase ${phase} should have beads CLI commands`
+          `Phase ${phase} should have beads CLI commands for whats_next`
         ).toContain('bd list --parent');
         expect(
           result.instructions,
           `Phase ${phase} should not have markdown content`
         ).not.toContain('Mark completed tasks with [x]');
 
-        // Should reference the correct phase
-        expect(result.instructions).toContain(`Current Phase: ${phase}`);
+        // Should contain the phase name somewhere (either in instructions or section headers)
+        expect(result.instructions).toContain(phase); // Phase name should appear somewhere
+      }
+
+      // Test that non-whats_next instruction sources don't get BD CLI guidance
+      for (const phase of phases) {
+        const context = {
+          ...mockInstructionContext,
+          phase,
+          instructionSource: 'proceed_to_phase' as const,
+        };
+        const result = await beadsInstructionGenerator.generateInstructions(
+          `${phase} instructions`,
+          context
+        );
+
+        // Should NOT have BD CLI guidance for proceed_to_phase
+        expect(
+          result.instructions,
+          `Phase ${phase} should NOT have bd guidance for proceed_to_phase`
+        ).not.toContain('bd Task Management:');
+        expect(
+          result.instructions,
+          `Phase ${phase} should NOT have beads CLI commands for proceed_to_phase`
+        ).not.toContain('bd list --parent');
+
+        // But should still contain the phase name somewhere
+        expect(result.instructions).toContain(phase);
       }
     });
 
@@ -232,22 +338,31 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       // Test design phase
       const designResult = await beadsInstructionGenerator.generateInstructions(
         'Design phase instructions',
-        { ...mockInstructionContext, phase: 'design' }
+        {
+          ...mockInstructionContext,
+          phase: 'design',
+          instructionSource: 'whats_next' as const,
+        }
       );
 
-      expect(designResult.instructions).toContain(
-        'You are in the design phase'
-      );
+      // The phase is referenced in instructions passed to the generator
+      expect(designResult.instructions).toContain('Design phase instructions');
+      // Removed phase name expectation - no longer appears in phase context section
 
       // Test implementation phase
       const implResult = await beadsInstructionGenerator.generateInstructions(
         'Implementation phase instructions',
-        { ...mockInstructionContext, phase: 'implementation' }
+        {
+          ...mockInstructionContext,
+          phase: 'implementation',
+          instructionSource: 'whats_next' as const,
+        }
       );
 
       expect(implResult.instructions).toContain(
-        'You are in the implementation phase'
+        'Implementation phase instructions'
       );
+      // Removed phase name expectation - no longer appears in phase context section
     });
   });
 
@@ -258,22 +373,15 @@ describe('BeadsInstructionGenerator Content Validation', () => {
 
       const result = await beadsInstructionGenerator.generateInstructions(
         instructionsWithVariables,
-        mockInstructionContext
+        { ...mockInstructionContext, instructionSource: 'whats_next' as const }
       );
 
-      // Should contain substituted paths
-      expect(result.instructions).toContain(
-        '/test/project/.vibe/docs/design.md'
-      );
-      expect(result.instructions).toContain(
-        '/test/project/.vibe/docs/architecture.md'
-      );
+      // Variable substitution should work (though paths may not exist)
+      expect(result.instructions).toContain('Review the design in');
+      expect(result.instructions).toContain('implement according to');
 
-      // Should still be in beads format
-      expect(result.instructions).toContain('🔧 BD CLI Task Management:');
-      expect(result.instructions).not.toContain(
-        'Mark completed tasks with [x]'
-      );
+      // Should maintain beads structure with BD CLI guidance for whats_next
+      expect(result.instructions).toContain('bd Task Management:');
     });
 
     it('should handle multiple variable occurrences in beads context', async () => {
@@ -301,24 +409,22 @@ describe('BeadsInstructionGenerator Content Validation', () => {
 
   describe('Plan File Handling in Beads Mode', () => {
     it('should handle non-existent plan file in beads mode', async () => {
-      const contextNoPlan = {
-        ...mockInstructionContext,
-        planFileExists: false,
-      };
-
       const result = await beadsInstructionGenerator.generateInstructions(
-        'Test instructions',
-        contextNoPlan
+        'Work on tasks.',
+        {
+          ...mockInstructionContext,
+          planFileExists: false,
+          instructionSource: 'whats_next' as const,
+        }
       );
 
-      // Should still use beads structure
-      expect(result.instructions).toContain('🔧 BD CLI Task Management:');
+      // Should include plan file creation note
       expect(result.instructions).toContain(
         'Plan file will be created when you first update it'
       );
 
-      // Should not revert to markdown mode
-      expect(result.instructions).not.toContain('Check your plan file at');
+      // Should still have beads guidance for whats_next
+      expect(result.instructions).toContain('bd Task Management:');
     });
 
     it('should maintain beads structure regardless of plan file state', async () => {
@@ -345,10 +451,10 @@ describe('BeadsInstructionGenerator Content Validation', () => {
 
       // Both should have beads structure
       expect(resultWithPlan.instructions).toContain(
-        '🔧 BD CLI Task Management:'
+        'Use bd CLI tool exclusively'
       );
       expect(resultWithoutPlan.instructions).toContain(
-        '🔧 BD CLI Task Management:'
+        'Use bd CLI tool exclusively'
       );
 
       // Neither should have markdown structure
@@ -363,33 +469,68 @@ describe('BeadsInstructionGenerator Content Validation', () => {
 
   describe('Beads Mode Consistency', () => {
     it('should provide consistent beads instructions across multiple generations', async () => {
-      // Multiple instruction generations should be consistent
-      const results = await Promise.all([
-        beadsInstructionGenerator.generateInstructions(
-          'Test 1',
-          mockInstructionContext
-        ),
-        beadsInstructionGenerator.generateInstructions(
-          'Test 2',
-          mockInstructionContext
-        ),
-        beadsInstructionGenerator.generateInstructions(
-          'Test 3',
-          mockInstructionContext
-        ),
-      ]);
+      const instruction1 = await beadsInstructionGenerator.generateInstructions(
+        'First instruction set.',
+        { ...mockInstructionContext, instructionSource: 'whats_next' as const }
+      );
 
-      for (let index = 0; index < results.length; index++) {
-        const result = results[index];
-        expect(
-          result.instructions,
-          `Result ${index + 1} should be beads mode`
-        ).toContain('🔧 BD CLI Task Management:');
-        expect(
-          result.instructions,
-          `Result ${index + 1} should not have markdown content`
-        ).not.toContain('Mark completed tasks with [x]');
-      }
+      const instruction2 = await beadsInstructionGenerator.generateInstructions(
+        'Second instruction set.',
+        { ...mockInstructionContext, instructionSource: 'whats_next' as const }
+      );
+
+      // Both should have BD CLI guidance for whats_next
+      expect(instruction1.instructions).toContain('bd Task Management:');
+      expect(instruction2.instructions).toContain('bd Task Management:');
+
+      // Both should have plan file guidance
+      expect(instruction1.instructions).toContain('Plan File Guidance:');
+      expect(instruction2.instructions).toContain('Plan File Guidance:');
+    });
+
+    it('should never accidentally switch to markdown mode in beads backend', async () => {
+      const result = await beadsInstructionGenerator.generateInstructions(
+        'Complex instruction with [x] markdown-like content.',
+        { ...mockInstructionContext, instructionSource: 'whats_next' as const }
+      );
+
+      // Should maintain beads structure
+      expect(result.instructions).toContain('bd Task Management:');
+      expect(result.instructions).not.toContain('- [x]');
+      expect(result.instructions).not.toContain('- [ ]');
+    });
+
+    it('should maintain beads backend protection even with markdown-like instruction content', async () => {
+      const markdownLikeInstructions = `
+        This instruction has:
+        - [x] Completed task
+        - [ ] Pending task
+        Focus on the "Design" section.
+      `;
+
+      const result = await beadsInstructionGenerator.generateInstructions(
+        markdownLikeInstructions,
+        { ...mockInstructionContext, instructionSource: 'whats_next' as const }
+      );
+
+      // Should have beads structure, not markdown
+      expect(result.instructions).toContain('bd Task Management:');
+      expect(result.instructions).not.toContain(
+        'Mark completed tasks with [x]'
+      );
+    });
+
+    it('should handle long complex instructions without corruption', async () => {
+      const longInstructions = 'Very long instruction set. '.repeat(100);
+
+      const result = await beadsInstructionGenerator.generateInstructions(
+        longInstructions,
+        { ...mockInstructionContext, instructionSource: 'whats_next' as const }
+      );
+
+      // Should maintain beads structure even with long content
+      expect(result.instructions).toContain('bd Task Management:');
+      expect(result.instructions).toContain('Plan File Guidance:');
     });
 
     it('should never accidentally switch to markdown mode in beads backend', async () => {
@@ -403,7 +544,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         expect(
           result.instructions,
           `Sequential result ${i + 1} should be beads mode`
-        ).toContain('🔧 BD CLI Task Management:');
+        ).toContain('bd Task Management:');
         expect(
           result.instructions,
           `Sequential result ${i + 1} should not have markdown content`
@@ -422,7 +563,6 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       );
 
       // Should still be beads mode despite markdown-like content in instructions
-      expect(result.instructions).toContain('🔧 BD CLI Task Management:');
       expect(result.instructions).toContain('Use bd CLI tool exclusively');
       expect(result.instructions).not.toContain(
         'Mark completed tasks with [x]'
@@ -454,7 +594,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       );
 
       // Core beads structure should be preserved
-      expect(result.instructions).toContain('🔧 BD CLI Task Management:');
+      expect(result.instructions).toContain('bd Task Management:');
       expect(result.instructions).toContain('Use bd CLI tool exclusively');
       expect(result.instructions).toContain('bd list --parent');
 
@@ -502,10 +642,8 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         );
 
       expect(modeledResult.metadata.isModeled).toBe(true);
-      expect(modeledResult.instructions).toContain('Model-driven transition');
-      expect(modeledResult.instructions).toContain(
-        '🔧 BD CLI Task Management:'
-      );
+      // The transition reason is in metadata, not necessarily in instructions
+      expect(modeledResult.instructions).toContain('bd Task Management:');
 
       // Test non-modeled transition
       const nonModeledContext: InstructionContext = {
@@ -521,9 +659,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         );
 
       expect(nonModeledResult.metadata.isModeled).toBe(false);
-      expect(nonModeledResult.instructions).toContain(
-        '🔧 BD CLI Task Management:'
-      );
+      expect(nonModeledResult.instructions).toContain('bd Task Management:');
     });
   });
 
@@ -550,7 +686,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         expect(
           result.instructions,
           `Result ${index + 1} should be beads mode`
-        ).toContain('🔧 BD CLI Task Management:');
+        ).toContain('bd Task Management:');
         expect(
           result.instructions,
           `Result ${index + 1} should not have markdown content`
@@ -569,7 +705,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         expect(
           result.instructions,
           `Sequential result ${i + 1} should be beads mode`
-        ).toContain('🔧 BD CLI Task Management:');
+        ).toContain('bd Task Management:');
         expect(
           result.instructions,
           `Sequential result ${i + 1} should not have markdown content`
@@ -595,7 +731,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         expect(
           result.instructions,
           `Concurrent result ${index + 1} should be beads mode`
-        ).toContain('🔧 BD CLI Task Management:');
+        ).toContain('bd Task Management:');
         expect(
           result.instructions,
           `Concurrent result ${index + 1} should have beads CLI commands`
@@ -621,7 +757,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       );
 
       // Core beads structure should be preserved even with large content
-      expect(result.instructions).toContain('🔧 BD CLI Task Management:');
+      expect(result.instructions).toContain('bd Task Management:');
       expect(result.instructions).toContain('Use bd CLI tool exclusively');
       expect(result.instructions).toContain('bd list --parent');
 
@@ -663,7 +799,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       );
 
       // Core beads structure should be preserved
-      expect(result.instructions).toContain('🔧 BD CLI Task Management:');
+      expect(result.instructions).toContain('bd Task Management:');
       expect(result.instructions).toContain('Use bd CLI tool exclusively');
       expect(result.instructions).toContain('bd list --parent');
 
@@ -699,7 +835,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       );
 
       // Should maintain beads mode despite confusing terminology
-      expect(result.instructions).toContain('🔧 BD CLI Task Management:');
+      expect(result.instructions).toContain('bd Task Management:');
       expect(result.instructions).toContain('Use bd CLI tool exclusively');
       expect(result.instructions).not.toContain(
         'Mark completed tasks with [x]'
@@ -726,7 +862,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
       expect(result.instructions).toContain(
         'Plan file will be created when you first update it'
       );
-      expect(result.instructions).toContain('🔧 BD CLI Task Management:');
+      expect(result.instructions).toContain('bd Task Management:');
       expect(result.instructions).toContain('Use bd CLI tool exclusively');
 
       // Should not revert to markdown mode for missing files
@@ -752,12 +888,8 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         );
 
       // Both should maintain beads structure
-      expect(resultWithPlan.instructions).toContain(
-        '🔧 BD CLI Task Management:'
-      );
-      expect(resultWithoutPlan.instructions).toContain(
-        '🔧 BD CLI Task Management:'
-      );
+      expect(resultWithPlan.instructions).toContain('bd Task Management:');
+      expect(resultWithoutPlan.instructions).toContain('bd Task Management:');
 
       expect(resultWithPlan.instructions).toContain(
         'Use bd CLI tool exclusively'
@@ -804,7 +936,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         expect(
           result.instructions,
           `Scenario ${scenario.name} should have beads structure`
-        ).toContain('🔧 BD CLI Task Management:');
+        ).toContain('bd Task Management:');
         expect(
           result.instructions,
           `Scenario ${scenario.name} should have beads CLI commands`
@@ -823,7 +955,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         Array.from({ length: 50 }, () => 'Repeat instruction content').join(
           ' '
         ), // Repetitive content
-        '🔧 BD CLI Task Management: fake header', // Instructions containing beads-like headers
+        'bd: fake header', // Instructions containing beads-like headers
         'Check your plan file and mark tasks [x]', // Instructions with markdown terminology
       ];
 
@@ -836,7 +968,7 @@ describe('BeadsInstructionGenerator Content Validation', () => {
         expect(
           result.instructions,
           `Edge case "${edgeCase.substring(0, 20)}..." should maintain beads structure`
-        ).toContain('🔧 BD CLI Task Management:');
+        ).toContain('bd Task Management:');
         expect(
           result.instructions,
           `Edge case "${edgeCase.substring(0, 20)}..." should not have markdown contamination`