refactor(desktop): unify sidecar controller interface

Author: Hona

packages/desktop/src/main/sidecar/controller.test.ts

@@ -0,0 +1,31 @@
+import { describe, expect, test, vi } from "bun:test"
+import { BaseSidecarController } from "./types"
+
+describe("SidecarController Interface", () => {
+  test("manages sidecar lifecycle state transitions", async () => {
+    const fetchUrl = vi.fn().mockResolvedValue("http://127.0.0.1:4096")
+    const stopFn = vi.fn().mockResolvedValue(undefined)
+
+    const controller = new BaseSidecarController("local-1", "local", fetchUrl, stopFn)
+
+    expect(controller.status).toBe("idle")
+
+    const url = await controller.getUrl()
+    expect(url).toBe("http://127.0.0.1:4096")
+    expect(controller.status).toBe("ready")
+
+    await controller.stop()
+    expect(stopFn).toHaveBeenCalled()
+    expect(controller.status).toBe("stopped")
+  })
+
+  test("handles startup failures gracefully", async () => {
+    const fetchUrl = vi.fn().mockRejectedValue(new Error("Failed to start server"))
+    const stopFn = vi.fn().mockResolvedValue(undefined)
+
+    const controller = new BaseSidecarController("wsl-ubuntu", "wsl", fetchUrl, stopFn)
+
+    expect(controller.getUrl()).rejects.toThrow("Failed to start server")
+    expect(controller.status).toBe("failed")
+  })
+})

packages/desktop/src/main/sidecar/types.ts

@@ -0,0 +1,45 @@
+export type SidecarKind = "local" | "wsl"
+export type SidecarStatus = "idle" | "starting" | "ready" | "failed" | "stopped"
+
+export interface SidecarInstance {
+  readonly id: string
+  readonly kind: SidecarKind
+  readonly status: SidecarStatus
+  getUrl(): Promise<string | null>
+  stop(): Promise<void>
+}
+
+export class BaseSidecarController implements SidecarInstance {
+  private _status: SidecarStatus = "idle"
+
+  constructor(
+    public readonly id: string,
+    public readonly kind: SidecarKind,
+    private fetchUrl: () => Promise<string | null>,
+    private stopFn: () => Promise<void>,
+  ) {}
+
+  get status(): SidecarStatus {
+    return this._status
+  }
+
+  async getUrl(): Promise<string | null> {
+    this._status = "starting"
+    try {
+      const url = await this.fetchUrl()
+      this._status = url ? "ready" : "failed"
+      return url
+    } catch (err) {
+      this._status = "failed"
+      throw err
+    }
+  }
+
+  async stop(): Promise<void> {
+    try {
+      await this.stopFn()
+    } finally {
+      this._status = "stopped"
+    }
+  }
+}