infra-kit 0.1.128 → 0.1.130

package/src/commands/worktrees-reload/worktrees-reload.ts

@@ -1,230 +0,0 @@
-import { z } from 'zod'
-
-import {
-  buildCmuxWorkspaceTitle,
-  canonicalizeCmuxTitle,
-  closeCmuxWorkspaceByTitle,
-  listCmuxWorkspaceTitles,
-  openCmuxWorkspaceWithLayout,
-} from 'src/integrations/cmux'
-import { ideProviderLabel, openIdeWorkspace } from 'src/integrations/ide'
-import type { IdeProvider } from 'src/integrations/ide'
-import { commandEcho } from 'src/lib/command-echo'
-import { WORKTREES_DIR_SUFFIX } from 'src/lib/constants'
-import { OperationError } from 'src/lib/errors/operation-error'
-import { getCurrentWorktrees, getProjectRoot, getRepoName } from 'src/lib/git-utils'
-import { logger } from 'src/lib/logger'
-import { defineMcpTool, textContent } from 'src/types'
-
-interface WorktreesReloadResult {
-  closedCmux: string[]
-  openedCmux: string[]
-  ideProviders: IdeProvider[]
-  ideFoldersAdded: number
-  ideFoldersRemoved: number
-}
-
-/**
- * Reload command: closes every open cmux workspace for the release worktrees on
- * disk, then reopens the full set — cmux workspaces are recreated fresh and the
- * configured editor workspace is reconciled and relaunched. The single
- * window-restore command: also serves the cold-start case (editor + cmux closed),
- * at the cost of tearing down any live cmux windows first.
- *
- * The editor has no per-window close primitive, so its "reload" is a reconcile +
- * relaunch of the configured workspace. Safe to run from any branch
- * (non-destructive to git — only cmux/editor view state is touched).
- */
-export const worktreesReload = async () => {
-  commandEcho.start('worktrees-reload')
-
-  try {
-    const projectRoot = await getProjectRoot()
-    const worktreeDir = `${projectRoot}${WORKTREES_DIR_SUFFIX}`
-    const currentBranches = await getCurrentWorktrees('release')
-    const repoName = await getRepoName()
-
-    // Phase 1 — close (cmux only; Cursor has no per-window close).
-    const closedCmux = await closeCmux({ currentBranches, repoName })
-
-    // Phase 2 — reopen. cmux and Cursor are independent surfaces, so reopen them
-    // concurrently to shrink the window where the just-closed cmux workspaces are gone.
-    const [ideOutcomes, { opened: openedCmux }] = await Promise.all([
-      openIdeWorkspace({ projectRoot, worktreeDir, currentBranches }),
-      reopenCmux({ worktreeDir, currentBranches, repoName }),
-    ])
-
-    const result: WorktreesReloadResult = {
-      closedCmux,
-      openedCmux,
-      ideProviders: ideOutcomes
-        .filter((outcome) => {
-          return outcome.ran
-        })
-        .map((outcome) => {
-          return outcome.provider
-        }),
-      ideFoldersAdded: ideOutcomes.reduce((sum, outcome) => {
-        return sum + outcome.added
-      }, 0),
-      ideFoldersRemoved: ideOutcomes.reduce((sum, outcome) => {
-        return sum + outcome.removed
-      }, 0),
-    }
-
-    logResults(result, {
-      ideRan: ideOutcomes.some((outcome) => {
-        return outcome.ran
-      }),
-    })
-
-    commandEcho.print()
-
-    return {
-      content: textContent(JSON.stringify(result, null, 2)),
-      structuredContent: { ...result },
-    }
-  } catch (error) {
-    logger.error({ error }, '❌ Error reloading worktrees')
-    throw new OperationError(error, {
-      operation: 'reload worktrees',
-      remediation: "run 'worktrees-list' to confirm the branches exist",
-    })
-  }
-}
-
-interface CloseCmuxArgs {
-  currentBranches: string[]
-  repoName: string
-}
-
-/**
- * Close the cmux workspace for each release worktree that is currently open.
- * Snapshots `listCmuxWorkspaceTitles()` up front purely so the returned list
- * reflects which workspaces were actually open (the close itself is best-effort
- * and per-title). Returns the titles a close was attempted for.
- */
-export const closeCmux = async (args: CloseCmuxArgs): Promise<string[]> => {
-  const { currentBranches, repoName } = args
-
-  if (currentBranches.length === 0) {
-    return []
-  }
-
-  const openBefore = await listCmuxWorkspaceTitles()
-
-  const closed: string[] = []
-
-  for (const branch of currentBranches) {
-    const title = buildCmuxWorkspaceTitle({ repoName, branch })
-
-    // openBefore holds canonical keys; match the built title the same way so a
-    // workspace stored under a drifted title (whitespace / old v-prefix) resolves.
-    if (!openBefore.has(canonicalizeCmuxTitle(title))) {
-      continue
-    }
-
-    await closeCmuxWorkspaceByTitle(title)
-    closed.push(title)
-  }
-
-  return closed
-}
-
-interface ReopenCmuxArgs {
-  worktreeDir: string
-  currentBranches: string[]
-  repoName: string
-}
-
-interface ReopenCmuxOutcome {
-  opened: string[]
-}
-
-/**
- * Recreate a cmux workspace for every release worktree on disk. Force-opens each
- * branch (no dedup against the live workspace list): reload has just awaited the
- * close of these workspaces, so it favours liveness — guaranteeing every branch
- * ends up open.
- */
-export const reopenCmux = async (args: ReopenCmuxArgs): Promise<ReopenCmuxOutcome> => {
-  const { worktreeDir, currentBranches, repoName } = args
-
-  const opened: string[] = []
-
-  for (const branch of currentBranches) {
-    const title = buildCmuxWorkspaceTitle({ repoName, branch })
-
-    try {
-      await openCmuxWorkspaceWithLayout({ cwd: `${worktreeDir}/${branch}`, title })
-      opened.push(title)
-    } catch (error) {
-      logger.warn({ error, title }, `⚠️ Failed to reopen cmux workspace for ${branch}`)
-    }
-  }
-
-  return { opened }
-}
-
-interface LogResultsContext {
-  ideRan: boolean
-}
-
-const logResults = (result: WorktreesReloadResult, context: LogResultsContext): void => {
-  if (result.closedCmux.length > 0) {
-    logger.info(`🧹 Closed ${result.closedCmux.length} cmux workspace(s)`)
-  }
-
-  if (context.ideRan) {
-    const ideLabels = result.ideProviders
-      .map((provider) => {
-        return ideProviderLabel(provider)
-      })
-      .join(', ')
-
-    if (result.ideFoldersAdded > 0) {
-      logger.info(`✅ Added ${result.ideFoldersAdded} folder(s) to ${ideLabels} workspace(s)`)
-    }
-
-    if (result.ideFoldersRemoved > 0) {
-      logger.info(`🧹 Removed ${result.ideFoldersRemoved} dangling folder(s) from ${ideLabels} workspace(s)`)
-    }
-  }
-
-  if (result.openedCmux.length > 0) {
-    logger.info('✅ Reopened cmux workspaces:')
-    for (const title of result.openedCmux) {
-      logger.info(title)
-    }
-  }
-
-  if (
-    !context.ideRan &&
-    result.closedCmux.length === 0 &&
-    result.openedCmux.length === 0 &&
-    result.ideFoldersAdded === 0 &&
-    result.ideFoldersRemoved === 0
-  ) {
-    logger.info('ℹ️ Nothing to reload')
-  }
-}
-
-// MCP Tool Registration
-export const worktreesReloadMcpTool = defineMcpTool({
-  name: 'worktrees-reload',
-  description:
-    'Close all open cmux workspaces for the current release worktrees, then reopen the full set — the single window-restore command. Every configured editor (Cursor and/or Zed) is reconciled + relaunched (no per-window close). Disruptive: closes live cmux windows first. Safe to run from any branch. Use to refresh stale windows or to restore everything after a cold start (editor + cmux closed).',
-  inputSchema: {},
-  outputSchema: {
-    closedCmux: z.array(z.string()).describe('Titles of cmux workspaces a close was attempted for (best-effort)'),
-    openedCmux: z.array(z.string()).describe('Titles of cmux workspaces reopened during this run'),
-    ideProviders: z
-      .array(z.string())
-      .describe('Configured IDE providers that were reloaded (cursor | zed); empty if none configured'),
-    ideFoldersAdded: z.number().describe('Total worktree folders added across all configured editor workspaces'),
-    ideFoldersRemoved: z
-      .number()
-      .describe('Total dangling worktree folders removed across all configured editor workspaces'),
-  },
-  handler: worktreesReload,
-})

