oh-my-opencode 5.0.0-beta.23 → 5.0.0-beta.25

package/dist/skills/debugging/SKILL.md

@@ -53,8 +53,6 @@ These are not "optional extras". They are the correct tool in their domain, and
 | **Ghidra** | Any binary without trustworthy source — third-party closed libs, malware, vendored binaries whose behavior contradicts docs, CTF, firmware. **Use Ghidra's decompiler before `strings`/`objdump` guessing. It turns machine code into readable C.** | 📖 **[references/tools/ghidra.md](references/tools/ghidra.md)** |
 | **pwndbg** | Any native binary debugging session. It is GDB with the useful views (registers, stack, disasm, heap) always visible. **If you'd reach for plain `gdb`, reach for `pwndbg` instead — it is strictly a superset.** | 📖 **[references/tools/pwndbg.md](references/tools/pwndbg.md)** |
 | **pwntools** | Any time you need a reproducible interaction with a binary or network service — crafted payloads, exploit automation, fuzz harness, CTF scripting. | 📖 **[references/tools/pwntools.md](references/tools/pwntools.md)** |
-| **Frida** | Any running process you must instrument live without source or symbols — hook a function and print real argument values, trace calls, stub a return. Complements Ghidra: Ghidra reads the bytes, Frida watches them execute. **If Ghidra's static decompile has hit its limit, reach for Frida.** | 📖 **[references/tools/frida.md](references/tools/frida.md)** |
-| **DAP client (`dap.mjs`)** | Any time you would drive a debugger through a PTY and screen-scrape its text. Debuggers already speak the machine-readable Debug Adapter Protocol (debugpy, `dlv dap`, lldb-dap, js-debug); this bundled script drives it with bounded, monitorable output. **Prefer it over scraping `gdb`/`pdb` whenever the debugger speaks DAP.** Design modeled on oh-my-pi's `debug` tool (github.com/can1357/oh-my-pi). | 📖 **[references/tools/dap.md](references/tools/dap.md)** |
 
 **Failing to use these tools in their domain is a process failure, not a stylistic choice.** If the bug is in a browser and you did Phase 8 without Playwright, you are doing it wrong. If the bug is in a stripped binary and you read hex with `xxd`, you are doing it wrong. The references tell you how. Read them.
 

package/dist/skills/debugging/references/methodology/00-setup.md

@@ -35,18 +35,6 @@ The shape of the answers you need (commands in the runtime reference):
 | Is there an existing failing test or known repro? | Prefer amplifying an existing repro over inventing one. |
 | Are watchers (file watchers, hot reloaders, supervisors) going to restart the process mid-session? | If yes, turn them off before attaching. Restarts drop inspector connections and invalidate breakpoints. |
 
-### Output budget
-
-**Bound every inspection before you run it; unbounded output destroys the investigation's working context.**
-- Set a frame count, element count, or byte range for every query.
-- Prefer one targeted query over one broad dump: ask for the value that distinguishes hypotheses.
-- Page through more data only when the first bounded result says it is necessary; do not re-dump the whole object.
-- Treat any command whose output size you cannot predict as a command that needs a limit.
-- Before: GDB `bt` — After: GDB `bt 10` to inspect only the top ten frames.
-- Before: LLDB `memory read 0x1000` — After: `memory read 0x1000 -c 32` for a bounded byte range.
-- Before: LLDB `frame variable` — After: `frame variable --children-count 10 user` for one variable's children.
-- This is not presentation etiquette: a flooded context hides the decisive observation and can end the investigation.
-
 ### 4. Gate check
 
 If any answer is "I'm not sure", you are not ready for Phase 1. Investigate until certain. Guessing here cascades into false-positive hypotheses in Phase 2.

package/dist/skills/debugging/references/methodology/02-investigate.md

@@ -39,27 +39,6 @@ If two hypotheses have identical distinguishing evidence, they aren't actually d
 
 ## Phase 3 — Parallel Investigation
 
