oh-my-opencode 4.5.1 → 4.5.12

package/packages/omo-codex/plugin/components/lsp/test/package-smoke.test.ts

@@ -0,0 +1,153 @@
+import { readdirSync, readFileSync } from "node:fs";
+import { describe, expect, it } from "vitest";
+
+type PackageJson = {
+	readonly version: string;
+	readonly type: string;
+	readonly packageManager: string;
+	readonly bin: Record<string, string>;
+	readonly dependencies: Record<string, string>;
+	readonly scripts: Record<string, string>;
+};
+
+type HookCommand = {
+	readonly command: string;
+};
+
+type HookEntry = {
+	readonly hooks: readonly HookCommand[];
+};
+
+type HooksJson = {
+	readonly hooks: Record<string, readonly HookEntry[]>;
+};
+
+type McpServer = {
+	readonly command: string;
+	readonly args: readonly string[];
+};
+
+type McpJson = {
+	readonly mcpServers: Record<string, McpServer>;
+};
+
+function readPackageJson(path: string): PackageJson {
+	const parsed: unknown = JSON.parse(readFileSync(path, "utf8"));
+	if (!isPackageJson(parsed)) throw new TypeError(`Invalid package metadata: ${path}`);
+	return parsed;
+}
+
+function readHooksJson(path: string): HooksJson {
+	const parsed: unknown = JSON.parse(readFileSync(path, "utf8"));
+	if (!isHooksJson(parsed)) throw new TypeError(`Invalid hooks metadata: ${path}`);
+	return parsed;
+}
+
+function readMcpJson(path: string): McpJson {
+	const parsed: unknown = JSON.parse(readFileSync(path, "utf8"));
+	if (!isMcpJson(parsed)) throw new TypeError(`Invalid MCP metadata: ${path}`);
+	return parsed;
+}
+
+describe("plugin package metadata", () => {
+	it("#given packaged component files #when validating entrypoints #then hook command stays local and MCP command references the package", () => {
+		// given
+		const packageJson = readPackageJson("package.json");
+		const hooksJson = readHooksJson("hooks/hooks.json");
+		const mcpJson = readMcpJson(".mcp.json");
+		const cliSource = readFileSync("src/cli.ts", "utf8");
+		const codexHookSource = readFileSync("src/codex-hook.ts", "utf8");
+		const sourceFiles = readdirSync("src");
+
+		// when
+		const command = hooksJson.hooks["PostToolUse"]?.[0]?.hooks[0]?.command;
+		const lspServer = mcpJson.mcpServers["lsp"];
+		const pluginRoot = ["$", "{PLUGIN_ROOT}"].join("");
+
+		// then
+		expect(packageJson.type).toBe("module");
+		expect(packageJson.packageManager).toBe("npm@11.12.1");
+		expect(packageJson.dependencies).toEqual({
+			"@code-yeongyu/lsp-tools-mcp": "file:../../../../lsp-tools-mcp",
+		});
+		expect(packageJson.bin["omo-lsp"]).toBe("./dist/cli.js");
+		expect(packageJson.bin["codex-lsp"]).toBeUndefined();
+		expect(packageJson.scripts["build"]).toBe("node scripts/clean-dist.mjs && tsc -p tsconfig.build.json");
+		expect(cliSource.startsWith("#!/usr/bin/env node")).toBe(true);
+		expect(cliSource).toContain("Usage: omo-lsp [mcp | hook post-tool-use]");
+		expect(command).toBe(`node "${pluginRoot}/dist/cli.js" hook post-tool-use`);
+		expect(lspServer?.command).toBe("node");
+		expect(lspServer?.args).toEqual(["../../../../lsp-tools-mcp/dist/cli.js", "mcp"]);
+		expect(cliSource).not.toContain("./lazy-lsp-mcp.js");
+		expect(cliSource).not.toContain("@code-yeongyu/lsp-tools-mcp");
+		expect(cliSource).toContain("../../../../../lsp-tools-mcp/dist/cli.js");
+		expect(codexHookSource).not.toContain("@code-yeongyu/lsp-tools-mcp");
+		expect(codexHookSource).toContain("../../../../../lsp-tools-mcp/dist/lsp/manager.js");
+		expect(codexHookSource).toContain("../../../../../lsp-tools-mcp/dist/tools.js");
+		expect(sourceFiles.filter((name) => name.startsWith("lazy-mcp") || name === "lazy-lsp-mcp.ts")).toEqual([]);
+	});
+
+	it("#given LSP skill guidance #when validating MCP tool instructions #then tool names are not framed as shell commands", () => {
+		// given
+		const skill = readFileSync("skills/lsp/SKILL.md", "utf8");
+
+		// when
+		const mentionsToolInterface = skill.includes("through the tool interface");
+		const rejectsShellExecution = skill.includes("not shell commands");
+
+		// then
+		expect(mentionsToolInterface).toBe(true);
+		expect(rejectsShellExecution).toBe(true);
+	});
+});
+
+function isPackageJson(value: unknown): value is PackageJson {
+	return (
+		isRecord(value) &&
+		typeof value["version"] === "string" &&
+		value["type"] === "module" &&
+		value["packageManager"] === "npm@11.12.1" &&
+		isStringRecord(value["bin"]) &&
+		isStringRecord(value["dependencies"]) &&
+		isStringRecord(value["scripts"])
+	);
+}
+
+function isHooksJson(value: unknown): value is HooksJson {
+	if (!isRecord(value) || !isRecord(value["hooks"])) return false;
+	return Object.values(value["hooks"]).every(isHookEntries);
+}
+
+function isHookEntries(value: unknown): value is readonly HookEntry[] {
+	return Array.isArray(value) && value.every(isHookEntry);
+}
+
+function isHookEntry(value: unknown): value is HookEntry {
+	return isRecord(value) && Array.isArray(value["hooks"]) && value["hooks"].every(isHookCommand);
+}
+
+function isHookCommand(value: unknown): value is HookCommand {
+	return isRecord(value) && typeof value["command"] === "string";
+}
+
+function isMcpJson(value: unknown): value is McpJson {
+	if (!isRecord(value) || !isRecord(value["mcpServers"])) return false;
+	return Object.values(value["mcpServers"]).every(isMcpServer);
+}
+
+function isMcpServer(value: unknown): value is McpServer {
+	return (
+		isRecord(value) &&
+		typeof value["command"] === "string" &&
+		Array.isArray(value["args"]) &&
+		value["args"].every((item) => typeof item === "string")
+	);
+}
+
+function isStringRecord(value: unknown): value is Record<string, string> {
+	return isRecord(value) && Object.values(value).every((item) => typeof item === "string");
+}
+
+function isRecord(value: unknown): value is Record<string, unknown> {
+	return typeof value === "object" && value !== null && !Array.isArray(value);
+}

