Skip to content

v0.3.45: fixes for organization invites, custom tool execution#1234

Merged
waleedlatif1 merged 2 commits intomainfrom
staging
Sep 3, 2025
Merged

v0.3.45: fixes for organization invites, custom tool execution#1234
waleedlatif1 merged 2 commits intomainfrom
staging

Conversation

@waleedlatif1
Copy link
Collaborator

@vercel
Copy link

vercel bot commented Sep 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
sim (staging) Ready Ready Comment Sep 3, 2025 3:27pm
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
docs Skipped Skipped Sep 3, 2025 3:27pm

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Summary

This PR contains two targeted bug fixes addressing issues with organization invitations and custom tool execution.

The first change fixes organization invitation URLs by simplifying their structure from a query parameter pattern (/invite/organization?id={id}) to a cleaner path parameter pattern (/invite/{id}). This affects the email rendering functions in the POST handler for organization invitations, making invitation URLs more concise and following better REST conventions. The change maintains the same invitation ID while just changing how it's encoded in the URL, suggesting the frontend routing has been updated to handle invitations with this simpler structure.

The second change ensures custom tools always use local VM execution instead of E2B (remote code execution) by adding a !isCustomTool condition to the execution decision logic. This modification forces custom tools to bypass E2B execution and use the local VM environment, which provides consistent variable and environment variable resolution. The change is necessary because custom tools require special parameter handling where their parameters are made directly accessible as variables in the execution context, and E2B execution has different variable resolution patterns that could lead to inconsistent behavior.

Both changes integrate well with the existing codebase architecture - the invitation URL change aligns with the existing invitation system while improving URL structure, and the custom tool execution change maintains the existing execution logic while ensuring proper parameter handling for custom tools.

Confidence score: 5/5

  • This PR is extremely safe to merge with virtually no risk of production issues
  • Score reflects simple, well-targeted changes that fix specific bugs without introducing complexity or side effects
  • No files require special attention as both changes are straightforward and isolated

2 files reviewed, no comments

Edit Code Review Bot Settings | Greptile

@waleedlatif1 waleedlatif1 merged commit 581929b into main Sep 3, 2025
16 checks passed
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