nodejs / nodejs/node

Referencing external dependency via `package.json` works for ESM, but not CJS

Offen
#49,257 7 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

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

v20.5.1

Platform

Linux vali 6.4.6-76060406-generic #202307241739169092810522.04~d567a38 SMP PREEMPT_DYNAMIC Tue A x86_64 x86_64 x86_64 GNU/Linux

Subsystem

No response

What steps will reproduce the bug?

Clone https://gist.github.com/remcohaszing/25c719651d340d9170dd75428dbcef8e

Given the following files:

// package.json
{
  "imports": {
    "#path": {
      "default": "path"
    }
  }
}
// index.cjs
require('#path')
// index.mjs
await import('#path')

Now run both scripts.

How often does it reproduce? Is there a required condition?

It reproduces consistently.

What is the expected behavior? Why is that the expected behavior?

Both scripts run and output nothing.

What do you see instead?

Running node index.mjs works fine, but node index.cjs throws:

node:internal/modules/cjs/loader:1018
        throw e;
        ^

TypeError [ERR_INVALID_URL_SCHEME]: The URL must be of scheme file
    at new NodeError (node:internal/errors:405:5)
    at fileURLToPath (node:internal/url:1397:11)
    at finalizeEsmResolution (node:internal/modules/cjs/loader:1059:20)
    at Module._resolveFilename (node:internal/modules/cjs/loader:1011:16)
    at Module._load (node:internal/modules/cjs/loader:901:27)
    at Module.require (node:internal/modules/cjs/loader:1115:19)
    at require (node:internal/modules/helpers:130:18)
    at Object.<anonymous> (/home/remco/Downloads/foo/index.cjs:1:1)
    at Module._compile (node:internal/modules/cjs/loader:1233:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1287:10) {
  code: 'ERR_INVALID_URL_SCHEME'
}

Node.js v20.5.1

Referencing a relative path works fine though.

Additional information

There was some discussion in this Twitter thread.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Reproduziere das Problem mit der package.json, index.cjs und index.mjs des Gist und verfolge anschließend die CommonJS-Auflösung durch node:internal/modules/cjs/loader, insbesondere finalizeEsmResolution. Erledigt ist die Aufgabe, wenn das imports-Mapping in der package.json zum integrierten path von beiden Skripten aus funktioniert, ohne ERR_INVALID_URL_SCHEME.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, node.js
Bereich
backend
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
52/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.