@omnidev-ai/cli 0.3.0 → 0.5.0

package/src/commands/profile.test.ts

@@ -1,339 +0,0 @@
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { mkdirSync } from "node:fs";
-import { setupTestDir } from "@omnidev-ai/core/test-utils";
-import { runProfileList, runProfileSet } from "./profile";
-
-describe("profile commands", () => {
-	setupTestDir("profile-test-", { chdir: true });
-	let originalExit: typeof process.exit;
-	let exitCode: number | undefined;
-	let consoleOutput: string[];
-	let consoleErrors: string[];
-	let originalLog: typeof console.log;
-	let originalError: typeof console.error;
-
-	beforeEach(() => {
-		// Mock process.exit
-		exitCode = undefined;
-		originalExit = process.exit;
-		process.exit = ((code?: number) => {
-			exitCode = code ?? 0;
-			throw new Error(`process.exit(${code})`);
-		}) as typeof process.exit;
-
-		// Mock console
-		consoleOutput = [];
-		consoleErrors = [];
-		originalLog = console.log;
-		originalError = console.error;
-		console.log = (...args: unknown[]) => {
-			consoleOutput.push(args.join(" "));
-		};
-		console.error = (...args: unknown[]) => {
-			consoleErrors.push(args.join(" "));
-		};
-	});
-
-	afterEach(() => {
-		process.exit = originalExit;
-		console.log = originalLog;
-		console.error = originalError;
-	});
-
-	describe("runProfileList", () => {
-		test("should show error when config file does not exist", async () => {
-			try {
-				await runProfileList();
-			} catch {
-				// Expected to throw due to process.exit mock
-			}
-
-			expect(exitCode).toBe(1);
-			expect(consoleOutput.join("\n")).toContain("No config file found");
-			expect(consoleOutput.join("\n")).toContain("Run: omnidev init");
-		});
-
-		test("should show message when no profiles defined", async () => {
-			// Create minimal config without profiles
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-active_profile = "default"
-`,
-			);
-
-			await runProfileList();
-
-			expect(exitCode).toBeUndefined();
-			expect(consoleOutput.join("\n")).toContain("No profiles defined");
-			expect(consoleOutput.join("\n")).toContain("Using default capabilities");
-		});
-
-		test("should list all profiles from config", async () => {
-			// Create config with profiles
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-active_profile = "default"
-
-[profiles.default]
-capabilities = []
-
-[profiles.planning]
-capabilities = ["tasks", "planner"]
-
-[profiles.coding]
-capabilities = []
-`,
-			);
-
-			await runProfileList();
-
-			expect(exitCode).toBeUndefined();
-			const output = consoleOutput.join("\n");
-			expect(output).toContain("Available Profiles:");
-			expect(output).toContain("default");
-			expect(output).toContain("planning");
-			expect(output).toContain("coding");
-		});
-
-		test("should show active profile with marker", async () => {
-			// Create config with profiles
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-active_profile = "planning"
-
-[profiles.default]
-capabilities = []
-
-[profiles.planning]
-capabilities = ["planner"]
-`,
-			);
-
-			await runProfileList();
-
-			expect(exitCode).toBeUndefined();
-			const output = consoleOutput.join("\n");
-			expect(output).toContain("● planning (active)");
-			expect(output).toContain("○ default");
-		});
-
-		test("should show profile capabilities", async () => {
-			// Create config with profiles
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-active_profile = "default"
-
-[profiles.default]
-capabilities = []
-
-[profiles.planning]
-capabilities = ["planner", "tasks"]
-`,
-			);
-
-			await runProfileList();
-
-			expect(exitCode).toBeUndefined();
-			const output = consoleOutput.join("\n");
-			expect(output).toContain("Capabilities: planner, tasks");
-		});
-
-		test("should use default_profile when no active profile", async () => {
-			// Create config with active_profile
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-active_profile = "planning"
-
-[profiles.planning]
-capabilities = ["planner"]
-`,
-			);
-
-			await runProfileList();
-
-			expect(exitCode).toBeUndefined();
-			const output = consoleOutput.join("\n");
-			expect(output).toContain("● planning (active)");
-		});
-
-		test("should handle invalid config gracefully", async () => {
-			// Create invalid config
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write("omni.toml", "invalid toml [[[");
-
-			try {
-				await runProfileList();
-			} catch {
-				// Expected to throw due to process.exit mock
-			}
-
-			expect(exitCode).toBe(1);
-			expect(consoleErrors.join("\n")).toContain("Error loading profiles");
-		});
-	});
-
-	describe("runProfileSet", () => {
-		test("should show error when config file does not exist", async () => {
-			try {
-				await runProfileSet("planning");
-			} catch {
-				// Expected to throw due to process.exit mock
-			}
-
-			expect(exitCode).toBe(1);
-			expect(consoleOutput.join("\n")).toContain("No config file found");
-			expect(consoleOutput.join("\n")).toContain("Run: omnidev init");
-		});
-
-		test("should show error when profile does not exist", async () => {
-			// Create config without the requested profile
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-
-[profiles.default]
-capabilities = []
-`,
-			);
-
-			try {
-				await runProfileSet("nonexistent");
-			} catch {
-				// Expected to throw due to process.exit mock
-			}
-
-			expect(exitCode).toBe(1);
-			const output = consoleOutput.join("\n");
-			expect(output).toContain('Profile "nonexistent" not found');
-			expect(output).toContain("Available profiles:");
-			expect(output).toContain("- default");
-		});
-
-		test("should set active profile", async () => {
-			// Create config with profiles
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-
-[profiles.default]
-capabilities = []
-
-[profiles.planning]
-capabilities = ["planner"]
-`,
-			);
-
-			await runProfileSet("planning");
-
-			expect(exitCode).toBeUndefined();
-			expect(consoleOutput.join("\n")).toContain("Active profile set to: planning");
-
-			// Verify active_profile was written to state file (not config.toml)
-			const stateContent = await Bun.file(".omni/state/active-profile").text();
-			expect(stateContent).toBe("planning");
-		});
-
-		test("should trigger agents sync after setting profile", async () => {
-			// Create config with profiles
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-active_profile = "default"
-
-[profiles.default]
-capabilities = []
-
-[profiles.planning]
-capabilities = []
-`,
-			);
-
-			await runProfileSet("planning");
-
-			expect(exitCode).toBeUndefined();
-			const output = consoleOutput.join("\n");
-			expect(output).toContain("Syncing agent configuration");
-		});
-
-		test("should show list of available profiles when profile not found", async () => {
-			// Create config with multiple profiles
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-
-[profiles.default]
-capabilities = []
-
-[profiles.planning]
-capabilities = []
-
-[profiles.coding]
-capabilities = []
-`,
-			);
-
-			try {
-				await runProfileSet("nonexistent");
-			} catch {
-				// Expected to throw due to process.exit mock
-			}
-
-			expect(exitCode).toBe(1);
-			const output = consoleOutput.join("\n");
-			expect(output).toContain("Available profiles:");
-			expect(output).toContain("- default");
-			expect(output).toContain("- planning");
-			expect(output).toContain("- coding");
-		});
-
-		test("should handle empty profiles config", async () => {
-			// Create config without any profiles
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write(
-				"omni.toml",
-				`project = "test-project"
-`,
-			);
-
-			try {
-				await runProfileSet("default");
-			} catch {
-				// Expected to throw due to process.exit mock
-			}
-
-			expect(exitCode).toBe(1);
-			const output = consoleOutput.join("\n");
-			expect(output).toContain('Profile "default" not found');
-			expect(output).toContain("(none defined)");
-		});
-
-		test("should handle invalid config gracefully", async () => {
-			// Create invalid config
-			mkdirSync(".omni", { recursive: true });
-			await Bun.write("omni.toml", "invalid toml [[[");
-
-			try {
-				await runProfileSet("planning");
-			} catch {
-				// Expected to throw due to process.exit mock
-			}
-
-			expect(exitCode).toBe(1);
-			expect(consoleErrors.join("\n")).toContain("Error setting profile");
-		});
-	});
-});

package/src/commands/profile.ts

@@ -1,153 +0,0 @@
-import { existsSync } from "node:fs";
-import { getEnabledAdapters } from "@omnidev-ai/adapters";
-import {
-	getActiveProfile,
-	loadConfig,
-	resolveEnabledCapabilities,
-	setActiveProfile,
-	syncAgentConfiguration,
-} from "@omnidev-ai/core";
-import { buildCommand, buildRouteMap } from "@stricli/core";
-
-const listCommand = buildCommand({
-	docs: {
-		brief: "List available profiles",
-	},
-	parameters: {},
-	async func() {
-		await runProfileList();
-	},
-});
-
-async function runSetCommand(_flags: Record<string, never>, profileName: string): Promise<void> {
-	await runProfileSet(profileName);
-}
-
-const setCommand = buildCommand({
-	docs: {
-		brief: "Set the active profile",
-	},
-	parameters: {
-		flags: {},
-		positional: {
-			kind: "tuple" as const,
-			parameters: [
-				{
-					brief: "Profile name",
-					parse: String,
-				},
-			],
-		},
-	},
-	func: runSetCommand,
-});
-
-export const profileRoutes = buildRouteMap({
-	routes: {
-		list: listCommand,
-		set: setCommand,
-	},
-	docs: {
-		brief: "Manage capability profiles",
-	},
-});
-
-export async function runProfileList(): Promise<void> {
-	try {
-		// Check if omni.toml exists
-		if (!existsSync("omni.toml")) {
-			console.log("✗ No config file found");
-			console.log("  Run: omnidev init");
-			process.exit(1);
-		}
-
-		// Load config
-		const config = await loadConfig();
-
-		// Get active profile
-		const activeProfile = (await getActiveProfile()) ?? config.active_profile ?? "default";
-
-		// Check if profiles exist
-		const profiles = config.profiles ?? {};
-		const profileNames = Object.keys(profiles);
-
-		if (profileNames.length === 0) {
-			console.log("No profiles defined in omni.toml");
-			console.log("");
-			console.log("Using default capabilities from omni.toml");
-			return;
-		}
-
-		// Display profiles
-		console.log("Available Profiles:");
-		console.log("");
-
-		for (const name of profileNames) {
-			const isActive = name === activeProfile;
-			const icon = isActive ? "●" : "○";
-			const profile = profiles[name];
-
-			if (profile === undefined) {
-				continue;
-			}
-
-			console.log(`${icon} ${name}${isActive ? " (active)" : ""}`);
-
-			// Show capabilities (including always-enabled)
-			const capabilities = resolveEnabledCapabilities(config, name);
-			if (capabilities.length > 0) {
-				console.log(`  Capabilities: ${capabilities.join(", ")}`);
-			} else {
-				console.log("  Capabilities: none");
-			}
-			console.log("");
-		}
-	} catch (error) {
-		console.error("✗ Error loading profiles:", error);
-		process.exit(1);
-	}
-}
-
-export async function runProfileSet(profileName: string): Promise<void> {
-	try {
-		// Check if omni.toml exists
-		if (!existsSync("omni.toml")) {
-			console.log("✗ No config file found");
-			console.log("  Run: omnidev init");
-			process.exit(1);
-		}
-
-		// Load config
-		const config = await loadConfig();
-
-		// Validate profile exists
-		const profiles = config.profiles ?? {};
-		if (!(profileName in profiles)) {
-			console.log(`✗ Profile "${profileName}" not found in omni.toml`);
-			console.log("");
-			console.log("Available profiles:");
-			const profileNames = Object.keys(profiles);
-			if (profileNames.length === 0) {
-				console.log("  (none defined)");
-			} else {
-				for (const name of profileNames) {
-					console.log(`  - ${name}`);
-				}
-			}
-			process.exit(1);
-		}
-
-		// Set active profile
-		await setActiveProfile(profileName);
-
-		console.log(`✓ Active profile set to: ${profileName}`);
-		console.log("");
-
-		// Auto-sync agent configuration with enabled adapters
-		const adapters = await getEnabledAdapters();
-		await syncAgentConfiguration({ adapters });
-	} catch (error) {
-		console.error("✗ Error setting profile:", error);
-		process.exit(1);
-	}
-}

package/src/commands/provider.test.ts

@@ -1,128 +0,0 @@
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { existsSync, mkdirSync, rmSync } from "node:fs";
-import { captureConsole, tmpdir } from "@omnidev-ai/core/test-utils";
-import { runProviderList, runProviderEnable, runProviderDisable } from "./provider";
-
-// Import the functions we need to test
-async function getProviderFunctions() {
-	const { readEnabledProviders, writeEnabledProviders } = await import("@omnidev-ai/core");
-	return { readEnabledProviders, writeEnabledProviders };
-}
-
-describe("provider commands", () => {
-	let testDir: string;
-	let originalCwd: string;
-
-	beforeEach(async () => {
-		originalCwd = process.cwd();
-		testDir = tmpdir("provider-test-");
-		process.chdir(testDir);
-
-		// Create basic OmniDev structure
-		mkdirSync(".omni/state", { recursive: true });
-		await Bun.write("omni.toml", 'project = "test"\n');
-	});
-
-	afterEach(() => {
-		process.chdir(originalCwd);
-		if (existsSync(testDir)) {
-			rmSync(testDir, { recursive: true, force: true });
-		}
-	});
-
-	describe("runProviderList", () => {
-		test("lists all providers with status", async () => {
-			const { writeEnabledProviders } = await getProviderFunctions();
-			await writeEnabledProviders(["claude-code", "cursor"]);
-
-			const { stdout } = await captureConsole(async () => {
-				await runProviderList();
-			});
-
-			const output = stdout.join("\n");
-			expect(output).toContain("Available providers:");
-			expect(output).toContain("Claude Code");
-			expect(output).toContain("Cursor");
-			expect(output).toContain("Codex");
-			expect(output).toContain("OpenCode");
-		});
-
-		test("shows enabled providers with marker", async () => {
-			const { writeEnabledProviders } = await getProviderFunctions();
-			await writeEnabledProviders(["claude-code"]);
-
-			const { stdout } = await captureConsole(async () => {
-				await runProviderList();
-			});
-
-			const output = stdout.join("\n");
-			// Should have at least one enabled marker
-			expect(output).toContain("●");
-		});
-
-		test("shows disabled providers with different marker", async () => {
-			const { writeEnabledProviders } = await getProviderFunctions();
-			await writeEnabledProviders(["claude-code"]);
-
-			const { stdout } = await captureConsole(async () => {
-				await runProviderList();
-			});
-
-			const output = stdout.join("\n");
-			// Should have disabled markers for other providers
-			expect(output).toContain("○");
-		});
-	});
-
-	describe("runProviderEnable", () => {
-		test("enables a provider", async () => {
-			const { writeEnabledProviders, readEnabledProviders } = await getProviderFunctions();
-			await writeEnabledProviders(["claude-code"]);
-
-			await captureConsole(async () => {
-				await runProviderEnable({}, "cursor");
-			});
-
-			const providers = await readEnabledProviders();
-			expect(providers).toContain("cursor");
-		});
-
-		test("shows success message", async () => {
-			const { writeEnabledProviders } = await getProviderFunctions();
-			await writeEnabledProviders(["claude-code"]);
-
-			const { stdout } = await captureConsole(async () => {
-				await runProviderEnable({}, "cursor");
-			});
-
-			const output = stdout.join("\n");
-			expect(output).toContain("✓ Enabled provider: Cursor");
-		});
-	});
-
-	describe("runProviderDisable", () => {
-		test("disables a provider", async () => {
-			const { writeEnabledProviders, readEnabledProviders } = await getProviderFunctions();
-			await writeEnabledProviders(["claude-code", "cursor"]);
-
-			await captureConsole(async () => {
-				await runProviderDisable({}, "cursor");
-			});
-
-			const providers = await readEnabledProviders();
-			expect(providers).not.toContain("cursor");
-		});
-
-		test("shows success message", async () => {
-			const { writeEnabledProviders } = await getProviderFunctions();
-			await writeEnabledProviders(["claude-code", "cursor"]);
-
-			const { stdout } = await captureConsole(async () => {
-				await runProviderDisable({}, "cursor");
-			});
-
-			const output = stdout.join("\n");
-			expect(output).toContain("✓ Disabled provider: Cursor");
-		});
-	});
-});