package/src/commands/worktrees-remove/index.ts

@@ -1 +0,0 @@
-export { worktreesRemove, worktreesRemoveMcpTool } from './worktrees-remove'

package/src/commands/worktrees-remove/worktrees-remove.ts

@@ -1,199 +0,0 @@
-import checkbox from '@inquirer/checkbox'
-import confirm from '@inquirer/confirm'
-import process from 'node:process'
-import { z } from 'zod'
-
-import { getReleasePRsWithInfo } from 'src/integrations/gh'
-import { removeIdeWorktreeFolders } from 'src/integrations/ide'
-import { commandEcho } from 'src/lib/command-echo'
-import { WORKTREES_DIR_SUFFIX } from 'src/lib/constants'
-import { isPromptCancellation } from 'src/lib/errors/is-prompt-cancellation'
-import { OperationError } from 'src/lib/errors/operation-error'
-import { assertManagementContext } from 'src/lib/git-guard'
-import { getCurrentWorktrees, getProjectRoot, getRepoName } from 'src/lib/git-utils'
-import { logger } from 'src/lib/logger'
-import { formatBranchName, parseReleaseRef } from 'src/lib/release-id'
-import { detectReleaseType, formatBranchChoices, getJiraDescriptions, releaseBranchLabels } from 'src/lib/release-utils'
-import type { ReleaseType } from 'src/lib/release-utils'
-import { removeWorktrees } from 'src/lib/worktrees'
-import { defineMcpTool, textContent } from 'src/types'
-import type { RequiredConfirmedOptionArg } from 'src/types'
-
-// Constants
-interface WorktreeManagementArgs extends RequiredConfirmedOptionArg {
-  all?: boolean
-  versions?: string
-}
-
-/**
- * Manage git worktrees for release branches
- * Creates worktrees for active release branches and removes unused ones
- */
-export const worktreesRemove = async (options: WorktreeManagementArgs) => {
-  const { confirmedCommand, all, versions } = options
-
-  commandEcho.start('worktrees-remove')
-
-  await assertManagementContext({ operation: 'remove worktrees', requiredBranch: 'dev' })
-
-  try {
-    const currentWorktrees = await getCurrentWorktrees('release')
-
-    if (currentWorktrees.length === 0) {
-      logger.info('ℹ️ No active worktrees to remove')
-
-      commandEcho.print()
-
-      return {
-        content: textContent(JSON.stringify({ removedWorktrees: [], count: 0 }, null, 2)),
-        structuredContent: { removedWorktrees: [], count: 0 },
-      }
-    }
-
-    const projectRoot = await getProjectRoot()
-
-    const worktreeDir = `${projectRoot}${WORKTREES_DIR_SUFFIX}`
-
-    let selectedReleaseBranches: string[] = []
-
-    if (all) {
-      selectedReleaseBranches = currentWorktrees
-    } else if (versions) {
-      selectedReleaseBranches = versions.split(',').map((v) => {
-        return formatBranchName(parseReleaseRef(v.trim()))
-      })
-    } else {
-      commandEcho.setInteractive()
-
-      const [descriptions, prInfo] = await Promise.all([getJiraDescriptions(), getReleasePRsWithInfo()])
-
-      const releaseTypes = new Map<string, ReleaseType>(
-        prInfo.map((pr) => {
-          return [pr.branch, detectReleaseType(pr.title)]
-        }),
-      )
-
-      selectedReleaseBranches = await checkbox({
-        required: true,
-        message: '🌿 Select release branches',
-        choices: formatBranchChoices({ branches: currentWorktrees, descriptions, types: releaseTypes }),
-      })
-    }
-
-    // Track --all flag if all branches were selected (either via flag or interactively)
-    const allSelected = selectedReleaseBranches.length === currentWorktrees.length
-
-    if (allSelected) {
-      commandEcho.addOption('--all', true)
-    } else {
-      commandEcho.addOption('--versions', releaseBranchLabels(selectedReleaseBranches))
-    }
-
-    // Ask for confirmation
-    const answer = confirmedCommand
-      ? true
-      : await confirm({
-          message: 'Are you sure you want to proceed with these worktree changes?',
-        })
-
-    if (!confirmedCommand) {
-      commandEcho.setInteractive()
-    }
-
-    if (!answer) {
-      logger.info('Operation cancelled. Exiting...')
-      process.exit(0)
-    }
-
-    // Track --yes flag if confirmation was interactive (user confirmed)
-    if (!confirmedCommand) {
-      commandEcho.addOption('--yes', true)
-    }
-
-    const repoName = await getRepoName()
-
-    const removedWorktrees = await removeWorktrees({
-      branches: selectedReleaseBranches,
-      worktreeDir,
-      repoName,
-      pruneFolder: allSelected,
-    })
-
-    // `!confirmedCommand` is the interactive path (a human used the picker/confirm). Only there
-    // do we let Zed's destructive `--reuse` relaunch fire; MCP/--yes runs (confirmedCommand=true)
-    // and worktrees-sync never relaunch an editor window.
-    await removeIdeWorktreeFolders({
-      projectRoot,
-      worktreeDir,
-      currentWorktrees,
-      removedWorktrees,
-      allowEditorRelaunch: !confirmedCommand,
-    })
-
-    logResults(removedWorktrees)
-
-    commandEcho.print()
-
-    const structuredContent = {
-      removedWorktrees,
-      count: removedWorktrees.length,
-    }
-
-    return {
-      content: textContent(JSON.stringify(structuredContent, null, 2)),
-      structuredContent,
-    }
-  } catch (error) {
-    // A cancelled prompt (Ctrl-C / Esc) is a user back-out, not a failure: let it
-    // reach the top-level boundary untouched so it exits cleanly, instead of being
-    // logged as an error with a misleading remediation.
-    if (isPromptCancellation(error)) throw error
-
-    logger.error({ error }, '❌ Error managing worktrees')
-    throw new OperationError(error, {
-      operation: 'remove worktrees',
-      remediation: "check 'git worktree list' for the path; uncommitted changes block removal",
-    })
-  }
-}
-
-/**
- * Log the results of worktree management
- */
-const logResults = (removed: string[]): void => {
-  if (removed.length > 0) {
-    logger.info('❌ Removed worktrees:')
-    for (const branch of removed) {
-      logger.info(branch)
-    }
-    logger.info('')
-  } else {
-    logger.info('ℹ️ No unused worktrees to remove')
-  }
-}
-
-// MCP Tool Registration
-export const worktreesRemoveMcpTool = defineMcpTool({
-  name: 'worktrees-remove',
-  description:
-    'Remove local git worktrees for release branches. When everything is removed, also runs "git worktree prune" to clear stale metadata; the worktrees directory and its release/feature subfolders are left in place. When invoked via MCP, pass either "versions" (comma-separated) or all=true — the branch picker is unreachable without a TTY, and the CLI confirmation is auto-skipped for MCP calls, so the caller is responsible for gating.',
-  inputSchema: {
-    all: z
-      .boolean()
-      .optional()
-      .describe(
-        'Remove every existing worktree. Either "all" or "versions" must be provided for MCP calls (the interactive picker is unavailable without a TTY). Ignored if "versions" is provided.',
-      ),
-    versions: z
-      .string()
-      .optional()
-      .describe(
-        'Comma-separated release versions or names to target (e.g. "1.2.5, 1.2.6" or "checkout-redesign, 1.2.5"). Either "versions" or all=true must be provided for MCP calls. Overrides "all" when set.',
-      ),
-  },
-  outputSchema: {
-    removedWorktrees: z.array(z.string()).describe('List of removed git worktree branches'),
-    count: z.number().describe('Number of git worktrees removed'),
-  },
-  handler: worktreesRemove,
-})

