Skip to content

fix(bundlecheck): Store and use actual_externals in cache - #494

Merged
aversini merged 1 commit into
mainfrom
fix(bundlecheck)-Store-and-use-actual_externals-in-cache
Feb 1, 2026
Merged

aversini merged 1 commit into
mainfrom
fix(bundlecheck)-Store-and-use-actual_externals-in-cache

Conversation

@aversini

@aversini aversini commented Feb 1, 2026

Copy link
Copy Markdown
Collaborator

Add an actual_externals column to the bundle_cache schema (with migration) and persist computed externals when caching results. getCachedResult now prefers actual_externals (falling back to legacy externals) and setCachedResult stores sorted actual_externals. A test was added to ensure cached results return the computed externals (e.g. auto-detected react/react-dom) when the cache key externals differ from the result externals. Backwards compatibility for older cache rows is preserved.

Add an actual_externals column to the bundle_cache schema (with migration) and persist computed externals when caching results. getCachedResult now prefers actual_externals (falling back to legacy externals) and setCachedResult stores sorted actual_externals. A test was added to ensure cached results return the computed externals (e.g. auto-detected react/react-dom) when the cache key externals differ from the result externals. Backwards compatibility for older cache rows is preserved.
@aversini
aversini merged commit 98bd42b into main Feb 1, 2026
6 checks passed
@aversini
aversini deleted the fix(bundlecheck)-Store-and-use-actual_externals-in-cache branch February 1, 2026 16:00
@aversini aversini mentioned this pull request Feb 1, 2026
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.

1 participant