oh-my-opencode 5.0.0-beta.1 → 5.0.0-beta.10

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

@@ -8,13 +8,18 @@ const FORMAT_OPTIONS = {
 	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 +30,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 +129,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 +146,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/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/windows-git-bash-bundled-rule.test.ts

@@ -71,25 +71,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/start-work-continuation/AGENTS.md

@@ -31,9 +31,10 @@ Conventions for human contributors and AI agents working on this repository.
 
 ## Layout
 
-- `src/boulder-reader.ts`: reads `.omo/boulder.json`, resolves the active work for the session, re-exports `getPlanChecklist`/`PlanChecklist` from `plan-checklist.ts`. `readContinuationState` returns null only when the plan has no readable top-level checklist (`total === 0`).
+- `src/boulder-reader.ts`: reads `.omo/boulder.json`, resolves the active work for the session, re-exports `getPlanChecklist`/`PlanChecklist` from `plan-checklist.ts`. `readContinuationState` returns null in four cases: `boulder.json` is missing or unparseable, no work matches the session, the work status is not continuable (only `active` and `paused` continue - `completed`/`abandoned` stop), or `checklist.total === 0`.
 - `src/plan-checklist.ts`: `PlanChecklist` (`completed`/`remaining`/`total`/`nextTaskLabel`) and `getPlanChecklist`/`parsePlanChecklist`. Counts structured `## TODOs` rows (`N. <title>`) and `## Final Verification Wave` rows (`F<number>. <title>`); falls back to simple top-level `- [ ]`/`- [x]` checkboxes. Skips fenced blocks and respects `#`/`##` section boundaries.
-- `src/codex-hook.ts`: Stop/SubagentStop hook; fills `REMAINING_COUNT`/`TOTAL_COUNT`/`NEXT_TASK_LABEL` from the checklist into `directive.md`.
+- `src/codex-hook.ts`: Stop/SubagentStop hook; fills nine placeholders into `directive.md` - `PLAN_NAME`, `PLAN_PATH`, `BOULDER_PATH`, `REMAINING_COUNT`, `TOTAL_COUNT`, `NEXT_TASK_LABEL`, `WORKTREE_BLOCK`, `LEDGER_PATH`, `SESSION_ID`. `WORKTREE_BLOCK` renders empty when the work has no worktree. When `remaining === 0`, `nextTaskLabel` is null and renders as "none (final gate pending)".
+- **Context-pressure suppression:** the hook reads `input.transcript_path` through the injected `ReadonlyFileSystem` and returns `""` (no continuation) when the transcript carries any context-pressure marker (`context compacted`, `context_length_exceeded`, `context_too_large`, `codex ran out of room in the model's context window`, and related phrasings). This is the safety valve against an infinite continuation loop once the context window is exhausted; it is pinned by a `#given context-window pressure` test.
 - `directive.md`: directive template with placeholders, applied per invocation.
 
 ## Build and Hooks

package/packages/omo-codex/plugin/components/start-work-continuation/hooks/hooks.json

@@ -7,7 +7,7 @@
 						"type": "command",
 						"command": "node \"${PLUGIN_ROOT}/components/start-work-continuation/dist/cli.js\" hook stop",
 						"timeout": 10,
-						"statusMessage": "(OmO 5.0.0-beta.1) Checking Start-Work Continuation"
+						"statusMessage": "(OmO 5.0.0-beta.10) Checking Start-Work Continuation"
 					}
 				]
 			}
@@ -19,7 +19,7 @@
 						"type": "command",
 						"command": "node \"${PLUGIN_ROOT}/components/start-work-continuation/dist/cli.js\" hook subagent-stop",
 						"timeout": 10,
-						"statusMessage": "(OmO 5.0.0-beta.1) Checking Start-Work Continuation"
+						"statusMessage": "(OmO 5.0.0-beta.10) Checking Start-Work Continuation"
 					}
 				]
 			}

package/packages/omo-codex/plugin/components/start-work-continuation/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@code-yeongyu/codex-start-work-continuation",
-	"version": "5.0.0-beta.1",
+	"version": "5.0.0-beta.10",
 	"description": "Codex Stop hook continuation injector for omo-codex start-work plans.",
 	"type": "module",
 	"packageManager": "npm@11.12.1",

