feat(lsp): add Kotlin Language Server support (#7)

* feat(lsp): add Kotlin Language Server support (#6)

Add support for JetBrains Kotlin LSP with auto-download capabilities:

- Platform detection for win-x64, linux-x64, linux-arm64, osx-x64, osx-arm64
- Auto-download JetBrains Kotlin LSP (v0.253.10629)
- Auto-download bundled JRE 21 from vscode-java releases
- Root detection via Gradle (build.gradle.kts, build.gradle) and Maven (pom.xml)
- Support for .kt and .kts file extensions
- Exported KotlinServer from index.ts
- Added comprehensive tests for KotlinServer

Closes #6

* docs: update README and add lsp package CLAUDE.md

- Add Kotlin LSP to README supported languages table
- Create CLAUDE.md for packages/lsp with architecture overview

* fix(lsp): improve error handling and documentation accuracy

Error handling improvements:
- Replace empty .catch(() => {}) with proper error logging
- Add specific ENOENT checks instead of broad catch blocks
- Add try-catch around spawn() with error/exit event listeners
- Add user guidance message for dependency setup failures
- Add error context to verification failure messages

Documentation fixes:
- Fix misleading extractZip comment (system unzip, not Bun native)
- Update CLAUDE.md with accurate extension lists for all servers
- Add missing root detection files (go.sum, pyrightconfig.json, etc.)
- Update auto-download pattern example to match implementation

Test improvements:
- Add .kts extension test for getServersForExtension

* style: fix whitespace in CLAUDE.md

Author: amondnet

.gitmodules

@@ -7,3 +7,6 @@
 [submodule "ref/opencode"]
 	path = ref/opencode
 	url = https://github.com/sst/opencode.git
+[submodule "ref/multispy"]
+	path = ref/multispy
+	url = https://github.com/microsoft/multilspy.git

CLAUDE.md

@@ -124,6 +124,7 @@ Claude/MCP Client <-> StdioTransport <-> McpServer <-> Providers
 | Python | pyright-langserver | pyproject.toml, setup.py, requirements.txt |
 | Go | gopls | go.mod, go.work |
 | Rust | rust-analyzer | Cargo.toml |
+| Kotlin | JetBrains Kotlin LSP (auto-download) | build.gradle.kts, build.gradle, pom.xml |
 
 ### Built-in Formatters
 

README.md

@@ -101,6 +101,7 @@ Create `dora.json` or `opencode.json` in your project root:
 | Python                | pyright                    | pyproject.toml, requirements.txt |
 | Go                    | gopls                      | go.mod                           |
 | Rust                  | rust-analyzer              | Cargo.toml                       |
+| Kotlin                | JetBrains Kotlin LSP       | build.gradle.kts, pom.xml        |
 
 ### Formatters
 

packages/lsp/CLAUDE.md

@@ -0,0 +1,117 @@
+# @pleaseai/code-lsp
+
+LSP (Language Server Protocol) client implementation for AI coding tools.
+
+## Overview
+
+This package provides a unified interface for interacting with multiple language servers, enabling real-time diagnostics, hover information, and symbol navigation.
+
+## Architecture
+
+```
+src/
+├── index.ts       # Public API, LSPManager class
+├── client.ts      # LSP client implementation (JSON-RPC)
+├── server.ts      # LSP server definitions
+└── language.ts    # Language ID mapping
+```
+
+## Supported Language Servers
+
+| Server | ID | Extensions | Root Detection |
+|--------|-----|------------|----------------|
+| TypeScript | `typescript` | .ts, .tsx, .js, .jsx, .mjs, .cjs, .mts, .cts | package-lock.json, bun.lockb, bun.lock, yarn.lock, pnpm-lock.yaml |
+| Deno | `deno` | .ts, .tsx, .js, .jsx, .mjs | deno.json, deno.jsonc |
+| Oxlint | `oxlint` | .ts, .tsx, .js, .jsx, .mjs, .cjs, .mts, .cts, .vue, .astro, .svelte | .oxlintrc.json, package-lock.json, bun.lockb, bun.lock, pnpm-lock.yaml, yarn.lock, package.json |
+| Pyright | `pyright` | .py, .pyi | pyproject.toml, setup.py, requirements.txt, pyrightconfig.json |
+| Gopls | `gopls` | .go | go.work, go.mod, go.sum |
+| Rust Analyzer | `rust-analyzer` | .rs | Cargo.toml, Cargo.lock |
+| Kotlin | `kotlin` | .kt, .kts | build.gradle.kts, build.gradle, settings.gradle.kts, settings.gradle, pom.xml |
+
+## Adding a New Server
+
+1. Define the server in `server.ts`:
+```typescript
+export const MyServer: LSPServerInfo = {
+  id: 'my-server',
+  extensions: ['.ext'],
+  root: nearestRoot(['config.json']), // or custom root function
+  async spawn(root) {
+    const proc = spawn('my-lsp', ['--stdio'], { cwd: root })
+    return { process: proc }
+  },
+}
+```
+
+2. Add to `LSP_SERVERS` array in `server.ts`
+
+3. Export from `index.ts`
+
+4. Add tests in `__tests__/server.test.ts`
+
+## Auto-Download Pattern (Kotlin Example)
+
+For servers requiring runtime dependencies:
+
+```typescript
+const KOTLIN_RUNTIME_DEPS = {
+  kotlinLsp: { url: '...', version: '...' },
+  java: {
+    'win-x64': { url: '...', javaHomePath: '...', javaPath: '...' },
+    'linux-x64': { url: '...', javaHomePath: '...', javaPath: '...' },
+    // ... other platforms
+  } as Record<PlatformId, { url: string, javaHomePath: string, javaPath: string }>,
+}
+
+async function setupKotlinDependencies(platformId: PlatformId) {
+  const cacheDir = path.join(os.homedir(), '.cache', 'dora', 'kotlin-lsp')
+  // Check if exists, download and extract if not
+  // Verify files exist after download
+  return { javaHomePath, kotlinLspPath }
+}
+```
+
+## Key APIs
+
+### LSPManager
+
+Main entry point for managing LSP clients:
+
+```typescript
+const manager = new LSPManager(projectPath)
+
+// Touch file to initialize LSP
+await manager.touchFile('src/index.ts', true)
+
+// Get diagnostics
+const diags = await manager.diagnostics()
+
+// Get hover info
+const hover = await manager.hover({ file, line, character })
+
+// Search symbols
+const symbols = await manager.workspaceSymbol('query')
+
+// Cleanup
+await manager.shutdown()
+```
+
+### Server Utilities
+
+```typescript
+import { getServerById, getServersForExtension } from '@pleaseai/code-lsp'
+
+const server = getServerById('typescript')
+const servers = getServersForExtension('.ts')
+```
+
+## Testing
+
+```bash
+bun test ./src
+```
+
+Tests cover:
+- Server definitions (ID, extensions, root, spawn functions)
+- LSP client lifecycle
+- Manager operations

packages/lsp/src/__tests__/server.test.ts

@@ -4,6 +4,7 @@ import {
   getServerById,
   getServersForExtension,
   GoplsServer,
+  KotlinServer,
   LSP_SERVERS,
   OxlintServer,
   PyrightServer,
@@ -22,6 +23,7 @@ describe('LSP_SERVERS', () => {
     expect(serverIds).toContain('pyright')
     expect(serverIds).toContain('gopls')
     expect(serverIds).toContain('rust-analyzer')
+    expect(serverIds).toContain('kotlin')
   })
 })
 
@@ -120,6 +122,25 @@ describe('RustAnalyzerServer', () => {
   })
 })
 
