getsentry / getsentry/sentry-javascript
Confirm bun-integration-tests need no span streaming port
- 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
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