package/packages/omo-codex/plugin/components/lsp/tsconfig.build.json

@@ -0,0 +1,12 @@
+{
+	"extends": "./tsconfig.json",
+	"compilerOptions": {
+		"allowImportingTsExtensions": false,
+		"declaration": true,
+		"outDir": "dist",
+		"rootDir": "src",
+		"noEmit": false
+	},
+	"include": ["src/**/*"],
+	"exclude": ["test/**/*"]
+}

package/packages/omo-codex/plugin/components/lsp/tsconfig.json

@@ -0,0 +1,27 @@
+{
+	"compilerOptions": {
+		"target": "ES2022",
+		"module": "Node16",
+		"moduleResolution": "Node16",
+		"lib": ["ES2022"],
+		"strict": true,
+		"exactOptionalPropertyTypes": true,
+		"noUncheckedIndexedAccess": true,
+		"noPropertyAccessFromIndexSignature": true,
+		"verbatimModuleSyntax": true,
+		"noImplicitOverride": true,
+		"noImplicitReturns": true,
+		"noFallthroughCasesInSwitch": true,
+		"noUnusedLocals": true,
+		"noUnusedParameters": true,
+		"esModuleInterop": true,
+		"allowImportingTsExtensions": true,
+		"skipLibCheck": true,
+		"forceConsistentCasingInFileNames": true,
+		"resolveJsonModule": true,
+		"useDefineForClassFields": false,
+		"types": ["node"],
+		"noEmit": true
+	},
+	"include": ["src/**/*", "test/**/*"]
+}

