oh-my-opencode 4.7.5 → 4.8.0

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

@@ -6,8 +6,8 @@ import { filterDiagnosticsBySeverity, formatApplyResult, formatDiagnostic, forma
 import { inferExtensionFromDirectory } from "./lsp/infer-extension.js";
 import { getLspManager } from "./lsp/manager.js";
 import { getAllServers } from "./lsp/server-resolution.js";
-import { handleMissingDependencyError } from "./lsp/utils.js";
 import { applyWorkspaceEdit } from "./lsp/workspace-edit.js";
+import { missingDependencyResult } from "./missing-dependency-result.js";
 const objectSchema = (properties, required = []) => ({
     type: "object",
     properties,
@@ -144,20 +144,16 @@ export async function executeLspDiagnostics(params, signal) {
         return text(output, details);
     }
     catch (error) {
-        const message = handleMissingDependencyError(error);
-        if (message) {
-            const details = {
-                filePath,
-                severity,
-                mode: "file",
-                diagnostics: [],
-                totalDiagnostics: 0,
-                truncated: false,
-                error: message,
-                errorKind: "missing_dependency",
-            };
-            return text(message, details);
-        }
+        const missingDependency = missingDependencyResult(error, {
+            filePath,
+            severity,
+            mode: "file",
+            diagnostics: [],
+            totalDiagnostics: 0,
+            truncated: false,
+        });
+        if (missingDependency)
+            return missingDependency;
         throw error;
     }
 }
@@ -174,17 +170,14 @@ async function executeLspGotoDefinition(params, signal) {
         return text(locations.map(formatLocation).join("\n"), details);
     }
     catch (error) {
-        const message = handleMissingDependencyError(error);
-        if (message) {
-            return text(message, {
-                filePath,
-                line,
-                character,
-                locations: [],
-                error: message,
-                errorKind: "missing_dependency",
-            });
-        }
+        const missingDependency = missingDependencyResult(error, {
+            filePath,
+            line,
+            character,
+            locations: [],
+        });
+        if (missingDependency)
+            return missingDependency;
         throw error;
     }
 }
@@ -216,19 +209,16 @@ async function executeLspFindReferences(params, signal) {
         return text(output, details);
     }
     catch (error) {
-        const message = handleMissingDependencyError(error);
-        if (message) {
-            return text(message, {
-                filePath,
-                line,
-                character,
-                references: [],
-                totalReferences: 0,
-                truncated: false,
-                error: message,
-                errorKind: "missing_dependency",
-            });
-        }
+        const missingDependency = missingDependencyResult(error, {
+            filePath,
+            line,
+            character,
+            references: [],
+            totalReferences: 0,
+            truncated: false,
+        });
+        if (missingDependency)
+            return missingDependency;
         throw error;
     }
 }
@@ -259,20 +249,17 @@ async function executeLspSymbols(params, signal) {
         return formatSymbolsResult(filePath, scope, symbols, limit);
     }
     catch (error) {
-        const message = handleMissingDependencyError(error);
-        if (message) {
-            const query = optionalString(params, "query");
-            return text(message, {
-                filePath,
-                scope,
-                symbols: [],
-                totalSymbols: 0,
-                truncated: false,
-                error: message,
-                errorKind: "missing_dependency",
-                ...(query === undefined ? {} : { query }),
-            });
-        }
+        const query = optionalString(params, "query");
+        const missingDependency = missingDependencyResult(error, {
+            filePath,
+            scope,
+            symbols: [],
+            totalSymbols: 0,
+            truncated: false,
+            ...(query === undefined ? {} : { query }),
+        });
+        if (missingDependency)
+            return missingDependency;
         throw error;
     }
 }
@@ -312,17 +299,14 @@ async function executeLspPrepareRename(params, signal) {
         return text(formatPrepareRenameResult(result), details);
     }
     catch (error) {
-        const message = handleMissingDependencyError(error);
-        if (message) {
-            return text(message, {
-                filePath,
-                line,
-                character,
-                result: null,
-                error: message,
-                errorKind: "missing_dependency",
-            });
-        }
+        const missingDependency = missingDependencyResult(error, {
+            filePath,
+            line,
+            character,
+            result: null,
+        });
+        if (missingDependency)
+            return missingDependency;
         throw error;
     }
 }
