getsentry / getsentry/sentry-javascript

Port cloudflare-integration-tests to span streaming

オープン
#24,132 コメント 1 件 リアクション 0 件 担当者 1 名 @JPeer264 が担当を希望しています GitHub で見る
javascript Task
主要言語
TypeScript
スター
8.7k
フォーク
1.8k
平均マージ
1日 17時間
マージ済み PR(30日)
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 を短くまとめたダイジェスト。