Skip to content

Update vue-language-tools monorepo to v3 #13

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.

Sign up for GitHub

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

Open
renovate wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update vue-language-tools monorepo to v3 #13

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"@playwright/cli": "0.1.13",
"@readonly-mcp/core": "github:readonly-mcp/core#1ff05ccedbe3a589decdbd67e73f98a5884f3286",
"@vtsls/language-server": "0.3.0",
"@vue/language-server": "2.2.12",
"@vue/language-server": "3.3.3",

@coderabbitai coderabbitai Bot Jun 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Read-only verification: confirm documented v3 incompatibility assumptions still exist.
set -euo pipefail

echo "== Check repo note about Vue LS v3 forwarding requirement =="
rg -n -C2 'stuck on v2|tsserver/request|vtsls|`@vue/language-server` 2.x|`@vue/language-server` 3' pnpmfile.cjs || true

echo
echo "== Check current pinned version in package.json =="
rg -n '"`@vue/language-server`"' package.json

echo
echo "== Search for any explicit forwarding implementation hints in repo =="
rg -n -C2 'tsserver/request|vtsls|forward.*request|language-server.*forward' || true

Repository: gtbuchanan/dotfiles

Length of output: 3648

🏁 Script executed:

#!/bin/bash
set -euo pipefail
rg -n '"`@vue/language-server`"\s*:\s*"[0-9.]+"' -S . || true
rg -n '`@vue/language-server`' pnpm-lock.yaml package.json 2>/dev/null || true
ls -la

Repository: gtbuchanan/dotfiles

Length of output: 1267

Gate or revert the @vue/language-server v3.3.3 bump to avoid the documented Claude Code LSP incompatibility

package.json pins @vue/language-server to 3.3.3, but pnpmfile.cjs explicitly documents that @vue/language-server v3 requires the LSP client to forward tsserver/request to vtsls, which Claude Code doesn’t implement. No forwarding implementation (or 2.x fallback pin) exists elsewhere in this repo.

Revert the bump (keep a v2 pin) or gate it on client support for the required forwarding path.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@package.json` at line 9, package.json currently pins "`@vue/language-server`"
to "3.3.3" which is incompatible with our LSP client because pnpmfile.cjs
documents that v3 requires forwarding tsserver/request to vtsls; either revert
this dependency to a v2.x pin (match the previous 2.x semver) or add a gate that
only installs/uses "`@vue/language-server`" v3 when the LSP client implements the
tsserver/request -> vtsls forwarding; update package.json to the v2 pin or add
logic around installation/resolution (referenced symbols:
"`@vue/language-server`", version "3.3.3", pnpmfile.cjs, tsserver/request, vtsls)
so the repo never installs v3 unless the forwarding support is present.

"tweakcc": "4.0.14",
"vscode-langservers-extracted": "4.10.0"
}
Expand Down