oh-my-opencode 4.19.1 → 4.19.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-opencode",
-  "version": "4.19.1",
+  "version": "4.19.3",
   "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",
@@ -209,21 +209,21 @@
     "bun-types": "1.3.14",
     "node-pty": "^1.1.0",
     "puppeteer-core": "^25.3.0",
-    "typescript": "^6.0.3"
+    "typescript": "^7.0.2"
   },
   "optionalDependencies": {
-    "oh-my-opencode-darwin-arm64": "4.19.1",
-    "oh-my-opencode-darwin-x64": "4.19.1",
-    "oh-my-opencode-darwin-x64-baseline": "4.19.1",
-    "oh-my-opencode-linux-arm64": "4.19.1",
-    "oh-my-opencode-linux-arm64-musl": "4.19.1",
-    "oh-my-opencode-linux-x64": "4.19.1",
-    "oh-my-opencode-linux-x64-baseline": "4.19.1",
-    "oh-my-opencode-linux-x64-musl": "4.19.1",
-    "oh-my-opencode-linux-x64-musl-baseline": "4.19.1",
-    "oh-my-opencode-windows-arm64": "4.19.1",
-    "oh-my-opencode-windows-x64": "4.19.1",
-    "oh-my-opencode-windows-x64-baseline": "4.19.1"
+    "oh-my-opencode-darwin-arm64": "4.19.3",
+    "oh-my-opencode-darwin-x64": "4.19.3",
+    "oh-my-opencode-darwin-x64-baseline": "4.19.3",
+    "oh-my-opencode-linux-arm64": "4.19.3",
+    "oh-my-opencode-linux-arm64-musl": "4.19.3",
+    "oh-my-opencode-linux-x64": "4.19.3",
+    "oh-my-opencode-linux-x64-baseline": "4.19.3",
+    "oh-my-opencode-linux-x64-musl": "4.19.3",
+    "oh-my-opencode-linux-x64-musl-baseline": "4.19.3",
+    "oh-my-opencode-windows-arm64": "4.19.3",
+    "oh-my-opencode-windows-x64": "4.19.3",
+    "oh-my-opencode-windows-x64-baseline": "4.19.3"
   },
   "overrides": {
     "@earendil-works/pi-agent-core": "0.80.3",

package/packages/lsp-core/src/lsp/client-diagnostics-freshness.integration.test.ts

@@ -51,21 +51,23 @@ describe("LspClient diagnostics freshness", () => {
 					{
 						trigger: "didChange",
 						diagnostics: [diagnostic("post-generation-versionless")],
+						awaitClientDelivery: true,
 					},
 				],
 			},
 			{ diagnosticsFreshnessTimeoutMs: 80, versionlessPublishQuiescenceMs: 20 },
 		);
 		await context.client.openFile(context.source);
-		await waitForEventCount(
+		const versionlessDelivery = waitForEventCount(
 			context.events,
-			(event) => event.type === "serverNotification" && event.method === "textDocument/publishDiagnostics",
+			(event) => event.type === "clientResponse" && event.method === "workspace/configuration",
 			1,
 		);
+		const startedAt = Date.now();
 		writeFileSync(context.source, "const after = 1;\n", "utf-8");
 		await context.client.openFile(context.source);
+		expect(await versionlessDelivery).toHaveLength(1);
 
-		const startedAt = Date.now();
 		const result = await context.client.diagnostics(context.source);
 		const elapsedMs = Date.now() - startedAt;
 
@@ -104,14 +106,21 @@ describe("LspClient diagnostics freshness", () => {
 						trigger: "didChange",
 						version: 1,
 						diagnostics: [diagnostic("stale")],
+						awaitClientDelivery: true,
 					},
 				],
 			},
-			{ diagnosticsFreshnessTimeoutMs: 30, versionlessPublishQuiescenceMs: 5 },
+			{ diagnosticsFreshnessTimeoutMs: 100, versionlessPublishQuiescenceMs: 5 },
 		);
 		await stale.client.openFile(stale.source);
+		const staleDelivery = waitForEventCount(
+			stale.events,
+			(event) => event.type === "clientResponse" && event.method === "workspace/configuration",
+			1,
+		);
 		writeFileSync(stale.source, "const stale = 1;\n", "utf-8");
 		await stale.client.openFile(stale.source);
+		expect(await staleDelivery).toHaveLength(1);
 
 		const staleResult = await stale.client.diagnostics(stale.source);
 
@@ -127,14 +136,21 @@ describe("LspClient diagnostics freshness", () => {
 						trigger: "didChange",
 						version: 3,
 						diagnostics: [diagnostic("future")],
+						awaitClientDelivery: true,
 					},
 				],
 			},
