oh-my-opencode 4.16.2 → 4.17.0

package/packages/omo-codex/plugin/test/aggregate-agents.test.mjs

@@ -17,15 +17,73 @@ const agentSchemaKeys = new Set([
 
 const lazycodexAgentInvariants = new Map([
 	[
-		"lazycodex-executor.toml",
+		"explorer.toml",
+		{
+			model: "gpt-5.6-luna",
+			effort: "low",
+			includes: [/Read-only/, /working tree/, /rg/],
+		},
+	],
+	[
+		"librarian.toml",
+		{
+			model: "gpt-5.6-luna",
+			effort: "low",
+			includes: [/Read-only/, /SHA-pinned GitHub permalink/, /external/],
+		},
+	],
+	[
+		"metis.toml",
+		{
+			model: "gpt-5.6-sol",
+			effort: "high",
+			includes: [/pre-planning analyst/i, /contradictions/, /Read-only/],
+		},
+	],
+	[
+		"momus.toml",
+		{
+			model: "gpt-5.6-sol",
+			effort: "ultra",
+			includes: [/plan reviewer/i, /OKAY, ITERATE, or REJECT/, /Read-only/],
+		},
+	],
+	[
+		"plan.toml",
+		{
+			model: "gpt-5.6-sol",
+			effort: "max",
+			includes: [/strategic planning consultant/i, /\.omo\/plans\/<slug>\.md/, /never implements/i],
+		},
+	],
+	[
+		"lazycodex-worker-low.toml",
 		{
+			model: "gpt-5.6-luna",
 			effort: "high",
-			includes: [/EVIDENCE_RECORDED: <path>/, /scenario/i, /artifact/i],
+			includes: [/EVIDENCE_RECORDED: <path>/, /low-difficulty/i, /smallest correct change/i],
+		},
+	],
+	[
+		"lazycodex-worker-medium.toml",
+		{
+			model: "gpt-5.6-luna",
+			effort: "max",
+			includes: [/EVIDENCE_RECORDED: <path>/, /medium-difficulty/i, /smallest correct change/i],
+		},
+	],
+	[
+		"lazycodex-worker-high.toml",
+		{
+			model: "gpt-5.6-sol",
+			effort: "max",
+			includes: [/EVIDENCE_RECORDED: <path>/, /high-difficulty/i, /smallest correct change/i],
 		},
 	],
 	[
 		"lazycodex-clone-fidelity-reviewer.toml",
 		{
+			model: "gpt-5.6-sol",
 			effort: "xhigh",
 			includes: [/recommendation/, /blockers/, /\.omo\/evidence\/<goal>-clone-fidelity\.md/],
 		},
@@ -33,22 +91,25 @@ const lazycodexAgentInvariants = new Map([
 	[
 		"lazycodex-code-reviewer.toml",
 		{
+			model: "gpt-5.6-sol",
 			effort: "xhigh",
-			includes: [/codeQualityStatus/, /recommendation/, /\.omo\/evidence\/<goal>-code-review\.md/],
+			includes: [/codeQualityStatus/, /recommendation/, /<attemptDir>\/<goalId>-code-review\.md/, /currentAttemptDir/],
 		},
 	],
 	[
 		"lazycodex-qa-executor.toml",
 		{
-			effort: "medium",
-			includes: [/not_applicable/, /surfaceEvidence/, /adversarialCases/],
+			model: "gpt-5.6-luna",
+			effort: "high",
+			includes: [/not_applicable/, /surfaceEvidence/, /adversarialCases/, /<attemptDir>\/<goalId>-manual-qa\.md/],
 		},
 	],
 	[
 		"lazycodex-gate-reviewer.toml",
 		{
-			effort: "xhigh",
-			includes: [/APPROVE\/REJECT/, /blockers/, /\.omo\/evidence\/<goal>-gate-review\.md/],
+			model: "gpt-5.6-sol",
+			effort: "high",
+			includes: [/APPROVE\/REJECT/, /blockers/, /<attemptDir>\/<goalId>-gate-review\.md/, /currentAttemptDir/],
 		},
 	],
 ]);
@@ -69,9 +130,11 @@ test("#given bundled Codex agents #when components/ultrawork/agents directory is
 		"explorer.toml",
 		"lazycodex-clone-fidelity-reviewer.toml",
 		"lazycodex-code-reviewer.toml",
-		"lazycodex-executor.toml",
 		"lazycodex-gate-reviewer.toml",
 		"lazycodex-qa-executor.toml",
+		"lazycodex-worker-high.toml",
+		"lazycodex-worker-low.toml",
+		"lazycodex-worker-medium.toml",
 		"librarian.toml",
 		"metis.toml",
 		"momus.toml",
@@ -94,11 +157,29 @@ test("#given bundled Codex agents #when components/ultrawork/agents directory is
 	}
 });
 
+test("#given bundled agent TOMLs #when nickname_candidates are inspected #then they use only the codex-accepted charset", async () => {
+	// given: codex_app_server ignores a role whose nickname has characters outside
+	// ASCII letters, digits, spaces, hyphens, underscores (observed live in task-15 QA)
+	const agentsDir = join(root, "components", "ultrawork", "agents");
+	const files = (await readdir(agentsDir)).filter((name) => name.endsWith(".toml"));
+
+	// when/then
+	for (const file of files) {
+		const text = await readFile(join(agentsDir, file), "utf8");
+		for (const match of text.matchAll(/nickname_candidates\s*=\s*\[([^\]]*)\]/g)) {
+			for (const nickname of match[1].matchAll(/"([^"]*)"/g)) {
+				assert.match(nickname[1], /^[A-Za-z0-9 _-]+$/, `${file}: nickname "${nickname[1]}"`);
+			}
+		}
+	}
+});
+
 test("#given planner agent prompt #when inspected #then generated artifacts stay under .omo", async () => {
 	const prompt = await readFile(join(root, "components", "ultrawork", "agents", "plan.toml"), "utf8");
 
 	assert.match(prompt, /\.omo\/plans\/<slug>\.md/);
-	assert.match(prompt, /\.omo\/evidence\/task-<N>-<slug>\.<ext>/);
+	assert.match(prompt, /<attemptDir>\/task-<N>-<slug>\.<ext>/);
+	assert.match(prompt, /\.omo\/evidence\/ulw\/<session>\/<goalId>\/a<attempt>/);
 	assert.doesNotMatch(prompt, /(?<!\.omo\/)plans\/<slug>\.md/);
 	assert.doesNotMatch(prompt, /(?<!\.omo\/)evidence\/task-/);
 });
@@ -109,7 +190,8 @@ test("#given lazycodex agent prompts #when inspected #then each role pins model
 	for (const [fileName, invariant] of lazycodexAgentInvariants) {
 		const prompt = await readFile(join(agentsDir, fileName), "utf8");
 
-		assert.match(prompt, /^model\s*=\s*"gpt-5\.5"$/m);
+		const escapedModel = invariant.model.replace(/\./g, "\\.");
+		assert.match(prompt, new RegExp(`^model\\s*=\\s*"${escapedModel}"$`, "m"));
 		assert.match(prompt, new RegExp(`^model_reasoning_effort\\s*=\\s*"${invariant.effort}"$`, "m"));
 		assert.doesNotMatch(prompt, /^tools\s*=/m);
 		assert.doesNotMatch(prompt, /^blocking\s*=/m);
@@ -161,3 +243,27 @@ test("#given LazyCodex reviewer prompts #when inspected #then anti-slop review c
 		"gate reviewer must perform the overfit/slop pass directly before checking report coverage",
 	);
 });
+
+test("#given done-gate reviewer prompts #when inspected #then burden of proof is approve-unless-cited and reject priors are gone", async () => {
+	const agentsDir = join(root, "components", "ultrawork", "agents");
+	const gateReviewer = await readFile(join(agentsDir, "lazycodex-gate-reviewer.toml"), "utf8");
+	const qaExecutor = await readFile(join(agentsDir, "lazycodex-qa-executor.toml"), "utf8");
+	const codeReviewer = await readFile(join(agentsDir, "lazycodex-code-reviewer.toml"), "utf8");
+
+	assert.match(gateReviewer, /APPROVE unless you can cite/);
+	assert.match(gateReviewer, /violatedCriterion/);
+	assert.match(gateReviewer, /evidencePointer/);
+	assert.match(gateReviewer, /top blockers inline/);
+	assert.match(gateReviewer, /is a NOTE, not a blocker/);
+	assert.match(gateReviewer, /You do NOT check/);
+	assert.doesNotMatch(gateReviewer, /Assume the work has already failed/);
+	assert.doesNotMatch(gateReviewer, /Return exactly one recommendation: APPROVE\/REJECT\./);
+
+	assert.match(qaExecutor, /one-line reason/);
+	assert.match(qaExecutor, /rejecting a legitimately untriggered class is itself an error/);
+	assert.doesNotMatch(qaExecutor, /Trust nothing\./);
+
+	assert.match(codeReviewer, /MEDIUM by default/);
+	assert.doesNotMatch(codeReviewer, /Treat useless tests or needless production complexity as CRITICAL\/HIGH/);
+});
+

package/packages/omo-codex/plugin/test/aggregate-hooks.test.mjs

@@ -48,6 +48,22 @@ test("#given isolated components #when hooks are inspected #then commands stay i
 	assert.equal(await exists("scripts/migrate-codex-config.mjs"), true);
 });
 
+test("#given aggregate Stop hooks #when inspected #then start-work continuation and ulw-loop resume are separate groups", async () => {
+	// given
+	const manifests = await readAggregateHookManifests();
+
+	// when
+	const stopCommands = manifests
+		.filter(({ hooks }) => hooks.hooks.Stop)
+		.flatMap(({ hooks }) => hooks.hooks.Stop)
+		.flatMap((group) => group.hooks.map((handler) => handler.command));
+
+	// then
+	assert.equal(stopCommands.length, 2);
+	assert.ok(stopCommands.some((command) => command.includes("start-work-continuation/dist/cli.js")));
+	assert.ok(stopCommands.some((command) => command.includes("ulw-loop/dist/cli.js\" hook stop")));
+});
+
 test("#given aggregate SubagentStop hooks #when inspected #then start-work and LazyCodex executor verifier are separate groups", async () => {
 	// given
 	const manifests = await readAggregateHookManifests();
@@ -64,7 +80,7 @@ test("#given aggregate SubagentStop hooks #when inspected #then start-work and L
 	// then
 	assert.equal(subagentStopGroups.length, 2);
 	assert.equal(subagentStopGroups[0]?.matcher, undefined);
-	assert.equal(subagentStopGroups[1]?.matcher, "^lazycodex-executor$");
+	assert.equal(subagentStopGroups[1]?.matcher, "^lazycodex-worker-(low|medium|high)$");
 	assert.equal(verifierGroups.length, 1);
 	assert.equal(verifierGroups[0]?.groupIndex, 0);
 	assert.equal(verifierGroups[0]?.handler.timeout, 10);
@@ -158,7 +174,12 @@ test("#given aggregate OMO plugin is enabled #when hooks are inspected #then she
 	assert.match(text, /Resetting Git Bash MCP Reminder/);
 	assert.match(text, /components\/ulw-loop\/dist\/cli\.js/);
 	assert.match(text, /hook pre-tool-use/);
-	assert.deepEqual(preToolUseGroups.map((group) => group.matcher), ["^Bash$", "^create_goal$"]);
+	assert.deepEqual(preToolUseGroups.map((group) => group.matcher), [
+		"^Bash$",
+		"^create_goal$",
+		"^(spawn_agent|collaborationspawn_agent|collaboration\\.spawn_agent)$",
+	]);
+	assert.match(text, /hook pre-tool-use-spawn/);
 });
 
 test("#given aggregate OMO plugin has a dedicated ultrawork trigger #when hooks are inspected #then ulw-loop does not duplicate ultrawork injection", async () => {

package/packages/omo-codex/plugin/test/aggregate-manifest.test.mjs

@@ -17,7 +17,7 @@ test("#given aggregate plugin manifest #when inspected #then it owns the omo nam
 	// then
 	assert.equal(manifest.name, "omo");
 	assert(Array.isArray(hookPaths));
-	assert.equal(hookPaths.length, 21);
+	assert.equal(hookPaths.length, 23);
 	assert(hookPaths.every((hookPath) => typeof hookPath === "string" && hookPath.startsWith("./hooks/")));
 	assert(!hookPaths.includes("./hooks/hooks.json"));
 	assert(!hookPaths.includes("./hooks/user-prompt-submit-selecting-lazycodex-workflow.json"));

package/packages/omo-codex/plugin/test/aggregate-model-catalog.test.mjs

@@ -8,17 +8,17 @@ import { root } from "./aggregate-plugin-fixture.mjs";
 test("#given bundled model catalog #when inspected #then default verifier and worker roles are pinned", async () => {
 	const catalog = JSON.parse(await readFile(join(root, "model-catalog.json"), "utf8"));
 
-	assert.equal(catalog.current.model, "gpt-5.5");
-	assert.equal(catalog.current.model_context_window, 400000);
+	assert.equal(catalog.current.model, "gpt-5.6-sol");
+	assert.equal(catalog.current.model_context_window, 372000);
 	assert.equal(catalog.current.model_reasoning_effort, "high");
 	assert.equal(catalog.current.plan_mode_reasoning_effort, "xhigh");
 	assert.deepEqual(catalog.roles.default, catalog.current);
 	assert.deepEqual(catalog.roles.verifier, {
-		model: "gpt-5.5",
+		model: "gpt-5.6-sol",
 		model_reasoning_effort: "high",
 	});
 	assert.deepEqual(catalog.roles.worker, {
-		model: "gpt-5.5",
+		model: "gpt-5.6-sol",
 		model_reasoning_effort: "high",
 	});
 });

package/packages/omo-codex/plugin/test/aggregate-skills.test.mjs

@@ -34,7 +34,8 @@ test('#given synced skills and bundled rules #when role-specific agents are spaw
 	const promptFiles = skillEntries
 		.filter((entry) => entry.isDirectory())
 		.map((entry) => join(skillsDir, entry.name, "SKILL.md"));
-	promptFiles.push(join(root, "components", "rules", "bundled-rules", "hephaestus.md"));
+	promptFiles.push(join(root, "components", "rules", "bundled-rules", "hephaestus", "gpt-5.5.md"));
+	promptFiles.push(join(root, "components", "rules", "bundled-rules", "hephaestus", "gpt-5.6.md"));
 
 	const missingForkContext = [];
 	for (const promptPath of promptFiles) {
@@ -53,7 +54,8 @@ test("#given long-running orchestration prompts #when waiting on child agents #t
 		join(root, "skills", "ulw-loop", "references", "full-workflow.md"),
 		join(root, "skills", "review-work", "SKILL.md"),
 		join(root, "skills", "start-work", "SKILL.md"),
-		join(root, "components", "rules", "bundled-rules", "hephaestus.md"),
+		join(root, "components", "rules", "bundled-rules", "hephaestus", "gpt-5.5.md"),
+		join(root, "components", "rules", "bundled-rules", "hephaestus", "gpt-5.6.md"),
 	];
 
 	const missingLivenessGuidance = [];

package/packages/omo-codex/plugin/test/auto-update-restart-notice.test.mjs

@@ -224,3 +224,36 @@ test("#given completed pending update #when hook session-start runs as CLI #then
 	assert.equal(repeat.status, 0);
 	assert.equal(repeat.stdout, "");
 });
+
+test("#given the script reached through a symlinked directory #when hook session-start runs as CLI #then the entry guard still fires", async () => {
+	const root = await mkdtemp(join(tmpdir(), "lazycodex-restart-notice-symlink-"));
+	const env = autoUpdateEnv(root, {
+		LAZYCODEX_CURRENT_VERSION: "1.0.1",
+		LAZYCODEX_LATEST_VERSION: "1.0.1",
+		LAZYCODEX_CONFIG_MIGRATION_DISABLED: "1",
+	});
+	await writeFile(env.LAZYCODEX_AUTO_UPDATE_STATE_PATH, JSON.stringify({
+		lastCheckedAt: Date.now() - 1_000,
+		lastStatus: "success",
+		pendingNotice: { fromVersion: "1.0.0", toVersion: "1.0.1", startedAt: 1 },
+	}));
+	// Simulate a symlinked plugin cache dir: argv[1] spells the symlink, while
+	// import.meta.url resolves to the real path. Seen in the wild; the old
+	// `pathToFileURL(process.argv[1])` guard never matched and the hook was a
+	// silent no-op.
+	const { symlink } = await import("node:fs/promises");
+	const { dirname } = await import("node:path");
+	const linkDir = join(root, "linked-scripts");
+	await symlink(dirname(SCRIPT_PATH), linkDir, "dir");
+	const linkedScript = join(linkDir, "auto-update.mjs");
+
+	const result = spawnSync(process.execPath, [linkedScript, "hook", "session-start"], {
+		encoding: "utf8",
+		env: { ...process.env, ...env },
+	});
+
+	assert.equal(result.status, 0);
+	const lines = result.stdout.split("\n").filter((line) => line.length > 0);
+	assert.equal(lines.length, 1, `expected one SessionStart JSON line, got stdout=${JSON.stringify(result.stdout)} stderr=${JSON.stringify(result.stderr)}`);
+	assert.equal(JSON.parse(lines[0]).hookSpecificOutput.hookEventName, "SessionStart");
+});

package/packages/omo-codex/plugin/test/auto-update.test.mjs

@@ -225,7 +225,7 @@ test("#given test command override #when running check #then records state and l
 			toVersion: "1.0.1",
 		},
 	]);
-	assert.match(await readFile(join(env.CODEX_HOME, "config.toml"), "utf8"), /model = "gpt-5\.5"/);
+	assert.match(await readFile(join(env.CODEX_HOME, "config.toml"), "utf8"), /model = "gpt-5\.6-sol"/);
 });
 
 test("#given failed waited update #when retry window passes #then next update is not blocked by success throttle", async () => {
@@ -279,7 +279,7 @@ test("#given active lock #when running check #then skips concurrent update", asy
 
 	assert.equal(result.started, false);
 	assert.equal(result.reason, "locked");
-	assert.match(await readFile(join(root, "codex-home", "config.toml"), "utf8"), /model_context_window = 400000/);
+	assert.match(await readFile(join(root, "codex-home", "config.toml"), "utf8"), /model_context_window = 372000/);
 });
 
 test("#given stale lock #when running check #then removes lock and runs update", async () => {
@@ -645,8 +645,8 @@ test("#given throttled updater and stale Codex config #when running check #then
 	const content = await readFile(join(codexHome, "config.toml"), "utf8");
 	assert.equal(result.started, false);
 	assert.equal(result.reason, "throttled");
-	assert.match(content, /model = "gpt-5\.5"/);
-	assert.match(content, /model_context_window = 400000/);
+	assert.match(content, /model = "gpt-5\.6-sol"/);
+	assert.match(content, /model_context_window = 372000/);
 	assert.match(content, /model_reasoning_effort = "high"/);
 	assert.match(content, /plan_mode_reasoning_effort = "xhigh"/);
 	assert.doesNotMatch(content, /gpt-5\.2/);

package/packages/omo-codex/plugin/test/component-hook-contract-cases.mjs

@@ -180,12 +180,12 @@ export function componentHookContractCases(tempRoot) {
 			},
 		},
 		{
-			name: "lazycodex executor verifier subagent-stop blocks missing evidence",
+			name: "lazycodex worker verifier subagent-stop blocks missing evidence",
 			component: "lazycodex-executor-verify",
 			event: "subagent-stop",
 			payload: {
 				hook_event_name: "SubagentStop",
-				agent_type: "lazycodex-executor",
+				agent_type: "lazycodex-worker-medium",
 				agent_id: "agent-task12",
 				session_id: "s-task12",
 				transcript_path: join(tempRoot, "transcript.jsonl"),

package/packages/omo-codex/plugin/test/lcx-bug-skills.test.mjs

@@ -7,109 +7,12 @@ import { fileURLToPath } from "node:url";
 const pluginRoot = dirname(dirname(fileURLToPath(import.meta.url)));
 const sharedSkillsRoot = join(pluginRoot, "..", "..", "shared-skills");
 
-async function readSkill(name) {
-	return readFile(join(sharedSkillsRoot, "skills", name, "SKILL.md"), "utf8");
-}
-
-test("#given synced lcx-report-bug skill #when inspected #then it files LazyCodex bug issues with generated labels", async () => {
-	// given
-	const skillRoot = join(sharedSkillsRoot, "skills", "lcx-report-bug");
-
-	// when
-	const skill = await readFile(join(skillRoot, "SKILL.md"), "utf8");
-	const interfaceMetadata = await readFile(join(skillRoot, "agents", "openai.yaml"), "utf8");
-
-	// then
-	assert.match(skill, /^---\r?\nname: lcx-report-bug\r?\n/m);
-	assert.match(skill, /Never create a PR or push a branch against `code-yeongyu\/lazycodex`/);
-	assert.match(skill, /gh pr create --repo openai\/codex/);
-	assert.doesNotMatch(skill, /gh pr create --repo "\$TARGET_REPO"/);
-	assert.doesNotMatch(skill, /gh pr create --repo code-yeongyu\/lazycodex/);
-	assert.match(interfaceMetadata, /display_name: "lcx-report-bug \(omo\)"/);
-	assert.match(interfaceMetadata, /- "lazycodex bug"/);
-	assert.match(interfaceMetadata, /- "openai codex bug"/);
-});
-
-test("#given synced lcx-contribute-bug-fix skill #when inspected #then it delivers LazyCodex fixes as issues and upstream fixes as fork PRs", async () => {
-	// given
-	const skillRoot = join(sharedSkillsRoot, "skills", "lcx-contribute-bug-fix");
-
-	// when
-	const skill = await readFile(join(skillRoot, "SKILL.md"), "utf8");
-	const interfaceMetadata = await readFile(join(skillRoot, "agents", "openai.yaml"), "utf8");
-
-	// then
-	assert.match(skill, /^---\r?\nname: lcx-contribute-bug-fix\r?\n/m);
-	assert.match(skill, /NEVER open a PR or push a branch against this repo/);
-	assert.match(skill, /gh issue create --repo code-yeongyu\/lazycodex/);
-	assert.match(skill, /gh pr create --repo openai\/codex/);
-	assert.doesNotMatch(skill, /gh pr create --repo "\$TARGET_REPO"/);
-	assert.doesNotMatch(skill, /gh pr create --repo code-yeongyu\/lazycodex/);
-	assert.match(interfaceMetadata, /display_name: "lcx-contribute-bug-fix \(omo\)"/);
-	assert.match(interfaceMetadata, /- "contribute a bug fix"/);
-	assert.match(interfaceMetadata, /- "fix bug pr"/);
-});
-
-test("#given synced lcx-doctor skill #when inspected #then it diagnoses installs against latest temp-root sources without mutating them", async () => {
-	// given
-	const skillRoot = join(sharedSkillsRoot, "skills", "lcx-doctor");
-
-	// when
-	const skill = await readFile(join(skillRoot, "SKILL.md"), "utf8");
-	const interfaceMetadata = await readFile(join(skillRoot, "agents", "openai.yaml"), "utf8");
-
-	// then
-	assert.match(skill, /^---\r?\nname: lcx-doctor\r?\n/m);
-	assert.match(interfaceMetadata, /display_name: "lcx-doctor \(omo\)"/);
-	assert.match(interfaceMetadata, /- "lazycodex doctor"/);
-	assert.match(interfaceMetadata, /- "lazycodex health check"/);
-});
-
-test("#given lcx source-sync skills #when inspected #then source caches are validated before reuse", async () => {
-	// given
-	const skillNames = ["lcx-doctor", "lcx-report-bug", "lcx-contribute-bug-fix"];
-
-	for (const skillName of skillNames) {
+test("#given lcx skills #when frontmatter is inspected #then each exposes the loader-parsed skill name", async () => {
+	for (const skillName of ["lcx-report-bug", "lcx-contribute-bug-fix", "lcx-doctor"]) {
 		// when
-		const skill = await readSkill(skillName);
+		const skill = await readFile(join(sharedSkillsRoot, "skills", skillName, "SKILL.md"), "utf8");
 
 		// then
-		assert.match(skill, /LAZYCODEX_SOURCE_ROOT="\$\{LAZYCODEX_SOURCE_ROOT:-\$\{TMPDIR:-\/tmp\}\/lazycodex-sources\}"/);
-		assert.match(skill, /git -C "\$DEST" rev-parse --is-inside-work-tree/);
-		assert.match(skill, /git -C "\$DEST" config --get remote\.origin\.url/);
-		assert.match(skill, /DEST\.corrupt\.\$\(date \+%Y%m%d%H%M%S\)/);
-		assert.doesNotMatch(skill, /if \[ ! -d "\$DEST\/\.git" \]; then/);
-		assert.doesNotMatch(skill, /sync_latest_source code-yeongyu\/lazycodex \/tmp\/lazycodex-source/);
-		assert.doesNotMatch(skill, /sync_latest_source openai\/codex \/tmp\/openai-codex-source/);
+		assert.match(skill, new RegExp(`^name: ${skillName}$`, "m"), `${skillName}: frontmatter must expose ${skillName}`);
 	}
 });
-
-test("#given lcx-doctor skill #when inspected #then aggregate hook validation follows the current manifest", async () => {
-	// given
-	const skill = await readSkill("lcx-doctor");
-
-	// then
-	assert.match(skill, /\.codex-plugin\/plugin\.json/);
-	assert.match(skill, /manifest declares a `hooks` array/);
-	assert.match(skill, /validate every direct hook path declared by the manifest/);
-	assert.match(skill, /`hooks\/hooks\.json` only when the manifest declares it/);
-	assert.match(skill, /do not require retired paths such as `components\/workflow-selector`/);
-	assert.match(skill, /`hooks\/user-prompt-submit-selecting-lazycodex-workflow\.json` unless the current manifest declares them/);
-	assert.doesNotMatch(skill, /Plugin payload present and non-empty: `hooks\/hooks\.json`/);
-});
-
-test("#given lcx-doctor skill #when inspected #then runtime probes and materialized payloads distinguish expected rewrites from missing files", async () => {
-	// given
-	const skill = await readSkill("lcx-doctor");
-
-	// then
-	assert.match(skill, /manifest-declared runtime payload/);
-	assert.match(skill, /`dist\/cli\/index\.js` and `dist\/cli-node\/index\.js`/);
-	assert.match(skill, /install-time materialization rewrites as expected/);
-	assert.match(skill, /absolute `\.mcp\.json` runtime paths/);
-	assert.match(skill, /Missing or zero-byte rewritten targets are FAIL/);
-	assert.match(skill, /Use the configured Codex default model/);
-	assert.match(skill, /unless the user explicitly passed a model override/);
-	assert.match(skill, /never force a guessed\/rejected model such as `gpt-5\.5-codex-mini`/);
-	assert.doesNotMatch(skill, /--model gpt-5\.5-codex-mini/);
-});