Skip to content

chore: eliminate duplicate pnpm configurations#16906

Merged
florian-lefebvre merged 1 commit into
withastro:mainfrom
btea:chore/eliminate-duplicate-config
May 29, 2026
Merged

chore: eliminate duplicate pnpm configurations#16906
florian-lefebvre merged 1 commit into
withastro:mainfrom
btea:chore/eliminate-duplicate-config

Conversation

@btea

@btea btea commented May 29, 2026

Copy link
Copy Markdown
Contributor

Changes

https://pnpm.io/settings#allowbuilds

allowBuilds includes the configuration for onlyBuiltDependencies, so the configuration related to onlyBuiltDependencies can be removed.

Testing

Docs

@changeset-bot

changeset-bot Bot commented May 29, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 9e405fc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ocavue ocavue 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.

LGTM

@florian-lefebvre florian-lefebvre left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@florian-lefebvre florian-lefebvre merged commit 7a29334 into withastro:main May 29, 2026
24 checks passed
@btea btea deleted the chore/eliminate-duplicate-config branch May 29, 2026 12:39
ematipico added a commit that referenced this pull request Jun 2, 2026
* chore(deps): resolve peer dependency issues (#16894)

* chore(deps): update eslint to v10 (#16896)

* test(vercel): add per-test timeout and raise suite timeout (#16898)

* chore: eliminate duplicate pnpm configurations (#16906)

* Fix i18n redirectToDefaultLocale not working with prefixDefaultLocale (#16887)

* fix(i18n): preserve ROUTE_TYPE_HEADER for i18n post-processing (#16800)

* chore: move fix somewhere else

* chore: update changeset

---------

Co-authored-by: ematipico <[email protected]>

* feat(fonts): better bold fallbacks (#16908)

* [ci] format

* fix(docs): replace last occurrences of `withastro/adapters` (#16693)

* fix: astro() marks all pipeline features as used (#16899)

* fix: astro() marks all pipeline features as used

AstroHandler is the batteries-included handler that wires up every
pipeline feature internally. Mark all PipelineFeatures bits at the
top of handle() so the one-shot warnMissingFeatures check in BaseApp
never fires a false positive when the first request short-circuits
(e.g. a redirect route skips middleware).

* add changeset

* move allFeatures mask to base-pipeline.ts per review

* docs: fix typos in astro:i18n middleware JSDoc (#16574)

* docs: fix outdated Partytown link (#16865)

* chore(deps): update devalue (#16900)

* Pre-bundle astro/fetch and astro/hono in Cloudflare optimizeDeps (#16914)

## Goal
Eliminate the 'new dependencies optimized: astro/fetch' warning during dev
in the Cloudflare adapter. The advanced routing feature (d69f858, #16366)
introduced astro/fetch and astro/hono exports, and the Cloudflare handler
entry points (01aa164, #16729) import from them — but neither commit
added these to the optimizeDeps.include list. Vite discovers them at runtime
and triggers a late re-optimization.

## Decisions
- Added both astro/fetch and astro/hono: the Cloudflare adapter imports from
  both (hono.ts imports FetchState from astro/fetch, and astro/hono is used
  similarly). Pre-bundling both prevents the same issue for either entry point.

## Changes
- packages/integrations/cloudflare/src/index.ts: added astro/fetch and
  astro/hono to the server environment optimizeDeps.include array, alongside
  the existing astro/app and astro/app/fetch/default-handler entries.

* fix(node): serve prerendered pages with build.format 'file' and 'preserve' by passing extensions option to send (#16922)

* [ci] format

* chore(deps): update `pnpm/action-setup` to v6 (#16904)

* chore(deps): update pnpm to v11.5.0 (#16903)

* Fix 404 route resolution for experimental.advancedRouting with astro/hono handlers (#16911)

Co-authored-by: Emanuele Stoppa <[email protected]>
Co-authored-by: Matthew Phillips <[email protected]>

* dedupe

---------

Co-authored-by: ocavue <[email protected]>
Co-authored-by: btea <[email protected]>
Co-authored-by: Houston (Bot) <[email protected]>
Co-authored-by: Florian Lefebvre <[email protected]>
Co-authored-by: Florian Lefebvre <[email protected]>
Co-authored-by: Armand Philippot <[email protected]>
Co-authored-by: Matthew Phillips <[email protected]>
Co-authored-by: dfedoryshchev <[email protected]>
Co-authored-by: Andreas Deininger <[email protected]>
Co-authored-by: Houston (Bot) <[email protected]>
Co-authored-by: Matthew Phillips <[email protected]>
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.

3 participants