Skip to content

feat: split childProcess integration into childProcess and worker integrations - #22884

Closed
atharv-sys32 wants to merge 1 commit into
getsentry:developfrom
atharv-sys32:feat/split-child-process-worker
Closed

feat: split childProcess integration into childProcess and worker integrations#22884
atharv-sys32 wants to merge 1 commit into
getsentry:developfrom
atharv-sys32:feat/split-child-process-worker

Conversation

@atharv-sys32

Copy link
Copy Markdown

The childProcess integration handled both child process and worker thread events under one integration. This splits it into two separate integrations:

  • childProcessIntegration: handles child process events only
  • workerIntegration: handles worker thread events only (new)

The old childProcessIntegration's captureWorkerErrors option is deprecated. Users should use workerIntegration() instead.

Fixes #18698

@atharv-sys32
atharv-sys32 requested a review from a team as a code owner July 30, 2026 19:56
@atharv-sys32
atharv-sys32 requested review from msonnb and stephanie-anderson and removed request for a team July 30, 2026 19:56
Comment thread packages/node/src/integrations/workerIntegration.ts
Comment thread packages/node/src/integrations/childProcess.ts
Comment thread packages/node/src/integrations/childProcess.ts
Comment thread packages/node/src/integrations/workerIntegration.ts
Comment thread packages/node/src/integrations/workerIntegration.ts
Comment thread packages/node/src/index.ts
@atharv-sys32
atharv-sys32 force-pushed the feat/split-child-process-worker branch from 11c03c7 to 7f333fe Compare July 30, 2026 20:10
Comment thread packages/node/src/integrations/workerIntegration.ts
Comment thread packages/node/src/integrations/workerIntegration.ts Outdated
@atharv-sys32
atharv-sys32 force-pushed the feat/split-child-process-worker branch from 7f333fe to fce1cb8 Compare July 30, 2026 21:03
@atharv-sys32
atharv-sys32 requested a review from a team as a code owner July 30, 2026 21:03
@atharv-sys32
atharv-sys32 requested review from isaacs and mydea and removed request for a team July 30, 2026 21:03
Comment thread packages/node/src/integrations/childProcess.ts
@atharv-sys32
atharv-sys32 force-pushed the feat/split-child-process-worker branch from fce1cb8 to adc7c01 Compare July 30, 2026 21:09
Comment thread packages/node/src/integrations/childProcess.ts Outdated
Comment thread packages/node/src/integrations/workerIntegration.ts Outdated
@atharv-sys32
atharv-sys32 force-pushed the feat/split-child-process-worker branch from adc7c01 to 1a25c95 Compare July 30, 2026 21:17
Comment thread packages/node/src/integrations/childProcess.ts
@atharv-sys32
atharv-sys32 force-pushed the feat/split-child-process-worker branch from 1a25c95 to 900acc1 Compare July 30, 2026 21:27

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 900acc1. Configure here.

Comment thread packages/node/src/integrations/childProcess.ts Outdated
@atharv-sys32
atharv-sys32 force-pushed the feat/split-child-process-worker branch from 900acc1 to e66783d Compare July 30, 2026 21:39
Comment thread packages/node/src/integrations/workerIntegration.ts
…egrations

The childProcess integration handled both child process and worker
thread events under one integration. This splits it into two:
- childProcessIntegration: handles child process events only
- workerIntegration: handles worker thread events only (new)

The old childProcessIntegration's captureWorkerErrors option is
deprecated. Users should use workerIntegration() instead.

Fixes getsentry#18698

Signed-off-by: Atharv Pandey <atharvpandey245@gmail.com>
@atharv-sys32
atharv-sys32 force-pushed the feat/split-child-process-worker branch from e66783d to 92a8a25 Compare July 30, 2026 21:47
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.

Split up childProcess integration into childProcess and worker integration

1 participant