node-integration-tests: port trace semantics suites to span streaming
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 52/100
- Issue type
- Refactor
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- node.js, typescript
- Domain
- testing
Research direction
Start with the porting recipe in the parent issue, then inspect the listed tracing/* and public-api/* suites and their -streamed twins. Port the specified suites to span streaming, fold back the named twins, and keep tracing/sampling-static and public-api/beforeSendSpan-static on the static pin. Done means the listed integration suites cover sampling, propagation, span linking, and public API behavior under span streaming.
Written by the indexing model from the issue text.
Description
Follow the porting recipe in the parent issue.
Sampling, DSC, envelope headers and span linking. These suites assert on envelope headers and trace propagation rather than on instrumentation output, so they are the ones most likely to expose a real product bug rather than a test rewrite.
Suites
- Envelope headers and DSC:
tracing/envelope-header/*(7 dirs),tracing/dsc-txn-name-update,tracing/double-baggage/*(3 dirs),tracing/meta-tags,tracing/meta-tags-twp,tracing/meta-tags-twp-errors - Sampling:
tracing/sampling-static,tracing/sample-rate-propagation/*(4 dirs),tracing/sample-rand-propagation,tracing/no-parent-span-client-report - Trace lifetime:
tracing/continueTrace,tracing/startNewTrace,tracing/traceid-recycling,tracing/traceid-recycling-with-spans - Span shaping:
tracing/linking,tracing/maxSpans,tracing/tracer-start-active-span-error - Public API span surface:
public-api/startSpan/*(7 dirs),public-api/beforeSendSpan-static,public-api/bindScopeToEmitter,public-api/setMeasurement,public-api/scopes/*
Twins to fold back
tracing/sampling-streamed, tracing/ignoreSpans-streamed/*, public-api/beforeSendSpan-streamed, context-streamed/*, and the 6 public-api/startSpan/*-streamed dirs.
Keep static trace lifecycle coverage
This is the group that owns the guard. Keep tracing/sampling-static and public-api/beforeSendSpan-static on the static pin. Everything else moves.
- Dominant language
- TypeScript
- Stars
- 8.7k
- Forks
- 1.9k
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 562
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from getsentry/sentry-javascript
-
Browser Waiting for: Product Owner
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
getsentry/sentry-javascript#24577 · 1 comment ·
-
Task
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
getsentry/sentry-javascript#24558 · 1 comment ·
-
Task
Difficulty 1/5 Under an hour Newbie friendliness 90/100
getsentry/sentry-javascript#24557 · 1 comment ·
-
Task
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
getsentry/sentry-javascript#24556 · 1 comment ·
-
Task
Difficulty 1/5 1-3 hours Newbie friendliness 90/100
getsentry/sentry-javascript#24555 · 1 comment ·
All issues in getsentry/sentry-javascript
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
safetrustcr/dApp-SafeTrust#426 ·
-
area:workflow bug ready-for-agent
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
fil-donadoni/tolaria#4409 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Fission-AI/OpenSpec#1960 ·
-
Add dependabot Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
corsairdev/corsair#1764 ·