Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ERR_LOADER_CHAIN_INCOMPLETE on Node.js 18.6 #36

Closed
ai opened this issue Jul 15, 2022 · 0 comments · Fixed by #37
Closed

ERR_LOADER_CHAIN_INCOMPLETE on Node.js 18.6 #36

ai opened this issue Jul 15, 2022 · 0 comments · Fixed by #37

Comments

@ai
Copy link
Contributor

ai commented Jul 15, 2022

After updating to Node.js 18.6.0 I start to have an error:

> tsm node_modules/uvu/bin.js . '\.test\.(ts|js)$'

(node:111446) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
(node:111446) DeprecationWarning: Obsolete loader hook(s) supplied and will be ignored: getFormat, transformSource
Error [ERR_LOADER_CHAIN_INCOMPLETE]: "file:///home/ai/Dev/nanostores/node_modules/.pnpm/tsm@2.2.1/node_modules/tsm/loader.mjs 'load'" did not call the next hook in its chain and did not explicitly signal a short circuit. If this is intentional, include `shortCircuit: true` in the hook's return.
    at new NodeError (node:internal/errors:387:5)
    at ESMLoader.load (node:internal/modules/esm/loader:624:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ESMLoader.moduleProvider (node:internal/modules/esm/loader:472:11)
    at async link (node:internal/modules/esm/module_job:67:21)

It looks like a conflict with a new loader API from Node.js 18.6

antongolub added a commit to antongolub-forks/tsm that referenced this issue Jul 16, 2022
antongolub added a commit to antongolub-forks/tsm that referenced this issue Jul 16, 2022
antongolub added a commit to antongolub-forks/tsm that referenced this issue Jul 16, 2022
lukeed added a commit that referenced this issue Jul 19, 2022
* fix: add support for Node.js v18.6.0

closes #36

* Apply suggestions from code review

* refactor: replace `withShortCircuit` wrapper with direct flag injects

* chore: impr comments

* revert style changes

* update tsc step

Co-authored-by: Luke Edwards <luke.edwards05@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant