infra-kit 0.1.128 → 0.1.129

package/src/lib/git-utils/__tests__/git-primitives.test.ts

@@ -1,217 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-
-import {
-  deleteLocalBranch,
-  deleteRemoteBranch,
-  getCurrentBranch,
-  isInsideLinkedWorktree,
-  isWorkingTreeClean,
-} from 'src/lib/git-utils'
-
-// Controllable per-command git output. The mocked `$` dispatches on the command
-// text so a single test can make `--absolute-git-dir` and `--git-common-dir`
-// return DIFFERENT values — which the shared single-stdout mock cannot express.
-const git = vi.hoisted(() => {
-  return {
-    toplevel: '/repo',
-    branch: 'dev',
-    status: '',
-    absoluteGitDir: '/repo/.git',
-    commonDir: '.git',
-    branchList: '',
-    lsRemote: '',
-    lsRemoteThrows: false,
-    localDeleteThrows: false,
-    remoteDeleteThrows: false,
-    calls: [] as string[],
-  }
-})
-
-vi.mock('zx', () => {
-  const run = (strings: TemplateStringsArray): Promise<{ stdout: string }> => {
-    const command = strings.join('')
-
-    git.calls.push(command)
-
-    if (command.includes('--show-toplevel')) return Promise.resolve({ stdout: `${git.toplevel}\n` })
-    if (command.includes('--abbrev-ref HEAD')) return Promise.resolve({ stdout: `${git.branch}\n` })
-    if (command.includes('status --porcelain')) return Promise.resolve({ stdout: git.status })
-    if (command.includes('--absolute-git-dir')) return Promise.resolve({ stdout: `${git.absoluteGitDir}\n` })
-    if (command.includes('--git-common-dir')) return Promise.resolve({ stdout: `${git.commonDir}\n` })
-    if (command.includes('branch --list')) return Promise.resolve({ stdout: git.branchList })
-    if (command.includes('branch -D')) {
-      return git.localDeleteThrows
-        ? Promise.reject(new Error('branch checked out in another worktree'))
-        : Promise.resolve({ stdout: '' })
-    }
-    if (command.includes('ls-remote --heads')) {
-      return git.lsRemoteThrows
-        ? Promise.reject(new Error('could not read from remote repository'))
-        : Promise.resolve({ stdout: git.lsRemote })
-    }
-    if (command.includes('push origin --delete')) {
-      return git.remoteDeleteThrows
-        ? Promise.reject(new Error('remote ref does not exist'))
-        : Promise.resolve({ stdout: '' })
-    }
-
-    return Promise.resolve({ stdout: '' })
-  }
-
-  // zx's `$` supports both tagged-template form (`$`...``) and an options form
-  // (`$({ cwd })`...``) that returns a fresh tagged-template function.
-  const $ = vi.fn((first: unknown) => {
-    if (!Array.isArray(first)) {
-      return (strings: TemplateStringsArray) => {
-        return run(strings)
-      }
-    }
-
-    return run(first as unknown as TemplateStringsArray)
-  })
-
-  return { $ }
-})
-
-describe('getCurrentBranch', () => {
-  beforeEach(() => {
-    git.branch = 'dev'
-  })
-
-  it('returns the trimmed branch name', async () => {
-    git.branch = 'release/v1.2.3'
-
-    await expect(getCurrentBranch()).resolves.toBe('release/v1.2.3')
-  })
-})
-
-describe('isWorkingTreeClean', () => {
-  it('returns true when porcelain output is empty', async () => {
-    git.status = ''
-
-    await expect(isWorkingTreeClean()).resolves.toBe(true)
-  })
-
-  it('returns true when porcelain output is whitespace only', async () => {
-    git.status = '\n'
-
-    await expect(isWorkingTreeClean()).resolves.toBe(true)
-  })
-
-  it('returns false when there are changes', async () => {
-    git.status = ' M src/foo.ts\n?? new.ts\n'
-
-    await expect(isWorkingTreeClean()).resolves.toBe(false)
-  })
-})
-
-describe('isInsideLinkedWorktree', () => {
-  beforeEach(() => {
-    git.toplevel = '/repo'
-  })
-
-  it('returns false in the main checkout (git dir resolves to the common dir)', async () => {
-    git.absoluteGitDir = '/repo/.git'
-    git.commonDir = '.git'
-
-    await expect(isInsideLinkedWorktree()).resolves.toBe(false)
-  })
-
-  it('returns true in a linked worktree (git dir under .git/worktrees, differs from common dir)', async () => {
-    git.absoluteGitDir = '/repo/.git/worktrees/release-v1.2.3'
-    git.commonDir = '/repo/.git'
-
-    await expect(isInsideLinkedWorktree()).resolves.toBe(true)
-  })
-})
-
-const ranLocalDelete = (): boolean => {
-  return git.calls.some((c) => {
-    return c.includes('branch -D')
-  })
-}
-const ranRemoteDelete = (): boolean => {
-  return git.calls.some((c) => {
-    return c.includes('push origin --delete')
-  })
-}
-
-describe('deleteLocalBranch', () => {
-  beforeEach(() => {
-    git.branch = 'dev'
-    git.branchList = ''
-    git.localDeleteThrows = false
-    git.calls = []
-  })
-
-  it('force-deletes the branch when it exists and is not current', async () => {
-    git.branchList = '  release/v1.2.3\n'
-
-    await deleteLocalBranch('release/v1.2.3')
-
-    expect(ranLocalDelete()).toBe(true)
-  })
-
-  it('is a no-op when the branch does not exist locally', async () => {
-    git.branchList = ''
-
-    await deleteLocalBranch('release/v1.2.3')
-
-    expect(ranLocalDelete()).toBe(false)
-  })
-
-  it('is a no-op when the branch is the current checkout', async () => {
-    git.branch = 'release/v1.2.3'
-    git.branchList = '* release/v1.2.3\n'
-
-    await deleteLocalBranch('release/v1.2.3')
-
-    expect(ranLocalDelete()).toBe(false)
-  })
-
-  it('propagates when the delete itself fails (e.g. checked out in another worktree)', async () => {
-    git.branchList = '  release/v1.2.3\n'
-    git.localDeleteThrows = true
-
-    await expect(deleteLocalBranch('release/v1.2.3')).rejects.toThrow()
-  })
-})
-
-describe('deleteRemoteBranch', () => {
-  beforeEach(() => {
-    git.lsRemote = ''
-    git.lsRemoteThrows = false
-    git.remoteDeleteThrows = false
-    git.calls = []
-  })
-
-  it('deletes the remote branch when ls-remote finds it', async () => {
-    git.lsRemote = 'abc123\trefs/heads/release/v1.2.3\n'
-
-    await deleteRemoteBranch('release/v1.2.3')
-
-    expect(ranRemoteDelete()).toBe(true)
-  })
-
-  it('is a no-op when the branch does not exist on the remote', async () => {
-    git.lsRemote = ''
-
-    await deleteRemoteBranch('release/v1.2.3')
-
-    expect(ranRemoteDelete()).toBe(false)
-  })
-
-  it('propagates a network/auth failure instead of treating it as "branch absent"', async () => {
-    git.lsRemoteThrows = true
-
-    await expect(deleteRemoteBranch('release/v1.2.3')).rejects.toThrow()
-    expect(ranRemoteDelete()).toBe(false)
-  })
-
-  it('propagates when the remote delete itself fails', async () => {
-    git.lsRemote = 'abc123\trefs/heads/release/v1.2.3\n'
-    git.remoteDeleteThrows = true
-
-    await expect(deleteRemoteBranch('release/v1.2.3')).rejects.toThrow()
-  })
-})