package/packages/omo-codex/plugin/components/start-work-continuation/test/cli.test.ts

@@ -26,9 +26,6 @@ describe("start-work continuation CLI", () => {
 		expect(result.status).toBe(0);
 		const output = parseStopHookOutput(result.stdout);
 		expect(output.decision).toBe("block");
-		expect(output.reason).toContain("Remaining top-level checkboxes: `2` of `4`");
-		expect(output.reason).toContain("Next incomplete task: `1. Implement checklist parser parity`");
-		expect(output.reason).toContain("Your session id in boulder.json: `codex:s1`");
 	});
 
 	it("#given valid SubagentStop stdin #when CLI runs #then stdout contains block JSON", () => {

package/packages/omo-codex/plugin/components/start-work-continuation/test/codex-hook.test.ts

@@ -58,16 +58,7 @@ describe("start-work Stop hook", () => {
 		const output = runStopHook(createStopInput(workspace), fs);
 
 		// then
-		const parsed = parseBlockOutput(output);
-		expect(parsed.decision).toBe("block");
-		expect(parsed.reason).toContain("- Plan: `launch-plan`");
-		expect(parsed.reason).toContain(`- Plan file: \`${join(workspace, ".omo", "plans", "plan.md")}\``);
-		expect(parsed.reason).toContain(`- Boulder state: \`${join(workspace, ".omo", "boulder.json")}\``);
-		expect(parsed.reason).toContain("- Remaining top-level checkboxes: `2` of `4`");
-		expect(parsed.reason).toContain("- Next incomplete task: `1. Implement checklist parser parity`");
-		expect(parsed.reason).toContain("- Worktree: `/tmp/worktree`");
-		expect(parsed.reason).toContain(`- Ledger: \`${join(workspace, ".omo", "start-work", "ledger.jsonl")}\``);
-		expect(parsed.reason).toContain("- Your session id in boulder.json: `codex:sess_abc`");
+		expect(parseBlockOutput(output).decision).toBe("block");
 	});
 
 	it("#given active codex work and an external-blocker marker #when hook runs #then returns empty output", () => {
@@ -187,12 +178,7 @@ describe("start-work Stop hook", () => {
 		const output = runStopHook(createStopInput(workspace), fs);
 
 		// then
-		const parsed = parseBlockOutput(output);
-		expect(parsed.decision).toBe("block");
-		expect(parsed.reason).toContain("- Remaining top-level checkboxes: `0` of `1`");
-		expect(parsed.reason).toContain("- Next incomplete task: `none (final gate pending)`");
-		expect(parsed.reason).toContain("When the remaining count is `0`, skip checkbox execution");
-		expect(parsed.reason).toContain("re-read the ledger record and verify the exact lane/SHA pair");
+		expect(parseBlockOutput(output).decision).toBe("block");
 	});
 
 	it("#given context-window pressure in transcript #when hook runs #then it does not inject continuation text", () => {

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.1) Checking Thread Title Hygiene"
+						"statusMessage": "(OmO 5.0.0-beta.10) 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.1",
+	"version": "5.0.0-beta.10",
 	"description": "Codex team-mode hook component that keeps background thread titles descriptive after create_thread.",
 	"type": "module",
 	"private": true,

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/dist/cli.js

@@ -174,6 +174,7 @@ function writePostHogActivityState(stateDir, nextState, diagnostics) {
 // ../../telemetry-core/src/constants.ts
 var DEFAULT_POSTHOG_HOST = "https://us.i.posthog.com";
 var DEFAULT_POSTHOG_API_KEY = "phc_CFJhj5HyvA62QPhvyaUCtaq23aUfznnijg5VaaGkNk74";
+var UNCONFIGURED_POSTHOG_API_KEY = "phc_REPLACE_ME_OMO_NATIVE";
 
 // ../../telemetry-core/src/diagnostics.ts
 import { appendFileSync, existsSync as existsSync2, mkdirSync as mkdirSync3, readFileSync as readFileSync2 } from "node:fs";
@@ -310,6 +311,9 @@ function shouldDisableTelemetry(input) {
   const env = input.env ?? process.env;
   const globalPrefix = input.globalEnvPrefix ?? "OMO";
   const prefixes = Array.from(new Set([globalPrefix, input.productEnvPrefix]));
+  if (isDisableFlag(env["DO_NOT_TRACK"])) {
+    return true;
+  }
   for (const prefix of prefixes) {
     if (isDisableFlag(env[`${prefix}_DISABLE_POSTHOG`])) {
       return true;
@@ -323,18 +327,15 @@ function shouldDisableTelemetry(input) {
 function getTelemetryApiKey(env = process.env, defaultApiKey = DEFAULT_POSTHOG_API_KEY) {
   return env["POSTHOG_API_KEY"]?.trim() ?? defaultApiKey;
 }
-function getTelemetryHost(env = process.env, defaultHost = DEFAULT_POSTHOG_HOST) {
-  return env["POSTHOG_HOST"]?.trim() || defaultHost;
+function isConfiguredTelemetryApiKey(apiKey) {
+  const normalized = apiKey.trim();
+  return normalized.length > 0 && normalized !== UNCONFIGURED_POSTHOG_API_KEY;
 }
-
-// ../../telemetry-core/src/machine-id.ts
-import { createHash } from "node:crypto";
-import os2 from "node:os";
-function getDefaultTelemetryOsProvider() {
-  return os2;
+function hasTelemetryApiKey(env, defaultApiKey) {
+  return isConfiguredTelemetryApiKey(getTelemetryApiKey(env, defaultApiKey));
 }
-function getTelemetryDistinctId(machineIdPrefix, osProvider = getDefaultTelemetryOsProvider()) {
-  return createHash("sha256").update(`${machineIdPrefix}${osProvider.hostname()}`).digest("hex");
+function getTelemetryHost(env = process.env, defaultHost = DEFAULT_POSTHOG_HOST) {
+  return env["POSTHOG_HOST"]?.trim() || defaultHost;
 }
 
 // ../../../node_modules/.bun/posthog-node@5.35.12/node_modules/posthog-node/dist/extensions/error-tracking/modifiers/module.node.mjs
@@ -5504,6 +5505,16 @@ class PostHog extends PostHogBackendClient {
   }
 }
 
+// ../../telemetry-core/src/machine-id.ts
+import { createHash } from "node:crypto";
+import os2 from "node:os";
+function getDefaultTelemetryOsProvider() {
+  return os2;
+}
+function getTelemetryDistinctId(machineIdPrefix, osProvider = getDefaultTelemetryOsProvider()) {
+  return createHash("sha256").update(`${machineIdPrefix}${osProvider.hostname()}`).digest("hex");
+}
+
 // ../../telemetry-core/src/posthog-client.ts
 var NO_OP_CLIENT = {
   enabled: false,
@@ -5538,7 +5549,7 @@ function createDefaultPostHogTransport(apiKey, options) {
 }
 function isTelemetryClientEnabled(input) {
   const env = input.env ?? process.env;
-  return !shouldDisableTelemetry({ env, productEnvPrefix: input.product.productEnvPrefix }) && getTelemetryApiKey(env, input.product.defaultApiKey).length > 0;
+  return !shouldDisableTelemetry({ env, productEnvPrefix: input.product.productEnvPrefix }) && hasTelemetryApiKey(env, input.product.defaultApiKey);
 }
 function createTelemetryClient(input) {
   if (!isTelemetryClientEnabled(input)) {
@@ -5604,7 +5615,8 @@ function createTransport(input) {
       flushAt: 1,
       flushInterval: 0,
       host: getTelemetryHost(env, input.product.defaultHost),
-      disableGeoip: false
+      disableGeoip: input.product.disableGeoip ?? false,
+      ...input.product.transportOptions
     });
   } catch (error) {
     input.diagnostics?.({

package/packages/omo-codex/plugin/components/telemetry/dist/posthog.js

@@ -170,6 +170,7 @@ function writePostHogActivityState(stateDir, nextState, diagnostics) {
 // ../../../../telemetry-core/src/constants.ts
 var DEFAULT_POSTHOG_HOST = "https://us.i.posthog.com";
 var DEFAULT_POSTHOG_API_KEY = "phc_CFJhj5HyvA62QPhvyaUCtaq23aUfznnijg5VaaGkNk74";
+var UNCONFIGURED_POSTHOG_API_KEY = "phc_REPLACE_ME_OMO_NATIVE";
 
 // ../../../../telemetry-core/src/diagnostics.ts
 import { appendFileSync, existsSync as existsSync2, mkdirSync as mkdirSync3, readFileSync as readFileSync2 } from "node:fs";
@@ -306,6 +307,9 @@ function shouldDisableTelemetry(input) {
   const env = input.env ?? process.env;
   const globalPrefix = input.globalEnvPrefix ?? "OMO";
   const prefixes = Array.from(new Set([globalPrefix, input.productEnvPrefix]));
+  if (isDisableFlag(env["DO_NOT_TRACK"])) {
+    return true;
+  }
   for (const prefix of prefixes) {
     if (isDisableFlag(env[`${prefix}_DISABLE_POSTHOG`])) {
       return true;
@@ -319,18 +323,15 @@ function shouldDisableTelemetry(input) {
 function getTelemetryApiKey(env = process.env, defaultApiKey = DEFAULT_POSTHOG_API_KEY) {
   return env["POSTHOG_API_KEY"]?.trim() ?? defaultApiKey;
 }
-function getTelemetryHost(env = process.env, defaultHost = DEFAULT_POSTHOG_HOST) {
-  return env["POSTHOG_HOST"]?.trim() || defaultHost;
+function isConfiguredTelemetryApiKey(apiKey) {
+  const normalized = apiKey.trim();
+  return normalized.length > 0 && normalized !== UNCONFIGURED_POSTHOG_API_KEY;
 }
-
-// ../../../../telemetry-core/src/machine-id.ts
-import { createHash } from "node:crypto";
-import os2 from "node:os";
-function getDefaultTelemetryOsProvider() {
-  return os2;
+function hasTelemetryApiKey(env, defaultApiKey) {
+  return isConfiguredTelemetryApiKey(getTelemetryApiKey(env, defaultApiKey));
 }
-function getTelemetryDistinctId(machineIdPrefix, osProvider = getDefaultTelemetryOsProvider()) {
-  return createHash("sha256").update(`${machineIdPrefix}${osProvider.hostname()}`).digest("hex");
+function getTelemetryHost(env = process.env, defaultHost = DEFAULT_POSTHOG_HOST) {
+  return env["POSTHOG_HOST"]?.trim() || defaultHost;
 }
 
 // ../../../../../node_modules/.bun/posthog-node@5.35.12/node_modules/posthog-node/dist/extensions/error-tracking/modifiers/module.node.mjs
@@ -5500,6 +5501,16 @@ class PostHog extends PostHogBackendClient {
   }
 }
 
+// ../../../../telemetry-core/src/machine-id.ts
+import { createHash } from "node:crypto";
+import os2 from "node:os";
+function getDefaultTelemetryOsProvider() {
+  return os2;
+}
+function getTelemetryDistinctId(machineIdPrefix, osProvider = getDefaultTelemetryOsProvider()) {
+  return createHash("sha256").update(`${machineIdPrefix}${osProvider.hostname()}`).digest("hex");
+}
+
 // ../../../../telemetry-core/src/posthog-client.ts
 var NO_OP_CLIENT = {
   enabled: false,
@@ -5534,7 +5545,7 @@ function createDefaultPostHogTransport(apiKey, options) {
 }
 function isTelemetryClientEnabled(input) {
   const env = input.env ?? process.env;
-  return !shouldDisableTelemetry({ env, productEnvPrefix: input.product.productEnvPrefix }) && getTelemetryApiKey(env, input.product.defaultApiKey).length > 0;
+  return !shouldDisableTelemetry({ env, productEnvPrefix: input.product.productEnvPrefix }) && hasTelemetryApiKey(env, input.product.defaultApiKey);
 }
 function createTelemetryClient(input) {
   if (!isTelemetryClientEnabled(input)) {
@@ -5600,7 +5611,8 @@ function createTransport(input) {
       flushAt: 1,
       flushInterval: 0,
       host: getTelemetryHost(env, input.product.defaultHost),
-      disableGeoip: false
+      disableGeoip: input.product.disableGeoip ?? false,
+      ...input.product.transportOptions
     });
   } catch (error) {
     input.diagnostics?.({

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

@@ -7,7 +7,7 @@
 						"type": "command",
 						"command": "node \"${PLUGIN_ROOT}/dist/cli.js\" hook session-start",
 						"timeout": 5,
-						"statusMessage": "(OmO 5.0.0-beta.1) Recording Session Telemetry"
+						"statusMessage": "(OmO 5.0.0-beta.10) Recording Session Telemetry"
 					}
 				]
 			}

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

@@ -1,6 +1,6 @@
 {
 	"name": "@code-yeongyu/codex-telemetry",
-	"version": "5.0.0-beta.1",
+	"version": "5.0.0-beta.10",
 	"description": "Codex plugin component that emits omo-codex anonymous daily-active telemetry on SessionStart.",
 	"type": "module",
 	"packageManager": "npm@11.12.1",

package/packages/omo-codex/plugin/components/ultrawork/directive.md

@@ -126,6 +126,12 @@ exactly `objective`; do not include `status`. Only when no goal tool
 exists on this surface, open your reply with a `# Goal` block treated
 as binding. Goals are unlimited; never invent a numeric budget or
 limit.
+Check `get_goal` first: continue a matching active goal instead of
+duplicating one; surface a conflicting one. Write the objective
+outcome-first: the concrete thing that will be TRUE when done (an
+outcome, never an activity), the named deliverable surfaces, and
+explicit scope bounds — a vague objective produces vague criteria,
+and vague criteria cannot be proven.
 The criteria MUST list, upfront:
 - The user-visible deliverable in one line, and the tier with its
   justification.

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

@@ -7,7 +7,7 @@
 						"type": "command",
 						"command": "node \"${PLUGIN_ROOT}/dist/cli.js\" hook user-prompt-submit",
 						"timeout": 5,
-						"statusMessage": "(OmO 5.0.0-beta.1) Checking Ultrawork Trigger"
+						"statusMessage": "(OmO 5.0.0-beta.10) Checking Ultrawork Trigger"
 					}
 				]
 			}

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

@@ -1,6 +1,6 @@
 {
 	"name": "@code-yeongyu/codex-ultrawork",
-	"version": "5.0.0-beta.1",
+	"version": "5.0.0-beta.10",
 	"description": "Codex plugin that injects the ultrawork orchestration directive and ships LazyCodex planning, review, QA, and gate agent roles.",
 	"type": "module",
 	"packageManager": "npm@11.12.1",

package/packages/omo-codex/plugin/components/ultrawork/skills/ultrawork/SKILL.md

@@ -133,6 +133,12 @@ exactly `objective`; do not include `status`. Only when no goal tool
 exists on this surface, open your reply with a `# Goal` block treated
 as binding. Goals are unlimited; never invent a numeric budget or
 limit.
+Check `get_goal` first: continue a matching active goal instead of
+duplicating one; surface a conflicting one. Write the objective
+outcome-first: the concrete thing that will be TRUE when done (an
+outcome, never an activity), the named deliverable surfaces, and
+explicit scope bounds — a vague objective produces vague criteria,
+and vague criteria cannot be proven.
 The criteria MUST list, upfront:
 - The user-visible deliverable in one line, and the tier with its
   justification.

package/packages/omo-codex/plugin/components/ultrawork/skills/ulw-plan/SKILL.md

@@ -34,7 +34,7 @@ Example opening (adapt the wording, keep every commitment):
 
 ## INTENT ROUTING - pick ONE intent reference
 
-**Review modifiers are a gate trigger, not a style cue.** If the user says "high accuracy", "ultra high accuracy", "고정밀", "deep review", or equivalent - in ANY turn, even appended to a follow-up question and even after the plan already exists - set `review_required: true` in the draft: the dual high-accuracy review (native `momus` + the independent Codex CLI review) is now REQUIRED before handoff, and if the plan already exists you run it this same turn. Answering the current question more carefully does NOT satisfy it. This does NOT choose CLEAR/UNCLEAR and does NOT suppress interview.
+**Review modifiers are a gate trigger, not a style cue.** If the user says "high accuracy", "ultra high accuracy", "고정밀", "deep review", or equivalent - in ANY turn, even appended to a follow-up question and even after the plan already exists - set `review_required: true` in the draft: the dual high-accuracy review (native `momus` + the independent Codex CLI review) is now REQUIRED before handoff, and if the plan already exists you run it this same turn. The review runs under the bounded convergence contract in `full-workflow.md`: a 5-round cap (unlimited only on explicit user request), evidence-backed blocker eligibility, and approval-with-notes counting as approval. Answering the current question more carefully does NOT satisfy it. This does NOT choose CLEAR/UNCLEAR and does NOT suppress interview.
 
 After grounding, make ONE judgment, record `intent: clear|unclear` plus `review_required`, **ANNOUNCE both to the user in one line**, then load ONE intent reference (you ALSO read `references/full-workflow.md` for the shared mechanics - see below). The test keys on whether the desired **OUTCOME** is clear, NOT on request length. This verdict line and the opening announcement above are the two mandatory user-visible signals of a planning session - it tells the user whether they will be interviewed and whether high-accuracy review is already requested; never skip either.
 
@@ -72,7 +72,7 @@ When producing the plan, encode every executable item as a column-zero Markdown
 - **Full scope is the default.** Plan the ENTIRE request; "MVP", "v1", "phase 1", or any reduced subset is never an option you invent or ask about - it exists only if the user introduces it. Scope OUT / Must-NOT-Have entries are guardrails against unrequested additions, never reductions of the request.
 - **Explore before asking.** Discoverable facts (repo/system/docs truth) -> research and cite, never ask. Preferences/tradeoffs -> the only things you bring to the user. When unsure which, treat it as a user-decision.
 - **CodeGraph first when present.** Use `codegraph_explore` for repo how/where/what/flow questions before wider reads; if codegraph_* tools are absent, inactive/uninitialized, or cold-start unavailable, continue with Read/Grep/Glob/LSP and the ast-grep skill.
-- **Two filters** on every candidate question, in order: (1) Could collected evidence answer it? -> explore instead. (2) Could the user's stated intent plus a defensible default answer it? -> adopt the default, record it, do not ask - UNLESS it is an owner-decision, which always survives as a question even when a default exists: anything irreversible / destructive / safety-critical, or a cross-cutting product choice the user lives with (public config surface, distribution / packaging, external dependency or pinned SHA, data / schema shape). Default the reversible internals; surface the owner-decisions.
+- **Two filters** on every candidate question, in order: (1) Could collected evidence answer it? -> explore instead. (2) Could the user's stated intent plus a defensible default answer it? -> adopt the default, record it, do not ask - UNLESS it is an owner-decision, which always survives as a question even when a default exists: anything irreversible / destructive / safety-critical, or a cross-cutting product choice the user lives with (public config surface, distribution / packaging, external dependency or pinned SHA, data / schema shape, real budget / paid-service spend, expected scale or capacity target, target-audience / compliance limits). Extrinsic constraints (budget, mandated stack, scale, audience) leave no repo evidence, so exploration can never surface them - sweep those axes explicitly once per plan and classify each as explored, defaulted (ledger), or asked. Default the reversible internals; surface the owner-decisions.
 - **Explore to sufficiency, then STOP.** One research wave per open question; stop when the clearance check is answerable; never re-explore to double-check.
 - **Parallel-dispatch** independent research in ONE turn and keep working while it runs. Subagent outputs are CLAIMS until you independently verify them.
 - **Approval is not execution.** Approval authorizes writing the plan ONLY, never implementation. ONE request -> ONE plan, however large.