oh-my-opencode 4.8.1 → 4.9.1

package/packages/omo-codex/scripts/install-hook-targets.test.mjs

@@ -0,0 +1,100 @@
+import assert from "node:assert/strict";
+import { mkdir, mkdtemp, rm, writeFile } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { dirname, join } from "node:path";
+import test from "node:test";
+
+import { assertHookCommandTargets, findMissingHookCommandTargets } from "./install/hook-targets.mjs";
+
+const HOOKS_MANIFEST = {
+	hooks: {
+		SessionStart: [
+			{
+				hooks: [
+					{
+						type: "command",
+						command: 'node "${PLUGIN_ROOT}/components/git-bash/dist/cli.js" hook session-start',
+					},
+				],
+			},
+			{
+				matcher: "^startup$",
+				hooks: [
+					{
+						type: "command",
+						command: 'node "${PLUGIN_ROOT}/scripts/auto-update.mjs" hook session-start',
+					},
+				],
+			},
+		],
+		Stop: [
+			{
+				hooks: [
+					{
+						type: "command",
+						command: 'node "${PLUGIN_ROOT}/components/ulw-loop/dist/cli.js" hook stop',
+					},
+				],
+			},
+		],
+	},
+};
+
+async function createPluginRoot({ withGitBashDist }) {
+	const pluginRoot = await mkdtemp(join(tmpdir(), "hook-targets-"));
+	await mkdir(join(pluginRoot, "hooks"), { recursive: true });
+	await writeFile(join(pluginRoot, "hooks", "hooks.json"), JSON.stringify(HOOKS_MANIFEST, null, "\t"));
+	const targets = [
+		"scripts/auto-update.mjs",
+		"components/ulw-loop/dist/cli.js",
+		...(withGitBashDist ? ["components/git-bash/dist/cli.js"] : []),
+	];
+	for (const target of targets) {
+		await mkdir(dirname(join(pluginRoot, target)), { recursive: true });
+		await writeFile(join(pluginRoot, target), "");
+	}
+	return pluginRoot;
+}
+
+test("#given a hook command target missing from the payload #when scanning #then exactly that path is reported", async (t) => {
+	const pluginRoot = await createPluginRoot({ withGitBashDist: false });
+	t.after(() => rm(pluginRoot, { recursive: true, force: true }));
+
+	const missing = await findMissingHookCommandTargets(pluginRoot);
+
+	assert.deepEqual(missing, [join(pluginRoot, "components/git-bash/dist/cli.js")]);
+});
+
+test("#given a complete payload #when scanning #then nothing is missing", async (t) => {
+	const pluginRoot = await createPluginRoot({ withGitBashDist: true });
+	t.after(() => rm(pluginRoot, { recursive: true, force: true }));
+
+	assert.deepEqual(await findMissingHookCommandTargets(pluginRoot), []);
+});
+
+test("#given no hooks manifest #when scanning #then nothing is missing", async (t) => {
+	const pluginRoot = await mkdtemp(join(tmpdir(), "hook-targets-"));
+	t.after(() => rm(pluginRoot, { recursive: true, force: true }));
+
+	assert.deepEqual(await findMissingHookCommandTargets(pluginRoot), []);
+});
+
+test("#given a broken payload #when asserting #then the error names the missing target and the plugin root", async (t) => {
+	const pluginRoot = await createPluginRoot({ withGitBashDist: false });
+	t.after(() => rm(pluginRoot, { recursive: true, force: true }));
+
+	await assert.rejects(
+		() => assertHookCommandTargets(pluginRoot),
+		(error) =>
+			error instanceof Error &&
+			error.message.includes("components/git-bash/dist/cli.js") &&
+			error.message.includes("hooks.json"),
+	);
+});
+
+test("#given a complete payload #when asserting #then it resolves", async (t) => {
+	const pluginRoot = await createPluginRoot({ withGitBashDist: true });
+	t.after(() => rm(pluginRoot, { recursive: true, force: true }));
+
+	await assertHookCommandTargets(pluginRoot);
+});

package/packages/omo-codex/scripts/install-lazycodex-version-stamp.test.mjs

@@ -1,5 +1,5 @@
 import assert from "node:assert/strict";
-import { readFile } from "node:fs/promises";
+import { mkdir, readFile, writeFile } from "node:fs/promises";
 import { join } from "node:path";
 import test from "node:test";
 
@@ -53,6 +53,8 @@ test("#given sisyphuslabs lazycodex install #when installing locally #then stamp
 			],
 		},
 	});
+	await mkdir(join(pluginRoot, "components", "comment-checker", "dist"), { recursive: true });
+	await writeFile(join(pluginRoot, "components", "comment-checker", "dist", "cli.js"), "console.log('comment checker cli')\n");
 
 	const result = await installMarketplaceLocally({
 		repoRoot,

package/packages/omo-codex/scripts/install-local.mjs

@@ -39,17 +39,8 @@ import {
 } from "./install/lazycodex-version-stamp.mjs";
 import { shouldBuildSourcePackages } from "./install/source-package-build.mjs";
 import { runLazyCodexManualUpdate } from "../plugin/scripts/auto-update.mjs";
-
-export function resolveCodexInstallerBinDir(options = {}) {
-	const homeDir = resolve(options.homeDir ?? homedir());
-	const env = options.env ?? process.env;
-	const explicitBinDir = nonEmptyEnvValue(env, "CODEX_LOCAL_BIN_DIR");
-	if (explicitBinDir !== undefined) return explicitBinDir;
-
-	const codexHome = resolve(options.codexHome ?? nonEmptyEnvValue(env, "CODEX_HOME") ?? join(homeDir, ".codex"));
-	const defaultCodexHome = resolve(join(homeDir, ".codex"));
-	return codexHome === defaultCodexHome ? join(homeDir, ".local", "bin") : join(codexHome, "bin");
-}
+export { nonEmptyEnvValue, resolveCodexInstallerBinDir } from "./install/bin-dir.mjs";
+import { nonEmptyEnvValue, resolveCodexInstallerBinDir } from "./install/bin-dir.mjs";
 
 export async function installMarketplaceLocally(options = {}) {
 	const repoRoot = resolve(options.repoRoot ?? process.cwd());
@@ -115,6 +106,10 @@ export async function installMarketplaceLocally(options = {}) {
 		if (marketplace.name === "sisyphuslabs" && plugin.name === "omo") {
 			const runtimeLink = await linkRootRuntimeBin({ binDir, codexHome, repoRoot, platform });
 			if (runtimeLink !== null) log(`Linked ${runtimeLink.name} -> ${runtimeLink.target}`);
+			else
+				log(
+					`Warning: skipped the omo runtime wrapper because ${join(repoRoot, "dist", "cli", "index.js")} is missing; omo sparkshell/ulw-loop commands will be unavailable until a package shipping dist/cli is installed`,
+				);
 		}
 		pluginSources.push({ name: entry.name, sourcePath });
 		installed.push(plugin);
@@ -161,6 +156,7 @@ export async function installMarketplaceLocally(options = {}) {
 		marketplaceSource: { sourceType: "local", source: marketplaceRoot },
 		pluginNames,
 		platform,
+		gitBashEnabled: platform === "win32" && gitBashResolution.found,
 		trustedHookStates,
 		agentConfigs: [...agentConfigs.values()].sort((left, right) => left.name.localeCompare(right.name)),
 		autonomousPermissions: options.autonomousPermissions !== false,
@@ -198,13 +194,6 @@ function agentNameFromToml(fileName) {
 	return fileName.endsWith(".toml") ? fileName.slice(0, -".toml".length) : fileName;
 }
 
-function nonEmptyEnvValue(env, key) {
-	const value = env[key];
-	if (typeof value !== "string") return undefined;
-	const trimmed = value.trim();
-	return trimmed.length === 0 ? undefined : trimmed;
-}
-
 function legacyCacheMarketplaces(marketplaceName) {
 	return marketplaceName === "sisyphuslabs" ? ["lazycodex", ["code", "yeongyu", "codex", "plugins"].join("-")] : [];
 }

package/packages/omo-codex/scripts/install-local.test.mjs

@@ -1,5 +1,5 @@
 import assert from "node:assert/strict";
-import { mkdir, readFile, readlink, writeFile } from "node:fs/promises";
+import { mkdir, readFile, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { dirname, join } from "node:path";
 import test from "node:test";
@@ -66,6 +66,38 @@ test("#given custom CODEX_HOME and PATH without omo #when installing locally wit
 	assert.match(wrapper, /omo-ulw-loop/);
 });
 
+test("#given repoRoot without root CLI dist #when installing locally #then warns about the skipped omo runtime wrapper", async () => {
+	const repoRoot = await makeTempDir();
+	const codexHome = await makeTempDir();
+	const homeDir = await makeTempDir();
+	const codexPackageRoot = join(repoRoot, "packages", "omo-codex");
+	const pluginRoot = join(codexPackageRoot, "plugin");
+
+	await writeJson(join(codexPackageRoot, "marketplace.json"), {
+		name: "sisyphuslabs",
+		plugins: [{ name: "omo", source: "./plugins/omo" }],
+	});
+	await writePluginAt(pluginRoot, "omo", "0.1.0");
+
+	const logs = [];
+	await installMarketplaceLocally({
+		repoRoot,
+		codexHome,
+		env: {},
+		homeDir,
+		platform: "linux",
+		runCommand: async () => {},
+		log: (line) => logs.push(String(line)),
+	});
+
+	const cliPath = join(repoRoot, "dist", "cli", "index.js");
+	assert.ok(
+		logs.some((line) => line.includes("omo runtime wrapper") && line.includes(cliPath)),
+		`expected a warning naming the missing ${cliPath}; got:\n${logs.join("\n")}`,
+	);
+	await assert.rejects(readFile(join(codexHome, "bin", "omo"), "utf8"));
+});
+
 test("#given explicit CODEX_LOCAL_BIN_DIR #when resolving local installer bin dir #then preserves installed omo precedence", () => {
 	const homeDir = join(tmpdir(), "omo-codex-home-explicit");
 	const codexHome = join(tmpdir(), "omo-codex-install-explicit");
@@ -121,6 +153,7 @@ test("#given plugin hooks #when installing #then records trusted hook hashes", a
 	});
 	const pluginRoot = join(codexPackageRoot, "plugin");
 	await writePluginAt(pluginRoot, "alpha", "1.2.3");
+	await writeFile(join(pluginRoot, "dist", "cli.js"), "console.log('plugin cli')\n");
 	await writeJson(join(pluginRoot, "hooks", "hooks.json"), {
 		hooks: {
 			UserPromptSubmit: [

package/packages/shared-skills/skills/lcx-contribute-bug-fix/SKILL.md

@@ -30,7 +30,22 @@ Create a PR that includes:
 
 1. Read the user's bug report and identify the affected surface.
 2. Invoke `$omo:debugging` for the investigation. If only unqualified skill names are exposed, invoke `$debugging` and state that it is the OMO debugging skill.
-3. Decide the target repository. If ownership is close, compare against upstream Codex source under `/tmp/openai-codex-source` before choosing.
+3. Materialize the latest sources, then decide the target repository. Sync both checkouts on every run and compare them before choosing — a stale checkout routes the fix to the wrong repo:
+
+```bash
+sync_latest_source() {
+  REPO="$1"; DEST="$2"
+  if [ ! -d "$DEST/.git" ]; then
+    gh repo clone "$REPO" "$DEST" -- --depth=1 \
+      || git clone --depth=1 "https://github.com/$REPO" "$DEST"
+  fi
+  DEFAULT_BRANCH="$(git -C "$DEST" remote show origin | sed -n '/HEAD branch/s/.*: //p')"
+  git -C "$DEST" fetch --depth=1 origin "$DEFAULT_BRANCH"
+  git -C "$DEST" checkout -B "$DEFAULT_BRANCH" FETCH_HEAD
+}
+sync_latest_source code-yeongyu/lazycodex /tmp/lazycodex-source
+sync_latest_source openai/codex /tmp/openai-codex-source
+```
 4. Create a fresh temporary clone and branch. Do not modify the user's current repository for the target fix unless the current repository is itself the requested target and the user explicitly asked for local edits.
 
 ```bash

package/packages/shared-skills/skills/lcx-doctor/SKILL.md

@@ -0,0 +1,93 @@
+---
+name: lcx-doctor
+description: "Diagnose LazyCodex and Codex CLI installation health against the latest sources. Use whenever the user asks for a doctor or health check, says LazyCodex, lazycodex-ai, omo-codex, or Codex behaves oddly after an install, update, or config change, suspects a stale, drifted, or broken setup, or wants the local install audited and compared with the latest LazyCodex and Codex code."
+metadata:
+  short-description: Diagnose LazyCodex/Codex install health against latest sources
+---
+
+# lcx-doctor
+
+You are a LazyCodex install doctor. Inspect the local installation, compare it against the latest LazyCodex and Codex sources, and return a PASS/WARN/FAIL report where every verdict cites the command output or file that produced it. Diagnose only: the only writes you make are under `/tmp`. Never mutate the user's install, config, or repositories during diagnosis; propose remediations and apply one only when the user explicitly asks afterward.
+
+Use GPT-5.5 style: outcome first, concise, evidence-bound.
+
+## Required Workflow
+
+1. Materialize the latest sources under `/tmp` first. Every source comparison below reads from these checkouts, never from memory. Re-sync on every run so a cached checkout cannot go stale:
+
+```bash
+sync_latest_source() {
+  REPO="$1"; DEST="$2"
+  if [ ! -d "$DEST/.git" ]; then
+    gh repo clone "$REPO" "$DEST" -- --depth=1 \
+      || git clone --depth=1 "https://github.com/$REPO" "$DEST"
+  fi
+  DEFAULT_BRANCH="$(git -C "$DEST" remote show origin | sed -n '/HEAD branch/s/.*: //p')"
+  git -C "$DEST" fetch --depth=1 origin "$DEFAULT_BRANCH"
+  git -C "$DEST" checkout -B "$DEFAULT_BRANCH" FETCH_HEAD
+}
+sync_latest_source code-yeongyu/lazycodex /tmp/lazycodex-source
+sync_latest_source openai/codex /tmp/openai-codex-source
+```
+
+2. Inventory the installed surface. Resolve `CODEX_HOME` (default `~/.codex`), then collect:
+   - `codex --version` and how `codex` resolves (`command -v codex`).
+   - Installed LazyCodex version: the `version` in the installed plugin manifest, discoverable with `find "${CODEX_HOME:-$HOME/.codex}/plugins" -path '*/.codex-plugin/plugin.json'`. Installed plugins live under `$CODEX_HOME/plugins/cache/<marketplace>/<name>/<version>/`.
+   - Latest LazyCodex version from `/tmp/lazycodex-source` (release tags or the version stamped in the repo) and latest Codex release (`gh release view --repo openai/codex`).
+   - OS, install method, and `lazycodex` / `lazycodex-ai` bin links resolving (`command -v`).
+3. Check config and wiring against the latest installer, not against assumptions. Read what the current installer under `/tmp/lazycodex-source` writes (installer sources live in the omo-codex package, e.g. `scripts/install/`), then verify the local equivalents:
+   - `$CODEX_HOME/config.toml` exists and parses; LazyCodex-managed entries match what the latest installer would write.
+   - Plugin payload present and non-empty: `hooks/hooks.json`, `skills/`, `.mcp.json`, components under the installed plugin root.
+   - Stale project-local leftovers the installer now removes (e.g. `.codex/hooks.json`, `.codex/skills` in the project) are flagged, not deleted.
+4. Probe the real surface. Run the built-in diagnostics first: `lazycodex doctor --json` (add `--verbose` when a check needs deeper traces) and fold its results into the report. A missing or crashing `lazycodex doctor` is itself a FAIL finding, not a reason to skip probing — fall back to `codex --version` plus a trivial non-interactive invocation that loads the plugin. Capture stderr verbatim; a clean exit with warnings is WARN, not PASS.
+5. Compare for drift. Where installed bundled files differ from the same files at the installed version, or the latest source renamed or removed something the local config still references, record it with both paths.
+6. Check whether each FAIL is already known: `gh issue list --repo code-yeongyu/lazycodex --search "<short symptom>" --state open` (and `openai/codex` when the failure points upstream). Link matches in the report instead of re-diagnosing from scratch.
+7. If a probe fails and the cause is not explained by config or source comparison, invoke `$omo:debugging` for the investigation. If Codex exposes only unqualified skill names in the current session, invoke `$debugging` and state that it is the OMO debugging skill.
+8. Emit the report.
+
+## Doctor Report Template
+
+```markdown
+## LazyCodex Doctor Report
+
+### Summary
+[One sentence: healthy, degraded, or broken — and the single most important next action.]
+
+### Environment
+- LazyCodex installed / latest:
+- Codex CLI installed / latest:
+- CODEX_HOME:
+- OS / install method:
+
+### Checks
+| Check | Verdict | Evidence |
+| --- | --- | --- |
+| Versions current | PASS/WARN/FAIL | [command output or file:line] |
+| config.toml integrity | PASS/WARN/FAIL | [evidence] |
+| Plugin payload wiring | PASS/WARN/FAIL | [evidence] |
+| Bin links / aliases | PASS/WARN/FAIL | [evidence] |
+| Runtime probe | PASS/WARN/FAIL | [evidence] |
+| Drift vs latest source | PASS/WARN/FAIL | [evidence, citing /tmp/lazycodex-source or /tmp/openai-codex-source paths] |
+
+### Remediations
+1. [Most important fix first: exact command or config edit, and what it resolves.]
+
+### Known Issues Matched
+- [issue URL — or "none found"]
+```
+
+## Follow-up Routing
+
+- Local misconfiguration or stale install: give the remediation; reinstalling via the standard LazyCodex install command is the default fix for payload drift.
+- Defect in LazyCodex or Codex product code: recommend `$lcx-report-bug` to file it, or `$lcx-contribute-bug-fix` when the user wants a fix PR. Both reuse the `/tmp` checkouts you already synced.
+
+## Stop Conditions
+
+Ask one narrow question only when a finding requires a destructive decision, such as deleting user-edited config or downgrading a version.
+
+Do not:
+
+- mutate config, installs, or repositories during diagnosis
+- report a verdict without captured evidence
+- compare against remembered source layout instead of `/tmp/lazycodex-source` and `/tmp/openai-codex-source`
+- declare healthy while any probe output was never captured

package/packages/shared-skills/skills/lcx-doctor/agents/openai.yaml

@@ -0,0 +1,11 @@
+interface:
+  display_name: "lcx-doctor (omo)"
+  short_description: "Diagnose LazyCodex/Codex install health against latest sources"
+  search_terms:
+    - "lcx-doctor"
+    - "lazycodex doctor"
+    - "lazycodex health check"
+    - "codex doctor"
+    - "lazycodex install broken"
+    - "omo-codex diagnose"
+  default_prompt: "Use $lcx-doctor to sync the latest LazyCodex and Codex sources into /tmp, inventory the local installation, compare versions, config, and wiring against the latest sources, probe the real surface, and report PASS/WARN/FAIL findings with evidence and remediations."

package/packages/shared-skills/skills/lcx-report-bug/SKILL.md

@@ -33,24 +33,28 @@ Create or prepare a GitHub issue or PR that includes:
 
 1. Read the user's bug report and identify the affected surface: LazyCodex installer, Codex plugin, skill, hook, MCP, CLI alias, GitHub marketplace sync, or web/docs.
 2. Invoke `$omo:debugging` for the investigation. If Codex exposes only unqualified skill names in the current session, invoke `$debugging` and state that it is the OMO debugging skill.
-3. Materialize upstream Codex source under `/tmp` before deciding ownership:
+3. Materialize the latest LazyCodex and upstream Codex sources under `/tmp` before deciding ownership. Re-sync on every run so a cached checkout cannot go stale — stale source produces wrong routing and dead line references:
 
 ```bash
-CODEX_SRC="/tmp/openai-codex-source"
-if [ ! -d "$CODEX_SRC/.git" ]; then
-  gh repo clone openai/codex "$CODEX_SRC" -- --depth=1
-else
-  git -C "$CODEX_SRC" fetch --depth=1 origin
-fi
+sync_latest_source() {
+  REPO="$1"; DEST="$2"
+  if [ ! -d "$DEST/.git" ]; then
+    gh repo clone "$REPO" "$DEST" -- --depth=1 \
+      || git clone --depth=1 "https://github.com/$REPO" "$DEST"
+  fi
+  DEFAULT_BRANCH="$(git -C "$DEST" remote show origin | sed -n '/HEAD branch/s/.*: //p')"
+  git -C "$DEST" fetch --depth=1 origin "$DEFAULT_BRANCH"
+  git -C "$DEST" checkout -B "$DEFAULT_BRANCH" FETCH_HEAD
+}
+sync_latest_source code-yeongyu/lazycodex /tmp/lazycodex-source
+sync_latest_source openai/codex /tmp/openai-codex-source
 ```
-
-If `gh` is unavailable, use `git clone --depth=1 https://github.com/openai/codex "$CODEX_SRC"`.
 4. Follow the debugging skill far enough to gather runtime evidence:
    - form at least three plausible hypotheses
    - run the smallest reproduction that exercises the real surface
    - confirm the root cause by observing the failing state
    - identify the minimal fix path or maintainer action
-5. Compare local LazyCodex evidence with `/tmp/openai-codex-source` before choosing the target repo. Cite exact files, commands, logs, or source paths that support the routing decision.
+5. Compare runtime evidence with both `/tmp/lazycodex-source` and `/tmp/openai-codex-source` before choosing the target repo. Cite exact files, commands, logs, or source paths that support the routing decision.
 6. Choose the target repo:
    - Use `code-yeongyu/lazycodex` when the bug is in LazyCodex integration, distribution, bundled plugin code, skills, hooks, MCP wiring, installer behavior, aliases, marketplace sync, docs, or any behavior that disappears in clean upstream Codex.
    - Use `openai/codex` when the bug reproduces in clean upstream Codex without LazyCodex, or the failing behavior comes from Codex CLI core, plugin API contracts, sandboxing, approvals, config loading, or built-in tool behavior.
@@ -106,7 +110,7 @@ Write the issue body in English and keep it direct:
 ## Repository Decision
 - Target repository:
 - Why this belongs there:
-- LazyCodex evidence:
+- LazyCodex evidence (runtime + `/tmp/lazycodex-source`):
 - Upstream Codex source evidence from `/tmp/openai-codex-source`:
 
 ## Reproduction
@@ -150,7 +154,7 @@ Use this when a PR is the right artifact:
 ## Repository Decision
 - Target repository:
 - Why this belongs there:
-- LazyCodex evidence:
+- LazyCodex evidence (runtime + `/tmp/lazycodex-source`):
 - Upstream Codex source evidence from `/tmp/openai-codex-source`:
 
 ## Root Cause
@@ -227,6 +231,6 @@ Do not file:
 - a vague issue without reproduction steps
 - an issue that claims a root cause not supported by runtime evidence
 - a duplicate when commenting on an existing issue is enough
-- an upstream Codex issue without checking `/tmp/openai-codex-source`
+- an issue without checking the latest `/tmp/lazycodex-source` and `/tmp/openai-codex-source` checkouts
 - a LazyCodex issue when the bug is proven to reproduce in clean upstream Codex
 - a fix PR without a concrete branch, implemented fix, and verification result

package/packages/shared-skills/skills/lsp-setup/SKILL.md

@@ -0,0 +1,139 @@
+---
+name: lsp-setup
+description: "Configure a Language Server (LSP) for a specific language so editor/agent tooling — diagnostics, go-to-definition, find-references, rename — works. Use when you need to: configure LSP, lsp setup, set up or install a language server, fix 'no LSP server configured' / 'server not installed', choose between servers (basedpyright vs pyright vs ty vs ruff), or wire .codex/lsp-client.json / .opencode/lsp.json. 언어서버 설정. Routes by file extension to references/<language>/README.md for the exact builtin server, per-OS install commands (macOS/Linux/Windows), config snippets for both config files, initialization options, alternatives, and troubleshooting. Ships scripts: detect-lsp.ts (scan a project for languages + each server's install/config status) and verify-lsp.ts (run a real diagnostics roundtrip). Covers typescript, python, go, rust, c/c++, java, kotlin, c#/razor, swift, ruby, php, dart, elixir, zig, lua, bash, yaml, terraform, haskell, julia."
+---
+
+# LSP Setup
+
+Configure the right Language Server for a project so the `lsp` MCP tools
+(`diagnostics`, `goto_definition`, `find_references`, `symbols`, `rename`)
+actually work. This skill is an index: detect what a project needs, install the
+server, write the config, then verify with a real roundtrip.
+
+The list of servers we ship as **builtin** is the source of truth in
+`packages/lsp-tools-mcp/src/lsp/server-definitions.ts` (`BUILTIN_SERVERS` +
+`LSP_INSTALL_HINTS`). The per-language references below mirror it.
+
+---
+
+## PHASE 0 — LANGUAGE GATE (run first)
+
+Identify the language from the file extension, then **read the matching
+reference before installing or configuring anything**.
+
+| Extension(s) | Reference |
+|---|---|
+| `.ts .tsx .js .jsx .mjs .cjs .mts .cts .vue .svelte .astro` | `references/typescript/README.md` |
+| `.py .pyi` | `references/python/README.md` |
+| `.go` | `references/go/README.md` |
+| `.rs` | `references/rust/README.md` |
+| `.c .cpp .cc .cxx .h .hpp .hh .hxx` | `references/c-cpp/README.md` |
+| `.java` | `references/java/README.md` |
+| `.kt .kts` | `references/kotlin/README.md` |
+| `.cs .razor .cshtml` | `references/csharp/README.md` |
+| `.swift` | `references/swift/README.md` |
+| `.rb .rake .gemspec .ru` | `references/ruby/README.md` |
+| `.php` | `references/php/README.md` |
+| `.dart` | `references/dart/README.md` |
+| `.ex .exs` | `references/elixir/README.md` |
+| `.zig .zon` | `references/zig/README.md` |
+| `.lua` | `references/lua/README.md` |
+| `.sh .bash .zsh .ksh` | `references/bash/README.md` |
+| `.yaml .yml` | `references/yaml/README.md` |
+| `.tf .tfvars` | `references/terraform/README.md` |
+| `.hs .lhs` | `references/haskell/README.md` |
+| `.jl` | `references/julia/README.md` |
+
+---
+
+## WORKFLOW — detect → install → configure → verify
+
+### 1. Detect
+
+Scan the project to see which languages are present and whether each server is
+installed and configured:
+
+```bash
+bun scripts/detect-lsp.ts <projectDir>      # human report (default: cwd)
+bun scripts/detect-lsp.ts <projectDir> --json
+```
+
+For each detected language it prints the builtin server id, the executable it
+needs on `PATH`, whether that executable is installed, an install hint, and
+whether a project config file already references it.
+
+### 2. Install
+
+Open `references/<language>/README.md` and run the install command for your OS.
+Then confirm the executable resolves:
+
+```bash
+command -v <server-executable>   # e.g. typescript-language-server, gopls, rust-analyzer
+```
+
+### 3. Configure
+
+Most builtin servers need **no config** — they are resolved automatically by
+file extension. Write config only to: pick between competing servers, set a
+`priority`, pass `initialization` options, override `extensions`, set `env`, or
+`disable` a server.
+
+Two project-scoped config files, **identical JSON shape**:
+
+- Codex harness → `.codex/lsp-client.json` (user: `~/.codex/lsp-client.json`)
+- OpenCode/omo harness → `.opencode/lsp.json` (also `.omo/lsp.json`)
+
+```jsonc
+{
+  "lsp": {
+    "<server-id>": {
+      "command": ["<bin>", "<args>"],   // optional for builtin ids (supplied automatically)
+      "extensions": [".ext"],            // optional override
+      "priority": 100,                    // higher wins when several servers match an extension
+      "initialization": { },              // server-specific initializationOptions
+      "env": { "KEY": "value" },          // optional
+      "disabled": false                   // set true to turn a server off
+    }
+  }
+}
+```
+
+Rules enforced by `config-loader.ts`:
+
+- In a **project** config (`.codex/lsp-client.json`, `.opencode/lsp.json`) an
+  entry whose id is a **builtin** server inherits `command` automatically — you
+  only override `extensions` / `priority` / `initialization`. A non-builtin id
+  in a project config is **ignored**.
+- To define a **fully custom** (non-builtin) server with its own `command`, put
+  it in the **user** config (`~/.codex/lsp-client.json`, or the path set by
+  `LSP_TOOLS_MCP_USER_CONFIG`), where `command` + `extensions` are honored.
+- Project entries win over user entries; both win over builtin defaults.
+
+Each language reference gives a ready-to-paste snippet.
+
+### 4. Verify
+
+Run a real diagnostics roundtrip against a source file. This spawns the server,
+opens the file, requests diagnostics, and reports `OK`/`FAIL`:
+
+```bash
+bun scripts/verify-lsp.ts <path/to/file.ext>
+bun scripts/verify-lsp.ts <file> --timeout=90000
+```
+
+`OK` = the server started and answered. `FAIL: language server not installed`
+= go back to step 2. Other `FAIL` text carries the server/startup error.
+`SKIP` = the engine source could not be located; run from inside the omo
+repo/worktree, or call the `lsp` MCP `diagnostics` tool directly.
+
+---
+
+## Scripts
+
+| Script | Purpose |
+|---|---|
+| `scripts/detect-lsp.ts` | Scan a directory; per detected language report server id, install status, install hint, config status. `--json` for machine output. |
+| `scripts/verify-lsp.ts` | Real LSP diagnostics roundtrip for one file via the `lsp-tools-mcp` engine; `OK`/`FAIL`/`SKIP` + exit code 0/1/3. |
+| `scripts/lsp-server-table.ts` | Embedded snapshot of the primary builtin server per language (mirrors `server-definitions.ts`). |
+
+Run with [Bun](https://bun.sh): `curl -fsSL https://bun.sh/install | bash`.

package/packages/shared-skills/skills/lsp-setup/references/bash/README.md

@@ -0,0 +1,60 @@
+# Bash — LSP setup
+
+- **Builtin server:** `bash` — `bash-language-server start`
+- **Extensions:** `.sh .bash .zsh .ksh`
+- **Install hint:** `npm install -g bash-language-server`
+
+An alias id `bash-ls` exists with the identical command; either id works.
+
+## Install
+
+- **macOS:** `npm install -g bash-language-server`
+- **Linux:** `npm install -g bash-language-server`
+- **Windows:** `npm install -g bash-language-server` (PowerShell)
+
+For real diagnostics, also install `shellcheck`:
+
+- **macOS:** `brew install shellcheck`
+- **Linux:** `apt install shellcheck` (or `dnf install ShellCheck`)
+- **Windows:** `scoop install shellcheck`
+
+Confirm it resolves:
+
+```bash
+command -v bash-language-server
+command -v shellcheck
+```
+
+## Configure
+
+Builtin — usually NO config needed (auto-resolved by extension). Configure only to set priority, init options, override extensions, or disable. Same JSON shape in `.codex/lsp-client.json` (Codex) AND `.opencode/lsp.json` (OpenCode/omo):
+
+```json
+{ "lsp": { "bash": { "priority": 100 } } }
+```
+
+For builtin ids in a PROJECT config, `command` is supplied automatically — only set `priority`/`initialization`/`extensions`/`disabled`/`env`. A fully custom (non-builtin) server with its own `command` must go in the USER config (`~/.codex/lsp-client.json`).
+
+### Initialization options (only if commonly needed)
+
+None commonly required. `bash-language-server` discovers `shellcheck` on PATH automatically. To point at a non-PATH binary, export `SHELLCHECK_PATH` via `env`:
+
+```json
+{ "lsp": { "bash": { "env": { "SHELLCHECK_PATH": "/opt/bin/shellcheck" } } } }
+```
+
+## Alternatives
+
+- `shellcheck` standalone as a linter-only flow (no LSP).
+- `shfmt` for formatting (complements, does not replace, the LSP).
+
+## Troubleshooting
+- **PATH:** `bash-language-server` on PATH; reopen shell after `npm -g` install.
+- **No diagnostics:** `shellcheck` missing — diagnostics are powered by it; install and reopen.
+- **Wrong shell dialect:** `.zsh`/`.ksh` are linted as bash; shellcheck may flag shell-specific syntax.
+
+## Verify
+
+```bash
+bun ../../scripts/verify-lsp.ts path/to/file.sh
+```