Merge pull request #2 from tsirysndr/feat/overrides-start-vm-options

Add CLI flag parsing for overriding virtual machine configuration in start command

deno.json

···
   },
   "imports": {
     "@cliffy/command": "jsr:@cliffy/command@^1.0.0-rc.8",
+    "@cliffy/flags": "jsr:@cliffy/flags@^1.0.0-rc.8",
     "@cliffy/table": "jsr:@cliffy/table@^1.0.0-rc.8",
     "@db/sqlite": "jsr:@db/sqlite@^0.12.0",
     "@paralleldrive/cuid2": "npm:@paralleldrive/cuid2@^3.0.4",

deno.lock

···
   "specifiers": {
     "jsr:@cliffy/command@^1.0.0-rc.8": "1.0.0-rc.8",
     "jsr:@cliffy/flags@1.0.0-rc.8": "1.0.0-rc.8",
+    "jsr:@cliffy/flags@^1.0.0-rc.8": "1.0.0-rc.8",
     "jsr:@cliffy/internal@1.0.0-rc.8": "1.0.0-rc.8",
     "jsr:@cliffy/table@1.0.0-rc.8": "1.0.0-rc.8",
     "jsr:@cliffy/table@^1.0.0-rc.8": "1.0.0-rc.8",
···
     "@cliffy/command@1.0.0-rc.8": {
       "integrity": "758147790797c74a707e5294cc7285df665422a13d2a483437092ffce40b5557",
       "dependencies": [
-        "jsr:@cliffy/flags",
+        "jsr:@cliffy/flags@1.0.0-rc.8",
         "jsr:@cliffy/internal",
         "jsr:@cliffy/table@1.0.0-rc.8",
         "jsr:@std/fmt@~1.0.2",
···
   "workspace": {
     "dependencies": [
       "jsr:@cliffy/command@^1.0.0-rc.8",
+      "jsr:@cliffy/flags@^1.0.0-rc.8",
       "jsr:@cliffy/table@^1.0.0-rc.8",
       "jsr:@db/sqlite@0.12",
       "jsr:@soapbox/kysely-deno-sqlite@^2.2.0",

main.ts

···
     })
     .command("start", "Start a virtual machine")
     .arguments("<vm-name:string>")
-    .option("--detach, -d", "Run VM in the background and print VM name")
+    .option("-c, --cpu <type:string>", "Type of CPU to emulate", {
+      default: "host",
+    })
+    .option("-C, --cpus <number:number>", "Number of CPU cores", {
+      default: 2,
+    })
+    .option("-m, --memory <size:string>", "Amount of memory for the VM", {
+      default: "2G",
+    })
+    .option("-i, --image <path:string>", "Path to VM disk image")
+    .option(
+      "--disk-format <format:string>",
+      "Disk image format (e.g., qcow2, raw)",
+      {
+        default: "raw",
+      },
+    )
+    .option(
+      "--size <size:string>",
+      "Size of the VM disk image to create if it doesn't exist (e.g., 20G)",
+      {
+        default: "20G",
+      },
+    )
+    .option(
+      "-b, --bridge <name:string>",
+      "Name of the network bridge to use for networking (e.g., br0)",
+    )
+    .option(
+      "-d, --detach",
+      "Run VM in the background and print VM name",
+    )
+    .option(
+      "-p, --port-forward <mappings:string>",
+      "Port forwarding rules in the format hostPort:guestPort (comma-separated for multiple)",
+    )
     .action(async (options: unknown, vmName: string) => {
       await start(vmName, Boolean((options as { detach: boolean }).detach));
     })

src/subcommands/start.ts

···
+import { parseFlags } from "@cliffy/flags";
 import _ from "lodash";
 import { LOGS_DIR } from "../constants.ts";
+import type { VirtualMachine } from "../db.ts";
 import { getInstanceState, updateInstanceState } from "../state.ts";
 import { setupNATNetworkArgs } from "../utils.ts";
 
 export default async function (name: string, detach: boolean = false) {
-  const vm = await getInstanceState(name);
+  let vm = await getInstanceState(name);
   if (!vm) {
     console.error(
       `Virtual machine with name or ID ${name} not found.`,
···
   }
 
   console.log(`Starting virtual machine ${vm.name} (ID: ${vm.id})...`);
+
+  vm = mergeFlags(vm);
 
   const qemuArgs = [
     ..._.compact([vm.bridge && "qemu-system-x86_64"]),
···
     }
   }
 }
+
+function mergeFlags(vm: VirtualMachine): VirtualMachine {
+  const { flags } = parseFlags(Deno.args);
+  return {
+    ...vm,
+    memory: flags.memory ? String(flags.memory) : vm.memory,
+    cpus: flags.cpus ? Number(flags.cpus) : vm.cpus,
+    cpu: flags.cpu ? String(flags.cpu) : vm.cpu,
+    diskFormat: flags.diskFormat ? String(flags.diskFormat) : vm.diskFormat,
+    portForward: flags.portForward ? String(flags.portForward) : vm.portForward,
+    drivePath: flags.image ? String(flags.image) : vm.drivePath,
+    bridge: flags.bridge ? String(flags.bridge) : vm.bridge,
+    diskSize: flags.size ? String(flags.size) : vm.diskSize,
+  };
+}