[vitest-pool-workers] Preserve deprecated Miniflare cache option - #14913
[vitest-pool-workers] Preserve deprecated Miniflare cache option#14913edmundhung wants to merge 6 commits into
Conversation
🦋 Changeset detectedLatest commit: b231e27 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
✅ All changesets look good |
|
No other references. The implementation is coherent and correct. Let me summarize my findings: The core logic is correct:
Minor observations (not bugs, not actionable):
None of these rise to the level of a logic bug, security issue, backward-compat violation, or incorrect API behavior. The one behavioral edge — There are no actionable issues. LGTM |
@cloudflare/autoconfig
@cloudflare/config
create-cloudflare
@cloudflare/deploy-helpers
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-auth
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
c91813f to
1f75897
Compare
Preserves the deprecated Miniflare cache worker option in @cloudflare/vitest-pool-workers while Miniflare v5 renames it to cacheAPI.
The pool normalizes worker options before Miniflare schema parsing, for both the test runner and auxiliary workers. cacheAPI takes precedence when both names are provided. The legacy cache field remains typed with a deprecation annotation so existing configurations continue to compile.
A picture of a cute animal (not mandatory, but encouraged)