package/packages/omo-codex/plugin/components/lsp/vitest.config.ts

@@ -0,0 +1,9 @@
+import { defineConfig } from "vitest/config";
+
+export default defineConfig({
+	test: {
+		include: ["test/**/*.test.ts"],
+		environment: "node",
+		pool: "threads",
+	},
+});

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

@@ -0,0 +1,3 @@
+{
+	"hooks": "./hooks/hooks.json"
+}

package/packages/omo-codex/plugin/components/rules/.gitattributes

@@ -0,0 +1,13 @@
+# Normalize line endings: store LF in git, check out LF on every platform.
+# Required so biome's --check passes on Windows (default core.autocrlf=true).
+* text=auto eol=lf
+
+# Explicit binary types
+*.png binary
+*.jpg binary
+*.jpeg binary
+*.gif binary
+*.ico binary
+*.zip binary
+*.tgz binary
+*.gz binary

package/packages/omo-codex/plugin/components/rules/.github/CODEOWNERS

@@ -0,0 +1,12 @@
+* @code-yeongyu
+
+.github/workflows/* @code-yeongyu
+.github/dependabot.yml @code-yeongyu
+package.json @code-yeongyu
+package-lock.json @code-yeongyu
+LICENSE @code-yeongyu
+NOTICE @code-yeongyu
+README.md @code-yeongyu
+CHANGELOG.md @code-yeongyu
+.codex-plugin/plugin.json @code-yeongyu
+hooks/hooks.json @code-yeongyu

package/packages/omo-codex/plugin/components/rules/.github/ISSUE_TEMPLATE/bug.yml

@@ -0,0 +1,49 @@
+name: Bug Report
+description: Report broken Codex rule injection or matching behavior
+labels: [bug]
+body:
+  - type: markdown
+    attributes:
+      value: |
+        Include the Codex hook payload, hook output, rule file, and plugin version needed to reproduce.
+
+  - type: textarea
+    id: what
+    attributes:
+      label: What happened?
+      description: Include exact output/errors.
+    validations:
+      required: true
+
+  - type: textarea
+    id: payload
+    attributes:
+      label: Hook payload
+      description: Paste the minimal SessionStart, UserPromptSubmit, or PostToolUse payload that reproduces the issue.
+      render: json
+    validations:
+      required: false
+
+  - type: textarea
+    id: rule
+    attributes:
+      label: Rule file
+      description: Paste the relevant rule file or frontmatter.
+      render: markdown
+    validations:
+      required: false
+
+  - type: textarea
+    id: expected
+    attributes:
+      label: Expected behavior
+    validations:
+      required: true
+
+  - type: input
+    id: version
+    attributes:
+      label: codex-rules version
+      placeholder: 0.1.0
+    validations:
+      required: false

package/packages/omo-codex/plugin/components/rules/.github/ISSUE_TEMPLATE/feature.yml

@@ -0,0 +1,27 @@
+name: Feature Request
+description: Propose a Codex rule source, matcher, or hook improvement
+labels: [enhancement]
+body:
+  - type: textarea
+    id: problem
+    attributes:
+      label: Problem
+      description: What workflow is blocked or awkward today?
+    validations:
+      required: true
+
+  - type: textarea
+    id: proposal
+    attributes:
+      label: Proposal
+      description: What should codex-rules do?
+    validations:
+      required: true
+
+  - type: textarea
+    id: alternatives
+    attributes:
+      label: Alternatives considered
+      description: What else could solve this?
+    validations:
+      required: false

package/packages/omo-codex/plugin/components/rules/.github/branch-ruleset.json

@@ -0,0 +1,45 @@
+{
+	"name": "main protection",
+	"target": "branch",
+	"enforcement": "active",
+	"conditions": {
+		"ref_name": {
+			"include": ["~DEFAULT_BRANCH"],
+			"exclude": []
+		}
+	},
+	"rules": [
+		{ "type": "deletion" },
+		{ "type": "non_fast_forward" },
+		{ "type": "required_linear_history" },
+		{
+			"type": "pull_request",
+			"parameters": {
+				"required_approving_review_count": 1,
+				"dismiss_stale_reviews_on_push": true,
+				"require_code_owner_review": true,
+				"require_last_push_approval": false,
+				"required_review_thread_resolution": true
+			}
+		},
+		{
+			"type": "required_status_checks",
+			"parameters": {
+				"strict_required_status_checks_policy": true,
+				"required_status_checks": [
+					{ "context": "test (ubuntu-latest · node 20)" },
+					{ "context": "test (ubuntu-latest · node 22)" },
+					{ "context": "test (macos-latest · node 20)" },
+					{ "context": "test (macos-latest · node 22)" }
+				]
+			}
+		}
+	],
+	"bypass_actors": [
+		{
+			"actor_id": 5,
+			"actor_type": "RepositoryRole",
+			"bypass_mode": "always"
+		}
+	]
+}

package/packages/omo-codex/plugin/components/rules/.github/dependabot.yml

@@ -0,0 +1,16 @@
+version: 2
+updates:
+  - package-ecosystem: npm
+    directory: /
+    schedule:
+      interval: weekly
+    open-pull-requests-limit: 5
+    groups:
+      dev-dependencies:
+        dependency-type: development
+
+  - package-ecosystem: github-actions
+    directory: /
+    schedule:
+      interval: weekly
+    open-pull-requests-limit: 5

package/packages/omo-codex/plugin/components/rules/.github/pull_request_template.md

@@ -0,0 +1,20 @@
+## Summary
+
+<!-- Brief description, 1-3 bullets -->
+
+-
+
+## Verification
+
+- [ ] `npm run check` (typecheck + biome + build)
+- [ ] `npm test` (unit tests)
+- [ ] `npm pack --dry-run` (release sanity)
+- [ ] Hook smoke-tested locally with `node dist/cli.js hook session-start`
+- [ ] Hook smoke-tested locally with `node dist/cli.js hook post-tool-use`
+
+## Codex plugin impact
+
+- [ ] `.codex-plugin/plugin.json` remains valid
+- [ ] `hooks/hooks.json` still uses stable Codex hook JSON
+- [ ] Session deduplication behavior is covered by tests
+- [ ] CHANGELOG entry added for user-facing changes

package/packages/omo-codex/plugin/components/rules/.github/workflows/ci.yml

@@ -0,0 +1,47 @@
+name: ci
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+permissions:
+  contents: read
+
+jobs:
+  test:
+    name: test (${{ matrix.os }} · node ${{ matrix.node }})
+    runs-on: ${{ matrix.os }}
+    timeout-minutes: 15
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+        node: ["20", "22"]
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v6
+
+      - name: Setup Node ${{ matrix.node }}
+        uses: actions/setup-node@v6
+        with:
+          node-version: ${{ matrix.node }}
+          cache: npm
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Check
+        run: npm run check
+
+      - name: Unit tests
+        run: npm test
+
+      - name: Package smoke
+        run: npm pack --dry-run

package/packages/omo-codex/plugin/components/rules/.github/workflows/publish.yml

@@ -0,0 +1,51 @@
+name: publish
+
+on:
+  release:
+    types: [published]
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: false
+
+permissions:
+  contents: read
+  id-token: write
+
+jobs:
+  publish:
+    runs-on: ubuntu-latest
+    timeout-minutes: 15
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v6
+
+      - name: Setup Node 22
+        uses: actions/setup-node@v6
+        with:
+          node-version: "22"
+          registry-url: https://registry.npmjs.org
+          cache: npm
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Check
+        run: npm run check
+
+      - name: Unit tests
+        run: npm test
+
+      - name: Package smoke
+        run: npm pack --dry-run
+
+      - name: Publish to npm
+        run: |
+          if [ -z "$NODE_AUTH_TOKEN" ]; then
+            echo "NODE_AUTH_TOKEN is not configured; skipping npm publish."
+            exit 0
+          fi
+          npm publish --access public --provenance
+        env:
+          NODE_AUTH_TOKEN: ${{ secrets.NODE_AUTH_TOKEN }}

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

@@ -0,0 +1,34 @@
+# Repository Conventions
+
+Conventions for human contributors and AI agents working on this repository.
+
+## Style
+
+- Terse technical prose. No emojis in commits, issues, PR comments, or code.
+- TypeScript strict mode. No `any`, no `@ts-ignore`, no `@ts-expect-error`, no enums.
+- ESM modules with `.js` suffix in runtime import paths.
+- Tabs for indentation. Double quotes for strings.
+- Tests use vitest with `#given .. #when .. #then` descriptions or plain `// given / // when / // then` body comments.
+
+## Commands
+
+- `npm install` - install dependencies.
+- `npm test` - run vitest once.
+- `npm run typecheck` - strict TypeScript check.
+- `npm run check` - type check, biome, and build.
+- `npm pack --dry-run` - release package smoke test.
+- `node dist/cli.js hook session-start < fixture.json` - smoke-test static rule injection.
+- `node dist/cli.js hook post-tool-use < fixture.json` - smoke-test dynamic rule injection.
+
+## Constraints
+
+- No Bun APIs. Runtime is Node only because Codex launches plugin hooks with Node.
+- Keep `SessionStart`, `UserPromptSubmit`, and `PostToolUse` hook behavior covered by tests.
+- Keep Codex file path extraction for reads, edits, `apply_patch`, and shell-style tools covered by tests.
+- Hook output must use the stable Codex hook JSON contract.
+- Do not couple this package back to pi, omo, or senpi internal source paths.
+
+## Don'ts
+
+- No `git add -A` or `git add .`. Stage only the files you changed.
+- No `git commit --no-verify`. No force pushes. No history rewriting on shared branches.

package/packages/omo-codex/plugin/components/rules/CHANGELOG.md

@@ -0,0 +1,19 @@
+# Changelog
+
+## Unreleased
+
+- Restrict the default `PostToolUse` hook matcher to Codex's canonical `apply_patch` tool name.
+- Add opt-in `NODE_DEBUG=codex-rules` phase timing logs for `PostToolUse` debugging.
+- Harden dynamic hook coverage for additional-context JSON output, disabled/static modes, failed tool responses, and duplicate suppression.
+- Remove redundant apply_patch path scanning and stale tracked-tool constants.
+- Use portable Codex hook interpolation and add package smoke coverage for hook entrypoints.
+- Cap recursive rule directory scans and run CI on Windows in addition to Ubuntu and macOS.
+- Replace the external glob matcher dependency with an internal matcher so clean Codex plugin installs run without `node_modules`.
+
+## 0.1.0 - 2026-05-15
+
+- Port `pi-rules` rule loading, matching, formatting, truncation, and deduplication to a Codex plugin.
+- Add `SessionStart`, `UserPromptSubmit`, and `PostToolUse` hooks for static and file-specific context injection.
+- Add persistent per-session deduplication under Codex plugin data.
+- Add Codex-aware path extraction for read, write, edit, multi-edit, `apply_patch`, and shell command payloads.
+- Add tests, CI, release workflow, marketplace metadata, and local install support.

package/packages/omo-codex/plugin/components/rules/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 Yeongyu Kim
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

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

@@ -0,0 +1,15 @@
+codex-rules
+
+This package implements rule/context loading for Codex plugins.
+Its behavior is ported from pi-rules in the pi coding-agent extension ecosystem
+and inspired by oh-my-openagent (omo) at https://github.com/code-yeongyu/oh-my-openagent,
+including omo's `.omo/rules/` workflow and rules-injector hook architecture.
+omo is originally licensed under the Sustainable Use License 1.0.
+
+Yeongyu Kim (https://github.com/code-yeongyu), author of omo, pi-rules, and this
+package, licenses the source distributed in this repository under the MIT License.
+If any source was ported from omo or pi-rules, that ported source is re-licensed
+here under MIT for distribution as a Codex plugin. See LICENSE for terms.
+
+picomatch is by Jon Schlinkert and contributors (https://github.com/micromatch/picomatch).
+Distributed under the MIT License.