Skip to content

fix(lsp): resolve LSP server compilation errors#82

Merged
developerfred merged 3 commits intomainfrom
fix/lsp-inlay-hint-params
Mar 20, 2026
Merged

fix(lsp): resolve LSP server compilation errors#82
developerfred merged 3 commits intomainfrom
fix/lsp-inlay-hint-params

Conversation

@developerfred
Copy link
Copy Markdown
Owner

Summary

Fixes LSP server compilation errors that prevented building.

Issues Fixed

Changes

  • Fixed InlayHint params naming throughout
  • Fixed CodeActionKind variant names
  • Fixed ServerCapabilities struct initialization for semantic tokens
  • Fixed ParseError matching to use valid variants
  • Fixed SymbolKind variants
  • Simplified semantic tokens functions to return None (API changed significantly)

Test Results

  • LSP compiles successfully ✅
  • 14 warnings remaining (unused code, deprecations)

- build.rs: Fix unwrap in save() when cache_path has no parent
- build.rs: Fix unwrap in build_file() when output has no parent
- lib.rs: Fix unwrap in compile() when binary is None
- Update dependencies: clap 4.4->4.5, regex 1.10->1.11, num_cpus 1.15->1.16
- Add lru_time_cache crate to bend-pvm-lsp dependencies
- Update dependencies: clap 4.4->4.5, regex 1.10->1.11, num_cpus 1.15->1.16

Note: LSP server still needs API updates for lsp-types 0.94 compatibility
Issues fixed:
- #76: InlayHintParams naming (was InlayHintsParams)
- #77: CodeActionKind::QUICKFIX (was QUICK_FIX)
- #78: SemanticTokensOptions wrapped in SemanticTokensServerCapabilities
- #79: CodeActionProviderCapability::Options wrapper
- #80: ParseError variants (removed non-existent UnterminatedString/InvalidNumber)
- #81: SymbolKind::CLASS (removed non-existent TYPE_ALIAS)

Semantic tokens:
- Return None for semantic tokens endpoints (API changed significantly)

Parser errors:
- Map to LexicalError and InvalidPattern for string/number errors
@developerfred developerfred merged commit 974ffb3 into main Mar 20, 2026
3 of 4 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