package/src/lib/git-utils/__tests__/git-utils.test.ts

@@ -1,49 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-
-import { getCurrentWorktrees } from 'src/lib/git-utils'
-
-const worktreeList = vi.hoisted(() => {
-  return { stdout: '' }
-})
-
-vi.mock('zx', () => {
-  return {
-    $: vi.fn(() => {
-      return Promise.resolve(worktreeList)
-    }),
-  }
-})
-
-const asWorktreeLine = (branch: string): string => {
-  return `/repos/project-worktrees/${branch}  abc1234 [${branch}]`
-}
-
-describe('getCurrentWorktrees', () => {
-  beforeEach(() => {
-    worktreeList.stdout = [
-      asWorktreeLine('main'),
-      asWorktreeLine('release/v1.18.22'),
-      asWorktreeLine('release/checkout-redesign'),
-      asWorktreeLine('release/Bad_Name'),
-      asWorktreeLine('feature/login-page'),
-      '/repos/project  abc1234 (bare)',
-      '',
-    ].join('\n')
-  })
-
-  it('returns versioned AND named release worktrees for type release', async () => {
-    await expect(getCurrentWorktrees('release')).resolves.toEqual(['release/v1.18.22', 'release/checkout-redesign'])
-  })
-
-  it('excludes junk release branches and non-release branches', async () => {
-    const branches = await getCurrentWorktrees('release')
-
-    expect(branches).not.toContain('release/Bad_Name')
-    expect(branches).not.toContain('feature/login-page')
-    expect(branches).not.toContain('main')
-  })
-
-  it('returns feature worktrees for type feature', async () => {
-    await expect(getCurrentWorktrees('feature')).resolves.toEqual(['feature/login-page'])
-  })
-})

package/src/lib/git-utils/git-utils.ts

