| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -142,8 +142,8 @@ function loadCJSModule(module, source, url, filename, isMain) { | |||
| 142 | 142 | specifier = `${pathToFileURL(path)}`; | |
| 143 | 143 | } | |
| 144 | 144 | ||
| 145 | - // FIXME(node:59666) Currently, the ESM loader re-invents require() here for imported CJS and this | ||
| 146 | - // requires a separate cache to be populated as well as introducing several quirks. This is not ideal. | ||
| 145 | + // NOTE: This re-invented require() is only used on the loader-hook worker thread. | ||
| 146 | + // On the main thread, the authentic require() is used instead (fixed by #60380). | ||
| 147 | 147 | const request = { specifier, attributes: importAttributes, phase: kEvaluationPhase, __proto__: null }; | |
| 148 | 148 | const job = cascadedLoader.getOrCreateModuleJob(url, request, kRequireInImportedCJS); | |
| 149 | 149 | job.runSync(); | |
| Back | FazBrowse Home | New Git URL |
0 commit comments