@codemcp/workflows 4.10.1 → 4.10.3

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();
     });
   });
 });