Skip to content

Conversation

@kada49
Copy link
Contributor

@kada49 kada49 commented Nov 20, 2025

The Language Server Specification added a serverInfo parameter in version 3.15.0 (01/14/2020).

How I implemented it seems a little hacky to me, but I couldn't find a better way to do it. Improvements are welcome!

Old

- vue_ls (id: 1)
  - Version: ? (no serverInfo.version response)
  - Root directory: ~/Desktop/website
  - Command: { "vue-language-server", "--stdio" }
  - Settings: {}
  - Attached buffers: 1

New

- vue_ls (id: 1)
  - Version: 3.1.4
  - Root directory: ~/Desktop/website
  - Command: { "/home/kada49/Desktop/language-tools/packages/language-server/bin/vue-language-server.js", "--stdio" }
  - Settings: {}
  - Attached buffers: 1

Note: the new command is changed because of me using a different build for the new one

@kada49 kada49 changed the title feat: expose ls version to client feat(language-server): expose ls version to client Nov 20, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 20, 2025

Open in StackBlitz

vue-component-meta

npm i https://pkg.pr.new/vuejs/language-tools/vue-component-meta@5767

vue-component-type-helpers

npm i https://pkg.pr.new/vuejs/language-tools/vue-component-type-helpers@5767

@vue/language-core

npm i https://pkg.pr.new/vuejs/language-tools/@vue/language-core@5767

@vue/language-plugin-pug

npm i https://pkg.pr.new/vuejs/language-tools/@vue/language-plugin-pug@5767

@vue/language-server

npm i https://pkg.pr.new/vuejs/language-tools/@vue/language-server@5767

@vue/language-service

npm i https://pkg.pr.new/vuejs/language-tools/@vue/language-service@5767

vue-tsc

npm i https://pkg.pr.new/vuejs/language-tools/vue-tsc@5767

@vue/typescript-plugin

npm i https://pkg.pr.new/vuejs/language-tools/@vue/typescript-plugin@5767

commit: a105ae2

@KazariEX
Copy link
Member

@johnsoncodehk Should we add a fourth parameter to the initialize method to pass this information?

@KazariEX KazariEX changed the title feat(language-server): expose ls version to client feat(language-server): add serverInfo to initialized result Nov 21, 2025
@KazariEX KazariEX merged commit 0b8a880 into vuejs:master Nov 21, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants