🦞 igor-claw: astro.md — adapter must go on adapter:, not integrations - #940
Open
ayal wants to merge 1 commit into
Open
🦞 igor-claw: astro.md — adapter must go on adapter:, not integrations#940ayal wants to merge 1 commit into
ayal wants to merge 1 commit into
Conversation
… integrations Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
skills/wix-headless/references/astro.mddocuments the always-onastro.config.mjsrequirements (Caveat A2:wixPages()+checkOrigin: false) but never mentions the production adapter's config placement.@wix/astro-wix-hosting-adapter, or the legacy@wix/cloud-provider-fetch-adapter) must be set via the top-leveladapter:key, never added to theintegrationsarray — even though it's imported/invoked exactly likewix()/wixPages()/react(). Putting it inintegrationslets its setup hooks run and log success while Astro's build-time guard (which only checks the staticadapter:field) still fails with a misleadingNoAdapterInstallederror.create-headless-link.mdto point at the new caveat.Context
Surfaced by a headless-feedback report: an agent hand-scaffolded and hand-wired
astro.config.mjsfor a pre-provisioned site (none ofheadless init/link/createsupport targeting an existing site+clientId with no local project), placed the adapter inintegrationsby pattern-matching the other entries, and lost real debugging time to the misleading build error before finding the root cause in Astro's own source. Companion docs fix (the CLI's own generated-doc description): wix-private/developer-docs#8060. Slack thread: https://wix.slack.com/archives/C0BDXM5LLE7/p1786316382185579Opened automatically by an AI agent on behalf of Ayal (ayalg@wix.com) — not authored by a human.
Test plan
headless linkbehavior inwix-private/wix-cli/packages/cli-project-extender/src/modify-astro-config.ts(config.adapter = wixHostingAdapter()).settings.config.adaptervs runtimesettings.adapter).