fix(deps): remove unused vscode-languageclient

[dependabot]

Removes the unused direct runtime dependency vscode-languageclient (6.0.0-next.9 -> removed).

The extension only used private range-conversion helpers from the package. Those calls are replaced with focused local conversions backed by regression coverage, leaving no runtime, build, test, type, configuration, packaging, or dynamic usage of vscode-languageclient.

Validation completed on Node.js 20:

• clean npm ci
• npm run tslint
• npm run compile
• npm run build
• VSIX packaging and content inspection
• npm test (34 passing)

[azure-pipelines]

Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

[Changyong Gong (chagong)]

Approved: the v10 migration removes unsupported private converter imports, preserves range conversion behavior, and passes the full CI matrix.

[Copilot]

🟡 Changes recommended

vscode-languageclient appears to be unused after the refactor (no remaining repo references), so keeping/bumping it adds avoidable dependency and supply-chain surface.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the extension’s dependency set by bumping vscode-languageclient to 10.1.0, while removing reliance on its internal range converters by introducing local VS Code ↔︎ LSP Range conversion helpers and adding a regression test for those helpers.

Changes:

• Bump vscode-languageclient from 6.0.0-next.9 to 10.1.0 (plus transitive dependency updates in the lockfile).
• Replace usage of vscode-languageclient/lib/*Converter internals with local toProtocolRange / toCodeRange helpers.
• Add a unit test validating the range conversion behavior.

File summaries

File	Description
test/JavaInlineValueProvider.test.ts	Adds coverage for VS Code ↔︎ protocol Range conversions.
src/JavaInlineValueProvider.ts	Removes dependency on vscode-languageclient internal converters and uses local conversion helpers.
package.json	Bumps vscode-languageclient version.
package-lock.json	Updates resolved dependency tree for the vscode-languageclient bump.

Review details

• Files reviewed: 3/4 changed files
• Comments generated: 1
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Withdrawing approval because vscode-languageclient is unused after the converter refactor and should be removed rather than upgraded.

[Copilot]

🟢 Approval recommended

The dependency removal is complete (no remaining references found) and the replacement conversions are straightforward and covered by a regression test.

Review details

• Files reviewed: 3/4 changed files
• Comments generated: 0 new
• Review effort level: Lite

[Changyong Gong (chagong)]

The unused direct dependency is removed, the replacement range conversion is covered, and all required checks pass on the current head.