+describe('KotlinServer', () => {
+  test('has correct id', () => {
+    expect(KotlinServer.id).toBe('kotlin')
+  })
+
+  test('supports Kotlin extensions', () => {
+    expect(KotlinServer.extensions).toContain('.kt')
+    expect(KotlinServer.extensions).toContain('.kts')
+  })
+
+  test('has root function', () => {
+    expect(typeof KotlinServer.root).toBe('function')
+  })
+
+  test('has spawn function', () => {
+    expect(typeof KotlinServer.spawn).toBe('function')
+  })
+})
+
 describe('getServerById', () => {
   test('returns typescript server', () => {
     const server = getServerById('typescript')
@@ -158,6 +179,22 @@ describe('getServersForExtension', () => {
     expect(serverIds).toContain('gopls')
   })
 
+  test('returns servers for .kt extension', () => {
+    const servers = getServersForExtension('.kt')
+    expect(servers.length).toBeGreaterThan(0)
+
+    const serverIds = servers.map(s => s.id)
+    expect(serverIds).toContain('kotlin')
+  })
+
+  test('returns servers for .kts extension', () => {
+    const servers = getServersForExtension('.kts')
+    expect(servers.length).toBeGreaterThan(0)
+
+    const serverIds = servers.map(s => s.id)
+    expect(serverIds).toContain('kotlin')
+  })
+
   test('returns empty array for unknown extension', () => {
     const servers = getServersForExtension('.unknown')
     expect(servers).toEqual([])

packages/lsp/src/index.ts

@@ -396,6 +396,7 @@ export {
   getServerById,
   getServersForExtension,
   GoplsServer,
+  KotlinServer,
   LSP_SERVERS,
   type LSPServerHandle,
   type LSPServerInfo,