Skip to content

feat(server-utils): Rewrite @opentelemetry/instrumentation-hapi to orchestrion#21866

Open
chargome wants to merge 4 commits into
developfrom
charlygomez/js-2404-rewrite-opentelemetryinstrumentation-hapi-to-orchestrion
Open

feat(server-utils): Rewrite @opentelemetry/instrumentation-hapi to orchestrion#21866
chargome wants to merge 4 commits into
developfrom
charlygomez/js-2404-rewrite-opentelemetryinstrumentation-hapi-to-orchestrion

Conversation

@chargome

@chargome chargome commented Jun 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Member
  • Adds hapiChannelIntegration in @sentry/server-utils for injecting orchestrion channels
  • A start-only subscriber swaps route handlers / ext methods
  • Span-building helpers are ported OTel-free into hapi-utils.ts / hapi-types.ts (structural types, no @hapi/* runtime dep), HTTP attribute keys come from @sentry/conventions/attributes.

closes #20753

@chargome chargome self-assigned this Jun 30, 2026
@linear-code

linear-code Bot commented Jun 30, 2026

Copy link
Copy Markdown

JS-2404

@chargome chargome changed the title feat(server-utils): Rewrite @opentelemetry/instrumentation-hapi to orchestrion feat(server-utils): Rewrite @opentelemetry/instrumentation-hapi to orchestrion Jun 30, 2026
@chargome

chargome commented Jun 30, 2026

Copy link
Copy Markdown
Member Author

bugbot run

cursor[bot]
cursor Bot reviewed Jun 30, 2026
View reviewed changes
Comment thread packages/server-utils/src/integrations/tracing-channel/hapi-utils.ts
@chargome

chargome commented Jun 30, 2026

Copy link
Copy Markdown
Member Author

bugbot run

cursor[bot]
cursor Bot reviewed Jun 30, 2026
View reviewed changes

@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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 993f38b. Configure here.

@chargome chargome force-pushed the charlygomez/js-2404-rewrite-opentelemetryinstrumentation-hapi-to-orchestrion branch from 993f38b to fadc582 Compare July 1, 2026 10:30
@chargome chargome marked this pull request as ready for review July 1, 2026 12:20
@chargome chargome requested a review from a team as a code owner July 1, 2026 12:20
@chargome chargome requested review from JPeer264, andreiborza and mydea and removed request for a team July 1, 2026 12:20
@nicohrubec nicohrubec self-requested a review July 1, 2026 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursor cursor[bot] cursor[bot] left review comments
@mydea mydea Awaiting requested review from mydea mydea is a code owner automatically assigned from getsentry/team-javascript-sdks-server
@JPeer264 JPeer264 Awaiting requested review from JPeer264 JPeer264 is a code owner automatically assigned from getsentry/team-javascript-sdks-server
@andreiborza andreiborza Awaiting requested review from andreiborza andreiborza is a code owner automatically assigned from getsentry/team-javascript-sdks-server
@nicohrubec nicohrubec Awaiting requested review from nicohrubec

At least 1 approving review is required to merge this pull request.

Assignees

@chargome chargome

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Rewrite @opentelemetry/instrumentation-hapi to orchestrion

1 participant

@chargome