Skip to content

Code Spell Checker extension crashes #63

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

Closed
ymmt2005 opened this issue Mar 7, 2019 · 8 comments
Closed

Code Spell Checker extension crashes #63

ymmt2005 opened this issue Mar 7, 2019 · 8 comments
Assignees
Labels
bug Something isn't working

Comments

@ymmt2005
Copy link

ymmt2005 commented Mar 7, 2019

Steps to Reproduce

  1. Download vsix file of the extension.
  2. Unzip vsix and copy extension directory under ~/.code-server/extensions/streetsidesoftware.code-spell-checker-1.6.10
  3. Enable the extension in code-server.
  4. The extension crashes with the following error message:
Error: Cannot find module 'vscode-languageserver'
    at Function.Module._resolveFilename (module.js:536:15)
    at Function.Module._load (module.js:466:25)
    at Module.require (module.js:579:17)
    at Module.r.require.e [as require] (/usr/local/bin/code-server:468:26234)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/home/ymmt/.vscode/extensions/streetsidesoftware.code-spell-checker-1.6.10/server/src/server.js:4:33)
    at Module._compile (module.js:635:30)
    at Object.Module._extensions..js (module.js:646:10)
    at Module.load (module.js:554:32)
    at tryModuleLoad (module.js:497:12)
[Info  - 10:44:40] Connection to server got closed. Server will restart.
@foresthoffman foresthoffman self-assigned this Mar 7, 2019
This was referenced Mar 7, 2019
@nhooyr nhooyr added bug Something isn't working needs-investigation labels Mar 8, 2019
@jondavidschober
Copy link

Same issue with TSLint.

@ymmt2005
Copy link
Author

Same issue with #200 (comment)

@schrej
Copy link

schrej commented Mar 25, 2019

It doesn't work for me either, ESLint outputs the follwing in the Output tab when selcting ESLint on the right:

[Info  - 15:44:10] ESLint server stopped.
[Info  - 15:44:12] ESLint server running in node v8.15.0
[Info  - 15:44:12] ESLint server is running.
 ›   Error: Unexpected arguments: var p = 
 ›   process;p.on('message',function(m){if(m.c==='e'){p.exit(0);}else 
 ›   if(m.c==='rs'){try{var 
 ›   r=require.resolve(m.a);p.send({c:'r',s:true,r:r});}catch(err){p.send({c:'r
 ›   ',s:false});}}});, 
 ›   See more help with --help

dbaeumer.vscode-selint version: 1.8.2
Note: I copied ESLint from my Windows machine to the extensions folder of code-server, might be relevant.

@zigego
Copy link

zigego commented Mar 25, 2019

@schrej Change the version of node, Eslint work with build-in node by default.
Open settings,then set eslint.runtime to your local node. It works to me both Debian and Mac.

@schrej
Copy link

schrej commented Mar 25, 2019

@zigego Thank you!
After setting "eslint.runtime": "/usr/bin/node" ESLint works fine and shows issues in the editor.

@foresthoffman
Copy link
Contributor

@ymmt2005 It appears that the current version of Code Spell Checker is quite outdated and using some invalid dependencies, which means that it is unpackageable. I'm seeing the same errors as the current failing build.

I've created an issue on the extension's repo: streetsidesoftware/vscode-spell-checker#318

@ymmt2005
Copy link
Author

@foresthoffman
Thank you very much for your investigation.

I'm going to close this issue. Code spell checker is a must have for me so I hope
someday it is packaged for code-server.

code-server is simply great. Thank you, and keep going!

@foresthoffman
Copy link
Contributor

streetsidesoftware/vscode-spell-checker#318 was merged this morning, so now the build works properly. I've got a PR up that should allow the spell checker build to be automated properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants