Skip to content

refactor: retire the LSP system - #2429

Merged
GCWing merged 2 commits into
1.0.0-explorefrom
gcwing/retire-lsp-system
Aug 22, 2026
Merged

refactor: retire the LSP system#2429
GCWing merged 2 commits into
1.0.0-explorefrom
gcwing/retire-lsp-system

Conversation

@GCWing

@GCWing GCWing commented Aug 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the built-in LSP contracts, services, runtime wiring, Tauri commands, settings UI, editor integration, tests, and locale resources
  • keep Monaco-native editor capabilities available and gate editor actions on their actual provider support
  • preserve upgrade behavior by returning explicit retirement errors for legacy lsp_* peer-host invokes instead of silently falling back
  • remove retired LSP claims and owners from the semantic capability catalog, regenerate its projections, and stabilize the interaction digest across Windows and Linux by sorting reviewed POSIX paths and excluding generated sources
  • clean ownership docs and boundary rules, retain upstream LSP facts as unsupported external metadata, and restore the DSH external-tool display mapping with coverage

Validation

  • cargo test -p bitfun-cli retired_lsp_commands (2 passed)
  • cargo check -p bitfun-sdk-host-app -p bitfun-server
  • pnpm run check:core-boundaries (rerun after rebase)
  • pnpm run capabilities:check
  • pnpm run capabilities:test (9 passed)
  • pnpm run website:test (6 passed)
  • pnpm run website:build (22 feature guides + 16 settings guides)
  • focused i18n contract tests (2 passed)
  • DSH tool-view mapping test and syntax checks for 9 touched TypeScript files
  • git diff --check origin/1.0.0-explore...HEAD

Desktop test sources compiled, but launching the local test binary was blocked by a rebuildable DLL artifact issue (STATUS_ENTRYPOINT_NOT_FOUND). The regular Web type-check was likewise blocked by an incomplete local node_modules installation; these are local build-product issues rather than source defects.

E2E was intentionally not run, as requested. Remote workspace/control/Peer Device/Detached Dispatch E2E scenarios were not exercised; legacy peer-host retirement behavior is covered by the focused CLI tests above.

Submission hygiene

Untracked design/QA documents, .pnpm-store/, and NUL* intermediates were excluded. The only added file is packages/dsh-acp/tests/tool-view.spec.ts.

@GCWing
GCWing merged commit be0e144 into 1.0.0-explore Aug 22, 2026
7 of 8 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.

1 participant