cli-lsp-client 1.4.0 → 1.6.0

package/README.md

@@ -13,23 +13,25 @@ CLI tool for getting LSP diagnostics. Uses a background daemon to keep LSP serve
 
 ## Supported Languages
 
-| Language | LSP Server | Auto-installed | Notes |
-|----------|------------|----------------|-------|
-| TypeScript/JavaScript | `typescript-language-server` | ✓ (via bunx) | `.ts`, `.tsx`, `.js`, `.jsx`, `.mjs`, `.cjs`, `.mts`, `.cts` |
-| Python | `pyright-langserver` | ✓ (via bunx) | `.py`, `.pyi` |
-| JSON | `vscode-json-language-server` | ✓ (via vscode-langservers-extracted) | `.json`, `.jsonc` - includes schema validation |
-| CSS | `vscode-css-language-server` | ✓ (via vscode-langservers-extracted) | `.css`, `.scss`, `.sass`, `.less` |
-| YAML | `yaml-language-server` | ✓ (via bunx) | `.yaml`, `.yml` - includes schema validation |
-| Bash/Shell | `bash-language-server` | ✓ (via bunx) | `.sh`, `.bash`, `.zsh` - **requires shellcheck** (`brew install shellcheck`) |
-| Go | `gopls` | ✗ | Requires manual install: `go install golang.org/x/tools/gopls@latest` |
-| Java | `jdtls` (Eclipse JDT) | ✗ | `.java` - see [Java Installation](#java-installation-guide) below |
-| Lua | `lua-language-server` | ✗ | `.lua` - requires manual install via package manager (brew, scoop) or from [releases](https://github.com/LuaLS/lua-language-server/releases) |
-
+| Language              | LSP Server                     | Auto-installed                       | Notes                                                                                                                                        |
+| --------------------- | ------------------------------ | ------------------------------------ | -------------------------------------------------------------------------------------------------------------------------------------------- |
+| TypeScript/JavaScript | `typescript-language-server`   | ✓ (via bunx)                         | `.ts`, `.tsx`, `.js`, `.jsx`, `.mjs`, `.cjs`, `.mts`, `.cts`                                                                                 |
+| Python                | `pyright-langserver`           | ✓ (via bunx)                         | `.py`, `.pyi`                                                                                                                                |
+| JSON                  | `vscode-json-language-server`  | ✓ (via vscode-langservers-extracted) | `.json`, `.jsonc` - includes schema validation                                                                                               |
+| CSS                   | `vscode-css-language-server`   | ✓ (via vscode-langservers-extracted) | `.css`, `.scss`, `.sass`, `.less`                                                                                                            |
+| YAML                  | `yaml-language-server`         | ✓ (via bunx)                         | `.yaml`, `.yml` - includes schema validation                                                                                                 |
+| Bash/Shell            | `bash-language-server`         | ✓ (via bunx)                         | `.sh`, `.bash`, `.zsh` - **requires shellcheck** (`brew install shellcheck`)                                                                 |
+| GraphQL               | `graphql-language-service-cli` | ✓ (via bunx)                         | `.graphql`, `.gql`                                                                                                                           |
+| **R**                 | **R languageserver**           | **✗**                                | **`.r`, `.R`, `.rmd`, `.Rmd` - see [R Installation](#r-installation-guide) below**                                                           |
+| **C#**                | **OmniSharp-Roslyn**           | **✗**                                | **`.cs` - see [C# Installation](#c-installation-guide) below**                                                                               |
+| Go                    | `gopls`                        | ✗                                    | Requires manual install: `go install golang.org/x/tools/gopls@latest`                                                                        |
+| Java                  | `jdtls` (Eclipse JDT)          | ✗                                    | `.java` - see [Java Installation](#java-installation-guide) below                                                                            |
+| Lua                   | `lua-language-server`          | ✗                                    | `.lua` - requires manual install via package manager (brew, scoop) or from [releases](https://github.com/LuaLS/lua-language-server/releases) |
 
 ## How It Works
 
 - Daemon starts automatically when needed
-- LSP servers spawn based on file type  
+- LSP servers spawn based on file type
 - Finds project roots using config files (tsconfig.json, etc.)
 - Servers stay running for subsequent requests
 
@@ -37,7 +39,7 @@ CLI tool for getting LSP diagnostics. Uses a background daemon to keep LSP serve
 
 ### Real-time Diagnostics Hook
 
-Get instant diagnostic feedback for TypeScript, Python, Go, JSON, CSS, YAML, Bash, Java, and Lua files as you edit in Claude Code.
+Get instant diagnostic feedback for TypeScript, Python, JSON, CSS, YAML, Bash, GraphQL, R, C#, Go, Java, and Lua files as you edit in Claude Code.
 
 #### Setup
 
@@ -53,7 +55,7 @@ Configure Claude Code to use the built-in hook command:
         "hooks": [
           {
             "type": "command",
-            "command": "npx -y cli-lsp-client warmup"
+            "command": "npx -y cli-lsp-client start"
           }
         ]
       }
@@ -75,9 +77,9 @@ Configure Claude Code to use the built-in hook command:
 
 #### How It Works
 
-- **SessionStart**: Automatically warms up LSP servers when Claude Code starts for faster initial diagnostics
+- **SessionStart**: Automatically starts LSP servers when Claude Code starts for faster initial diagnostics
 - **PostToolUse**: Runs diagnostics after each file edit (Edit, MultiEdit, Write tools)
-- Built-in file filtering for all supported languages (11 file types)
+- Built-in file filtering for all supported languages (16 file types)
 - Shows errors, warnings, and hints inline
 - Graceful error handling - never breaks your editing experience
 - Uses the same fast daemon as the regular diagnostics command
@@ -88,7 +90,7 @@ When you save a file with errors, you'll see immediate feedback:
 
 ```
 Edit operation feedback:
-- [npx -y cli-lsp-client claude-code-hook]: 
+- [npx -y cli-lsp-client claude-code-hook]:
 ERROR at line 3, column 9:
   Type 'number' is not assignable to type 'string'.
   Source: typescript
@@ -106,6 +108,8 @@ npx cli-lsp-client diagnostics src/example.ts
 # Check any supported file type
 npx cli-lsp-client diagnostics app.py
 npx cli-lsp-client diagnostics main.go
+npx cli-lsp-client diagnostics analysis.R
+npx cli-lsp-client diagnostics Program.cs
 ```
 
 Exit codes: 0 for no issues, 2 for issues found.
@@ -126,20 +130,23 @@ npx cli-lsp-client hover src/main.ts myFunction
 
 # Get hover info for a variable or type
 npx cli-lsp-client hover app.py MyClass
+npx cli-lsp-client hover analysis.R mean
+npx cli-lsp-client hover Program.cs Console
 ```
 
-```bash
+````bash
 $ npx cli-lsp-client hover src/client.ts runCommand
 Location: src/client.ts:370:17
 ```typescript
 export function runCommand(command: string, commandArgs: string[]): Promise<void>
-```
-```
+````
+
+````
 
 ### Daemon Management
 
 ```bash
-# Check daemon status and memory usage
+# Check daemon status with uptime and running language servers
 npx cli-lsp-client status
 
 # List all running daemons across directories
@@ -156,6 +163,22 @@ npx cli-lsp-client --version
 
 # Show help
 npx cli-lsp-client help
+````
+
+The `status` command shows the current daemon's uptime and running language servers:
+
+```bash
+$ npx cli-lsp-client status
+LSP Daemon Status
+================
+PID: 33502
+Uptime: 1m 38s
+
+Language Servers:
+- typescript (.) - running 1m 33s
+- pyright (.) - running 1m 10s
+
+Total: 2 language servers running
 ```
 
 The `list` command shows all running daemon instances with their working directories, PIDs, and status:
@@ -165,7 +188,7 @@ $ npx cli-lsp-client list
 
 Running Daemons:
 ================
-Hash   | PID   | Status    | Working Directory             
+Hash   | PID   | Status    | Working Directory
 ----------------------------------------------------------
 h0gx9u | 12345 | ● Running | /Users/user/project-a
 94yi9w | 12346 | ● Running | /Users/user/project-b
@@ -204,11 +227,13 @@ java -Declipse.application=org.eclipse.jdt.ls.core.id1 \
 ### Alternative Installation Methods
 
 **Homebrew (macOS/Linux)**:
+
 ```bash
 brew install jdtls
 ```
 
 **Arch Linux**:
+
 ```bash
 pacman -S jdtls
 ```
@@ -221,14 +246,141 @@ pacman -S jdtls
 
 For detailed setup instructions, see the [official Eclipse JDT.LS documentation](https://github.com/eclipse-jdtls/eclipse.jdt.ls).
 
+## R Installation Guide
+
+The R language server requires R runtime and the `languageserver` package:
+
+### Installation Steps
+
+1. **Install R**: Download and install R from [CRAN](https://cran.r-project.org/) or use a package manager:
+
+   **macOS (Homebrew)**:
+   ```bash
+   brew install r
+   ```
+
+   **Ubuntu/Debian**:
+   ```bash
+   sudo apt-get update
+   sudo apt-get install r-base
+   ```
+
+   **Windows**: Download installer from [CRAN Windows](https://cran.r-project.org/bin/windows/base/)
+
+2. **Install R languageserver package**: Open R and run:
+   ```r
+   install.packages("languageserver")
+   ```
+
+   Or from command line:
+   ```bash
+   R --slave -e 'install.packages("languageserver", repos="https://cran.rstudio.com/")'
+   ```
+
+### Verification
+
+Test that the language server works:
+```bash
+R --slave -e 'languageserver::run()'
+```
+
+### Project Detection
+
+The R LSP automatically detects R projects based on these files:
+- `DESCRIPTION` (R packages)
+- `NAMESPACE` (R packages) 
+- `.Rproj` (RStudio projects)
+- `renv.lock` (renv dependency management)
+- Any `.r`, `.R`, `.rmd`, `.Rmd` files
+
+For more information, see the [R languageserver documentation](https://github.com/REditorSupport/languageserver).
+
+## C# Installation Guide
+
+The C# language server requires .NET SDK and OmniSharp-Roslyn:
+
+### Installation Steps
+
+1. **Install .NET SDK**: Download .NET 6.0+ from [Microsoft .NET](https://dotnet.microsoft.com/download) or use a package manager:
+
+   **macOS (Homebrew)**:
+   ```bash
+   brew install dotnet
+   ```
+
+   **Ubuntu/Debian**:
+   ```bash
+   # Add Microsoft package repository
+   wget https://packages.microsoft.com/config/ubuntu/$(lsb_release -rs)/packages-microsoft-prod.deb -O packages-microsoft-prod.deb
+   sudo dpkg -i packages-microsoft-prod.deb
+   sudo apt-get update
+   sudo apt-get install -y dotnet-sdk-8.0
+   ```
+
+   **Windows**: Download installer from [.NET Downloads](https://dotnet.microsoft.com/download)
+
+2. **Install OmniSharp-Roslyn**: Download the latest release from [OmniSharp releases](https://github.com/OmniSharp/omnisharp-roslyn/releases):
+
+   **Automatic script** (recommended):
+   ```bash
+   # Download and extract OmniSharp to ~/.omnisharp/
+   mkdir -p ~/.omnisharp
+   curl -L https://github.com/OmniSharp/omnisharp-roslyn/releases/latest/download/omnisharp-osx-arm64-net6.0.tar.gz | tar -xz -C ~/.omnisharp/
+   
+   # Create symlink to make omnisharp available in PATH
+   sudo ln -sf ~/.omnisharp/OmniSharp /usr/local/bin/omnisharp
+   ```
+
+   **Manual installation**:
+   - Download the appropriate release for your platform (Windows: `omnisharp-win-x64-net6.0.zip`, Linux: `omnisharp-linux-x64-net6.0.tar.gz`)
+   - Extract to a directory (e.g., `~/.omnisharp/`)
+   - Add the executable to your PATH or create a symlink
+
+3. **Set environment variables**:
+
+   **Fish shell**:
+   ```bash
+   set -Ux DOTNET_ROOT ~/.dotnet
+   ```
+
+   **Bash/Zsh**:
+   ```bash
+   echo 'export DOTNET_ROOT=~/.dotnet' >> ~/.bashrc  # or ~/.zshrc
+   source ~/.bashrc  # or ~/.zshrc
+   ```
+
+   **Note**: `DOTNET_ROOT` must be set in your shell environment for the C# language server to work. The CLI will only load OmniSharp if this environment variable is defined. Restart your terminal after setting the environment variable to ensure it's available.
+
+### Verification
+
+Test that OmniSharp works:
+```bash
+# Verify DOTNET_ROOT is set
+echo $DOTNET_ROOT
+
+# Test OmniSharp command
+omnisharp --help
+```
+
+### Project Detection
+
+The C# LSP automatically detects C# projects based on these files:
+- `*.sln` (Solution files)
+- `*.csproj` (Project files)
+- `project.json` (Legacy project files)
+- `global.json` (.NET global configuration)
+- Any `.cs` files
+
+For more information, see the [OmniSharp documentation](https://github.com/OmniSharp/omnisharp-roslyn).
+
 ### Additional Commands
 
 ```bash
-# Warm up LSP servers for current directory (faster subsequent requests)
-npx cli-lsp-client warmup
+# Start LSP servers for current directory (faster subsequent requests)
+npx cli-lsp-client start
 
-# Warm up for specific directory
-npx cli-lsp-client warmup /path/to/project
+# Start servers for specific directory
+npx cli-lsp-client start /path/to/project
 
 # View daemon logs
 npx cli-lsp-client logs

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cli-lsp-client",
-  "version": "1.4.0",
+  "version": "1.6.0",
   "description": "CLI tool for fast LSP diagnostics with background daemon and multi-project support",
   "type": "module",
   "main": "src/cli.ts",
@@ -23,14 +23,27 @@
     "build": "bun build src/cli.ts --compile --outfile cli-lsp-client",
     "dev": "bun run src/cli.ts",
     "typecheck": "bun --bun tsc --noEmit",
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix",
+    "format": "prettier --check .",
+    "format:fix": "prettier --write .",
     "test": "bun test",
     "test:watch": "bun test tests/ --watch",
     "prepublish": "bun test && bun run build"
   },
   "devDependencies": {
+    "@eslint/js": "^9.33.0",
     "@types/bun": "latest",
     "@types/node": "latest",
-    "typescript": "latest"
+    "eslint": "^9.33.0",
+    "eslint-config-prettier": "10.1.8",
+    "eslint-import-resolver-typescript": "^4.4.4",
+    "eslint-plugin-import-x": "^4.16.1",
+    "eslint-plugin-unused-imports": "^4.1.4",
+    "globals": "^16.3.0",
+    "prettier": "3.6.2",
+    "typescript": "latest",
+    "typescript-eslint": "^8.39.1"
   },
   "peerDependencies": {
     "typescript": "^5.0.0"
@@ -38,6 +51,7 @@
   "dependencies": {
     "pyright": "^1.1.403",
     "vscode-jsonrpc": "^8.2.1",
-    "vscode-languageserver-types": "^3.17.5"
+    "vscode-languageserver-types": "^3.17.5",
+    "zod": "^4.0.17"
   }
 }

package/src/cli.ts

@@ -1,6 +1,7 @@
 #!/usr/bin/env bun
 
 import path from 'path';
+import { z } from 'zod';
 import { startDaemon } from './daemon.js';
 import { runCommand, sendToExistingDaemon } from './client.js';
 import { formatDiagnosticsPlain } from './lsp/formatter.js';
@@ -9,64 +10,104 @@ import { HELP_MESSAGE } from './constants.js';
 import { ensureDaemonRunning } from './utils.js';
 import packageJson from '../package.json' with { type: 'json' };
 
-export async function handleClaudeCodeHook(filePath: string): Promise<{ hasIssues: boolean; output: string; daemonFailed?: boolean }> {
+// Schema for Claude Code PostToolUse hook payload
+const HookDataSchema = z.object({
+  session_id: z.string().optional(),
+  transcript_path: z.string().optional(),
+  cwd: z.string().optional(),
+  hook_event_name: z.string().optional(),
+  tool_name: z.string().optional(),
+  tool_input: z
+    .object({
+      file_path: z.string().optional(),
+      content: z.string().optional(),
+    })
+    .optional(),
+  tool_response: z.any().optional(),
+});
+
+export async function handleClaudeCodeHook(
+  filePath: string
+): Promise<{ hasIssues: boolean; output: string; daemonFailed?: boolean }> {
   // Check if file exists
-  if (!await Bun.file(filePath).exists()) {
+  if (!(await Bun.file(filePath).exists())) {
     return { hasIssues: false, output: '' };
   }
-  
+
   // Filter supported file types
   const supportedExts = [
-    '.ts', '.tsx', '.js', '.jsx', '.mjs', '.cjs', '.mts', '.cts',
-    '.py', '.pyi',
+    '.ts',
+    '.tsx',
+    '.js',
+    '.jsx',
+    '.mjs',
+    '.cjs',
+    '.mts',
+    '.cts',
+    '.py',
+    '.pyi',
     '.go',
-    '.json', '.jsonc',
-    '.css', '.scss', '.sass', '.less',
-    '.yaml', '.yml',
-    '.sh', '.bash', '.zsh',
+    '.json',
+    '.jsonc',
+    '.css',
+    '.scss',
+    '.sass',
+    '.less',
+    '.yaml',
+    '.yml',
+    '.sh',
+    '.bash',
+    '.zsh',
     '.java',
     '.lua',
-    '.graphql', '.gql'
+    '.graphql',
+    '.gql',
+    '.r',
+    '.R',
+    '.rmd',
+    '.Rmd',
+    '.cs',
   ];
   const ext = path.extname(filePath);
   if (!supportedExts.includes(ext)) {
     return { hasIssues: false, output: '' };
   }
-  
+
   // Get diagnostics (suppress errors to stdout)
   try {
     // Ensure daemon is running
     const daemonStarted = await ensureDaemonRunning();
-    
+
     if (!daemonStarted) {
       // Failed to start daemon - return with flag so caller can handle
-      return { 
-        hasIssues: false, 
-        output: 'Failed to start LSP daemon. Please try running "cli-lsp-client stop" and retry.',
-        daemonFailed: true 
+      return {
+        hasIssues: false,
+        output:
+          'Failed to start LSP daemon. Please try running "cli-lsp-client stop" and retry.',
+        daemonFailed: true,
       };
     }
-    
+
     const result = await sendToExistingDaemon('diagnostics', [filePath]);
-    
+
     // The diagnostics command returns an array of diagnostics
     if (!Array.isArray(result) || result.length === 0) {
       return { hasIssues: false, output: '' };
     }
-    
+
     const diagnostics = result as Diagnostic[];
-    
+
     // Format output for Claude Code hook (plain text, no ANSI codes)
     const formatted = formatDiagnosticsPlain(filePath, diagnostics);
     return { hasIssues: true, output: formatted || '' };
-  } catch (error) {
+  } catch (_error) {
     // Silently fail - don't break Claude Code experience
     return { hasIssues: false, output: '' };
   }
 }
 
 function showHelp(): void {
-  console.log(HELP_MESSAGE);
+  process.stdout.write(HELP_MESSAGE + '\n');
 }
 
 async function run(): Promise<void> {
@@ -88,7 +129,7 @@ async function run(): Promise<void> {
 
   // Handle version command directly (no daemon needed)
   if (command === 'version' || command === '--version' || command === '-v') {
-    console.log(packageJson.version);
+    process.stdout.write(packageJson.version + '\n');
     return;
   }
 
@@ -112,10 +153,14 @@ async function run(): Promise<void> {
       }
 
       // Parse the JSON to get the file path
-      const hookData = JSON.parse(stdinData);
-      // Handle both PostToolUse format (tool_input.file_path) and simple format (file_path)
-      const filePath = hookData.tool_input?.file_path || hookData.file_path || hookData.filePath;
-      
+      const parseResult = HookDataSchema.safeParse(JSON.parse(stdinData));
+      if (!parseResult.success) {
+        process.exit(0); // Invalid JSON format, silently exit
+      }
+      const hookData = parseResult.data;
+      // Extract file_path from PostToolUse tool_input
+      const filePath = hookData.tool_input?.file_path;
+
       if (!filePath) {
         process.exit(0); // No file path, silently exit
       }
@@ -123,15 +168,15 @@ async function run(): Promise<void> {
       const result = await handleClaudeCodeHook(filePath);
       if (result.daemonFailed) {
         // Daemon failed to start - exit with status 1 to show error to user
-        console.error(result.output);
+        process.stderr.write(result.output + '\n');
         process.exit(1);
       }
       if (result.hasIssues) {
-        console.error(result.output);
+        process.stderr.write(result.output + '\n');
         process.exit(2);
       }
       process.exit(0);
-    } catch (error) {
+    } catch (_error) {
       // Silently fail for hook commands to not break Claude Code
       process.exit(0);
     }