Skip to content

fix(playground): avoid duplicate browser commands#169

Merged
hongjr03 merged 1 commit into
masterfrom
fix/playground-multiple-labs
May 27, 2026
Merged

fix(playground): avoid duplicate browser commands#169
hongjr03 merged 1 commit into
masterfrom
fix/playground-multiple-labs

Conversation

@hongjr03
Copy link
Copy Markdown
Member

@hongjr03 hongjr03 commented May 27, 2026

Summary

  • Omit desktop-only execute-command capabilities from the browser LSP initialize result so multiple playground embeds can start on the same page without registering duplicate global VS Code commands.
  • Keep the regular desktop/server capabilities unchanged.
  • Add a Rust test for the browser-specific capability shape.

Validation

  • cargo fmt --check
  • cargo test -p vide --lib
  • npm --prefix playground run build:embed
  • npm --prefix docs run build

Notes

  • npm --prefix playground run build:wasm was not run locally because Emscripten SDK is not configured in this environment; docs-preview CI has the Emscripten setup and rebuilds WASM.

@github-actions
Copy link
Copy Markdown

Docs preview: https://vide.pascal-lab.net/preview/pr-169/

@hongjr03 hongjr03 force-pushed the fix/playground-multiple-labs branch from 4de4922 to 84be5f9 Compare May 27, 2026 09:01
@hongjr03 hongjr03 marked this pull request as ready for review May 27, 2026 09:09
@hongjr03 hongjr03 merged commit 6b009fd into master May 27, 2026
7 checks passed
@hongjr03 hongjr03 deleted the fix/playground-multiple-labs branch May 27, 2026 09:09
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