lazycodex-ai 5.0.0-beta.4 → 5.0.0-beta.42

package/packages/omo-codex/plugin/test/teammode-transport.test.mjs

@@ -3,7 +3,6 @@ import { existsSync, readFileSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import test from "node:test";
 
-import { root } from "./aggregate-plugin-fixture.mjs";
 import {
 	cleanupTeamRoot,
 	createTeamRoot,
@@ -53,45 +52,6 @@ function addV2Members(tempRoot, sessionId) {
 	);
 }
 
-test("#given flat V2 tools are available #when the leader reads teammode #then transport selection and user announcement precede init", () => {
-	const skill = readFileSync(join(root, "components", "teammode", "skills", "teammode", "SKILL.md"), "utf8");
-	const inspectIndex = skill.indexOf("Inspect your active tool list");
-	const initIndex = skill.indexOf('team.mjs" init');
-
-	assert.notEqual(inspectIndex, -1, "skill must inspect the active tool list");
-	assert.notEqual(initIndex, -1, "skill must still document init");
-	assert.ok(inspectIndex < initIndex, "transport inspection must happen before init");
-	assert.match(skill, /spawn_agent.*task_name[\s\S]*send_message[\s\S]*followup_task[\s\S]*wait_agent[\s\S]*list_agents[\s\S]*interrupt_agent/);
-	assert.match(skill, /tell the user|announce.*transport/i);
-	assert.match(skill, /MultiAgentV2/i);
-	assert.match(skill, /Codex App.*fallback/i);
-});
-
-test("#given neither transport's tools are visible #when the leader reads teammode #then search hits are revalidated and fallback is capability-aware without team state", () => {
-	const skill = readFileSync(join(root, "components", "teammode", "skills", "teammode", "SKILL.md"), "utf8");
-
-	const searchIndex = skill.indexOf("tool_search");
-	const unavailableIndex = skill.indexOf("Teammode unavailable");
-	assert.notEqual(searchIndex, -1, "skill must route hidden tools through the tool_search check");
-	assert.notEqual(unavailableIndex, -1, "skill must give the leader unavailable announcement templates");
-	assert.ok(searchIndex < unavailableIndex, "tool_search must precede the unavailable conclusion");
-	assert.match(skill, /revalidate.*COMPLETE.*compatible transport set/is, "a deferred-tool hit must be revalidated as a complete transport");
-	assert.match(skill, /another visible plain-subagent mechanism.*spawn.*communicate.*observe/is, "plain-subagent fallback requires a complete visible lifecycle");
-	assert.match(skill, /Otherwise continue serially.*capability limitation/is, "missing all transports must continue serially rather than promise workers");
-	assert.match(skill, /Do NOT run `init`/, "the fallback must not create team state");
-	assert.doesNotMatch(skill, /splitting the work across plain fire-and-forget subagents/i, "the skill must not promise unavailable plain subagents");
-});
-
-test("#given a MultiAgentV2 team #when the leader spawns members #then guidance limits calls to exposed V2 fields", () => {
-	const skill = readFileSync(join(root, "components", "teammode", "skills", "teammode", "SKILL.md"), "utf8");
-
-	assert.match(skill, /using only the V2 schema fields/i);
-	assert.match(skill, /`task_name`[\s\S]*`message`[\s\S]*`fork_turns`/);
-	assert.match(skill, /role, priority, or task-specific routing instruction in `message`/i);
-	assert.match(skill, /V2 does not accept[\s\S]*`agent_type`, `model`, `reasoning_effort`, or `service_tier`/);
-	assert.match(skill, /inherit the[\s\S]*session model/i);
-});
-
 test("#given MultiAgentV2 transport #when members are added and bound #then task names and canonical agent paths form the address book", () => {
 	const tempRoot = createTeamRoot("omo-codex-teammode-v2-transport-");
 	try {
@@ -123,7 +83,6 @@ test("#given MultiAgentV2 transport #when members are added and bound #then task
 				{ id: "B", taskName: "mailbox_delivery", agentPath: "/root/mailbox_delivery", threadId: null, threadTitle: null, status: "active" },
 			],
 		);
-		assert.match(guide, /Transport:.*MultiAgentV2/i);
 		assert.match(guide, /members\[\]\.agentPath/);
 		assert.match(guide, /send_message/);
 		assert.match(guide, /followup_task/);
@@ -223,9 +182,7 @@ test("#given a V2 team #when the whole team is archived without a note #then no
 		const archiveEntry = team.log.find((entry) => entry.event === "archive");
 
 		assert.equal(team.status, "archived");
-		assert.doesNotMatch(result.stdout, /closed all members/i, "V2 has no runtime close/archive operation to claim");
 		assert.match(result.stdout, /no runtime archive operation/i);
-		assert.doesNotMatch(archiveEntry.detail, /all members closed/i);
 		assert.match(archiveEntry.detail, /no runtime archive operation/i);
 	} finally {
 		cleanupTeamRoot(tempRoot);
@@ -261,7 +218,6 @@ test("#given a codex_app team #when one member is archived #then the existing wo
 		const result = runTeam(tempRoot, "archive", "--team", "app-archive-member", "--id", "A");
 
 		assert.match(result.stdout, /archived member A/);
-		assert.doesNotMatch(result.stdout, /no runtime archive operation/i);
 	} finally {
 		cleanupTeamRoot(tempRoot);
 	}

package/packages/omo-codex/plugin/test/teammode-worktree.test.mjs

@@ -83,24 +83,21 @@ test("#given a worktree team #when worktree-add A #then a real git worktree on a
 		assert.match(result.stdout, /A/);
 		assert.match(result.stdout, /wait for the real Codex thread id/);
 		assert.match(result.stdout, /bind-thread before sending bootstrap/);
-		assert.doesNotMatch(result.stdout, /Tell that member to:/);
 	} finally {
 		cleanupTeamRoot(tempRoot);
 	}
 });
 
-test("#given worktree-add ran #when the field manual is regenerated #then it flips the member's guide to ISOLATION IS ON with the derived branch", () => {
+test("#given worktree-add ran #when the field manual is regenerated #then it shows the member's derived worktree branch", () => {
 	const tempRoot = createTeamRoot("omo-codex-teammode-wt-guide-");
 	try {
 		initGitRepo(tempRoot);
 		bootstrapTeam(tempRoot, "wt-guide", { worktree: false });
-		// before: no isolation, so the manual tells members to signal collisions instead
-		assert.match(readFileSync(join(teamDir(tempRoot, "wt-guide"), "guide.md"), "utf8"), /does not use isolated git worktrees/);
 
 		runTeam(tempRoot, "worktree-add", "--team", "wt-guide", "--id", "A");
 
+		// then - the regenerated guide carries the derived branch recorded in team state
 		const guide = readFileSync(join(teamDir(tempRoot, "wt-guide"), "guide.md"), "utf8");
-		assert.match(guide, /ISOLATION IS ON/);
 		assert.match(guide, /team\/wt-guide\/A/, "the member's row must show its derived worktree branch");
 	} finally {
 		cleanupTeamRoot(tempRoot);
@@ -225,7 +222,6 @@ test("#given a member branch that cannot be merged #when integrate runs #then it
 		const out = `${result.stdout}${result.stderr}`;
 		assert.notEqual(result.status, 0);
 		assert.match(out, /could not integrate|not something we can merge|ghost-branch-404/i, "must surface git's actual reason");
-		assert.doesNotMatch(out, /Conflicting files/, "a non-conflict failure must NOT be mislabeled as a merge conflict");
 	} finally {
 		cleanupTeamRoot(tempRoot);
 	}

package/packages/omo-codex/plugin/test/ultrawork-skill-pointer.test.mjs

@@ -4,7 +4,7 @@ import { existsSync, readFileSync } from "node:fs";
 import { join } from "node:path";
 import test from "node:test";
 
-import { root } from "./aggregate-plugin-fixture.mjs";
+import { repoRoot, root } from "./aggregate-plugin-fixture.mjs";
 
 const POINTER_MAX_BYTES = 4096;
 const skillPath = join(root, "skills", "ultrawork", "SKILL.md");
@@ -30,10 +30,12 @@ function runUserPromptSubmitCli(component, extraArgs = []) {
 test("#given synced skills #when the ultrawork skill is inspected #then it carries the full directive body", () => {
 	assert.equal(existsSync(skillPath), true);
 	const skill = readFileSync(skillPath, "utf8");
-	const directive = readFileSync(join(root, "components", "ultrawork", "directive.md"), "utf8");
+	const directive = readFileSync(
+		join(repoRoot, "packages", "prompts-core", "prompts", "ultrawork", "codex.md"),
+		"utf8",
+	);
 	assert.match(skill, /^---\nname: ultrawork\n/);
 	assert.equal(skill.endsWith(directive), true);
-	assert.match(skill, /## 1\. Create the goal with binding success criteria/);
 });
 
 test("#given the built ultrawork CLI #when a ulw prompt is submitted #then a compact skill pointer is emitted", () => {
@@ -42,10 +44,8 @@ test("#given the built ultrawork CLI #when a ulw prompt is submitted #then a com
 	const output = JSON.parse(result.stdout);
 	const context = output.hookSpecificOutput.additionalContext;
 	assert.match(context, /^<ultrawork-mode>/);
-	assert.match(context, /First user-visible line this turn MUST be exactly:/);
 	assert.match(context, /create_goal/);
 	assert.equal(context.includes(skillPath), true);
-	assert.equal(context.includes("Tier triage"), false);
 	assert.equal(Buffer.byteLength(context, "utf8") < POINTER_MAX_BYTES, true);
 });
 

package/packages/omo-codex/plugin/test/ulw-plan-review-state-contract.test.mjs

@@ -54,15 +54,12 @@ function matchesCas(state, expected, clauses) {
 
 for (const surface of surfaces) {
 	test(`#given ${surface.name} #when deeper review becomes required before plan completion #then durable request state covers explicit and automatic review without inventing a digest`, async () => {
-		const skill = await readFile(surface.skillPath, "utf8");
 		const workflow = await readFile(surface.workflowPath, "utf8");
 		const contract = readJsonContract(workflow, "ulw-plan-review-request-state-contract");
 
 		assert.equal(contract.transition, "replace");
 		assert.equal(contract.phase, "review_requested");
 		assert.deepEqual(contract.applies_when, ["explicit_review_modifier_before_complete_plan", "intent=unclear_and_nontrivial"]);
-		assert.match(skill, /Include `--review-required`[\s\S]{0,180}non-Trivial UNCLEAR/);
-		assert.match(workflow, /--draft-only/);
 		assert.equal(contract.atomic, true);
 		assert.equal(contract.review_required, true);
 		assert.equal(contract.plan_path, ".omo/plans/<slug>.md");