fix: defer source-map-support loading to prevent OOM with Sentry debug injection

[akramcodez]

Description

Fixes #2920 - Prevents OOM crashes when using Sentry's debug ID injection with large projects on small machines.

Problem

• Sentry injects new Error().stack into all .mjs files
• source-map-support eagerly parses all sourcemaps during module loading
• 2000+ files × sourcemap parsing = OOM on small-1x (0.5GB RAM)

Solution

• Defer source-map-support installation until after config loads
• Use async import to avoid blocking module initialization
• Add sourceMapSupport config option (default: true) to allow disabling
• Add error handling for graceful degradation

Changes

• packages/cli-v3/src/entryPoints/dev-index-worker.ts - Deferred loading
• packages/cli-v3/src/entryPoints/dev-run-worker.ts - Deferred loading
• packages/cli-v3/src/entryPoints/managed-index-worker.ts - Deferred loading
• packages/cli-v3/src/entryPoints/managed-run-worker.ts - Deferred loading
• packages/core/src/v3/config.ts - New sourceMapSupport option

[changeset-bot]

⚠️ No Changeset found

Latest commit: 15091dc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[coderabbitai]

Walkthrough

This pull request refactors source-map-support initialization across four entry point files in the CLI package from eager, top-level installation to lazy, deferred installation. Each entry point now includes an installSourceMapSupport() function that dynamically imports source-map-support at runtime with error handling, guarded by a flag to prevent duplicate installations. Installation is deferred until after configuration is loaded and only executed if config.sourceMapSupport is not explicitly false. A new optional sourceMapSupport boolean property is added to TriggerConfig to provide explicit control over this behavior, defaulting to true with a note about potential OOM issues in large projects using Sentry's debug ID injection.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	The description provides context (problem, solution, changes) and lists affected files. However, it does not fully follow the template structure, missing testing details, changelog summary, and the provided template checklist.	Add testing procedures, expand changelog section, and include the template checklist to meet repository standards.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: deferring source-map-support loading to prevent OOM issues with Sentry debug injection. It is concise and directly related to the core problem being solved.
Linked Issues check	✅ Passed	All coding objectives from issue #2920 are met: source-map-support installation is deferred to post-config, async import is used, sourceMapSupport config option is added with default true, and error handling is implemented.
Out of Scope Changes check	✅ Passed	All changes are directly aligned with addressing the OOM issue: deferred loading in four worker entry points and a config option addition. No unrelated modifications are present.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 23ec5ff and 15091dc.

📒 Files selected for processing (5)

• packages/cli-v3/src/entryPoints/dev-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-run-worker.ts
• packages/cli-v3/src/entryPoints/managed-index-worker.ts
• packages/cli-v3/src/entryPoints/managed-run-worker.ts
• packages/core/src/v3/config.ts

🧰 Additional context used

📓 Path-based instructions (6)

**/*.{ts,tsx}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

**/*.{ts,tsx}: Use types over interfaces for TypeScript
Avoid using enums; prefer string unions or const objects instead

**/*.{ts,tsx}: Always import tasks from @trigger.dev/sdk, never use @trigger.dev/sdk/v3 or deprecated client.defineJob pattern
Every Trigger.dev task must be exported and have a unique id property with no timeouts in the run function

Files:

• packages/core/src/v3/config.ts
• packages/cli-v3/src/entryPoints/managed-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-run-worker.ts
• packages/cli-v3/src/entryPoints/managed-run-worker.ts

{packages/core,apps/webapp}/**/*.{ts,tsx}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Use zod for validation in packages/core and apps/webapp

Files:

• packages/core/src/v3/config.ts

**/*.{ts,tsx,js,jsx}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Use function declarations instead of default exports

Import from @trigger.dev/core using subpaths only, never import from root

Files:

• packages/core/src/v3/config.ts
• packages/cli-v3/src/entryPoints/managed-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-run-worker.ts
• packages/cli-v3/src/entryPoints/managed-run-worker.ts

**/*.ts

📄 CodeRabbit inference engine (.cursor/rules/otel-metrics.mdc)

**/*.ts: When creating or editing OTEL metrics (counters, histograms, gauges), ensure metric attributes have low cardinality by using only enums, booleans, bounded error codes, or bounded shard IDs
Do not use high-cardinality attributes in OTEL metrics such as UUIDs/IDs (envId, userId, runId, projectId, organizationId), unbounded integers (itemCount, batchSize, retryCount), timestamps (createdAt, startTime), or free-form strings (errorMessage, taskName, queueName)
When exporting OTEL metrics via OTLP to Prometheus, be aware that the exporter automatically adds unit suffixes to metric names (e.g., 'my_duration_ms' becomes 'my_duration_ms_milliseconds', 'my_counter' becomes 'my_counter_total'). Account for these transformations when writing Grafana dashboards or Prometheus queries

Files:

• packages/core/src/v3/config.ts
• packages/cli-v3/src/entryPoints/managed-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-run-worker.ts
• packages/cli-v3/src/entryPoints/managed-run-worker.ts

**/*.{js,ts,jsx,tsx,json,md,yaml,yml}

📄 CodeRabbit inference engine (AGENTS.md)

Format code using Prettier before committing

Files:

• packages/core/src/v3/config.ts
• packages/cli-v3/src/entryPoints/managed-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-run-worker.ts
• packages/cli-v3/src/entryPoints/managed-run-worker.ts

{packages,integrations}/**/*

📄 CodeRabbit inference engine (CLAUDE.md)

Add a changeset when modifying any public package in packages/* or integrations/* using pnpm run changeset:add

Files:

• packages/core/src/v3/config.ts
• packages/cli-v3/src/entryPoints/managed-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-run-worker.ts
• packages/cli-v3/src/entryPoints/managed-run-worker.ts

🧠 Learnings (14)

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger.config.ts : Configure OpenTelemetry instrumentations and exporters in trigger.config.ts for enhanced logging

Applied to files:

• packages/core/src/v3/config.ts
• packages/cli-v3/src/entryPoints/managed-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-index-worker.ts

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger.config.ts : Use build extensions in trigger.config.ts (additionalFiles, additionalPackages, aptGet, prismaExtension, etc.) to customize the build

Applied to files:

• packages/core/src/v3/config.ts

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger.config.ts : Configure build process in trigger.config.ts using `build` object with external packages, extensions, and JSX settings

Applied to files:

• packages/core/src/v3/config.ts

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger.config.ts : Configure log level in trigger.config.ts using `logLevel` property

Applied to files:

• packages/core/src/v3/config.ts

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger.config.ts : Set default maximum duration in trigger.config.ts using `maxDuration` property

Applied to files:

• packages/core/src/v3/config.ts

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger.config.ts : Configure Trigger.dev project in `trigger.config.ts` using `defineConfig()` with project ref and task directories

Applied to files:

• packages/core/src/v3/config.ts

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger.config.ts : Set default machine configuration in trigger.config.ts using `defaultMachine` property

Applied to files:

• packages/core/src/v3/config.ts

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger/**/*.{ts,tsx,js,jsx} : Attach metadata to task runs using the metadata option when triggering, and access/update it inside runs using metadata functions

Applied to files:

• packages/core/src/v3/config.ts

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger/**/*.{ts,tsx,js,jsx} : Use `trigger.dev/sdk/v3` for all imports in Trigger.dev tasks

Applied to files:

• packages/core/src/v3/config.ts

📚 Learning: 2025-11-27T16:27:35.304Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger/**/*.{ts,tsx,js,jsx} : Use `schemaTask()` from `trigger.dev/sdk/v3` with Zod schema for payload validation

Applied to files:

• packages/cli-v3/src/entryPoints/managed-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-index-worker.ts

📚 Learning: 2025-11-27T16:26:58.661Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/webapp.mdc:0-0
Timestamp: 2025-11-27T16:26:58.661Z
Learning: Applies to apps/webapp/**/*.{ts,tsx} : When importing from `trigger.dev/core` in the webapp, use subpath exports from the package.json instead of importing from the root path

Applied to files:

• packages/cli-v3/src/entryPoints/dev-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-run-worker.ts

📚 Learning: 2026-01-15T11:50:06.067Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-01-15T11:50:06.067Z
Learning: Applies to **/*.{ts,tsx} : Always import tasks from `trigger.dev/sdk`, never use `trigger.dev/sdk/v3` or deprecated `client.defineJob` pattern

Applied to files:

• packages/cli-v3/src/entryPoints/dev-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-run-worker.ts

📚 Learning: 2026-01-15T11:50:06.067Z

Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-01-15T11:50:06.067Z
Learning: Applies to **/*.{ts,tsx,js,jsx} : Import from `trigger.dev/core` using subpaths only, never import from root

Applied to files:

• packages/cli-v3/src/entryPoints/dev-index-worker.ts
• packages/cli-v3/src/entryPoints/dev-run-worker.ts

📚 Learning: 2025-11-26T14:40:07.146Z

Learnt from: ericallam
Repo: triggerdotdev/trigger.dev PR: 2710
File: packages/schema-to-json/package.json:0-0
Timestamp: 2025-11-26T14:40:07.146Z
Learning: Node.js 24+ has native TypeScript support and can execute .ts files directly without tsx or ts-node for scripts that use only erasable TypeScript syntax (type annotations, interfaces, etc.). The trigger.dev repository uses Node.js 24.11.1+ and scripts like updateVersion.ts can be run with `node` instead of `tsx`.

Applied to files:

• packages/cli-v3/src/entryPoints/dev-run-worker.ts
• packages/cli-v3/src/entryPoints/managed-run-worker.ts

🔇 Additional comments (9)

packages/cli-v3/src/entryPoints/dev-index-worker.ts (2)

22-38: Deferred install helper looks solid.
The guard and try/catch make repeated calls safe and prevent startup failures.

---

94-97: Config-gated install aligns with the new flag.
Loading config before deciding to install keeps defaults while enabling opt-out.

packages/cli-v3/src/entryPoints/dev-run-worker.ts (2)

71-87: Deferred install helper looks solid.
The guard and try/catch make repeated calls safe and prevent startup failures.

---

285-288: Config-gated install aligns with the new flag.
Loading config before deciding to install keeps defaults while enabling opt-out.

packages/cli-v3/src/entryPoints/managed-index-worker.ts (2)

22-38: Deferred install helper looks solid.
The guard and try/catch make repeated calls safe and prevent startup failures.

---

94-97: Config-gated install aligns with the new flag.
Loading config before deciding to install keeps defaults while enabling opt-out.

packages/cli-v3/src/entryPoints/managed-run-worker.ts (2)

71-87: Deferred install helper looks solid.
The guard and try/catch make repeated calls safe and prevent startup failures.

---

264-267: Config-gated install aligns with the new flag.
Loading config before deciding to install keeps defaults while enabling opt-out.

packages/core/src/v3/config.ts (1)

174-181: No action needed. The sourceMapSupport field is properly defined in TriggerConfig and correctly used throughout the codebase. TriggerConfig is a TypeScript type definition without Zod validation; config loading in the CLI uses c12 and applies optional defaults merging, not schema validation. The field will not be rejected or stripped.

Likely an incorrect or invalid review comment.

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[vibe-kanban-cloud]

Review Complete

Your review story is ready!

View Story

Comment !reviewfast on this PR to re-generate the story.