Wrong error annotation when commonjs `require`s an ES module
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 122k
- Forks
- 37.3k
- Ø Merge
- 4 T. 2 Std.
- Gemergte PRs (30 T.)
- 283
Beschreibung
Version
^22.4.0mainwith the--no-experimental-require-moduleflag.
Platform
macOS arm64
Subsystem
esm,module
What steps will reproduce the bug?
mkdir undefined && cd undefined
echo '{"type":"module"}' > package.json
echo "import nothing from 'somewhere'" > app.js
echo "require('./app.js')" > test.cjs
node test.cjs
How often does it reproduce? Is there a required condition?
It starts to happen with v22.4.0. v20.x and <=22.3.0 are not affected.
What is the expected behavior? Why is that the expected behavior?
$ node test.cjs
/Users/mzasso/git/test/undefined/test.cjs:1
require('./app.js')
^
Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/mzasso/git/test/undefined/app.js from /Users/mzasso/git/test/undefined/test.cjs not supported.
Instead change the require of app.js in /Users/mzasso/git/test/undefined/test.cjs to a dynamic import() which is available in all CommonJS modules.
at Object.<anonymous> (/Users/mzasso/git/test/undefined/test.cjs:1:1) {
code: 'ERR_REQUIRE_ESM'
}
Node.js v22.3.0
What do you see instead?
$ node test.cjs
/Users/mzasso/git/test/undefined/test.cjs:315
undefined
^
Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/mzasso/git/test/undefined/app.js from /Users/mzasso/git/test/undefined/test.cjs not supported.
Instead change the require of app.js in /Users/mzasso/git/test/undefined/test.cjs to a dynamic import() which is available in all CommonJS modules.
at TracingChannel.traceSync (node:diagnostics_channel:315:14)
at Object.<anonymous> (/Users/mzasso/git/test/undefined/test.cjs:1:1) {
code: 'ERR_REQUIRE_ESM'
}
Node.js v22.4.0
Additional information
As you can see, this seems to be caused by the presence of TracingChannel.traceSync in the stack trace.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Reproduktionsbefehle im Issue mit Node.js v22.4.0 und dem Flag --no-experimental-require-module auszuführen, und verfolge dann den ESM/CommonJS require-Fehler sowie die Stack-Annotation von TracingChannel.traceSync. Als abgeschlossen gilt die Aufgabe, wenn die Fehlerannotation wie in der erwarteten Ausgabe auf die Zeile require('./app.js') verweist und ein Regressionstest die gemeldete Versionsänderung abdeckt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100