Skip to content

test(e2e): Port the SolidStart 2 E2E app to span streaming - #23918

Merged
RulaKhaled merged 1 commit into
developfrom
rolaabuhasna/js-3494/solidstart-2
Sep 2, 2026
Merged

test(e2e): Port the SolidStart 2 E2E app to span streaming#23918
RulaKhaled merged 1 commit into
developfrom
rolaabuhasna/js-3494/solidstart-2

Conversation

@RulaKhaled

@RulaKhaled RulaKhaled commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

What

Ports solidstart-2 to span streaming.

Why

Span streaming is the default now, so the E2E suite has to exercise it. The http.server segment on pageload is matched on url.path rather than its name, because an unparameterized streamed server span is named after the method alone.

Part of #23810

@linear-code

linear-code Bot commented Sep 2, 2026

Copy link
Copy Markdown

JS-3494

@github-actions

github-actions Bot commented Sep 2, 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.

Path Size % Change Change
@sentry/browser 28.57 kB - -
@sentry/browser - with treeshaking flags 26.94 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 26.83 kB - -
@sentry/browser (incl. Tracing) 48.87 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 48.88 kB - -
@sentry/browser (incl. Tracing, Profiling) 51.8 kB - -
@sentry/browser (incl. Tracing, Replay) 88.36 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 77.76 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 93.05 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 106.02 kB - -
@sentry/browser (incl. Feedback) 46.06 kB - -
@sentry/browser (incl. sendFeedback) 33.64 kB - -
@sentry/browser (incl. FeedbackAsync) 38.74 kB - -
@sentry/browser (incl. Metrics) 29.52 kB - -
@sentry/browser (incl. Logs) 29.81 kB - -
@sentry/browser (incl. Metrics & Logs) 30.45 kB - -
@sentry/react 30.32 kB - -
@sentry/react (incl. Tracing) 51.08 kB - -
@sentry/vue 35.74 kB - -
@sentry/vue (incl. Tracing) 51.14 kB - -
@sentry/svelte 28.6 kB - -
CDN Bundle 30.36 kB - -
CDN Bundle (incl. Tracing) 49.52 kB - -
CDN Bundle (incl. Logs, Metrics) 32.59 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 51.43 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 73.19 kB - -
CDN Bundle (incl. Tracing, Replay) 87.01 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 88.88 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 92.94 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 94.84 kB - -
CDN Bundle - uncompressed 89.97 kB - -
CDN Bundle (incl. Tracing) - uncompressed 147.58 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 96.27 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 153.27 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 225.43 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 267.07 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 272.75 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 280.77 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 286.44 kB - -
@sentry/nextjs (client) 53.7 kB - -
@sentry/sveltekit (client) 49.3 kB - -
@sentry/core/server 40.74 kB - -
@sentry/core/browser 13.42 kB - -
@sentry/node 124.05 kB +0.02% +22 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 81.51 kB - -
@sentry/node - without tracing 88.3 kB +0.03% +22 B 🔺
@sentry/node - without channel injection 103.35 kB +0.02% +17 B 🔺
@sentry/aws-serverless 96.66 kB +0.03% +24 B 🔺
@sentry/cloudflare (withSentry) - minified 201.23 kB - -
@sentry/cloudflare (withSentry) 500.94 kB - -

View base workflow run

Removes the static trace lifecycle pin and rewrites the specs against streamed
span v2. Server action children are collected across envelopes; the http.server
segment is matched on url.path because an unparameterized streamed name is the
method alone.

Co-Authored-By: Cursor Grok 4.6 <cursoragent@cursor.com>
@RulaKhaled
RulaKhaled force-pushed the rolaabuhasna/js-3494/solidstart branch from b809db3 to e0f4da9 Compare September 2, 2026 10:58
@RulaKhaled
RulaKhaled force-pushed the rolaabuhasna/js-3494/solidstart-2 branch from 397dfc4 to 045963d Compare September 2, 2026 10:58
@RulaKhaled
RulaKhaled changed the base branch from rolaabuhasna/js-3494/solidstart to develop September 2, 2026 11:00
@RulaKhaled
RulaKhaled marked this pull request as ready for review September 2, 2026 11:52
@RulaKhaled
RulaKhaled merged commit 13e1316 into develop Sep 2, 2026
43 checks passed
@RulaKhaled
RulaKhaled deleted the rolaabuhasna/js-3494/solidstart-2 branch September 2, 2026 13:20
Sign up for free to 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.

2 participants