@miaws/miaw 1.18.5 → 1.18.6

package/test/cli/help/help-snapshots.test.ts

@@ -1,137 +0,0 @@
-// Help-text snapshots for every CLI command + key subcommand. Catches
-// accidental flag removals, renames, and reordering in a single sweep —
-// any change to the user-visible CLI surface shows up here as a diff.
-//
-// This is the broad coverage layer that makes the future Effect CLI
-// migration (yargs → effect-smol/cli) safe to attempt: if a refactor
-// preserves the surface, the snapshots stay green; if it doesn't, the
-// diff tells you exactly which command(s) changed.
-//
-// Snapshots are taken at COLUMNS=120 so wrapping is stable across
-// terminal sizes. The default miaw tui command is excluded —
-// `miaw --help` includes an ASCII banner that pulls in the install
-// version (changes per release), so we'd snapshot a moving target.
-import { describe, expect } from "bun:test"
-import { Effect } from "effect"
-import { EOL } from "os"
-import { cliIt } from "../../lib/cli-process"
-import { normalizeForSnapshot, PATH_SEP } from "../../lib/snapshot"
-
-// Composes `normalizeForSnapshot` (CRLF + tmpdir) with two help-specific
-// rules:
-//
-//   1. The harness's `oc-cli-XXX` subdir under TMPDIR collapses to `<HOME>`.
-//      `PATH_SEP` matches `/` and `\\` so the rule works on POSIX + Windows.
-//
-//   2. yargs wraps the `[string] [default: "..."]` clause based on the
-//      pre-normalized default's character length, so different random home
-//      path widths produce different leading-whitespace counts (or even
-//      line-wraps onto a fresh line on Windows). `\s+` matches both forms.
-function normalize(text: string): string {
-  return normalizeForSnapshot(text, {
-    pathReplacements: [
-      // Mixed-case [A-Za-z0-9] because node's mkdtemp suffix is mixed-case
-      // (the harness now uses FileSystem.makeTempDirectoryScoped under the
-      // hood). A `[a-z0-9]+` regex would leave uppercase chars trailing.
-      [new RegExp(`<TMPDIR>${PATH_SEP}oc-cli-[A-Za-z0-9]+`, "g"), "<HOME>"],
-      [/\s+\[string\] \[default: "<HOME>"\]/g, ' [string] [default: "<HOME>"]'],
-    ],
-  })
-}
-
-// Top-level commands. Order matches what `miaw --help` prints today;
-// keep it in that order so the snapshot file reads as a table of contents.
-// `completion` is intentionally excluded — it's a yargs built-in that emits
-// top-level help on `--help` and exits 1; not a real miaw command.
-const TOP_LEVEL = [
-  "acp",
-  "mcp",
-  "attach",
-  "run",
-  "debug",
-  "providers", // aliased to `auth`
-  "agent",
-  "upgrade",
-  "uninstall",
-  "serve",
-  "web",
-  "models",
-  "stats",
-  "export",
-  "import",
-  "github",
-  "pr",
-  "session",
-  "plugin",
-  "db",
-] as const
-
-// Subcommands worth pinning. Not exhaustive — the goal is one snapshot per
-// distinct argv shape, not every leaf. Add new entries when a subcommand
-// gains user-visible flags that we want to lock in.
-const SUBCOMMANDS = [
-  ["mcp", "list"],
-  ["mcp", "add"],
-  ["mcp", "auth"],
-  ["mcp", "logout"],
-  ["providers", "list"],
-  ["providers", "login"],
-  ["providers", "logout"],
-  ["agent", "create"],
-  ["agent", "list"],
-  ["session", "list"],
-  ["session", "delete"],
-  ["github", "install"],
-  ["github", "run"],
-  ["db", "path"],
-] as const
-
-// Fixed wrap width so a developer's terminal doesn't affect snapshots.
-// yargs honors COLUMNS; CI runners typically default to 80 which produces
-// different wraps from a 200-col local terminal.
-const SNAPSHOT_ENV = { COLUMNS: "120" }
-
-describe("miaw CLI help-text snapshots", () => {
-  // Single test, parallel spawns. Each command's help fires under
-  // `concurrency: 8` — wall-clock stays under ~10s even for ~35 commands,
-  // versus ~1 minute if we serialized.
-  cliIt.live(
-    "every documented command emits stable help text",
-    ({ miaw }) =>
-      Effect.gen(function* () {
-        const topLevel = yield* miaw.spawn(["--help"], { env: SNAPSHOT_ENV })
-        expect(topLevel.exitCode).toBe(0)
-        expect(topLevel.stderr.endsWith(EOL)).toBe(true)
-
-        const argvs: Array<readonly string[]> = [...TOP_LEVEL.map((c) => [c] as const), ...SUBCOMMANDS]
-
-        // Spawn in parallel, then assert in argv order so snapshot output is
-        // deterministic and per-command failures don't abort the rest of
-        // the sweep. `Effect.partition` is the canonical "run all, separate
-        // failures from successes" primitive — no mutable accumulator needed.
-        const [failures, results] = yield* Effect.partition(
-          argvs,
-          (argv) =>
-            Effect.gen(function* () {
-              const result = yield* miaw.spawn([...argv, "--help"], { env: SNAPSHOT_ENV })
-              if (result.exitCode !== 0) {
-                return yield* Effect.fail(`miaw ${argv.join(" ")}: exit ${result.exitCode}`)
-              }
-              return { argv, result }
-            }),
-          { concurrency: 8 },
-        )
-
-        for (const { argv, result } of results) {
-          // yargs writes --help to stderr, not stdout. Snapshotting stderr
-          // means our test catches the help body; stdout for these commands
-          // is expected to be empty.
-          expect(normalize(result.stderr)).toMatchSnapshot(`miaw ${argv.join(" ")} --help`)
-        }
-        if (failures.length > 0) {
-          throw new Error(`Help text failed for:\n  ${failures.join("\n  ")}`)
-        }
-      }),
-    180_000,
-  )
-})

package/test/cli/import.test.ts

@@ -1,54 +0,0 @@
-import { test, expect } from "bun:test"
-import {
-  parseShareUrl,
-  shouldAttachShareAuthHeaders,
-  transformShareData,
-  type ShareData,
-} from "../../src/cli/cmd/import"
-
-// parseShareUrl tests
-test("parses valid share URLs", () => {
-  expect(parseShareUrl("https://opncd.ai/share/Jsj3hNIW")).toBe("Jsj3hNIW")
-  expect(parseShareUrl("https://custom.example.com/share/abc123")).toBe("abc123")
-  expect(parseShareUrl("http://localhost:3000/share/test_id-123")).toBe("test_id-123")
-})
-
-test("rejects invalid URLs", () => {
-  expect(parseShareUrl("https://opncd.ai/s/Jsj3hNIW")).toBeNull() // legacy format
-  expect(parseShareUrl("https://opncd.ai/share/")).toBeNull()
-  expect(parseShareUrl("https://opncd.ai/share/id/extra")).toBeNull()
-  expect(parseShareUrl("not-a-url")).toBeNull()
-})
-
-test("only attaches share auth headers for same-origin URLs", () => {
-  expect(shouldAttachShareAuthHeaders("https://control.example.com/share/abc", "https://control.example.com")).toBe(
-    true,
-  )
-  expect(shouldAttachShareAuthHeaders("https://other.example.com/share/abc", "https://control.example.com")).toBe(false)
-  expect(shouldAttachShareAuthHeaders("https://control.example.com:443/share/abc", "https://control.example.com")).toBe(
-    true,
-  )
-  expect(shouldAttachShareAuthHeaders("not-a-url", "https://control.example.com")).toBe(false)
-})
-
-// transformShareData tests
-test("transforms share data to storage format", () => {
-  const data: ShareData[] = [
-    { type: "session", data: { id: "sess-1", title: "Test" } as any },
-    { type: "message", data: { id: "msg-1", sessionID: "sess-1" } as any },
-    { type: "part", data: { id: "part-1", messageID: "msg-1" } as any },
-    { type: "part", data: { id: "part-2", messageID: "msg-1" } as any },
-  ]
-
-  const result = transformShareData(data)!
-
-  expect(result.info.id).toBe("sess-1")
-  expect(result.messages).toHaveLength(1)
-  expect(result.messages[0].parts).toHaveLength(2)
-})
-
-test("returns null for invalid share data", () => {
-  expect(transformShareData([])).toBeNull()
-  expect(transformShareData([{ type: "message", data: {} as any }])).toBeNull()
-  expect(transformShareData([{ type: "session", data: { id: "s" } as any }])).toBeNull() // no messages
-})

package/test/cli/mcp-add.test.ts

@@ -1,74 +0,0 @@
-import { describe, expect } from "bun:test"
-import { Effect } from "effect"
-import path from "path"
-import { cliIt } from "../lib/cli-process"
-
-describe("miaw mcp add (non-interactive subprocess)", () => {
-  cliIt.concurrent(
-    "adds a remote server with HTTP headers",
-    ({ home, miaw }) =>
-      Effect.gen(function* () {
-        const result = yield* miaw.spawn([
-          "mcp",
-          "add",
-          "github",
-          "--url",
-          "https://example.com/mcp",
-          "--header",
-          "Authorization=Bearer {env:GITHUB_TOKEN}",
-          "--header",
-          "X-Option=one=two",
-        ])
-        miaw.expectExit(result, 0)
-
-        const config = yield* Effect.promise(() =>
-          Bun.file(path.join(home, ".config", "miaw", "miaw.json")).json(),
-        )
-        expect(config.mcp.github).toEqual({
-          type: "remote",
-          url: "https://example.com/mcp",
-          headers: {
-            Authorization: "Bearer {env:GITHUB_TOKEN}",
-            "X-Option": "one=two",
-          },
-        })
-      }),
-    60_000,
-  )
-
-  cliIt.concurrent(
-    "adds a local server while preserving argv and environment values",
-    ({ home, miaw }) =>
-      Effect.gen(function* () {
-        const result = yield* miaw.spawn([
-          "mcp",
-          "add",
-          "local",
-          "--env",
-          "API_KEY=secret",
-          "--env",
-          "VALUE=one=two",
-          "--",
-          "npx",
-          "-y",
-          "@example/server",
-          "--label",
-          "two words",
-        ])
-        miaw.expectExit(result, 0)
-
-        const config = yield* Effect.promise(() =>
-          Bun.file(path.join(home, ".config", "miaw", "miaw.json")).json(),
-        )
-        expect(config.mcp.local).toEqual({
-          type: "local",
-          command: ["npx", "-y", "@example/server", "--label", "two words"],
-          environment: {
-            API_KEY: "secret",
-            VALUE: "one=two",
-          },
-        })
-      }),
-    60_000,
-  )
-})

