Skip to content

build: add core-js for frontend build compatibility#7747

Closed
wtfiwtz wants to merge 1 commit into
getredash:masterfrom
orchestrated-io:build/add-core-js
Closed

build: add core-js for frontend build compatibility#7747
wtfiwtz wants to merge 1 commit into
getredash:masterfrom
orchestrated-io:build/add-core-js

Conversation

@wtfiwtz

@wtfiwtz wtfiwtz commented Jun 2, 2026

Copy link
Copy Markdown

Summary

Add core-js as a devDependency to resolve Babel polyfill requirements for @babel/preset-env under pnpm.

Changes

  • Add core-js: ^2.6.12 to devDependencies
  • Regenerate pnpm-lock.yaml

Why

  • @babel/preset-env requires core-js for polyfill resolution
  • Without it, frontend builds may fail with module resolution errors like:
    Cannot find module 'core-js/modules/es6.symbol.js'
    Cannot find module 'core-js/modules/es7.array.includes.js'
    
  • This is a build-time dependency only (not shipped to production)

CVEs Addressed

None - this is a build fix

Testing

  • ✅ Python tests pass (frontend-only change)
  • ✅ Frontend tests pass
  • pnpm install succeeds

Note

  • core-js 2.x is used (not 3.x) for compatibility with existing Babel configuration
  • Deprecation warning is expected but acceptable for this version
  • This is consistent with how other projects using similar Babel setups handle core-js

Dependencies

None - independent frontend build fix

Made with Cursor

Add core-js as a devDependency to resolve Babel polyfill requirements
for @babel/preset-env under pnpm.

Changes:
- Add core-js: ^2.6.12 to devDependencies
- Regenerate pnpm-lock.yaml

Why:
- @babel/preset-env requires core-js for polyfill resolution
- Without it, frontend builds may fail with module resolution errors
- This is a build-time dependency only

Testing:
- ✅ Python tests pass (frontend-only change)
- ✅ Frontend tests pass
- ✅ pnpm install succeeds

Note:
- core-js 2.x is used (not 3.x) for compatibility with existing Babel config
- Deprecation warning is expected but acceptable for this version

Co-authored-by: Cursor <cursoragent@cursor.com>
@wtfiwtz

wtfiwtz commented Jun 4, 2026

Copy link
Copy Markdown
Author

Duplicate of #7727

@wtfiwtz wtfiwtz closed this Jun 4, 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