@omnidev-ai/cli 0.1.0 → 0.3.0

package/src/lib/dynamic-app.ts

@@ -6,9 +6,8 @@ import { capabilityRoutes } from "../commands/capability";
 // Core commands
 import { doctorCommand } from "../commands/doctor";
 import { initCommand } from "../commands/init";
-import { mcpRoutes } from "../commands/mcp";
 import { profileRoutes } from "../commands/profile";
-import { serveCommand } from "../commands/serve";
+import { providerRoutes } from "../commands/provider";
 import { syncCommand } from "../commands/sync";
 import { debug } from "./debug";
 
@@ -20,11 +19,10 @@ export async function buildDynamicApp() {
 	const routes: Record<string, unknown> = {
 		init: initCommand,
 		doctor: doctorCommand,
-		serve: serveCommand,
 		sync: syncCommand,
 		capability: capabilityRoutes,
 		profile: profileRoutes,
-		mcp: mcpRoutes,
+		provider: providerRoutes,
 	};
 
 	debug("Core routes registered", Object.keys(routes));

package/src/prompts/provider.ts

@@ -1,15 +1,17 @@
 import { checkbox } from "@inquirer/prompts";
-import type { Provider } from "@omnidev-ai/core";
+import type { ProviderId } from "@omnidev-ai/core";
 
-export async function promptForProvider(): Promise<Provider[]> {
+export async function promptForProviders(): Promise<ProviderId[]> {
 	const answers = await checkbox({
 		message: "Select your AI provider(s):",
 		choices: [
-			{ name: "Claude (recommended)", value: "claude", checked: true },
+			{ name: "Claude Code (Claude CLI)", value: "claude-code", checked: true },
+			{ name: "Cursor", value: "cursor", checked: false },
 			{ name: "Codex", value: "codex", checked: false },
+			{ name: "OpenCode", value: "opencode", checked: false },
 		],
 		required: true,
 	});
 
-	return answers as Provider[];
+	return answers as ProviderId[];
 }

package/src/commands/mcp.ts

@@ -1,113 +0,0 @@
-import { existsSync } from "node:fs";
-import { buildCommand, buildRouteMap } from "@stricli/core";
-
-const STATUS_FILE = ".omni/state/mcp-status.json";
-
-// Local type definition to avoid circular dependency with @omnidev-ai/mcp
-interface McpChildProcess {
-	capabilityId: string;
-	pid: number | null;
-	status: "starting" | "connected" | "disconnected" | "error";
-	transport: "stdio" | "sse" | "http";
-	lastHealthCheck?: string;
-	error?: string;
-	toolCount?: number;
-}
-
-interface McpStatusFile {
-	lastUpdated: string;
-	relayPort: number;
-	children: McpChildProcess[];
-}
-
-/**
- * Run the mcp status command.
- */
-export async function runMcpStatus(): Promise<void> {
-	try {
-		if (!existsSync(STATUS_FILE)) {
-			console.log("No MCP status found.");
-			console.log("");
-			console.log("Is the OmniDev server running? Start it with:");
-			console.log("  omnidev serve");
-			return;
-		}
-
-		const statusText = await Bun.file(STATUS_FILE).text();
-		const status = JSON.parse(statusText) as McpStatusFile;
-
-		console.log("");
-		console.log("=== MCP Controller Status ===");
-		console.log("");
-		console.log(`Last updated: ${status.lastUpdated}`);
-		console.log(`Relay port:   ${status.relayPort}`);
-		console.log("");
-
-		if (status.children.length === 0) {
-			console.log("No MCP children running.");
-			console.log("");
-			console.log("Enable a capability with [mcp] configuration to spawn child MCPs.");
-			return;
-		}
-
-		console.log(`Child Processes (${status.children.length}):`);
-		console.log("");
-
-		for (const child of status.children) {
-			const statusIcon = getStatusIcon(child.status);
-			console.log(`  ${statusIcon} ${child.capabilityId}`);
-			console.log(`      Status:    ${child.status}`);
-			console.log(`      Transport: ${child.transport}`);
-			if (child.pid) {
-				console.log(`      PID:       ${child.pid}`);
-			}
-			if (child.toolCount !== undefined) {
-				console.log(`      Tools:     ${child.toolCount}`);
-			}
-			if (child.lastHealthCheck) {
-				console.log(`      Last check: ${child.lastHealthCheck}`);
-			}
-			if (child.error) {
-				console.log(`      Error:     ${child.error}`);
-			}
-			console.log("");
-		}
-	} catch (error) {
-		console.error("Error reading MCP status:", error);
-		process.exit(1);
-	}
-}
-
-function getStatusIcon(status: string): string {
-	switch (status) {
-		case "connected":
-			return "\u2713"; // checkmark
-		case "starting":
-			return "\u2022"; // bullet
-		case "disconnected":
-			return "\u2717"; // x mark
-		case "error":
-			return "\u2717"; // x mark
-		default:
-			return "?";
-	}
-}
-
-const statusCommand = buildCommand({
-	docs: {
-		brief: "Show MCP controller status",
-	},
-	parameters: {},
-	async func() {
-		await runMcpStatus();
-	},
-});
-
-export const mcpRoutes = buildRouteMap({
-	routes: {
-		status: statusCommand,
-	},
-	docs: {
-		brief: "MCP controller commands",
-	},
-});

package/src/commands/serve.test.ts

@@ -1,184 +0,0 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
-import { existsSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { runServe } from "./serve";
-
-// Create test fixtures directory
-const testDir = join(process.cwd(), "test-fixtures-serve");
-
-beforeEach(() => {
-	// Clean up and create fresh test directory
-	if (existsSync(testDir)) {
-		rmSync(testDir, { recursive: true, force: true });
-	}
-	mkdirSync(testDir, { recursive: true });
-	process.chdir(testDir);
-});
-
-afterEach(() => {
-	// Return to original directory and clean up
-	process.chdir(join(testDir, ".."));
-	if (existsSync(testDir)) {
-		rmSync(testDir, { recursive: true, force: true });
-	}
-});
-
-describe("serve command", () => {
-	test("should fail when OmniDev is not initialized", async () => {
-		const mockExit = mock((code?: number) => {
-			throw new Error(`process.exit: ${code}`);
-		}) as typeof process.exit;
-		const originalExit = process.exit;
-		process.exit = mockExit;
-
-		try {
-			await expect(runServe({})).rejects.toThrow("process.exit: 1");
-			expect(mockExit).toHaveBeenCalledWith(1);
-		} finally {
-			process.exit = originalExit;
-		}
-	});
-
-	test("should fail when .omni/ directory is missing", async () => {
-		// Don't create .omni/ - test expects it to be missing
-
-		const mockExit = mock((code?: number) => {
-			throw new Error(`process.exit: ${code}`);
-		}) as typeof process.exit;
-		const originalExit = process.exit;
-		process.exit = mockExit;
-
-		try {
-			await expect(runServe({})).rejects.toThrow("process.exit: 1");
-			expect(mockExit).toHaveBeenCalledWith(1);
-		} finally {
-			process.exit = originalExit;
-		}
-	});
-
-	test("should fail when profile does not exist", async () => {
-		// Set up directories
-		mkdirSync(".omni", { recursive: true });
-		mkdirSync(".omni", { recursive: true });
-
-		// Create a config without the requested profile
-		writeFileSync(
-			"omni.toml",
-			`
-[capability]
-project = "test"
-
-[profiles.default]
-`,
-		);
-
-		const mockExit = mock((code?: number) => {
-			throw new Error(`process.exit: ${code}`);
-		}) as typeof process.exit;
-		const originalExit = process.exit;
-		process.exit = mockExit;
-
-		try {
-			await expect(runServe({ profile: "nonexistent" })).rejects.toThrow("process.exit: 1");
-			expect(mockExit).toHaveBeenCalledWith(1);
-		} finally {
-			process.exit = originalExit;
-		}
-	});
-
-	test("should set profile when provided and valid", async () => {
-		// Set up directories
-		mkdirSync(".omni", { recursive: true });
-		mkdirSync(".omni", { recursive: true });
-
-		// Create config with profiles
-		writeFileSync(
-			"omni.toml",
-			`project = "test"
-active_profile = "default"
-
-[profiles.default]
-capabilities = []
-
-[profiles.testing]
-capabilities = []
-`,
-		);
-
-		// Mock startServer to prevent actual server start
-		const mockStartServer = mock(async () => {
-			// Server started successfully, do nothing
-		});
-
-		// Mock the import of @omnidev-ai/mcp
-		const originalImport = globalThis[Symbol.for("Bun.lazy")];
-		// biome-ignore lint/suspicious/noExplicitAny: Testing requires dynamic mocking
-		(globalThis as any).import = mock(async (module: string) => {
-			if (module === "@omnidev-ai/mcp") {
-				return { startServer: mockStartServer };
-			}
-			throw new Error(`Unexpected import: ${module}`);
-		});
-
-		const mockExit = mock((code?: number) => {
-			throw new Error(`process.exit: ${code}`);
-		}) as typeof process.exit;
-		const originalExit = process.exit;
-		process.exit = mockExit;
-
-		try {
-			// This should fail because startServer will actually run, but that's OK for this test
-			// We just want to verify that setActiveProfile was called
-			await runServe({ profile: "testing" }).catch(() => {
-				// Ignore the error from startServer
-			});
-
-			// Check that active profile was written to state file (not config.toml)
-			const stateContent = await Bun.file(".omni/state/active-profile").text();
-			expect(stateContent).toBe("testing");
-		} finally {
-			process.exit = originalExit;
-			// biome-ignore lint/suspicious/noExplicitAny: Restore original import
-			if (originalImport) (globalThis as any).import = originalImport;
-		}
-	});
-
-	test("should start server without profile flag", async () => {
-		// Set up directories
-		mkdirSync(".omni", { recursive: true });
-		mkdirSync(".omni", { recursive: true });
-
-		// Create config
-		writeFileSync(
-			"omni.toml",
-			`
-[capability]
-project = "test"
-default_profile = "default"
-
-[profiles.default]
-`,
-		);
-
-		// We can't actually test server startup without complex mocking,
-		// so we'll just verify the command passes initial checks
-		const mockExit = mock((code?: number) => {
-			throw new Error(`process.exit: ${code}`);
-		}) as typeof process.exit;
-		const originalExit = process.exit;
-		process.exit = mockExit;
-
-		try {
-			// This will fail at the import stage, but that's expected
-			await runServe({}).catch((error) => {
-				// Should fail on import or server start, not on validation
-				expect(error).toBeDefined();
-			});
-
-			// No profile should be written when flag not provided
-			expect(existsSync(".omni/active-profile")).toBe(false);
-		} finally {
-			process.exit = originalExit;
-		}
-	});
-});

package/src/commands/serve.ts

@@ -1,63 +0,0 @@
-import { existsSync } from "node:fs";
-import { setActiveProfile } from "@omnidev-ai/core";
-import { buildCommand } from "@stricli/core";
-
-interface ServeFlags {
-	profile?: string;
-}
-
-export async function runServe(flags: ServeFlags): Promise<void> {
-	console.log("Starting OmniDev MCP server...");
-
-	// Check if OmniDev is initialized
-	if (!existsSync(".omni")) {
-		console.error("✗ OmniDev not initialized. Run: omnidev init");
-		process.exit(1);
-	}
-
-	// Set profile if provided
-	if (flags.profile) {
-		console.log(`Setting profile to: ${flags.profile}`);
-		try {
-			// Validate profile exists in config
-			const { loadConfig } = await import("@omnidev-ai/core");
-			const config = await loadConfig();
-			if (!config.profiles?.[flags.profile]) {
-				console.error(`✗ Profile "${flags.profile}" not found in config`);
-				process.exit(1);
-			}
-			await setActiveProfile(flags.profile);
-		} catch (error) {
-			console.error(`✗ Failed to set profile: ${error}`);
-			process.exit(1);
-		}
-	}
-
-	// Import and start the MCP server
-	try {
-		const { startServer } = await import("@omnidev-ai/mcp");
-		console.log("✓ Server starting...");
-		await startServer();
-	} catch (error) {
-		console.error(`✗ Failed to start server: ${error}`);
-		process.exit(1);
-	}
-}
-
-export const serveCommand = buildCommand({
-	docs: {
-		brief: "Start the OmniDev MCP server",
-		fullDescription: "Starts the MCP server that exposes omni_query and omni_execute tools to LLMs",
-	},
-	parameters: {
-		flags: {
-			profile: {
-				kind: "parsed" as const,
-				parse: String,
-				brief: "Set active profile before starting server",
-				optional: true,
-			},
-		},
-	},
-	func: runServe,
-});