Skip to content

Fix: core internal import in CJS legacy envs#5509

Merged
OEvgeny merged 2 commits intomainfrom
fix/core-export
Jul 3, 2025
Merged

Fix: core internal import in CJS legacy envs#5509
OEvgeny merged 2 commits intomainfrom
fix/core-export

Conversation

@OEvgeny
Copy link
Copy Markdown
Collaborator

@OEvgeny OEvgeny commented Jul 3, 2025

Fixes #

Changelog Entry

  • Fixed core internal import in legacy CommonJS environments, in 5509, by @OEvgeny

Description

This adds missing files to the core npm package, so the internal module can be properly located in environments lacking exports field support.

Specific Changes

  • Fixed files field to include top level js files

-

  • I have added tests and executed them locally
  • I have updated CHANGELOG.md
  • I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • Browser and platform compatibilities reviewed
  • CSS styles reviewed (minimal rules, no z-index)
  • Documents reviewed (docs, samples, live demo)
  • Internationalization reviewed (strings, unit formatting)
  • package.json and package-lock.json reviewed
  • Security reviewed (no data URIs, check for nonce leak)
  • Tests reviewed (coverage, legitimacy)

@OEvgeny OEvgeny marked this pull request as ready for review July 3, 2025 22:24
@OEvgeny OEvgeny force-pushed the fix/core-export branch from 012ef7e to 2c54d9f Compare July 3, 2025 22:25
@OEvgeny OEvgeny enabled auto-merge (squash) July 3, 2025 22:32
@OEvgeny OEvgeny merged commit da78356 into main Jul 3, 2025
51 of 52 checks passed
@OEvgeny OEvgeny deleted the fix/core-export branch July 3, 2025 23:15
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.

2 participants