-			{ diagnosticsFreshnessTimeoutMs: 30, versionlessPublishQuiescenceMs: 5 },
+			{ diagnosticsFreshnessTimeoutMs: 100, versionlessPublishQuiescenceMs: 5 },
 		);
 		await future.client.openFile(future.source);
+		const futureDelivery = waitForEventCount(
+			future.events,
+			(event) => event.type === "clientResponse" && event.method === "workspace/configuration",
+			1,
+		);
 		writeFileSync(future.source, "const future = 1;\n", "utf-8");
 		await future.client.openFile(future.source);
+		expect(await futureDelivery).toHaveLength(1);
 
 		const futureResult = await future.client.diagnostics(future.source);
 

package/packages/lsp-core/src/lsp/fixtures/workspace-edit-server.mjs

@@ -51,6 +51,14 @@ function finishRename(context) {
 	}, delay);
 }
 
+function sendPublishDeliveryBarrier() {
+	const id = nextServerRequestId;
+	nextServerRequestId += 1;
+	pendingServerRequests.set(String(id), { kind: "publishDeliveryBarrier", method: "workspace/configuration" });
+	record({ type: "serverRequest", method: "workspace/configuration", id });
+	send({ jsonrpc: "2.0", id, method: "workspace/configuration", params: { items: [] } });
+}
+
 function schedulePublish(step, fallbackUri, fallbackVersion) {
 	const delay = Number(step.delayMs ?? 0);
 	setTimeout(() => {
@@ -61,6 +69,7 @@ function schedulePublish(step, fallbackUri, fallbackVersion) {
 		};
 		record({ type: "serverNotification", method: "textDocument/publishDiagnostics", params });
 		send({ jsonrpc: "2.0", method: "textDocument/publishDiagnostics", params });
+		if (step.awaitClientDelivery === true) sendPublishDeliveryBarrier();
 	}, delay);
 }
 
