getsentry / getsentry/sentry-javascript

Port cloudflare-integration-tests to span streaming

未关闭
#24,132 1 条评论 0 个 reaction 已指派 1 人 已被 @JPeer264 认领 在 GitHub 查看
javascript Task
主要语言
TypeScript
星标
8.7k
派生
1.8k
平均合并
1 天 17 小时
30 天内合并 PR
523

描述

Port `dev-packages/cloudflare-integration-tests` to span streaming. Span streaming is the default, so every suite that still pins `traceLifecycle: 'static'` covers behaviour we no longer ship.

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

## Scope

* 102 `traceLifecycle: 'static'` pins across 82 files
* 2 suites already have a `-streamed` twin: `tracing/ignoreSpans-streamed` and `public-api/startSpan-streamed`
* `tracing/d1` uses an env-gated `STREAMED` switch
* This package has no shared span helpers. Every streamed test hand-rolls its own `getSpanContainer`, so the helper work comes first.

## How to port a suite

1. Remove the `traceLifecycle: 'static'` pin from every `Sentry.init` in the suite.
2. If a `-streamed` twin exists, fold it back: delete the static original and rename the twin to the base name.
3. Rewrite the assertions from transaction envelopes to span v2 against the shared helper added in the first sub-issue:
* `event.transaction` becomes `span.name`
* `contexts.trace.op` becomes the `sentry.op` attribute plus `span.is_segment`
* `contexts.trace.data['x']` becomes `span.attributes['x']`, which has the shape `{ value, type }`
* `transaction_info.source` becomes the `sentry.segment.name.source` attribute

Watch for span names that become low cardinality under streaming: `http.server`, `http.client`, `db`, `cache` and `gen_ai`.

Reference suite: `suites/public-api/startSpan-streamed`.

## Keep static trace lifecycle coverage

Keep one suite on the static pin as a guard rather than duplicating each one. Name it in the PR.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。