Conversation
- Add index.mjs as ESM entry point with require() polyfill - Update package.json with module and exports fields - Add comprehensive tests for ESM bundling - Maintain backward compatibility with CommonJS Co-authored-by: naorpeled <6171622+naorpeled@users.noreply.github.com>
Co-authored-by: naorpeled <6171622+naorpeled@users.noreply.github.com>
|
@copilot make sure to also add tests to make sure there's no regression for non ESM usage |
- Enhanced CommonJS require test with full end-to-end functionality - Added test for CommonJS bundling with esbuild - Verified all existing tests still pass - Ensured no regression for CommonJS users Co-authored-by: naorpeled <6171622+naorpeled@users.noreply.github.com>
Added comprehensive regression tests for non-ESM usage:
All 28 test suites pass (486 total tests including 4 new tests in the ESM test suite). Commit: b8f4f3c |
|
@copilot you haven't committed the cjs tests |
The CJS tests are committed in b8f4f3c. The test file
The file was created in commit 15516da with 3 tests, then modified in commit b8f4f3c to add the 4th CJS bundling test. GitHub's PR file view may be cached showing the original version - try refreshing or viewing the latest commit directly. |
Fix ESM compatibility when bundling with esbuild
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.