Skip to content

test(react-router): Verify client fetch instrumentation hook fires - #21402

Merged
chargome merged 1 commit into
developfrom
cg/verify-native-client-hooks
Jun 10, 2026
Merged

test(react-router): Verify client fetch instrumentation hook fires#21402
chargome merged 1 commit into
developfrom
cg/verify-native-client-hooks

Conversation

@chargome

Copy link
Copy Markdown
Member

Converts the fetcher test.fixme into a passing test: a fetcher submission produces a function.react_router.fetcher transaction (origin auto.function.react_router.instrumentation_api) that nests the client action span and the http.client span(s) for the underlying .data request.

closes#21390

@chargomechargome self-assigned this Jun 9, 2026
@github-actions

github-actionsBot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

⚠️Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

PathSize% ChangeChange
@sentry/browser27.39 kB--
@sentry/browser - with treeshaking flags25.82 kB--
@sentry/browser (incl. Tracing)45.68 kB--
@sentry/browser (incl. Tracing + Span Streaming)47.92 kB--
@sentry/browser (incl. Tracing, Profiling)50.48 kB--
@sentry/browser (incl. Tracing, Replay)84.9 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags74.51 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)89.6 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)102.28 kB--
@sentry/browser (incl. Feedback)44.55 kB--
@sentry/browser (incl. sendFeedback)32.19 kB--
@sentry/browser (incl. FeedbackAsync)37.3 kB--
@sentry/browser (incl. Metrics)28.46 kB--
@sentry/browser (incl. Logs)28.69 kB--
@sentry/browser (incl. Metrics & Logs)29.39 kB--
@sentry/react29.18 kB--
@sentry/react (incl. Tracing)47.98 kB--
@sentry/vue32.4 kB--
@sentry/vue (incl. Tracing)47.57 kB--
@sentry/svelte27.41 kB--
CDN Bundle29.78 kB--
CDN Bundle (incl. Tracing)48.16 kB--
CDN Bundle (incl. Logs, Metrics)31.32 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)49.48 kB--
CDN Bundle (incl. Replay, Logs, Metrics)70.61 kB--
CDN Bundle (incl. Tracing, Replay)85.51 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)86.75 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)91.34 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)92.6 kB--
CDN Bundle - uncompressed88.46 kB--
CDN Bundle (incl. Tracing) - uncompressed145.67 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed93.17 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed149.65 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed217.99 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed264.54 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed268.5 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed278.24 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed282.19 kB--
@sentry/nextjs (client)50.43 kB--
@sentry/sveltekit (client)46.1 kB--
@sentry/core/server76.01 kB--
@sentry/core/browser63.14 kB--
@sentry/node-core61.72 kB-0.01%-1 B 🔽
@sentry/node130.4 kB-0.01%-1 B 🔽
@sentry/node - without tracing74.1 kB-0.01%-1 B 🔽
@sentry/aws-serverless86.29 kB--
@sentry/cloudflare (withSentry) - minified173.69 kB--
@sentry/cloudflare (withSentry)433.85 kB--

View base workflow run

@chargome
chargome marked this pull request as ready for review June 9, 2026 13:45
Base automatically changed from charlygomez/js-2712-add-redis-backed-db-span-coverage-to-the-react-router-7 to developJune 10, 2026 08:23
@chargome
chargomeforce-pushed the cg/verify-native-client-hooks branch from 17fa01e to bf11c75CompareJune 10, 2026 08:23
@chargome
chargome merged commit 8fe16a2 into developJun 10, 2026
41 checks passed
@chargome
chargome deleted the cg/verify-native-client-hooks branch June 10, 2026 08:42
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Verify native client hooks fire in Framework Mode

2 participants

@chargome@nicohrubec