Skip to content

Bump vscode-textmate-languageservice from 0.2.1 to 1.1.0#174

Merged
Gimly merged 17 commits intomasterfrom
unknown repository
Feb 26, 2023
Merged

Bump vscode-textmate-languageservice from 0.2.1 to 1.1.0#174
Gimly merged 17 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jan 27, 2023

This is a full rewrite of the library that kills all tech debt, perf issues, and also adds browser support.

All providers exit within 100ms except for the definition provider which takes 1200ms. Not perfect but we've cut overheads by 1-2 orders of magnitude!
3 running theories for improvements.. mainly putting a stack variable for variable-property text merging in the tokenizer, & coalescing output data further to kill any overhead.

Not sure we are at the finish line here, but this will close a bunch of issues for sure.

These issues had regression test cases built to test them.. definitely killed these bugs 😄

If the changelogs interest you... 1.0.0-rc-1 and 1.0.0-rc-2

M00TSDNS added 2 commits January 27, 2023 23:32
Since `getCoreNodeModule` was removed, the bundler works again :)
@ghost
Copy link
Copy Markdown
Author

ghost commented Jan 28, 2023

Testing: extension is fully tested (vsce package) on my local machine, and via npm test against the library's test suite.

If you would like browser support, we can now enable it. But we'd need to conditionally kill features

M00TSDNS added 3 commits January 29, 2023 00:43
@ghost ghost changed the title Bump vscode-textmate-languageservice from 0.2.1 to 1.0.0-rc-2 Bump vscode-textmate-languageservice from 0.2.1 to 1.0.0 Jan 29, 2023
@ghost ghost changed the title Bump vscode-textmate-languageservice from 0.2.1 to 1.0.0 Bump vscode-textmate-languageservice from 0.2.1 to 1.0.2 Jan 29, 2023
@ghost ghost changed the title Bump vscode-textmate-languageservice from 0.2.1 to 1.0.2 Bump vscode-textmate-languageservice from 0.2.1 to 1.0.0 Jan 29, 2023
@jankap
Copy link
Copy Markdown

jankap commented Feb 2, 2023

This is awesome! Big question: will @Gimly merge this or not in the future? Can I try this out on my PC already?

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 2, 2023

Run it in PWSH for Windows or your bash console on Unix:

$ cd ~/.vscode/extensions/gimly81.matlab-2.*
$ npm i [email protected]

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 2, 2023

Shoot... you will have to clone this out, npx vsce package then install the VSIX file.
https://gitlab.com/SNDST00M/vscode-textmate-languageservice/-/blob/main/CHANGELOG.md#100-rc-1

"Significant changes to the shape of the API exports."

@jiecui
Copy link
Copy Markdown

jiecui commented Feb 15, 2023

Huge thanks!!
However, after upgrading to 1.0.0, mlint does not appear to work anymore.

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 16, 2023

Did you upgrade via npm i, or did you clone my repository and rebuild the extension? There are some changes to the way the LSP is initialised.. hopefully for the last time but yeah

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 19, 2023

Hack to fix vscode-matlab performance for yourself

Put all these commands in the console:

$ git clone https://github.com/Gimly/vscode-matlab.git
$ cd vscode-matlab
$ git submodule update --init
$ npm install npm-run-all --save-dev
$ vsce package
$ code --install-extension matlab-2.3.1.vsix
$ cd ..

(Colourization is fixed in the snippet now)

@jiecui
Copy link
Copy Markdown

jiecui commented Feb 21, 2023

Sorry for the late reply. Thanks a lot for the comments!
All the above commands were executed successfully, but mlint problem persisted. In addition, the editor also lost colorization of some of the code. I switched to Matlab extension older version v2.0.0, which works all right.

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 22, 2023

I am downloading a fresh purchased copy of Matlab to debug today

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 22, 2023

Works for me:

image

@jiecui
Copy link
Copy Markdown

jiecui commented Feb 22, 2023

Thanks! Probably, the issue is at my end. How can I remove Xavier Hahn's Matlab extension completely and try a fresh start?

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 22, 2023

code --install-extension matlab-2.3.1.vsix should have done it 🤔

@jiecui
Copy link
Copy Markdown

jiecui commented Feb 23, 2023

Still doesn't work well for me after a complete re-installation, but I guess the issue should be at my end. Thanks anyway!

image

M00TSDNS added 2 commits February 23, 2023 20:39
@ghost ghost changed the title Bump vscode-textmate-languageservice from 0.2.1 to 1.0.0 Bump vscode-textmate-languageservice from 0.2.1 to 1.1.0 Feb 24, 2023
@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 24, 2023

We now have web support! Better than Anycode 🎁❣

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 25, 2023

@jiecui go to github.dev/jiecui/mpact - please install [EXPERIMENTAL] Matlab extension and try it.

@jiecui
Copy link
Copy Markdown

jiecui commented Feb 26, 2023

Nice!

@Gimly Gimly merged commit e71fd80 into Gimly:master Feb 26, 2023
@ghost ghost deleted the hotfix-lsp-perfs branch February 26, 2023 09:55
@eluum
Copy link
Copy Markdown

eluum commented Mar 3, 2023

I have not been able to get this working for vscode on windows, seems like I have the same issue as @jiecui based on his screenshot. Building and installing the package from the cloned repo doesn't seem to work, using the version in the vscode extension Marketplace works but lags too much to be usable.

@ghost
Copy link
Copy Markdown
Author

ghost commented Mar 3, 2023

Riiight.. you've got to do git submodule update --init to have the grammar.

EDIT: I updated the comment w/ instructions for local fix - #174 (comment).


@Gimly you'll need to do npm i npm-run-all --save-dev and commit the new dependency to fix the build..

@eluum
Copy link
Copy Markdown

eluum commented Mar 4, 2023

Thanks, everything is working now. Great job making this extension usable!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment