infra-kit 0.1.114 → 0.1.115

package/eslint.config.js

@@ -1,3 +1,35 @@
 import config from '@wl/eslint-config'
 
-export default config()
+/**
+ * No-React boundary: the entry/MCP/command code must never STATICALLY import the
+ * Ink TUI (or ink/react). The TUI is reached only via dynamic `await import()`
+ * from the TTY branch, so React stays off the MCP / `--json` / non-TTY paths.
+ * `no-restricted-imports` only flags static imports — dynamic import() is allowed.
+ */
+const noTuiOnMachinePaths = {
+  files: ['src/entry/cli.ts', 'src/entry/mcp.ts', 'src/commands/**/*.ts', 'src/commands/**/*.tsx'],
+  rules: {
+    'no-restricted-imports': [
+      'error',
+      {
+        paths: [
+          { name: 'ink', message: 'Do not import ink outside src/tui/* — load the TUI via dynamic import().' },
+          { name: 'react', message: 'Do not import react outside src/tui/* — load the TUI via dynamic import().' },
+        ],
+        patterns: [
+          {
+            group: ['src/tui', 'src/tui/*', '**/tui', '**/tui/*'],
+            message:
+              'Do not statically import the Ink TUI from CLI/MCP/command code; use a dynamic import() in the TTY branch.',
+          },
+        ],
+      },
+    ],
+  },
+}
+
+// `config()` returns a Promise resolving to the flat-config array; ESLint awaits
+// the default export, so resolve it and append our local boundary rule.
+export default config().then((base) => {
+  return [...base, noTuiOnMachinePaths]
+})

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "infra-kit",
   "type": "module",
-  "version": "0.1.114",
+  "version": "0.1.115",
   "description": "infra-kit",
   "main": "dist/index.js",
   "module": "dist/index.js",
@@ -42,16 +42,20 @@
     "@inquirer/select": "^5.2.1",
     "@modelcontextprotocol/sdk": "^1.29.0",
     "commander": "^15.0.0",
+    "ink": "^7.1.0",
     "pino": "^10.3.1",
     "pino-pretty": "^13.1.3",
+    "react": "catalog:",
     "yaml": "^2.9.0",
     "zod": "^4.4.3",
     "zx": "^8.8.5"
   },
   "devDependencies": {
+    "@types/react": "catalog:",
     "@wl/eslint-config": "workspace:*",
     "@wl/vitest-config": "workspace:*",
     "esbuild": "^0.28.1",
+    "ink-testing-library": "^4.0.0",
     "typescript": "^6.0.3"
   }
 }

package/scripts/build.js

@@ -28,7 +28,17 @@ await esbuild.build({
   outdir: OUT_DIR,
   sourcemap: true,
   minify: true,
-  external: Object.keys(packageJson.dependencies),
+  // `splitting` is required so the dynamically-imported Ink TUI (src/tui/*) lands
+  // in a separate lazy chunk. Without it, the auto-injected `react/jsx-runtime`
+  // import would be hoisted into the eager cli.js/mcp.js bundle and load React on
+  // every invocation — breaking the "no React on machine paths" guarantee.
+  splitting: true,
+  // Automatic JSX runtime so .tsx needs no `import React`. Pairs with
+  // tsconfig `"jsx": "react-jsx"`.
+  jsx: 'automatic',
+  // Externalize deps + the React JSX runtime subpaths (the `react` key alone
+  // does not cover subpaths in esbuild's external matching).
+  external: [...Object.keys(packageJson.dependencies), 'react/jsx-runtime', 'react/jsx-dev-runtime'],
 })
 
 for (const entryPoint of entryPoints) {

package/src/.omc/state/agent-replay-c513ef38-cc3e-4a29-9c4c-1e0137245dd5.jsonl

@@ -0,0 +1,4 @@
+{"t":0,"agent":"a6f037b","agent_type":"architect","event":"agent_start","parent_mode":"none"}
+{"t":0,"agent":"a6f037b","agent_type":"architect","event":"agent_stop","success":true,"duration_ms":132042}
+{"t":0,"agent":"ac9a1b4","agent_type":"critic","event":"agent_start","parent_mode":"none"}
+{"t":0,"agent":"ac9a1b4","agent_type":"critic","event":"agent_stop","success":true,"duration_ms":236050}

package/src/.omc/state/agent-replay-ed3e2cfb-a666-4bb1-bcc1-569d5d0b837a.jsonl

@@ -0,0 +1,2 @@
+{"t":0,"agent":"system","event":"keyword_detected","keyword":"ralph"}
+{"t":0,"agent":"system","event":"mode_change","mode_from":"none","mode_to":"ralph"}

package/src/.omc/state/idle-notif-cooldown.json

@@ -1,3 +1,3 @@
 {
-  "lastSentAt": "2026-06-21T11:09:39.757Z"
+  "lastSentAt": "2026-06-22T13:44:36.933Z"
 }

package/src/.omc/state/sessions/c513ef38-cc3e-4a29-9c4c-1e0137245dd5/mission-state.json

@@ -0,0 +1,79 @@
+{
+  "updatedAt": "2026-06-27T13:35:47.967Z",
+  "missions": [
+    {
+      "id": "session:c513ef38-cc3e-4a29-9c4c-1e0137245dd5:none",
+      "source": "session",
+      "name": "none",
+      "objective": "Session mission",
+      "createdAt": "2026-06-27T13:29:10.793Z",
+      "updatedAt": "2026-06-27T13:35:47.967Z",
+      "status": "done",
+      "workerCount": 2,
+      "taskCounts": {
+        "total": 2,
+        "pending": 0,
+        "blocked": 0,
+        "inProgress": 0,
+        "completed": 2,
+        "failed": 0
+      },
+      "agents": [
+        {
+          "name": "architect:a6f037b",
+          "role": "architect",
+          "ownership": "a6f037b2aebb4ef11",
+          "status": "done",
+          "currentStep": null,
+          "latestUpdate": "completed",
+          "completedSummary": null,
+          "updatedAt": "2026-06-27T13:31:22.835Z"
+        },
+        {
+          "name": "critic:ac9a1b4",
+          "role": "critic",
+          "ownership": "ac9a1b43d56c3ca70",
+          "status": "done",
+          "currentStep": null,
+          "latestUpdate": "completed",
+          "completedSummary": null,
+          "updatedAt": "2026-06-27T13:35:47.967Z"
+        }
+      ],
+      "timeline": [
+        {
+          "id": "session-start:a6f037b2aebb4ef11:2026-06-27T13:29:10.793Z",
+          "at": "2026-06-27T13:29:10.793Z",
+          "kind": "update",
+          "agent": "architect:a6f037b",
+          "detail": "started architect:a6f037b",
+          "sourceKey": "session-start:a6f037b2aebb4ef11"
+        },
+        {
+          "id": "session-stop:a6f037b2aebb4ef11:2026-06-27T13:31:22.835Z",
+          "at": "2026-06-27T13:31:22.835Z",
+          "kind": "completion",
+          "agent": "architect:a6f037b",
+          "detail": "completed",
+          "sourceKey": "session-stop:a6f037b2aebb4ef11"
+        },
+        {
+          "id": "session-start:ac9a1b43d56c3ca70:2026-06-27T13:31:51.917Z",
+          "at": "2026-06-27T13:31:51.917Z",
+          "kind": "update",
+          "agent": "critic:ac9a1b4",
+          "detail": "started critic:ac9a1b4",
+          "sourceKey": "session-start:ac9a1b43d56c3ca70"
+        },
+        {
+          "id": "session-stop:ac9a1b43d56c3ca70:2026-06-27T13:35:47.967Z",
+          "at": "2026-06-27T13:35:47.967Z",
+          "kind": "completion",
+          "agent": "critic:ac9a1b4",
+          "detail": "completed",
+          "sourceKey": "session-stop:ac9a1b43d56c3ca70"
+        }
+      ]
+    }
+  ]
+}

package/src/.omc/state/sessions/c513ef38-cc3e-4a29-9c4c-1e0137245dd5/pre-tool-advisory-throttle.json

@@ -0,0 +1,22 @@
+{
+  "version": 1,
+  "entries": {
+    "79a93d4a2f8f50b95f852280616242fee1855dc99a3c75211917f55e72e95fae": {
+      "last_emitted_at_ms": 1782567157541,
+      "message": "Use parallel execution for independent tasks. Use run_in_background for long operations (npm install, builds, tests)."
+    },
+    "466399dafa2d20f60587180bad0a07358eb7f2bce724df0ff682f038ad33f5ad": {
+      "last_emitted_at_ms": 1782567145707,
+      "message": "Read multiple files in parallel when possible for faster analysis."
+    },
+    "f5aaba48c1f35bcc584c1486fcb615715c685334220b3f63e3c02d30e32e3273": {
+      "last_emitted_at_ms": 1782567111776,
+      "message": "Spawning agent: oh-my-claudecode:critic (opus) | Task: Critic evaluation of validation plan"
+    },
+    "2bbd4e7762c885bc584b86ba2061c9117aeda36cd34b8641a610e68d917fb1cc": {
+      "last_emitted_at_ms": 1782566950662,
+      "message": "Spawning agent: oh-my-claudecode:architect (opus) | Task: Architect review of validation plan"
+    }
+  },
+  "updated_at": "2026-06-27T13:32:37.541Z"
+}

package/src/.omc/state/sessions/c513ef38-cc3e-4a29-9c4c-1e0137245dd5/subagent-tracking-state.json

@@ -0,0 +1,26 @@
+{
+  "agents": [
+    {
+      "agent_id": "a6f037b2aebb4ef11",
+      "agent_type": "oh-my-claudecode:architect",
+      "started_at": "2026-06-27T13:29:10.793Z",
+      "parent_mode": "none",
+      "status": "completed",
+      "completed_at": "2026-06-27T13:31:22.835Z",
+      "duration_ms": 132042
+    },
+    {
+      "agent_id": "ac9a1b43d56c3ca70",
+      "agent_type": "oh-my-claudecode:critic",
+      "started_at": "2026-06-27T13:31:51.917Z",
+      "parent_mode": "none",
+      "status": "completed",
+      "completed_at": "2026-06-27T13:35:47.967Z",
+      "duration_ms": 236050
+    }
+  ],
+  "total_spawned": 2,
+  "total_completed": 2,
+  "total_failed": 0,
+  "last_updated": "2026-06-27T13:35:48.069Z"
+}

package/src/.omc/state/sessions/ed3e2cfb-a666-4bb1-bcc1-569d5d0b837a/pre-tool-advisory-throttle.json

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

package/src/.omc/state/sessions/ed3e2cfb-a666-4bb1-bcc1-569d5d0b837a/ralph-state.json

@@ -0,0 +1,13 @@
+{
+  "active": true,
+  "iteration": 1,
+  "max_iterations": 100,
+  "started_at": "2026-06-22T13:48:06.524Z",
+  "prompt": "/ralph do it",
+  "session_id": "ed3e2cfb-a666-4bb1-bcc1-569d5d0b837a",
+  "project_path": "/Users/arthur/projects/infra-kit/apps/infra-kit/cli/src",
+  "linked_ultrawork": true,
+  "awaiting_confirmation": true,
+  "awaiting_confirmation_set_at": "2026-06-22T13:48:06.524Z",
+  "last_checked_at": "2026-06-22T13:48:06.524Z"
+}

package/src/.omc/state/sessions/ed3e2cfb-a666-4bb1-bcc1-569d5d0b837a/ultrawork-state.json

@@ -0,0 +1,11 @@
+{
+  "active": true,
+  "started_at": "2026-06-22T13:48:06.540Z",
+  "original_prompt": "/ralph do it",
+  "session_id": "ed3e2cfb-a666-4bb1-bcc1-569d5d0b837a",
+  "project_path": "/Users/arthur/projects/infra-kit/apps/infra-kit/cli/src",
+  "reinforcement_count": 0,
+  "awaiting_confirmation": true,
+  "awaiting_confirmation_set_at": "2026-06-22T13:48:06.540Z",
+  "last_checked_at": "2026-06-22T13:48:06.540Z"
+}

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

@@ -0,0 +1,7 @@
+{
+  "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

@@ -0,0 +1,18 @@
+{
+  "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,45 +1,13 @@
 import fs from 'node:fs/promises'
-import os from 'node:os'
 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'
 
-/**
- * Resolve whether a file is reachable, suppressing ENOENT into a boolean.
- *
- * @example
- * await fileExists('/etc/hosts')   // => true
- * await fileExists('/nope.txt')    // => false
- */
-const fileExists = async (filePath: string): Promise<boolean> => {
-  try {
-    await fs.access(filePath)
-
-    return true
-  } catch {
-    return false
-  }
-}
-
-/**
- * Replace the user's home prefix with `~` so logged paths stay short and
- * portable across machines. Leaves non-home paths untouched.
- *
- * @example
- * // os.homedir() === '/Users/arthur'
- * tildify('/Users/arthur/.infra-kit/config.json') // => '~/.infra-kit/config.json'
- * tildify('/etc/hosts')                          // => '/etc/hosts'
- */
-const tildify = (filePath: string): string => {
-  const home = os.homedir()
-
-  return filePath.startsWith(home) ? `~${filePath.slice(home.length)}` : filePath
-}
-
 /**
  * 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
@@ -50,7 +18,7 @@ const tildify = (filePath: string): string => {
  * // INFO: Project name: api
  * // INFO: Config merge chain (later overrides earlier):
  * // INFO:   [✓] project (committed)    ~/projects/api/infra-kit.json
- * // INFO:   [ ] user global            ~/.infra-kit/config.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> => {
@@ -155,13 +123,15 @@ 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/config.json — top-level keys
-// (environments, envManagement, ide, taskManager, worktrees) replace wholesale.
+// 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 }
+  // "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/doctor/__tests__/agent-files.test.ts

@@ -61,50 +61,28 @@ describe('checkAgentFiles', () => {
     )
   })
 
-  it('passes all three checks right after init runs the agent-files step', async () => {
+  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(statusOf(checks, 'AGENTS.md block')).toBe('pass')
-      expect(statusOf(checks, 'CLAUDE.md import')).toBe('pass')
-      expect(statusOf(checks, '.cursor/rules block')).toBe('pass')
+      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 AGENTS.md present but CLAUDE.md import removed', async () => {
+  it('flags a missing CLAUDE.md block', async () => {
     await withTmpRepo(async (tmp) => {
       await writeAgentFiles()
-      // User deletes CLAUDE.md (drops the @AGENTS.md import) but keeps AGENTS.md.
       fs.rmSync(path.join(tmp, 'CLAUDE.md'))
 
       const checks = await checkAgentFiles()
 
-      expect(statusOf(checks, 'AGENTS.md block')).toBe('pass')
-      expect(statusOf(checks, 'CLAUDE.md import')).toBe('fail')
-    })
-  })
-
-  it('flags a missing .cursor/rules block', async () => {
-    await withTmpRepo(async (tmp) => {
-      await writeAgentFiles()
-      fs.rmSync(path.join(tmp, '.cursor', 'rules', 'infra-kit.mdc'))
-
-      const checks = await checkAgentFiles()
-
-      expect(statusOf(checks, '.cursor/rules block')).toBe('fail')
-    })
-  })
-
-  it('flags a missing AGENTS.md block', async () => {
-    await withTmpRepo(async (tmp) => {
-      await writeAgentFiles()
-      fs.rmSync(path.join(tmp, 'AGENTS.md'))
-
-      const checks = await checkAgentFiles()
-
-      expect(statusOf(checks, 'AGENTS.md block')).toBe('fail')
+      expect(statusOf(checks, 'CLAUDE.md block')).toBe('fail')
     })
   })
 })

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

@@ -0,0 +1,110 @@
+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/)
+  })
+})