@@ -1,179 +0,0 @@
-import path from 'node:path'
-import { $ } from 'zx'
-
-import { isReleaseBranch } from 'src/lib/release-id'
-
-/**
- * Get current git worktrees
- *
- * @returns [release/v1.18.22, release/v1.18.23, release/v1.18.24] or [feature/mobile-app, feature/explore-page, feature/login-page]
- */
-export const getCurrentWorktrees = async (type: 'release' | 'feature'): Promise<string[]> => {
-  const worktreesOutput = await $`git worktree list`
-
-  const worktreeLines = worktreesOutput.stdout.split('\n').filter(Boolean)
-
-  const worktreePredicateMap = {
-    release: releaseWorktreePredicate,
-    feature: featureWorktreePredicate,
-  }
-
-  return worktreeLines.map(worktreePredicateMap[type]).filter((branch) => {
-    return branch !== null
-  })
-}
-
-/**
- * Extract the branch name from a `git worktree list` output line.
- *
- * `git worktree list` formats each line as:
- *   <path>  <hash> [<branch>]
- *
- * Reads the branch from the trailing `[branch]` token so it works for the
- * main checkout too (whose path does not encode the branch name).
- */
-const parseWorktreeBranch = (line: string): string | null => {
-  const trimmed = line.trimEnd()
-
-  if (!trimmed.endsWith(']')) return null
-
-  const open = trimmed.lastIndexOf('[')
-
-  if (open === -1) return null
-
-  const branch = trimmed.slice(open + 1, -1)
-
-  return branch.length > 0 ? branch : null
-}
-
-/**
- * Extract a release branch name from a `git worktree list` output line.
- *
- * Returns `null` for lines that are not release worktrees.
- *
- * @example
- * releaseWorktreePredicate('/path/to/release/v1.18.22  abc1234 [release/v1.18.22]')
- * // => 'release/v1.18.22'
- *
- * @example
- * releaseWorktreePredicate('/path/to/feature/login  abc1234 [feature/login]')
- * // => null
- */
-const releaseWorktreePredicate = (line: string): string | null => {
-  const branch = parseWorktreeBranch(line)
-
-  return isReleaseBranch(branch) ? branch : null
-}
-
-/**
- * Extract a feature branch name from a `git worktree list` output line.
- *
- * Returns `null` for lines that are not feature worktrees.
- *
- * @example
- * featureWorktreePredicate('/path/to/feature/login-page  abc1234 [feature/login-page]')
- * // => 'feature/login-page'
- *
- * @example
- * featureWorktreePredicate('/path/to/release/v1.18.22  abc1234 [release/v1.18.22]')
- * // => null
- */
-const featureWorktreePredicate = (line: string): string | null => {
-  const branch = parseWorktreeBranch(line)
-
-  return branch?.startsWith('feature/') ? branch : null
-}
-
-/**
- * Get the current project root directory
- */
-export const getProjectRoot = async (): Promise<string> => {
-  const result = await $`git rev-parse --show-toplevel`
-
-  return result.stdout.trim()
-}
-
-/**
- * Get the current git branch name (e.g. `dev`, `main`, `release/v1.2.3`).
- */
-export const getCurrentBranch = async (): Promise<string> => {
-  const result = await $`git rev-parse --abbrev-ref HEAD`
-
-  return result.stdout.trim()
-}
-
-/**
- * Whether the working tree has no staged, unstaged, or untracked changes.
- */
-export const isWorkingTreeClean = async (): Promise<boolean> => {
-  const result = await $`git status --porcelain`
-
-  return result.stdout.trim().length === 0
-}
-
-/**
- * Whether the current checkout is a linked git worktree rather than the main
- * repository checkout.
- *
- * A linked worktree's git dir lives under `<main>/.git/worktrees/<name>`, so it
- * differs from the shared common dir; in the main checkout the two resolve to
- * the same path. Both are anchored to the toplevel so `--git-common-dir` (which
- * git may report relative to cwd) resolves consistently.
- */
-export const isInsideLinkedWorktree = async (): Promise<boolean> => {
-  const cwd = await getProjectRoot()
-
-  const [gitDirResult, commonDirResult] = await Promise.all([
-    $({ cwd })`git rev-parse --absolute-git-dir`,
-    $({ cwd })`git rev-parse --git-common-dir`,
-  ])
-
-  const gitDir = gitDirResult.stdout.trim()
-  const commonDir = path.resolve(cwd, commonDirResult.stdout.trim())
-
-  return gitDir !== commonDir
-}
-
-/**
- * Get the current repository name (basename of the project root)
- */
-export const getRepoName = async (): Promise<string> => {
-  const projectRoot = await getProjectRoot()
-
-  return path.basename(projectRoot)
-}
-
-/**
- * Delete a local branch if it exists and is not the current checkout.
- *
- * Idempotent: a no-op when the branch is absent (`git branch --list` prints
- * nothing). Uses force `-D` because a delivered release branch was
- * squash-merged — its tip is unreachable from the base, so `-d` would refuse
- * with "not fully merged". The delete itself still rejects if the branch is
- * checked out in another worktree; callers decide how to handle that.
- */
-export const deleteLocalBranch = async (branch: string): Promise<void> => {
-  const listed = await $`git branch --list ${branch}`
-
-  if (listed.stdout.trim().length === 0) return
-
-  if ((await getCurrentBranch()) === branch) return
-
-  await $`git branch -D ${branch}`
-}
-
-/**
- * Delete a branch on the `origin` remote if it exists.
- *
- * Idempotent: a no-op when the branch is absent on the remote. Existence is
- * probed with `git ls-remote --heads` (empty stdout = absent) rather than
- * `--exit-code`, so a genuine network/auth failure rejects and propagates to
- * the caller instead of being silently misread as "branch absent".
- */
-export const deleteRemoteBranch = async (branch: string): Promise<void> => {
-  const refs = await $`git ls-remote --heads origin ${branch}`
-
-  if (refs.stdout.trim().length === 0) return
-
-  await $`git push origin --delete ${branch}`
-}

