You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Installing both this vscode-java in a codespace breaks the codeql extension.
Environment
Operating System: codespaces
JDK version:
Visual Studio Code version:
Java extension version:
Steps To Reproduce
Install both extensions
Open the codeql pane (load codeql extension)
Wait for codeql to startup (can take some time due to needing to download codeql itself).
In codeql extension choose download a database from lgtm; Pick g/apache/maven, then java.
Open the codeql pane (load codeql extension again)
Open a java file (load java extension)
In the workspace root look in the newly added top level root starting apache_maven_...
An error about accessing pipes on undefined happens.
Additional Informations
This is a very complex 3 way interaction between the 2 extensions and codespaces. The underlying bug is isaacs/node-graceful-fs#170 which is fixed with newer versions of graceful-fs. This is still used by vscode-java so things break I can't quite work out how an old version ends up in vscode-java as the only runtime dependency is optional but specifying --no-optional doesn't fix it (I am not a js/npm expert).
The full issue happens due to the following steps:
Basically codespaces patches fs (using graceful-fs) so that some values become computed properties.
vscode-codeql "subclasses" classes in fs due to (using graceful-fs) internally.
The broken version of graceful-fs in vscode-java is loaded and accidentally mutates fs (due to the use of computed properties in the patched version of fs).
The "classes" in vscode-codeql don't have the new prototype in the chain so the new constructors from the broken version of fs create new objects rather than modifying this.
Note that the issues with fs having properties is also the case with node 15+ but vscode is on 14 so this will break in future whenever vscode updates nodejs that far.
To fix it you need to ensure the version of graceful-fs is greater than 4.2.2 where the bug is fixed.
The text was updated successfully, but these errors were encountered:
Installing both this vscode-java in a codespace breaks the codeql extension.
Environment
Steps To Reproduce
codeql
itself).g/apache/maven
, thenjava
.apache_maven_
...pipes
onundefined
happens.Additional Informations
This is a very complex 3 way interaction between the 2 extensions and codespaces. The underlying bug is isaacs/node-graceful-fs#170 which is fixed with newer versions of graceful-fs. This is still used by
vscode-java
so things break I can't quite work out how an old version ends up in vscode-java as the only runtime dependency is optional but specifying--no-optional
doesn't fix it (I am not a js/npm expert).The full issue happens due to the following steps:
fs
(usinggraceful-fs
) so that some values become computed properties.vscode-codeql
"subclasses" classes infs
due to (usinggraceful-fs
) internally.vscode-java
is loaded and accidentally mutatesfs
(due to the use of computed properties in the patched version offs
).vscode-codeql
don't have the new prototype in the chain so the new constructors from the broken version offs
create new objects rather than modifyingthis
.Note that the issues with
fs
having properties is also the case with node 15+ but vscode is on 14 so this will break in future whenever vscode updates nodejs that far.To fix it you need to ensure the version of
graceful-fs
is greater than 4.2.2 where the bug is fixed.The text was updated successfully, but these errors were encountered: