getsentry / getsentry/sentry-javascript

Confirm bun-integration-tests need no span streaming port

Open Beginner friendly
#24,134 1 comment 0 reactions 0 assignees View on GitHub
javascript Task
Dominant language
TypeScript
Stars
8.7k
Forks
1.8k
Avg merge
1d 17h
Merged PRs (30d)
523

Description

Part of [SDK-1425](https://linear.app/getsentry/issue/SDK-1425/port-integration-tests-to-span-streaming), one tracking issue per integration test package.

`dev-packages/bun-integration-tests` carries no `traceLifecycle` pin at all, so it already runs on the default streaming lifecycle. `suites/fetch` and `suites/hono-sdk` already assert on streamed spans.

This is a verification issue, not a port. Confirm and close, or split off whatever gap turns up.

## What to check

* No suite pins `traceLifecycle: 'static'` and none uses an env-gated `STREAMED` switch.
* The 5 suites (`basic`, `fetch`, `hono-sdk`, `orchestrion-mysql`, `orchestrion-postgres`) assert on span v2 payloads, not on transaction envelopes. The remaining `transaction:` reference in `suites/hono-sdk/test.ts` is the error event's `transaction` field, which stays.
* The orchestrion suites assert low cardinality `db` span names.

Contributor guide

Open the contributing guide

Research direction

Inspect the five suites under dev-packages/bun-integration-tests: basic, fetch, hono-sdk, orchestrion-mysql, and orchestrion-postgres. Search them for traceLifecycle, STREAMED, transaction envelope assertions, and orchestrion span names, then run the Bun integration tests. Done means the listed checks are confirmed or any discovered gap is split into a follow-up issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
bun, typescript
Domain
testing-qa
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
82/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.