feat(nestjs): add orchestrion diagnostics-channel instrumentation#22176
Merged
Conversation
2ace4ec to
d7b514a
Compare
Contributor
size-limit report 📦
|
1eb7633 to
fa0de85
Compare
d7b514a to
9d8bcb6
Compare
fa0de85 to
dd92103
Compare
9d8bcb6 to
f3a502e
Compare
dd92103 to
b74dcbd
Compare
f3a502e to
d63141c
Compare
b74dcbd to
9315716
Compare
d63141c to
9e50193
Compare
9315716 to
28b5141
Compare
03c4d95 to
e554559
Compare
28b5141 to
915478f
Compare
JPeer264
approved these changes
Jul 13, 2026
915478f to
b49e065
Compare
e554559 to
2fa98d0
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 2fa98d0. Configure here.
b49e065 to
5714894
Compare
2fa98d0 to
a01cbf6
Compare
5714894 to
139ac5e
Compare
a01cbf6 to
481c2cc
Compare
139ac5e to
b4e6fdc
Compare
481c2cc to
f16df75
Compare
b4e6fdc to
868f6a8
Compare
f16df75 to
93a8cbe
Compare
868f6a8 to
0ebbf46
Compare
93a8cbe to
588a170
Compare
0ebbf46 to
8f95306
Compare
588a170 to
e493838
Compare
e493838 to
47b1bce
Compare
Add the NestJS channel subscriber in `@sentry/nestjs`, plus its code-transform config and channel names in `@sentry/server-utils`. `nestIntegration` now uses the diagnostics-channel path when orchestrion is injected and the vendored OTel path otherwise. Only the span origin differs between the two implementations. Fix: #20947 Fix: #20904 Fix: #20905 Fix: #20906 Fix: #20907 Fix: JS-2519 Fix: JS-2477 Fix: JS-2478 Fix: JS-2479 Fix: JS-2480
47b1bce to
b5cd115
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.

Add the NestJS channel subscriber in
@sentry/nestjs, plus its code-transform config and channel names in@sentry/server-utils.nestIntegrationnow uses the diagnostics-channel path when orchestrion is injected and the vendored OTel path otherwise. Only the span origin differs between the two implementations.Fix: #20947
Fix: #20904
Fix: #20905
Fix: #20906
Fix: #20907
Fix: JS-2519
Fix: JS-2477
Fix: JS-2478
Fix: JS-2479
Fix: JS-2480