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