oh-my-opencode 4.18.2 → 4.19.0

package/packages/omo-codex/plugin/components/start-work-continuation/dist/cli.js

@@ -1,37 +1,22 @@
 #!/usr/bin/env node
 
 // components/start-work-continuation/src/cli.ts
-import { readFileSync as readFileSync3 } from "node:fs";
+import { readFileSync as readFileSync4 } from "node:fs";
 import { stdin as processStdin, stdout as processStdout } from "node:process";
 
 // components/start-work-continuation/src/boulder-reader.ts
-import { existsSync, readFileSync } from "node:fs";
+import { existsSync as existsSync2, readFileSync as readFileSync2 } from "node:fs";
 import { isAbsolute, join, relative, resolve } from "node:path";
-var TODO_HEADING = "TODOs";
-var FINAL_VERIFICATION_HEADING = "Final Verification Wave";
-var CHECKBOX_PREFIX_LENGTH = "- [ ] ".length;
-var SESSION_ID_PREFIX_PATTERN = /^(codex|opencode):/;
-function readContinuationState(cwd, sessionId) {
-  const boulderPath = getBoulderFilePath(cwd);
-  const boulderState = readBoulderState(boulderPath);
-  if (boulderState === null)
-    return null;
-  const work = getWorkForSession(boulderState, normalizeSessionId(sessionId, "codex"));
-  if (work === null || !isContinuableStatus(work.status))
-    return null;
-  const planPath = resolveBoulderPlanPathForWork(cwd, work);
-  const checklist = getPlanChecklist(planPath);
-  if (checklist.total === 0)
-    return null;
-  return {
-    planName: work.planName,
-    planPath,
-    boulderPath,
-    ledgerPath: join(cwd, ".omo", "start-work", "ledger.jsonl"),
-    worktreePath: work.worktreePath ?? null,
-    checklist
-  };
-}
+
+// components/start-work-continuation/src/plan-checklist.ts
+import { existsSync, readFileSync } from "node:fs";
+var TODO_HEADING_PATTERN = /^##[ \t]+TODOs(?:[ \t]+#+)?[ \t]*$/i;
+var FINAL_VERIFICATION_HEADING_PATTERN = /^##[ \t]+Final Verification Wave(?:[ \t]+#+)?[ \t]*$/i;
+var SECTION_BOUNDARY_HEADING_PATTERN = /^#{1,2}(?:[ \t]+|$)/;
+var FENCE_PATTERN = /^[ \t]{0,3}(`{3,}|~{3,})(.*)$/;
+var SIMPLE_CHECKBOX_PATTERN = /^[-*][ \t]*\[[ \t]*([xX]?)[ \t]*\][ \t]+(.+)$/;
+var TODO_CHECKBOX_PATTERN = /^- \[([ xX])\] ([1-9]\d*\. .+)$/;
+var FINAL_WAVE_CHECKBOX_PATTERN = /^- \[([ xX])\] (F[1-9]\d*\. .+)$/i;
 function getPlanChecklist(planPath) {
   if (!existsSync(planPath))
     return emptyChecklist();
@@ -45,34 +30,155 @@ function getPlanChecklist(planPath) {
 }
 function parsePlanChecklist(markdown) {
   const lines = markdown.split(/\r?\n/);
-  const hasCountedSections = lines.some((line) => isCountedHeading(parseLevelTwoHeading(line)));
+  if (!hasStructuredSection(lines))
+    return parseSimpleChecklist(lines);
   let completed = 0;
   let remaining = 0;
   let nextTaskLabel = null;
-  let isCountedSection = !hasCountedSections;
+  let section = "other";
+  let fence = null;
   for (const line of lines) {
-    const heading = parseLevelTwoHeading(line);
-    if (heading !== null) {
-      isCountedSection = isCountedHeading(heading);
+    if (fence !== null) {
+      if (isClosingFence(line, fence))
+        fence = null;
+      continue;
+    }
+    const openingFence = parseOpeningFence(line);
+    if (openingFence !== null) {
+      fence = openingFence;
       continue;
     }
-    if (!isCountedSection)
+    if (SECTION_BOUNDARY_HEADING_PATTERN.test(line)) {
+      section = parseStructuredSectionHeading(line);
       continue;
-    const checkbox = parseTopLevelCheckbox(line);
+    }
+    if (section === "other")
+      continue;
+    const checkbox = parseStructuredCheckbox(line, section);
     if (checkbox === null)
       continue;
-    if (checkbox.checked) {
+    if (checkbox.checked)
       completed += 1;
-    } else {
+    else {
       remaining += 1;
       nextTaskLabel = nextTaskLabel ?? checkbox.label;
     }
   }
   return { completed, remaining, total: completed + remaining, nextTaskLabel };
 }
+function hasStructuredSection(lines) {
+  let fence = 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) {
+  let completed = 0;
+  let remaining = 0;
+  let nextTaskLabel = null;
+  let fence = 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) {
+  if (TODO_HEADING_PATTERN.test(line))
+    return "todo";
+  if (FINAL_VERIFICATION_HEADING_PATTERN.test(line))
+    return "final-wave";
+  return "other";
+}
+function parseStructuredCheckbox(line, section) {
+  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) {
+  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) {
+  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, fence) {
+  const run = line.match(/^[ \t]{0,3}(`{3,}|~{3,})[ \t]*$/)?.[1];
+  return run?.charAt(0) === fence.marker && run.length >= fence.length;
+}
+function emptyChecklist() {
+  return { completed: 0, remaining: 0, total: 0, nextTaskLabel: null };
+}
+
+// components/start-work-continuation/src/boulder-reader.ts
+var SESSION_ID_PREFIX_PATTERN = /^(codex|opencode):/;
+function readContinuationState(cwd, sessionId) {
+  const boulderPath = getBoulderFilePath(cwd);
+  const boulderState = readBoulderState(boulderPath);
+  if (boulderState === null)
+    return null;
+  const work = getWorkForSession(boulderState, normalizeSessionId(sessionId, "codex"));
+  if (work === null || !isContinuableStatus(work.status))
+    return null;
+  const planPath = resolveBoulderPlanPathForWork(cwd, work);
+  const checklist = getPlanChecklist(planPath);
+  if (checklist.total === 0)
+    return null;
+  return {
+    planName: work.planName,
+    planPath,
+    boulderPath,
+    ledgerPath: join(cwd, ".omo", "start-work", "ledger.jsonl"),
+    worktreePath: work.worktreePath ?? null,
+    checklist
+  };
+}
 function readBoulderState(path) {
   try {
-    const parsed = JSON.parse(readFileSync(path, "utf8"));
+    const parsed = JSON.parse(readFileSync2(path, "utf8"));
     return parseBoulderState(parsed);
   } catch (error) {
     if (error instanceof Error)
@@ -150,27 +256,11 @@ function resolveBoulderPlanPathForWork(cwd, work) {
     return absolutePlanPath;
   }
   const worktreePlanPath = resolve(resolveTrackedPath(cwd, worktreePath), relativePlanPath);
-  return existsSync(worktreePlanPath) ? worktreePlanPath : absolutePlanPath;
+  return existsSync2(worktreePlanPath) ? worktreePlanPath : absolutePlanPath;
 }
 function resolveTrackedPath(baseDirectory, trackedPath) {
   return isAbsolute(trackedPath) ? resolve(trackedPath) : resolve(baseDirectory, trackedPath);
 }
-function parseTopLevelCheckbox(line) {
-  if (line.startsWith("- [ ] "))
-    return { checked: false, label: line.slice(CHECKBOX_PREFIX_LENGTH) };
-  if (line.startsWith("- [x] ") || line.startsWith("- [X] ")) {
-    return { checked: true, label: line.slice(CHECKBOX_PREFIX_LENGTH) };
-  }
-  return null;
-}
-function parseLevelTwoHeading(line) {
-  if (!line.startsWith("## "))
-    return null;
-  return line.slice("## ".length).trim();
-}
-function isCountedHeading(heading) {
-  return heading === TODO_HEADING || heading === FINAL_VERIFICATION_HEADING;
-}
 function parseBoulderWorkStatus(value) {
   if (value === "active" || value === "paused" || value === "completed" || value === "abandoned")
     return value;
@@ -203,16 +293,13 @@ function isContinuableStatus(status) {
 function getBoulderFilePath(cwd) {
   return join(cwd, ".omo", "boulder.json");
 }
-function emptyChecklist() {
-  return { completed: 0, remaining: 0, total: 0, nextTaskLabel: null };
-}
 function isRecord(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
 
 // components/start-work-continuation/src/directive.ts
-import { readFileSync as readFileSync2 } from "node:fs";
-var START_WORK_CONTINUATION_DIRECTIVE = readFileSync2(new URL("../directive.md", import.meta.url), "utf8");
+import { readFileSync as readFileSync3 } from "node:fs";
+var START_WORK_CONTINUATION_DIRECTIVE = readFileSync3(new URL("../directive.md", import.meta.url), "utf8");
 
 // components/start-work-continuation/src/codex-hook.ts
 function runStopHook(input, fs) {
@@ -285,7 +372,7 @@ function isRecord2(value) {
 // components/start-work-continuation/src/cli.ts
 var nodeFileSystem = {
   readFileSync(path, encoding) {
-    return readFileSync3(path, encoding);
+    return readFileSync4(path, encoding);
   }
 };
 var command = process.argv[2];

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 4.18.2) Checking Start-Work Continuation"
+						"statusMessage": "(OmO 4.19.0) 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 4.18.2) Checking Start-Work Continuation"
+						"statusMessage": "(OmO 4.19.0) 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": "4.18.2",
+	"version": "4.19.0",
 	"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/src/boulder-reader.ts

@@ -1,12 +1,9 @@
 import { existsSync, readFileSync } from "node:fs";
 import { isAbsolute, join, relative, resolve } from "node:path";
+import { getPlanChecklist, type PlanChecklist } from "./plan-checklist.js";
 
-export type PlanChecklist = {
-	readonly completed: number;
-	readonly remaining: number;
-	readonly total: number;
-	readonly nextTaskLabel: string | null;
-};
+export type { PlanChecklist } from "./plan-checklist.js";
+export { getPlanChecklist } from "./plan-checklist.js";
 
 type BoulderWorkStatus = "active" | "paused" | "completed" | "abandoned";
 
@@ -35,9 +32,6 @@ export type ContinuationState = {
 	readonly checklist: PlanChecklist;
 };
 
-const TODO_HEADING = "TODOs";
-const FINAL_VERIFICATION_HEADING = "Final Verification Wave";
-const CHECKBOX_PREFIX_LENGTH = "- [ ] ".length;
 const SESSION_ID_PREFIX_PATTERN = /^(codex|opencode):/;
 
 export function readContinuationState(cwd: string, sessionId: string): ContinuationState | null {
@@ -62,47 +56,6 @@ export function readContinuationState(cwd: string, sessionId: string): Continuat
 	};
 }
 
-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;
-	}
-}
-
-function parsePlanChecklist(markdown: string): PlanChecklist {
-	const lines = markdown.split(/\r?\n/);
-	const hasCountedSections = lines.some((line) => isCountedHeading(parseLevelTwoHeading(line)));
-	let completed = 0;
-	let remaining = 0;
-	let nextTaskLabel: string | null = null;
-	let isCountedSection = !hasCountedSections;
-
-	for (const line of lines) {
-		const heading = parseLevelTwoHeading(line);
-		if (heading !== null) {
-			isCountedSection = isCountedHeading(heading);
-			continue;
-		}
-		if (!isCountedSection) continue;
-
-		const checkbox = parseTopLevelCheckbox(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 readBoulderState(path: string): BoulderState | null {
 	try {
 		const parsed: unknown = JSON.parse(readFileSync(path, "utf8"));
@@ -193,23 +146,6 @@ function resolveTrackedPath(baseDirectory: string, trackedPath: string): string
 	return isAbsolute(trackedPath) ? resolve(trackedPath) : resolve(baseDirectory, trackedPath);
 }
 
-function parseTopLevelCheckbox(line: string): { readonly checked: boolean; readonly label: string } | null {
-	if (line.startsWith("- [ ] ")) return { checked: false, label: line.slice(CHECKBOX_PREFIX_LENGTH) };
-	if (line.startsWith("- [x] ") || line.startsWith("- [X] ")) {
-		return { checked: true, label: line.slice(CHECKBOX_PREFIX_LENGTH) };
-	}
-	return null;
-}
-
-function parseLevelTwoHeading(line: string): string | null {
-	if (!line.startsWith("## ")) return null;
-	return line.slice("## ".length).trim();
-}
-
-function isCountedHeading(heading: string | null): boolean {
-	return heading === TODO_HEADING || heading === FINAL_VERIFICATION_HEADING;
-}
-
 function parseBoulderWorkStatus(value: unknown): BoulderWorkStatus | undefined {
 	if (value === "active" || value === "paused" || value === "completed" || value === "abandoned") return value;
 	return undefined;
@@ -243,10 +179,6 @@ function getBoulderFilePath(cwd: string): string {
 	return join(cwd, ".omo", "boulder.json");
 }
 
-function emptyChecklist(): PlanChecklist {
-	return { completed: 0, remaining: 0, total: 0, nextTaskLabel: null };
-}
-
 function isRecord(value: unknown): value is Record<string, unknown> {
 	return typeof value === "object" && value !== null && !Array.isArray(value);
 }

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 };
+}