@@ -79,11 +88,11 @@ function handleClientResponse(message) {
 	pendingServerRequests.delete(String(message.id));
 	record({
 		type: "clientResponse",
-		method: "workspace/applyEdit",
+		method: context.method ?? "workspace/applyEdit",
 		result: message.result ?? null,
 		error: message.error ?? null,
 	});
-	if (context.kind === "unscoped") return;
+	if (context.kind === "publishDeliveryBarrier" || context.kind === "unscoped") return;
 	if (typeof context.pendingApplyResponses === "number") {
 		context.pendingApplyResponses -= 1;
 		if (context.pendingApplyResponses > 0) return;

package/packages/omo-codex/THIRD-PARTY-NOTICES.md

@@ -7,13 +7,13 @@ payloads, and component NOTICE files.
 
 ## Components
 
-### @colbymchenry/codegraph@1.0.1
+### @colbymchenry/codegraph@1.5.0
 - License: MIT, from npm package metadata and root third-party notice inspection.
 - Copyright: CodeGraph contributors.
 - Upstream URL: https://www.npmjs.com/package/@colbymchenry/codegraph
 - Where-bundled: optional runtime dependency of `plugin/components/codegraph`.
 
-### @colbymchenry/codegraph platform bundles@1.0.1
+### @colbymchenry/codegraph platform bundles@1.5.0
 - License: MIT for CodeGraph package metadata; bundled payloads also include a Node.js v24.16.0 runtime and vendored JavaScript/WASM dependencies. The Codex CodeGraph component ships the Node.js runtime license and bundled third-party license text at `plugin/components/codegraph/NODE-RUNTIME-LICENSES.md`.
 - Copyright: CodeGraph contributors, Node.js contributors, and bundled dependency contributors.
 - Upstream URL: https://www.npmjs.com/package/@colbymchenry/codegraph

package/packages/omo-codex/plugin/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "omo",
-  "version": "4.19.1",
+  "version": "4.19.3",
   "description": "One Codex plugin namespace for Yeongyu's local Codex components.",
   "author": {
     "name": "Yeongyu Kim",

package/packages/omo-codex/plugin/components/bootstrap/dist/cli.js

@@ -737,7 +737,11 @@ function defaultFileExists(filePath) {
   }
 }
 function defaultVersionProbe(binaryPath) {
-  return String(execFileSync(binaryPath, ["--version"], { encoding: "utf8", timeout: 5000 }));
+  return String(execFileSync(binaryPath, ["--version"], {
+    encoding: "utf8",
+    stdio: ["ignore", "pipe", "ignore"],
+    timeout: 5000
+  }));
 }
 function isAstGrepVersionOutput(output) {
   return output.toLowerCase().includes("ast-grep");
@@ -751,8 +755,8 @@ function hasAstGrepVersion(binaryPath, runVersionProbeSync) {
     return false;
   }
 }
-function pathCommandCandidates(platform) {
-  return platform === "linux" ? ["ast-grep", "sg"] : ["sg", "ast-grep"];
+function pathCommandCandidates() {
+  return ["ast-grep", "sg"];
 }
 function findSgBinarySync(options = {}) {
   const env = options.env ?? process.env;
@@ -769,7 +773,7 @@ function findSgBinarySync(options = {}) {
       if (fileExists(runtimeCandidate))
         return runtimeCandidate;
     }
-    for (const commandName of pathCommandCandidates(platform)) {
+    for (const commandName of pathCommandCandidates()) {
       const pathCandidate = which(commandName);
       if (pathCandidate === null || !fileExists(pathCandidate))
         continue;

package/packages/omo-codex/plugin/components/bootstrap/hooks/hooks.json

@@ -8,7 +8,7 @@
 						"command": "node \"${PLUGIN_ROOT}/components/bootstrap/dist/cli.js\" hook session-start",
 						"commandWindows": "powershell -NoProfile -ExecutionPolicy Bypass -File \"${PLUGIN_ROOT}\\components\\bootstrap\\scripts\\bootstrap.ps1\"",
 						"timeout": 30,
-						"statusMessage": "(OmO 4.19.1) Checking Bootstrap Provisioning"
+						"statusMessage": "(OmO 4.19.3) Checking Bootstrap Provisioning"
 					}
 				]
 			}

package/packages/omo-codex/plugin/components/bootstrap/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@sisyphuslabs/codex-bootstrap",
-	"version": "4.19.1",
+	"version": "4.19.3",
 	"description": "Codex SessionStart bootstrap component that provisions LazyCodex runtime dependencies from a detached worker.",
 	"type": "module",
 	"private": true,

package/packages/omo-codex/plugin/components/codegraph/AGENTS.md

@@ -4,7 +4,7 @@
 
 ## OVERVIEW
 
-`@sisyphuslabs/codex-codegraph` (private, Node >=20, bin `omo-codegraph`). Wraps the external CodeGraph CLI (`@colbymchenry/codegraph`, pinned `1.0.1` as optionalDependency + `CODEGRAPH_VERSION` constant) behind a stdio MCP server plus Codex lifecycle hooks. Two committed dists ship in the published payload; rebuild `dist/` in the same change as `src/` edits.
+`@sisyphuslabs/codex-codegraph` (private, Node >=20, bin `omo-codegraph`). Wraps the external CodeGraph CLI (`@colbymchenry/codegraph`, pinned `1.5.0` through the shared `CODEGRAPH_PINNED_VERSION` and the component optionalDependency) behind a stdio MCP server plus Codex lifecycle hooks. Two committed dists ship in the published payload; rebuild `dist/` in the same change as `src/` edits.
 
 Codex wiring (all repo-relative to `packages/omo-codex/plugin/`):
 - **MCP:** `.mcp.json` server `codegraph` → `node components/codegraph/dist/serve.js` (`required: false`).
@@ -14,7 +14,7 @@ Codex wiring (all repo-relative to `packages/omo-codex/plugin/`):
 
 Serve pipeline (`src/serve.ts`): OMO SOT config gate (`[codex].codegraph.enabled=false` → unavailable stub) → resolve binary (`OMO_CODEGRAPH_BIN` env → bundled → provisioned `~/.omo/codegraph/bin` → PATH) → Node support gate (major 20-24; >=25 crashes CodeGraph mid-indexing; `CODEGRAPH_ALLOW_UNSAFE_NODE=1` overrides) → auto-provision into `~/.omo/codegraph` unless `codegraph.auto_provision=false` → bridge JSON-RPC to `codegraph serve --mcp`. Project cwd resolved from `OMO_CODEGRAPH_PROJECT_CWD` → `OMO_CODEGRAPH_SESSION_START_CWD` → `PWD` → wrapper cwd. When the binary is missing/disabled, `mcp-unavailable.ts` still answers `initialize`/`tools/*` with the skip reason so Codex startup never fails.
 
-Hook pipeline (`src/hook.ts` + `src/session-start-worker.ts`): `hook session-start` probes `status --json` (2s timeout); uninitialized project → spawns a detached `hook session-start-worker` (prepare workspace + gitignore → `status --json` → `init` or `sync`, 60s per command) and emits a SessionStart `additionalContext` notice. Worker outcomes append to `~/.omo/codegraph/session-start.jsonl`. `hook post-tool-use` emits init guidance when a codegraph tool result indicates an uninitialized project. `cli.js` with no hook subcommand falls through to serve.
+Hook pipeline (`src/hook.ts` + `src/session-start-*.ts`): `hook session-start` checks only the exact filesystem marker `<projectRoot>/.codegraph/codegraph.db`; it never invokes the upstream CLI for status. An exact database skips as initialized, an ancestor database records `skipped-nested-root`, and only a definitive no-database result may acquire the per-project atomic lock and spawn `hook session-start-worker`. Live locks record `skipped-locked`; stale locks recover into an exponential cooldown instead of spawning immediately. Worker failures and skips persist a cooldown stamp (15 minutes by default, doubling to a 24-hour cap; configurable with Codex-only `codegraph.session_start_cooldown_ms`, minimum 60000), while success clears it. The worker prepares the workspace, invokes only `codegraph init` with a 60-second tree timeout, and records success only after the exact project database appears. Hook suppressions and worker outcomes append to `~/.omo/codegraph/session-start.jsonl`. `hook post-tool-use` emits init guidance when a codegraph tool result indicates an uninitialized project. `cli.js` with no hook subcommand falls through to serve.
 
 ## includeCode CONTRACT (commit 4cf383c5b)
 
@@ -26,8 +26,14 @@ Hook pipeline (`src/hook.ts` + `src/session-start-worker.ts`): `hook session-sta
 |------|---------|
 | `src/serve.ts` | MCP entry: config/resolution/Node gates, provisioning, bridge or unavailable stub |
 | `src/cli.ts` | Hook CLI router: `hook session-start` / `hook post-tool-use` / `hook session-start-worker` / serve fallback |
-| `src/hook.ts` | SessionStart probe + detached worker spawn, PostToolUse guidance emission |
-| `src/session-start-worker.ts` | Background bootstrap: provision → workspace prep → `init`/`sync`, jsonl outcome log |
+| `src/hook.ts` | SessionStart exact-root decision pipeline + detached worker spawn |
+| `src/post-tool-use-hook.ts` | PostToolUse init guidance emission |
+| `src/session-start-project.ts` | Exact database and ancestor-coverage filesystem probes |
+| `src/session-start-lock.ts` | Atomic per-project worker lock + stale recovery |
+| `src/session-start-cooldown.ts` | Persistent exponential failure cooldown state |
+| `src/session-start-worker.ts` | Background bootstrap orchestration and lock lifecycle |
+| `src/session-start-command.ts` | Provision/resolve + bounded `codegraph init` invocation |
+| `src/session-start-outcome.ts` | `session-start.jsonl` outcome writer |
 | `src/mcp-bridge.ts` | Stdio JSON-RPC forwarder; per-request framed/line response-mode tracking; codegraph_node contract rewrites; awaits stdio writes and SIGKILLs the child on response-forwarding failure |
 | `src/mcp-unavailable.ts` | Reason-bearing stub MCP server for disabled/missing binary |
 | `src/serve-invocation.ts` | win32 invocation shim: `.cmd`/`.bat` via `cmd.exe /d /s /c`, `.js`/`.mjs`/`.cjs` via `process.execPath` (mirrored by `resolveCodegraphCommandInvocation` in session-start-worker) |
@@ -46,7 +52,7 @@ Hook pipeline (`src/hook.ts` + `src/session-start-worker.ts`): `hook session-sta
 
 - This component tests with `bun test` (unlike the vitest-based `lsp` sibling); given/when/then style.
 - `src/` imports reach sibling packages via relative paths (`../../../../../utils/src/...`); they are inlined by `bun build`, so runtime `dist/` has zero deps beyond Node.
-- `CODEGRAPH_VERSION` is duplicated in `serve.ts` and `session-start-worker.ts` and must match the `@colbymchenry/codegraph` optionalDependency version.
+- `serve.ts` and SessionStart command provisioning derive their version from the shared `CODEGRAPH_PINNED_VERSION`; the component optionalDependency must match that pin.
 - `trustedCodegraphInstallDir` overrides the `~/.omo/codegraph` install dir and is forwarded to children as `CODEGRAPH_INSTALL_DIR`.
 - `resolution.source === "env"` is never auto-provisioned over: a user-set `OMO_CODEGRAPH_BIN` pointing at a missing file skips the MCP instead of silently substituting a download.
 - `runBridgedCodegraphProcess` awaits every stdio write and races `childExit` against response forwarding; a forwarding error (e.g. parent stdout `EPIPE`) rejects the serve promise, destroys the child pipes, and `SIGKILL`s the codegraph child if still alive so a held-open child is never orphaned. Pinned by `test/serve-mcp-bridge-lifecycle.test.ts` and the held-open case in `test/serve-mcp-bridge.test.ts`.

package/packages/omo-codex/plugin/components/codegraph/NOTICE

@@ -1,6 +1,6 @@
 codex-codegraph adapts the CodeGraph MCP runtime for the Codex plugin surface.
 
-This component optionally installs @colbymchenry/codegraph@1.0.1. That package
+This component optionally installs @colbymchenry/codegraph@1.5.0. That package
 selects platform bundles for darwin, linux, and win32 on arm64 and x64. The
 CodeGraph packages declare MIT license metadata.