@mrclrchtr/supi-code-intelligence 6.0.1 → 6.2.0

package/README.md

@@ -81,7 +81,7 @@ The package adds eight tools that Pi selects as needed:
 | `code_graph` | Follow LSP references and implementations plus AST calls made by a symbol |
 | `code_find` | Search LSP workspace symbols or parsed AST source structure rather than raw text |
 | `code_health` | Check live diagnostics, language servers, and code-intelligence availability |
-| `code_refactor_plan` | Preview a precise rename or extraction without changing files |
+| `code_refactor_plan` | Preview a precise rename, extraction, import cleanup, or dead-code deletion without changing files |
 | `code_refactor_apply` | Apply a fresh refactor plan after safety checks |
 
 Pi's built-in `grep` remains the right tool for literal or regular-expression searches; these tools add symbol and source-structure awareness.
@@ -123,7 +123,7 @@ If a server is missing, the agent can still use available workspace and structur
 
 ## Status and settings
 
-Open the status view to see detected languages, running or missing servers, and capability warnings:
+Open the status view to see each LSP route with its workspace-relative root, running or missing servers, typed route issue counts, and capability warnings:
 
 ```text
 /supi-ci-status

package/node_modules/@mrclrchtr/supi-code-runtime/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrclrchtr/supi-code-runtime",
-  "version": "6.0.1",
+  "version": "6.2.0",
   "description": "Shared workspace context and capability contracts for code intelligence",
   "license": "MIT",
   "repository": {

package/node_modules/@mrclrchtr/supi-code-runtime/src/capability/types.ts

@@ -75,9 +75,14 @@ export interface SemanticProvider {
     filePath: string,
     control?: CodeRequestControl,
   ): Promise<CodeQueryResult<DocumentCodeSymbol[]>>;
+  /**
+   * Collect workspace symbols from routes that can contribute within the
+   * optional canonical path scopes.
+   */
   workspaceSymbols(
     query: string,
     control?: CodeRequestControl,
+    scopes?: readonly string[],
   ): Promise<CodeQueryResult<CodeSymbol[]>>;
 
   /** Optional definition capability with explicit completed-empty semantics. */

package/node_modules/@mrclrchtr/supi-core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrclrchtr/supi-core",
-  "version": "6.0.1",
+  "version": "6.2.0",
   "description": "Shared settings, configuration, reporting, and session infrastructure",
   "license": "MIT",
   "repository": {

package/node_modules/@mrclrchtr/supi-core/src/debug-identity.ts

@@ -0,0 +1,11 @@
+/** Maximum UTF-16 code-unit length of one debug identity string. */
+export const MAX_DEBUG_IDENTITY_STRING = 512;
+
+/** Marker appended to a truncated debug identity string. */
+export const DEBUG_IDENTITY_TRUNCATION_MARKER = "…";
+
+/** Bound one debug identity string, including its truncation marker. */
+export function truncateDebugIdentity(value: string): string {
+  if (value.length <= MAX_DEBUG_IDENTITY_STRING) return value;
+  return `${value.slice(0, MAX_DEBUG_IDENTITY_STRING - 1)}${DEBUG_IDENTITY_TRUNCATION_MARKER}`;
+}

package/node_modules/@mrclrchtr/supi-core/src/debug.ts

@@ -3,6 +3,11 @@
 // Kept separate from debug-registry.ts so debug-timing.ts can import the
 // registry without creating an import cycle through the barrel re-export.
 
+export {
+  DEBUG_IDENTITY_TRUNCATION_MARKER,
+  MAX_DEBUG_IDENTITY_STRING,
+  truncateDebugIdentity,
+} from "./debug-identity.ts";
 // biome-ignore lint/performance/noReExportAll: preserve the stable debug domain entry point
 export * from "./debug-registry.ts";
 // biome-ignore lint/performance/noReExportAll: preserve the stable debug domain entry point

package/node_modules/@mrclrchtr/supi-lsp/README.md

@@ -82,6 +82,24 @@ Configuration overrides merge with the built-in server definitions. Use `.pi/sup
 
 Gopls pull diagnostics stay opt-in while golang/go#70199 is open; without the option the built-in Go configuration stays in push mode. Kotlin's `--stdio` argument is already part of the built-in configuration and needs no override.
 
+### Automatic workspace path policy
+
+Automatic LSP work uses one path policy that does not change for each workspace runtime. It covers project discovery, route startup, warm-up, sentinel and source-file lists, created-file tracking, runtime guidance, and diagnostic summaries not tied to one request.
+
+The policy excludes these directories by default: `.git`, `.cache`, `.pi`, `.pnpm`, `node_modules`, `dist`, `build`, `out`, `coverage`, `.next`, `.nuxt`, `.turbo`, and `__pycache__`. It also applies `lsp.exclude` patterns and root or nested `.gitignore` rules. Patterns use gitignore syntax, including rules relative to each directory and `!` rules. Built-in exclusions cannot be enabled again. Symbolic-link directories are not visited. Other dot-directories, such as `.github` and `.storybook`, remain allowed.
+
+Set `lsp.exclude` in project or global SuPi configuration:
+
+```json
+{
+  "lsp": {
+    "exclude": ["generated/**", "!generated/keep.ts"]
+  }
+}
+```
+
+An exact semantic request can still route an excluded file when a compatible server is available. This does not add the file to automatic work. Configured diagnostic suppression still applies to diagnostic output.
+
 ### Custom server configuration
 
 A custom server needs a command and at least one file type:
@@ -104,7 +122,7 @@ A custom server needs a command and at least one file type:
 
 File types do not include a leading dot. If `rootMarkers` is omitted, the server uses the session root.
 
-The `lsp.enabled` and `lsp.active` settings are deprecated and ignored. Disable one language with `lsp.servers.<language>.enabled: false`. Use `/supi-ci-status` from `@mrclrchtr/supi-code-intelligence` to see detected, running, and missing servers.
+Disable one language with `lsp.servers.<language>.enabled: false`. Use `/supi-ci-status` from `@mrclrchtr/supi-code-intelligence` to see detected, running, and missing servers.
 
 ## What it provides
 
@@ -146,7 +164,11 @@ Retained and persisted LSP debug events may identify local workspaces, servers,
 - `method` — exact LSP method, e.g. `textDocument/hover`
 - `root` — server root, absolute where present
 
