Skip to content
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

typescript: languages provided by plugins not synced #75088

Closed
kyliau opened this issue Jun 7, 2019 · 3 comments · Fixed by #75371
Closed

typescript: languages provided by plugins not synced #75088

kyliau opened this issue Jun 7, 2019 · 3 comments · Fixed by #75371
Assignees
Labels
feature-request Request for new features or functionality javascript JavaScript support issues typescript Typescript support issues
Milestone

Comments

@kyliau
Copy link

kyliau commented Jun 7, 2019

tsserver plugins could provide a list of languages so that when a document other than .js[x] and ts[x] is opened, the plugins would be notified.

Angular language service relies on this to parse .html files in Angular templates.

  "contributes": {
    "typescriptServerPlugins": [
      {
        "name": "@angular/language-service",
        "enableForWorkspaceTypeScriptVersions": true,
        "languages": [
          "html"
        ]
      }
    ]
  },

There seems to be a regression whereby languages provided by tsserver plugins are no longer synced (does not have buffer sync support).

This was brought to light by @andrius-pra in angular/vscode-ng-language-service#338 (comment)

For potential fix, please see a5853d6

cc: @mjbvz

@kyliau kyliau changed the title Languages provided by plugins not synced typescript: languages provided by plugins not synced Jun 7, 2019
@mjbvz mjbvz assigned mjbvz and unassigned jrieken Jun 7, 2019
@mjbvz
Copy link
Collaborator

mjbvz commented Jun 7, 2019

Related microsoft/TypeScript#17151

We only prototyped the support for external languages in TypeScript so this is not really an official API. You'd have to check with the TS team to see if this is still something they are interested in properly supporting as there is other work required beyond this

@mjbvz mjbvz added this to the June 2019 milestone Jun 13, 2019
@mjbvz mjbvz added javascript JavaScript support issues typescript Typescript support issues labels Jun 13, 2019
@mjbvz
Copy link
Collaborator

mjbvz commented Jun 13, 2019

I merged a pr that fixes this so you can test it and provide feedback, but you must not ship anything to customers that relies on this API. It is not official and may be changed or be removed at any point in the future

@mjbvz mjbvz added External feature-request Request for new features or functionality labels Jun 13, 2019
@kyliau
Copy link
Author

kyliau commented Jun 13, 2019

@mjbvz Thanks for the update. I'll test it out locally.
I'd like to organize a meeting to finalize the design with the Typescript team, so that we can move on with the implementation.

@vscodebot vscodebot bot locked and limited conversation to collaborators Jul 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality javascript JavaScript support issues typescript Typescript support issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants