Skip to content

fix(vscode): lower vscode engine to ^1.105.0 and drop unused velocity bundle#58

Open
codersasha wants to merge 1 commit into
eclipseguru:mainfrom
codersasha:fix/vscode-engine-1.105-drop-velocity
Open

fix(vscode): lower vscode engine to ^1.105.0 and drop unused velocity bundle#58
codersasha wants to merge 1 commit into
eclipseguru:mainfrom
codersasha:fix/vscode-engine-1.105-drop-velocity

Conversation

@codersasha

@codersasha codersasha commented Jun 5, 2026

Copy link
Copy Markdown

Two small fixes to vscode/package.json.

1. Lower the VS Code engine floor to ^1.105.0.
^1.106.0 stops the extension installing on editors still on 1.105 — notably Cursor, which runs VS Code 1.105.1. The floor doesn't need to be that high: the extension is typed against @types/vscode ^1.80.0, so it never touches a 1.106 API.

2. Drop org.apache.velocity.engine-core.jar from javaExtensions.
It's a dead reference. The p2 repo has no such bundle, so scripts/server.mjs never packages it (the VSIX ships 9 of the 10 listed jars), and nothing in the Salesforce bundles imports Velocity. But BundleUtils.loadBundles aborts the whole set if a single jar is missing — so on a clean JDT workspace this one bad entry stops the entire Bazel plugin from loading, and nothing resolves. Removing the line fixes it.

… bundle

- engines.vscode ^1.106.0 -> ^1.105.0 so the extension installs on editors
  built on VS Code 1.105.x (e.g. Cursor 3.6.21 / VS Code 1.105.1). The code is
  typed against @types/vscode ^1.80.0, so no 1.106-only API is used.
- remove ./server/org.apache.velocity.engine-core.jar from javaExtensions: the
  p2 repo ships no such bundle (so it is never packaged in the VSIX) and no
  Salesforce bundle requires Velocity. The dangling entry makes
  BundleUtils.loadBundles abort the whole plugin set on a clean JDT workspace.
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.

1 participant