oh-my-opencode 5.0.0-beta.49 → 5.0.0-beta.50

package/packages/omo-codex/scripts/install-config.test.mjs

@@ -8,7 +8,7 @@ import test from "node:test";
 
 import { updateCodexConfig } from "./install-dist/install-local.mjs";
 
-test("#given empty Codex config #when script installer updates config #then sets subagent thread limits without forcing MultiAgentV2", async () => {
+test("#given empty Codex config #when script installer updates config #then leaves thread limits unset without forcing MultiAgentV2", async () => {
 	// given
 	const root = await mkdtemp(join(tmpdir(), "omo-codex-script-config-multi-agent-"));
 	const configPath = join(root, "config.toml");
@@ -28,10 +28,10 @@ test("#given empty Codex config #when script installer updates config #then sets
 	const config = await readFile(configPath, "utf8");
 	assert.doesNotMatch(config, /^\s*multi_agent_mode\s*=/m);
 	assert.doesNotMatch(config, /^\s*max_threads\s*=/m);
-	assert.match(config, /\[features\.multi_agent_v2\]/);
+	assert.doesNotMatch(config, /\[features\.multi_agent_v2\]/);
 	const v2Section = multiAgentV2Section(config);
 	assert.doesNotMatch(v2Section, /^enabled\s*=/m);
-	assert.match(v2Section, /max_concurrent_threads_per_session = 16/);
+	assert.doesNotMatch(v2Section, /max_concurrent_threads_per_session\s*=/);
 });
 
 test("#given queue multi-agent mode #when script installer updates config #then removes unsupported root key", async () => {
@@ -337,7 +337,7 @@ test("#given empty Codex config #when script installer updates config #then sets
 	const config = await readFile(configPath, "utf8");
 	const v2Section = config.slice(config.indexOf("[features.multi_agent_v2]"));
 	assert.doesNotMatch(config, /^\s*max_threads\s*=/m);
-	assert.match(v2Section, /max_concurrent_threads_per_session = 16/);
+	assert.doesNotMatch(v2Section, /max_concurrent_threads_per_session\s*=/);
 	assert.doesNotMatch(v2Section, /hide_spawn_agent_metadata/);
 });
 
@@ -404,7 +404,7 @@ test("#given legacy boolean MultiAgentV2 flag and table #when script installer u
 	const v2Section = multiAgentV2Section(config);
 	assert.doesNotMatch(v2Section, /^enabled\s*=/m);
 	assert.match(v2Section, /usage_hint_enabled = false/);
-	assert.match(v2Section, /max_concurrent_threads_per_session = 16/);
+	assert.doesNotMatch(v2Section, /max_concurrent_threads_per_session\s*=/);
 });
 
 test("#given legacy boolean MultiAgentV2 flag false #when script installer updates config #then normalizes to a disabled table config", async () => {
@@ -440,12 +440,12 @@ test("#given legacy boolean MultiAgentV2 flag false #when script installer updat
 	assert.match(config, /\[features\.multi_agent_v2\]/);
 	const disabledV2Section = multiAgentV2Section(config);
 	assert.match(disabledV2Section, /^enabled = false$/m);
-	assert.match(disabledV2Section, /^max_concurrent_threads_per_session = 16$/m);
+	assert.doesNotMatch(disabledV2Section, /^max_concurrent_threads_per_session\s*=/m);
 });
 
-test("#given legacy agents max_threads #when script installer updates config #then raises the root subagent thread cap", async () => {
+test("#given legacy agents max_threads #when script installer updates config #then preserves the user root subagent thread cap", async () => {
 	// given
-	// A pinned v1 model keeps the legacy low-cap raise path exercised; the
+	// A pinned v1 model keeps user V1 cap preservation exercised; the
 	// stamped v2-preferred default would remove agents.max_threads instead.
 	const root = await mkdtemp(join(tmpdir(), "omo-codex-script-config-multi-agent-legacy-threads-"));
 	const configPath = join(root, "config.toml");
@@ -473,20 +473,19 @@ test("#given legacy agents max_threads #when script installer updates config #th
 
 	// then
 	const config = await readFile(configPath, "utf8");
-	assert.match(config, /\[features\.multi_agent_v2\]/);
+	assert.doesNotMatch(config, /\[features\.multi_agent_v2\]/);
 	const v2Section = multiAgentV2Section(config);
 	assert.doesNotMatch(v2Section, /^enabled\s*=/m);
-	assert.match(v2Section, /max_concurrent_threads_per_session = 16/);
+	assert.doesNotMatch(v2Section, /max_concurrent_threads_per_session\s*=/);
 	assert.match(config, /\[agents\]/);
-	assert.match(config, /max_threads = 1000/);
-	assert.doesNotMatch(config, /max_threads = 16/);
+	assert.match(config, /^\s*max_threads\s*=\s*16$/m);
 	assert.match(config, /max_depth = 4/);
 	assert.match(config, /job_max_runtime_seconds = 3600/);
 });
 
-test("#given managed agent role sections #when script installer updates config #then preserves role config while raising only root agents max_threads", async () => {
+test("#given managed agent role sections #when script installer updates config #then preserves role config without raising root agents max_threads", async () => {
 	// given
-	// A pinned v1 model keeps the legacy low-cap raise path exercised; the
+	// A pinned v1 model keeps user V1 cap preservation exercised; the
 	// stamped v2-preferred default would remove agents.max_threads instead.
 	const root = await mkdtemp(join(tmpdir(), "omo-codex-script-config-multi-agent-role-section-"));
 	const configPath = join(root, "config.toml");
@@ -517,8 +516,7 @@ test("#given managed agent role sections #when script installer updates config #
 
 	// then
 	const config = await readFile(configPath, "utf8");
-	assert.match(config, /max_threads = 1000/);
-	assert.doesNotMatch(config, /max_threads = 16/);
+	assert.match(config, /^\s*max_threads\s*=\s*16$/m);
 	assert.match(config, /\[agents\.explorer\]/);
 	assert.match(config, /description = "read-only explorer"/);
 	assert.match(config, /config_file = "\.\/agents\/explorer\.toml"/);
@@ -572,7 +570,7 @@ test("#given existing trust and lsp blocks #when updating config #then existing
 function multiAgentV2Section(config) {
 	const header = "[features.multi_agent_v2]";
 	const start = config.indexOf(header);
-	assert.notEqual(start, -1);
+	if (start === -1) return "";
 	const rest = config.slice(start);
 	const nextHeader = rest.slice(1).search(/^\[/m);
 	return nextHeader === -1 ? rest : rest.slice(0, nextHeader + 1);