pantheon-opencode 1.3.5-beta.60.efa533f → 1.3.6

package/scripts/doctor.mjs

@@ -187,6 +187,26 @@ function showHelp() {
 // Helper utilities
 // ---------------------------------------------------------------------------
 
+/** @type {Set<string>} agent-defining frontmatter fields */
+const AGENT_FRONTMATTER_FIELDS = new Set(['name', 'description', 'mode'])
+
+/**
+ * Return true if the file content has YAML frontmatter containing at least
+ * one agent-defining field (name, description, or mode). Non-agent .md files
+ * such as README.md are excluded.
+ * @param {string} content
+ * @returns {boolean}
+ */
+export function isValidAgentFile(content) {
+  const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---/)
+  if (!match) return false
+  const fm = match[1]
+  for (const field of AGENT_FRONTMATTER_FIELDS) {
+    if (new RegExp(`(?:^|\\r?\\n)${field}\\s*:`).test(fm)) return true
+  }
+  return false
+}
+
 /**
  * Collect canonical agent names from agents/*.md
  * @returns {string[]} sorted list of agent names (without extension)
@@ -195,7 +215,9 @@ function getCanonicalAgentNames() {
   if (!existsSync(AGENTS_DIR)) return []
   return readdirSync(AGENTS_DIR)
     .filter((f) => f.endsWith('.md'))
-    .map((f) => f.replace(/\.agent\.md$/, ''))
+    .map((f) => ({ name: f, path: join(AGENTS_DIR, f) }))
+    .filter(({ path: p }) => isValidAgentFile(readFileSync(p, 'utf8')))
+    .map(({ name }) => name.replace(/\.agent\.md$/, ''))
     .sort()
 }
 
@@ -329,30 +351,41 @@ function checkAgentFiles(args) {
 // Check B: MCP Configuration
 // ---------------------------------------------------------------------------
 
+/** Resolve the effective OpenCode user config directory. */
+export function resolveOpenCodeConfigDir(env = process.env) {
+  if (env.PANTHEON_HOME) return resolve(env.PANTHEON_HOME)
+  const base = env.XDG_CONFIG_HOME || join(env.HOME ?? homedir(), '.config')
+  return join(base, 'opencode')
+}
+
 /**
  * Collect every reachable opencode.json (project → repository → user config)
  * with its parsed content, for the config, spawn-path and runtime layers.
  * @returns {{label:string, path:string, data:object}[]}
  */
-function collectMcpConfigs(args) {
+export function collectMcpConfigs(args) {
   const candidates = [
     { label: 'project root', path: join(args.target, 'opencode.json') },
     { label: 'repository root', path: join(ROOT, 'opencode.json') },
   ]
 
-  // Also check user config if target matches Pantheon ROOT
-  if (args.target === ROOT) {
-    candidates.push({
-      label: 'user config',
-      path: join(ROOT, '.opencode', 'opencode.json'),
-    })
-  } else {
-    const homeConfig = join(process.env.HOME ?? '~', '.config', 'opencode', 'opencode.json')
-    candidates.push({ label: 'user config', path: homeConfig })
-  }
+  // OpenCode resolves user configuration independently of cwd. Keep the
+  // project-local .opencode config for repository checks, but always include
+  // the effective user config so a sandbox HOME is not mistaken for cwd.
+  candidates.push({
+    label: 'project .opencode config',
+    path: join(args.target, '.opencode', 'opencode.json'),
+  })
+  candidates.push({
+    label: 'user config',
+    path: join(resolveOpenCodeConfigDir(args.env ?? process.env), 'opencode.json'),
+  })
 
   const configs = []
+  const seen = new Set()
   for (const c of candidates) {
+    if (seen.has(c.path)) continue
+    seen.add(c.path)
     const data = readJson(c.path)
     if (data) {
       configs.push({ label: c.label, data, path: c.path })
@@ -369,7 +402,9 @@ function checkMcpConfig(args) {
 
   if (configs.length === 0) {
     if (requiredMcpNames.length > 0) {
-      error(`No opencode.json found; required MCPs missing: ${requiredMcpNames.join(', ')}`)
+      error(
+        `No opencode.json found (searched project/repository and ${resolveOpenCodeConfigDir(args.env ?? process.env)}); required MCPs missing: ${requiredMcpNames.join(', ')}`,
+      )
     } else {
       info(`No opencode.json found for ${args.profile} profile — MCP check skipped`)
     }
@@ -928,7 +963,7 @@ function checkRequirementLine(line, installed) {
 function checkVenvLayer(args) {
   section('F. Runtime Layer — Venv, Python & Dependencies')
 
-  const configDir = join(homedir(), '.config', 'opencode')
+  const configDir = resolveOpenCodeConfigDir()
   const venvPython = join(configDir, '.venv', 'bin', 'python3')
 
   if (!existsSync(venvPython)) {
@@ -1125,7 +1160,15 @@ export function deriveInstalledAgentFiles(configs) {
       continue
     }
     for (const entry of entries) {
-      if (entry.isFile() && entry.name.endsWith('.md')) files.push(join(directory, entry.name))
+      if (!entry.isFile() || !entry.name.endsWith('.md')) continue
+      const filePath = join(directory, entry.name)
+      try {
+        if (isValidAgentFile(readFileSync(filePath, 'utf8'))) {
+          files.push(filePath)
+        }
+      } catch {
+        // unreadable — skip
+      }
     }
   }
   return [...new Set(files)].sort()

package/scripts/hooks/format-multi-language.sh

@@ -16,9 +16,12 @@ import sys, json
 d = json.load(sys.stdin)
 print(d.get('tool_name', ''))
 " 2>/dev/null || echo "")
+  # Normalize to lowercase — opencode sends lowercase (edit/write/bash),
+  # Claude Code protocol may send capitalized (Edit/Write/Bash).
+  TOOL_NAME_LOWER=$(echo "$TOOL_NAME" | tr '[:upper:]' '[:lower:]')
 
-  case "$TOOL_NAME" in
-    Edit|Write)
+  case "$TOOL_NAME_LOWER" in
+    edit|write)
       JSON_PATH=$(echo "$STDIN_JSON" | python3 -c "
 import sys, json
 d = json.load(sys.stdin)
@@ -28,7 +31,7 @@ print(d.get('tool_input', {}).get('file_path', ''))
         FILE_PATH="$JSON_PATH"
       fi
       ;;
-    Bash)
+    bash)
       COMMAND=$(echo "$STDIN_JSON" | python3 -c "
 import sys, json
 d = json.load(sys.stdin)

package/scripts/hooks/log-session-start.sh

@@ -4,10 +4,16 @@
 set -euo pipefail
 
 # Project-local by default; set XDG_STATE_HOME for system-wide logging
+# Anchor to PANTHEON_HOME or PANTHEON_PROJECT when available to avoid
+# CWD-dependent relative path ("logs/agent-sessions" wrote to cwd).
 if [ -n "${LOG_DIR:-}" ]; then
     LOG_DIR="$LOG_DIR"
 elif [ -n "${XDG_STATE_HOME:-}" ]; then
     LOG_DIR="$XDG_STATE_HOME/pantheon/hooks"
+elif [ -n "${PANTHEON_HOME:-}" ]; then
+    LOG_DIR="$PANTHEON_HOME/logs/agent-sessions"
+elif [ -n "${PANTHEON_PROJECT:-}" ]; then
+    LOG_DIR="$PANTHEON_PROJECT/logs/agent-sessions"
 else
     LOG_DIR="logs/agent-sessions"
 fi

package/scripts/hooks/on-subagent-delegation-start.sh

@@ -24,10 +24,16 @@
 set -euo pipefail
 
 # Project-local by default; set XDG_STATE_HOME for system-wide logging
+# Anchor to PANTHEON_HOME or PANTHEON_PROJECT when available to avoid
+# CWD-dependent relative path ("logs/agent-sessions" wrote to cwd).
 if [ -n "${LOG_DIR:-}" ]; then
     LOG_DIR="$LOG_DIR"
 elif [ -n "${XDG_STATE_HOME:-}" ]; then
     LOG_DIR="$XDG_STATE_HOME/pantheon/hooks"
+elif [ -n "${PANTHEON_HOME:-}" ]; then
+    LOG_DIR="$PANTHEON_HOME/logs/agent-sessions"
+elif [ -n "${PANTHEON_PROJECT:-}" ]; then
+    LOG_DIR="$PANTHEON_PROJECT/logs/agent-sessions"
 else
     LOG_DIR="logs/agent-sessions"
 fi
@@ -133,20 +139,30 @@ echo "[DELEGATION] $AGENT_NAME started → $LOG_DIR/delegations.log" >&2
 
 # --- Dispatch-time validation: check target against routing.yml ---
 SCRIPT_DIR="$(cd -- "$(dirname -- "${BASH_SOURCE[0]}")" && pwd)"
-REPO_ROOT="$(cd -- "$SCRIPT_DIR/../.." && pwd)"
-ROUTING_FILE="$REPO_ROOT/routing.yml"
-if [ -f "$ROUTING_FILE" ] && [ -n "$AGENT_NAME" ] && [ "$AGENT_NAME" != "unknown" ]; then
+REPO_ROOT="$(cd -- "$SCRIPT_DIR/../.." 2>/dev/null && pwd)"
+
+# Find routing.yml: try repo root first, then PANTHEON_HOME, then PANTHEON_PROJECT.
+# Silently skip if none found (deployed users may not have routing.yml locally).
+ROUTING_FILE=""
+if [ -n "$REPO_ROOT" ] && [ -f "$REPO_ROOT/routing.yml" ]; then
+    ROUTING_FILE="$REPO_ROOT/routing.yml"
+elif [ -n "${PANTHEON_HOME:-}" ] && [ -f "$PANTHEON_HOME/routing.yml" ]; then
+    ROUTING_FILE="$PANTHEON_HOME/routing.yml"
+elif [ -n "${PANTHEON_PROJECT:-}" ] && [ -f "$PANTHEON_PROJECT/routing.yml" ]; then
+    ROUTING_FILE="$PANTHEON_PROJECT/routing.yml"
+fi
+
+if [ -n "$ROUTING_FILE" ] && [ -n "$AGENT_NAME" ] && [ "$AGENT_NAME" != "unknown" ]; then
     ESCAPED_AGENT_NAME=$(printf '%s\n' "$AGENT_NAME" | sed 's/[][(){}.^$*+?|\/\\-]/\\&/g')
     if grep -qE "^[[:space:]]*-[[:space:]]+target:[[:space:]]+$ESCAPED_AGENT_NAME[[:space:]]*$" "$ROUTING_FILE" 2>/dev/null; then
         echo "[VALIDATION] ✅ Target '$AGENT_NAME' found in routing.yml" >&2
     else
         echo "[VALIDATION] ⚠️ Target '$AGENT_NAME' NOT in routing.yml delegation rules" >&2
     fi
-elif [ -f "$ROUTING_FILE" ]; then
+elif [ -n "$ROUTING_FILE" ]; then
     echo "[VALIDATION] ⚠️ Could not identify the delegated agent (agent='$AGENT_NAME') — skipping routing.yml validation" >&2
-elif [ ! -f "$ROUTING_FILE" ]; then
-    echo "[VALIDATION] ⚠️ Could not read routing.yml for validation" >&2
 fi
+# else: no routing.yml found — silently skip (deployed users may not have it locally)
 
 # --- Cost estimation for delegation ---
 declare -A TIER_MAP

package/scripts/hooks/validate-post-conditions.sh

@@ -9,7 +9,7 @@ SESSION_ID=$(echo "$INPUT" | python3 -c "import sys,json; print(json.load(sys.st
 
 REVIEW_DIR=".pantheon/memory-bank/.tmp"
 if [ -d "$REVIEW_DIR" ]; then
-    REVIEW_COUNT=$(ls "$REVIEW_DIR"/REVIEW-* 2>/dev/null | wc -l)
+    REVIEW_COUNT=$(find "$REVIEW_DIR" -maxdepth 1 -name 'REVIEW-*' -type f 2>/dev/null | wc -l)
     if [ "$REVIEW_COUNT" -eq 0 ]; then
         echo "[POST-CONDITION] ⚠️ No REVIEW artifact found in $REVIEW_DIR/" >&2
         echo "[POST-CONDITION] 💡 Remember: implementation agents MUST call @themis for review" >&2

package/scripts/hooks/validate-tool-safety.sh

@@ -18,8 +18,11 @@ BLOCKED_PATTERNS=(
     "DROP\s+DATABASE"
     "TRUNCATE\s+TABLE"
     "DELETE\s+FROM\s+.*;?\s*$"
-    "rm\s+.*\s+--no-preserve-root"
-    ":(){ :|:& };:"
+    # DEAD PATTERN REMOVED: `rm\s+.*\s+--no-preserve-root` — redundant with
+    # pattern #1 which already blocks `rm -rf /` regardless of flags.
+    # DEAD PATTERN REMOVED: `:(){ :|:& };:` — fork bomb syntax never appears
+    # in real tool_input.command payloads (the entire stdin would need to be
+    # exactly this 13-char token with zero surrounding JSON/context).
     ">\s*/dev/\(sd[a-z]\|hd[a-z]\|disk[0-9]*\)"
     "dd\s+if=.*of=/dev/"
     "mkfs\."

package/scripts/install/agents-md.mjs

@@ -76,7 +76,9 @@ export function readCanonicalAgents() {
   if (!existsSync(AGENTS_DIR)) return []
 
   const files = readdirSync(AGENTS_DIR)
-    .filter((f) => f.endsWith('.agent.md') || f.endsWith('.md'))
+    .filter(
+      (f) => (f.endsWith('.agent.md') || f.endsWith('.md')) && f.toLowerCase() !== 'readme.md',
+    )
     .sort()
 
   const agents = []

package/scripts/install/cli-ui.mjs

@@ -95,22 +95,42 @@ export function summaryTable(items) {
   const bot = `\u2514${'\u2500'.repeat(nameWidth + 2)}\u2534${'\u2500'.repeat(statusWidth + 2)}\u2518`
 
   console.log(tag(top))
-  console.log(tag(`\u2502 ${'Componente'.padEnd(nameWidth)} \u2502 ${'Status'.padEnd(statusWidth)} \u2502`))
+  console.log(
+    tag(`\u2502 ${'Componente'.padEnd(nameWidth)} \u2502 ${'Status'.padEnd(statusWidth)} \u2502`),
+  )
   console.log(tag(sep))
   for (const item of items) {
-    console.log(tag(`\u2502 ${item.name.padEnd(nameWidth)} \u2502 ${String(item.status).padEnd(statusWidth)} \u2502`))
+    console.log(
+      tag(
+        `\u2502 ${item.name.padEnd(nameWidth)} \u2502 ${String(item.status).padEnd(statusWidth)} \u2502`,
+      ),
+    )
   }
   console.log(tag(bot))
 }
 
 export function spinner(message) {
-  const frames = ['\u280b', '\u2819', '\u2839', '\u2838', '\u283c', '\u2834', '\u2826', '\u2827', '\u2807', '\u280f']
+  const frames = [
+    '\u280b',
+    '\u2819',
+    '\u2839',
+    '\u2838',
+    '\u283c',
+    '\u2834',
+    '\u2826',
+    '\u2827',
+    '\u2807',
+    '\u280f',
+  ]
   let i = 0
   let running = true
 
   if (tty() && !_quiet) {
     const interval = setInterval(() => {
-      if (!running) { clearInterval(interval); return }
+      if (!running) {
+        clearInterval(interval)
+        return
+      }
       process.stdout.write(`\r${frames[i % frames.length]} ${tag(message)}...`)
       i++
     }, 80)

package/scripts/install/interactive.mjs

@@ -31,7 +31,10 @@ function teardownStdin() {
   }
 }
 
-export async function promptMultiSelect(options, { title = '', selected = new Set(), pageSize = 6 } = {}) {
+export async function promptMultiSelect(
+  options,
+  { title = '', selected = new Set(), pageSize = 6 } = {},
+) {
   if (options.length === 0) return new Set()
 
   const preSelected = selected instanceof Set ? selected : new Set(selected)
@@ -83,7 +86,11 @@ export async function promptMultiSelect(options, { title = '', selected = new Se
     const count = items.filter((i) => i.checked).length
     const total = items.length
     const noun = count === 1 ? 'selected' : 'selected'
-    lines.push(colors.dim(`(${count}/${total} ${noun}) — ${colors.cyan('↑')}${colors.dim('/')}${colors.cyan('↓')} navigate · ${colors.cyan('Space')}${colors.dim(' toggle · ')}${colors.cyan('Enter')}${colors.dim(' confirm')}`))
+    lines.push(
+      colors.dim(
+        `(${count}/${total} ${noun}) — ${colors.cyan('↑')}${colors.dim('/')}${colors.cyan('↓')} navigate · ${colors.cyan('Space')}${colors.dim(' toggle · ')}${colors.cyan('Enter')}${colors.dim(' confirm')}`,
+      ),
+    )
 
     const lineCount = lines.length
     if (previousLineCount > 0) cursorUp(previousLineCount)
@@ -247,7 +254,11 @@ export async function promptList(options, { title = '' } = {}) {
 
     if (hasMoreBelow) lines.push(colors.dim('  ...'))
 
-    lines.push(colors.dim(`(${colors.cyan('↑')}${colors.dim('/')}${colors.cyan('↓')} navigate · ${colors.cyan('Enter')}${colors.dim(' select')}`))
+    lines.push(
+      colors.dim(
+        `(${colors.cyan('↑')}${colors.dim('/')}${colors.cyan('↓')} navigate · ${colors.cyan('Enter')}${colors.dim(' select')}`,
+      ),
+    )
 
     const lineCount = lines.length
     if (previousLineCount > 0) cursorUp(previousLineCount)
@@ -307,9 +318,7 @@ export async function promptList(options, { title = '' } = {}) {
 }
 
 export async function runInteractiveInstall({ components = [], defaultComponents = [] } = {}) {
-  const defaultSet = new Set(
-    defaultComponents.length > 0 ? defaultComponents : components,
-  )
+  const defaultSet = new Set(defaultComponents.length > 0 ? defaultComponents : components)
 
   const componentOptions = [
     {
@@ -362,7 +371,9 @@ export async function runInteractiveInstall({ components = [], defaultComponents
 
   if (selectedComponents.size === 0) {
     process.stdout.write(`${colors.yellow('No components selected — using defaults\n')}`)
-    defaultSet.forEach((v) => selectedComponents.add(v))
+    defaultSet.forEach((v) => {
+      selectedComponents.add(v)
+    })
   }
 
   const targetOptions = [

package/scripts/install/migrate-config.mjs

@@ -16,20 +16,20 @@
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs'
 import { join } from 'node:path'
 import {
-  readState,
-  writeState,
-  createInitialState,
-  addMigrationRecord,
-  updateVersion,
-} from './state.mjs'
-import {
-  readCheckpoint,
-  writeCheckpoint,
   advanceCheckpoint,
+  listAppliedMigrations,
+  readCheckpoint,
   removeCheckpoint,
   resumePendingMigration,
-  listAppliedMigrations,
+  writeCheckpoint,
 } from './migrate.mjs'
+import {
+  addMigrationRecord,
+  createInitialState,
+  readState,
+  updateVersion,
+  writeState,
+} from './state.mjs'
 
 // ---------------------------------------------------------------------------
 // Migration step definitions
@@ -290,16 +290,10 @@ function main() {
     addMigrationRecord(state, config.from, config.to, 'completed')
     removeCheckpoint(config.target)
     writeState(config.target, state)
-    console.log(
-      `\n  \u2705 Migration complete: ${config.from} \u2192 ${config.to}`,
-    )
-    console.log(
-      `     State written to: ${join(config.target, '.pantheon', 'install-state.json')}`,
-    )
+    console.log(`\n  \u2705 Migration complete: ${config.from} \u2192 ${config.to}`)
+    console.log(`     State written to: ${join(config.target, '.pantheon', 'install-state.json')}`)
   } else {
-    console.log(
-      `\n  \u2705 Migration would complete (dry-run): ${config.from} \u2192 ${config.to}`,
-    )
+    console.log(`\n  \u2705 Migration would complete (dry-run): ${config.from} \u2192 ${config.to}`)
   }
 }
 

package/scripts/install/migrate.mjs

@@ -11,7 +11,7 @@
 import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs'
 import { dirname, join } from 'node:path'
 import { fileURLToPath } from 'node:url'
-import { readState, writeState, createInitialState } from './state.mjs'
+import { createInitialState, readState, writeState } from './state.mjs'
 
 const __dirname = dirname(fileURLToPath(import.meta.url))
 const ROOT = join(__dirname, '..', '..')
@@ -243,9 +243,7 @@ export function rollbackMigration(target, version) {
   }
 
   const migrations = state.applied_migrations || []
-  const targetMigration = migrations.find(
-    (m) => m.to === version && m.status === 'completed',
-  )
+  const targetMigration = migrations.find((m) => m.to === version && m.status === 'completed')
   if (!targetMigration) {
     console.log(`  \u26A0\uFE0F  No completed migration found for version ${version}`)
     return { success: false, reason: 'Migration not found' }
@@ -341,7 +339,12 @@ function main() {
     console.log(`  Previous version: ${history.previousVersion || 'N/A'}`)
     console.log(`  Applied migrations: ${history.migrations.length}`)
     for (const m of history.migrations) {
-      const icon = m.status === 'completed' ? '\u2705' : m.status === 'rolled_back' ? '\u{1F504}' : '\u26A0\uFE0F'
+      const icon =
+        m.status === 'completed'
+          ? '\u2705'
+          : m.status === 'rolled_back'
+            ? '\u{1F504}'
+            : '\u26A0\uFE0F'
       console.log(`    ${icon} ${m.from} \u2192 ${m.to} [${m.status}] (${m.applied_at})`)
     }
     return

package/scripts/install/model-picker.mjs

@@ -55,7 +55,10 @@ export function writeActivePreset(
     copyFileSync(filePath, backupPath)
     backupMade = true
   }
-  writeFileSync(`${filePath}.tmp-${process.pid}`, `${JSON.stringify(buildActivePresetFile(presetName, source), null, 2)}\n`)
+  writeFileSync(
+    `${filePath}.tmp-${process.pid}`,
+    `${JSON.stringify(buildActivePresetFile(presetName, source), null, 2)}\n`,
+  )
   renameSync(`${filePath}.tmp-${process.pid}`, filePath)
   return { written: true, path: filePath, backupPath: backupMade ? backupPath : null }
 }

package/scripts/install/opencode.mjs

@@ -452,6 +452,7 @@ export async function installOpenCode(
       '_pantheon_paths.py',
       'mcp_persistence_server.py',
       'pantheon_vision_server.py',
+      'eval_store.py',
     ]
     const srcScriptsDir = join(ROOT, 'scripts')
     // Canonical MCP server sources live in src/mcp/. Map them explicitly so the
@@ -466,6 +467,7 @@ export async function installOpenCode(
       '_pantheon_paths.py': join(ROOT, 'src', 'mcp', '_pantheon_paths.py'),
       'mcp_persistence_server.py': join(ROOT, 'src', 'mcp', 'mcp_persistence_server.py'),
       'pantheon_vision_server.py': join(ROOT, 'src', 'mcp', 'pantheon_vision_server.py'),
+      'eval_store.py': join(ROOT, 'src', 'mcp', 'eval_store.py'),
     }
     const dstScriptsDir = join(runtimeTarget, 'scripts')
     if (!dryRun) mkdirSync(dstScriptsDir, { recursive: true })
@@ -558,7 +560,9 @@ export async function installOpenCode(
     const agentsDir = join(ROOT, 'src', 'agents')
     if (!existsSync(agentsDir)) return {}
     const sources = {}
-    const files = readdirSync(agentsDir).filter((f) => f.endsWith('.agent.md') || f.endsWith('.md'))
+    const files = readdirSync(agentsDir).filter(
+      (f) => (f.endsWith('.agent.md') || f.endsWith('.md')) && f.toLowerCase() !== 'readme.md',
+    )
     for (const f of files) {
       const name = f.replace(/\.(agent\.)?md$/, '')
       sources[name] = `${agentPrefix}/${name}.md`
@@ -570,7 +574,9 @@ export async function installOpenCode(
     const agentsDir = join(ROOT, 'src', 'agents')
     if (!existsSync(agentsDir)) return {}
     const config = {}
-    const files = readdirSync(agentsDir).filter((f) => f.endsWith('.agent.md') || f.endsWith('.md'))
+    const files = readdirSync(agentsDir).filter(
+      (f) => (f.endsWith('.agent.md') || f.endsWith('.md')) && f.toLowerCase() !== 'readme.md',
+    )
     for (const f of files) {
       const name = f.replace(/\.(agent\.)?md$/, '')
       const content = readFileSync(join(agentsDir, f), 'utf8')

package/scripts/install/shared.mjs

@@ -15,10 +15,10 @@ import {
 import { basename, dirname, join } from 'node:path'
 import { fileURLToPath } from 'node:url'
 
-const REQUIRED_NODE_MAJOR = 18;
+const REQUIRED_NODE_MAJOR = 18
 if (parseInt(process.versions.node.split('.')[0], 10) < REQUIRED_NODE_MAJOR) {
-  console.error(`❌ Node.js >= ${REQUIRED_NODE_MAJOR} required (current: ${process.versions.node})`);
-  process.exit(1);
+  console.error(`❌ Node.js >= ${REQUIRED_NODE_MAJOR} required (current: ${process.versions.node})`)
+  process.exit(1)
 }
 
 import yaml from 'js-yaml'
@@ -33,7 +33,9 @@ export const PLATFORM_DIR = join(ROOT, 'platform')
 export function getAgentNames() {
   if (!existsSync(AGENTS_DIR)) return []
   return readdirSync(AGENTS_DIR)
-    .filter((f) => f.endsWith('.agent.md') || f.endsWith('.md'))
+    .filter(
+      (f) => (f.endsWith('.agent.md') || f.endsWith('.md')) && f.toLowerCase() !== 'readme.md',
+    )
     .map((f) => f.replace(/\.(agent\.)?md$/, ''))
     .sort()
 }
@@ -277,9 +279,11 @@ export function copyFiles(srcDir, dstDir, dryRun, renameMap = null, clean = fals
   // Build set of expected destination filenames (after rename)
   const dstNames = new Set()
   for (const entry of entries) {
+    if (entry.toLowerCase() === 'readme.md') continue
     const srcFile = join(srcDir, entry)
     if (!existsSync(srcFile)) continue
     const dstName = renameMap ? (renameMap(entry) ?? entry) : entry
+    if (dstName.toLowerCase() === 'readme.md') continue
     dstNames.add(dstName)
 
     const dstFile = join(dstDir, dstName)

package/scripts/install/state.mjs

@@ -95,12 +95,15 @@ export function upgradeStateV1toV2(state) {
     updated_at: state.updated_at || now,
     migration_checkpoint: null,
     applied_migrations: Array.isArray(state.applied_migrations) ? state.applied_migrations : [],
-    components: state.components && typeof state.components === 'object' ? state.components : {
-      agents: { version: 1, status: 'installed' },
-      skills: { version: 1, status: 'installed' },
-      plugins: { version: 1, status: 'unknown' },
-      runtime: { version: 1, status: 'unknown' },
-    },
+    components:
+      state.components && typeof state.components === 'object'
+        ? state.components
+        : {
+            agents: { version: 1, status: 'installed' },
+            skills: { version: 1, status: 'installed' },
+            plugins: { version: 1, status: 'unknown' },
+            runtime: { version: 1, status: 'unknown' },
+          },
   }
 }