git:20260713.d9d98d1 to git:20260817.3dd8826
35 added, 3 removed. Audit A to A.
- # omo-senpi LSP Component
+ # lsp component
- Daemon-backed Senpi LSP adapter. This component retains only Senpi-facing descriptors, schemas, renderers, path extraction, post-edit wiring, and migration-warning helpers adapted from `pi-lsp-client` / oh-my-pi by Yeongyu Kim. Client, transport, JSON-RPC, manager, server-resolution, project-trust, diagnostics, workspace-edit, and process execution live in `@oh-my-opencode/lsp-core` plus the packaged `@code-yeongyu/lsp-daemon` runtime.
+ ## Overview
- Project-local `.pi/lsp-client.json` commands are intentionally ignored here. Users who still need custom LSP commands must move those definitions to their user `.pi/lsp-client.json`; project configs may keep safe fields such as extensions and priorities, but command/env entries only produce a migration warning.
+ Daemon-backed Senpi LSP adapter. This component retains only Senpi-facing descriptors, schemas, renderers, path extraction, post-edit wiring, and migration-warning helpers adapted from `pi-lsp-client` / oh-my-pi by Yeongyu Kim. Client, transport, JSON-RPC, manager, server-resolution, project-trust, diagnostics, workspace-edit, and process execution live in `@oh-my-opencode/lsp-core` plus the packaged `@code-yeongyu/lsp-daemon` runtime. Every tool call is forwarded out of process: the component never spawns language servers itself.
+
+ ## Structure
+
+ | Path | Purpose |
+ |------|---------|
+ | `index.ts` | Component factory (`createLspComponent`): flag registration, six-tool registration with the daemon runtime wrapped over each `execute`, post-edit `tool_result` hook, session lifecycle (`session_start`/`session_compact`/`session_shutdown` state reset plus daemon-client cache clear). |
+ | `daemon-runtime.ts` | Resolves the packaged daemon CLI at `../runtime/lsp-daemon/dist/cli.js` (path must exist, version validated against a strict pattern); `OMO_LSP_DAEMON_CLI` + `OMO_LSP_DAEMON_VERSION` override both-or-neither, absolute path only. |
+ | `daemon-tool-client.ts` | Loads the daemon's `client.js` beside the CLI (cached per path), maps Senpi tool names to daemon names (`lsp_diagnostics` -> `diagnostics`, ...), and builds the per-call request context: canonical cwd, project `.pi/lsp-client.json` path, user `~/.pi/lsp-client.json`, install-decisions path, `installDecisionTool: false`. |
+ | `post-edit-diagnostics.ts` | Post-edit diagnostics over `@oh-my-opencode/lsp-core/post-edit`: mutation tools are `write`/`edit`/`apply_patch`, concurrency 4, per-session not-configured cache, widget lines under key `omo-senpi-lsp`. |
+ | `adapter/descriptors.ts` | The six ToolDefinitions (`lsp_diagnostics`, `lsp_goto_definition`, `lsp_find_references`, `lsp_symbols`, `lsp_prepare_rename`, `lsp_rename`) with typed details and TUI renderers. |
+ | `adapter/renderers-*.ts` / `rendering.ts` | Call/result renderers per tool family (diagnostics, navigation, rename, symbols). |
+ | `adapter/migration-notices.ts` | Scans project `.pi/lsp-client.json` for non-disabled entries carrying `command`/`env` and produces `untrusted_project_lsp_command` notices. |
+ | `adapter/schema.ts` / `language-mappings.ts` | TypeBox `defineTool` helper and extension-to-language tables. |
+
+ ## Wiring
+
+ - `registerLspFlags` always runs; two boolean flags, both default `true`: `omo-senpi-lsp-tools-enabled` gates the whole component (early return when `false`), `omo-senpi-lsp-post-edit-diagnostics-enabled` gates only the post-edit hook.
+ - Each registered tool's `execute` ignores the Senpi-provided context and calls `callPackagedDaemonTool(name, args, {signal})`, which lazily imports the packaged daemon client and forwards with the current request context.
+ - Post-edit: on `tool_result` for a mutation tool, `handlePostEditDiagnosticsToolResult` reports "(OmO) Checking LSP Diagnostics" tool-hook status, runs `lsp_diagnostics` with `severity: "error"` through the daemon, appends diagnostic blocks to the tool result content, and syncs the `omo-senpi-lsp` widget. A daemon `not_configured` availability answer becomes a cached per-extension skip.
+ - `session_shutdown` deletes per-session post-edit state and clears the daemon client module cache; `session_compact` resets only the session's not-configured cache.
+
+ ## Conventions
+
+ - Config-notice warnings fire once at register time through `ctx.logger.warn`, before tool registration.
+ - The daemon version and CLI override are validated fail-closed: mismatched override pair, relative path, or missing file all throw at resolution.
+ - Session ids come from `sessionManager.getSessionId()` on the event context; a missing id degrades to an anonymous state slot, never a crash.
+
+ ## Anti-patterns
+
+ - Do not read project-local `.pi/lsp-client.json` commands. They're intentionally ignored here; users who still need custom LSP commands must move those definitions to their user `.pi/lsp-client.json`. Project configs may keep safe fields such as extensions and priorities, but command/env entries only produce a migration warning.
+ - Do not add LSP client, transport, or server-management logic in this component; that belongs to `lsp-core` and the daemon package.
+ - Do not enable `installDecisionTool` in the request context; the Senpi adapter has no interactive install-decision surface.