Draft PR: Effect Integration for Sentry Node SDK #17432
Open
+1,758
−2,171
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1. Goal
This PR adds support for tracing Effect computations in the Sentry Node SDK. The goal is to automatically create Sentry spans for Effect operations and capture errors that occur within Effect computations as Sentry exceptions, providing better observability for users of the Effect library.
2. Pathway Taken
effectIntegration
) and instrumentation (instrumentEffect
) under@sentry/node
.effect
package as a dependency and provided type definitions for Effect’s tracer API.3. How the integration has happened and where it is included now
packages/node/src/integrations/tracing/effect/
.4. How users should integrate Effect into their own project
Basic usage:
Sentry.effectIntegration()
to your integrations array.5. Where am i stuck / need help
6. What is my thinking about these errors?
I think these errors are due to me not being able to setup the test environment properly as in most errors..the problem is just Span processes being undefined.
I would appreciate some pointers of where i am going wrong and how should i setup the test environment properly to test.