oh-my-opencode 4.18.2 → 4.19.1

package/packages/omo-codex/plugin/components/start-work-continuation/src/plan-checklist.ts

@@ -0,0 +1,171 @@
+import { existsSync, readFileSync } from "node:fs";
+
+export type PlanChecklist = {
+	readonly completed: number;
+	readonly remaining: number;
+	readonly total: number;
+	readonly nextTaskLabel: string | null;
+};
+
+const TODO_HEADING_PATTERN = /^##[ \t]+TODOs(?:[ \t]+#+)?[ \t]*$/i;
+const FINAL_VERIFICATION_HEADING_PATTERN = /^##[ \t]+Final Verification Wave(?:[ \t]+#+)?[ \t]*$/i;
+const SECTION_BOUNDARY_HEADING_PATTERN = /^#{1,2}(?:[ \t]+|$)/;
+const FENCE_PATTERN = /^[ \t]{0,3}(`{3,}|~{3,})(.*)$/;
+const SIMPLE_CHECKBOX_PATTERN = /^[-*][ \t]*\[[ \t]*([xX]?)[ \t]*\][ \t]+(.+)$/;
+const TODO_CHECKBOX_PATTERN = /^- \[([ xX])\] ([1-9]\d*\. .+)$/;
+const FINAL_WAVE_CHECKBOX_PATTERN = /^- \[([ xX])\] (F[1-9]\d*\. .+)$/i;
+
+type ChecklistSection = "todo" | "final-wave" | "other";
+
+type ParsedCheckbox = {
+	readonly checked: boolean;
+	readonly label: string;
+};
+
+type MarkdownFence = {
+	readonly marker: "`" | "~";
+	readonly length: number;
+};
+
+export function getPlanChecklist(planPath: string): PlanChecklist {
+	if (!existsSync(planPath)) return emptyChecklist();
+
+	try {
+		return parsePlanChecklist(readFileSync(planPath, "utf8"));
+	} catch (error) {
+		if (error instanceof Error) return emptyChecklist();
+		throw error;
+	}
+}
+
+export function parsePlanChecklist(markdown: string): PlanChecklist {
+	const lines = markdown.split(/\r?\n/);
+	if (!hasStructuredSection(lines)) return parseSimpleChecklist(lines);
+
+	let completed = 0;
+	let remaining = 0;
+	let nextTaskLabel: string | null = null;
+	let section: ChecklistSection = "other";
+	let fence: MarkdownFence | null = null;
+
+	for (const line of lines) {
+		if (fence !== null) {
+			if (isClosingFence(line, fence)) fence = null;
+			continue;
+		}
+		const openingFence = parseOpeningFence(line);
+		if (openingFence !== null) {
+			fence = openingFence;
+			continue;
+		}
+
+		if (SECTION_BOUNDARY_HEADING_PATTERN.test(line)) {
+			section = parseStructuredSectionHeading(line);
+			continue;
+		}
+		if (section === "other") continue;
+
+		const checkbox = parseStructuredCheckbox(line, section);
+		if (checkbox === null) continue;
+		if (checkbox.checked) completed += 1;
+		else {
+			remaining += 1;
+			nextTaskLabel = nextTaskLabel ?? checkbox.label;
+		}
+	}
+
+	return { completed, remaining, total: completed + remaining, nextTaskLabel };
+}
+
+function hasStructuredSection(lines: readonly string[]): boolean {
+	let fence: MarkdownFence | null = null;
+	for (const line of lines) {
+		if (fence !== null) {
+			if (isClosingFence(line, fence)) fence = null;
+			continue;
+		}
+		const openingFence = parseOpeningFence(line);
+		if (openingFence !== null) {
+			fence = openingFence;
+			continue;
+		}
+		if (parseStructuredSectionHeading(line) !== "other") return true;
+	}
+	return false;
+}
+
+function parseSimpleChecklist(lines: readonly string[]): PlanChecklist {
+	let completed = 0;
+	let remaining = 0;
+	let nextTaskLabel: string | null = null;
+	let fence: MarkdownFence | null = null;
+
+	for (const line of lines) {
+		if (fence !== null) {
+			if (isClosingFence(line, fence)) fence = null;
+			continue;
+		}
+		const openingFence = parseOpeningFence(line);
+		if (openingFence !== null) {
+			fence = openingFence;
+			continue;
+		}
+
+		const checkbox = parseSimpleTopLevelCheckbox(line);
+		if (checkbox === null) continue;
+		if (checkbox.checked) completed += 1;
+		else {
+			remaining += 1;
+			nextTaskLabel = nextTaskLabel ?? checkbox.label;
+		}
+	}
+
+	return { completed, remaining, total: completed + remaining, nextTaskLabel };
+}
+
+function parseStructuredSectionHeading(line: string): ChecklistSection {
+	if (TODO_HEADING_PATTERN.test(line)) return "todo";
+	if (FINAL_VERIFICATION_HEADING_PATTERN.test(line)) return "final-wave";
+	return "other";
+}
+
+function parseStructuredCheckbox(line: string, section: "todo" | "final-wave"): ParsedCheckbox | null {
+	const pattern = section === "todo" ? TODO_CHECKBOX_PATTERN : FINAL_WAVE_CHECKBOX_PATTERN;
+	const match = line.match(pattern);
+	const marker = match?.[1];
+	const label = match?.[2];
+	if (marker === undefined || label === undefined) return null;
+	return { checked: marker.toLowerCase() === "x", label };
+}
+
+function parseSimpleTopLevelCheckbox(line: string): ParsedCheckbox | null {
+	const match = line.match(SIMPLE_CHECKBOX_PATTERN);
+	const marker = match?.[1];
+	const label = match?.[2];
+	if (marker === undefined || label === undefined) return null;
+	return { checked: marker.toLowerCase() === "x", label };
+}
+
+function parseOpeningFence(line: string): MarkdownFence | null {
+	const match = line.match(FENCE_PATTERN);
+	const run = match?.[1];
+	const info = match?.[2];
+	const marker = run?.charAt(0);
+	if (
+		run === undefined ||
+		info === undefined ||
+		(marker !== "`" && marker !== "~") ||
+		(marker === "`" && info.includes("`"))
+	)
+		return null;
+	return { marker, length: run.length };
+}
+
+function isClosingFence(line: string, fence: MarkdownFence): boolean {
+	const run = line.match(/^[ \t]{0,3}(`{3,}|~{3,})[ \t]*$/)?.[1];
+	return run?.charAt(0) === fence.marker && run.length >= fence.length;
+}
+
+function emptyChecklist(): PlanChecklist {
+	return { completed: 0, remaining: 0, total: 0, nextTaskLabel: null };
+}

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

@@ -1,40 +1,44 @@
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
+import { mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { afterEach, describe, expect, it } from "vitest";
 import { getPlanChecklist, readContinuationState } from "../src/boulder-reader.js";
 
 const cleanupRoots: string[] = [];
+const SCAFFOLD_PLAN_MARKDOWN = readFileSync(new URL("./fixtures/plan-scaffold.md", import.meta.url), "utf8");
 
 afterEach(() => {
 	for (const root of cleanupRoots.splice(0)) rmSync(root, { recursive: true, force: true });
 });
 
 describe("start-work plan checklist consumption", () => {
-	it("#given top-level completed and incomplete checkboxes #when parsed #then counts remaining and total", () => {
+	it("#given scaffold headings and canonical numbered rows #when parsed #then structured totals and next label match", () => {
 		// given
-		const planPath = createPlan(
-			["# Plan", "", "## TODOs", "- [ ] First", "- [x] Done", "- [X] Also done", "- [ ] Second"].join("\n"),
-		);
+		const planPath = createPlan(SCAFFOLD_PLAN_MARKDOWN);
 
 		// when
 		const checklist = getPlanChecklist(planPath);
 
 		// then
-		expect(checklist).toEqual({ completed: 2, remaining: 2, total: 4, nextTaskLabel: "First" });
+		expect(checklist).toEqual({
+			completed: 2,
+			remaining: 2,
+			total: 4,
+			nextTaskLabel: "1. Implement checklist parser parity",
+		});
 	});
 
 	it("#given nested checkboxes #when parsed #then ignores non-column-zero items", () => {
 		// given
 		const planPath = createPlan(
-			["## TODOs", "- [ ] Top-level", "  - [ ] Nested", "\t- [ ] Tab nested", "- [x] Complete"].join("\n"),
+			["## TODOs", "- [ ] 1. Top-level", "  - [ ] Nested", "\t- [ ] Tab nested", "- [x] 2. Complete"].join("\n"),
 		);
 
 		// when
 		const checklist = getPlanChecklist(planPath);
 
 		// then
-		expect(checklist).toEqual({ completed: 1, remaining: 1, total: 2, nextTaskLabel: "Top-level" });
+		expect(checklist).toEqual({ completed: 1, remaining: 1, total: 2, nextTaskLabel: "1. Top-level" });
 	});
 
 	it("#given checkboxes outside counted sections #when parsed #then ignores unrelated top-level tasks", () => {
@@ -44,12 +48,12 @@ describe("start-work plan checklist consumption", () => {
 				"# Plan",
 				"- [ ] Preamble task",
 				"## TODOs",
-				"- [ ] Build hook",
+				"- [ ] 1. Build hook",
 				"## Acceptance Criteria",
 				"- [ ] Acceptance item",
 				"## Final Verification Wave",
-				"- [x] Run tests",
-				"- [ ] Run smoke",
+				"- [x] F1. Run tests",
+				"- [ ] F2. Run smoke",
 			].join("\n"),
 		);
 
@@ -57,18 +61,184 @@ describe("start-work plan checklist consumption", () => {
 		const checklist = getPlanChecklist(planPath);
 
 		// then
-		expect(checklist).toEqual({ completed: 1, remaining: 2, total: 3, nextTaskLabel: "Build hook" });
+		expect(checklist).toEqual({ completed: 1, remaining: 2, total: 3, nextTaskLabel: "1. Build hook" });
 	});
 
 	it("#given all top-level tasks complete #when parsed #then next task is null", () => {
 		// given
-		const planPath = createPlan(["## TODOs", "- [x] First", "- [X] Second"].join("\n"));
+		const planPath = createPlan(
+			["## Todos", "- [x] 1. First", "- [X] 2. Second", "## Final verification wave", "- [x] F1. Final"].join("\n"),
+		);
+
+		// when
+		const checklist = getPlanChecklist(planPath);
+
+		// then
+		expect(checklist).toEqual({ completed: 3, remaining: 0, total: 3, nextTaskLabel: null });
+	});
+
+	it("#given no structured headings #when parsed #then legacy top-level checkbox fallback remains", () => {
+		// given
+		const planPath = createPlan(["# Plan", "- [ ] First", "- [x] Done", "  - [ ] Nested"].join("\n"));
+
+		// when
+		const checklist = getPlanChecklist(planPath);
+
+		// then
+		expect(checklist).toEqual({ completed: 1, remaining: 1, total: 2, nextTaskLabel: "First" });
+	});
+
+	it("#given a heading-free legacy star checklist #when parsed #then fallback behavior is preserved", () => {
+		// given
+		const planPath = createPlan(["# Plan", "* [ ] First", "* [x] Done", "  * [ ] Nested"].join("\n"));
+
+		// when
+		const checklist = getPlanChecklist(planPath);
+
+		// then
+		expect(checklist).toEqual({ completed: 1, remaining: 1, total: 2, nextTaskLabel: "First" });
+	});
+
+	it("#given noncanonical structured rows #when parsed #then only exact positive-number grammar is counted", () => {
+		// given
+		const planPath = createPlan(
+			[
+				"## Todos",
+				"- [ ] 0. Zero is invalid",
+				"- [ ] 01. Leading zero is invalid",
+				"* [ ] 2. Star marker is invalid",
+				"-[ ] 3. Missing spaces are invalid",
+				"- [ ] 4. Canonical implementation",
+				"## Final verification wave",
+				"- [ ] F0. Zero final verifier is invalid",
+				"- [ ] F01. Leading zero final verifier is invalid",
+				"- [x] F2. Canonical final verifier",
+			].join("\n"),
+		);
+
+		// when
+		const checklist = getPlanChecklist(planPath);
+
+		// then
+		expect(checklist).toEqual({
+			completed: 1,
+			remaining: 1,
+			total: 2,
+			nextTaskLabel: "4. Canonical implementation",
+		});
+	});
+
+	it("#given fenced examples and a higher-level heading #when parsed #then section scope excludes them", () => {
+		// given
+		const planPath = createPlan(
+			[
+				"## Todos",
+				"- [ ] 1. Counted implementation",
+				"```md",
+				"- [ ] 2. Fenced example",
+				"```",
+				"# Appendix",
+				"- [ ] 3. Appendix checkbox",
+				"## Final verification wave",
+				"- [x] F1. Counted verifier",
+			].join("\n"),
+		);
+
+		// when
+		const checklist = getPlanChecklist(planPath);
+
+		// then
+		expect(checklist).toEqual({
+			completed: 1,
+			remaining: 1,
+			total: 2,
+			nextTaskLabel: "1. Counted implementation",
+		});
+	});
+
+	it("#given a child heading inside TODOs #when parsed #then canonical rows remain in the parent section", () => {
+		// given
+		const planPath = createPlan(["## TODOs", "- [x] 1. First task", "### Notes", "- [ ] 2. Second task"].join("\n"));
+
+		// when
+		const checklist = getPlanChecklist(planPath);
+
+		// then
+		expect(checklist).toEqual({ completed: 1, remaining: 1, total: 2, nextTaskLabel: "2. Second task" });
+	});
+
+	it("#given a four-backtick fence containing triple-backtick examples #when parsed #then shorter fences do not close it", () => {
+		// given
+		const planPath = createPlan(
+			[
+				"## Todos",
+				"- [x] 1. Counted implementation",
+				"````md",
+				"```ts",
+				"- [ ] 2. Fenced example",
+				"```",
+				"````",
+				"## Final verification wave",
+				"- [ ] F1. Counted verifier",
+			].join("\n"),
+		);
+
+		// when
+		const checklist = getPlanChecklist(planPath);
+
+		// then
+		expect(checklist).toEqual({
+			completed: 1,
+			remaining: 1,
+			total: 2,
+			nextTaskLabel: "F1. Counted verifier",
+		});
+	});
+
+	it("#given structured headings with ATX closing markers #when parsed #then canonical tasks remain structured", () => {
+		// given
+		const planPath = createPlan(
+			["## TODOs ##", "- [ ] 1. Implement", "## Final Verification Wave ###", "- [ ] F1. Verify"].join("\n"),
+		);
+
+		// when
+		const checklist = getPlanChecklist(planPath);
+
+		// then
+		expect(checklist).toEqual({
+			completed: 0,
+			remaining: 2,
+			total: 2,
+			nextTaskLabel: "1. Implement",
+		});
+	});
+
+	it("#given inline backtick code before a canonical task #when parsed #then it does not open a fence", () => {
+		// given
+		const planPath = createPlan(["## TODOs", "```example```", "- [ ] 1. Implement"].join("\n"));
+
+		// when
+		const checklist = getPlanChecklist(planPath);
+
+		// then
+		expect(checklist.total).toBe(1);
+		expect(checklist.nextTaskLabel).toBe("1. Implement");
+	});
+
+	it("#given a heading-free fenced checkbox example #when parsed #then legacy fallback ignores it", () => {
+		// given
+		const planPath = createPlan(["````md", "- [ ] 1. Example only", "````"].join("\n"));
 
 		// when
 		const checklist = getPlanChecklist(planPath);
 
 		// then
-		expect(checklist).toEqual({ completed: 2, remaining: 0, total: 2, nextTaskLabel: null });
+		expect(checklist).toEqual({
+			completed: 0,
+			remaining: 0,
+			total: 0,
+			nextTaskLabel: null,
+		});
 	});
 });
 