-### State freshness invariant
-
-**A debug session's state is a snapshot with a lifetime, not a fact you can carry forward indefinitely.**
-- Before any action that mutates the debuggee, re-observe the session's current state.
-- This includes continue, step, side-effecting evaluation, setting or removing breakpoints, and termination.
-- Record the current thread state and stop reason; those are the concrete tells for whether the old plan still applies.
-- An observation from an earlier turn may already be stale when the next action is issued.
-- Turns can be minutes apart while the debuggee runs at full speed, so this failure mode is normal, not exceptional.
-- If the thread state or stop reason has moved, re-observe the new state and do not replay the old plan.
-- Treat every mutation as conditional on the state you just observed, not on a remembered stop.
-
-### Failure-to-recovery taxonomy
-
-| Failure signature | Most likely cause | Next move |
-|---|---|---|
-| Adapter or debugger process failed to start | Wrong binary or missing installation | Verify the tool exists, then re-launch; do not retry blindly. |
-| Breakpoint accepted but never bound or verified | Source path mismatch, optimized-out code, or missing debug symbols | Check the binary was built with symbols and that the debugger-resolved path matches your file. |
-| Attach refused | Permissions, ptrace scope, SIP, or wrong PID | Fix the environment or target identity; a retry will not change it. |
-| No stop event within the expected window | Process is running, the breakpoint is unreachable, or the wait was too short | Pause and inspect threads rather than waiting longer. |
-| Session terminated unexpectedly | The debuggee crashed or exited | Capture the exit and crash evidence before restarting. |
-
 Branch depending on what's available.
 
 ### Path A: Team mode ENABLED

package/dist/skills/debugging/references/runtimes/go.md

@@ -42,24 +42,10 @@ dlv exec ./bin/myserver
 # Attach to a running process
 dlv attach $(pgrep myserver)
 
-# Delve's JSON-RPC API (IDE / remote attach) — default port 2345
+# Headless mode (IDE / remote attach) — default port 2345
 dlv debug --headless --listen=:2345 --api-version=2 ./cmd/server
 ```
 
-Delve exposes two distinct machine-readable interfaces:
-
-```bash
-# Debug Adapter Protocol: use this for editors and DAP-aware automation
-dlv dap --listen=127.0.0.1:2345
-
-# Delve's JSON-RPC API: use this for clients built directly against Delve's API
-dlv debug --headless --listen=127.0.0.1:2345 --api-version=2 ./cmd/server
-```
-
-Pick `dlv dap` when the client speaks DAP; pick headless JSON-RPC when you need Delve's
-native API or an existing JSON-RPC client. Both expose structured state, unlike scraping
-an interactive `(dlv)` session.
-
 ### Building a debuggable binary
 
 The compiler inlines and optimizes aggressively in normal builds, which makes stepping confusing. For serious debugging:

package/dist/skills/debugging/references/runtimes/native-binary.md

@@ -107,24 +107,6 @@ print(repr(data[max(0,pos-100):pos+200]))
 
 If you must keep using `strings`, lower the threshold: `strings -n 1 -t x ./target | rg ...`. The signal-to-noise drops sharply but short content is preserved.
 
-### Sanitizer build for a crashing C/C++ binary
-
-If the binary can be rebuilt from C/C++ sources, reproduce the crash with sanitizers before
-trying to infer the fault from stripped machine code. AddressSanitizer catches out-of-bounds
-accesses and use-after-free; UndefinedBehaviorSanitizer reports undefined operations.
-
-```bash
-# Rebuild with source locations and runtime checks enabled
-CFLAGS='-g -O1 -fsanitize=address,undefined' \
-CXXFLAGS='-g -O1 -fsanitize=address,undefined' \
-make clean all
-./target
-```
-
-The sanitizer report identifies the invalid access and its allocation/free or undefined
-operation stack, often making the root cause clear before dynamic analysis. For a binary
-that cannot be rebuilt, continue with the native tracing and debugger workflow below.
-
 Write the triage summary to the journal:
 
 ```markdown
