Skip to content

fix(e2e): Pin astro version in astro-6 test app - #20709

Merged
nicohrubec merged 2 commits into
masterfrom
hotfix/pin-astro-6-version
May 7, 2026
Merged

fix(e2e): Pin astro version in astro-6 test app#20709
nicohrubec merged 2 commits into
masterfrom
hotfix/pin-astro-6-version

Conversation

@nicohrubec

@nicohrubecnicohrubec commented May 7, 2026

Copy link
Copy Markdown
Member

Seems like the latest astro release included a breaking change that breaks our E2E test apps and broke our backsync: #20708

Pinning on master to unblock CI and the backsync.

Astro 6.3.0 removed `getAdapterLogger()` from `BaseApp`, breaking
compatibility with `@astrojs/node@10.0.6` which still calls it.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions

github-actionsBot commented May 7, 2026

Copy link
Copy Markdown
Contributor

⚠️ This PR is opened against master. You probably want to open it against develop.

@github-actions

github-actionsBot commented May 7, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser26.3 kBaddedadded
@sentry/browser - with treeshaking flags24.78 kBaddedadded
@sentry/browser (incl. Tracing)44.17 kBaddedadded
@sentry/browser (incl. Tracing + Span Streaming)46.39 kBaddedadded
@sentry/browser (incl. Tracing, Profiling)49.14 kBaddedadded
@sentry/browser (incl. Tracing, Replay)83.63 kBaddedadded
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags73.08 kBaddedadded
@sentry/browser (incl. Tracing, Replay with Canvas)88.33 kBaddedadded
@sentry/browser (incl. Tracing, Replay, Feedback)100.93 kBaddedadded
@sentry/browser (incl. Feedback)43.44 kBaddedadded
@sentry/browser (incl. sendFeedback)31.11 kBaddedadded
@sentry/browser (incl. FeedbackAsync)36.19 kBaddedadded
@sentry/browser (incl. Metrics)27.6 kBaddedadded
@sentry/browser (incl. Logs)27.73 kBaddedadded
@sentry/browser (incl. Metrics & Logs)28.43 kBaddedadded
@sentry/react28.04 kBaddedadded
@sentry/react (incl. Tracing)46.4 kBaddedadded
@sentry/vue31.18 kBaddedadded
@sentry/vue (incl. Tracing)46.02 kBaddedadded
@sentry/svelte26.32 kBaddedadded
CDN Bundle28.91 kBaddedadded
CDN Bundle (incl. Tracing)46.94 kBaddedadded
CDN Bundle (incl. Logs, Metrics)30.34 kBaddedadded
CDN Bundle (incl. Tracing, Logs, Metrics)48.04 kBaddedadded
CDN Bundle (incl. Replay, Logs, Metrics)69.46 kBaddedadded
CDN Bundle (incl. Tracing, Replay)84.13 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)85.2 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Feedback)89.94 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)91.02 kBaddedadded
CDN Bundle - uncompressed84.88 kBaddedadded
CDN Bundle (incl. Tracing) - uncompressed140.44 kBaddedadded
CDN Bundle (incl. Logs, Metrics) - uncompressed89.08 kBaddedadded
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed143.9 kBaddedadded
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed213.29 kBaddedadded
CDN Bundle (incl. Tracing, Replay) - uncompressed258.54 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed261.99 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed272.24 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed275.68 kBaddedadded
@sentry/nextjs (client)48.9 kBaddedadded
@sentry/sveltekit (client)44.64 kBaddedadded
@sentry/node-core60.48 kBaddedadded
@sentry/node165.51 kBaddedadded
@sentry/node - without tracing73.49 kBaddedadded
@sentry/aws-serverless107.64 kBaddedadded
@sentry/cloudflare (withSentry) - minified169.35 kBaddedadded
@sentry/cloudflare (withSentry)427.5 kBaddedadded

`@astrojs/node@10.0.6` requires `getAbortControllerCleanup` (exported
since astro ~6.2.0) and `getAdapterLogger()` (removed in astro 6.3.0).
Pin to ~6.2.x to satisfy both constraints.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nicohrubec
nicohrubec merged commit e0ffe34 into masterMay 7, 2026
35 checks passed
@nicohrubec
nicohrubec deleted the hotfix/pin-astro-6-version branch May 7, 2026 11:08
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.

2 participants

@nicohrubec@mydea