package/src/commands/worktrees-sync/index.ts

@@ -1 +0,0 @@
-export { worktreesSync, worktreesSyncMcpTool } from './worktrees-sync'

package/src/commands/worktrees-sync/worktrees-sync.ts

@@ -1,198 +0,0 @@
-import confirm from '@inquirer/confirm'
-import process from 'node:process'
-import { z } from 'zod'
-import { $ } from 'zx'
-
-import { buildCmuxWorkspaceTitle, closeCmuxWorkspaceByTitle } from 'src/integrations/cmux'
-import { getReleasePRs } from 'src/integrations/gh'
-import { removeIdeWorktreeFolders } from 'src/integrations/ide'
-import { commandEcho } from 'src/lib/command-echo'
-import { WORKTREES_DIR_SUFFIX } from 'src/lib/constants'
-import { isPromptCancellation } from 'src/lib/errors/is-prompt-cancellation'
-import { OperationError } from 'src/lib/errors/operation-error'
-import { assertManagementContext } from 'src/lib/git-guard'
-import { getCurrentWorktrees, getProjectRoot, getRepoName } from 'src/lib/git-utils'
-import { logger } from 'src/lib/logger'
-import { isReleaseBranch } from 'src/lib/release-id'
-import { defineMcpTool, textContent } from 'src/types'
-import type { RequiredConfirmedOptionArg } from 'src/types'
-
-interface WorktreeSyncArgs extends RequiredConfirmedOptionArg {}
-
-/**
- * Manage git worktrees for release branches.
- *
- * Creates worktrees for active release branches and removes unused ones
- */
-export const worktreesSync = async (options: WorktreeSyncArgs) => {
-  const { confirmedCommand } = options
-
-  commandEcho.start('worktrees-sync')
-
-  await assertManagementContext({ operation: 'sync worktrees', requiredBranch: 'dev' })
-
-  try {
-    const currentWorktrees = await getCurrentWorktrees('release')
-    const projectRoot = await getProjectRoot()
-
-    const worktreeDir = `${projectRoot}${WORKTREES_DIR_SUFFIX}`
-
-    const releasePRsList = await getReleasePRs()
-
-    // Ask for confirmation
-    const answer = confirmedCommand
-      ? true
-      : await confirm({
-          message: 'Are you sure you want to proceed with these worktree changes?',
-        })
-
-    if (!confirmedCommand) {
-      commandEcho.setInteractive()
-    }
-
-    if (!answer) {
-      logger.info('Operation cancelled. Exiting...')
-      process.exit(0)
-    }
-
-    // Track --yes flag if confirmation was interactive (user confirmed)
-    if (!confirmedCommand) {
-      commandEcho.addOption('--yes', true)
-    }
-
-    const { branchesToRemove } = categorizeWorktrees({
-      releasePRsList,
-      currentWorktrees,
-    })
-
-    const repoName = await getRepoName()
-
-    const removedWorktrees = await removeWorktrees({
-      branches: branchesToRemove,
-      worktreeDir,
-      repoName,
-    })
-
-    // Hard `false`: sync is background/stale-cleanup (predominantly the MCP path) — it must never
-    // relaunch and overwrite a focused Zed window. Zed removal stays a no-op here.
-    await removeIdeWorktreeFolders({
-      projectRoot,
-      worktreeDir,
-      currentWorktrees,
-      removedWorktrees,
-      allowEditorRelaunch: false,
-    })
-
-    logResults(removedWorktrees)
-
-    commandEcho.print()
-
-    const structuredContent = {
-      removedWorktrees,
-      count: removedWorktrees.length,
-    }
-
-    return {
-      content: textContent(JSON.stringify(structuredContent, null, 2)),
-      structuredContent,
-    }
-  } catch (error) {
-    // A cancelled prompt (Ctrl-C / Esc) is a user back-out, not a failure: let it
-    // reach the top-level boundary untouched so it exits cleanly, instead of being
-    // logged as an error with a misleading remediation.
-    if (isPromptCancellation(error)) throw error
-
-    logger.error({ error }, '❌ Error managing worktrees')
-    throw new OperationError(error, {
-      operation: 'sync worktrees with remote',
-      remediation: "ensure 'gh auth status' is ok and you can reach origin",
-    })
-  }
-}
-
-interface CategorizeWorktreesArgs {
-  releasePRsList: string[]
-  currentWorktrees: string[]
-}
-
-/**
- * Categorize worktrees into those that need to be created or removed
- */
-const categorizeWorktrees = (args: CategorizeWorktreesArgs): { branchesToRemove: string[] } => {
-  const { releasePRsList, currentWorktrees } = args
-
-  const currentBranchNames = currentWorktrees.filter((branch) => {
-    return isReleaseBranch(branch)
-  })
-
-  const branchesToRemove = currentBranchNames.filter((branch) => {
-    return !releasePRsList.includes(branch)
-  })
-
-  return { branchesToRemove }
-}
-
-interface RemoveWorktreesArgs {
-  branches: string[]
-  worktreeDir: string
-  repoName: string
-}
-
-/**
- * Remove worktrees for the specified branches and close their cmux workspaces
- */
-const removeWorktrees = async (args: RemoveWorktreesArgs): Promise<string[]> => {
-  const { branches, worktreeDir, repoName } = args
-
-  const removed: string[] = []
-
-  for (const branch of branches) {
-    try {
-      const worktreePath = `${worktreeDir}/${branch}`
-
-      const title = buildCmuxWorkspaceTitle({ repoName, branch })
-
-      await closeCmuxWorkspaceByTitle(title)
-
-      await $`git worktree remove ${worktreePath}`
-      removed.push(branch)
-    } catch (error) {
-      const err = new OperationError(error, {
-        operation: `remove stale worktree for ${branch}`,
-        remediation: 'inspect the worktree dir manually; rerun with the branch checked out elsewhere',
-      })
-
-      logger.error({ error, branch, msg: err.message })
-    }
-  }
-
-  return removed
-}
-
-/**
- * Log the results of worktree management
- */
-const logResults = (removed: string[]): void => {
-  if (removed.length > 0) {
-    logger.info('❌ Removed worktrees:')
-    for (const branch of removed) {
-      logger.info(branch)
-    }
-    logger.info('')
-  } else {
-    logger.info('ℹ️ No unused worktrees to remove')
-  }
-}
-
-// MCP Tool Registration
-export const worktreesSyncMcpTool = defineMcpTool({
-  name: 'worktrees-sync',
-  description:
-    'Remove worktrees whose release PR is no longer open (stale cleanup). Only removes — never creates; use worktrees-add to create worktrees for new releases. The CLI confirmation is auto-skipped for MCP calls, so the caller is responsible for gating.',
-  inputSchema: {},
-  outputSchema: {
-    removedWorktrees: z.array(z.string()).describe('List of removed worktree branches'),
-    count: z.number().describe('Number of worktrees removed during sync'),
-  },
-  handler: worktreesSync,
-})