@@ -332,25 +316,25 @@ async function executeLspRename(params, signal) {
     const character = requireNumber(params, "character");
     const newName = requireString(params, "newName");
     try {
-        const edit = await withLspClient(filePath, async (client) => client.rename(filePath, line, character, newName), "rename", clientOptions(signal));
-        const apply = applyWorkspaceEdit(edit);
-        const details = { filePath, line, character, newName, apply, edit };
+        const edit = await withLspClient(filePath, async (client, workspaceRoot) => ({
+            edit: await client.rename(filePath, line, character, newName),
+            workspaceRoot,
+        }), "rename", clientOptions(signal));
+        const apply = applyWorkspaceEdit(edit.edit, { workspaceRoot: edit.workspaceRoot });
+        const details = { filePath, line, character, newName, apply, edit: edit.edit };
         return text(formatApplyResult(apply), details, !apply.success);
     }
     catch (error) {
-        const message = handleMissingDependencyError(error);
-        if (message) {
-            return text(message, {
-                filePath,
-                line,
-                character,
-                newName,
-                apply: null,
-                edit: null,
-                error: message,
-                errorKind: "missing_dependency",
-            });
-        }
+        const missingDependency = missingDependencyResult(error, {
+            filePath,
+            line,
+            character,
+            newName,
+            apply: null,
+            edit: null,
+        });
+        if (missingDependency)
+            return missingDependency;
         throw error;
     }
 }

package/packages/lsp-tools-mcp/package.json

@@ -3,7 +3,7 @@
 	"version": "0.1.0",
 	"description": "Standalone Language Server Protocol tools exposed as a stdio MCP server.",
 	"type": "module",
