oh-my-opencode 5.1.2 → 5.1.4

package/packages/lsp-core/src/missing-dependency-result.ts

@@ -1,4 +1,5 @@
 import { LspServerLookupError } from "./lsp/errors.js";
+import { type InstallDecision, loadInstallDecision } from "./lsp/server-install-state.js";
 import { handleMissingDependencyError } from "./lsp/startup-failure.js";
 import { lspRequestContext } from "./request-context.js";
 import type { ToolExecutionResult } from "./tools.js";
@@ -20,6 +21,7 @@ export type MissingDependencyAvailability =
 			readonly installHint: string;
 			readonly installDecisionTool: boolean;
 			readonly installDecisionsPath: string;
+			readonly decision: InstallDecision | null;
 	  };
 
 export function missingDependencyResult<TDetails extends object>(
@@ -67,6 +69,7 @@ function missingDependencyAvailability(error: unknown): MissingDependencyAvailab
 				installHint: error.lookup.installHint,
 				installDecisionTool: context.capabilities.installDecisionTool,
 				installDecisionsPath: context.installDecisionsPath,
+				decision: loadInstallDecision(error.lookup.server.id)?.decision ?? null,
 			};
 		default: {
 			const exhaustive: never = error.lookup;

package/packages/lsp-core/src/post-edit/file-location.ts

@@ -0,0 +1,50 @@
+import { tmpdir } from "node:os";
+import { dirname, resolve } from "node:path";
+
+import { nearestMarkedAncestor } from "../lsp/outside-context-workspace.js";
+import { canonicalizeExistingOrNearestAncestor, isPathInside } from "../request-context.js";
+
+export type PostEditFileLocation = "project" | "outside_project" | "agent_config" | "temp";
+
+export interface PostEditFileLocationOptions {
+	readonly cwd: string;
+	readonly agentDirs: readonly string[];
+	/** Temp roots; defaults to the canonical OS temp dir (plus `/tmp` off Windows). */
+	readonly tempDirs?: readonly string[];
+}
+
+/**
+ * Classifies an edited file for install guidance.
+ *
+ * A project root is the nearest ancestor carrying one of lsp-core's `WORKSPACE_MARKERS` (the marker set the
+ * out-of-cwd workspace resolver uses). Inside a temp dir, only a marked root strictly below the temp root
+ * counts, so a stray `package.json` left in the temp root does not turn every scratch file into a project.
+ * Paths are canonicalized first because macOS `/tmp` and `/var` are symlinks.
+ */
+export function classifyPostEditFileLocation(
+	filePath: string,
+	options: PostEditFileLocationOptions,
+): PostEditFileLocation {
+	const file = canonicalizeExistingOrNearestAncestor(resolve(options.cwd, filePath));
+	if (canonicalDirectories(options.agentDirs).some((directory) => isPathInside(directory, file))) {
+		return "agent_config";
+	}
+
+	const projectRoot = nearestMarkedAncestor(dirname(file));
+	const tempRoot = canonicalDirectories(options.tempDirs ?? defaultTempDirectories()).find((directory) =>
+		isPathInside(directory, file),
+	);
+	if (tempRoot !== undefined) {
+		const ownProject = projectRoot !== undefined && projectRoot !== tempRoot && isPathInside(tempRoot, projectRoot);
+		return ownProject ? "project" : "temp";
+	}
+	return projectRoot === undefined ? "outside_project" : "project";
+}
+
+function defaultTempDirectories(): readonly string[] {
+	return process.platform === "win32" ? [tmpdir()] : [tmpdir(), "/tmp"];
+}
+
+function canonicalDirectories(directories: readonly string[]): readonly string[] {
+	return directories.map((directory) => canonicalizeExistingOrNearestAncestor(directory));
+}

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

@@ -1 +1,3 @@
+export * from "./file-location.js";
+export * from "./not-installed-guidance.js";
 export * from "./orchestration.js";

package/packages/lsp-core/src/post-edit/not-installed-guidance.ts

@@ -0,0 +1,36 @@
+import type { PostEditFileLocation } from "./file-location.js";
+
+export interface PostEditNotInstalledOutcome {
+	readonly kind: "not_installed";
+	readonly serverId: string;
+	readonly installDecisionTool: boolean;
+	/** A recorded install decision; absent while the user has not decided. */
+	readonly decision?: "declined" | "allowed";
+	readonly text: string;
+}
+
+export interface NotInstalledGuidanceState {
+	readonly notInstalledServers: Set<string>;
+}
+
+/**
+ * Decides whether a missing-server nudge reaches the model.
+ *
+ * Recorded decisions keep their own text. Undecided nudges are skipped for files outside a project
+ * (no marked root, the agent's config dir, or a temp dir) because a repo-local install there would
+ * create a manifest in HOME or a temp dir. Without a decision tool nothing can be recorded, so a
+ * server is nudged at most once per session state.
+ */
+export function notInstalledGuidance(
+	filePath: string,
+	outcome: PostEditNotInstalledOutcome,
+	state: NotInstalledGuidanceState,
+	locateFile: ((filePath: string) => PostEditFileLocation) | undefined,
+): string | undefined {
+	if (outcome.decision !== undefined) return outcome.text;
+	if (locateFile !== undefined && locateFile(filePath) !== "project") return undefined;
+	if (outcome.installDecisionTool) return outcome.text;
+	if (state.notInstalledServers.has(outcome.serverId)) return undefined;
+	state.notInstalledServers.add(outcome.serverId);
+	return outcome.text;
+}

package/packages/lsp-core/src/post-edit/not-installed.test.ts

@@ -0,0 +1,199 @@
+import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { afterEach, describe, expect, it } from "bun:test";
+
+import {
+	classifyPostEditFileLocation,
+	collectPostEditDiagnostics,
+	createPostEditNotConfiguredCache,
+	type PostEditDiagnosticsOutcome,
+	type PostEditFileLocation,
+	resetPostEditNotConfiguredCache,
+} from "./index.js";
+
+const tempDirectories: string[] = [];
+
+afterEach(() => {
+	for (const directory of tempDirectories.splice(0)) {
+		rmSync(directory, { recursive: true, force: true });
+	}
+});
+
+function tempRoot(prefix: string): string {
+	const root = mkdtempSync(join(tmpdir(), prefix));
+	tempDirectories.push(root);
+	return root;
+}
+
+function touch(path: string, text = ""): string {
+	mkdirSync(join(path, ".."), { recursive: true });
+	writeFileSync(path, text);
+	return path;
+}
+
+const FULL_NUDGE = [
+	"LSP server 'typescript' for .ts is NOT INSTALLED.",
+	"To install in THIS repository (preferred — no global install needed):",
+	"ACTION REQUIRED — ASK THE USER whether to install this LSP server.",
+].join("\n");
+
+function notInstalled(
+	serverId: string,
+	overrides: Partial<Extract<PostEditDiagnosticsOutcome, { kind: "not_installed" }>> = {},
+): PostEditDiagnosticsOutcome {
+	return {
+		kind: "not_installed",
+		serverId,
+		installDecisionTool: false,
+		text: `${FULL_NUDGE} (${serverId})`,
+		...overrides,
+	};
+}
+
+function at(location: PostEditFileLocation): () => PostEditFileLocation {
+	return () => location;
+}
+
+describe("classifyPostEditFileLocation", () => {
+	it("#given a file with no workspace marker above it #when classified #then it is outside any project", () => {
+		const root = tempRoot("lsp-postedit-noproject-");
+		const file = touch(join(root, "scratch", "note.mjs"));
+
+		expect(classifyPostEditFileLocation(file, { cwd: root, agentDirs: [], tempDirs: [] })).toBe("outside_project");
+	});
+
+	it("#given a file under the agent config dir inside a git repository #when classified #then the agent config dir wins", () => {
+		const root = tempRoot("lsp-postedit-agentdir-");
+		mkdirSync(join(root, ".git"));
+		const agentDir = join(root, ".omo", "agent");
+		const file = touch(join(agentDir, "models.json"), "{}\n");
+
+		expect(classifyPostEditFileLocation(file, { cwd: root, agentDirs: [agentDir], tempDirs: [] })).toBe(
+			"agent_config",
+		);
+	});
+
+	it("#given a temp dir holding a stray package.json #when a scratch file there is classified #then it is a temp file", () => {
+		const temp = tempRoot("lsp-postedit-temp-");
+		touch(join(temp, "package.json"), "{}\n");
+		const file = touch(join(temp, "rules_test.mjs"));
+
+		expect(classifyPostEditFileLocation(file, { cwd: temp, agentDirs: [], tempDirs: [temp] })).toBe("temp");
+	});
+
+	it("#given the default temp dirs #when an OS temp file is classified #then the canonical temp dir matches", () => {
+		const root = tempRoot("lsp-postedit-ostemp-");
+		const file = touch(join(root, "scratch.mjs"));
+
+		expect(classifyPostEditFileLocation(file, { cwd: root, agentDirs: [] })).toBe("temp");
+		if (process.platform !== "win32") {
+			const slashTmp = join("/tmp", `lsp-postedit-missing-${process.pid}`, "rules_test.mjs");
+			expect(classifyPostEditFileLocation(slashTmp, { cwd: root, agentDirs: [] })).toBe("temp");
+		}
+	});
+
+	it("#given a project with its own marker inside a temp dir #when classified #then it is a project", () => {
+		const temp = tempRoot("lsp-postedit-tempproject-");
+		const project = join(temp, "checkout");
+		touch(join(project, "package.json"), "{}\n");
+		const file = touch(join(project, "src", "a.ts"));
+
+		expect(classifyPostEditFileLocation(file, { cwd: project, agentDirs: [], tempDirs: [temp] })).toBe("project");
+		expect(classifyPostEditFileLocation("src/a.ts", { cwd: project, agentDirs: [], tempDirs: [temp] })).toBe(
+			"project",
+		);
+	});
+});
+
+describe("collectPostEditDiagnostics not-installed guidance", () => {
+	for (const location of ["outside_project", "agent_config", "temp"] as const) {
+		it(`#given a missing server for a ${location} file #when diagnostics run #then the install nudge is skipped`, async () => {
+			const result = await collectPostEditDiagnostics({
+				filePaths: ["file.ts"],
+				runDiagnostics: async () => notInstalled("typescript"),
+				locateFile: at(location),
+			});
+
+			expect(result.blocks).toEqual([]);
+			expect(result.observations).toEqual([{ filePath: "file.ts", kind: "not_installed" }]);
+		});
+	}
+
+	it("#given a missing server for a project file #when diagnostics run #then today's full guidance is kept", async () => {
+		const result = await collectPostEditDiagnostics({
+			filePaths: ["src/a.ts"],
+			runDiagnostics: async () => notInstalled("typescript"),
+			locateFile: at("project"),
+		});
+
+		expect(result.blocks).toEqual([{ filePath: "src/a.ts", diagnostics: `${FULL_NUDGE} (typescript)` }]);
+	});
+
+	it("#given no decision tool #when repeated edits hit missing servers #then each server is nudged once per session", async () => {
+		const cache = createPostEditNotConfiguredCache();
+		const serverFor = (filePath: string): string => (filePath.endsWith(".json") ? "biome" : "typescript");
+		const run = (filePaths: readonly string[]) =>
+			collectPostEditDiagnostics({
+				filePaths,
+				cache,
+				runDiagnostics: async (filePath) => notInstalled(serverFor(filePath)),
+				locateFile: at("project"),
+			});
+
+		const first = await run(["a.ts", "b.ts"]);
+		const second = await run(["c.ts", "d.json"]);
+		const third = await run(["e.json", "f.ts"]);
+
+		expect(first.blocks.map((block) => block.filePath)).toEqual(["a.ts"]);
+		expect(second.blocks.map((block) => block.filePath)).toEqual(["d.json"]);
+		expect(third.blocks).toEqual([]);
+
+		resetPostEditNotConfiguredCache(cache);
+		const afterReset = await run(["g.ts"]);
+		expect(afterReset.blocks.map((block) => block.filePath)).toEqual(["g.ts"]);
+	});
+
+	it("#given a decision tool #when repeated edits hit a missing server #then every edit keeps the guidance", async () => {
+		const cache = createPostEditNotConfiguredCache();
+		const run = (filePath: string) =>
+			collectPostEditDiagnostics({
+				filePaths: [filePath],
+				cache,
+				runDiagnostics: async () => notInstalled("typescript", { installDecisionTool: true }),
+				locateFile: at("project"),
+			});
+
+		expect((await run("a.ts")).blocks).toHaveLength(1);
+		expect((await run("b.ts")).blocks).toHaveLength(1);
+	});
+
+	for (const decision of ["declined", "allowed"] as const) {
+		it(`#given a recorded ${decision} decision #when edits repeat anywhere #then that branch's text is unchanged every time`, async () => {
+			const cache = createPostEditNotConfiguredCache();
+			const text = `LSP server 'typescript' ${decision} branch text`;
+			const outcomes: PostEditFileLocation[] = ["project", "outside_project", "project"];
+			const blocks: string[] = [];
+			for (const location of outcomes) {
+				const result = await collectPostEditDiagnostics({
+					filePaths: ["a.ts"],
+					cache,
+					runDiagnostics: async () => notInstalled("typescript", { decision, text }),
+					locateFile: at(location),
+				});
+				blocks.push(...result.blocks.map((block) => block.diagnostics));
+			}
+
+			expect(blocks).toEqual([text, text, text]);
+		});
+	}
+
+	it("#given no file locator #when a missing server is reported #then the full guidance is kept", async () => {
+		const result = await collectPostEditDiagnostics({
+			filePaths: ["a.ts"],
+			runDiagnostics: async () => notInstalled("typescript"),
+		});
+
+		expect(result.blocks).toEqual([{ filePath: "a.ts", diagnostics: `${FULL_NUDGE} (typescript)` }]);
+	});
+});

package/packages/lsp-core/src/post-edit/orchestration.ts

@@ -1,5 +1,8 @@
 import { extname } from "node:path";
 
+import type { PostEditFileLocation } from "./file-location.js";
+import { notInstalledGuidance, type PostEditNotInstalledOutcome } from "./not-installed-guidance.js";
+
 const DEFAULT_MAX_CONCURRENCY = 4;
 const CLEAN_DIAGNOSTICS_TEXT = "No diagnostics found";
 
@@ -8,7 +11,8 @@ export type PostEditDiagnosticsOutcome =
 	| {
 			readonly kind: "not_configured";
 			readonly extension: string;
-	  };
+	  }
+	| PostEditNotInstalledOutcome;
 
 export type DiagnosticsRunner = (filePath: string) => Promise<PostEditDiagnosticsOutcome>;
 
@@ -19,7 +23,7 @@ export interface PostEditDiagnosticsBlock {
 
 export interface PostEditDiagnosticsObservation {
 	readonly filePath: string;
-	readonly kind: "block" | "clean" | "not_configured";
+	readonly kind: "block" | "clean" | "not_configured" | "not_installed";
 }
 
 export interface PostEditDiagnosticsResult {
@@ -29,6 +33,7 @@ export interface PostEditDiagnosticsResult {
 
 export interface PostEditNotConfiguredCache {
 	readonly notConfiguredExtensions: Set<string>;
+	readonly notInstalledServers: Set<string>;
 }
 
 export interface CollectPostEditDiagnosticsInput {
@@ -36,6 +41,7 @@ export interface CollectPostEditDiagnosticsInput {
 	readonly runDiagnostics: DiagnosticsRunner;
 	readonly cache?: PostEditNotConfiguredCache;
 	readonly maxConcurrency?: number;
+	readonly locateFile?: (filePath: string) => PostEditFileLocation;
 }
 
 interface IndexedFilePath {
@@ -50,11 +56,12 @@ interface IndexedDiagnostics {
 }
 
 export function createPostEditNotConfiguredCache(): PostEditNotConfiguredCache {
-	return { notConfiguredExtensions: new Set() };
+	return { notConfiguredExtensions: new Set(), notInstalledServers: new Set() };
 }
 
 export function resetPostEditNotConfiguredCache(cache: PostEditNotConfiguredCache): void {
 	cache.notConfiguredExtensions.clear();
+	cache.notInstalledServers.clear();
 }
 
 export async function collectPostEditDiagnostics(
@@ -67,8 +74,15 @@ export async function collectPostEditDiagnostics(
 	const observations: PostEditDiagnosticsObservation[] = [];
 
 	for (const result of results) {
-		const classification = classifyDiagnostics(result.outcome);
+		const outcome = result.outcome;
+		const classification =
+			typeof outcome !== "string" && outcome.kind === "not_installed"
+				? classifyNotInstalled(notInstalledGuidance(result.filePath, outcome, cache, input.locateFile))
+				: classifyDiagnostics(outcome);
 		switch (classification.kind) {
+			case "not_installed":
+				observations.push({ filePath: result.filePath, kind: "not_installed" });
+				break;
 			case "clean":
 				observations.push({ filePath: result.filePath, kind: "clean" });
 				break;
@@ -146,9 +160,16 @@ async function collectFileDiagnostics(
 
 function normalizeDiagnosticsOutcome(outcome: PostEditDiagnosticsOutcome): PostEditDiagnosticsOutcome {
 	if (typeof outcome === "string") return normalizeDiagnosticsText(outcome);
+	if (outcome.kind === "not_installed") return { ...outcome, text: normalizeDiagnosticsText(outcome.text) };
 	return outcome;
 }
 
+function classifyNotInstalled(
+	guidance: string | undefined,
+): { readonly kind: "not_installed" } | { readonly kind: "block"; readonly diagnostics: string } {
+	return guidance === undefined ? { kind: "not_installed" } : { kind: "block", diagnostics: guidance };
+}
+
 function normalizeDiagnosticsText(text: string): string {
 	return text.trim();
 }
@@ -159,7 +180,7 @@ function formatDiagnosticsError(error: Error): string {
 }
 
 function classifyDiagnostics(
-	outcome: PostEditDiagnosticsOutcome,
+	outcome: Exclude<PostEditDiagnosticsOutcome, PostEditNotInstalledOutcome>,
 ):
 	| { readonly kind: "clean" }
 	| { readonly kind: "not_configured"; readonly extension: string }

package/packages/lsp-daemon/dist/cli.js

@@ -5350,7 +5350,8 @@ function missingDependencyAvailability(error) {
         extensions: [...error.lookup.server.extensions],
         installHint: error.lookup.installHint,
         installDecisionTool: context.capabilities.installDecisionTool,
-        installDecisionsPath: context.installDecisionsPath
+        installDecisionsPath: context.installDecisionsPath,
+        decision: loadInstallDecision(error.lookup.server.id)?.decision ?? null
       };
     default: {
       const exhaustive = error.lookup;

package/packages/lsp-daemon/dist/client.js

@@ -5586,7 +5586,8 @@ function missingDependencyAvailability(error) {
         extensions: [...error.lookup.server.extensions],
         installHint: error.lookup.installHint,
         installDecisionTool: context.capabilities.installDecisionTool,
-        installDecisionsPath: context.installDecisionsPath
+        installDecisionsPath: context.installDecisionsPath,
+        decision: loadInstallDecision(error.lookup.server.id)?.decision ?? null
       };
     default: {
       const exhaustive = error.lookup;

package/packages/lsp-daemon/dist/index.js

@@ -5581,7 +5581,8 @@ function missingDependencyAvailability(error) {
         extensions: [...error.lookup.server.extensions],
         installHint: error.lookup.installHint,
         installDecisionTool: context.capabilities.installDecisionTool,
-        installDecisionsPath: context.installDecisionsPath
+        installDecisionsPath: context.installDecisionsPath,
+        decision: loadInstallDecision(error.lookup.server.id)?.decision ?? null
       };
     default: {
       const exhaustive = error.lookup;

package/packages/lsp-tools-mcp/dist/cli.js

@@ -4919,7 +4919,8 @@ function missingDependencyAvailability(error) {
         extensions: [...error.lookup.server.extensions],
         installHint: error.lookup.installHint,
         installDecisionTool: context.capabilities.installDecisionTool,
-        installDecisionsPath: context.installDecisionsPath
+        installDecisionsPath: context.installDecisionsPath,
+        decision: loadInstallDecision(error.lookup.server.id)?.decision ?? null
       };
     default: {
       const exhaustive = error.lookup;

package/packages/lsp-tools-mcp/dist/mcp.js

@@ -4915,7 +4915,8 @@ function missingDependencyAvailability(error) {
         extensions: [...error.lookup.server.extensions],
         installHint: error.lookup.installHint,
         installDecisionTool: context.capabilities.installDecisionTool,
-        installDecisionsPath: context.installDecisionsPath
+        installDecisionsPath: context.installDecisionsPath,
+        decision: loadInstallDecision(error.lookup.server.id)?.decision ?? null
       };
     default: {
       const exhaustive = error.lookup;

package/packages/lsp-tools-mcp/dist/tools.js

@@ -4603,7 +4603,8 @@ function missingDependencyAvailability(error) {
         extensions: [...error.lookup.server.extensions],
         installHint: error.lookup.installHint,
         installDecisionTool: context.capabilities.installDecisionTool,
-        installDecisionsPath: context.installDecisionsPath
+        installDecisionsPath: context.installDecisionsPath,
+        decision: loadInstallDecision(error.lookup.server.id)?.decision ?? null
       };
     default: {
       const exhaustive = error.lookup;

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

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

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 5.1.2) Checking Bootstrap Provisioning"
+						"statusMessage": "(OmO 5.1.4) Checking Bootstrap Provisioning"
 					}
 				]
 			}

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

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

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

@@ -146,6 +146,34 @@ function joinPatchLines(lines) {
 `)}
 `;
 }
+// ../../comment-checker-core/src/package-binary.ts
+import { createRequire } from "node:module";
+import { dirname, join } from "node:path";
+var COMMENT_CHECKER_PACKAGE_NAME = "@code-yeongyu/comment-checker";
+function isModuleResolutionMiss(error) {
+  return error instanceof Error || typeof error === "object" && error !== null && "name" in error && error.name === "ResolveMessage";
+}
+function findCommentCheckerPackageBinary(input) {
+  const packageName = input.packageName ?? COMMENT_CHECKER_PACKAGE_NAME;
+  const platformKey = `${input.platform ?? process.platform}-${input.arch ?? process.arch}`;
+  const packageDir = dirname(input.packageJsonPath);
+  const candidates = [
+    resolvePlatformPackageBinary(input.packageJsonPath, `${packageName}-${platformKey}`, input.binaryName),
+    join(packageDir, "vendor", platformKey, input.binaryName),
+    join(packageDir, "bin", input.binaryName)
+  ];
+  return candidates.find((candidate) => candidate !== null && input.existsSync(candidate)) ?? null;
+}
+function resolvePlatformPackageBinary(packageJsonPath, platformPackageName, binaryName) {
+  try {
+    const manifestPath = createRequire(packageJsonPath).resolve(`${platformPackageName}/package.json`);
+    return join(dirname(manifestPath), "bin", binaryName);
+  } catch (error) {
+    if (isModuleResolutionMiss(error))
+      return null;
+    throw error;
+  }
+}
 // components/comment-checker/src/hook-input.ts
 function toHookInput(request, context) {
   return {
@@ -302,8 +330,7 @@ function getContentText(content) {
 // components/comment-checker/src/runner.ts
 import { spawn } from "node:child_process";
 import { existsSync } from "node:fs";
-import { createRequire } from "node:module";
-import { dirname, join } from "node:path";
+import { createRequire as createRequire2 } from "node:module";
 var MAX_PROCESS_OUTPUT_BYTES = 64 * 1024;
 async function runCommentChecker2(input, options = {}) {
   const binaryPath = options.binaryPath ?? (options.resolveBinary ? options.resolveBinary() : resolveCommentCheckerBinary2());
@@ -361,7 +388,7 @@ function resolveCommentCheckerBinary2() {
 }
 function resolvePackageApiBinary() {
   try {
-    const require2 = createRequire(import.meta.url);
+    const require2 = createRequire2(import.meta.url);
     const packageExports = require2(commentCheckerPackageName());
     if (!isCommentCheckerPackage(packageExports))
       return;
@@ -373,10 +400,9 @@ function resolvePackageApiBinary() {
 }
 function resolvePackageBinary(binaryName) {
   try {
-    const require2 = createRequire(import.meta.url);
-    const packagePath = require2.resolve(`${commentCheckerPackageName()}/package.json`);
-    const binaryPath = join(dirname(packagePath), "bin", binaryName);
-    return existsSync(binaryPath) ? binaryPath : undefined;
+    const require2 = createRequire2(import.meta.url);
+    const packageJsonPath = require2.resolve(`${commentCheckerPackageName()}/package.json`);
+    return findCommentCheckerPackageBinary({ packageJsonPath, binaryName, existsSync }) ?? undefined;
   } catch {
     return;
   }

package/packages/omo-codex/plugin/components/comment-checker/hooks/hooks.json

@@ -8,7 +8,7 @@
 						"type": "command",
 						"command": "node \"${PLUGIN_ROOT}/dist/cli.js\" hook post-tool-use",
 						"timeout": 30,
-						"statusMessage": "(OmO 5.1.2) Checking Comments"
+						"statusMessage": "(OmO 5.1.4) Checking Comments"
 					}
 				]
 			}

package/packages/omo-codex/plugin/components/comment-checker/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@code-yeongyu/codex-comment-checker",
-	"version": "5.1.2",
+	"version": "5.1.4",
 	"description": "Codex plugin that runs comment-checker after edit-like PostToolUse hooks.",
 	"type": "module",
 	"packageManager": "npm@11.12.1",

package/packages/omo-codex/plugin/components/comment-checker/src/runner.ts

@@ -1,7 +1,8 @@
 import { spawn } from "node:child_process";
 import { existsSync } from "node:fs";
 import { createRequire } from "node:module";
-import { dirname, join } from "node:path";
+
+import { findCommentCheckerPackageBinary } from "@oh-my-opencode/comment-checker-core";
 
 import type { CommentCheckerHookInput } from "./core.js";
 
@@ -108,9 +109,8 @@ function resolvePackageApiBinary(): string | undefined {
 function resolvePackageBinary(binaryName: string): string | undefined {
 	try {
 		const require = createRequire(import.meta.url);
-		const packagePath = require.resolve(`${commentCheckerPackageName()}/package.json`);
-		const binaryPath = join(dirname(packagePath), "bin", binaryName);
-		return existsSync(binaryPath) ? binaryPath : undefined;
+		const packageJsonPath = require.resolve(`${commentCheckerPackageName()}/package.json`);
+		return findCommentCheckerPackageBinary({ packageJsonPath, binaryName, existsSync }) ?? undefined;
 	} catch {
 		return undefined;
 	}

package/packages/omo-codex/plugin/components/git-bash/hooks/hooks.json

@@ -8,7 +8,7 @@
 						"type": "command",
 						"command": "node \"${PLUGIN_ROOT}/dist/cli.js\" hook pre-tool-use",
 						"timeout": 5,
-						"statusMessage": "(OmO 5.1.2) Recommending Git Bash MCP"
+						"statusMessage": "(OmO 5.1.4) Recommending Git Bash MCP"
 					}
 				]
 			}
@@ -20,7 +20,7 @@
 						"type": "command",
 						"command": "node \"${PLUGIN_ROOT}/dist/cli.js\" hook post-compact",
 						"timeout": 5,
-						"statusMessage": "(OmO 5.1.2) Resetting Git Bash MCP Reminder"
+						"statusMessage": "(OmO 5.1.4) Resetting Git Bash MCP Reminder"
 					}
 				]
 			}

package/packages/omo-codex/plugin/components/git-bash/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@sisyphuslabs/codex-git-bash-hook",
-	"version": "5.1.2",
+	"version": "5.1.4",
 	"description": "Codex hook component that reminds Windows sessions to prefer the OMO git_bash MCP.",
 	"type": "module",
 	"private": true,

package/packages/omo-codex/plugin/components/lazycodex-executor-verify/hooks/hooks.json

@@ -8,7 +8,7 @@
 						"type": "command",
 						"command": "node \"${PLUGIN_ROOT}/components/lazycodex-executor-verify/dist/cli.js\" hook subagent-stop",
 						"timeout": 10,
-						"statusMessage": "(OmO 5.1.2) Verifying LazyCodex Executor Evidence"
+						"statusMessage": "(OmO 5.1.4) Verifying LazyCodex Executor Evidence"
 					}
 				]
 			}