Skip to content

Commit bad1163

Browse files
authored
fix: temporarily disable MCP Session ID error handling to resolve stdio transport issues (#373)
1 parent 8a9197b commit bad1163

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

.gitignore

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,5 +35,3 @@ __pycache__/
3535
*.pyo
3636
*.pyd
3737
.Python
38-
39-
mcp/

src/mcp/server.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -631,7 +631,8 @@ export class ActorsMcpServer {
631631
const mcpSessionId = typeof request.params.mcpSessionId === 'string' ? request.params.mcpSessionId : undefined;
632632
if (!mcpSessionId) {
633633
log.error('MCP Session ID is missing in tool call request. This should never happen.');
634-
throw new Error('MCP Session ID is required for tool calls');
634+
// TEMP: do not throw for now as it causes issues with stdio transport
635+
// throw new Error('MCP Session ID is required for tool calls');
635636
}
636637
// Remove apifyToken from request.params just in case
637638
delete request.params.apifyToken;

src/tools/actor.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -205,9 +205,10 @@ Actor description: ${definition.description}`;
205205
openWorldHint: true,
206206
},
207207
// Allow long running tasks for Actor tools, make it optional for now
208-
execution: {
209-
taskSupport: 'optional',
210-
},
208+
// TEMP: disable for now as it causes issues with session id error for stdio transport
209+
// execution: {
210+
// taskSupport: 'optional',
211+
// },
211212
});
212213
}
213214
return tools;

0 commit comments

Comments
 (0)