-`runtime.transition` events carry `cwd` and a bounded `servers` array (name, status, ready; at most 16 entries) alongside the aggregate counts. `readiness.*` events carry `cwd`, `server`, and `root`; their messages and data never embed raw progress-token values. `request.timing` events carry the exact `method`, `server`, and `cwd`, plus the JSON-RPC error code: the server-reported code for failed requests, and the defined constant `-32095` (`LSP_REQUEST_TIMEOUT_ERROR_CODE`) for local timeouts; cancellations carry no code. `diagnostics.timing` events carry `cwd`, `server`, and a workspace-relative `file` for `sync-file` operations; `refresh-open` stays aggregate. `runtime.recovery` events carry `cwd` and bounded attempted/restarted server names; a cancelled pass records the server names still running at cancellation, while restart identity requires the pass result. `capability.transition` events fire only on semantic ready↔pending transitions and carry `cwd` and the ready state — never for initialize, registration, or unregistration traffic. Code-intelligence events (`code-operation.*`, `workflow.timing`, `ast-scan.timing`) carry `cwd` only.
+`runtime.transition` events carry `cwd` and a bounded `servers` array (name, status, ready, and an optional process-crash status reason; at most 16 entries) alongside the aggregate counts. `readiness.*` events carry `cwd`, `server`, and `root`; their messages and data never embed raw progress-token values. `request.timing` events carry the exact `method`, `server`, and `cwd`, plus the JSON-RPC error code: the server-reported code for failed requests, and the defined constant `-32095` (`LSP_REQUEST_TIMEOUT_ERROR_CODE`) for local timeouts; cancellations carry no code. `diagnostics.timing` events carry `cwd`, `server`, and a workspace-relative `file` for `sync-file` operations; `refresh-open` stays aggregate. `runtime.recovery` events carry `cwd`, bounded server and route-root identity, an outcome, and elapsed time for process-crash recovery. Diagnostic recovery events keep their existing bounded attempted/restarted server names. `capability.transition` events fire only on semantic ready↔pending transitions and carry `cwd` and the ready state — never for initialize, registration, or unregistration traffic. Code-intelligence events (`code-operation.*`, `workflow.timing`, `ast-scan.timing`) carry `cwd` only.
+
+`ProjectServerInfo.statusReason` is present only for process-crash recovery states: `process-crashed`, `process-crash-recovery-pending`, or `process-crash-recovery-exhausted`. An LSP route is one configured server and workspace root. A route stays in `error` status for all three states.
+
+Semantic evidence operations recover each required, previously running crashed route and wait for the shared replacement. Scoped workspace-symbol operations select routes by operation support and root intersection; unscoped operations select every known supporting route. Required routes start in parallel. File diagnostics can also recover their route. An explicit broad diagnostic refresh can recover crashed routes that retain tracked files in its scope. Server inventory, workspace readiness, and passive diagnostic snapshots do not start recovery. Recovery never cold-starts a route that did not initialize successfully.
 
 Identity strings — `cwd`, `server`, `file`, `method`, and `root` — are bounded to 512 UTF-16 code units (marker included; truncation appends `…`) and server lists to 16 entries. No raw protocol dumps, request/response params, diagnostic text, progress tokens, or unbounded file lists are recorded; `openFiles` stays a count. Identity fields are intentionally **not** secret-redacted — the debug registry still redacts secret keys and values, but server names, workspace-relative files, and method names pass through unredacted so local protocol failures stay diagnosable. The supi-debug package documents this disclosure for retained and persisted events.
 
