@dreki-gg/pi-lsp 0.1.2 → 0.2.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @dreki-gg/pi-lsp
 
+## 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
+
+- [`d133c3d`](https://github.com/dreki-gg/pi-extensions/commit/d133c3da917e7e5def568d27d6cde8ae8a6c00d2) Thanks [@jalbarrang](https://github.com/jalbarrang)! - Mark pi peer dependencies as optional so npm does not auto-install pi internals when installing extension packages.
+
 ## 0.1.2
 
 ### 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

@@ -76,7 +76,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/index.ts

@@ -9,7 +9,7 @@
  * Any LSP server can be added via config.
  */
 
-import type { ExtensionAPI } from '@mariozechner/pi-coding-agent';
+import type { ExtensionAPI } from '@earendil-works/pi-coding-agent';
 
 import { LspClient } from './client';
 import { loadConfig, scaffoldGlobalConfig, serversForExtension, type LoadedConfig } from './config';

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/extensions/lsp/tools.ts

@@ -4,9 +4,9 @@
  * 11 operations routed to the right server by file extension.
  */
 
-import type { ExtensionAPI } from '@mariozechner/pi-coding-agent';
+import type { ExtensionAPI } from '@earendil-works/pi-coding-agent';
 import { Type } from 'typebox';
-import { StringEnum } from '@mariozechner/pi-ai';
+import { StringEnum } from '@earendil-works/pi-ai';
 
 import type { LspClient } from './client';
 import {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dreki-gg/pi-lsp",
-  "version": "0.1.2",
+  "version": "0.2.0",
   "description": "Language-agnostic LSP code intelligence for pi — diagnostics, hover, definitions, references, symbols, and call hierarchy",
   "keywords": [
     "pi-package"
@@ -26,7 +26,7 @@
     ]
   },
   "devDependencies": {
-    "@mariozechner/pi-ai": "^0.69.0",
+    "@earendil-works/pi-ai": "^0.74.0",
     "@types/node": "24",
     "bun-types": "latest",
     "oxfmt": "^0.43.0",
@@ -35,8 +35,19 @@
     "typescript-language-server": "^5.1.3"
   },
   "peerDependencies": {
-    "@mariozechner/pi-ai": "*",
-    "@mariozechner/pi-coding-agent": "*",
+    "@earendil-works/pi-ai": "*",
+    "@earendil-works/pi-coding-agent": "*",
     "typebox": "*"
+  },
+  "peerDependenciesMeta": {
+    "@earendil-works/pi-ai": {
+      "optional": true
+    },
+    "@earendil-works/pi-coding-agent": {
+      "optional": true
+    },
+    "typebox": {
+      "optional": true
+    }
   }
 }

package/test/tools.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, test } from 'bun:test';
 
 import { registerLspTool } from '../extensions/lsp/tools';
-import type { ToolDefinition } from '@mariozechner/pi-coding-agent';
+import type { ToolDefinition } from '@earendil-works/pi-coding-agent';
 
 function captureTool() {
   let tool: ToolDefinition<any, any> | null = null;