-
Notifications
You must be signed in to change notification settings - Fork 2
feat: configure jest for esm dependency compatibility #430
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
- Add babel.config.js for ESM→CJS transformation targeting current Node.js - Update jest.config.js with transformIgnorePatterns for chalk/execa/commander - Add @babel/core, @babel/preset-env, babel-jest as dev dependencies - Tests still passing (18 suites, 643 tests) Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
✅ Coverage Check PassedOverall Coverage
Coverage comparison generated by |
Smoke Test Results ✅Last 2 Merged PRs:
Test Results:
Overall Status: PASS cc: @Copilot @Mossaka
|
|
Smoke Test Results - Claude Last 2 merged PRs:
✅ GitHub MCP: Retrieved PR data Overall Status: PASS
|
Jest cannot parse ESM-only packages, blocking dependency updates (chalk 5.x, execa 9.x, commander 14.x). This adds Babel transformation to handle ESM→CJS conversion in tests.
Changes
@babel/preset-envtargeting current Node.jsbabel-jesttransformer for JS files andtransformIgnorePatternsto transformchalk|execa|commanderin node_modules@babel/core,@babel/preset-env,babel-jestdev dependenciesScope
Fixes Jest's ability to parse ESM modules. Does not address API breaking changes in execa/commander upgrades - those require separate source modifications.
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
8.8.4.4REDACTED, pid is -1(packet block)8.8.8.8REDACTED, pid is -1(packet block)If you need me to access, download, or install something from one of these locations, you can either:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.