Skip to content

fix(trigger): deps to include e2b and daytona - #6205

Merged
icecrasher321 merged 1 commit into
stagingfrom
fix/trigger-dev-deps
Aug 3, 2026
Merged

fix(trigger): deps to include e2b and daytona#6205
icecrasher321 merged 1 commit into
stagingfrom
fix/trigger-dev-deps

Conversation

@icecrasher321

Copy link
Copy Markdown
Collaborator

Summary

Trigger dev deps to include e2b and daytona.

Type of Change

  • Bug fix

Testing

Tested manually

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

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Aug 3, 2026 4:29pm

Request Review

@cursor

cursor Bot commented Aug 3, 2026

Copy link
Copy Markdown

PR Summary

Cursor Bugbot is generating a summary for commit e8ce63f. Configure here.

@greptile-apps

greptile-apps Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR updates Trigger.dev deployment packaging so E2B and Daytona SDKs remain available as runtime dependencies instead of being statically bundled.

  • Externalizes e2b, @e2b/code-interpreter, and @daytona/sdk during Trigger.dev builds.
  • Adds the E2B code-interpreter and Daytona SDK packages to deployed runtime dependencies.

Confidence Score: 5/5

The PR appears safe to merge because the changed Trigger.dev configuration consistently packages the SDKs used by the E2B and Daytona runtime paths.

The newly externalized SDKs are also included as deployment dependencies, while the bare E2B package is supplied transitively by the included code-interpreter package.

Important Files Changed

Filename Overview
apps/sim/trigger.config.ts Aligns Trigger.dev externalization and deployment dependency configuration for the E2B and Daytona runtime SDKs; no actionable defect was established.

Reviews (1): Last reviewed commit: "fix(trigger): deps to include e2b and da..." | Re-trigger Greptile

@icecrasher321
icecrasher321 merged commit f07160a into staging Aug 3, 2026
27 checks passed
@icecrasher321
icecrasher321 deleted the fix/trigger-dev-deps branch August 3, 2026 16:39
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