Skip to content

fix(deps): override starlight-llms-txt astro peer for Astro 7 (unblock Pages deploy)#13

Merged
zircote merged 1 commit into
mainfrom
fix/starlight-llms-txt-astro7-override
Jun 30, 2026
Merged

fix(deps): override starlight-llms-txt astro peer for Astro 7 (unblock Pages deploy)#13
zircote merged 1 commit into
mainfrom
fix/starlight-llms-txt-astro7-override

Conversation

@zircote

@zircote zircote commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

The catalog site runs astro@7.0.4, but starlight-llms-txt@0.10.0 declares peer astro@^6, so npm ci fails ERESOLVE and the Pages deploy build errors (run on #12's merge). Adds overrides.starlight-llms-txt.astro = "$astro" (the same pattern the MIF and research-harness-template sites use) and regenerates the lock. npm ci resolves and npm run build is green (7 pages).

The catalog site is on astro@7.0.4 but starlight-llms-txt@0.10.0 only peers
astro@^6, so npm ci ERESOLVEs and the Pages deploy build fails. Add the same
overrides.starlight-llms-txt.astro = $astro pin the MIF and research-harness-
template sites use, so the resolver accepts the installed Astro 7.
Copilot AI review requested due to automatic review settings June 30, 2026 19:10
@zircote zircote merged commit de43975 into main Jun 30, 2026
24 checks passed
@zircote zircote deleted the fix/starlight-llms-txt-astro7-override branch June 30, 2026 19:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR unblocks GitHub Pages deploys for the catalog site by resolving an npm peer-dependency conflict between Astro 7 and starlight-llms-txt@0.10.0 (which declares a peer range of astro@^6).

Changes:

  • Add an npm overrides entry to force starlight-llms-txt to use the root astro version via "astro": "$astro".
  • Keep the existing esbuild override intact while extending the overrides map.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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