infra-kit 0.1.128 → 0.1.129

package/src/commands/audit/audit.ts

@@ -1,177 +0,0 @@
-import path from 'node:path'
-import process from 'node:process'
-import { z } from 'zod'
-
-import { getProjectRoot } from 'src/lib/git-utils'
-import { logger } from 'src/lib/logger'
-import { ROOT_DEFAULT_RULES } from 'src/lib/package-config'
-import type { ResolvedPackageRules } from 'src/lib/package-config'
-import { discoverPackages, pathExists, validatePackage } from 'src/lib/package-validator'
-import type { PackageValidationResult } from 'src/lib/package-validator'
-import { defineMcpTool, textContent } from 'src/types'
-
-// TODO [DO]: extract `audit` into its own standalone CLI tool, decoupled from infra-kit.
-
-interface AuditOptions {
-  /** Audit every non-vendor workspace package instead of just the current one. */
-  all?: boolean
-  /** Audit the monorepo root (turbo pipeline + root commands) instead of a package. */
-  root?: boolean
-  /** Directory to resolve the current package from. Defaults to `process.cwd()`. */
-  cwd?: string
-}
-
-/** A directory to audit plus the under-the-hood defaults that apply to it. */
-interface AuditTarget {
-  dir: string
-  baseline?: Readonly<ResolvedPackageRules>
-}
-
-/**
- * Walk upward from `start` to the nearest directory containing a package.json.
- * Used so `infra-kit audit` (no `--all`) targets the package whose package.json
- * script invoked it, regardless of the exact working directory.
- *
- * @example
- * await findPackageRoot('/repo/packages/serverless-config/src')
- * // => '/repo/packages/serverless-config'
- */
-const findPackageRoot = async (start: string): Promise<string> => {
-  let current = path.resolve(start)
-
-  while (current !== path.dirname(current)) {
-    if (await pathExists(path.join(current, 'package.json'))) {
-      return current
-    }
-
-    current = path.dirname(current)
-  }
-
-  if (await pathExists(path.join(current, 'package.json'))) {
-    return current
-  }
-
-  throw new Error(`No package.json found in or above ${start}`)
-}
-
-/**
- * Resolve which directories to audit, and the baseline defaults each uses:
- * `root` → the monorepo root with {@link ROOT_DEFAULT_RULES}; `all` → every
- * discovered non-vendor package; otherwise the package walked up from cwd.
- */
-const resolveTargets = async (options: AuditOptions): Promise<AuditTarget[]> => {
-  if (options.root) {
-    return [{ dir: await getProjectRoot(), baseline: ROOT_DEFAULT_RULES }]
-  }
-
-  if (options.all) {
-    const dirs = await discoverPackages(await getProjectRoot())
-
-    return dirs.map((dir) => {
-      return { dir }
-    })
-  }
-
-  return [{ dir: await findPackageRoot(options.cwd ?? process.cwd()) }]
-}
-
-/**
- * Print a package's audit result as doctor-style `[PASS]`/`[FAIL]` lines.
- */
-const logResult = (result: PackageValidationResult): void => {
-  const header = result.passed ? 'PASS' : 'FAIL'
-
-  logger.info(`\n${result.packageName} — ${header}`)
-
-  for (const check of result.checks) {
-    const icon = check.status === 'pass' ? '[PASS]' : '[FAIL]'
-
-    logger.info(`  ${icon} ${check.name}: ${check.message}`)
-  }
-}
-
-/**
- * Audit the monorepo root (`root`), every non-vendor workspace package (`all`),
- * or the package resolved by walking up from the working directory (default —
- * the shape used by a package's `"infra-kit-check": "infra-kit audit"` script). The
- * returned `structuredContent.allPassed` lets the CLI set a non-zero exit code so
- * the audit fails CI; this function never calls `process.exit` so the MCP tool
- * can reuse it.
- *
- * @example
- * // CLI inside packages/serverless-config: `infra-kit audit`
- * await audit()              // audits the current package
- * await audit({ all: true }) // audits every non-vendor workspace package
- * await audit({ root: true }) // audits the monorepo root (turbo + root commands)
- */
-export const audit = async (options: AuditOptions = {}) => {
-  const targets = await resolveTargets(options)
-
-  const results: PackageValidationResult[] = []
-
-  for (const target of targets) {
-    results.push(await validatePackage(target.dir, target.baseline))
-  }
-
-  for (const result of results) {
-    logResult(result)
-  }
-
-  const allPassed = results.every((result) => {
-    return result.passed
-  })
-
-  logger.info(`\n${allPassed ? '✅ All valid' : '❌ Audit failed'} (${results.length} checked)`)
-
-  const structuredContent = {
-    allPassed,
-    packages: results.map((result) => {
-      return {
-        name: result.packageName,
-        passed: result.passed,
-        checks: result.checks,
-      }
-    }),
-  }
-
-  return {
-    content: textContent(JSON.stringify(structuredContent, null, 2)),
-    structuredContent,
-  }
-}
-
-const auditInputSchema = {
-  all: z.boolean().optional().describe('Audit every non-vendor workspace package'),
-  root: z.boolean().optional().describe('Audit the monorepo root (turbo pipeline + root commands)'),
-}
-
-const auditOutputSchema = {
-  allPassed: z.boolean().describe('Whether every audited package passed all checks'),
-  packages: z
-    .array(
-      z.object({
-        name: z.string(),
-        passed: z.boolean(),
-        checks: z.array(
-          z.object({
-            name: z.string(),
-            status: z.enum(['pass', 'fail']),
-            message: z.string(),
-          }),
-        ),
-      }),
-    )
-    .describe('Per-package check results'),
-}
-
-// MCP Tool Registration
-export const auditMcpTool = defineMcpTool({
-  name: 'audit',
-  description:
-    'Audit packages against infra-kit.config.ts rules (config present and valid, required scripts, required files, and turbo tasks for the root). Defaults to the current package; all=true audits every non-vendor workspace package; root=true audits the monorepo root.',
-  inputSchema: auditInputSchema,
-  outputSchema: auditOutputSchema,
-  handler: (params) => {
-    return audit({ all: params.all, root: params.root })
-  },
-})

package/src/commands/audit/index.ts

@@ -1 +0,0 @@
-export { audit, auditMcpTool } from './audit'

package/src/commands/config/.omc/state/sessions/41336850-028c-476b-be1a-2164e1e46cac/last-tool-error-state.json

@@ -1,7 +0,0 @@
-{
-  "tool_name": "Bash",
-  "tool_input_preview": "{\"command\":\"cd src/commands/init && echo \\\"=== os. usage (excl import/comment) ===\\\" && rtk grep -n \\\"os\\\\.\\\" migrate-config.ts | grep -v \\\"node:os\\\" | grep -v \\\"homedir()\\\"; echo \\\"--- all os. lines ...",
-  "error": "Exit code 2\n(eval):cd:1: no such file or directory: src/commands/init\n--- all os. lines ---\ngrep: migrate-config.ts: No such file or directory\n0 matches for 'os\\.'\n=== fs. usage count ===\ngrep: migrate-config.ts: No such file or directory",
-  "timestamp": "2026-06-25T14:31:26.386Z",
-  "retry_count": 1
-}

package/src/commands/config/.omc/state/sessions/41336850-028c-476b-be1a-2164e1e46cac/pre-tool-advisory-throttle.json

@@ -1,18 +0,0 @@
-{
-  "version": 1,
-  "entries": {
-    "79a93d4a2f8f50b95f852280616242fee1855dc99a3c75211917f55e72e95fae": {
-      "last_emitted_at_ms": 1782397886227,
-      "message": "Use parallel execution for independent tasks. Use run_in_background for long operations (npm install, builds, tests)."
-    },
-    "466399dafa2d20f60587180bad0a07358eb7f2bce724df0ff682f038ad33f5ad": {
-      "last_emitted_at_ms": 1782397881323,
-      "message": "Read multiple files in parallel when possible for faster analysis."
-    },
-    "445ed27a3872b681d98190bae61ccb84954e1bc4e140df5370be958dee776b3a": {
-      "last_emitted_at_ms": 1782397873064,
-      "message": "Verify changes work after editing. Test functionality before marking complete."
-    }
-  },
-  "updated_at": "2026-06-25T14:31:26.227Z"
-}

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/)
-  })
-})