@@ -77,7 +247,7 @@ describe("start-work boulder state reader", () => {
 		// given
 		const workspace = createWorkspace({
 			boulderJson: createBoulderJson({ status: "active", sessionIds: ["codex:sess_abc"] }),
-			planMarkdown: "# Plan\n\n## TODOs\n- [ ] First\n- [x] Done\n- [ ] Second\n",
+			planMarkdown: SCAFFOLD_PLAN_MARKDOWN,
 		});
 
 		// when
@@ -90,7 +260,12 @@ describe("start-work boulder state reader", () => {
 			boulderPath: join(workspace, ".omo", "boulder.json"),
 			ledgerPath: join(workspace, ".omo", "start-work", "ledger.jsonl"),
 			worktreePath: null,
-			checklist: { completed: 1, remaining: 2, total: 3, nextTaskLabel: "First" },
+			checklist: {
+				completed: 2,
+				remaining: 2,
+				total: 4,
+				nextTaskLabel: "1. Implement checklist parser parity",
+			},
 		});
 	});
 
@@ -98,7 +273,7 @@ describe("start-work boulder state reader", () => {
 		// given
 		const workspace = createWorkspace({
 			boulderJson: createBoulderJson({ status: "completed", sessionIds: ["codex:sess_abc"] }),
-			planMarkdown: "# Plan\n\n## TODOs\n- [ ] First\n",
+			planMarkdown: "# Plan\n\n## TODOs\n- [ ] 1. First\n",
 		});
 
 		// when
@@ -112,7 +287,7 @@ describe("start-work boulder state reader", () => {
 		// given
 		const workspace = createWorkspace({
 			boulderJson: createBoulderJson({ status: "paused", sessionIds: ["codex:sess_abc"] }),
-			planMarkdown: "# Plan\n\n## TODOs\n- [ ] First\n",
+			planMarkdown: "# Plan\n\n## TODOs\n- [ ] 1. First\n",
 		});
 
 		// when
@@ -120,14 +295,14 @@ describe("start-work boulder state reader", () => {
 
 		// then
 		expect(state?.planName).toBe("launch-plan");
-		expect(state?.checklist).toEqual({ completed: 0, remaining: 1, total: 1, nextTaskLabel: "First" });
+		expect(state?.checklist).toEqual({ completed: 0, remaining: 1, total: 1, nextTaskLabel: "1. First" });
 	});
 
 	it("#given active codex work with no remaining checklist items #when state is read #then final gate continuation remains present", () => {
 		// given
 		const workspace = createWorkspace({
 			boulderJson: createBoulderJson({ status: "active", sessionIds: ["codex:sess_abc"] }),
-			planMarkdown: "# Plan\n\n## TODOs\n- [x] First\n",
+			planMarkdown: "# Plan\n\n## TODOs\n- [x] 1. First\n",
 		});
 
 		// when
@@ -155,7 +330,7 @@ describe("start-work boulder state reader", () => {
 		// given
 		const workspace = createWorkspace({
 			boulderJson: "{",
-			planMarkdown: "# Plan\n\n## TODOs\n- [ ] First\n",
+			planMarkdown: "# Plan\n\n## TODOs\n- [ ] 1. First\n",
 		});
 
 		// when
@@ -169,7 +344,7 @@ describe("start-work boulder state reader", () => {
 		// given
 		const workspace = createWorkspace({
 			boulderJson: createBoulderJson({ status: "active", sessionIds: ["sess_abc"] }),
-			planMarkdown: "# Plan\n\n## TODOs\n- [ ] First\n",
+			planMarkdown: "# Plan\n\n## TODOs\n- [ ] 1. First\n",
 		});
 
 		// when
@@ -201,7 +376,7 @@ describe("start-work boulder state reader", () => {
 				started_at: "2026-06-12T00:00:00.000Z",
 				session_ids: ["codex:sess_abc"],
 			}),
-			planMarkdown: "# Plan\n\n## TODOs\n- [ ] First\n",
+			planMarkdown: "# Plan\n\n## TODOs\n- [ ] 1. First\n",
 		});
 
 		// when

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

@@ -6,6 +6,7 @@ import { execPath } from "node:process";
 import { afterEach, describe, expect, it } from "vitest";
 
 const cleanupRoots: string[] = [];
+const SCAFFOLD_PLAN_MARKDOWN = readFileSync(new URL("./fixtures/plan-scaffold.md", import.meta.url), "utf8");
 
 afterEach(() => {
 	for (const root of cleanupRoots.splice(0)) rmSync(root, { recursive: true, force: true });
@@ -25,8 +26,8 @@ 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 `3`");
-		expect(output.reason).toContain("Next incomplete task: `Task one`");
+		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`");
 	});
 
@@ -106,7 +107,7 @@ function createWorkspace(sessionIds: readonly string[]): string {
 	const root = mkdtempSync(join(tmpdir(), "codex-continuation-cli-"));
 	cleanupRoots.push(root);
 	mkdirSync(join(root, ".omo", "plans"), { recursive: true });
-	writeFileSync(join(root, ".omo", "plans", "plan.md"), "## TODOs\n\n- [ ] Task one\n- [x] Done\n- [ ] Task two\n");
+	writeFileSync(join(root, ".omo", "plans", "plan.md"), SCAFFOLD_PLAN_MARKDOWN);
 	const work = {
 		work_id: "w1",
 		active_plan: ".omo/plans/plan.md",

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

@@ -1,4 +1,4 @@
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
+import { mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { afterEach, describe, expect, it } from "vitest";
@@ -8,6 +8,7 @@ import type { ReadonlyFileSystem, StopInput } from "../src/types.js";
 
 const DEFAULT_WORKSPACE = "/repo";
 const cleanupRoots: string[] = [];
+const SCAFFOLD_PLAN_MARKDOWN = readFileSync(new URL("./fixtures/plan-scaffold.md", import.meta.url), "utf8");
 
 afterEach(() => {
 	for (const root of cleanupRoots.splice(0)) rmSync(root, { recursive: true, force: true });
@@ -49,7 +50,7 @@ describe("start-work Stop hook", () => {
 				status: "active",
 				worktreePath: "/tmp/worktree",
 			}),
-			planMarkdown: ["# Plan", "", "## TODOs", "- [ ] First", "- [x] Done", "- [ ] Second"].join("\n"),
+			planMarkdown: SCAFFOLD_PLAN_MARKDOWN,
 		});
 		const fs = createMemoryFs();
 
@@ -62,8 +63,8 @@ describe("start-work Stop hook", () => {
 		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 `3`");
-		expect(parsed.reason).toContain("- Next incomplete task: `First`");
+		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`");
@@ -73,7 +74,7 @@ describe("start-work Stop hook", () => {
 		// given
 		const workspace = createWorkspace({
 			boulderJson: createBoulderJson({ sessionIds: ["codex:sess_abc"], status: "active" }),
-			planMarkdown: ["# Plan", "", "## TODOs", "- [x] Done"].join("\n"),
+			planMarkdown: ["# Plan", "", "## TODOs", "- [x] 1. Done"].join("\n"),
 		});
 		const fs = createMemoryFs();
 

package/packages/omo-codex/plugin/components/start-work-continuation/test/fixtures/plan-scaffold.md

@@ -0,0 +1,15 @@
+# Scaffold Parser Parity
+
+## Todos
+- [ ] 1. Implement checklist parser parity
+  - [ ] Nested acceptance detail
+- [x] 2. Preserve completed implementation rows
+- [ ] Missing numeric prefix must be ignored
+
+## Acceptance Criteria
+- [ ] Outside tracked sections must be ignored
+
+## Final verification wave
+- [ ] F1. Exercise the Codex Stop surface
+- [X] F2. Preserve completed final verification rows
+- [ ] 3. Wrong final-wave label must be ignored

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 4.18.2) Checking Thread Title Hygiene"
+						"statusMessage": "(OmO 4.19.1) Checking Thread Title Hygiene"
 					}
 				]
 			}

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

@@ -1,6 +1,6 @@
 {
 	"name": "@sisyphuslabs/codex-teammode",
-	"version": "4.18.2",
+	"version": "4.19.1",
 	"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/telemetry/hooks/hooks.json

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

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

@@ -1,6 +1,6 @@
 {
 	"name": "@code-yeongyu/codex-telemetry",
-	"version": "4.18.2",
+	"version": "4.19.1",
 	"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/AGENTS.md

@@ -13,19 +13,19 @@ Conventions for human contributors and AI agents working on this repository.
 
 ## Layout
 
-- `src/cli.ts` — `UserPromptSubmit` hook CLI. Reads JSON on stdin, writes a compact `<ultrawork-mode>` skill pointer to stdout when the keyword matches (full directive as fallback), exits 0 otherwise.
-- `src/codex-hook.ts` — pure detector/hook behavior; skips on context-pressure/recovery transcripts and when the directive is already in the transcript.
-- `src/skill-pointer.ts` — `buildUltraworkAdditionalContext()`: emits the <4096-byte pointer (Codex App truncates large hook output) directing the model to `create_goal` then read the bundled `ultrawork` skill; falls back to the full `ULTRAWORK_DIRECTIVE` when the plugin skills tree is absent. Pointer size + `ulw-loop` mirror byte-identity pinned by `plugin/test/ultrawork-skill-pointer.test.mjs`.
-- `directive.md` — bundled ultrawork directive text.
-- `skills/ultrawork/SKILL.md` — full directive as a skill, generated by `scripts/sync-directive.mjs`; pinned byte-for-byte to `directive.md` by `test/directive-source.test.ts`. `plugin/scripts/sync-skills.mjs` materializes it into the aggregate `plugin/skills/` tree the pointer path targets.
-- `skills/ulw-plan/` — dual-maintained copy of the ulw-plan skill (component copy wins over `packages/shared-skills/skills/ulw-plan/` in sync-skills; keep both in step by hand).
-- `agents/*.toml` — bundled Codex agent role files. Installed as regular files into `CODEX_HOME/agents/` by `src/cli/install-codex/link-cached-plugin-agents.ts` at install time. Public `sisyphuslabs` installs source them from Codex's installed-marketplace snapshot, not the versioned plugin cache, so they survive Codex auto-update cache pruning and temporary snapshot cleanup. No runtime `SessionStart` hook is involved.
-- `hooks/hooks.json` — registers the prompt-detector hook only.
-- `.codex-plugin/plugin.json` — Codex plugin manifest. Marketplace metadata lives here, not in `package.json`.
+- `src/cli.ts`: `UserPromptSubmit` hook CLI. Reads JSON on stdin, writes a compact `<ultrawork-mode>` skill pointer to stdout when the keyword matches (full directive as fallback), exits 0 otherwise.
+- `src/codex-hook.ts`: pure detector/hook behavior; skips on context-pressure/recovery transcripts and when the directive is already in the transcript.
+- `src/skill-pointer.ts`: `buildUltraworkAdditionalContext()`: emits the <4096-byte pointer (Codex App truncates large hook output) directing the model to `create_goal` then read the bundled `ultrawork` skill; falls back to the full `ULTRAWORK_DIRECTIVE` when the plugin skills tree is absent. Pointer size + `ulw-loop` mirror byte-identity pinned by `plugin/test/ultrawork-skill-pointer.test.mjs`.
+- `directive.md`: bundled ultrawork directive text.
+- `skills/ultrawork/SKILL.md`: full directive as a skill, generated by `scripts/sync-directive.mjs`; pinned byte-for-byte to `directive.md` by `test/directive-source.test.ts`. `plugin/scripts/sync-skills.mjs` materializes it into the aggregate `plugin/skills/` tree the pointer path targets.
+- `skills/ulw-plan/`: dual-maintained copy of the ulw-plan skill (component copy wins over `packages/shared-skills/skills/ulw-plan/` in sync-skills; keep both in step by hand).
+- `agents/*.toml`: bundled Codex agent role files. Installed as regular files into `CODEX_HOME/agents/` by `src/cli/install-codex/link-cached-plugin-agents.ts` at install time. Public `sisyphuslabs` installs source them from Codex's installed-marketplace snapshot, not the versioned plugin cache, so they survive Codex auto-update cache pruning and temporary snapshot cleanup. No runtime `SessionStart` hook is involved.
+- `hooks/hooks.json`: registers the prompt-detector hook only.
+- `.codex-plugin/plugin.json`: Codex plugin manifest. Marketplace metadata lives here, not in `package.json`.
 
 ## Constraints
 
-- Never let the hook block a turn — exit code is always 0.
+- Never let the hook block a turn. Exit code is always 0.
 - Never make a network call from the hook.
 - Keep the directive in `directive.md`. Do not inline it into TypeScript files.
 - Keep bundled agent role prompts concise and model-specific; measure prompt length when changing them.