improvement(tools): removed transformError, isInternalRoute, directExecution#928
Merged
waleedlatif1 merged 9 commits intostagingfrom Aug 11, 2025
Merged
improvement(tools): removed transformError, isInternalRoute, directExecution#928waleedlatif1 merged 9 commits intostagingfrom
waleedlatif1 merged 9 commits intostagingfrom
Conversation
…ifferent error formats
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
arenadeveloper02
pushed a commit
to arenadeveloper02/p2-sim
that referenced
this pull request
Sep 19, 2025
…ecution (simstudioai#928) * standardized response format for transformError * removed trasnformError, moved error handling to executeTool for all different error formats * remove isInternalRoute, make it implicit in executeTool * removed directExecution, everything on the server nothing on the client * fix supabase * fix(tag-dropdown): fix values for parallel & loop blocks (simstudioai#929) * fix(search-modal): add parallel and loop blocks to search modal * reordered tool params * update docs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
removed transformError, isInternalRoute, directExecution. simpler tool definition makes it easier to add more and all functionality was maintained and error handling overall was improved from tools.
Type of Change
Testing
Tested manually, ensured everything works and that error handling was improved
Checklist