package/src/lib/git-utils/index.ts

@@ -1,10 +0,0 @@
-export {
-  deleteLocalBranch,
-  deleteRemoteBranch,
-  getCurrentBranch,
-  getCurrentWorktrees,
-  getProjectRoot,
-  getRepoName,
-  isInsideLinkedWorktree,
-  isWorkingTreeClean,
-} from './git-utils'

package/src/lib/infra-kit-config/.omc/state/sessions/8dd0ad2f-3a90-4158-b9de-ec8e19a7c983/pre-tool-advisory-throttle.json

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

package/src/lib/infra-kit-config/__tests__/env-autoload-schema.test.ts

@@ -1,71 +0,0 @@
-import { describe, expect, it } from 'vitest'
-
-import { infraKitConfigSchema, infraKitOverrideConfigSchema } from '../infra-kit-config'
-
-const baseConfig = {
-  environments: ['dev', 'prod'],
-  envManagement: { provider: 'doppler', config: { name: 'my-project' } },
-}
-
-describe('envAutoLoad schema', () => {
-  it('is optional — a config without it parses', () => {
-    const result = infraKitConfigSchema.safeParse(baseConfig)
-
-    expect(result.success).toBe(true)
-    expect(result.success && result.data.envAutoLoad).toBeUndefined()
-  })
-
-  it.each(['shell-startup', 'cli-invocation'] as const)('accepts trigger "%s"', (trigger) => {
-    const result = infraKitConfigSchema.safeParse({ ...baseConfig, envAutoLoad: { trigger, config: 'dev' } })
-
-    expect(result.success).toBe(true)
-    expect(result.success && result.data.envAutoLoad).toEqual({ trigger, config: 'dev' })
-  })
-
-  it('rejects an unknown trigger value (e.g. "both")', () => {
-    const result = infraKitConfigSchema.safeParse({ ...baseConfig, envAutoLoad: { trigger: 'both', config: 'dev' } })
-
-    expect(result.success).toBe(false)
-  })
-
-  it('rejects a missing trigger', () => {
-    const result = infraKitConfigSchema.safeParse({ ...baseConfig, envAutoLoad: { config: 'dev' } })
-
-    expect(result.success).toBe(false)
-  })
-
-  it('rejects an empty config name', () => {
-    const result = infraKitConfigSchema.safeParse({
-      ...baseConfig,
-      envAutoLoad: { trigger: 'shell-startup', config: '' },
-    })
-
-    expect(result.success).toBe(false)
-  })
-
-  it('rejects unknown keys (strict object)', () => {
-    const result = infraKitConfigSchema.safeParse({
-      ...baseConfig,
-      envAutoLoad: { trigger: 'shell-startup', config: 'dev', extra: true },
-    })
-
-    expect(result.success).toBe(false)
-  })
-
-  // Important: an env name not in `environments` must NOT fail the schema — it is
-  // validated (and disabled) at resolve time so a typo never bricks every command.
-  it('does NOT reject a config name absent from environments (resolve-time concern)', () => {
-    const result = infraKitConfigSchema.safeParse({
-      ...baseConfig,
-      envAutoLoad: { trigger: 'shell-startup', config: 'not-listed' },
-    })
-
-    expect(result.success).toBe(true)
-  })
-
-  it('is accepted as a partial override layer', () => {
-    const result = infraKitOverrideConfigSchema.safeParse({ envAutoLoad: { trigger: 'cli-invocation', config: 'dev' } })
-
-    expect(result.success).toBe(true)
-  })
-})