vm.Module.evaluate() morphs loader errors into module evaluation rejections

Offen

#60,242 erstellt am 13.10.2025 6 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 5/5
Geschätzter Aufwand: Über eine Woche
Anfängerfreundlichkeit: 25/100

Issue-Typ: Feature
Klarheit: Muss geklärt werden
Aktivitätsstatus: Aktiv
Tech-Stack: javascript
Bereich: backend

Rechercherichtung

Beginne mit vm.Module.evaluate() und lies die Diskussion zum verlinkten Pull Request. Konzentriere dich darauf, wie Validierungs- und Loader-Fehler derzeit gemeldet werden. Das Issue ist abgeschlossen, wenn sich das Projekt auf eine Möglichkeit geeinigt hat, mit der synchrone Loader Loader-Fehler von Fehlern bei der Modulauswertung unterscheiden können.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

esm stale vm

This came up in https://github.com/nodejs/node/pull/60205#discussion_r2423587581 when I was trying to make vm.Module.evaluate() return the module evaluation promise as-is (so synchronously fulfilled for synthetic modules and source text modules without TLA). Currently the validation errors, ERR_VM_MODULE_STATUS, THROW_ERR_SCRIPT_EXECUTION_TIMEOUT, and THROW_ERR_SCRIPT_EXECUTION_INTERRUPTED would always be rejected, so for a synchronous loader, they would either have to expose these underlying rejections to users when the loading goes wrong, or be forced to become asynchronous as it cannot catch these errors synchronously and paint it over. i.e. they cannot do something like this:

function syncLoad() {
  try {
    mod.evaluate();
  } catch(e) {  // This allows the user to get the loader errors synchronously
    if (e.code === 'ERR_VM_MODULE_STATUS' ||
        e.code === 'ERR_SCRIPT_EXECUTION_INTERRUPTED' ||
        e.code === 'ERR_SCRIPT_EXECUTION_TIMEOUT') {
      // fix it up and try again, or error and explain to user what to do
      // but hide the errors that do not directly come from module code
      // as implementation detail
    }
  }
  if (mod.status === 'errored') {
    throw mod.error;  // It's an evaluation error from the module code
  }
  return mod.namespace;
}

Opening a separate issue to discuss if there's another way to make this possible other than just making it throw these loader errors synchronously.

Vorherrschende Sprache: JavaScript
Sterne: 122k
Forks: 37.4k
Ø Merge: 4 T. 3 Std.
Gemergte PRs (30 T.): 272

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus nodejs/node

Stream.prototype.forEach will block in first promise in queue before read more chunk Offen

doc

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100

nodejs/node#66106 · Erstellt am 18.9.2026
nodejs 26.8.2 fails to build with clang/LLVM Offen

build

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100

nodejs/node#66076 · 1 Kommentar · Erstellt am 17.9.2026
`TextEncoder.encodeInto()` underfills the destination for some non-ASCII text Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100

nodejs/node#65994 · 2 Kommentare · 2 Reaktionen · Erstellt am 11.9.2026
worker: `messageerror` listeners remain during `exit` Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100

nodejs/node#65782 · Erstellt am 04.9.2026
[`util.styleText`] Allow `text` param to be any scalar value (ex. `number`, `bool` etc additionaly to `string`) which will be casted internally to `string` Offen

feature request

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

nodejs/node#63841 · 1 Kommentar · Erstellt am 10.6.2026

Alle Issues in nodejs/node