switchroom 0.8.1 → 0.10.0

package/telegram-plugin/tests/setup-flow.test.ts

@@ -1,510 +0,0 @@
-/**
- * Tests for the /setup wizard state machine (setup-flow.ts).
- *
- * Pure function tests — no grammY, no SQLite, no network.
- *
- * Covers:
- *   - startSetupFlow: no slug, valid slug, invalid slug
- *   - handleSetupText: full happy-path step transitions
- *   - Validator helpers: isValidSlug, isValidPersonaName, isValidModel, isValidEmoji
- *   - Skip / cancel / error paths at each step
- *   - makeSetupInitialState / advanceSetupState / setupStepLabel helpers
- */
-
-import { describe, it, expect } from 'vitest'
-import {
-  startSetupFlow,
-  handleSetupText,
-  makeSetupInitialState,
-  advanceSetupState,
-  setupStepLabel,
-  isValidSlug,
-  isValidPersonaName,
-  isValidModel,
-  isValidEmoji,
-  isSkip,
-  isCancel,
-} from '../foreman/setup-flow.js'
-import type { SetupFlowState } from '../foreman/setup-state.js'
-
-const CALLER = '12345678'
-
-// #190: setup-flow now needs the operator's profile list. Tests that don't
-// care about the profile validation just pass `default` so the asked-profile
-// step accepts any input that maps to one of these.
-const PROFILES = ['default', 'coding', 'health-coach', 'executive-assistant']
-
-function makeState(overrides: Partial<SetupFlowState> = {}): SetupFlowState {
-  return {
-    chatId: 'chat1',
-    step: 'asked-slug',
-    slug: null,
-    persona: null,
-    model: null,
-    emoji: null,
-    profile: null,
-    botToken: null,
-    allowedUserId: null,
-    authSessionName: null,
-    loginUrl: null,
-    startedAt: 1000,
-    updatedAt: 1000,
-    ...overrides,
-  }
-}
-
-/**
- * Test helper — wraps handleSetupText with a default profiles list so the
- * 30+ existing call sites don't need to be rewritten one-by-one. Tests that
- * specifically exercise the asked-profile validation pass their own
- * profiles via the `opts` argument.
- */
-function call(
-  state: SetupFlowState | null,
-  text: string,
-  opts: { callerId?: string; profiles?: string[] } = {},
-) {
-  return handleSetupText({
-    state,
-    text,
-    callerId: opts.callerId ?? CALLER,
-    profiles: opts.profiles ?? PROFILES,
-  })
-}
-
-// ─── isValidSlug ─────────────────────────────────────────────────────────
-
-describe('isValidSlug', () => {
-  it('accepts simple lowercase', () => expect(isValidSlug('gymbro')).toBe(true))
-  it('accepts hyphens', () => expect(isValidSlug('gym-bro')).toBe(true))
-  it('accepts underscores', () => expect(isValidSlug('gym_bro')).toBe(true))
-  it('accepts leading digit', () => expect(isValidSlug('1agent')).toBe(true))
-  it('rejects uppercase', () => expect(isValidSlug('GymBro')).toBe(false))
-  it('rejects spaces', () => expect(isValidSlug('gym bro')).toBe(false))
-  it('rejects empty', () => expect(isValidSlug('')).toBe(false))
-  it('accepts 51-char slug', () => expect(isValidSlug('a'.repeat(51))).toBe(true))
-  it('rejects 52-char slug', () => expect(isValidSlug('a'.repeat(52))).toBe(false))
-})
-
-// ─── isValidPersonaName ───────────────────────────────────────────────────
-
-describe('isValidPersonaName', () => {
-  it('accepts normal name', () => expect(isValidPersonaName('Gym Bro')).toBe(true))
-  it('accepts emoji in name', () => expect(isValidPersonaName('Clerk 💼')).toBe(true))
-  it('rejects empty string', () => expect(isValidPersonaName('')).toBe(false))
-  it('rejects control char', () => expect(isValidPersonaName('bad\x00name')).toBe(false))
-  it('rejects 81-char name', () => expect(isValidPersonaName('a'.repeat(81))).toBe(false))
-  it('accepts 80-char name', () => expect(isValidPersonaName('a'.repeat(80))).toBe(true))
-})
-
-// ─── isValidModel ─────────────────────────────────────────────────────────
-
-describe('isValidModel', () => {
-  it('accepts sonnet alias', () => expect(isValidModel('sonnet')).toBe(true))
-  it('accepts opus alias', () => expect(isValidModel('opus')).toBe(true))
-  it('accepts haiku alias', () => expect(isValidModel('haiku')).toBe(true))
-  it('accepts inherit alias', () => expect(isValidModel('inherit')).toBe(true))
-  it('accepts full model ID', () => expect(isValidModel('claude-sonnet-4-5')).toBe(true))
-  it('rejects spaces', () => expect(isValidModel('bad model')).toBe(false))
-  it('rejects empty', () => expect(isValidModel('')).toBe(false))
-})
-
-// ─── isValidEmoji ─────────────────────────────────────────────────────────
-
-describe('isValidEmoji', () => {
-  it('accepts single emoji', () => expect(isValidEmoji('🏋️')).toBe(true))
-  it('accepts simple ascii (single char)', () => expect(isValidEmoji('x')).toBe(true))
-  it('rejects empty string', () => expect(isValidEmoji('')).toBe(false))
-  it('rejects only whitespace', () => expect(isValidEmoji('   ')).toBe(false))
-})
-
-// ─── isSkip / isCancel ────────────────────────────────────────────────────
-
-describe('isSkip', () => {
-  it('matches "skip"', () => expect(isSkip('skip')).toBe(true))
-  it('matches "s"', () => expect(isSkip('s')).toBe(true))
-  it('matches "-"', () => expect(isSkip('-')).toBe(true))
-  it('ignores case', () => expect(isSkip('SKIP')).toBe(true))
-  it('does not match other words', () => expect(isSkip('no')).toBe(false))
-})
-
-describe('isCancel', () => {
-  it('matches "cancel"', () => expect(isCancel('cancel')).toBe(true))
-  it('matches "/cancel"', () => expect(isCancel('/cancel')).toBe(true))
-  it('matches "abort"', () => expect(isCancel('abort')).toBe(true))
-  it('ignores case', () => expect(isCancel('CANCEL')).toBe(true))
-  it('does not match "yes"', () => expect(isCancel('yes')).toBe(false))
-})
-
-// ─── startSetupFlow ───────────────────────────────────────────────────────
-
-describe('startSetupFlow', () => {
-  it('asks for slug when no inline arg', () => {
-    const action = startSetupFlow(null)
-    expect(action.kind).toBe('ask-slug')
-  })
-
-  it('asks for persona when valid inline slug given', () => {
-    const action = startSetupFlow('gymbro')
-    expect(action.kind).toBe('ask-persona')
-    if (action.kind === 'ask-persona') expect(action.slug).toBe('gymbro')
-  })
-
-  it('returns error for invalid inline slug', () => {
-    const action = startSetupFlow('INVALID SLUG!')
-    expect(action.kind).toBe('error')
-  })
-})
-
-// ─── handleSetupText: cancel at any step ─────────────────────────────────
-
-describe('handleSetupText: cancel', () => {
-  const steps = [
-    'asked-slug', 'asked-persona', 'asked-model', 'asked-emoji',
-    'asked-bot-token', 'confirming-allowlist',
-  ] as const
-
-  for (const step of steps) {
-    it(`cancels at step ${step}`, () => {
-      const state = makeState({ step, slug: 'gymbro', persona: 'Gym Bro' })
-      const action = handleSetupText({ state, text: 'cancel', callerId: CALLER })
-      expect(action.kind).toBe('cancel')
-      if (action.kind === 'cancel') expect(action.reason).toBe('user-cancelled')
-    })
-  }
-})
-
-// ─── handleSetupText: null state ──────────────────────────────────────────
-
-describe('handleSetupText: null state', () => {
-  it('returns cancel with no-active-flow reason', () => {
-    const action = handleSetupText({ state: null, text: 'gymbro', callerId: CALLER })
-    expect(action.kind).toBe('cancel')
-    if (action.kind === 'cancel') expect(action.reason).toBe('no-active-flow')
-  })
-})
-
-// ─── handleSetupText: step asked-slug ────────────────────────────────────
-
-describe('handleSetupText: asked-slug', () => {
-  it('advances to ask-persona on valid slug', () => {
-    const state = makeState({ step: 'asked-slug' })
-    const action = handleSetupText({ state, text: 'gymbro', callerId: CALLER })
-    expect(action.kind).toBe('ask-persona')
-    if (action.kind === 'ask-persona') expect(action.slug).toBe('gymbro')
-  })
-
-  it('returns error on invalid slug', () => {
-    const state = makeState({ step: 'asked-slug' })
-    const action = handleSetupText({ state, text: 'BAD SLUG', callerId: CALLER })
-    expect(action.kind).toBe('error')
-    if (action.kind === 'error') expect(action.stayInStep).toBe(true)
-  })
-})
-
-// ─── handleSetupText: step asked-persona ─────────────────────────────────
-
-describe('handleSetupText: asked-persona', () => {
-  it('advances to ask-model on valid persona', () => {
-    const state = makeState({ step: 'asked-persona', slug: 'gymbro' })
-    const action = handleSetupText({ state, text: 'Gym Bro', callerId: CALLER })
-    expect(action.kind).toBe('ask-model')
-    if (action.kind === 'ask-model') {
-      expect(action.slug).toBe('gymbro')
-      expect(action.persona).toBe('Gym Bro')
-    }
-  })
-
-  it('returns error on empty persona', () => {
-    const state = makeState({ step: 'asked-persona', slug: 'gymbro' })
-    const action = handleSetupText({ state, text: '', callerId: CALLER })
-    expect(action.kind).toBe('error')
-  })
-})
-
-// ─── handleSetupText: step asked-model ───────────────────────────────────
-
-describe('handleSetupText: asked-model', () => {
-  it('advances to ask-emoji with skip', () => {
-    const state = makeState({ step: 'asked-model', slug: 'gymbro', persona: 'Gym Bro' })
-    const action = handleSetupText({ state, text: 'skip', callerId: CALLER })
-    expect(action.kind).toBe('ask-emoji')
-    if (action.kind === 'ask-emoji') expect(action.model).toBeNull()
-  })
-
-  it('advances to ask-emoji with valid model', () => {
-    const state = makeState({ step: 'asked-model', slug: 'gymbro', persona: 'Gym Bro' })
-    const action = handleSetupText({ state, text: 'sonnet', callerId: CALLER })
-    expect(action.kind).toBe('ask-emoji')
-    if (action.kind === 'ask-emoji') expect(action.model).toBe('sonnet')
-  })
-
-  it('returns error on model string with spaces', () => {
-    const state = makeState({ step: 'asked-model', slug: 'gymbro', persona: 'Gym Bro' })
-    // Spaces are not allowed in model IDs
-    const action = handleSetupText({ state, text: 'bad model name', callerId: CALLER })
-    expect(action.kind).toBe('error')
-    if (action.kind === 'error') expect(action.stayInStep).toBe(true)
-  })
-})
-
-// ─── handleSetupText: step asked-emoji ───────────────────────────────────
-
-describe('handleSetupText: asked-emoji (#190 — now transitions to ask-profile)', () => {
-  it('advances to ask-profile with skip (no emoji)', () => {
-    const state = makeState({ step: 'asked-emoji', slug: 'gymbro', persona: 'Gym Bro', model: 'sonnet' })
-    const action = call(state, 'skip')
-    expect(action.kind).toBe('ask-profile')
-    if (action.kind === 'ask-profile') {
-      expect(action.emoji).toBeNull()
-      expect(action.profiles).toEqual(PROFILES)
-    }
-  })
-
-  it('advances to ask-profile carrying the emoji', () => {
-    const state = makeState({ step: 'asked-emoji', slug: 'gymbro', persona: 'Gym Bro', model: null })
-    const action = call(state, '🏋️')
-    expect(action.kind).toBe('ask-profile')
-    if (action.kind === 'ask-profile') expect(action.emoji).toBe('🏋️')
-  })
-})
-
-// ─── handleSetupText: step asked-profile (#190) ──────────────────────────
-
-describe('handleSetupText: asked-profile (#190)', () => {
-  function profileState(): SetupFlowState {
-    return makeState({
-      step: 'asked-profile',
-      slug: 'gymbro',
-      persona: 'Gym Bro',
-      model: 'sonnet',
-      emoji: '🏋️',
-    })
-  }
-
-  it('advances to ask-bot-token when profile is in the live list', () => {
-    const action = call(profileState(), 'health-coach')
-    expect(action.kind).toBe('ask-bot-token')
-    if (action.kind === 'ask-bot-token') {
-      expect(action.profile).toBe('health-coach')
-      expect(action.slug).toBe('gymbro')
-      expect(action.emoji).toBe('🏋️')
-    }
-  })
-
-  it('returns error stayInStep when profile is unknown', () => {
-    const action = call(profileState(), 'nonexistent')
-    expect(action.kind).toBe('error')
-    if (action.kind === 'error') {
-      expect(action.stayInStep).toBe(true)
-      expect(action.message).toContain('nonexistent')
-    }
-  })
-
-  it('lists valid profiles in error message', () => {
-    const action = call(profileState(), 'bogus')
-    if (action.kind === 'error') {
-      for (const p of PROFILES) expect(action.message).toContain(p)
-    }
-  })
-
-  it('cancels when slug or persona missing', () => {
-    const action = call(makeState({ step: 'asked-profile' }), 'default')
-    expect(action.kind).toBe('cancel')
-  })
-
-  it('respects a different profiles list per call', () => {
-    const action = call(profileState(), 'tiny-bundle', { profiles: ['tiny-bundle'] })
-    expect(action.kind).toBe('ask-bot-token')
-  })
-})
-
-// ─── handleSetupText: step asked-bot-token ───────────────────────────────
-
-describe('handleSetupText: asked-bot-token', () => {
-  it('advances to confirm-allowlist with valid token shape', () => {
-    const state = makeState({
-      step: 'asked-bot-token',
-      slug: 'gymbro',
-      persona: 'Gym Bro',
-      model: null,
-      emoji: null,
-    })
-    const action = handleSetupText({ state, text: '1234567890:AAHxxxxxxxxxxxxxxxxxxxxxxx', callerId: CALLER })
-    expect(action.kind).toBe('confirm-allowlist')
-    if (action.kind === 'confirm-allowlist') expect(action.callerId).toBe(CALLER)
-  })
-
-  it('returns error on bad token shape', () => {
-    const state = makeState({
-      step: 'asked-bot-token',
-      slug: 'gymbro',
-      persona: 'Gym Bro',
-    })
-    const action = handleSetupText({ state, text: 'notavalidtoken', callerId: CALLER })
-    expect(action.kind).toBe('error')
-    if (action.kind === 'error') expect(action.stayInStep).toBe(true)
-  })
-
-  it('returns cancel when slug or persona is missing', () => {
-    const state = makeState({
-      step: 'asked-bot-token',
-      slug: null,
-      persona: null,
-    })
-    const action = handleSetupText({ state, text: '1234567890:AAHxxxxxxxxxxxxxxxxxxxxxxx', callerId: CALLER })
-    expect(action.kind).toBe('cancel')
-  })
-})
-
-// ─── handleSetupText: step confirming-allowlist ───────────────────────────
-
-describe('handleSetupText: confirming-allowlist (#189 — now transitions to call-create-agent)', () => {
-  const baseState = makeState({
-    step: 'confirming-allowlist',
-    slug: 'gymbro',
-    persona: 'Gym Bro',
-    model: null,
-    emoji: null,
-    profile: 'default',
-    botToken: '1234567890:AAHxxxxxxxxxxxxxxxxxxxxxxx',
-  })
-
-  it('advances to call-create-agent on "yes"', () => {
-    const action = call(baseState, 'yes')
-    expect(action.kind).toBe('call-create-agent')
-    if (action.kind === 'call-create-agent') {
-      expect(action.allowedUserId).toBe(CALLER)
-      expect(action.slug).toBe('gymbro')
-      expect(action.persona).toBe('Gym Bro')
-      expect(action.profile).toBe('default')
-    }
-  })
-
-  it('advances to call-create-agent on "y"', () => {
-    const action = call(baseState, 'y')
-    expect(action.kind).toBe('call-create-agent')
-    if (action.kind === 'call-create-agent') expect(action.allowedUserId).toBe(CALLER)
-  })
-
-  it('uses custom user_id when not "yes"', () => {
-    const action = call(baseState, '99999999')
-    expect(action.kind).toBe('call-create-agent')
-    if (action.kind === 'call-create-agent') expect(action.allowedUserId).toBe('99999999')
-  })
-
-  it('falls back to "default" profile for legacy in-flight flows where profile is null', () => {
-    // Simulates a flow that started before the #190 schema migration —
-    // SQLite returns NULL for the new `profile` column, the wizard
-    // shouldn't break.
-    const legacy = makeState({
-      step: 'confirming-allowlist',
-      slug: 'oldgymbro',
-      persona: 'Old Gym',
-      botToken: '1234567890:AAHxxxxxxxxxxxxxxxxxxxxxxx',
-      profile: null,
-    })
-    const action = call(legacy, 'yes')
-    expect(action.kind).toBe('call-create-agent')
-    if (action.kind === 'call-create-agent') expect(action.profile).toBe('default')
-  })
-})
-
-// ─── handleSetupText: step asked-oauth-code (#189) ───────────────────────
-
-describe('handleSetupText: asked-oauth-code (#189)', () => {
-  function oauthState(): SetupFlowState {
-    return makeState({
-      step: 'asked-oauth-code',
-      slug: 'gymbro',
-      persona: 'Gym Bro',
-      profile: 'default',
-      botToken: '1234567890:AAHxxxxxxxxxxxxxxxxxxxxxxx',
-      authSessionName: 'gymbro-foreman',
-      loginUrl: 'https://example.com/login',
-    })
-  }
-
-  it('advances to call-complete-creation with a valid-shape code', () => {
-    const action = call(oauthState(), 'a1b2c3d4e5')
-    expect(action.kind).toBe('call-complete-creation')
-    if (action.kind === 'call-complete-creation') {
-      expect(action.slug).toBe('gymbro')
-      expect(action.code).toBe('a1b2c3d4e5')
-    }
-  })
-
-  it('returns error stayInStep on too-short code', () => {
-    const action = call(oauthState(), 'abc')
-    expect(action.kind).toBe('error')
-    if (action.kind === 'error') expect(action.stayInStep).toBe(true)
-  })
-
-  it('cancels when slug missing (corrupt state)', () => {
-    const action = call(makeState({ step: 'asked-oauth-code' }), 'a1b2c3d4e5')
-    expect(action.kind).toBe('cancel')
-  })
-})
-
-// ─── handleSetupText: terminal steps ─────────────────────────────────────
-
-describe('handleSetupText: terminal steps', () => {
-  it('returns cancel for reconciling step', () => {
-    const state = makeState({ step: 'reconciling' })
-    const action = handleSetupText({ state, text: 'anything', callerId: CALLER })
-    expect(action.kind).toBe('cancel')
-  })
-
-  it('returns cancel for done step', () => {
-    const state = makeState({ step: 'done' })
-    const action = handleSetupText({ state, text: 'anything', callerId: CALLER })
-    expect(action.kind).toBe('cancel')
-  })
-})
-
-// ─── makeSetupInitialState ────────────────────────────────────────────────
-
-describe('makeSetupInitialState', () => {
-  it('sets step to asked-slug when no slug', () => {
-    const s = makeSetupInitialState('chat1', null)
-    expect(s.step).toBe('asked-slug')
-    expect(s.slug).toBeNull()
-  })
-
-  it('sets step to asked-persona when slug provided', () => {
-    const s = makeSetupInitialState('chat1', 'gymbro')
-    expect(s.step).toBe('asked-persona')
-    expect(s.slug).toBe('gymbro')
-  })
-})
-
-// ─── advanceSetupState ────────────────────────────────────────────────────
-
-describe('advanceSetupState', () => {
-  it('merges updates and bumps updatedAt', () => {
-    const original = makeState({ updatedAt: 1000 })
-    const advanced = advanceSetupState(original, { step: 'asked-persona', slug: 'gymbro' })
-    expect(advanced.step).toBe('asked-persona')
-    expect(advanced.slug).toBe('gymbro')
-    expect(advanced.chatId).toBe(original.chatId)
-    expect(advanced.updatedAt).toBeGreaterThanOrEqual(original.updatedAt)
-  })
-})
-
-// ─── setupStepLabel ───────────────────────────────────────────────────────
-
-describe('setupStepLabel', () => {
-  const cases: [import('../foreman/setup-state.js').SetupFlowStep, string][] = [
-    ['asked-slug', 'waiting for agent slug'],
-    ['asked-persona', 'waiting for persona name'],
-    ['asked-model', 'waiting for model choice'],
-    ['asked-emoji', 'waiting for emoji'],
-    ['asked-bot-token', 'waiting for BotFather token'],
-    ['confirming-allowlist', 'waiting for allowlist confirmation'],
-    ['reconciling', 'provisioning agent'],
-    ['done', 'done'],
-  ]
-  for (const [step, expected] of cases) {
-    it(`labels ${step} correctly`, () => expect(setupStepLabel(step)).toBe(expected))
-  }
-})

