Skip to content
This repository has been archived by the owner on Dec 25, 2023. It is now read-only.

Extension appears to be running but doesn't provide any functionality #59

Closed
halfnibble opened this issue Sep 4, 2020 · 17 comments
Closed
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@halfnibble
Copy link

Unfortunately, the extension does not work at all (except for syntax highlighting) in Nova 1.0b20.

To Reproduce

Steps to reproduce the behavior:

  • Update Nova to 1.0b20
  • Edit any TypeScript file and try to access properties on imported, typed objects.
  • Or even try something simpler like, let test: number = "hello";
  • No errors. :(

Expected behavior
I would expect autocomplete to show a list of properties for typed objects.

And I would expect obvious type assignment errors to display an error.

Note: this worked in 1.0b19.

Screenshots
If applicable, add screenshots to help explain your problem.

Environment (please complete the following information):

  • OS: macOS 10.15.6 (19G2021)
  • Nova version: 1.0b20 Build 199815
  • Extension version: 1.6.1
  • TypeScript version: 3.7.5
  • Sidebar information: Running. Version 4.0.2
  • NodeJS information:
    • node version: 10.16.3
    • npm version: 6.11.3
    • installation method: Rush.js > pnpm

Additional context
The extension was working until the most recent Nova update (and the extension has not changed to my knowledge).

Since I am working in a monorepo, I also tried setting the path to the TypeScript server, and I still don't get TypeScript support in Nova.

npm-debug.log

Nothing output.

@halfnibble halfnibble added the bug Something isn't working label Sep 4, 2020
@halfnibble
Copy link
Author

Update: this only happens in one of my two repos. I have no idea why. Is there any way to get more debugging info from the extension? It looks like it's not even running on my one repo despite reporting "activated"

@apexskier
Copy link
Owner

The extension is working for me on Beta 20, however, I do see some stuff in the extension console that indicates some changes to the runtime behavior that could be problematic.

Specifically, I haven't see this before - it looks like Panic might have changed the execution environment for the language server to be readonly.
Screen Shot 2020-09-07 at 08 51 23

If you clone this project locally and run it locally you'll see more information in the extension console (I hide a few things when not in dev mode).

@apexskier
Copy link
Owner

Also - you indicated that your expected typescript version is 3.7.5 and the sidebar says you're getting 4.0.2. I haven't used Rush.js or pnpm, but they could be causing the wrong version of typescript to be used, which could cause the issue.

@halfnibble
Copy link
Author

I don't know what happened, but it started working again. 🤷‍♂️ I guess we can close this issue.

Very, very odd. Do you want me to file a bug with the Nova team about the readonly language server environment?

@joshdick
Copy link

FWIW I seem to be having the same issue, with b20. the sidebar shows that the TypeScript server is running but the extension is otherwise not working.

@apexskier
Copy link
Owner

@halfnibble Forgot to mention - I filed a bug for the readonly environment here.

@apexskier
Copy link
Owner

@joshdick Are you able to clone this and run locally? I'd like to see what the Extension Console says for someone who can reproduce.

@apexskier apexskier changed the title Does not work on Nova 1.0b20 Sometimes does not work on Nova 1.0b20 Sep 10, 2020
@joshdick
Copy link

joshdick commented Sep 11, 2020

@apexskier A few interesting things:

I tried upgrading Nova from b20 to v1.0 and starting completely fresh by removing all of Nova's supporting files with AppCleaner. After doing that:

  • The "run locally" test you mentioned works fine and shows errors as expected inside the locally-cloned extension project.
  • Running the release version of the extension installed via Extension Library now causes Nova to hard-crash when I attempt to open a .ts file in my own existing project. Disabling the extension stops the hard crashing. This happens even when nova-typescript is the only extension enabled.

@apexskier
Copy link
Owner

  • Running the release version extension installed via Extension Library now causes Nova to hard-crash when I attempt to open a .ts file in my own existing project. Disabling the extension stops the hard crashing. This happens even when nova-typescript is the only extension enabled.

Yikes. Can you file a bug with Panic for this? Any hard crashes indicate something not being handled gracefully by Nova, and will obscure any info the extension is providing in the console.

@apexskier apexskier changed the title Sometimes does not work on Nova 1.0b20 Extension appears to be running but doesn't provide any functionality Sep 14, 2020
@apexskier
Copy link
Owner

Unfortunately, the extension does not work at all (except for syntax highlighting)

Also, I forgot to mention - this extension doesn't provide syntax highlighting, that's something Nova provides directly.

@chillax
Copy link

chillax commented Sep 15, 2020

  • Running the release version of the extension installed via Extension Library now causes Nova to hard-crash when I attempt to open a .ts file in my own existing project. Disabling the extension stops the hard crashing. This happens even when nova-typescript is the only extension enabled.

After upgrading to 1.0 having this extension enabled causes Nova to crash every time I try to open/close a tab in a project, even in non-TS projects. I submitted a problem report via the Panic's new issue form.

@ishanray
Copy link

This doesn't work and is very buggy. Also autocomplete of any kind doesn't show up.

@apexskier
Copy link
Owner

@ishanray Can you please provide some more details? Anything in the extension console? Does this reproduce if you run the extension locally?

@apexskier apexskier added the help wanted Extra attention is needed label Sep 18, 2020
@halfnibble
Copy link
Author

Unrelated to my original issue (which thankfully resolved itself 🤞), I saw this note on the typescript-language-server repo.
"looking for maintainers"
typescript-language-server/typescript-language-server#141

Do we know how Theia is able to use the VS Code extension, and is that possible for Nova?

@apexskier
Copy link
Owner

My guess is that Theia is providing a compatible extension api for VSCode, which would be in Panic's hands. My opinion is that the bet on the Language Server Protocol is the right one, but Nova's support is still pretty basic. Since Microsoft is behind Typescript, VSCode, and LSP I'm hoping things converge (microsoft/TypeScript#39459).

I haven't run into any missing functionality yet, but I'd consider taking over maintenance of typescript-language-server if we discover any bugs or missing features.

@apexskier
Copy link
Owner

This issue is getting a little messy and I'm going to be closing it, since it's pre-official release and I've updated my bug report template since.

Sounds like @halfnibble and @joshdick aren't experiencing it anymore, @chillax's issue sounds more like #74, and @ishanray please file a new issue with more details if you're still experiencing this as well on the latest release.

@joshdick
Copy link

I am actually still having crashing issues with Nova 1.1 and v1.7.0 of this extension, but will prod #74 instead.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants