Skip to content

test(cloudflare): Cover withSentry and ExecutionContextCompat - #24559

Open
JPeer264 wants to merge 1 commit into
developfrom
jp/cloudflare-two-uncovered-spots
Open

JPeer264 wants to merge 1 commit into
developfrom
jp/cloudflare-two-uncovered-spots

Conversation

@JPeer264

Copy link
Copy Markdown
Member

closes #24041

Add unit tests for the WorkerEntrypoint and error-swallowing paths of withSentry, and type tests that keep ExecutionContextCompat assignable from both the v4 and v5 ExecutionContext shapes.

Add unit tests for the WorkerEntrypoint and error-swallowing paths of
withSentry, and type tests that keep ExecutionContextCompat assignable
from both the v4 and v5 ExecutionContext shapes.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@JPeer264 JPeer264 self-assigned this Sep 21, 2026
@JPeer264
JPeer264 requested a review from a team as a code owner September 21, 2026 12:37
@JPeer264
JPeer264 requested review from andreiborza and isaacs and removed request for a team September 21, 2026 12:37
@github-actions

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 29.07 kB - -
@sentry/browser - with treeshaking flags 27.34 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 27.24 kB - -
@sentry/browser (incl. Tracing) 50.55 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 50.56 kB - -
@sentry/browser (incl. Tracing, Profiling) 53.55 kB - -
@sentry/browser (incl. Tracing, Replay) 90.09 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 79.18 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 94.78 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 107.76 kB - -
@sentry/browser (incl. Feedback) 46.6 kB - -
@sentry/browser (incl. sendFeedback) 34.13 kB - -
@sentry/browser (incl. FeedbackAsync) 39.25 kB - -
@sentry/browser (incl. Metrics) 30.08 kB - -
@sentry/browser (incl. Logs) 30.34 kB - -
@sentry/browser (incl. Metrics & Logs) 31 kB - -
@sentry/react 30.82 kB - -
@sentry/react (incl. Tracing) 52.86 kB - -
@sentry/vue 36.33 kB - -
@sentry/vue (incl. Tracing) 52.84 kB - -
@sentry/svelte 29.1 kB - -
CDN Bundle 30.77 kB - -
CDN Bundle (incl. Tracing) 51.06 kB - -
CDN Bundle (incl. Logs, Metrics) 33.02 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 53.05 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 73.71 kB - -
CDN Bundle (incl. Tracing, Replay) 88.61 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 90.54 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 94.66 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 96.67 kB - -
CDN Bundle - uncompressed 91.07 kB - -
CDN Bundle (incl. Tracing) - uncompressed 152.44 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 97.64 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 158.4 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 227.05 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 272.01 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 277.95 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 285.72 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 291.65 kB - -
@sentry/nextjs (client) 55.19 kB - -
@sentry/sveltekit (client) 50.98 kB - -
@sentry/core/server 39.97 kB - -
@sentry/core/browser 13.63 kB - -
@sentry/node 133.33 kB +0.02% +24 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 82.15 kB - -
@sentry/node - without tracing 89.92 kB +0.03% +20 B 🔺
@sentry/node - without channel injection 112.11 kB +0.02% +12 B 🔺
@sentry/aws-serverless 98.19 kB +0.02% +18 B 🔺
@sentry/cloudflare (withSentry) - minified 204.67 kB - -
@sentry/cloudflare (withSentry) 509.34 kB - -

View base workflow run

This branch has not been deployed

No deployments
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.

cloudflare: Two uncovered spots

1 participant