Skip to content

Conversation

@haxzie
Copy link
Collaborator

@haxzie haxzie commented Oct 17, 2025

Note

Removes redundant connected-account lookups and consolidates tool execution with centralized before/after modifiers and improved API error-to-SDK error mapping.

  • Core (tool execution):
    • Remove pre-execution connected account fetch and per-tool execution handlers; add executeComposioTool to call API and transform responses.
    • Centralize modifier application in execute() for both custom and Composio tools; route to customTools.executeCustomTool or executeComposioTool.
  • Errors:
    • Add handleToolExecutionError to parse APIError and map codes (e.g., 1803) to specific errors like ComposioConnectedAccountNotFoundError.
    • Simplify ComposioToolExecutionError constructor and expand possible fixes.
  • Tests:
    • Update test utilities to remove getConnectedAccountIdForTool spying and related setup.

Written by Cursor Bugbot for commit 05a6a7c. This will update automatically on new commits. Configure here.

cursor[bot]

This comment was marked as outdated.

@haxzie haxzie changed the title Fix: Remove unecessary API calls during tool execution and Handle execution errors better Fix: Remove unnecessary API calls during tool execution and Handle execution errors better Oct 17, 2025
@haxzie haxzie merged commit 1c13c52 into next Oct 17, 2025
5 checks passed
@haxzie haxzie deleted the haxzie/fix/improve-execution-performance-2025-10-17 branch October 17, 2025 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants