From 5a8a36956f0d6a139d0b69c80f4b828295505b0c Mon Sep 17 00:00:00 2001 From: RonGamzu <37371774+RonGamzu@users.noreply.github.com> Date: Thu, 21 May 2026 00:15:15 +0300 Subject: [PATCH] lib: fix typos in esm loader comments Three typos in comments in the importSyncForRequire path: - "check the whether" -> "check whether" - "kEvaluaing" -> "kEvaluating" (matches the symbol used elsewhere in the file) - "the the problematic" -> "the problematic" No functional change. Signed-off-by: RonGamzu <37371774+RonGamzu@users.noreply.github.com> --- lib/internal/modules/esm/loader.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/internal/modules/esm/loader.js b/lib/internal/modules/esm/loader.js index 34a9393ac18f01..b39b29fff7779b 100644 --- a/lib/internal/modules/esm/loader.js +++ b/lib/internal/modules/esm/loader.js @@ -289,8 +289,8 @@ class ModuleLoader { let job = this.loadCache.get(url, kImplicitTypeAttribute); // This module job is already created: // 1. If it was loaded by `require()` before, at this point the instantiation - // is already completed and we can check the whether it is in a cycle - // (in that case the module status is kEvaluaing), and whether the + // is already completed and we can check whether it is in a cycle + // (in that case the module status is kEvaluating), and whether the // required graph is synchronous. // 2. If it was loaded by `import` before, only allow it if it's already evaluated // to forbid cycles. @@ -298,7 +298,7 @@ class ModuleLoader { // synchronously so that any previously imported synchronous graph is already // evaluated at this point. // TODO(joyeecheung): add something similar to CJS loader's requireStack to help - // debugging the the problematic links in the graph for import. + // debugging the problematic links in the graph for import. debug('importSyncForRequire', parent?.filename, '->', filename, job); if (job !== undefined) { mod[kRequiredModuleSymbol] = job.module;