Skip to content

[Review] fix: Fix config to make esm build esm compatible (box/box-codegen#915)#11

Open
govambam wants to merge 1 commit intobase-for-pr-1315from
review-pr-1315
Open

[Review] fix: Fix config to make esm build esm compatible (box/box-codegen#915)#11
govambam wants to merge 1 commit intobase-for-pr-1315from
review-pr-1315

Conversation

@govambam
Copy link
Owner

Recreated from box#1315 for Macroscope review.

Original PR: box#1315
Status: closed (merged)

Recreated using squash merge commit - exact merged state preserved.

Original PR: box#1315

@macroscopeapp
Copy link

macroscopeapp bot commented Feb 18, 2026

Make the ESM build esm compatible by writing lib-esm/package.json with type: "module" and browser mappings for ./box/eventStream.js and ./internal/utilsNode.js

Add a postbuild script to generate lib-esm/package.json, update package.json browser mappings, switch ESM tsconfig to ES2022 with bundler resolution, and change tests to namespace imports.

📍Where to Start

Start with the build script in update-esm.mjs.


Macroscope summarized fab73b1.

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