-	"packageManager": "npm@11.12.1",
+	"packageManager": "npm@11.16.0",
 	"license": "MIT",
 	"homepage": "https://github.com/code-yeongyu/lsp-tools-mcp",
 	"repository": {
@@ -41,10 +41,10 @@
 		"check": "tsc --noEmit && biome check . && npm run build"
 	},
 	"devDependencies": {
-		"@biomejs/biome": "2.4.15",
-		"@types/node": "^25.7.0",
+		"@biomejs/biome": "2.4.16",
+		"@types/node": "^25.9.2",
 		"typescript": "^6.0.3",
-		"vitest": "^4.1.5"
+		"vitest": "^4.1.8"
 	},
 	"engines": {
 		"node": ">=20.0.0"

package/packages/omo-codex/lazycodex-repository/.github/workflows/pr-source-guidance.yml

@@ -0,0 +1,36 @@
+name: PR source guidance
+
+on:
+  pull_request_target:
+    types: [opened, reopened, ready_for_review]
+
+permissions:
+  contents: read
+  issues: write
+  pull-requests: read
+
+jobs:
+  comment-source-repo:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Comment with source repository guidance
+        env:
+          GH_TOKEN: ${{ github.token }}
+          PR_NUMBER: ${{ github.event.pull_request.number }}
+        run: |
+          set -euo pipefail
+
+          marker="canonical LazyCodex source lives in"
+          if gh pr view "$PR_NUMBER" --repo "$GITHUB_REPOSITORY" --json comments --jq '.comments[].body' | grep -F "$marker" >/dev/null; then
+            echo "Source guidance comment already exists."
+            exit 0
+          fi
+
+          body="$(mktemp)"
+          cat > "$body" <<'MARKDOWN'
+          Thanks for opening this PR. The canonical LazyCodex source lives in [oh-my-openagent](https://github.com/code-yeongyu/oh-my-openagent), and the Codex Light implementation is under [`packages/omo-codex`](https://github.com/code-yeongyu/oh-my-openagent/tree/dev/packages/omo-codex).
+
+          Direct edits in this marketplace repository can be overwritten by the release sync. Please open the implementation PR in `code-yeongyu/oh-my-openagent` and add the `lazycodex` label there so it gets routed faster.
+          MARKDOWN
+
+          gh pr comment "$PR_NUMBER" --repo "$GITHUB_REPOSITORY" --body-file "$body"

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

@@ -17,7 +17,7 @@
 	"interface": {
 		"displayName": "OMO",
 		"shortDescription": "Unified local Codex components",
-		"longDescription": "OMO exposes the local Codex Rules, Comment Checker, LSP, Ultrawork, and ulw-loop components as one plugin namespace while keeping each component isolated under components/ for maintenance.",
+		"longDescription": "OMO exposes local Codex Rules, Comment Checker, Git Bash, LSP, Ultrawork, ulw-loop, start-work continuation, and telemetry components as one plugin namespace while keeping each component isolated under components/ for maintenance.",
 		"developerName": "Yeongyu Kim",
 		"category": "Developer Tools",
 		"capabilities": ["Hooks", "MCP Tools", "Code Intelligence", "Workflow", "Context Injection"],

package/packages/omo-codex/plugin/README.md

@@ -7,11 +7,13 @@ Internally each component remains isolated under `components/`:
 - `components/comment-checker`
 - `components/rules`
 - `components/lsp`
+- `components/git-bash`
+- `components/start-work-continuation`
 - `components/ultrawork`
 - `components/ulw-loop`
 - `components/telemetry`
 
-The root plugin manifest exports one Codex plugin named `omo`, with aggregate hooks, skills, and the LSP MCP server.
+The root plugin manifest exports one Codex plugin named `omo`, with aggregate hooks, skills, and plugin-scoped MCP servers for `ast_grep`, `grep_app`, `context7`, `git_bash`, and `lsp`.
 
 ## Telemetry
 

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

@@ -0,0 +1,188 @@
+import { getString, isRecord } from "./record.js";
+import type { CommentCheckRequest } from "./types.js";
+
+type ApplyPatchAccumulator = {
+	operation: "add" | "delete" | "update";
+	filePath: string;
+	movePath?: string;
+	oldLines: string[];
+	newLines: string[];
+};
+
+type ApplyPatchFileMetadata = {
+	filePath: string;
+	movePath?: string;
+	before: string;
+	after: string;
+	type?: string;
+};
+
+export function extractApplyPatchRequests(event: {
+	details?: unknown;
+	input: Record<string, unknown>;
+	toolName: string;
+}): CommentCheckRequest[] {
+	const metadataRequests = extractApplyPatchMetadataRequests(event.details, event.toolName);
+	if (metadataRequests.length > 0) return metadataRequests;
+
+	const patch = getString(event.input, ["input", "patch", "command"]);
+	if (!patch) return [];
+	return parseApplyPatchRequests(patch, event.toolName);
+}
+
+export function parseApplyPatchRequests(patch: string, sourceToolName = "apply_patch"): CommentCheckRequest[] {
+	const requests: CommentCheckRequest[] = [];
+	let current: ApplyPatchAccumulator | undefined;
+
+	const flush = (): void => {
+		if (!current) return;
+		if (current.operation === "add") {
+			const content = joinPatchLines(current.newLines);
+			if (content.length > 0) {
+				requests.push({
+					sourceToolName,
+					toolName: "Write",
+					filePath: current.filePath,
+					toolInput: {
+						file_path: current.filePath,
+						content,
+					},
+				});
+			}
+		}
+		if (current.operation === "update") {
+			const newString = joinPatchLines(current.newLines);
+			if (newString.length > 0) {
+				const filePath = current.movePath ?? current.filePath;
+				requests.push({
+					sourceToolName,
+					toolName: "Edit",
+					filePath,
+					toolInput: {
+						file_path: filePath,
+						old_string: joinPatchLines(current.oldLines),
+						new_string: newString,
+					},
+				});
+			}
+		}
+		current = undefined;
+	};
+
+	for (const line of patch.split(/\r?\n/)) {
+		if (line === "*** Begin Patch" || line === "*** End Patch") continue;
+		if (line.startsWith("*** Add File: ")) {
+			flush();
+			current = makeAccumulator("add", line.slice("*** Add File: ".length).trim());
+			continue;
+		}
+		if (line.startsWith("*** Update File: ")) {
+			flush();
+			current = makeAccumulator("update", line.slice("*** Update File: ".length).trim());
+			continue;
+		}
+		if (line.startsWith("*** Delete File: ")) {
+			flush();
+			current = makeAccumulator("delete", line.slice("*** Delete File: ".length).trim());
+			continue;
+		}
+		if (line.startsWith("*** Move to: ")) {
+			if (current?.operation === "update") current.movePath = line.slice("*** Move to: ".length).trim();
+			continue;
+		}
+		if (!current) continue;
+		if (line.startsWith("@@")) continue;
+		if (current.operation === "add") {
+			if (line.startsWith("+")) current.newLines.push(line.slice(1));
+			continue;
+		}
+		if (current.operation === "update") {
+			if (line.startsWith("+")) current.newLines.push(line.slice(1));
+			if (line.startsWith("-")) current.oldLines.push(line.slice(1));
+		}
+	}
+
+	flush();
+	return requests;
+}
+
+function extractApplyPatchMetadataRequests(details: unknown, sourceToolName: string): CommentCheckRequest[] {
+	const metadataFiles = getApplyPatchMetadataFiles(details);
+	if (metadataFiles.length === 0) return [];
+
+	const requests: CommentCheckRequest[] = [];
+	for (const file of metadataFiles) {
+		if (file.type === "delete") continue;
+		const filePath = file.movePath ?? file.filePath;
+		if (file.before.length === 0) {
+			requests.push({
+				sourceToolName,
+				toolName: "Write",
+				filePath,
+				toolInput: {
+					file_path: filePath,
+					content: file.after,
+				},
+			});
+			continue;
+		}
+		requests.push({
+			sourceToolName,
+			toolName: "Edit",
+			filePath,
+			toolInput: {
+				file_path: filePath,
+				old_string: file.before,
+				new_string: file.after,
+			},
+		});
+	}
+	return requests;
+}
+
+function getApplyPatchMetadataFiles(details: unknown): ApplyPatchFileMetadata[] {
+	if (!isRecord(details)) return [];
+	const direct = readApplyPatchMetadataFiles(details["files"]);
+	if (direct.length > 0) return direct;
+	const resultDetails = details["result"];
+	const result = isRecord(resultDetails) ? readApplyPatchMetadataFiles(resultDetails["files"]) : [];
+	if (result.length > 0) return result;
+	const metadataDetails = details["metadata"];
+	const metadata = isRecord(metadataDetails) ? readApplyPatchMetadataFiles(metadataDetails["files"]) : [];
+	return metadata;
+}
+
+function readApplyPatchMetadataFiles(value: unknown): ApplyPatchFileMetadata[] {
+	if (!Array.isArray(value)) return [];
+	const files: ApplyPatchFileMetadata[] = [];
+	for (const item of value) {
+		if (!isRecord(item)) continue;
+		const filePath = getString(item, ["filePath", "file_path", "path"]);
+		const movePath = getString(item, ["movePath", "move_path"]);
+		const before = getString(item, ["before", "old", "oldString", "old_string"]);
+		const after = getString(item, ["after", "new", "newString", "new_string"]);
+		const type = getString(item, ["type", "operation"]);
+		if (!filePath || before === undefined || after === undefined) continue;
+		files.push({
+			filePath,
+			before,
+			after,
+			...(movePath === undefined ? {} : { movePath }),
+			...(type === undefined ? {} : { type }),
+		});
+	}
+	return files;
+}
+
+function makeAccumulator(operation: ApplyPatchAccumulator["operation"], filePath: string): ApplyPatchAccumulator {
+	return {
+		operation,
+		filePath,
+		oldLines: [],
+		newLines: [],
+	};
+}
+
+function joinPatchLines(lines: string[]): string {
+	return lines.length === 0 ? "" : `${lines.join("\n")}\n`;
+}

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

@@ -0,0 +1 @@
+export { getString, isRecord } from "./record.js";