infra-kit 0.1.127 → 0.1.129

package/src/commands/config/config.ts

@@ -1,137 +0,0 @@
-import fs from 'node:fs/promises'
-import path from 'node:path'
-import process from 'node:process'
-import { $ } from 'zx'
-
-import { getInfraKitConfigPaths, resetInfraKitConfigCache } from 'src/lib/infra-kit-config'
-import { logger } from 'src/lib/logger'
-import { fileExists, tildify } from 'src/lib/path-display'
-import type { ToolsExecutionResult } from 'src/types'
-
-/**
- * Print the file paths that participate in the config merge chain along with
- * existence markers, so the user can see at a glance which override layers
- * are active.
- *
- * @example
- * // CLI: `infra-kit config path`
- * // INFO: Project name: api
- * // INFO: Config merge chain (later overrides earlier):
- * // INFO:   [✓] project (committed)    ~/projects/api/infra-kit.json
- * // INFO:   [ ] user global            ~/.infra-kit/infra-kit.json
- * // INFO:   [✓] user project           ~/.infra-kit/projects/api/infra-kit.json
- */
-export const configPath = async (): Promise<ToolsExecutionResult> => {
-  const paths = await getInfraKitConfigPaths()
-
-  const rows: { label: string; path: string; exists: boolean }[] = await Promise.all(
-    [
-      { label: 'project (committed)', path: paths.main },
-      { label: 'user global', path: paths.userGlobal },
-      { label: 'user project', path: paths.userProject },
-    ].map(async (row) => {
-      return { ...row, exists: await fileExists(row.path) }
-    }),
-  )
-
-  logger.info(`Project name: ${paths.projectName}\n`)
-  logger.info('Config merge chain (later overrides earlier):\n')
-
-  for (const row of rows) {
-    const marker = row.exists ? '  [✓]' : '  [ ]'
-
-    logger.info(`${marker} ${row.label.padEnd(22)} ${tildify(row.path)}`)
-  }
-
-  const structuredContent = {
-    projectName: paths.projectName,
-    layers: rows.map((r) => {
-      return { label: r.label, path: r.path, exists: r.exists }
-    }),
-  }
-
-  return {
-    content: [{ type: 'text', text: JSON.stringify(structuredContent, null, 2) }],
-    structuredContent,
-  }
-}
-
-/**
- * Open the user-scope per-project override file in $EDITOR, creating the
- * parent directory and a stub file on first use. Resets the config cache
- * after the editor exits so subsequent reads pick up edits without a restart.
- *
- * @example
- * // CLI: `infra-kit config edit`
- * // first run — creates ~/.infra-kit/projects/api/infra-kit.json ({}) + a sibling
- * //             infra-kit.example.jsonc reference, then $EDITOR opens the .json
- * // subsequent runs — opens the existing file as-is
- */
-export const configEdit = async (): Promise<ToolsExecutionResult> => {
-  const paths = await getInfraKitConfigPaths()
-  const editor = process.env.EDITOR || process.env.VISUAL || 'vi'
-
-  await fs.mkdir(path.dirname(paths.userProject), { recursive: true })
-
-  if (!(await fileExists(paths.userProject))) {
-    // JSON can't carry comments, so seed an empty-but-valid config and drop the
-    // annotated guidance next to it in a non-loaded .example.jsonc the loader
-    // never reads (it only globs the three exact `infra-kit.json` filenames).
-    const examplePath = exampleSiblingPath(paths.userProject)
-
-    await fs.writeFile(paths.userProject, '{}\n', 'utf-8')
-    await fs.writeFile(examplePath, buildUserProjectExample(paths.projectName), 'utf-8')
-
-    logger.info(`Created ${tildify(paths.userProject)} — see ${tildify(examplePath)} for the annotated reference.`)
-  }
-
-  logger.info(`Opening ${tildify(paths.userProject)} in ${editor}`)
-
-  await $({ stdio: 'inherit' })`${editor} ${paths.userProject}`
-
-  resetInfraKitConfigCache()
-
-  const structuredContent = { path: paths.userProject, editor }
-
-  return {
-    content: [{ type: 'text', text: JSON.stringify(structuredContent, null, 2) }],
-    structuredContent,
-  }
-}
-
-/**
- * Derive the non-loaded `.example.jsonc` sibling for a config path.
- *
- * @example
- * exampleSiblingPath('/u/.infra-kit/projects/api/infra-kit.json')
- * // => '/u/.infra-kit/projects/api/infra-kit.example.jsonc'
- */
-const exampleSiblingPath = (jsonPath: string): string => {
-  return jsonPath.replace(/\.json$/, '.example.jsonc')
-}
-
-/**
- * Annotated JSONC reference for the user-scope per-project override layer.
- * Written alongside the real `{}` config so the guidance the old YAML stub
- * carried in comments survives the move to JSON.
- *
- * @example
- * buildUserProjectExample('api')
- * // => '// infra-kit user override for api …\n{ … }\n'
- */
-const buildUserProjectExample = (projectName: string): string => {
-  return `// infra-kit user override for ${projectName} — ~/.infra-kit/projects/${projectName}/infra-kit.json
-//
-// Layer 3 (highest precedence) of the config merge chain. Shallow-merged on top
-// of <repo>/infra-kit.json and ~/.infra-kit/infra-kit.json — top-level keys
-// (environments, envManagement, ide, taskManager, worktrees, envAutoLoad) replace wholesale.
-//
-// This .example.jsonc is reference only — it is NOT loaded. Put real overrides
-// in the sibling infra-kit.json (strict JSON: no comments, double-quoted keys).
-{
-  // "worktrees": { "openInGithubDesktop": false, "openInCmux": true, "cmux": { "layout": "two-columns" } },
-  // // Auto-load Doppler env here. trigger (pick one): shell-startup | cli-invocation; config: an environment name.
-  // "envAutoLoad": { "trigger": "shell-startup", "config": "dev" }
-}
-`
-}

package/src/commands/config/index.ts

@@ -1 +0,0 @@
-export { configEdit, configPath } from './config'

package/src/commands/doctor/.omc/state/sessions/8dd0ad2f-3a90-4158-b9de-ec8e19a7c983/pre-tool-advisory-throttle.json

@@ -1,14 +0,0 @@
-{
-  "version": 1,
-  "entries": {
-    "79a93d4a2f8f50b95f852280616242fee1855dc99a3c75211917f55e72e95fae": {
-      "last_emitted_at_ms": 1782040957960,
-      "message": "Use parallel execution for independent tasks. Use run_in_background for long operations (npm install, builds, tests)."
-    },
-    "445ed27a3872b681d98190bae61ccb84954e1bc4e140df5370be958dee776b3a": {
-      "last_emitted_at_ms": 1782040931719,
-      "message": "Verify changes work after editing. Test functionality before marking complete."
-    }
-  },
-  "updated_at": "2026-06-21T11:22:37.960Z"
-}

package/src/commands/doctor/__tests__/agent-files.test.ts

@@ -1,88 +0,0 @@
-import fs from 'node:fs'
-import os from 'node:os'
-import path from 'node:path'
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-
-import { writeAgentFiles } from 'src/commands/init/agent-files'
-import { getProjectRoot, getRepoName } from 'src/lib/git-utils'
-
-import { checkAgentFiles } from '../doctor'
-
-vi.mock('src/lib/git-utils', () => {
-  return {
-    getProjectRoot: vi.fn(),
-    getRepoName: vi.fn(),
-  }
-})
-
-const writeFile = (filePath: string, content: string): void => {
-  fs.mkdirSync(path.dirname(filePath), { recursive: true })
-  fs.writeFileSync(filePath, content, 'utf-8')
-}
-
-const withTmpRepo = async (fn: (tmp: string) => Promise<void>, opts: { repo?: boolean } = {}): Promise<void> => {
-  const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'infra-kit-doctor-agents-test-'))
-
-  vi.mocked(getProjectRoot).mockResolvedValue(tmp)
-  vi.mocked(getRepoName).mockResolvedValue(path.basename(tmp))
-
-  if (opts.repo !== false) {
-    writeFile(path.join(tmp, 'infra-kit.json'), '{"environments":["dev"]}\n')
-  }
-
-  try {
-    await fn(tmp)
-  } finally {
-    fs.rmSync(tmp, { recursive: true, force: true })
-  }
-}
-
-const statusOf = (checks: { name: string; status: string }[], name: string): string | undefined => {
-  return checks.find((c) => {
-    return c.name === name
-  })?.status
-}
-
-describe('checkAgentFiles', () => {
-  beforeEach(() => {
-    vi.clearAllMocks()
-  })
-
-  afterEach(() => {
-    vi.clearAllMocks()
-  })
-
-  it('returns no checks outside an infra-kit repo (never crashes)', async () => {
-    await withTmpRepo(
-      async () => {
-        await expect(checkAgentFiles()).resolves.toEqual([])
-      },
-      { repo: false },
-    )
-  })
-
-  it('checks only CLAUDE.md after init runs the agent-files step — no AGENTS.md or .cursor check', async () => {
-    await withTmpRepo(async () => {
-      await writeAgentFiles()
-
-      const checks = await checkAgentFiles()
-
-      expect(checks).toHaveLength(1)
-      expect(statusOf(checks, 'CLAUDE.md block')).toBe('pass')
-      // The legacy AGENTS.md and .cursor/rules checks no longer exist.
-      expect(statusOf(checks, 'AGENTS.md block')).toBeUndefined()
-      expect(statusOf(checks, '.cursor/rules block')).toBeUndefined()
-    })
-  })
-
-  it('flags a missing CLAUDE.md block', async () => {
-    await withTmpRepo(async (tmp) => {
-      await writeAgentFiles()
-      fs.rmSync(path.join(tmp, 'CLAUDE.md'))
-
-      const checks = await checkAgentFiles()
-
-      expect(statusOf(checks, 'CLAUDE.md block')).toBe('fail')
-    })
-  })
-})

package/src/commands/doctor/__tests__/check-ide-installed.test.ts

@@ -1,142 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-
-import { checkIdeInstalled } from '../doctor'
-
-const config = vi.hoisted(() => {
-  return { value: {} as { ide?: unknown }, shouldThrow: false }
-})
-
-vi.mock('src/lib/infra-kit-config', () => {
-  return {
-    resetInfraKitConfigCache: vi.fn(),
-    getInfraKitConfig: vi.fn(() => {
-      if (config.shouldThrow) {
-        return Promise.reject(new Error('bad config'))
-      }
-
-      return Promise.resolve(config.value)
-    }),
-    // Mirror the real normalizer (single → [ide], array → as-is, undefined → []).
-    resolveConfiguredIdes: vi.fn((cfg: { ide?: unknown }) => {
-      const ide = cfg.ide
-
-      if (!ide) return []
-
-      return Array.isArray(ide) ? ide : [ide]
-    }),
-  }
-})
-
-const zx = vi.hoisted(() => {
-  return { shouldThrow: false }
-})
-
-vi.mock('zx', () => {
-  return {
-    $: vi.fn(() => {
-      if (zx.shouldThrow) {
-        return Promise.reject(new Error('command not found'))
-      }
-
-      return Promise.resolve({ stdout: '' })
-    }),
-  }
-})
-
-vi.mock('src/lib/logger', () => {
-  return { logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() } }
-})
-
-describe('checkIdeInstalled', () => {
-  beforeEach(() => {
-    config.value = {}
-    config.shouldThrow = false
-    zx.shouldThrow = false
-    vi.clearAllMocks()
-  })
-
-  it('passes informationally when no IDE is configured', async () => {
-    config.value = { ide: undefined }
-
-    const result = await checkIdeInstalled()
-
-    expect(result.status).toBe('pass')
-    expect(result.message).toMatch(/No IDE configured/)
-  })
-
-  it('passes when the configured Cursor binary is present', async () => {
-    config.value = { ide: { provider: 'cursor', config: { workspaceConfigPath: 'ws' } } }
-
-    const result = await checkIdeInstalled()
-
-    expect(result.status).toBe('pass')
-    expect(result.message).toMatch(/Installed: Cursor/)
-  })
-
-  it('fails when the configured Cursor binary is missing', async () => {
-    config.value = { ide: { provider: 'cursor', config: { workspaceConfigPath: 'ws' } } }
-    zx.shouldThrow = true
-
-    const result = await checkIdeInstalled()
-
-    expect(result.status).toBe('fail')
-    expect(result.message).toMatch(/Cursor is not installed/)
-  })
-
-  it('passes when the configured Zed binary is present', async () => {
-    config.value = { ide: { provider: 'zed', config: {} } }
-
-    const result = await checkIdeInstalled()
-
-    expect(result.status).toBe('pass')
-    expect(result.message).toMatch(/Installed: Zed/)
-  })
-
-  it('fails when the configured Zed binary is missing', async () => {
-    config.value = { ide: { provider: 'zed', config: {} } }
-    zx.shouldThrow = true
-
-    const result = await checkIdeInstalled()
-
-    expect(result.status).toBe('fail')
-    expect(result.message).toMatch(/Zed is not installed/)
-  })
-
-  it('passes listing all editors when multiple are configured and present', async () => {
-    config.value = {
-      ide: [
-        { provider: 'cursor', config: { workspaceConfigPath: 'ws' } },
-        { provider: 'zed', config: {} },
-      ],
-    }
-
-    const result = await checkIdeInstalled()
-
-    expect(result.status).toBe('pass')
-    expect(result.message).toMatch(/Installed: Cursor, Zed/)
-  })
-
-  it('fails when one of several configured editors is missing', async () => {
-    config.value = {
-      ide: [
-        { provider: 'cursor', config: { workspaceConfigPath: 'ws' } },
-        { provider: 'zed', config: {} },
-      ],
-    }
-    zx.shouldThrow = true
-
-    const result = await checkIdeInstalled()
-
-    expect(result.status).toBe('fail')
-    expect(result.message).toMatch(/Cursor is not installed/)
-  })
-
-  it('passes informationally when the config cannot be read', async () => {
-    config.shouldThrow = true
-
-    const result = await checkIdeInstalled()
-
-    expect(result.status).toBe('pass')
-    expect(result.message).toMatch(/Skipped/)
-  })
-})

package/src/commands/doctor/__tests__/check-legacy-user-global-config.test.ts

@@ -1,110 +0,0 @@
-import fs from 'node:fs'
-import os from 'node:os'
-import path from 'node:path'
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-
-import { checkLegacyUserGlobalConfig } from '../doctor'
-
-const cfg = vi.hoisted(() => {
-  return { userGlobal: '', shouldThrow: false }
-})
-
-vi.mock('src/lib/infra-kit-config', () => {
-  return {
-    getInfraKitConfig: vi.fn(),
-    resetInfraKitConfigCache: vi.fn(),
-    resolveConfiguredIdes: vi.fn(() => {
-      return []
-    }),
-    getInfraKitConfigPaths: vi.fn(() => {
-      if (cfg.shouldThrow) {
-        return Promise.reject(new Error('no repo'))
-      }
-
-      return Promise.resolve({
-        main: '',
-        userGlobal: cfg.userGlobal,
-        userProject: '',
-        projectName: 'api',
-      })
-    }),
-  }
-})
-
-vi.mock('src/lib/logger', () => {
-  return { logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() } }
-})
-
-const withTmpHome = async (fn: (dir: string) => Promise<void>): Promise<void> => {
-  const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'infra-kit-doctor-legacy-test-'))
-  const dir = path.join(tmp, '.infra-kit')
-
-  fs.mkdirSync(dir, { recursive: true })
-  cfg.userGlobal = path.join(dir, 'infra-kit.json')
-  const homedirSpy = vi.spyOn(os, 'homedir').mockReturnValue(tmp)
-
-  try {
-    await fn(dir)
-  } finally {
-    homedirSpy.mockRestore()
-    fs.rmSync(tmp, { recursive: true, force: true })
-  }
-}
-
-describe('checkLegacyUserGlobalConfig', () => {
-  beforeEach(() => {
-    cfg.shouldThrow = false
-    vi.clearAllMocks()
-  })
-
-  afterEach(() => {
-    vi.clearAllMocks()
-  })
-
-  it('fails with a migrate message when only config.json exists (overrides not applied)', async () => {
-    await withTmpHome(async (dir) => {
-      fs.writeFileSync(path.join(dir, 'config.json'), '{}\n')
-
-      const result = await checkLegacyUserGlobalConfig()
-
-      expect(result.status).toBe('fail')
-      expect(result.message).toMatch(/run `infra-kit init` to migrate/)
-      expect(result.message).toMatch(/not being applied/)
-    })
-  })
-
-  it('fails with a stale message when both config.json and infra-kit.json exist', async () => {
-    await withTmpHome(async (dir) => {
-      fs.writeFileSync(path.join(dir, 'config.json'), '{}\n')
-      fs.writeFileSync(path.join(dir, 'infra-kit.json'), '{}\n')
-
-      const result = await checkLegacyUserGlobalConfig()
-
-      expect(result.status).toBe('fail')
-      expect(result.message).toMatch(/Stale legacy config\.json/)
-      expect(result.message).toMatch(/remove the old file/)
-      // Must NOT falsely claim overrides aren't applied — infra-kit.json is active.
-      expect(result.message).not.toMatch(/not being applied/)
-    })
-  })
-
-  it('passes when there is no legacy config.json', async () => {
-    await withTmpHome(async (dir) => {
-      fs.writeFileSync(path.join(dir, 'infra-kit.json'), '{}\n')
-
-      const result = await checkLegacyUserGlobalConfig()
-
-      expect(result.status).toBe('pass')
-      expect(result.message).toMatch(/No legacy user-global config\.json/)
-    })
-  })
-
-  it('passes informationally when the config paths cannot be resolved', async () => {
-    cfg.shouldThrow = true
-
-    const result = await checkLegacyUserGlobalConfig()
-
-    expect(result.status).toBe('pass')
-    expect(result.message).toMatch(/Skipped/)
-  })
-})