infra-kit 0.1.128 → 0.1.130

package/src/commands/init/__tests__/migrate-factory-config.test.ts

@@ -1,93 +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 { logger } from 'src/lib/logger'
-
-import { migrateFactoryConfigToJson } from '../migrate-config'
-
-let home: string
-let dir: string
-let oldTs: string
-let newJson: string
-let infoSpy: ReturnType<typeof vi.spyOn>
-
-const messages = (): string => {
-  return infoSpy.mock.calls
-    .map((c: unknown[]) => {
-      return String(c[0])
-    })
-    .join('\n')
-}
-
-beforeEach(() => {
-  home = fs.mkdtempSync(path.join(os.tmpdir(), 'migrate-factory-test-'))
-  vi.spyOn(os, 'homedir').mockReturnValue(home)
-  dir = path.join(home, '.infra-kit')
-  oldTs = path.join(dir, 'vendor.config.ts')
-  newJson = path.join(dir, 'vendor.json')
-  fs.mkdirSync(dir, { recursive: true })
-  infoSpy = vi.spyOn(logger, 'info').mockImplementation(() => {})
-})
-
-afterEach(() => {
-  vi.restoreAllMocks()
-  fs.rmSync(home, { recursive: true, force: true })
-})
-
-describe('migrateFactoryConfigToJson', () => {
-  it('converts an object-form vendor.config.ts to vendor.json and removes the .ts (plain ✓, no warning)', async () => {
-    fs.writeFileSync(oldTs, `export default { workspaceDir: '~/projects', targets: ['repo-a'] }`)
-
-    await migrateFactoryConfigToJson()
-
-    expect(fs.existsSync(oldTs)).toBe(false)
-    expect(JSON.parse(fs.readFileSync(newJson, 'utf-8'))).toEqual({ workspaceDir: '~/projects', targets: ['repo-a'] })
-    expect(messages()).toMatch(/✓ Migrated/)
-    expect(messages()).not.toMatch(/STATIC SNAPSHOT/)
-  })
-
-  it('converts a factory-function form AND emits the STATIC SNAPSHOT warning (not a plain ✓)', async () => {
-    fs.writeFileSync(oldTs, `export default () => ({ workspaceDir: '/abs/repos', targets: ['repo-b'] })`)
-
-    await migrateFactoryConfigToJson()
-
-    expect(fs.existsSync(oldTs)).toBe(false)
-    expect(JSON.parse(fs.readFileSync(newJson, 'utf-8'))).toEqual({ workspaceDir: '/abs/repos', targets: ['repo-b'] })
-    expect(messages()).toMatch(/STATIC SNAPSHOT/)
-    expect(messages()).not.toMatch(/✓ Migrated/)
-  })
-
-  it('is a no-op when there is no legacy vendor.config.ts (idempotent)', async () => {
-    await migrateFactoryConfigToJson()
-
-    expect(fs.existsSync(newJson)).toBe(false)
-    expect(infoSpy).not.toHaveBeenCalled()
-  })
-
-  it('never overwrites an existing vendor.json; leaves the .ts in place and warns', async () => {
-    const existing = { workspaceDir: '~/active', targets: ['keep-me'] }
-
-    fs.writeFileSync(oldTs, `export default { workspaceDir: '~/stale', targets: ['old'] }`)
-    fs.writeFileSync(newJson, JSON.stringify(existing))
-
-    await migrateFactoryConfigToJson()
-
-    // Active vendor.json untouched; the stale .ts is preserved (not deleted).
-    expect(JSON.parse(fs.readFileSync(newJson, 'utf-8'))).toEqual(existing)
-    expect(fs.existsSync(oldTs)).toBe(true)
-    expect(messages()).toMatch(/already exists/)
-  })
-
-  it('leaves an invalid old .ts in place (non-fatal) and writes no vendor.json', async () => {
-    // Missing required `targets` → fails factoryConfigSchema.
-    fs.writeFileSync(oldTs, `export default { workspaceDir: '~/projects' }`)
-
-    await expect(migrateFactoryConfigToJson()).resolves.toBeUndefined()
-
-    expect(fs.existsSync(oldTs)).toBe(true)
-    expect(fs.existsSync(newJson)).toBe(false)
-    expect(messages()).toMatch(/invalid factory config/)
-  })
-})

package/src/commands/init/__tests__/seed-user-global-config.test.ts

@@ -1,64 +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 { seedUserGlobalConfig } from '../init'
-
-let home: string
-let dir: string
-
-beforeEach(() => {
-  home = fs.mkdtempSync(path.join(os.tmpdir(), 'seed-user-global-test-'))
-  vi.spyOn(os, 'homedir').mockReturnValue(home)
-  dir = path.join(home, '.infra-kit')
-})
-
-afterEach(() => {
-  vi.restoreAllMocks()
-  fs.rmSync(home, { recursive: true, force: true })
-})
-
-describe('seedUserGlobalConfig', () => {
-  it('seeds the infra-kit.json stub and both fully-commented example files', () => {
-    seedUserGlobalConfig()
-
-    expect(fs.readFileSync(path.join(dir, 'infra-kit.json'), 'utf-8')).toBe('{}\n')
-
-    const infraExample = fs.readFileSync(path.join(dir, 'infra-kit.example.jsonc'), 'utf-8')
-
-    // Every recognized infra-kit property is documented.
-    for (const key of ['environments', 'envManagement', 'ide', 'taskManager', 'worktrees']) {
-      expect(infraExample).toContain(key)
-    }
-
-    const vendorExample = fs.readFileSync(path.join(dir, 'vendor.example.jsonc'), 'utf-8')
-
-    expect(vendorExample).toContain('workspaceDir')
-    expect(vendorExample).toContain('targets')
-  })
-
-  it('never seeds a real vendor.json (only the example)', () => {
-    seedUserGlobalConfig()
-
-    expect(fs.existsSync(path.join(dir, 'vendor.example.jsonc'))).toBe(true)
-    expect(fs.existsSync(path.join(dir, 'vendor.json'))).toBe(false)
-  })
-
-  it('refreshes the example files on a re-run while leaving an existing infra-kit.json byte-unchanged', () => {
-    fs.mkdirSync(dir, { recursive: true })
-    const realConfig = '{ "environments": ["dev"] }\n'
-
-    fs.writeFileSync(path.join(dir, 'infra-kit.json'), realConfig)
-    // Stale example a previous version might have written.
-    fs.writeFileSync(path.join(dir, 'infra-kit.example.jsonc'), '// stale\n')
-
-    seedUserGlobalConfig()
-
-    // Real config preserved exactly…
-    expect(fs.readFileSync(path.join(dir, 'infra-kit.json'), 'utf-8')).toBe(realConfig)
-    // …examples refreshed to the current template (no longer "stale").
-    expect(fs.readFileSync(path.join(dir, 'infra-kit.example.jsonc'), 'utf-8')).not.toBe('// stale\n')
-    expect(fs.existsSync(path.join(dir, 'vendor.example.jsonc'))).toBe(true)
-  })
-})

package/src/commands/init/__tests__/shell-body.test.ts

@@ -1,131 +0,0 @@
-import { describe, expect, it } from 'vitest'
-
-import { buildShellBody } from '../init'
-
-describe('buildShellBody — env auto-load shell-startup block', () => {
-  const body = buildShellBody()
-
-  it('defines and invokes the one-shot startup autoload function', () => {
-    expect(body).toContain('_infra_kit_startup_autoload()')
-    expect(body).toContain('_infra_kit_startup_autoload\n')
-  })
-
-  it('gates on a session and skips the spawn only when the env is for THIS project root', () => {
-    expect(body).toContain('[[ -z "$INFRA_KIT_SESSION" ]] && return')
-    expect(body).toContain('[[ -n "$INFRA_KIT_ENV_CONFIG" && "$dir" == "$INFRA_KIT_ENV_PROJECT_ROOT" ]] && return')
-  })
-
-  it('walks up for infra-kit.json with a fixed-point break (no infinite loop on empty $PWD)', () => {
-    expect(body).toContain('[[ -f "$dir/infra-kit.json" ]]')
-    expect(body).toContain('while [[ -n "$dir" && "$dir" != "$prev" ]]')
-  })
-
-  it('spawns env-autoload backgrounded and silenced, passing the canonical --project-dir', () => {
-    expect(body).toContain('infra-kit env-autoload --project-dir "$canon" & ) >/dev/null 2>&1')
-  })
-
-  it('keeps the precmd hook as a sourcer of env-load.sh', () => {
-    expect(body).toContain('add-zsh-hook precmd _infra_kit_autoload')
-  })
-
-  it('loads the zsh/sched module used by the startup poll', () => {
-    expect(body).toContain('zmodload zsh/sched 2>/dev/null')
-  })
-
-  it('defines a startup poll that re-runs the sourcer via sched', () => {
-    expect(body).toContain('_infra_kit_poll_autoload()')
-    expect(body).toContain('  _infra_kit_autoload\n')
-    expect(body).toContain('sched +1 _infra_kit_poll_autoload')
-  })
-
-  it('bounds the poll: re-arm only until loaded and within the startup window', () => {
-    expect(body).toContain('_INFRA_KIT_AUTOLOAD_WINDOW=6')
-    expect(body).toContain(
-      // eslint-disable-next-line no-template-curly-in-string
-      '  if (( _INFRA_KIT_LAST_LOAD_MTIME < _INFRA_KIT_SHELL_STARTED )) && (( ${EPOCHSECONDS:-0} - _INFRA_KIT_SHELL_STARTED < _INFRA_KIT_AUTOLOAD_WINDOW )); then',
-    )
-  })
-
-  it('arms the poll from the startup autoload right after the backgrounded spawn', () => {
-    const spawnIdx = body.indexOf('infra-kit env-autoload --project-dir "$canon" & ) >/dev/null 2>&1')
-    const armIdx = body.indexOf('sched +1 _infra_kit_poll_autoload', spawnIdx)
-
-    // arm line follows the spawn (same startup branch), before its `return`
-    expect(spawnIdx).toBeGreaterThanOrEqual(0)
-    expect(armIdx).toBeGreaterThan(spawnIdx)
-  })
-})
-
-describe('buildShellBody — warm cache', () => {
-  const body = buildShellBody()
-
-  it('defaults the warm TTL to 7200s (== DEFAULT_WARM_TTL_SECONDS) and exports it', () => {
-    // eslint-disable-next-line no-template-curly-in-string
-    expect(body).toContain(': ${_INFRA_KIT_WARM_TTL:=7200}')
-    expect(body).toContain('_INFRA_KIT_WARM_TTL')
-  })
-
-  it('derives the key with sha256 of the canonical dir (byte-identical to node)', () => {
-    expect(body).toContain('_infra_kit_warm_source()')
-    expect(body).toContain('printf %s "$canon" | shasum -a 256 | cut -c1-64')
-    expect(body).toContain('printf %s "$canon" | sha256sum | cut -c1-64')
-  })
-
-  it('gates warm-source on clear-marker (clear wins ties) and TTL', () => {
-    expect(body).toContain('(( cm >= wm )) && return')
-    // eslint-disable-next-line no-template-curly-in-string
-    expect(body).toContain('(( ${EPOCHSECONDS:-0} - wm >= _INFRA_KIT_WARM_TTL )) && return')
-  })
-
-  it('sources the warm file but does NOT set _INFRA_KIT_LAST_LOAD_MTIME (so the fresh session file still upgrades)', () => {
-    const fnStart = body.indexOf('_infra_kit_warm_source()')
-    const fnEnd = body.indexOf('_infra_kit_startup_autoload()')
-    const warmFn = body.slice(fnStart, fnEnd)
-
-    expect(warmFn).toContain('source "$warm"')
-    expect(warmFn).not.toContain('_INFRA_KIT_LAST_LOAD_MTIME')
-  })
-
-  it('warm-sources with the canonical dir BEFORE spawning the refresh', () => {
-    const warmCallIdx = body.indexOf('_infra_kit_warm_source "$canon"')
-    const spawnIdx = body.indexOf('infra-kit env-autoload --project-dir "$canon"')
-
-    expect(warmCallIdx).toBeGreaterThanOrEqual(0)
-    expect(spawnIdx).toBeGreaterThan(warmCallIdx)
-  })
-
-  it('forwards args on the env-clear wrapper so `env-clear --purge` reaches the CLI', () => {
-    expect(body).toContain('env-clear "$@"')
-  })
-})
-
-describe('buildShellBody — async notice (prompt-safe printing)', () => {
-  const body = buildShellBody()
-
-  it('defines a _infra_kit_notify helper guarded on ZLE that redraws the prompt', () => {
-    expect(body).toContain('_infra_kit_notify() {')
-    // Guards on ZLE being active: erases the current prompt line, then redraws.
-    expect(body).toContain("  zle && print -u2 -n $'\\r\\e[K'")
-    expect(body).toContain('  zle && zle reset-prompt')
-  })
-
-  it('routes the async notices through _infra_kit_notify (never a bare print at the idle prompt)', () => {
-    // eslint-disable-next-line no-template-curly-in-string
-    expect(body).toContain('_infra_kit_notify "infra-kit: auto-loaded vars for ${INFRA_KIT_ENV_CONFIG:-?}"')
-    expect(body).toContain('_infra_kit_notify "infra-kit: auto-cleared env"')
-    expect(body).toContain(
-      // eslint-disable-next-line no-template-curly-in-string
-      '_infra_kit_notify "infra-kit: loaded cached vars for ${INFRA_KIT_ENV_CONFIG:-?} (refreshing…)"',
-    )
-    // No async notice should still use a bare `print -u2 "infra-kit:` form.
-    expect(body).not.toContain('print -u2 "infra-kit:')
-  })
-
-  it('defines _infra_kit_notify before the startup autoload that (transitively) calls it', () => {
-    const notifyIdx = body.indexOf('_infra_kit_notify() {')
-    const startupInvokeIdx = body.lastIndexOf('_infra_kit_startup_autoload\n')
-
-    expect(notifyIdx).toBeGreaterThanOrEqual(0)
-    expect(startupInvokeIdx).toBeGreaterThan(notifyIdx)
-  })
-})

package/src/commands/init/agent-files.ts

@@ -1,219 +0,0 @@
-import fs from 'node:fs'
-import path from 'node:path'
-
-import { getInfraKitConfigPaths } from 'src/lib/infra-kit-config'
-import { logger } from 'src/lib/logger'
-import { hasManagedBlock, removeManagedBlock, upsertManagedBlock } from 'src/lib/managed-block'
-
-import packageJson from '../../../package.json' with { type: 'json' }
-
-/** Markers for the generated guidance block (HTML comments — invisible in rendered markdown). */
-export const AGENTS_MARKER_START = '<!-- infra-kit:begin -->'
-export const AGENTS_MARKER_END = '<!-- infra-kit:end -->'
-/** The version line lives on its own line (OMC `OMC:VERSION:` precedent) so the markers stay constant-matchable. */
-const AGENTS_VERSION_PREFIX = '<!-- infra-kit:version '
-
-/**
- * Marker pair for the legacy `@AGENTS.md` import region once injected into CLAUDE.md.
- * Migration-only now: `writeAgentFiles` strips this region so the full guidance body
- * can take its place. TODO(remove after ~2 release cycles once repos have re-run init).
- */
-export const AGENTS_IMPORT_START = '<!-- infra-kit:import:begin -->'
-export const AGENTS_IMPORT_END = '<!-- infra-kit:import:end -->'
-
-const AGENTS_FILE = 'AGENTS.md'
-const CLAUDE_FILE = 'CLAUDE.md'
-
-export type WriteAction = 'created' | 'updated' | 'unchanged' | 'removed'
-
-export interface AgentFileWrite {
-  path: string
-  action: WriteAction
-}
-
-export interface WriteAgentFilesResult {
-  /** True when run outside an infra-kit repo (no `infra-kit.json` at the git root). */
-  skipped: boolean
-  /** Repo root the files were written under, or null when skipped. */
-  root: string | null
-  written: AgentFileWrite[]
-}
-
-/** The version comment line, e.g. `<!-- infra-kit:version 0.1.105 -->`. */
-const buildVersionLine = (version: string): string => {
-  return `${AGENTS_VERSION_PREFIX}${version} -->`
-}
-
-/**
- * The generated guidance body (without the surrounding markers). Describes the
- * infra-kit CLI surface so any AI agent working in the repo learns it. Curated
- * from the real command set; the version line records which CLI version generated it.
- */
-const buildAgentsBody = (version: string): string => {
-  return [
-    buildVersionLine(version),
-    '',
-    '# infra-kit',
-    '',
-    'This repository uses the **infra-kit** CLI for environment, worktree, and release workflows.',
-    'This block is generated by `infra-kit init` — edit text *outside* the markers, never inside.',
-    '',
-    '## Commands (`ik` = `pnpm exec infra-kit`)',
-    '',
-    '- `ik env-load -c <config>` / `ik env-clear` / `ik env-status` — load, clear, or inspect Doppler env vars for a config (e.g. `dev`). Source the returned file to apply.',
-    '- `ik worktrees add` / `worktrees list` / `worktrees reload` / `worktrees remove` / `worktrees sync` — manage release and feature git worktrees.',
-    '- `ik release create` / `release list` / `release deploy-all` / `release deploy-selected` / `release deliver` / `release desc-edit` — release-branch and deploy flow.',
-    '- `ik release merge-dev` — merge the dev branch into every release branch.',
-    '- `ik audit` — audit packages against `infra-kit.config.ts` rules.',
-    '- `ik doctor` — check gh / doppler / aws / rtk install + auth and repo setup.',
-    '- `ik init` — (re)install shell integration and regenerate these agent-instruction files.',
-    '',
-    '## Conventions',
-    '',
-    '- Tickets are prefixed by area: `[FE]` frontend, `[BE]` backend, `[DO]` DevOps/infra, `[APP]` mobile app, `[IDEA]` proposal, `[ROOT]` cross-cutting. Pick the single dominant area.',
-    '- Environment variables are managed through Doppler via `ik env-load` — never commit secrets.',
-  ].join('\n')
-}
-
-const assertNotSymlink = (filePath: string): void => {
-  if (fs.existsSync(filePath) && fs.lstatSync(filePath).isSymbolicLink()) {
-    throw new Error(`Refusing to write ${filePath} because the destination is a symlink`)
-  }
-}
-
-/** Copy a file to a timestamped `<file>.backup.<timestamp>` sibling. */
-const backupFile = (filePath: string): void => {
-  const stamp = new Date().toISOString().replace(/[:.]/g, '-')
-
-  fs.copyFileSync(filePath, `${filePath}.backup.${stamp}`)
-}
-
-/**
- * Write `nextContent` to `filePath` with OMC-style safety: refuse symlinks,
- * back up the prior file with a timestamp before overwriting, and skip the
- * write entirely when nothing changed (keeps re-runs churn-free).
- */
-const writeManaged = (filePath: string, nextContent: string): WriteAction => {
-  assertNotSymlink(filePath)
-
-  const existed = fs.existsSync(filePath)
-  const previous = existed ? fs.readFileSync(filePath, 'utf-8') : null
-
-  if (previous === nextContent) return 'unchanged'
-
-  if (existed) backupFile(filePath)
-
-  fs.mkdirSync(path.dirname(filePath), { recursive: true })
-  fs.writeFileSync(filePath, nextContent, 'utf-8')
-
-  return existed ? 'updated' : 'created'
-}
-
-const readOr = (filePath: string, fallback: string): string => {
-  return fs.existsSync(filePath) ? fs.readFileSync(filePath, 'utf-8') : fallback
-}
-
-const assertBlockPresent = (filePath: string, start: string, end: string): void => {
-  const content = fs.readFileSync(filePath, 'utf-8')
-
-  if (!hasManagedBlock(content, start, end)) {
-    throw new Error(`Post-write validation failed: managed block missing from ${filePath}`)
-  }
-}
-
-/**
- * Migrate a legacy `AGENTS.md` now that the guidance lives solely in `CLAUDE.md`.
- * Strips the infra-kit managed block, then asymmetrically:
- * - no infra-kit block at all (hand-authored, not ours) → leave untouched (`unchanged`),
- * - file was purely generated (nothing but whitespace remains) → back up + delete (`removed`),
- * - hand-authored content surrounds the block → back up + write the block-free remainder (`updated`).
- * Every destructive path leaves a timestamped `.backup.` first.
- */
-const migrateLegacyAgentsFile = (agentsPath: string): WriteAction => {
-  if (!fs.existsSync(agentsPath)) return 'unchanged'
-
-  assertNotSymlink(agentsPath)
-
-  const content = fs.readFileSync(agentsPath, 'utf-8')
-  const stripped = removeManagedBlock(content, AGENTS_MARKER_START, AGENTS_MARKER_END)
-
-  if (stripped === null) return 'unchanged'
-
-  if (stripped.trim() === '') {
-    backupFile(agentsPath)
-    fs.rmSync(agentsPath)
-
-    return 'removed'
-  }
-
-  return writeManaged(agentsPath, stripped)
-}
-
-/**
- * Generate (or refresh) the repo agent-instruction guidance, now hosted solely in
- * `CLAUDE.md` (preserving hand-authored content outside the markers). Also migrates
- * legacy setups — stripping the old `@AGENTS.md` import region from `CLAUDE.md` and
- * backing up/removing a now-redundant `AGENTS.md`. Repo-gated: a no-op outside an
- * infra-kit repo. Idempotent (within a CLI version) and non-destructive.
- *
- * @example
- * await writeAgentFiles()
- * // INFO: Agent-instruction files synced (infra-kit 0.1.105)
- */
-export const writeAgentFiles = async (): Promise<WriteAgentFilesResult> => {
-  let mainConfigPath: string
-
-  try {
-    mainConfigPath = (await getInfraKitConfigPaths()).main
-  } catch {
-    logger.info('Skipped agent-instruction files — not inside an infra-kit repo')
-
-    return { skipped: true, root: null, written: [] }
-  }
-
-  if (!fs.existsSync(mainConfigPath)) {
-    logger.info('Skipped agent-instruction files — no infra-kit.json at the repo root')
-
-    return { skipped: true, root: null, written: [] }
-  }
-
-  const root = path.dirname(mainConfigPath)
-  const version = packageJson.version
-  const body = buildAgentsBody(version)
-
-  const agentsPath = path.join(root, AGENTS_FILE)
-  const claudePath = path.join(root, CLAUDE_FILE)
-
-  // CLAUDE.md now hosts the full guidance body. Strip the legacy `@AGENTS.md`
-  // import region first (if present), then upsert the body under the begin/end
-  // markers — disjoint marker pairs, so the two operations never collide.
-  let claudeContent = readOr(claudePath, '')
-
-  claudeContent = removeManagedBlock(claudeContent, AGENTS_IMPORT_START, AGENTS_IMPORT_END) ?? claudeContent
-  claudeContent = upsertManagedBlock({
-    content: claudeContent,
-    body,
-    startMarker: AGENTS_MARKER_START,
-    endMarker: AGENTS_MARKER_END,
-    placement: 'replace-in-place',
-  })
-
-  // CLAUDE.md is the only file we manage. A legacy AGENTS.md is migrated away.
-  const written: AgentFileWrite[] = [
-    { path: claudePath, action: writeManaged(claudePath, claudeContent) },
-    { path: agentsPath, action: migrateLegacyAgentsFile(agentsPath) },
-  ]
-
-  assertBlockPresent(claudePath, AGENTS_MARKER_START, AGENTS_MARKER_END)
-
-  for (const file of written) {
-    // Always report CLAUDE.md; only mention a migrated file when it actually changed.
-    if (file.action === 'unchanged' && file.path !== claudePath) continue
-
-    logger.info(`  ${file.action.padEnd(9)} ${path.relative(root, file.path)}`)
-  }
-
-  logger.info(`Agent-instruction files synced (infra-kit ${version})`)
-
-  return { skipped: false, root, written }
-}

package/src/commands/init/index.ts

@@ -1,8 +0,0 @@
-export {
-  AGENTS_IMPORT_END,
-  AGENTS_IMPORT_START,
-  AGENTS_MARKER_END,
-  AGENTS_MARKER_START,
-  writeAgentFiles,
-} from './agent-files'
-export { init } from './init'