Skip to content

Cache package dependency collection results to avoid redundant filesystem walks - #14819

Draft
dario-piotrowicz wants to merge 1 commit into
mainfrom
dario/cache-pkg-deps
Draft

Cache package dependency collection results to avoid redundant filesystem walks#14819
dario-piotrowicz wants to merge 1 commit into
mainfrom
dario/cache-pkg-deps

Conversation

@dario-piotrowicz

Copy link
Copy Markdown
Member

collectPackageDependencies now accepts an optional cacheDir parameter. When provided, collected results are persisted to disk and reused on subsequent calls as long as neither package.json nor the project's lockfile have been modified. This speeds up repeated deploys when dependencies haven't changed.


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: performance improvement

A picture of a cute animal (not mandatory, but encouraged)

@changeset-bot

changeset-bot Bot commented Jul 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c172836

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@cloudflare/deploy-helpers Patch
@cloudflare/remote-bindings Patch

Not sure what this means? Click here to learn what changesets are.

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

@github-actions

Copy link
Copy Markdown
Contributor

✅ All changesets look good

@pkg-pr-new

pkg-pr-new Bot commented Jul 23, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

npm i https://pkg.pr.new/@cloudflare/autoconfig@14819

@cloudflare/config

npm i https://pkg.pr.new/@cloudflare/config@14819

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@14819

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@14819

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@14819

miniflare

npm i https://pkg.pr.new/miniflare@14819

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@14819

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@14819

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@14819

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@14819

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@14819

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@14819

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@14819

wrangler

npm i https://pkg.pr.new/wrangler@14819

commit: c172836

ask-bonk[bot]

This comment was marked as resolved.

@ask-bonk

This comment was marked as resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.

2 participants