@dreki-gg/pi-lsp 0.1.3 → 0.2.1

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @dreki-gg/pi-lsp
 
+## 0.2.1
+
+### Patch Changes
+
+- [`32797ff`](https://github.com/dreki-gg/pi-extensions/commit/32797ff18d968e22c6c44e95c46e3393d8928cef) Thanks [@jalbarrang](https://github.com/jalbarrang)! - feat(plan-mode): add Windows compatibility — replace Unix shell commands with cross-platform Bun/Node APIs
+
+  Plan-mode no longer shells out to `cat`, `bash`, or `mkdir` via `pi.exec()`. File I/O now uses `Bun.file()` / `Bun.write()` and `node:fs/promises` `mkdir`, making the extension fully cross-platform. Destructive and safe command pattern lists now include Windows equivalents (`del`, `rd`, `copy`, `move`, `powershell`, `dir`, `where`, `tasklist`, etc.).
+
+  Also fixes Windows compatibility in three other packages:
+
+  - **browser-tools**: `spawn` now uses `shell: true` on Windows so `.cmd` wrappers resolve correctly; `shellEscape` uses double-quote style on Windows; install guidance is platform-aware (Homebrew shown only on macOS).
+  - **subagent**: `spawn` uses `shell: true` on Windows when the command is bare `pi`, allowing `pi.cmd` resolution.
+  - **lsp**: `globalConfigPath()` now uses `os.homedir()` on Windows instead of the unreliable `process.env.HOME`.
+
+## 0.2.0
+
+### Minor Changes
+
+- [`d1c6d0b`](https://github.com/dreki-gg/pi-extensions/commit/d1c6d0b7da843700a7381790d9323f78dd26b152) Thanks [@jalbarrang](https://github.com/jalbarrang)! - Add Windows support for the LSP extension: spawn with `shell: true` on win32 for `.cmd` wrappers, use `where` instead of `which` for command lookup, normalize file URIs with `file:///` and forward slashes for drive-letter paths per RFC 8089, and fix `uriToPath` to correctly round-trip Windows URIs.
+
 ## 0.1.3
 
 ### Patch Changes

package/extensions/lsp/client.ts

@@ -29,12 +29,19 @@ import type {
 // ── Helpers ─────────────────────────────────────────────────────────────────
 
 export function pathToUri(filePath: string): string {
-  const abs = filePath.startsWith('/') ? filePath : resolve(filePath);
-  return `file://${abs}`;
+  const abs = resolve(filePath);
+  const normalized = abs.replace(/\\/g, '/');
+  // Windows paths need file:///C:/... (three slashes)
+  if (/^[A-Za-z]:/.test(normalized)) return `file:///${normalized}`;
+  return `file://${normalized}`;
 }
 
 export function uriToPath(uri: string): string {
-  return uri.startsWith('file://') ? uri.slice(7) : uri;
+  if (!uri.startsWith('file://')) return uri;
+  const path = uri.slice(7);
+  // Remove leading slash before Windows drive letter: /C:/... → C:/...
+  if (/^\/[A-Za-z]:/.test(path)) return path.slice(1);
+  return path;
 }
 
 function languageIdForFile(filePath: string): string {

package/extensions/lsp/config.ts

@@ -21,7 +21,8 @@ import type { LspConfigFile, LspServerUserConfig, ResolvedServerConfig } from '.
 // ── Paths ───────────────────────────────────────────────────────────────────
 
 function globalConfigPath(): string {
-  return join(process.env.HOME ?? homedir(), '.pi', 'agent', 'extensions', 'lsp', 'config.json');
+  const home = process.platform === 'win32' ? homedir() : (process.env.HOME ?? homedir());
+  return join(home, '.pi', 'agent', 'extensions', 'lsp', 'config.json');
 }
 
 function projectConfigPath(cwd: string): string {
@@ -76,7 +77,8 @@ async function loadJsonFile<T>(path: string): Promise<T | null> {
 
 function commandAvailableVia(command: string, cwd: string): 'global' | 'npx' | null {
   try {
-    execSync(`which ${command}`, { stdio: 'pipe', timeout: 5_000 });
+    const whichCmd = process.platform === 'win32' ? 'where' : 'which';
+    execSync(`${whichCmd} ${command}`, { stdio: 'pipe', timeout: 5_000 });
     return 'global';
   } catch {
     // not global

package/extensions/lsp/protocol.ts

@@ -43,6 +43,7 @@ export class LspConnection {
       stdio: ['pipe', 'pipe', 'pipe'],
       cwd: this.options?.cwd,
       env: { ...process.env, ...this.options?.env },
+      shell: process.platform === 'win32',
     });
 
     this.process.stdout!.on('data', (chunk: Buffer) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dreki-gg/pi-lsp",
-  "version": "0.1.3",
+  "version": "0.2.1",
   "description": "Language-agnostic LSP code intelligence for pi — diagnostics, hover, definitions, references, symbols, and call hierarchy",
   "keywords": [
     "pi-package"