oh-my-opencode 5.0.0-beta.2 → 5.0.0-beta.21

package/packages/omo-codex/plugin/components/rules/test/formatter.test.ts

@@ -1,20 +1,24 @@
-import { describe, expect, it } from "vitest";
-
-import { formatDynamicBlock, formatStaticBlock } from "@oh-my-opencode/rules-engine/engine";
 import type { LoadedRule, MatchReason, RuleSource } from "@oh-my-opencode/rules-engine/engine";
+import { formatDynamicBlock, formatStaticBlock } from "@oh-my-opencode/rules-engine/engine";
+import { describe, expect, it } from "vitest";
 
 const FORMAT_OPTIONS = {
 	maxRuleChars: 10_000,
 	maxResultChars: 10_000,
 };
 
+// Runtime sentinel: the rules hook emits and re-greps this marker when
+// deduplicating injected rules against the transcript (transcript-rule-filter.ts).
+const INSTRUCTIONS_FROM = "Instructions from: ";
+
 describe("rules formatter hook context", () => {
 	it("#given multiline dynamic rules #when formatting PostToolUse context #then labels and bodies render on separate lines", () => {
 		// given
+		const body = "DYN_BODY_SENTINEL_LINE_1\n\nDYN_BODY_SENTINEL_LINE_2";
 		const rule = loadedRule({
 			path: "/repo/packages/CONTEXT.md",
 			relativePath: "packages/CONTEXT.md",
-			body: ["# packages", "", "## OVERVIEW", "23 sibling packages.", "", "## CONVENTIONS", "Use npm."].join("\n"),
+			body,
 		});
 
 		// when
@@ -25,124 +29,86 @@ describe("rules formatter hook context", () => {
 		);
 
 		// then
-		expect(block).toBe(
-			[
-				"Additional project instructions matched for packages/omo-codex/plugin/components/ulw-loop/src/paths.ts:",
-				"",
-				"Instructions from: /repo/packages/CONTEXT.md",
-				"",
-				"# packages",
-				"",
-				"## OVERVIEW",
-				"23 sibling packages.",
-				"",
-				"## CONVENTIONS",
-				"Use npm.",
-			].join("\n"),
-		);
+		expect(block).toContain(`${INSTRUCTIONS_FROM}/repo/packages/CONTEXT.md\n\n${body}`);
+		expect(occurrenceCount(block, INSTRUCTIONS_FROM)).toBe(1);
 	});
 
 	it("#given static rules #when formatting SessionStart context #then it injects rule bodies inline", () => {
 		// given
+		const body = "STATIC_BODY_SENTINEL";
 		const rule = loadedRule({
 			path: "/repo/CONTEXT.md",
 			relativePath: "CONTEXT.md",
-			body: "Keep generated hook context readable.",
+			body,
 		});
 
 		// when
 		const block = formatStaticBlock([rule], FORMAT_OPTIONS);
 
 		// then
-		expect(block).toBe(
-			[
-				"## Project Instructions",
-				"",
-				"Instructions from: /repo/CONTEXT.md",
-				"",
-				"Keep generated hook context readable.",
-			].join("\n"),
-		);
+		expect(block).toContain(`${INSTRUCTIONS_FROM}/repo/CONTEXT.md\n\n${body}`);
 	});
 
 	it("#given CRLF and bare CR rule bodies #when formatting context #then it normalizes line endings", () => {
 		// given
 		const rule = loadedRule({
-			body: "First line\r\n  indented second line\rThird line",
+			body: "A1\r\n  B2\rC3",
 		});
 
 		// when
 		const block = formatDynamicBlock([rule], "src/app.ts", FORMAT_OPTIONS);
 
 		// then
-		expect(block).toContain("First line\n  indented second line\nThird line");
+		expect(block).toContain("A1\n  B2\nC3");
 		expect(block).not.toContain("\r");
 	});
 
 	it("#given duplicate static rules with different line endings #when formatting context #then it injects one copy", () => {
 		// given
+		const sharedBody = "SHARED_L1\r\nSHARED_L2";
+		const normalizedSharedBody = "SHARED_L1\nSHARED_L2";
 		const lfRule = loadedRule({
 			path: "/repo/CONTEXT.md",
 			relativePath: "CONTEXT.md",
-			body: "Shared rule\nKeep one copy.",
+			body: normalizedSharedBody,
 		});
 		const crlfRule = loadedRule({
 			path: "/repo/packages/CONTEXT.md",
 			relativePath: "packages/CONTEXT.md",
-			body: "Shared rule\r\nKeep one copy.",
+			body: sharedBody,
 		});
 
 		// when
 		const block = formatStaticBlock([lfRule, crlfRule], FORMAT_OPTIONS);
 
 		// then
-		expect(occurrenceCount(block, "Instructions from: /repo/CONTEXT.md")).toBe(1);
-		expect(occurrenceCount(block, "Shared rule\nKeep one copy.")).toBe(1);
+		expect(occurrenceCount(block, `${INSTRUCTIONS_FROM}/repo/CONTEXT.md`)).toBe(1);
+		expect(occurrenceCount(block, normalizedSharedBody)).toBe(1);
 		expect(block).not.toContain("/repo/packages/CONTEXT.md");
 	});
 
 	it("#given a Hephaestus static rule #when formatting SessionStart context #then it injects its body before other rule bodies", () => {
 		// given
 		const rules = [
-			loadedRule({ path: "/repo/alpha.md", relativePath: "alpha.md", body: "Alpha guidance." }),
+			loadedRule({ path: "/repo/alpha.md", relativePath: "alpha.md", body: "ALPHA_BODY_SENTINEL" }),
 			loadedRule({
 				path: "/repo/bundled-rules/hephaestus.md",
 				relativePath: "bundled-rules/hephaestus.md",
-				body: "Hephaestus guidance.",
+				body: "HEPHAESTUS_BODY_SENTINEL",
 			}),
-			loadedRule({ path: "/repo/beta.md", relativePath: "beta.md", body: "Beta guidance." }),
+			loadedRule({ path: "/repo/beta.md", relativePath: "beta.md", body: "BETA_BODY_SENTINEL" }),
 		];
 
 		// when
 		const block = formatStaticBlock(rules, FORMAT_OPTIONS);
 
 		// then
-		expect(block).toContain("Instructions from: /repo/bundled-rules/hephaestus.md");
-		expect(block).toContain("Hephaestus guidance.");
-		expect(block).toContain("Alpha guidance.");
-		expect(block).toContain("Beta guidance.");
-		expect(block.indexOf("Hephaestus guidance.")).toBeLessThan(block.indexOf("Alpha guidance."));
-		expect(block.indexOf("Alpha guidance.")).toBeLessThan(block.indexOf("Beta guidance."));
-		expect(block).not.toContain("must read project rules:");
-		expect(block).not.toContain("- [hephaestus.md]");
-	});
-
-	it("#given only a Hephaestus static rule #when formatting SessionStart context #then it emits no project rule link section", () => {
-		// given
-		const rule = loadedRule({
-			path: "/repo/bundled-rules/hephaestus.md",
-			relativePath: "bundled-rules/hephaestus.md",
-			body: "Hephaestus guidance.",
-		});
-
-		// when
-		const block = formatStaticBlock([rule], FORMAT_OPTIONS);
-
-		// then
-		expect(block).toContain("Instructions from: /repo/bundled-rules/hephaestus.md");
-		expect(block).toContain("Hephaestus guidance.");
-		expect(block).not.toContain("- [hephaestus.md]");
-		expect(block).not.toContain("must read project rules:");
+		expect(block).toContain(`${INSTRUCTIONS_FROM}/repo/bundled-rules/hephaestus.md`);
+		expect(block).toContain("HEPHAESTUS_BODY_SENTINEL");
+		expect(block).toContain("ALPHA_BODY_SENTINEL");
+		expect(block).toContain("BETA_BODY_SENTINEL");
+		expect(block.indexOf("HEPHAESTUS_BODY_SENTINEL")).toBeLessThan(block.indexOf("ALPHA_BODY_SENTINEL"));
+		expect(block.indexOf("ALPHA_BODY_SENTINEL")).toBeLessThan(block.indexOf("BETA_BODY_SENTINEL"));
 	});
 
 	it("#given an oversized Hephaestus static rule #when formatting under a tight result budget #then its body is never truncated", () => {
@@ -162,7 +128,6 @@ describe("rules formatter hook context", () => {
 
 		// then
 		expect(block).toContain(tailMarker);
-		expect(block).not.toContain("[Truncated. Full:");
 	});
 
 	it("#given multiple oversized rules #when formatting under a tight result budget #then every rule receives a fair truncated share with a read-full guide", () => {
@@ -180,13 +145,15 @@ describe("rules formatter hook context", () => {
 		});
 
 		// then
-		expect(block).toContain("Instructions from: /repo/alpha.md");
-		expect(block).toContain("Instructions from: /repo/beta.md");
-		expect(block).toContain("Instructions from: /repo/gamma.md");
-		expect(block).toContain("[Truncated. Full: alpha.md]");
-		expect(block).toContain("[Truncated. Full: beta.md]");
-		expect(block).toContain("[Truncated. Full: gamma.md]");
-		expect(occurrenceCount(block, "[Truncated. Full:")).toBe(3);
+		expect(block).toContain(`${INSTRUCTIONS_FROM}/repo/alpha.md`);
+		expect(block).toContain(`${INSTRUCTIONS_FROM}/repo/beta.md`);
+		expect(block).toContain(`${INSTRUCTIONS_FROM}/repo/gamma.md`);
+		expect(block).toContain("alpha-");
+		expect(block).toContain("beta-");
+		expect(block).toContain("gamma-");
+		expect(block).not.toContain("A".repeat(400));
+		expect(block).not.toContain("B".repeat(400));
+		expect(block).not.toContain("C".repeat(400));
 	});
 
 	it("#given no matching rules #when formatting hook context #then it emits no context", () => {

package/packages/omo-codex/plugin/components/rules/test/hephaestus-model-variant.test.ts

@@ -1,10 +1,9 @@
 import { mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
+import { findPluginBundledCandidates } from "@oh-my-opencode/rules-engine/engine";
 import { afterEach, describe, expect, it } from "vitest";
-
 import { type CodexSessionStartInput, runSessionStartHook } from "../src/codex-hook.js";
-import { findPluginBundledCandidates } from "@oh-my-opencode/rules-engine/engine";
 
 const GPT_55_VARIANT_PATH = "bundled-rules/hephaestus/gpt-5.5.md";
 const GPT_56_VARIANT_PATH = "bundled-rules/hephaestus/gpt-5.6.md";

package/packages/omo-codex/plugin/components/rules/test/hook-output.test.ts

@@ -5,14 +5,14 @@ import { formatAdditionalContextOutput } from "../src/hook-output.js";
 describe("formatAdditionalContextOutput", () => {
 	it("#given context with outer whitespace and CRLF #when serializing hook JSON #then additional context is newline-normalized", () => {
 		// given
-		const context = "\r\n\r\nFirst line\r\nSecond line\rThird line\r\n";
+		const context = "\r\n\r\nA1\r\nB2\rC3\r\n";
 
 		// when
 		const output = formatAdditionalContextOutput("PostToolUse", context);
 		const parsed: unknown = JSON.parse(output);
 
 		// then
-		expect(readAdditionalContext(parsed)).toBe("First line\nSecond line\nThird line");
+		expect(readAdditionalContext(parsed)).toBe("A1\nB2\nC3");
 		expect(output.endsWith("\n")).toBe(true);
 	});
 
@@ -40,7 +40,6 @@ describe("formatAdditionalContextOutput", () => {
 		expect(additionalContext.length).toBeLessThanOrEqual(32_000);
 		expect(additionalContext).toContain("first");
 		expect(additionalContext).not.toContain("last");
-		expect(additionalContext).toContain("[Truncated hook additional context to 32000 chars");
 	});
 });
 

package/packages/omo-codex/plugin/components/rules/test/matcher.test.ts

@@ -1,7 +1,6 @@
-import { describe, expect, it } from "vitest";
-
-import { matchRule, normalizeGlobs } from "@oh-my-opencode/rules-engine/engine";
 import type { RuleFrontmatter } from "@oh-my-opencode/rules-engine/engine";
+import { matchRule, normalizeGlobs } from "@oh-my-opencode/rules-engine/engine";
+import { describe, expect, it } from "vitest";
 
 function matchFrontmatter(
 	frontmatter: RuleFrontmatter,

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

@@ -32,7 +32,7 @@ describe("plugin package metadata", () => {
 		// then
 		expect(packageJson.type).toBe("module");
 		expect(packageJson.packageManager).toBe("npm@11.12.1");
-		expect(packageJson.dependencies ?? {}).toEqual({ picomatch: "^4.0.3" });
+		expect(packageJson.dependencies ?? {}).toEqual({ picomatch: "^4.0.7" });
 		expect(packageJson.bin["omo-rules"]).toBe("./dist/cli.js");
 		expect(packageFiles).toContain("bundled-rules");
 		expect(bundledRules).toContain("windows-git-bash.md");

package/packages/omo-codex/plugin/components/rules/test/parser.test.ts

@@ -1,6 +1,5 @@
-import { describe, expect, it } from "vitest";
-
 import { parseRule } from "@oh-my-opencode/rules-engine/engine";
+import { describe, expect, it } from "vitest";
 
 describe("parseRule", () => {
 	it("#given content without frontmatter #when parsing #then body is preserved", () => {

package/packages/omo-codex/plugin/components/rules/test/post-compact-budget.test.ts

@@ -1,10 +1,9 @@
 import { mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import path from "node:path";
+import type { PiRulesConfig } from "@oh-my-opencode/rules-engine/engine";
 import { afterEach, describe, expect, it } from "vitest";
-
 import { withPostCompactBudget } from "../src/post-compact-budget.js";
-import type { PiRulesConfig } from "@oh-my-opencode/rules-engine/engine";
 
 const tempDirectories: string[] = [];
 const CONFIG: PiRulesConfig = {

package/packages/omo-codex/plugin/components/rules/test/rules-engine-consumption.test.ts

@@ -1,15 +1,14 @@
 import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
-import { afterEach, describe, expect, it } from "vitest";
-
 import {
 	findPluginBundledCandidates,
 	matchRule,
 	parseRule,
-	scanRuleFiles,
 	SOURCE_PRIORITY,
+	scanRuleFiles,
 } from "@oh-my-opencode/rules-engine/engine";
+import { afterEach, describe, expect, it } from "vitest";
 
 const tempDirectories: string[] = [];
 
@@ -37,9 +36,7 @@ describe("rules-engine package consumption", () => {
 
 		// then
 		expect(SOURCE_PRIORITY.get("plugin-bundled")).toBe(200);
-		expect(nonWindowsCandidates.map((candidate) => candidate.relativePath)).toEqual([
-			"bundled-rules/hephaestus.md",
-		]);
+		expect(nonWindowsCandidates.map((candidate) => candidate.relativePath)).toEqual(["bundled-rules/hephaestus.md"]);
 		expect(windowsCandidates.map((candidate) => candidate.relativePath)).toEqual([
 			"bundled-rules/hephaestus.md",
 			"bundled-rules/windows-git-bash.md",

package/packages/omo-codex/plugin/components/rules/test/scanner.test.ts

@@ -1,9 +1,8 @@
 import { mkdirSync, mkdtempSync, rmSync, symlinkSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
-import { afterEach, describe, expect, it } from "vitest";
-
 import { scanRuleFiles } from "@oh-my-opencode/rules-engine/engine";
+import { afterEach, describe, expect, it } from "vitest";
 
 const tempDirectories: string[] = [];
 

package/packages/omo-codex/plugin/components/rules/test/sources.test.ts

@@ -1,9 +1,6 @@
-import { describe, expect, it } from "vitest";
-
-import { SOURCE_PRIORITY } from "@oh-my-opencode/rules-engine/engine";
-import { defaultConfig } from "@oh-my-opencode/rules-engine/engine";
-import { disabledSourcesFromConfig } from "@oh-my-opencode/rules-engine/engine";
 import type { PiRulesConfig } from "@oh-my-opencode/rules-engine/engine";
+import { defaultConfig, disabledSourcesFromConfig, SOURCE_PRIORITY } from "@oh-my-opencode/rules-engine/engine";
+import { describe, expect, it } from "vitest";
 
 describe("rules source selection", () => {
 	it("#given default config #when disabled sources are derived #then opt-out sources stay disabled", () => {

package/packages/omo-codex/plugin/components/rules/test/windows-git-bash-bundled-rule.test.ts

@@ -1,10 +1,9 @@
 import { mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
+import { findPluginBundledCandidates } from "@oh-my-opencode/rules-engine/engine";
 import { afterEach, describe, expect, it } from "vitest";
-
 import { type CodexSessionStartInput, runSessionStartHook } from "../src/codex-hook.js";
-import { findPluginBundledCandidates } from "@oh-my-opencode/rules-engine/engine";
 
 const WINDOWS_RULE_DESCRIPTION = "Windows Git Bash guidance for Codex";
 const WINDOWS_RULE_PATH = "bundled-rules/windows-git-bash.md";
@@ -71,25 +70,11 @@ describe("Windows Git Bash bundled rule content", () => {
 		expect(body).toContain("alwaysApply: true");
 	});
 
-	it('#given the bundled rule #when read #then it NEVER recommends bare shell:"bash"', () => {
-		const body = readWindowsRuleBody();
-
-		expect(body).not.toContain('shell: "bash"');
-	});
-
-	it("#given the bundled rule #when read #then it warns that bare bash often resolves to WSL System32", () => {
-		const body = readWindowsRuleBody();
-
-		expect(body).toMatch(/WSL/i);
-		expect(body).toMatch(/System32/i);
-	});
-
-	it("#given the bundled rule #when read #then it prefers the git_bash MCP and the absolute Git Bash path", () => {
+	it("#given the bundled rule #when read #then it names the Git Bash runtime identifiers", () => {
 		const body = readWindowsRuleBody();
 
 		expect(body).toContain("git_bash");
 		expect(body).toContain("OMO_CODEX_GIT_BASH_PATH");
-		expect(body).toContain("C:\\Program Files\\Git\\bin\\bash.exe");
 	});
 });
 

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

@@ -8,7 +8,7 @@
 						"type": "command",
 						"command": "node \"${PLUGIN_ROOT}/dist/cli.js\" hook post-tool-use",
 						"timeout": 10,
-						"statusMessage": "(OmO 5.0.0-beta.2) Checking Thread Title Hygiene"
+						"statusMessage": "(OmO 5.0.0-beta.21) Checking Thread Title Hygiene"
 					}
 				]
 			}

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

@@ -1,6 +1,6 @@
 {
 	"name": "@sisyphuslabs/codex-teammode",
-	"version": "5.0.0-beta.2",
+	"version": "5.0.0-beta.21",
 	"description": "Codex team-mode hook component that keeps background thread titles descriptive after create_thread.",
 	"type": "module",
 	"private": true,
@@ -15,10 +15,10 @@
 		"typecheck": "tsc --noEmit"
 	},
 	"devDependencies": {
-		"@types/node": "^25.9.3",
-		"bun-types": "^1.3.1",
-		"typescript": "^6.0.3",
-		"vitest": "^4.1.8"
+		"@types/node": "^26.2.0",
+		"bun-types": "^1.4.0",
+		"typescript": "^7.0.2",
+		"vitest": "^4.1.11"
 	},
 	"engines": {
 		"node": ">=20.0.0"

package/packages/omo-codex/plugin/components/teammode/test/thread-title-hook.test.ts

@@ -29,9 +29,7 @@ describe("thread title PostToolUse guidance", () => {
 		expect(isHookOutput(parsed)).toBe(true);
 		if (!isHookOutput(parsed)) return;
 		expect(parsed.hookSpecificOutput.hookEventName).toBe("PostToolUse");
-		expect(parsed.hookSpecificOutput.additionalContext).toBe(
-			"THREAD ID thread-123: CALL codex_app.set_thread_title NOW. USE THE REAL TASK/ROLE.",
-		);
+		expect(parsed.hookSpecificOutput.additionalContext).toContain("thread-123");
 	});
 
 	it("#given Codex reports create_thread output as a JSON string #when the hook runs #then it still extracts the thread id", () => {
@@ -59,9 +57,7 @@ describe("thread title PostToolUse guidance", () => {
 		// then
 		expect(isHookOutput(parsed)).toBe(true);
 		if (!isHookOutput(parsed)) return;
-		expect(parsed.hookSpecificOutput.additionalContext).toBe(
-			"THREAD ID 019ef350-ee78-72a3-bd5e-e40cebc3d814: CALL codex_app.set_thread_title NOW. USE THE REAL TASK/ROLE.",
-		);
+		expect(parsed.hookSpecificOutput.additionalContext).toContain("019ef350-ee78-72a3-bd5e-e40cebc3d814");
 	});
 
 	it("#given an unrelated tool completed #when the hook runs #then it stays silent", () => {
@@ -141,9 +137,7 @@ describe("thread title PostToolUse guidance", () => {
 		// then
 		expect(isHookOutput(parsed)).toBe(true);
 		if (!isHookOutput(parsed)) return;
-		expect(parsed.hookSpecificOutput.additionalContext).toBe(
-			"PENDING WORKTREE ID remote-control:env:test-worktree: WORKTREE THREAD IS NOT READY YET. DO NOT bind-thread OR SEND THE MEMBER BOOTSTRAP UNTIL A REAL THREAD ID EXISTS. THEN CALL codex_app.set_thread_title USING THE REAL TASK/ROLE.",
-		);
+		expect(parsed.hookSpecificOutput.additionalContext).toContain("remote-control:env:test-worktree");
 	});
 });
 

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

@@ -10,6 +10,14 @@ Conventions for human contributors and AI agents working on this component.
 - Tabs for indentation. Double quotes for strings.
 - Tests use vitest with `#given .. #when .. #then` descriptions or plain `// given / // when / // then` body comments.
 
+## Layout
+
+- `src/cli.ts`: SessionStart hook CLI (stdin JSON in, empty stdout out).
+- `src/codex-hook.ts`: `runSessionStartHook` — the single hook handler.
+- `src/posthog.ts`: client construction/capture/shutdown; `DEFAULT_POSTHOG_API_KEY`, `DEFAULT_POSTHOG_HOST`, `getComponentVersion`.
+- `src/product-identity.ts`: the constants that must stay byte-equivalent with `packages/omo-codex/src/telemetry/product-identity.ts`.
+- `hooks/hooks.json`: component-local wiring (twin of the aggregate hook JSON).
+
 ## Commands
 
 - `npm install` - install dependencies.

package/packages/omo-codex/plugin/components/telemetry/biome.json

@@ -1,9 +1,9 @@
 {
-	"$schema": "https://biomejs.dev/schemas/2.4.15/schema.json",
+	"$schema": "https://biomejs.dev/schemas/2.5.9/schema.json",
 	"linter": {
 		"enabled": true,
 		"rules": {
-			"recommended": true,
+			"preset": "recommended",
 			"style": {
 				"noDefaultExport": "error",
 				"noEnum": "error",