Skip to content

improvement(tools): removed transformError, isInternalRoute, directExecution#928

Merged
waleedlatif1 merged 9 commits intostagingfrom
feat/block
Aug 11, 2025
Merged

improvement(tools): removed transformError, isInternalRoute, directExecution#928
waleedlatif1 merged 9 commits intostagingfrom
feat/block

Conversation

@waleedlatif1
Copy link
Collaborator

  • 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

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

  • Other: Refactoring

Testing

Tested manually, ensured everything works and that error handling was improved

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link

vercel bot commented Aug 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 10, 2025 8:19pm
sim ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 10, 2025 8:19pm

@vercel vercel bot temporarily deployed to Preview – docs August 10, 2025 20:13 Inactive
@vercel vercel bot temporarily deployed to Preview – sim August 10, 2025 20:14 Inactive
@waleedlatif1 waleedlatif1 merged commit 56ede1c into staging Aug 11, 2025
5 checks passed
@waleedlatif1 waleedlatif1 deleted the feat/block branch August 11, 2025 00:19
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
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.

1 participant