-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
in CI, works locally in a Dev Container
#22795
Comments
I think it may be related to TypeStrong/ts-node#1714 I have the same error after updating nodejs to |
Can someone please create a ts-node issue to discuss? It's easier for me to keep track of those. The short-circuit thing is potentially misleading, because often loaders should short-circuit, but node's loaders API demands that we signal the intent via a flag in the return value. This error happens when that intention is not signaled. It's complicated. |
No idea on the issue internals or specifics but this just occurred in our system too. No changes (code or else) except Node.js upgrade. |
Using
|
Sounds like Cypress needs to version-bump |
I was able to get the However, it only logs the error and continues with the test. Is that behavior you're seeing or is your test run failing? I also found that having What displays if you run |
Mine comes to a halt and doesn't run any further tests but doesn't exit. I'll try your branch. |
Clean install,
|
@brianjenkins94 or @nevenduranec, Would one of you be able to provide a reproduction repo? @chrisbreiding 's attempt might be a good place to start. |
Here is a minimal reproducible sample using Cypress: ts-node-1839.zip Using VSCode to open the above as a dev container, reproduce with:
|
@brianjenkins94, when running the provided steps i get this result
So i tried the 'start' script but that errors with this:
I think the test project may be missing the |
I'm seeing the I think you're probably not on a new enough Node:
|
@brianjenkins94, I get the same results with the never version of node. The test project seems to want to run a |
I took another look and was able to recreate the error. @brianjenkins94 thanks for bearing with me. This appears to work on node v18.2.0 but does not work on node 18.6.0 |
18.6.0 introduced loader chaining, that error code, and a new requirement which loaders must implement in some situations to avoid it. So if any loaders are being used, and if they're out-of-date, they may trigger that error. |
I'm getting the same error stack with: To note: I've |
@isgj Same issue here. Cypress v10 is broken on current Node LTS. Node |
I will look into this, most minimal repo is import { defineConfig } from "cypress";
export default defineConfig({
e2e: {
supportFile: false,
},
}); {
"type": "module",
"dependencies": {
"cypress": "^10.7.0",
"typescript": "^4.8.2"
}
} Basically any combination of
|
I updated Edit: Node.js 18.6 also seems okay. I'll leave the below debugging info for future reference, we will need to figure out why our tests are doing something different to production for Node.js 18.6.
Hmm this seems to only happen in our internal system tests - could be a red herring. Perhaps the |
@jsolly #23637 should fix this. In the meantime you could install Node.js 16.16, and that should work, too. Node.js 16.17 had a breaking change (surprise minor version breaking change). Edit: see below - worth nothing that my above comment isn't accurate - while this did catch us off guard, es modules in Node.js is still experimental, so there is not commitment to server. |
It was a breaking change in an experimental feature which does not follow semver, right? |
Right, yes, my bad -- I've been using this feature for so long I'd forgotten it's still experimental. Thanks for pointing that out. I've updated my comment to clarify (but left original one there, too, for context). Either way, @cspotcode's great Thanks for making |
I confirm it's a node version issue. I'm using now nvm with a node@16.15.1 an it works well now |
@YoLoADR This is solved, update to https://github.com/cypress-io/cypress/releases/tag/v10.9.0 |
@SchulteMarkus I get this error with |
That's not the error this issue is about. |
Current behavior
Stacktrace:
I noticed this too:
But otherwise didn't find any useful search results when searching for "ERR_LOADER_CHAIN_INCOMPLETE" so I'm going to file this so I can answer it when I figure it out :)
Desired behavior
Don't short circuit on
resolve
.Test code to reproduce
Not sure yet... I'm going to start by trying v10.2.* because it only recently stopped working in CI, so maybe it broke with v10.2.*.
I am using this
.gitlab-ci.yml
:and here are the relevant scripts from my
package.json
, which are a bit gnarly:and my
cypress.config.ts
, even though it's pretty plain:Cypress Version
v10.3.0
Other
No response
The text was updated successfully, but these errors were encountered: