infra-kit 0.1.127 → 0.1.129

package/src/lib/managed-block/__tests__/managed-block.test.ts

@@ -1,121 +0,0 @@
-import { describe, expect, it } from 'vitest'
-
-import { extractVersion, hasManagedBlock, removeManagedBlock, upsertManagedBlock } from '../managed-block'
-
-const START = '<!-- ik:begin -->'
-const END = '<!-- ik:end -->'
-
-describe('hasManagedBlock', () => {
-  it('detects a well-formed block', () => {
-    expect(hasManagedBlock(`x ${START} y ${END} z`, START, END)).toBe(true)
-  })
-
-  it('returns false when a marker is missing', () => {
-    expect(hasManagedBlock(`only ${START} here`, START, END)).toBe(false)
-  })
-
-  it('treats reversed markers (end before start) as absent', () => {
-    expect(hasManagedBlock(`${END} middle ${START}`, START, END)).toBe(false)
-  })
-})
-
-describe('removeManagedBlock', () => {
-  it('removes the block and preserves surrounding text', () => {
-    const content = `top\n${START}\nmid\n${END}\nbot`
-
-    expect(removeManagedBlock(content, START, END)).toBe('top\nbot')
-  })
-
-  it('returns null when no block is present', () => {
-    expect(removeManagedBlock('nothing here', START, END)).toBeNull()
-  })
-
-  it('returns null for reversed markers (guard)', () => {
-    expect(removeManagedBlock(`${END}\nx\n${START}`, START, END)).toBeNull()
-  })
-})
-
-describe('extractVersion', () => {
-  const PREFIX = '<!-- ik:version '
-
-  it('reads the version token after the prefix', () => {
-    expect(extractVersion('<!-- ik:version 0.1.105 -->', PREFIX)).toBe('0.1.105')
-  })
-
-  it('returns null when the prefix is absent', () => {
-    expect(extractVersion('no version here', PREFIX)).toBeNull()
-  })
-})
-
-describe('upsertManagedBlock', () => {
-  it('inserts into an empty file (replace-in-place default)', () => {
-    const result = upsertManagedBlock({ content: '', body: 'hello', startMarker: START, endMarker: END })
-
-    expect(result).toBe(`${START}\nhello\n${END}\n`)
-  })
-
-  it('inserts into an empty file (append-end)', () => {
-    const result = upsertManagedBlock({
-      content: '',
-      body: 'hello',
-      startMarker: START,
-      endMarker: END,
-      placement: 'append-end',
-    })
-
-    expect(result).toBe(`${START}\nhello\n${END}\n`)
-  })
-
-  it('replaces an existing block in place, keeping surrounding content verbatim', () => {
-    const content = `# top heading\n\n${START}\nold body\n${END}\n\n# bottom heading\n`
-    const result = upsertManagedBlock({ content, body: 'new body', startMarker: START, endMarker: END })
-
-    expect(result).toBe(`# top heading\n\n${START}\nnew body\n${END}\n\n# bottom heading\n`)
-  })
-
-  it('append-end relocates an existing mid-file block to end-of-file', () => {
-    const content = `intro\n\n${START}\nold\n${END}\n\nuser tail`
-    const result = upsertManagedBlock({
-      content,
-      body: 'fresh',
-      startMarker: START,
-      endMarker: END,
-      placement: 'append-end',
-    })
-
-    expect(result).toBe(`intro\nuser tail\n${START}\nfresh\n${END}\n`)
-  })
-
-  it('append-end lands the block at end-of-file when absent', () => {
-    const content = 'existing user content'
-    const result = upsertManagedBlock({
-      content,
-      body: 'b',
-      startMarker: START,
-      endMarker: END,
-      placement: 'append-end',
-    })
-
-    expect(result.endsWith(`${START}\nb\n${END}\n`)).toBe(true)
-    expect(result.startsWith('existing user content')).toBe(true)
-  })
-
-  it('is idempotent — re-running does not nest or duplicate blocks', () => {
-    const once = upsertManagedBlock({ content: 'pre\n', body: 'b', startMarker: START, endMarker: END })
-    const twice = upsertManagedBlock({ content: once, body: 'b', startMarker: START, endMarker: END })
-
-    expect(twice).toBe(once)
-    expect(twice.match(new RegExp(START, 'g'))?.length).toBe(1)
-  })
-
-  it('preserves content above and below across an update', () => {
-    const first = upsertManagedBlock({ content: 'ABOVE\n', body: 'v1', startMarker: START, endMarker: END })
-    const withTail = `${first}BELOW\n`
-    const updated = upsertManagedBlock({ content: withTail, body: 'v2', startMarker: START, endMarker: END })
-
-    expect(updated).toContain('ABOVE')
-    expect(updated).toContain('BELOW')
-    expect(updated).toContain('v2')
-    expect(updated).not.toContain('v1')
-  })
-})

package/src/lib/managed-block/index.ts

@@ -1,8 +0,0 @@
-export {
-  buildManagedBlock,
-  extractVersion,
-  hasManagedBlock,
-  removeManagedBlock,
-  upsertManagedBlock,
-} from './managed-block'
-export type { BlockPlacement, UpsertManagedBlockArgs } from './managed-block'

package/src/lib/managed-block/managed-block.ts

@@ -1,145 +0,0 @@
-/**
- * Generic "managed block" helpers — a marker-delimited region a tool owns and
- * rewrites idempotently while never touching content outside the markers.
- *
- * This is the same mechanism `infra-kit init` already uses for the `~/.zshrc`
- * shell block (`# -- infra-kit:begin -- … # -- infra-kit:end --`), lifted here so
- * it can be reused for the repo agent-instruction guidance block in `CLAUDE.md`.
- * It mirrors the design of OMC's
- * `<!-- OMC:START --> … <!-- OMC:END -->` CLAUDE.md installer.
- */
-
-/**
- * Where a freshly upserted block lands relative to existing content.
- *
- * - `replace-in-place`: if the block already exists, rewrite it where it sits
- *   (surrounding text untouched, verbatim); if absent, append it at end-of-file.
- * - `append-end`: strip any existing block from wherever it is, then append the
- *   fresh block at end-of-file (i.e. always relocate to the end). This preserves
- *   the historical `~/.zshrc` behavior of `removeExistingBlock` + append.
- */
-export type BlockPlacement = 'replace-in-place' | 'append-end'
-
-export interface UpsertManagedBlockArgs {
-  /** Existing file content (`''` for a new file). */
-  content: string
-  /** Inner body to place between the markers (WITHOUT the markers). */
-  body: string
-  startMarker: string
-  endMarker: string
-  /** Defaults to `replace-in-place`. */
-  placement?: BlockPlacement
-}
-
-/**
- * Whether `content` contains a well-formed `start … end` block. A reversed pair
- * (end before start) is treated as absent — the same guard `doctor.ts` applies
- * to the zshrc block, ported here so corrupted markers never match.
- *
- * @example
- * hasManagedBlock('a<!--s-->x<!--e-->b', '<!--s-->', '<!--e-->') // => true
- * hasManagedBlock('<!--e--><!--s-->', '<!--s-->', '<!--e-->')    // => false (reversed)
- */
-export const hasManagedBlock = (content: string, startMarker: string, endMarker: string): boolean => {
-  const startIdx = content.indexOf(startMarker)
-  const endIdx = content.indexOf(endMarker)
-
-  return startIdx !== -1 && endIdx !== -1 && endIdx >= startIdx
-}
-
-/**
- * Remove the first complete `start … end` block, preserving surrounding text.
- * Returns `null` when no well-formed block is present (no markers, or reversed),
- * so callers can fall through to legacy-format handling — this matches the
- * `string | null` contract of `init.ts`'s original `removeBetween`, with the
- * added reversed-marker guard.
- *
- * @example
- * removeManagedBlock('top\n<!--s-->mid<!--e-->\nbot', '<!--s-->', '<!--e-->')
- * // => 'top\nbot'
- */
-export const removeManagedBlock = (content: string, startMarker: string, endMarker: string): string | null => {
-  const startIdx = content.indexOf(startMarker)
-  const endIdx = content.indexOf(endMarker)
-
-  if (startIdx === -1 || endIdx === -1 || endIdx < startIdx) return null
-
-  // eslint-disable-next-line sonarjs/super-linear-regex
-  const before = content.slice(0, startIdx).replace(/\n+$/, '')
-  const after = content.slice(endIdx + endMarker.length).replace(/^\n+/, '')
-
-  return before + (after ? `\n${after}` : '')
-}
-
-/**
- * Read the version token that follows `versionPrefix` (e.g.
- * `'<!-- infra-kit:version '`). Returns the token up to the next whitespace or
- * `>`, or `null` if the prefix is absent. Mirrors OMC's `OMC:VERSION:` line.
- *
- * @example
- * extractVersion('<!-- infra-kit:version 0.1.105 -->', '<!-- infra-kit:version ')
- * // => '0.1.105'
- */
-export const extractVersion = (content: string, versionPrefix: string): string | null => {
-  const idx = content.indexOf(versionPrefix)
-
-  if (idx === -1) return null
-
-  const rest = content.slice(idx + versionPrefix.length)
-  const match = rest.match(/^([^\s>]+)/)
-
-  return match ? match[1]! : null
-}
-
-/**
- * Compose a full block string: `start\n{body}\nend`. Kept identical in shape to
- * the historical `buildShellBlock()` output so doctor's exact-match comparison
- * stays valid.
- */
-export const buildManagedBlock = (startMarker: string, body: string, endMarker: string): string => {
-  return `${startMarker}\n${body}\n${endMarker}`
-}
-
-/**
- * Insert or update a managed block in `content`, preserving everything outside
- * the markers. Idempotent: re-running with the same body yields the same block
- * and never nests duplicates.
- *
- * @example
- * // fresh file
- * upsertManagedBlock({ content: '', body: 'hi', startMarker: '<!--s-->', endMarker: '<!--e-->' })
- * // => '<!--s-->\nhi\n<!--e-->\n'
- *
- * @example
- * // existing block replaced in place, surrounding text kept
- * upsertManagedBlock({
- *   content: 'top\n<!--s-->\nold\n<!--e-->\nbot',
- *   body: 'new', startMarker: '<!--s-->', endMarker: '<!--e-->',
- * })
- * // => 'top\n<!--s-->\nnew\n<!--e-->\nbot'
- */
-export const upsertManagedBlock = ({
-  content,
-  body,
-  startMarker,
-  endMarker,
-  placement = 'replace-in-place',
-}: UpsertManagedBlockArgs): string => {
-  const block = buildManagedBlock(startMarker, body, endMarker)
-  const present = hasManagedBlock(content, startMarker, endMarker)
-
-  if (placement === 'replace-in-place' && present) {
-    const startIdx = content.indexOf(startMarker)
-    const endIdx = content.indexOf(endMarker) + endMarker.length
-
-    return content.slice(0, startIdx) + block + content.slice(endIdx)
-  }
-
-  // append-end, or replace-in-place with no existing block: drop any stale block
-  // then append the fresh one at end-of-file.
-  const stripped = present ? (removeManagedBlock(content, startMarker, endMarker) ?? content) : content
-  // eslint-disable-next-line sonarjs/super-linear-regex
-  const base = stripped.replace(/\n+$/, '')
-
-  return base.length > 0 ? `${base}\n${block}\n` : `${block}\n`
-}

package/src/lib/package-config/.omc/state/sessions/dde22a92-a0e9-4948-905d-25c9a812ab35/last-tool-error-state.json

@@ -1,7 +0,0 @@
-{
-  "tool_name": "Bash",
-  "tool_input_preview": "{\"command\":\"rtk ls -1 infra-kit.config.ts apps/*/*/infra-kit.config.* packages/*/infra-kit.config.* vendor/packages/*/infra-kit.config.* vendor/configs/*/infra-kit.config.* 2>/dev/null\",\"description\":...",
-  "error": "Exit code 1\n(eval):1: no matches found: apps/*/*/infra-kit.config.*",
-  "timestamp": "2026-06-20T09:23:27.849Z",
-  "retry_count": 1
-}

package/src/lib/package-config/.omc/state/sessions/dde22a92-a0e9-4948-905d-25c9a812ab35/pre-tool-advisory-throttle.json

@@ -1,10 +0,0 @@
-{
-  "version": 1,
-  "entries": {
-    "79a93d4a2f8f50b95f852280616242fee1855dc99a3c75211917f55e72e95fae": {
-      "last_emitted_at_ms": 1781947394826,
-      "message": "Use parallel execution for independent tasks. Use run_in_background for long operations (npm install, builds, tests)."
-    }
-  },
-  "updated_at": "2026-06-20T09:23:14.826Z"
-}

package/src/lib/package-config/__tests__/package-config.test.ts

@@ -1,95 +0,0 @@
-import { describe, expect, it } from 'vitest'
-
-import { DEFAULT_RULES, ROOT_DEFAULT_RULES, defineConfig, resolvePackageConfig } from '../package-config'
-import { packageConfigSchema } from '../package-config-schema'
-
-describe('defineConfig', () => {
-  it('returns an object input unchanged (identity)', () => {
-    const input = { requiredScripts: ['build'] }
-
-    expect(defineConfig(input)).toBe(input)
-  })
-
-  it('returns a factory input unchanged so the loader can resolve it', () => {
-    const factory = () => {
-      return { requiredFiles: ['a.txt'] }
-    }
-
-    expect(defineConfig(factory)).toBe(factory)
-  })
-})
-
-describe('resolvePackageConfig', () => {
-  it('falls back to defaults for every unset key', () => {
-    const rules = resolvePackageConfig({})
-
-    expect(rules.requiredScripts).toEqual(DEFAULT_RULES.requiredScripts)
-    expect(rules.requiredFiles).toEqual(DEFAULT_RULES.requiredFiles)
-  })
-
-  it('replaces a key wholesale when provided, including an empty array opt-out', () => {
-    const rules = resolvePackageConfig({ requiredScripts: [], requiredFiles: ['serverless.common.yml'] })
-
-    expect(rules.requiredScripts).toEqual([])
-    expect(rules.requiredFiles).toEqual(['serverless.common.yml'])
-  })
-
-  it('does not share the default array reference with the resolved result', () => {
-    const rules = resolvePackageConfig({})
-
-    rules.requiredScripts.push('mutated')
-
-    expect(DEFAULT_RULES.requiredScripts).not.toContain('mutated')
-  })
-
-  it('falls back to the supplied baseline (root) for unset keys, including turbo tasks', () => {
-    const rules = resolvePackageConfig({}, ROOT_DEFAULT_RULES)
-
-    expect(rules.requiredScripts).toEqual(ROOT_DEFAULT_RULES.requiredScripts)
-    expect(rules.turboTasks).toEqual(ROOT_DEFAULT_RULES.turboTasks)
-  })
-
-  it('lets a config override turbo.requiredTasks', () => {
-    const rules = resolvePackageConfig({ turbo: { requiredTasks: ['build'] } }, ROOT_DEFAULT_RULES)
-
-    expect(rules.turboTasks).toEqual(['build'])
-  })
-
-  it('defaults turboTasks to an empty array for packages', () => {
-    const rules = resolvePackageConfig({})
-
-    expect(rules.turboTasks).toEqual([])
-  })
-})
-
-describe('packageConfigSchema', () => {
-  it('rejects unknown keys so config typos surface as errors', () => {
-    const result = packageConfigSchema.safeParse({ requiredScript: ['build'] })
-
-    expect(result.success).toBe(false)
-  })
-
-  it('rejects a non-array requiredScripts', () => {
-    const result = packageConfigSchema.safeParse({ requiredScripts: 'build' })
-
-    expect(result.success).toBe(false)
-  })
-
-  it('accepts a well-formed config', () => {
-    const result = packageConfigSchema.safeParse({ requiredScripts: ['build'], requiredFiles: ['tsconfig.json'] })
-
-    expect(result.success).toBe(true)
-  })
-
-  it('accepts a turbo.requiredTasks block', () => {
-    const result = packageConfigSchema.safeParse({ turbo: { requiredTasks: ['build', 'validate'] } })
-
-    expect(result.success).toBe(true)
-  })
-
-  it('rejects an unknown key inside turbo', () => {
-    const result = packageConfigSchema.safeParse({ turbo: { tasks: ['build'] } })
-
-    expect(result.success).toBe(false)
-  })
-})

package/src/lib/package-config/index.ts

@@ -1,3 +0,0 @@
-export { DEFAULT_RULES, defineConfig, resolvePackageConfig, ROOT_DEFAULT_RULES } from './package-config'
-export type { InfraKitPackageConfig, InfraKitPackageConfigInput, ResolvedPackageRules } from './package-config'
-export { packageConfigSchema } from './package-config-schema'

package/src/lib/package-config/package-config-schema.ts

@@ -1,19 +0,0 @@
-import { z } from 'zod'
-
-/**
- * Schema for the resolved (post-factory) package config object. `strictObject`
- * rejects unknown keys so typos in `infra-kit.config.ts` surface as validation
- * errors instead of being silently ignored.
- *
- * Kept in its own module — separate from the public `defineConfig`/types entry —
- * so the published `infra-kit` type surface stays free of a `zod` import.
- */
-export const packageConfigSchema = z.strictObject({
-  requiredScripts: z.array(z.string().min(1)).optional(),
-  requiredFiles: z.array(z.string().min(1)).optional(),
-  turbo: z
-    .strictObject({
-      requiredTasks: z.array(z.string().min(1)).optional(),
-    })
-    .optional(),
-})

package/src/lib/package-config/package-config.ts

@@ -1,97 +0,0 @@
-/**
- * Validation rules for a single workspace package, declared in its
- * `infra-kit.config.js`. Every field is optional: a key left unset falls back to
- * the active baseline ({@link DEFAULT_RULES} for packages, {@link ROOT_DEFAULT_RULES}
- * for the monorepo root), and a key set replaces that default wholesale (per-key,
- * no array concatenation) so a package can opt out with an explicit empty array.
- *
- * Most packages need none of these — the standard rules live in the baseline, so
- * a typical config is just `defineConfig(() => ({}))`.
- *
- * @example
- * // infra-kit.config.js
- * import { defineConfig } from 'infra-kit'
- *
- * export default defineConfig(() => ({}))
- */
-export interface InfraKitPackageConfig {
-  /** Scripts that must be present in the package's package.json `scripts` map. */
-  requiredScripts?: string[]
-  /** Files (relative to the package root) that must exist on disk. */
-  requiredFiles?: string[]
-  /** Turborepo expectations — only meaningful where a turbo.json lives (the root). */
-  turbo?: {
-    /** Tasks that must be defined in turbo.json `tasks`. */
-    requiredTasks?: string[]
-  }
-}
-
-/**
- * Accepted shapes for a package config's default export — mirrors Vite's
- * `defineConfig` input: a plain object, a sync factory, or an async factory.
- */
-export type InfraKitPackageConfigInput =
-  InfraKitPackageConfig | (() => InfraKitPackageConfig) | (() => Promise<InfraKitPackageConfig>)
-
-/**
- * Identity helper that gives `infra-kit.config.js` authors full type inference
- * and editor autocomplete without changing the value — exactly like Vite's
- * `defineConfig`. Resolution of the factory form happens in the loader, not here.
- *
- * @example
- * export default defineConfig(() => ({}))
- *
- * @example
- * export default defineConfig(() => ({ requiredScripts: [] }))
- */
-export const defineConfig = (config: InfraKitPackageConfigInput): InfraKitPackageConfigInput => {
-  return config
-}
-
-/** Fully-resolved rules with every defaultable field present. */
-export interface ResolvedPackageRules {
-  requiredScripts: string[]
-  requiredFiles: string[]
-  turboTasks: string[]
-}
-
-/**
- * Baseline rules for a standard TypeScript workspace package, applied to any key
- * a package leaves unset. These are the "under the hood" defaults so a conforming
- * package's config can stay empty; non-standard packages override the relevant key.
- */
-export const DEFAULT_RULES: Readonly<ResolvedPackageRules> = {
-  requiredScripts: ['build', 'ts-check', 'eslint-check', 'prettier-check', 'test'],
-  requiredFiles: ['tsconfig.json', 'eslint.config.js', 'readme.md'],
-  turboTasks: [],
-}
-
-/**
- * Baseline rules for the monorepo root (`infra-kit audit --root`). Checks the
- * root commands, the workspace/turbo files, and that the turbo pipeline defines
- * the expected tasks — so the root's own config can also stay empty.
- */
-export const ROOT_DEFAULT_RULES: Readonly<ResolvedPackageRules> = {
-  requiredScripts: ['build', 'dev', 'test', 'qa', 'check', 'fix'],
-  requiredFiles: ['turbo.json', 'pnpm-workspace.yaml'],
-  turboTasks: ['build', 'test', 'ts-check', 'eslint-check', 'prettier-check', 'check'],
-}
-
-/**
- * Merge a parsed package config over a baseline. Each key is replaced wholesale
- * when the package provides it, otherwise the baseline value is used.
- *
- * @example
- * resolvePackageConfig({ requiredScripts: [] })
- * // => { requiredScripts: [], requiredFiles: [...DEFAULT_RULES.requiredFiles], turboTasks: [] }
- */
-export const resolvePackageConfig = (
-  config: InfraKitPackageConfig,
-  baseline: Readonly<ResolvedPackageRules> = DEFAULT_RULES,
-): ResolvedPackageRules => {
-  return {
-    requiredScripts: config.requiredScripts ?? [...baseline.requiredScripts],
-    requiredFiles: config.requiredFiles ?? [...baseline.requiredFiles],
-    turboTasks: config.turbo?.requiredTasks ?? [...baseline.turboTasks],
-  }
-}