Skip to content

Comments

feat(v6): update cloudflare default image service#13267

Open
Princesseuh wants to merge 6 commits intov6from
feat/cloudflare-image-default
Open

feat(v6): update cloudflare default image service#13267
Princesseuh wants to merge 6 commits intov6from
feat/cloudflare-image-default

Conversation

@Princesseuh
Copy link
Member

Description (required)

withastro/astro#15435 changes the default to use the bindings instead of compile because compile's dev behavior is not intuitive anymore (yet)

@netlify
Copy link

netlify bot commented Feb 13, 2026

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit c7af404
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/699c346257a1e200081562ef
😎 Deploy Preview https://deploy-preview-13267--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@astrobot-houston
Copy link
Contributor

astrobot-houston commented Feb 13, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
en/guides/integrations-guide/cloudflare.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@sarah11918
Copy link
Member

sarah11918 commented Feb 16, 2026

Just noting that this will need an entry in the Cloudflare upgrade guide here too, where the change is clearly noted and with any upgrade guidance! https://v6.docs.astro.build/en/guides/integrations-guide/cloudflare/#upgrading-to-v13-and-astro-6

Normally when a default setting has changed, we say things like "inspect your deployed project for anything funky" in case this affects them. And if possible, we like to give advice on what to do to "revert to the previous behavior" (since presumably that was already working), which in this case seems to be explicitly setting imageService: "compile"

Edited to add: Will note that this won't go in the main v6 upgrade guide since it's just for Cloudflare, but you can see what other "Changed default" entries typically look like there for inspiration! https://v6.docs.astro.build/en/guides/upgrade-to/v6/#changed-defaults

@sarah11918 sarah11918 added this to the v6.0.0 docs milestone Feb 16, 2026
@sarah11918 sarah11918 added improve or update documentation Enhance / update existing documentation (e.g. add example, improve description, update for changes) merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) 6.0 labels Feb 16, 2026
- **`cloudflare-binding`:** Uses the [Cloudflare Images binding](https://developers.cloudflare.com/images/transform-images/bindings/) for image transformation. The binding is automatically provisioned when you deploy.
- **`passthrough`:** Uses the existing [`noop`](/en/guides/images/#configure-no-op-passthrough-service) service.
- **`compile`:** Uses Astro's default service (Sharp), but only on pre-rendered routes at build time. For pages rendered on demand, all `astro:assets` features are disabled.
- **`compile`:** Transforms images at build time for pre-rendered routes. Uses passthrough for on-demand rendered pages.
Copy link
Member

@sarah11918 sarah11918 Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should still say what it uses, like all the other options, since this option is to define which image service is being used.

And, is the previous sentence now incorrect, that all astro:assets features are disabled for pages rendered on demand? That seemed a very clear way of stating it, so if it hasn't changed, I'm wondering why the wording has? Or do you want to emphasize the passthrough runtime behaviour? If so, then I'd write passthrough in code, something like:

Suggested change
- **`compile`:** Transforms images at build time for pre-rendered routes. Uses passthrough for on-demand rendered pages.
- **`compile`:** Uses XXXXXXX to transform images at build time for prerendered routes. The noop `passthrough` option is configured for on-demand rendered pages.

(Also, note that we're standardizing docs around prerendered (no hyphen) as we get around to them, so might as well update that here!)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c7af404

The reason it just says locally instead of explaining exactly what it uses to transform is because its using sharp but through miniflare so its not exactly the same as it was before when we just said it uses sharp

OliverSpeir and others added 4 commits February 23, 2026 00:29
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.0 improve or update documentation Enhance / update existing documentation (e.g. add example, improve description, update for changes) merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants