feat(desktop): expand Create Workspace wizard with source config and image catalog

desktop/electron-builder.yml

@@ -15,6 +15,8 @@ extraResources:
     to: bin/
     filter:
       - "**/*"
+  - from: resources/image-catalog-seed.json
+    to: image-catalog-seed.json
 
 mac:
   category: public.app-category.developer-tools

desktop/resources/image-catalog-seed.json

@@ -0,0 +1,92 @@
+{
+  "version": 1,
+  "categories": [
+    { "id": "languages", "label": "Languages" },
+    { "id": "universal", "label": "Universal" }
+  ],
+  "images": [
+    {
+      "id": "universal",
+      "ref": "mcr.microsoft.com/devcontainers/universal:2",
+      "name": "Universal",
+      "description": "Multi-language image with common tools preinstalled.",
+      "categories": ["universal"],
+      "icon": "universal",
+      "featured": true
+    },
+    {
+      "id": "python-3.12",
+      "ref": "mcr.microsoft.com/devcontainers/python:3.12",
+      "name": "Python 3.12",
+      "description": "Python 3.12 development image.",
+      "categories": ["languages"],
+      "icon": "Python",
+      "featured": true
+    },
+    {
+      "id": "node-20",
+      "ref": "mcr.microsoft.com/devcontainers/javascript-node:20",
+      "name": "Node.js 20",
+      "description": "Node.js 20 development image.",
+      "categories": ["languages"],
+      "icon": "Node.js",
+      "featured": true
+    },
+    {
+      "id": "go-1",
+      "ref": "mcr.microsoft.com/devcontainers/go:1",
+      "name": "Go",
+      "description": "Go development image.",
+      "categories": ["languages"],
+      "icon": "Go"
+    },
+    {
+      "id": "rust-1",
+      "ref": "mcr.microsoft.com/devcontainers/rust:1",
+      "name": "Rust",
+      "description": "Rust development image.",
+      "categories": ["languages"],
+      "icon": "Rust"
+    },
+    {
+      "id": "java-21",
+      "ref": "mcr.microsoft.com/devcontainers/java:21",
+      "name": "Java",
+      "description": "Java development image.",
+      "categories": ["languages"],
+      "icon": "Java"
+    },
+    {
+      "id": "php-8",
+      "ref": "mcr.microsoft.com/devcontainers/php:8",
+      "name": "PHP",
+      "description": "PHP development image.",
+      "categories": ["languages"],
+      "icon": "PHP"
+    },
+    {
+      "id": "cpp-1",
+      "ref": "mcr.microsoft.com/devcontainers/cpp:1",
+      "name": "C++",
+      "description": "C++ development image.",
+      "categories": ["languages"],
+      "icon": "C++"
+    },
+    {
+      "id": "dotnet-8",
+      "ref": "mcr.microsoft.com/devcontainers/dotnet:8",
+      "name": ".NET",
+      "description": ".NET development image.",
+      "categories": ["languages"],
+      "icon": ".NET"
+    },
+    {
+      "id": "ruby-3",
+      "ref": "mcr.microsoft.com/devcontainers/ruby:3",
+      "name": "Ruby",
+      "description": "Ruby development image.",
+      "categories": ["languages"],
+      "icon": "Ruby"
+    }
+  ]
+}

desktop/src/main/__tests__/image-catalog.test.ts

@@ -0,0 +1,177 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest"
+import { mkdtemp, rm, writeFile, readFile } from "node:fs/promises"
+import { tmpdir } from "node:os"
+import { join } from "node:path"
+import { loadCatalog, __setFetchForTest } from "../image-catalog.js"
+
+const SEED = {
+  version: 1,
+  categories: [{ id: "languages", label: "Languages" }],
+  images: [
+    { id: "seed-img", ref: "seed:1", name: "Seed", categories: ["languages"] },
+  ],
+}
+const REMOTE = {
+  version: 1,
+  categories: [{ id: "languages", label: "Languages" }],
+  images: [
+    { id: "remote-img", ref: "remote:1", name: "Remote", categories: ["languages"] },
+  ],
+}
+
+let dir: string
+let cachePath: string
+let seedPath: string
+
+beforeEach(async () => {
+  dir = await mkdtemp(join(tmpdir(), "catalog-"))
+  cachePath = join(dir, "image-catalog.json")
+  seedPath = join(dir, "seed.json")
+  await writeFile(seedPath, JSON.stringify(SEED))
+})
+
+afterEach(async () => {
+  __setFetchForTest(undefined)
+  await rm(dir, { recursive: true, force: true })
+})
+
+describe("loadCatalog", () => {
+  it("fetches remote and writes cache when remote succeeds", async () => {
+    __setFetchForTest(async () => new Response(JSON.stringify(REMOTE)))
+    const result = await loadCatalog({
+      url: "https://example/catalog.json",
+      cachePath,
+      seedPath,
+      ttlMs: 1000,
+      force: true,
+    })
+    expect(result.catalog.images[0].id).toBe("remote-img")
+    expect(result.origin).toBe("remote")
+    const cached = JSON.parse(await readFile(cachePath, "utf8"))
+    expect(cached.catalog.images[0].id).toBe("remote-img")
+    expect(typeof cached.fetchedAt).toBe("number")
+  })
+
+  it("falls back to on-disk cache when remote fails", async () => {
+    await writeFile(
+      cachePath,
+      JSON.stringify({ fetchedAt: Date.now(), catalog: REMOTE }),
+    )
+    __setFetchForTest(async () => {
+      throw new Error("network down")
+    })
+    const result = await loadCatalog({
+      url: "https://example/catalog.json",
+      cachePath,
+      seedPath,
+      ttlMs: 1000,
+      force: true,
+    })
+    expect(result.catalog.images[0].id).toBe("remote-img")
+    expect(result.origin).toBe("cache")
+  })
+
+  it("falls back to seed when remote fails and no cache exists", async () => {
+    __setFetchForTest(async () => {
+      throw new Error("network down")
+    })
+    const result = await loadCatalog({
+      url: "https://example/catalog.json",
+      cachePath,
+      seedPath,
+      ttlMs: 1000,
+      force: true,
+    })
+    expect(result.catalog.images[0].id).toBe("seed-img")
+    expect(result.origin).toBe("seed")
+  })
+
+  it("returns fresh cache without fetching when within TTL and not forced", async () => {
+    await writeFile(
+      cachePath,
+      JSON.stringify({ fetchedAt: Date.now(), catalog: REMOTE }),
+    )
+    const fetchSpy = vi.fn()
+    __setFetchForTest(fetchSpy as unknown as typeof fetch)
+    const result = await loadCatalog({
+      url: "https://example/catalog.json",
+      cachePath,
+      seedPath,
+      ttlMs: 60_000,
+      force: false,
+    })
+    expect(fetchSpy).not.toHaveBeenCalled()
+    expect(result.catalog.images[0].id).toBe("remote-img")
+    expect(result.origin).toBe("cache")
+  })
+
+  it("refetches and overwrites when cache is stale even without force", async () => {
+    await writeFile(
+      cachePath,
+      JSON.stringify({ fetchedAt: Date.now() - 10_000, catalog: SEED }),
+    )
+    __setFetchForTest(async () => new Response(JSON.stringify(REMOTE)))
+    const result = await loadCatalog({
+      url: "https://example/catalog.json",
+      cachePath,
+      seedPath,
+      ttlMs: 1000,
+      force: false,
+    })
+    expect(result.catalog.images[0].id).toBe("remote-img")
+    expect(result.origin).toBe("remote")
+    const cached = JSON.parse(await readFile(cachePath, "utf8"))
+    expect(cached.catalog.images[0].id).toBe("remote-img")
+  })
+
+  it("ignores a corrupt cache file and falls back to seed", async () => {
+    await writeFile(cachePath, "{ this is not valid json")
+    __setFetchForTest(async () => {
+      throw new Error("network down")
+    })
+    const result = await loadCatalog({
+      url: "https://example/catalog.json",
+      cachePath,
+      seedPath,
+      ttlMs: 1000,
+      force: false,
+    })
+    expect(result.catalog.images[0].id).toBe("seed-img")
+    expect(result.origin).toBe("seed")
+  })
+
+  it("rejects a structurally malformed remote payload and falls back to seed", async () => {
+    // images present but entries miss required fields (no ref/categories).
+    __setFetchForTest(
+      async () =>
+        new Response(
+          JSON.stringify({ version: 1, categories: [], images: [{ id: "x" }] }),
+        ),
+    )
+    const result = await loadCatalog({
+      url: "https://example/catalog.json",
+      cachePath,
+      seedPath,
+      ttlMs: 1000,
+      force: true,
+    })
+    expect(result.catalog.images[0].id).toBe("seed-img")
+    expect(result.origin).toBe("seed")
+  })
+
+  it("passes an abort signal to the remote fetch", async () => {
+    let receivedSignal: AbortSignal | undefined
+    __setFetchForTest(async (_url, init) => {
+      receivedSignal = (init as RequestInit | undefined)?.signal ?? undefined
+      return new Response(JSON.stringify(REMOTE))
+    })
+    await loadCatalog({
+      url: "https://example/catalog.json",
+      cachePath,
+      seedPath,
+      ttlMs: 1000,
+      force: true,
+    })
+    expect(receivedSignal).toBeInstanceOf(AbortSignal)
+  })
+})