improvement(tools): retire direct execution - #7207
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
@greptile-apps review this PR |
|
@cubic-dev-ai review this PR |
@icecrasher321 I can't start this review because your workspace has reached its free monthly review limit. cubic has reviewed 360,373 of the 360,000 allowed lines of code this month. Reviews resume on 1 September 2026 (in 4 days). Paid plans include much higher monthly review limits. Upgrade now to resume reviews. To help optimise your usage, you can tune cubic to get the most out of your usage limits:
|
Greptile SummaryThe PR replaces retired direct tool execution with registered, server-side semantic operations while retaining the existing tool contracts.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/internal/tool-operations/execute.ts | Establishes the common registered-operation execution boundary and forwards cancellation into implementations. |
| apps/sim/lib/internal/tool-operations/registry.server.ts | Registers server-only integration-family dispatchers without exposing their implementations through client metadata. |
| apps/sim/lib/internal/browser-use/operations/run-task.ts | Moves BrowserUse execution into a typed operation with provider-response validation, normalized errors, cancellation propagation, and session cleanup. |
| apps/sim/tools/types.ts | Removes the retired direct-execution type surface and defines the registered internal-operation configuration. |
| scripts/check-tool-registry-boundary.ts | Extends repository auditing to enforce the executable-registry and internal-operation boundary. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
Caller[Workflow or trusted caller] --> Tool[Tool definition]
Tool --> Projection[Operation input projection]
Projection --> Registry[Server operation registry]
Registry --> Family[Integration-family dispatcher]
Family --> Operation[Semantic operation]
Operation --> Provider[External provider or internal capability]
Provider --> Result[Structured tool response]
Reviews (7): Last reviewed commit: "fix(tools): update supabase buckets atom..." | Re-trigger Greptile
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
05b66a3 to
52f86da
Compare
|
@cubic-dev-ai review this PR |
@icecrasher321 I have started the AI code review. It will take a few minutes to complete. |
cf5f56f to
3d64d27
Compare
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 229 files
Note: This PR contains a large number of files. cubic selects up to 200 of the highest-priority eligible files for this review, so some files may not have been reviewed.
Tip: instead of fixing issues one by one fix them all with cubic
Re-trigger cubic
|
@cubic-dev-ai review this PR |
@icecrasher321 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 237 files
Note: This PR contains a large number of files. cubic selects up to 200 of the highest-priority eligible files for this review, so some files may not have been reviewed.
Tip: instead of fixing issues one by one fix them all with cubic
Re-trigger cubic
|
@cubic-dev-ai review this PR |
|
@greptile review this PR |
@icecrasher321 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 238 files
Note: This PR contains a large number of files. cubic selects up to 200 of the highest-priority eligible files for this review, so some files may not have been reviewed.
Fix all with cubic | Re-trigger cubic
afe5e33 to
870d7ef
Compare
|
@cubic-dev-ai review this PR |
|
@greptile review this PR |
@icecrasher321 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 238 files
Note: This PR contains a large number of files. cubic selects up to 200 of the highest-priority eligible files for this review, so some files may not have been reviewed.
Fix all with cubic | Re-trigger cubic
870d7ef to
ddcbb26
Compare
|
@cubic-dev-ai review this PR |
|
@greptile review this PR |
@icecrasher321 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
No issues found across 238 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Note: This PR contains a large number of files. cubic selects up to 200 of the highest-priority eligible files for this review, so some files may not have been reviewed.
You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.
Re-trigger cubic
|
@cubic-dev-ai review this PR |
|
@greptile review this PR |
@icecrasher321 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
No issues found across 238 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Note: This PR contains a large number of files. cubic selects up to 200 of the highest-priority eligible files for this review, so some files may not have been reviewed.
You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.
Re-trigger cubic
1 participant
Summary
directExecutionruntime and type surfacelib/internal/<family>/Type of Change
Testing
bun run lintbun run check:auditsbun run type-checkChecklist