package/telegram-plugin/tests/setup-state.test.ts

@@ -1,146 +0,0 @@
-/**
- * Tests for /setup wizard SQLite state (setup-state.ts).
- *
- * Uses bun:test (not vitest) because setup-state.ts imports bun:sqlite.
- * Run with: bun test telegram-plugin/tests/setup-state.test.ts
- */
-
-import { describe, it, expect, beforeEach, afterEach } from 'bun:test'
-import { mkdtempSync, rmSync } from 'fs'
-import { tmpdir } from 'os'
-
-let tmpDir: string
-
-beforeEach(() => {
-  tmpDir = mkdtempSync(tmpdir() + '/setup-state-test-')
-  process.env.SWITCHROOM_FOREMAN_DIR = tmpDir
-})
-
-afterEach(async () => {
-  const { _resetSetupDbForTest } = await import('../foreman/setup-state.js')
-  _resetSetupDbForTest()
-  delete process.env.SWITCHROOM_FOREMAN_DIR
-  try { rmSync(tmpDir, { recursive: true, force: true }) } catch { /* ignore */ }
-})
-
-function makeState(chatId = 'chat1') {
-  const now = Date.now()
-  return {
-    chatId,
-    step: 'asked-slug' as const,
-    slug: null,
-    persona: null,
-    model: null,
-    emoji: null,
-    botToken: null,
-    allowedUserId: null,
-    startedAt: now,
-    updatedAt: now,
-  }
-}
-
-// ─── Round-trip: setSetupState + getSetupState ────────────────────────────
-
-describe('setup-state: round-trip', () => {
-  it('stores and retrieves initial state', async () => {
-    const { setSetupState, getSetupState } = await import('../foreman/setup-state.js')
-    const state = makeState()
-    setSetupState(state)
-    const retrieved = getSetupState('chat1')
-    expect(retrieved).not.toBeNull()
-    expect(retrieved?.step).toBe('asked-slug')
-    expect(retrieved?.slug).toBeNull()
-    expect(retrieved?.chatId).toBe('chat1')
-  })
-
-  it('returns null for unknown chatId', async () => {
-    const { getSetupState } = await import('../foreman/setup-state.js')
-    expect(getSetupState('nonexistent')).toBeNull()
-  })
-
-  it('upserts state on repeat setSetupState', async () => {
-    const { setSetupState, getSetupState } = await import('../foreman/setup-state.js')
-    const state = makeState()
-    setSetupState(state)
-    setSetupState({ ...state, step: 'asked-persona', slug: 'gymbro' })
-    const retrieved = getSetupState('chat1')
-    expect(retrieved?.step).toBe('asked-persona')
-    expect(retrieved?.slug).toBe('gymbro')
-  })
-
-  it('stores all fields', async () => {
-    const { setSetupState, getSetupState } = await import('../foreman/setup-state.js')
-    const now = Date.now()
-    setSetupState({
-      chatId: 'chat2',
-      step: 'asked-bot-token',
-      slug: 'myagent',
-      persona: 'My Agent',
-      model: 'sonnet',
-      emoji: '🤖',
-      botToken: '1234567890:AAHxxxxxxxxxxxxxxxxxxxxxxx',
-      allowedUserId: '99999999',
-      startedAt: now - 1000,
-      updatedAt: now,
-    })
-    const retrieved = getSetupState('chat2')
-    expect(retrieved?.slug).toBe('myagent')
-    expect(retrieved?.persona).toBe('My Agent')
-    expect(retrieved?.model).toBe('sonnet')
-    expect(retrieved?.emoji).toBe('🤖')
-    expect(retrieved?.botToken).toBe('1234567890:AAHxxxxxxxxxxxxxxxxxxxxxxx')
-    expect(retrieved?.allowedUserId).toBe('99999999')
-  })
-})
-
-// ─── clearSetupState ──────────────────────────────────────────────────────
-
-describe('setup-state: clearSetupState', () => {
-  it('removes state for given chat', async () => {
-    const { setSetupState, clearSetupState, getSetupState } = await import('../foreman/setup-state.js')
-    setSetupState(makeState('chatA'))
-    setSetupState(makeState('chatB'))
-    clearSetupState('chatA')
-    expect(getSetupState('chatA')).toBeNull()
-    expect(getSetupState('chatB')).not.toBeNull()
-  })
-
-  it('is a no-op when chat has no state', async () => {
-    const { clearSetupState, getSetupState } = await import('../foreman/setup-state.js')
-    // Should not throw
-    clearSetupState('nobody')
-    expect(getSetupState('nobody')).toBeNull()
-  })
-})
-
-// ─── listActiveSetupFlows ─────────────────────────────────────────────────
-
-describe('setup-state: listActiveSetupFlows', () => {
-  it('returns only non-done flows within maxAge', async () => {
-    const { setSetupState, listActiveSetupFlows } = await import('../foreman/setup-state.js')
-    const now = Date.now()
-
-    setSetupState({ ...makeState('chat1'), step: 'asked-slug', updatedAt: now })
-    setSetupState({ ...makeState('chat2'), step: 'done', updatedAt: now })
-    setSetupState({ ...makeState('chat3'), step: 'asked-persona', updatedAt: now - 2 * 60 * 60 * 1000 }) // 2 hours old
-
-    const active = listActiveSetupFlows(60 * 60 * 1000) // 1 hour
-    expect(active.length).toBe(1)
-    expect(active[0].chatId).toBe('chat1')
-  })
-
-  it('returns empty list when nothing active', async () => {
-    const { listActiveSetupFlows } = await import('../foreman/setup-state.js')
-    const active = listActiveSetupFlows()
-    expect(active).toEqual([])
-  })
-
-  it('returns multiple active flows', async () => {
-    const { setSetupState, listActiveSetupFlows } = await import('../foreman/setup-state.js')
-    const now = Date.now()
-    setSetupState({ ...makeState('c1'), updatedAt: now })
-    setSetupState({ ...makeState('c2'), step: 'asked-persona', updatedAt: now })
-    const active = listActiveSetupFlows()
-    expect(active.length).toBe(2)
-  })
-})