package/src/dev/__tests__/dev-server-options.test.ts

@@ -1,57 +0,0 @@
-import { describe, expect, it } from 'vitest'
-
-import { toDevServerOptions } from 'src/entry/dev-server'
-import type { DevCliOptions } from 'src/entry/dev-server'
-
-/**
- * Flag/option parsing for the dev-server CLI entry. `toDevServerOptions` maps the
- * raw Commander flag object (comma-joined strings) to the typed `DevServerOptions`
- * the orchestrator consumes. Pure function — no fixtures.
- */
-describe('toDevServerOptions — CLI flag parsing', () => {
-  it('splits --app into an include list', () => {
-    const opts = toDevServerOptions({ app: 'client,backoffice' } satisfies DevCliOptions)
-
-    expect(opts.include).toEqual(['client', 'backoffice'])
-  })
-
-  it('trims whitespace and drops empty segments in comma lists', () => {
-    const opts = toDevServerOptions({ app: ' client , , backoffice ,' } satisfies DevCliOptions)
-
-    expect(opts.include).toEqual(['client', 'backoffice'])
-  })
-
-  it('maps a missing --app to null (means "all apps")', () => {
-    const opts = toDevServerOptions({} satisfies DevCliOptions)
-
-    expect(opts.include).toBeNull()
-  })
-
-  it('maps an all-empty comma list to null rather than an empty array', () => {
-    const opts = toDevServerOptions({ app: ' , , ' } satisfies DevCliOptions)
-
-    expect(opts.include).toBeNull()
-  })
-
-  it('defaults watch to false when the flag is absent', () => {
-    expect(toDevServerOptions({} satisfies DevCliOptions).watch).toBe(false)
-  })
-
-  it('passes --watch through as true', () => {
-    expect(toDevServerOptions({ watch: true } satisfies DevCliOptions).watch).toBe(true)
-  })
-
-  it('parses a combined flag set (--app --watch) in one pass', () => {
-    const opts = toDevServerOptions({ app: 'a,b', watch: true } satisfies DevCliOptions)
-
-    expect(opts).toEqual({ watch: true, include: ['a', 'b'], ui: false })
-  })
-
-  it('defaults ui to false when --ui is absent (api-only, backward compatible)', () => {
-    expect(toDevServerOptions({} satisfies DevCliOptions).ui).toBe(false)
-  })
-
-  it('passes --ui through as true', () => {
-    expect(toDevServerOptions({ ui: true } satisfies DevCliOptions).ui).toBe(true)
-  })
-})