@@ -247,26 +229,6 @@ lldb ./target
 (lldb) image dump symtab ./target
 ```
 
-### Data watchpoints — catch the corruption, not just the crash
-
-When a value is already wrong by the time execution reaches the crash, watch the memory
-address for writes and stop at the instruction that corrupts it. Use the address of the
-field or buffer identified from the static/dynamic hypothesis; see [tools/pwndbg.md](../tools/pwndbg.md)
-for the richer pwndbg watchpoint workflow rather than duplicating it here.
-
-```text
-# lldb: watch writes to an address (replace with a valid expression/address)
-(lldb) watchpoint set expression -- <addr>
-
-# gdb: stop on writes, or on reads when a read is the first observable symptom
-(gdb) watch *(<type> *)<addr>
-(gdb) rwatch *(<type> *)<addr>
-```
-
-A watchpoint is most useful after narrowing the suspect object: continue from its
-allocation or initialization, and inspect the backtrace and writer when it triggers.
-Hardware watchpoint slots are limited, so remove broad watchpoints before adding more.
-
 **Function interception via `DYLD_INSERT_LIBRARIES`** (macOS equivalent of `LD_PRELOAD`):
 
 ```bash

package/dist/skills/debugging/references/runtimes/node.md

@@ -98,31 +98,6 @@ bun test --inspect-brk                        # debug test runner
 
 **Critical**: Bun uses WebKit Inspector Protocol, not V8. `chrome://inspect` cannot connect directly. Use `debug.bun.sh` or the (currently buggy, per Bun docs) VS Code extension.
 
-### Scripted V8 inspector / CDP attach
-
-The inspector port also serves machine-readable HTTP discovery endpoints. `/json` lists
-inspectable targets; `/json/version` includes the WebSocket debugger URL. Connect that
-URL with the Chrome DevTools Protocol (CDP), rather than scraping `node inspect` text.
-`chrome-remote-interface` is the canonical scriptable npm client. When browser automation
-is already in play, use Playwright's `CDPSession` instead.
-
-```bash
-curl -s http://127.0.0.1:9229/json/version
-```
-
-```js
-const CDP = require('chrome-remote-interface');
-const version = await fetch('http://127.0.0.1:9229/json/version').then(r => r.json());
-const client = await CDP({ target: version.webSocketDebuggerUrl });
-await client.Runtime.enable();
-await client.Debugger.pause();
-```
-
-The same CDP commands can be sent through `chrome-remote-interface`; with Playwright,
-create a session with `await context.newCDPSession(page)` and call
-`await session.send('Runtime.enable')`. These V8/CDP examples do not apply to Bun's
-WebKit inspector protocol.
-
 ### Deno (native V8, Chrome DevTools / VS Code compatible)
 
 ```bash

package/dist/skills/debugging/references/runtimes/python.md

@@ -41,19 +41,6 @@ python -c 'import asyncio; print(asyncio.__version__)'
 | **post-mortem `pdb.pm()`** | Exception already happened, you want to inspect state | In an exception-caught REPL: `import pdb; pdb.pm()` after the exception propagates |
 | **debugpy (remote / IDE)** | IDE attach, remote host, containerized process | `python -m debugpy --listen 5678 --wait-for-client script.py` then attach from VS Code / PyCharm |
 
-`debugpy` is a Debug Adapter Protocol (DAP) adapter, not merely an IDE convenience.
-Its listener exposes a machine-readable protocol that an agent or other client can drive
-programmatically; it is a better boundary for automation than the human-oriented pdb REPL.
-
-```bash
-# Listen on the requested interface and wait before executing the script
-python -m debugpy --listen <host:port> --wait-for-client <script>
-```
-
-Use a DAP client to send structured requests and receive structured stopped/continued
-and stack/variable events. By contrast, `pdb` exposes prompt-oriented text: an agent
-using it must screen-scrape output and parse a human-facing REPL.
-
 ### Prefer `ipdb` or `pudb` over plain `pdb` when available
 
 - **ipdb** — drop-in replacement with tab completion, syntax highlighting. `pip install ipdb`, then `PYTHONBREAKPOINT=ipdb.set_trace` or use `import ipdb; ipdb.set_trace()`.

package/dist/skills/debugging/references/runtimes/rust.md

@@ -144,47 +144,6 @@ Rust symbols are mangled. Use either:
 
 ---
 
-## Sanitizers — memory and race bugs
-
-Nightly Rust exposes LLVM sanitizers through `-Z sanitizer`. Use them when a crash,
-data race, or leak is otherwise too timing- or state-dependent to explain from a
-backtrace; they replace guesswork with a report at the invalid operation.
-
-```bash
-# AddressSanitizer: out-of-bounds accesses and use-after-free
-cargo +nightly run -Z sanitizer=address
-
-# ThreadSanitizer: data races between threads
-cargo +nightly test -Z sanitizer=thread
-
-# LeakSanitizer: allocations that remain reachable at process exit
-cargo +nightly run -Z sanitizer=leak
-```
-
-`address` catches invalid memory access and use-after-free, `thread` catches unsynchronized
-concurrent accesses, and `leak` reports memory that was allocated but not released. Run
-the invocation that matches the suspected class; sanitizer diagnostics are especially
-useful before attaching a debugger to optimized or timing-sensitive code.
-
----
-
-## rr — time-travel debugging
-
-On supported Linux systems, record the failure once, then replay it deterministically and
-run backwards from the failure with `reverse-continue` or `reverse-step`:
-
-```bash
-rr record ./target/debug/my_binary
-rr replay
-# (rr) reverse-continue
-# (rr) reverse-step
-```
-
-`rr` is Linux/x86-64 only. It does not apply on Apple Silicon, including arm64 macOS;
-use `rust-lldb` or a sanitizer there instead.
-
----
-
 ## tokio-console — async task debugging
 
 For tokio-based async apps, this is essential when tasks are stuck or leaking.

package/dist/tools/delegate-task/types.d.ts

@@ -1,5 +1,4 @@
 import type { BackgroundManager } from "../../features/background-agent";
-import type { OhMyOpenCodeConfig } from "../../config";
 import type { CategoriesConfig, GitMasterConfig, BrowserAutomationProvider, AgentOverrides, SisyphusAgentConfig } from "../../config/schema";
 import type { ModelFallbackControllerAccessor } from "../../hooks/model-fallback";
 import type { LoadedSkill } from "../../features/opencode-skill-loader/types";
@@ -120,8 +119,6 @@ export interface DelegateTaskToolOptions {
     availableCategories?: AvailableCategory[];
     availableSkills?: AvailableSkill[];
     agentOverrides?: AgentOverrides;
-    /** Reload model-bearing config at task invocation time so edits are honored without rebuilding tools. */
-    loadCurrentModelConfig?: () => Pick<OhMyOpenCodeConfig, "agents" | "categories">;
     sisyphusAgentConfig?: SisyphusAgentConfig;
     modelFallbackControllerAccessor?: ModelFallbackControllerAccessor;
     onSyncSessionCreated?: (event: SyncSessionCreatedEvent) => Promise<void>;

package/dist/tui.js

@@ -17499,25 +17499,6 @@ var init_telemetry = __esm(() => {
   }).strict();
 });
 
-// packages/omo-config-core/src/schema/format-on-mutation.ts
-var mode, languages, OmoFormatOnMutationLayerSchema, OmoFormatOnMutationSchema;
-var init_format_on_mutation = __esm(() => {
-  init_zod();
-  mode = _enum2(["off", "best-effort", "required"]);
-  languages = record(string2(), boolean2()).optional();
-  OmoFormatOnMutationLayerSchema = object({
-    mode: mode.optional(),
-    languages,
-    maxFileBytes: number2().int().positive().optional(),
-    timeoutMs: number2().int().positive().optional()
-  }).strict();
-  OmoFormatOnMutationSchema = OmoFormatOnMutationLayerSchema.extend({
-    mode: mode.default("best-effort"),
-    maxFileBytes: number2().int().positive().default(1048576),
-    timeoutMs: number2().int().positive().default(3000)
-  }).strict();
-});
-
 // packages/omo-config-core/src/schema/config.ts
 var OmoOpenCodeHarnessConfigSchema, OmoTypedHarnessConfigSchema, OmoConfigProfileSchema, OmoConfigSchema, OmoConfigLayerSchema;
 var init_config = __esm(() => {
@@ -17531,10 +17512,8 @@ var init_config = __esm(() => {
   init_task();
   init_team();
   init_telemetry();
-  init_format_on_mutation();
   OmoOpenCodeHarnessConfigSchema = record(string2(), unknown());
   OmoTypedHarnessConfigSchema = object({
-    formatOnMutation: OmoFormatOnMutationLayerSchema.optional(),
     categories: OmoCategoriesConfigSchema.optional(),
     agents: OmoAgentsConfigSchema.optional(),
     codegraph: OmoCodegraphSettingsLayerSchema.optional(),
@@ -17546,7 +17525,6 @@ var init_config = __esm(() => {
     telemetry: OmoTelemetrySettingsLayerSchema.optional()
   }).strict();
   OmoConfigProfileSchema = object({
-    formatOnMutation: OmoFormatOnMutationLayerSchema.optional(),
     categories: OmoCategoriesConfigSchema.optional(),
     agents: OmoAgentsConfigSchema.optional(),
     codegraph: OmoCodegraphSettingsLayerSchema.optional(),
@@ -17561,7 +17539,6 @@ var init_config = __esm(() => {
     "[codex]": OmoTypedHarnessConfigSchema.optional()
   }).strict();
   OmoConfigSchema = object({
-    formatOnMutation: OmoFormatOnMutationSchema.optional(),
     $schema: string2().optional(),
     categories: OmoCategoriesConfigSchema.optional(),
     agents: OmoAgentsConfigSchema.optional(),
@@ -17580,7 +17557,6 @@ var init_config = __esm(() => {
     legacy_migrations: record(string2(), unknown()).optional()
   }).strict();
   OmoConfigLayerSchema = object({
-    formatOnMutation: OmoFormatOnMutationLayerSchema.optional(),
     $schema: string2().optional(),
     categories: OmoCategoriesConfigSchema.optional(),
     agents: OmoAgentsConfigSchema.optional(),
@@ -17607,7 +17583,6 @@ var init_schema = __esm(() => {
   init_codegraph();
   init_config();
   init_fallback_models();
-  init_format_on_mutation();
   init_git_master();
   init_harness();
   init_memory();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-opencode",
-  "version": "5.0.0-beta.23",
+  "version": "5.0.0-beta.25",
   "description": "The Best AI Agent Harness - Batteries-Included OpenCode Plugin with Multi-Model Orchestration, Parallel Background Agents, and Crafted LSP/AST Tools",
   "main": "./dist/index.js",
   "types": "dist/index.d.ts",
@@ -181,7 +181,7 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@code-yeongyu/senpi": "2026.8.27",
+    "@code-yeongyu/senpi": "2026.8.26-2",
     "@oh-my-opencode/agents-md-core": "workspace:*",
     "@oh-my-opencode/ast-grep-mcp": "workspace:*",
     "@oh-my-opencode/boulder-state": "workspace:*",
@@ -222,18 +222,18 @@
     "typescript": "^7.0.2"
   },
   "optionalDependencies": {
-    "oh-my-opencode-darwin-arm64": "5.0.0-beta.23",
-    "oh-my-opencode-darwin-x64": "5.0.0-beta.23",
-    "oh-my-opencode-darwin-x64-baseline": "5.0.0-beta.23",
-    "oh-my-opencode-linux-arm64": "5.0.0-beta.23",
-    "oh-my-opencode-linux-arm64-musl": "5.0.0-beta.23",
-    "oh-my-opencode-linux-x64": "5.0.0-beta.23",
-    "oh-my-opencode-linux-x64-baseline": "5.0.0-beta.23",
-    "oh-my-opencode-linux-x64-musl": "5.0.0-beta.23",
-    "oh-my-opencode-linux-x64-musl-baseline": "5.0.0-beta.23",
-    "oh-my-opencode-windows-arm64": "5.0.0-beta.23",
-    "oh-my-opencode-windows-x64": "5.0.0-beta.23",
-    "oh-my-opencode-windows-x64-baseline": "5.0.0-beta.23"
+    "oh-my-opencode-darwin-arm64": "5.0.0-beta.25",
+    "oh-my-opencode-darwin-x64": "5.0.0-beta.25",
+    "oh-my-opencode-darwin-x64-baseline": "5.0.0-beta.25",
+    "oh-my-opencode-linux-arm64": "5.0.0-beta.25",
+    "oh-my-opencode-linux-arm64-musl": "5.0.0-beta.25",
+    "oh-my-opencode-linux-x64": "5.0.0-beta.25",
+    "oh-my-opencode-linux-x64-baseline": "5.0.0-beta.25",
+    "oh-my-opencode-linux-x64-musl": "5.0.0-beta.25",
+    "oh-my-opencode-linux-x64-musl-baseline": "5.0.0-beta.25",
+    "oh-my-opencode-windows-arm64": "5.0.0-beta.25",
+    "oh-my-opencode-windows-x64": "5.0.0-beta.25",
+    "oh-my-opencode-windows-x64-baseline": "5.0.0-beta.25"
   },
   "overrides": {
     "@earendil-works/pi-agent-core": "0.84.2",
@@ -249,8 +249,5 @@
   },
   "trustedDependencies": [
     "@code-yeongyu/comment-checker"
-  ],
-  "patchedDependencies": {
-    "@code-yeongyu/senpi@2026.8.27": "patches/@code-yeongyu%2Fsenpi@2026.8.27.patch"
-  }
+  ]
 }

package/packages/lsp-core/src/index.ts

@@ -7,7 +7,6 @@ export * from "./lsp/constants.js";
 export * from "./lsp/directory-diagnostics.js";
 export * from "./lsp/effective-extension.js";
 export * from "./lsp/errors.js";
-export * from "./lsp/format-document.js";
 export * from "./lsp/formatters.js";
 export * from "./lsp/infer-extension.js";
 export * from "./lsp/json-rpc-connection.js";

package/packages/lsp-core/src/lsp/client-wrapper.ts

@@ -17,7 +17,6 @@ import {
 } from "./errors.js";
 import { getLspManager, type LspManager } from "./manager.js";
 import { findWorkspaceRootOutsideContext } from "./outside-context-workspace.js";
-import { LSP_LOCAL_INSTALL_HINTS } from "./server-definitions.js";
 import { loadInstallDecision } from "./server-install-state.js";
 import { findServerForExtension } from "./server-resolution.js";
 import type { ServerLookupResult } from "./types.js";
@@ -139,26 +138,6 @@ export function formatServerLookupError(result: Exclude<ServerLookupResult, { st
 	].join("\n");
 }
 
-/**
- * Builds the install guidance, offering the repo-local install first.
- *
- * A repo-local install is resolvable without touching the global environment, so it is the
- * recommended route whenever the server ships as a project dependency.
- */
-function formatInstallOptions(serverId: string, installHint: string): string[] {
-	const localHint = LSP_LOCAL_INSTALL_HINTS[serverId];
-	if (localHint === undefined) {
-		return ["To install, run:", `  ${installHint}`];
-	}
-	return [
-		"To install in THIS repository (preferred — no global install needed):",
-		`  ${localHint}`,
-		"",
-		"Or install it globally:",
-		`  ${installHint}`,
-	];
-}
-
 function formatNotInstalled(result: Extract<ServerLookupResult, { status: "not_installed" }>): string {
 	const { server, installHint } = result;
 	const extensions = server.extensions.join(", ");
@@ -175,20 +154,20 @@ function formatNotInstalled(result: Extract<ServerLookupResult, { status: "not_i
 		`Command not found: ${server.command[0]}`,
 		"",
 	];
-	const installOptions = formatInstallOptions(server.id, installHint);
 
 	if (decision === "allowed") {
 		return [
 			...header,
 			"The user has pre-authorized LSP installation. Run the install command, then retry this tool:",
-			...installOptions.slice(1),
+			`  ${installHint}`,
 		].join("\n");
 	}
 
 	if (!context.capabilities.installDecisionTool) {
 		return [
 			...header,
-			...installOptions,
+			"To install, run:",
+			`  ${installHint}`,
 			"",
 			"ACTION REQUIRED — ASK THE USER whether to install this LSP server.",
 			"Install-decision recording is unavailable in this harness; proceed without LSP if the user declines.",
@@ -197,7 +176,8 @@ function formatNotInstalled(result: Extract<ServerLookupResult, { status: "not_i
 
 	return [
 		...header,
-		...installOptions,
+		"To install, run:",
+		`  ${installHint}`,
 		"",
 		"ACTION REQUIRED — ASK THE USER whether to install this LSP server.",
 		"- If the user agrees: run the install command above, then retry this tool.",

package/packages/lsp-core/src/lsp/client.ts

@@ -6,7 +6,6 @@ import type { LspClientTimeoutOptions } from "./transport.js";
 import type {
 	Diagnostic,
 	DocumentSymbol,
-	FormattingOptions,
 	Location,
 	LocationLink,
 	PrepareRenameDefaultBehavior,
@@ -14,7 +13,6 @@ import type {
 	Range,
 	ResolvedServer,
 	SymbolInfo,
-	TextEdit,
 	WorkspaceEdit,
 } from "./types.js";
 import { LspRequestTimeoutError } from "./errors.js";
@@ -135,30 +133,6 @@ export class LspClient extends LspClientConnection {
 		}, options);
 	}
 
-	/**
-	 * Requests whole-document formatting edits, returning null when the server never advertised the
-	 * capability so callers can report unavailability instead of surfacing a method-not-found error.
-	 */
-	async formatDocument(
-		filePath: string,
-		options: FormattingOptions,
-		signal?: AbortSignal,
-	): Promise<TextEdit[] | null> {
-		if (!this.isDocumentFormattingSupported()) return null;
-		const absPath = this.resolveWorkspacePath(filePath);
-		await this.openFile(absPath);
-		const requestOptions = signal === undefined ? {} : { signal };
-		const edits = await this.sendRequest<TextEdit[] | null>(
-			"textDocument/formatting",
-			{
-				textDocument: { uri: pathToFileURL(absPath).href },
-				options,
-			},
-			requestOptions,
-		);
-		return edits ?? [];
-	}
-
 	async workspaceSymbols(query: string, signal?: AbortSignal): Promise<SymbolInfo[]> {
 		const options = signal === undefined ? {} : { signal };
 		return this.sendRequest<SymbolInfo[]>("workspace/symbol", { query }, options);

package/packages/lsp-core/src/lsp/connection.ts

@@ -4,7 +4,6 @@ import { LspClientTransport } from "./transport.js";
 
 interface InitializeCapabilities {
 	readonly diagnosticProvider?: unknown;
-	readonly documentFormattingProvider?: unknown;
 }
 
 function supportsDiagnosticPull(capabilities: InitializeCapabilities | undefined): boolean {
@@ -12,19 +11,6 @@ function supportsDiagnosticPull(capabilities: InitializeCapabilities | undefined
 	return Object.hasOwn(capabilities, "diagnosticProvider");
 }
 
-/**
- * Reads the whole-document formatting capability the server advertised at initialize time.
- *
- * The provider is either a boolean or an options object, and an explicit `false` means the
- * server refuses the request, so both shapes are checked instead of mere key presence.
- */
-function supportsDocumentFormatting(capabilities: InitializeCapabilities | undefined): boolean {
-	if (capabilities === undefined) return false;
-	const provider = capabilities.documentFormattingProvider;
-	if (provider === undefined || provider === null || provider === false) return false;
-	return provider === true || typeof provider === "object";
-}
-
 export class LspClientConnection extends LspClientTransport {
 	async initialize(): Promise<void> {
 		const rootUri = pathToFileURL(this.root).href;
@@ -42,7 +28,6 @@ export class LspClientConnection extends LspClientTransport {
 						references: {},
 						documentSymbol: { hierarchicalDocumentSymbolSupport: true },
 						publishDiagnostics: {},
-						formatting: { dynamicRegistration: false },
 						rename: {
 							prepareSupport: true,
 							prepareSupportDefaultBehavior: 1,
@@ -86,7 +71,6 @@ export class LspClientConnection extends LspClientTransport {
 			{ timeoutMs: this.initializeTimeoutMs },
 		);
 		this.setDiagnosticPullSupported(supportsDiagnosticPull(result?.capabilities));
-		this.setDocumentFormattingSupported(supportsDocumentFormatting(result?.capabilities));
 		await this.sendNotification("initialized", {});
 		await this.sendNotification("workspace/didChangeConfiguration", {
 			settings: { json: { validate: { enable: true } } },

package/packages/lsp-core/src/lsp/constants.ts

@@ -6,7 +6,6 @@ export const DEFAULT_MAX_DIRECTORY_FILES = 50;
 export const REQUEST_TIMEOUT_MS = 15_000;
 export const INIT_TIMEOUT_MS = 60_000;
 export const IDLE_TIMEOUT_MS = 5 * 60_000;
-export const MAX_RESIDENT_CLIENTS = 6;
 export const REAPER_INTERVAL_MS = 60_000;
 export const STOP_HARD_KILL_TIMEOUT_MS = 5_000;
 export const STOP_SIGKILL_GRACE_MS = 1_000;

package/packages/lsp-core/src/lsp/manager.ts

@@ -1,6 +1,6 @@
 import { reportBestEffortCleanupError } from "./cleanup-errors.js";
 import { LspClient } from "./client.js";
-import { IDLE_TIMEOUT_MS, INIT_TIMEOUT_MS, MAX_RESIDENT_CLIENTS, REAPER_INTERVAL_MS } from "./constants.js";
+import { IDLE_TIMEOUT_MS, INIT_TIMEOUT_MS, REAPER_INTERVAL_MS } from "./constants.js";
 import { installProcessSignalCleanup } from "./process-signal-cleanup.js";
 import type { ResolvedServer } from "./types.js";
 
@@ -28,7 +28,6 @@ export interface ClientSnapshot {
 export interface LspManagerOptions {
 	idleTimeoutMs?: number;
 	initTimeoutMs?: number;
-	maxResidentClients?: number;
 	reaperIntervalMs?: number;
 	clientFactory?: (root: string, server: ResolvedServer) => LspClient;
 	now?: () => number;
@@ -81,7 +80,6 @@ export class LspManager {
 
 	private readonly idleTimeoutMs: number;
 	private readonly initTimeoutMs: number;
-	private readonly maxResidentClients: number;
 	private readonly reaperIntervalMs: number;
 	private readonly clientFactory: (root: string, server: ResolvedServer) => LspClient;
 	private readonly now: () => number;
@@ -89,7 +87,6 @@ export class LspManager {
 	constructor(options: LspManagerOptions = {}) {
 		this.idleTimeoutMs = options.idleTimeoutMs ?? IDLE_TIMEOUT_MS;
 		this.initTimeoutMs = options.initTimeoutMs ?? INIT_TIMEOUT_MS;
-		this.maxResidentClients = options.maxResidentClients ?? MAX_RESIDENT_CLIENTS;
 		this.reaperIntervalMs = options.reaperIntervalMs ?? REAPER_INTERVAL_MS;
 		this.clientFactory = options.clientFactory ?? ((root, server) => new LspClient(root, server));
 		this.now = options.now ?? (() => Date.now());
@@ -137,32 +134,6 @@ export class LspManager {
 		}
 	}
 
-	/**
-	 * Frees capacity for one new client by stopping the least recently used idle client.
-	 *
-	 * Only clients with no in-flight work are eligible, so a resident server is never torn down
-	 * underneath a live request; when every resident client is busy the cap is exceeded instead.
-	 */
-	private evictForAdmission(): void {
-		while (this.clients.size >= this.maxResidentClients) {
-			const victim = this.leastRecentlyUsedIdleClient();
-			if (!victim) return;
-			this.clients.delete(victim.key);
-			void stopClientBestEffort(victim.managed.client);
-		}
-	}
-
-	private leastRecentlyUsedIdleClient(): { readonly key: string; readonly managed: ManagedClient } | null {
-		let candidate: { readonly key: string; readonly managed: ManagedClient } | null = null;
-		for (const [key, managed] of this.clients) {
-			if (managed.refCount > 0 || managed.pendingWaiters > 0 || managed.isInitializing) continue;
-			if (candidate === null || managed.lastUsedAt < candidate.managed.lastUsedAt) {
-				candidate = { key, managed };
-			}
-		}
-		return candidate;
-	}
-
 	private async tryDeleteIfOrphaned(key: string, managed: ManagedClient): Promise<void> {
 		if (
 			managed.refCount === 0 &&
@@ -226,8 +197,6 @@ export class LspManager {
 			return managed.client;
 		}
 
-		this.evictForAdmission();
-
 		const client = this.clientFactory(root, server);
 		const initStartedAt = this.now();
 		const initPromise = (async () => {
@@ -295,8 +264,6 @@ export class LspManager {
 		const key = this.getKey(root, server.id);
 		if (this.clients.has(key)) return;
 
-		this.evictForAdmission();
-
 		const client = this.clientFactory(root, server);
 		const initStartedAt = this.now();
 		const initPromise = (async () => {