@@ -187,7 +209,7 @@ Detected servers start concurrently. In a polyglot workspace, disable unneeded l
 }
 ```
 
-The old `lsp.enabled` and `lsp.active` settings are deprecated and ignored. If every server definition is disabled, the controller publishes an explicit `disabled` runtime state instead of an empty `ready` runtime. When a ready owner has no active client yet, it stays published for lazy routing while semantic capability remains pending.
+If every server definition is disabled, the controller publishes an explicit `disabled` runtime state instead of an empty `ready` runtime. When a ready owner has no active client yet, it stays published for lazy routing while semantic capability remains pending.
 
 ## Architecture
 
@@ -208,7 +230,7 @@ See [`docs/adr/0016-workspace-lsp-runtime-interface.md`](../../docs/adr/0016-wor
 
 ## Package exports
 
-- `@mrclrchtr/supi-lsp/api` — runtime/controller/config types and registry operations
+- `@mrclrchtr/supi-lsp/api` — runtime/controller/config types, registry operations, and automatic path-policy helpers
 - `@mrclrchtr/supi-lsp/provider/lsp-semantic-provider` — semantic provider adapter
 
 ## Source

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-code-runtime/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrclrchtr/supi-code-runtime",
-  "version": "6.0.1",
+  "version": "6.2.0",
   "description": "Shared workspace context and capability contracts for code intelligence",
   "license": "MIT",
   "repository": {

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-code-runtime/src/capability/types.ts

@@ -75,9 +75,14 @@ export interface SemanticProvider {
     filePath: string,
     control?: CodeRequestControl,
   ): Promise<CodeQueryResult<DocumentCodeSymbol[]>>;
+  /**
+   * Collect workspace symbols from routes that can contribute within the
+   * optional canonical path scopes.
+   */
   workspaceSymbols(
     query: string,
     control?: CodeRequestControl,
+    scopes?: readonly string[],
   ): Promise<CodeQueryResult<CodeSymbol[]>>;
 
   /** Optional definition capability with explicit completed-empty semantics. */

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrclrchtr/supi-core",
-  "version": "6.0.1",
+  "version": "6.2.0",
   "description": "Shared settings, configuration, reporting, and session infrastructure",
   "license": "MIT",
   "repository": {

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-core/src/debug-identity.ts

@@ -0,0 +1,11 @@
+/** Maximum UTF-16 code-unit length of one debug identity string. */
+export const MAX_DEBUG_IDENTITY_STRING = 512;
+
+/** Marker appended to a truncated debug identity string. */
+export const DEBUG_IDENTITY_TRUNCATION_MARKER = "…";
+
+/** Bound one debug identity string, including its truncation marker. */
+export function truncateDebugIdentity(value: string): string {
+  if (value.length <= MAX_DEBUG_IDENTITY_STRING) return value;
+  return `${value.slice(0, MAX_DEBUG_IDENTITY_STRING - 1)}${DEBUG_IDENTITY_TRUNCATION_MARKER}`;
+}

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-core/src/debug.ts

@@ -3,6 +3,11 @@
 // Kept separate from debug-registry.ts so debug-timing.ts can import the
 // registry without creating an import cycle through the barrel re-export.
 
+export {
+  DEBUG_IDENTITY_TRUNCATION_MARKER,
+  MAX_DEBUG_IDENTITY_STRING,
+  truncateDebugIdentity,
+} from "./debug-identity.ts";
 // biome-ignore lint/performance/noReExportAll: preserve the stable debug domain entry point
 export * from "./debug-registry.ts";
 // biome-ignore lint/performance/noReExportAll: preserve the stable debug domain entry point

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-jsonrpc/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "vscode-jsonrpc",
 	"description": "A json rpc implementation over streams",
-	"version": "9.0.1",
+	"version": "9.0.2",
 	"author": "Microsoft Corporation",
 	"license": "MIT",
 	"repository": {

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-languageserver-protocol/lib/common/protocol.colorProvider.d.ts

@@ -62,7 +62,7 @@ export interface ColorPresentationParams extends WorkDoneProgressParams, Partial
 export declare namespace ColorPresentationRequest {
     const method: 'textDocument/colorPresentation';
     const messageDirection: MessageDirection;
-    const type: ProtocolRequestType<ColorPresentationParams, ColorPresentation[], ColorPresentation[], void, WorkDoneProgressOptions & TextDocumentRegistrationOptions>;
+    const type: ProtocolRequestType<ColorPresentationParams, ColorPresentation[], ColorPresentation[], void, DocumentColorRegistrationOptions>;
     type HandlerSignature = RequestHandler<ColorPresentationParams, ColorPresentation[], void>;
     const capabilities: CM<"textDocument.colorProvider", "colorProvider">;
 }

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-languageserver-protocol/lib/common/protocol.d.ts

@@ -2778,12 +2778,12 @@ export interface CodeActionClientCapabilities {
      * Client supports the tag property on a code action. Clients
      * supporting tags have to handle unknown tags gracefully.
      *
-     * @since 3.18.0 - proposed
+     * @since 3.18.0
      */
     tagSupport?: CodeActionTagOptions;
 }
 /**
- * @since 3.18.0 - proposed
+ * @since 3.18.0
  */
 export type CodeActionTagOptions = {
     /**

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-languageserver-protocol/lib/common/protocol.fileOperations.d.ts

@@ -1,5 +1,5 @@
 import { NotificationHandler, RequestHandler } from 'vscode-jsonrpc';
-import { WorkspaceEdit } from 'vscode-languageserver-types';
+import { WorkspaceEdit, type DocumentUri } from 'vscode-languageserver-types';
 import { CM, MessageDirection, ProtocolNotificationType, ProtocolRequestType } from './messages';
 /**
  * Options for notifications/requests for user operations on files.
@@ -172,9 +172,9 @@ export interface CreateFilesParams {
  */
 export interface FileCreate {
     /**
-     * A file:// URI for the location of the file/folder being created.
+     * A URI for the location of the file/folder being created.
      */
-    uri: string;
+    uri: DocumentUri;
 }
 /**
  * The parameters sent in notifications/requests for user-initiated renames of
@@ -196,13 +196,13 @@ export interface RenameFilesParams {
  */
 export interface FileRename {
     /**
-     * A file:// URI for the original location of the file/folder being renamed.
+     * A URI for the original location of the file/folder being renamed.
      */
-    oldUri: string;
+    oldUri: DocumentUri;
     /**
-     * A file:// URI for the new location of the file/folder being renamed.
+     * A URI for the new location of the file/folder being renamed.
      */
-    newUri: string;
+    newUri: DocumentUri;
 }
 /**
  * The parameters sent in notifications/requests for user-initiated deletes of
@@ -223,9 +223,9 @@ export interface DeleteFilesParams {
  */
 export interface FileDelete {
     /**
-     * A file:// URI for the location of the file/folder being deleted.
+     * A URI for the location of the file/folder being deleted.
      */
-    uri: string;
+    uri: DocumentUri;
 }
 /**
  * The will create files request is sent from the client to the server before files are actually

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-languageserver-protocol/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "vscode-languageserver-protocol",
 	"description": "VSCode Language Server Protocol implementation",
-	"version": "3.18.2",
+	"version": "3.18.3",
 	"author": "Microsoft Corporation",
 	"license": "MIT",
 	"repository": {
@@ -27,8 +27,8 @@
 		}
 	},
 	"dependencies": {
-		"vscode-jsonrpc": "9.0.1",
-		"vscode-languageserver-types": "3.18.0"
+		"vscode-jsonrpc": "9.0.2",
+		"vscode-languageserver-types": "3.18.3"
 	},
 	"scripts": {
 		"prepublishOnly": "echo \"⛔ Can only publish from a secure pipeline ⛔\" && node ../build/npm/fail",

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-languageserver-types/lib/esm/main.d.ts

@@ -2517,7 +2517,7 @@ export type CodeActionDisabled = {
 /**
  * Code action tags are extra annotations that tweak the behavior of a code action.
  *
- * @since 3.18.0 - proposed
+ * @since 3.18.0
  */
 export declare namespace CodeActionTag {
     /**
@@ -2599,7 +2599,7 @@ export interface CodeAction {
     /**
      * Tags for this code action.
      *
-     * @since 3.18.0 - proposed
+     * @since 3.18.0
      */
     tags?: CodeActionTag[];
 }

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-languageserver-types/lib/esm/main.js

@@ -1758,7 +1758,7 @@ export var CodeActionContext;
 /**
  * Code action tags are extra annotations that tweak the behavior of a code action.
  *
- * @since 3.18.0 - proposed
+ * @since 3.18.0
  */
 export var CodeActionTag;
 (function (CodeActionTag) {

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-languageserver-types/lib/umd/main.d.ts

@@ -2517,7 +2517,7 @@ export type CodeActionDisabled = {
 /**
  * Code action tags are extra annotations that tweak the behavior of a code action.
  *
- * @since 3.18.0 - proposed
+ * @since 3.18.0
  */
 export declare namespace CodeActionTag {
     /**
@@ -2599,7 +2599,7 @@ export interface CodeAction {
     /**
      * Tags for this code action.
      *
-     * @since 3.18.0 - proposed
+     * @since 3.18.0
      */
     tags?: CodeActionTag[];
 }

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-languageserver-types/lib/umd/main.js

@@ -1790,7 +1790,7 @@
     /**
      * Code action tags are extra annotations that tweak the behavior of a code action.
      *
-     * @since 3.18.0 - proposed
+     * @since 3.18.0
      */
     var CodeActionTag;
     (function (CodeActionTag) {

package/node_modules/@mrclrchtr/supi-lsp/node_modules/vscode-languageserver-types/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "vscode-languageserver-types",
 	"description": "Types used by the Language server for node",
-	"version": "3.18.0",
+	"version": "3.18.3",
 	"author": "Microsoft Corporation",
 	"license": "MIT",
 	"repository": {

package/node_modules/@mrclrchtr/supi-lsp/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrclrchtr/supi-lsp",
-  "version": "6.0.1",
+  "version": "6.2.0",
   "description": "Language Server Protocol runtime for SuPi code intelligence",
   "license": "MIT",
   "repository": {
@@ -37,8 +37,8 @@
     "vscode-jsonrpc": "^9.0.0",
     "vscode-languageserver-protocol": "^3.18.0",
     "vscode-languageserver-types": "^3.18.0",
-    "@mrclrchtr/supi-code-runtime": "6.0.1",
-    "@mrclrchtr/supi-core": "6.0.1"
+    "@mrclrchtr/supi-code-runtime": "6.2.0",
+    "@mrclrchtr/supi-core": "6.2.0"
   },
   "bundledDependencies": [
     "@mrclrchtr/supi-code-runtime",
@@ -53,7 +53,6 @@
   "exports": {
     "./api": "./src/api.ts",
     "./package.json": "./package.json",
-    "./provider/lsp-semantic-provider": "./src/provider/lsp-semantic-provider.ts",
-    "./debug-telemetry": "./src/debug-telemetry.ts"
+    "./provider/lsp-semantic-provider": "./src/provider/lsp-semantic-provider.ts"
   }
 }

package/node_modules/@mrclrchtr/supi-lsp/src/api.ts

@@ -1,13 +1,8 @@
 // Public API surface for the LSP session-scoped service.
 
 export { type LoadConfigOptions, loadConfig } from "./config/config.ts";
-export type { DeprecatedLspKeys, LspSettings } from "./config/lsp-settings.ts";
-export {
-  getDeprecatedLspKeys,
-  getLspDisabledMessage,
-  hasDeprecatedLspKeys,
-  loadLspSettings,
-} from "./config/lsp-settings.ts";
+export type { LspSettings } from "./config/lsp-settings.ts";
+export { loadLspSettings } from "./config/lsp-settings.ts";
 export {
   clearTsconfigCache,
   type FileScopeDecision,
@@ -30,6 +25,7 @@ export type {
   MissingServer,
   Position,
   ProjectServerInfo,
+  ProjectServerStatusReason,
   Range,
   SymbolInformation,
   WorkspaceEdit,
@@ -42,6 +38,8 @@ export { isLikelyStaleDiagnostic } from "./diagnostics/stale-diagnostics.ts";
 export {
   scanWorkspaceSentinels,
   syncWorkspaceSentinelSnapshot,
+  type WorkspaceSentinelScanOptions,
+  type WorkspaceSentinelSyncResult,
 } from "./diagnostics/workspace-sentinels.ts";
 export { raceReadinessValue, raceRequestControl } from "./session/readiness.ts";
 export type {
@@ -57,6 +55,8 @@ export type {
   DiagnosticEvidenceStatus,
   DiagnosticEvidenceSummary,
   OutstandingDiagnosticSummaryEntry,
+  ProcessCrashDiagnosticDemand,
+  ProcessCrashRecoverySummary,
   RecoverDiagnosticsResult,
   RoutedMutationResponse,
   SemanticReadinessResult,
@@ -69,8 +69,15 @@ export type {
 } from "./session/runtime-registry.ts";
 export {
   clearWorkspaceLspRuntime,
+  emptyProcessCrashRecoverySummary,
   getWorkspaceLspRuntime,
   setWorkspaceLspRuntimeState,
   waitForWorkspaceLspRuntime,
 } from "./session/runtime-registry.ts";
 export { scanMissingServers } from "./session/scanner.ts";
+export {
+  AUTOMATIC_LSP_EXCLUDED_DIRECTORIES,
+  type AutomaticLspPathPolicy,
+  createAutomaticLspPathPolicy,
+  createDefaultAutomaticLspPathPolicy,
+} from "./workspace-path-policy.ts";

package/node_modules/@mrclrchtr/supi-lsp/src/client/client-diagnostic-cache.ts

@@ -1,5 +1,5 @@
+import { uriToFile } from "@mrclrchtr/supi-core/path";
 import type { PublishDiagnosticsParams } from "../config/types.ts";
-import { uriToFile } from "../utils.ts";
 import {
   type DiagnosticCacheEntry,
   isTentativePushEntry,

package/node_modules/@mrclrchtr/supi-lsp/src/client/client-diagnostic-capabilities.ts

@@ -66,11 +66,6 @@ export class ClientDynamicRegistrations {
     return (this.idsByMethod.get(method)?.size ?? 0) > 0;
   }
 
-  /** Snapshot of the active registration ids for a method. */
-  ids(method: string): readonly string[] {
-    return [...(this.idsByMethod.get(method) ?? [])];
-  }
-
   /** Drop all registrations — capability loss on shutdown, crash, or disposal. */
   clear(): void {
     this.idsByMethod.clear();

package/node_modules/@mrclrchtr/supi-lsp/src/client/client-diagnostic-publication.ts

@@ -9,9 +9,12 @@
 // and timing data. They never carry diagnostic payloads or source text.
 
 import * as path from "node:path";
-import { recordDebugEvent } from "@mrclrchtr/supi-core/debug";
-import { boundCwd, truncateIdentity } from "../debug-telemetry.ts";
-import { uriToFile } from "../utils.ts";
+import {
+  recordDebugEvent,
+  truncateDebugIdentity as truncateIdentity,
+} from "@mrclrchtr/supi-core/debug";
+import { uriToFile } from "@mrclrchtr/supi-core/path";
+import { boundCwd } from "../debug-telemetry.ts";
 
 /** Maximum tracked synchronizations before the oldest entry is evicted. */
 export const MAX_TRACKED_SYNCHRONIZATIONS = 64;

package/node_modules/@mrclrchtr/supi-lsp/src/client/client-diagnostic-refresh.ts

@@ -5,12 +5,13 @@ import {
   isCodeRequestInterruption,
   throwIfCodeRequestInterrupted,
 } from "@mrclrchtr/supi-code-runtime/api";
+import { fileToUri, uriToFile } from "@mrclrchtr/supi-core/path";
 import type { TextDocumentIdentifier } from "../config/types.ts";
 import {
   type DiagnosticEvidenceSummary,
   summarizeDiagnosticEvidence,
 } from "../diagnostics/evidence.ts";
-import { detectLanguageId, fileToUri, uriToFile } from "../utils.ts";
+import { detectLanguageId } from "../utils.ts";
 import {
   type DiagnosticCacheEntry,
   type DiagnosticSynchronization,

package/node_modules/@mrclrchtr/supi-lsp/src/client/client-diagnostic-timing.ts

@@ -1,6 +1,9 @@
 import type { CodeRequestControl } from "@mrclrchtr/supi-code-runtime/api";
-import { startDebugTimer } from "@mrclrchtr/supi-core/debug";
-import { boundCwd, truncateIdentity } from "../debug-telemetry.ts";
+import {
+  startDebugTimer,
+  truncateDebugIdentity as truncateIdentity,
+} from "@mrclrchtr/supi-core/debug";
+import { boundCwd } from "../debug-telemetry.ts";
 
 type DiagnosticCollection = "cache" | "fallback" | "none" | "pull" | "push";
 type DiagnosticFreshness = "not-observed" | "observed";

package/node_modules/@mrclrchtr/supi-lsp/src/client/client-diagnostics.ts

@@ -7,9 +7,10 @@ import {
   throwIfCodeRequestInterrupted,
   unavailableCodeQuery,
 } from "@mrclrchtr/supi-code-runtime/api";
+import { fileToUri, uriToFile } from "@mrclrchtr/supi-core/path";
 import type { Diagnostic, TextDocumentItem } from "../config/types.ts";
 import type { DiagnosticEvidenceSummary } from "../diagnostics/evidence.ts";
-import { detectLanguageId, fileToUri, uriToFile } from "../utils.ts";
+import { detectLanguageId } from "../utils.ts";
 import { applyPushDiagnostics, buildClientDiagnosticSnapshot } from "./client-diagnostic-cache.ts";
 import { collectSynchronizedFileDiagnostics } from "./client-diagnostic-collection.ts";
 import {

package/node_modules/@mrclrchtr/supi-lsp/src/client/client.ts

@@ -12,7 +12,11 @@ import {
   throwIfCodeRequestInterrupted,
   unavailableCodeQuery,
 } from "@mrclrchtr/supi-code-runtime/api";
-import { recordDebugEvent } from "@mrclrchtr/supi-core/debug";
+import {
+  recordDebugEvent,
+  truncateDebugIdentity as truncateIdentity,
+} from "@mrclrchtr/supi-core/debug";
+import { fileToUri } from "@mrclrchtr/supi-core/path";
 import { type ProgressToken, TextDocumentSyncKind } from "vscode-languageserver-protocol";
 import { CLIENT_CAPABILITIES } from "../config/capabilities.ts";
 import type {
@@ -35,10 +39,9 @@ import type {
   WorkspaceEdit,
   WorkspaceSymbol,
 } from "../config/types.ts";
-import { boundCwd, truncateIdentity } from "../debug-telemetry.ts";
+import { boundCwd } from "../debug-telemetry.ts";
 import type { DiagnosticEvidenceSummary } from "../diagnostics/evidence.ts";
 import { raceRequestControl } from "../session/readiness.ts";
-import { fileToUri } from "../utils.ts";
 import {
   ClientDynamicRegistrations,
   DOCUMENT_DIAGNOSTIC_METHOD,
@@ -141,6 +144,15 @@ function killProcessTree(pid: number): void {
   }
 }
 
+/** Terminate a failed startup without leaving server descendants running. */
+function killFailedStartupProcess(child: ChildProcess): void {
+  if (child.pid) {
+    killProcessTree(child.pid);
+    return;
+  }
+  child.kill();
+}
+
 // ── Types ─────────────────────────────────────────────────────────────
 export type ClientStatus = "initializing" | "running" | "error" | "shutdown";
 
@@ -285,7 +297,7 @@ export class LspClient {
     if (!this.process.stdin || !this.process.stdout) {
       const failure = new Error(`${cmd}: missing stdin/stdout`);
       this.handleProcessFailure(failure);
-      this.process.kill();
+      killFailedStartupProcess(this.process);
       throw failure;
     }
 
@@ -335,8 +347,14 @@ export class LspClient {
       if (positionEncoding !== "utf-16") {
         throw new Error(`Server selected unsupported position encoding "${positionEncoding}".`);
       }
+      if (this._status !== "initializing") {
+        throw new Error(`${this.name}: client shutdown during initialize`);
+      }
       this.capabilities = result.capabilities;
       await this.rpc.sendNotification("initialized", {});
+      if (this._status !== "initializing") {
+        throw new Error(`${this.name}: client shutdown during initialize`);
+      }
       this._status = "running";
       this.publishLifecycle("startup");
 
@@ -344,7 +362,7 @@ export class LspClient {
     } catch (err) {
       const failure = new Error(`${this.name}: initialize failed: ${err}`, { cause: err });
       this.handleProcessFailure(failure);
-      this.process.kill();
+      killFailedStartupProcess(this.process);
       throw failure;
     }
   }