package/test/cli/plugin-auth-picker.test.ts

@@ -1,120 +0,0 @@
-import { test, expect, describe } from "bun:test"
-import { resolvePluginProviders } from "../../src/cli/cmd/providers"
-import type { Hooks } from "@miaw/plugin"
-
-function hookWithAuth(provider: string): Hooks {
-  return {
-    auth: {
-      provider,
-      methods: [],
-    },
-  }
-}
-
-function hookWithoutAuth(): Hooks {
-  return {}
-}
-
-describe("resolvePluginProviders", () => {
-  test("returns plugin providers not in models.dev", () => {
-    const result = resolvePluginProviders({
-      hooks: [hookWithAuth("portkey")],
-      existingProviders: {},
-      disabled: new Set(),
-      providerNames: {},
-    })
-    expect(result).toEqual([{ id: "portkey", name: "portkey" }])
-  })
-
-  test("skips providers already in models.dev", () => {
-    const result = resolvePluginProviders({
-      hooks: [hookWithAuth("anthropic")],
-      existingProviders: { anthropic: {} },
-      disabled: new Set(),
-      providerNames: {},
-    })
-    expect(result).toEqual([])
-  })
-
-  test("deduplicates across plugins", () => {
-    const result = resolvePluginProviders({
-      hooks: [hookWithAuth("portkey"), hookWithAuth("portkey")],
-      existingProviders: {},
-      disabled: new Set(),
-      providerNames: {},
-    })
-    expect(result).toEqual([{ id: "portkey", name: "portkey" }])
-  })
-
-  test("respects disabled_providers", () => {
-    const result = resolvePluginProviders({
-      hooks: [hookWithAuth("portkey")],
-      existingProviders: {},
-      disabled: new Set(["portkey"]),
-      providerNames: {},
-    })
-    expect(result).toEqual([])
-  })
-
-  test("respects enabled_providers when provider is absent", () => {
-    const result = resolvePluginProviders({
-      hooks: [hookWithAuth("portkey")],
-      existingProviders: {},
-      disabled: new Set(),
-      enabled: new Set(["anthropic"]),
-      providerNames: {},
-    })
-    expect(result).toEqual([])
-  })
-
-  test("includes provider when in enabled set", () => {
-    const result = resolvePluginProviders({
-      hooks: [hookWithAuth("portkey")],
-      existingProviders: {},
-      disabled: new Set(),
-      enabled: new Set(["portkey"]),
-      providerNames: {},
-    })
-    expect(result).toEqual([{ id: "portkey", name: "portkey" }])
-  })
-
-  test("resolves name from providerNames", () => {
-    const result = resolvePluginProviders({
-      hooks: [hookWithAuth("portkey")],
-      existingProviders: {},
-      disabled: new Set(),
-      providerNames: { portkey: "Portkey AI" },
-    })
-    expect(result).toEqual([{ id: "portkey", name: "Portkey AI" }])
-  })
-
-  test("falls back to id when no name configured", () => {
-    const result = resolvePluginProviders({
-      hooks: [hookWithAuth("portkey")],
-      existingProviders: {},
-      disabled: new Set(),
-      providerNames: {},
-    })
-    expect(result).toEqual([{ id: "portkey", name: "portkey" }])
-  })
-
-  test("skips hooks without auth", () => {
-    const result = resolvePluginProviders({
-      hooks: [hookWithoutAuth(), hookWithAuth("portkey"), hookWithoutAuth()],
-      existingProviders: {},
-      disabled: new Set(),
-      providerNames: {},
-    })
-    expect(result).toEqual([{ id: "portkey", name: "portkey" }])
-  })
-
-  test("returns empty for no hooks", () => {
-    const result = resolvePluginProviders({
-      hooks: [],
-      existingProviders: {},
-      disabled: new Set(),
-      providerNames: {},
-    })
-    expect(result).toEqual([])
-  })
-})