pantheon-opencode 1.6.0-beta.2 → 1.6.0-beta.4

package/scripts/cost.mjs

@@ -3,19 +3,42 @@
  * cost.mjs — read-only token aggregation from opencode.db.
  *
  * CLI fallback for the `pantheon_cost` tool when node:sqlite is unavailable
- * in the plugin process. Uses node:sqlite itself and prints a single JSON
- * object to stdout:
- *   { ok: true, days, rows: [{agent, phase, tokensInput, tokensOutput, tokensTotal}] }
+ * in the plugin process. Prints a single JSON object to stdout:
+ *   { ok: true, days, rows: [{agent, phase, tokensInput, tokensOutput, tokensTotal}], coverage }
  *   { ok: false, error: "<message>" }          (exit code 1)
  *
- * NEVER writes to the database — opened read-only. Malformed rows are
- * skipped; assistant messages carry the authoritative token usage.
+ * ## This file is a MIRROR of src/pantheon/cost-command.ts
+ *
+ * It cannot import that module — it exists precisely to run on a plain Node
+ * where the plugin runtime cannot — so the two implementations duplicate the
+ * same contract on purpose:
+ *   - ledger families are detected by TABLE PRESENCE, never by filename and
+ *     never by a pinned host generation;
+ *   - PANTHEON_OPENCODE_VERSION only NARROWS an already-detected set;
+ *   - V1 marks assistant turns with `data.role`, V2 with the `type` COLUMN
+ *     (V2 payloads carry no `role` key at all);
+ *   - the coverage counters mean the same thing in both, because the caller
+ *     feeds them to the same diagnosis.
+ *
+ * Every one of those was a real divergence once: this script kept the V1
+ * predicate and dropped 100% of a V2 ledger, while the caller reported a
+ * successful, empty report. A change to one side that is not made on the other
+ * is a regression, not a refactor.
+ *
+ * NEVER writes to the database — opened read-only. Malformed rows are skipped;
+ * assistant messages carry the authoritative token usage.
  *
  * Usage: node scripts/cost.mjs --db-path <dbPath> --days <days>
  */
 import { existsSync } from 'node:fs'
 import { DatabaseSync } from 'node:sqlite'
 
+/** Ledger families a host database can carry; a migrated one carries both. */
+const COST_SOURCES = ['message', 'session_message']
+
+/** Which family each host generation stores its ledger in. */
+const VERSION_SOURCES = { v1: 'message', v2: 'session_message' }
+
 /**
  * Fail via exitCode + natural process exit (NEVER process.exit right after
  * stdout.write — the pipe is async and the write can be truncated).
@@ -26,22 +49,201 @@ function fail(message) {
   process.exitCode = 1
 }
 
-function assertCompatibleSchema(db) {
-  const table = db
-    .prepare("SELECT name FROM sqlite_master WHERE type = 'table' AND name = 'message'")
-    .get()
-  if (!table)
-    throw new Error('incompatible opencode.db schema: required "message" table is missing')
-  const columns = db.prepare('PRAGMA table_info(message)').all()
-  const names = new Set(columns.map((column) => column.name))
-  const missing = ['data', 'time_created'].filter((name) => !names.has(name))
+/** Read and validate the optional host-generation selector. */
+function requestedVersion() {
+  const configured = process.env.PANTHEON_OPENCODE_VERSION
+  if (configured === undefined || configured === '') return undefined
+  if (configured !== 'v1' && configured !== 'v2') {
+    throw new Error(`invalid PANTHEON_OPENCODE_VERSION "${configured}"; expected "v1" or "v2"`)
+  }
+  return configured
+}
+
+/**
+ * Detect the ledger families by TABLE PRESENCE, then narrow if asked.
+ *
+ * Presence is the only reliable discriminator: every real installation writes
+ * `opencode.db`, and a migrated database carries BOTH families at once.
+ */
+function resolveSources(db) {
+  const present = new Set(
+    db
+      .prepare(
+        "SELECT name FROM sqlite_master WHERE type = 'table' AND name IN ('message', 'session_message')",
+      )
+      .all()
+      .map((row) => String(row.name)),
+  )
+  const detected = COST_SOURCES.filter((source) => present.has(source))
+  if (detected.length === 0) {
+    throw new Error(
+      'incompatible opencode.db schema: neither the "message" nor the "session_message" table exists, so there is no token ledger to read',
+    )
+  }
+  const requested = requestedVersion()
+  if (requested === undefined) return detected
+  const narrowed = detected.filter((source) => source === VERSION_SOURCES[requested])
+  if (narrowed.length === 0) {
+    throw new Error(
+      `incompatible opencode.db schema: PANTHEON_OPENCODE_VERSION=${requested} selected no table family, but this database contains ${detected.join(' and ')}`,
+    )
+  }
+  return narrowed
+}
+
+/** The columns this reader needs from the family it is about to query. */
+function assertColumns(db, source) {
+  const names = new Set(
+    db
+      .prepare(`PRAGMA table_info(${source})`)
+      .all()
+      .map((column) => String(column.name)),
+  )
+  const required =
+    source === 'session_message' ? ['data', 'time_created', 'type'] : ['data', 'time_created']
+  const missing = required.filter((name) => !names.has(name))
   if (missing.length > 0) {
     throw new Error(
-      `incompatible opencode.db schema: message table is missing ${missing.join(', ')}; select the matching V1/V2 database`,
+      `incompatible opencode.db schema: ${source} table is missing ${missing.join(', ')}; select a database whose token layout this build understands`,
     )
   }
 }
 
+/**
+ * Parse one payload. Returns undefined for anything unparseable, which is how
+ * both readers treat it: the V1 counter requires json_valid(data) and the V2
+ * aggregate does too, so a malformed row contributes to nothing.
+ *
+ * Deliberately NOT unwrapping a nested `data` string: the in-process reader
+ * extracts `$.role`/`$.agent` from the TOP level, so a wrapped payload yields
+ * NULL there and is dropped. Unwrapping here would resurrect rows the primary
+ * path cannot see, and the two readers would disagree on the same file.
+ */
+function readPayload(data) {
+  try {
+    const parsed = JSON.parse(String(data))
+    return typeof parsed === 'object' && parsed !== null ? parsed : undefined
+  } catch {
+    return undefined
+  }
+}
+
+/** CAST(json_extract(...) AS INTEGER) semantics: absent or non-numeric → 0. */
+function toToken(value) {
+  const parsed = Number(value)
+  return Number.isFinite(parsed) ? parsed : 0
+}
+
+/**
+ * Map key for one aggregated row.
+ *
+ * NUL-separated: agent names cannot contain it, so an agent whose name ends
+ * in the phase cannot collide with a phase-only row.
+ */
+function rowKey(agent, phase) {
+  return `${agent}\u0000${phase}`
+}
+
+/** Read one family.
+ *
+ * `assistantRows` follows the per-generation discriminator exactly, including
+ * the asymmetry that matters: V2 counts an assistant row by its COLUMN whether
+ * or not the payload parses, because that counter SQL has no json_valid gate.
+ */
+function readSource(db, source, since) {
+  const windowRow = db
+    .prepare(
+      `SELECT COUNT(*) AS scanned, MIN(time_created) AS windowStart FROM ${source} WHERE time_created >= ?`,
+    )
+    .get(since)
+  const databaseRow = db.prepare(`SELECT MIN(time_created) AS databaseStart FROM ${source}`).get()
+  const select = source === 'session_message' ? 'SELECT type, data' : 'SELECT data'
+  const rows = db.prepare(`${select} FROM ${source} WHERE time_created >= ?`).all(since)
+
+  const groups = new Map()
+  let assistantRows = 0
+  for (const row of rows) {
+    const info = readPayload(row.data)
+    if (source === 'session_message') {
+      if (row.type !== 'assistant') continue
+      assistantRows += 1
+      if (info === undefined) continue
+    } else {
+      if (info === undefined || info.role !== 'assistant') continue
+      assistantRows += 1
+    }
+    const rawAgent = info.agent
+    const agent = rawAgent == null ? '' : typeof rawAgent === 'string' ? rawAgent : String(rawAgent)
+    if (agent === '') continue
+    // V2 has no phase anywhere: its SQL emits the literal, so the mirror does
+    // too rather than inventing a column the reader never had.
+    const metadata = info.metadata
+    const phase =
+      source === 'session_message'
+        ? 'unknown'
+        : (typeof info.phase === 'string' && info.phase) ||
+          (typeof metadata?.phase === 'string' && metadata.phase) ||
+          'unknown'
+    const tokensInput = toToken(info.tokens?.input)
+    const tokensOutput = toToken(info.tokens?.output)
+    const key = rowKey(agent, phase)
+    const acc = groups.get(key) ?? {
+      agent,
+      phase,
+      tokensInput: 0,
+      tokensOutput: 0,
+      tokensTotal: 0,
+    }
+    acc.tokensInput += tokensInput
+    acc.tokensOutput += tokensOutput
+    acc.tokensTotal += tokensInput + tokensOutput
+    groups.set(key, acc)
+  }
+
+  return {
+    groups,
+    scanned: Number(windowRow?.scanned ?? 0) || 0,
+    windowStart: numberOrUndefined(windowRow?.windowStart),
+    databaseStart: numberOrUndefined(databaseRow?.databaseStart),
+    assistantRows,
+  }
+}
+
+function numberOrUndefined(value) {
+  if (value === null || value === undefined) return undefined
+  const parsed = Number(value)
+  return Number.isFinite(parsed) ? parsed : undefined
+}
+
+/**
+ * The same verdict the in-process reader reaches, with the same phrase: the
+ * caller maps "no readable token source" to UNSUPPORTED, so emitting anything
+ * softer here would resurrect the silent-empty-report failure this replaced.
+ *
+ * `scanned === 0` is NOT unreadable — an empty window is a real, reportable
+ * answer, and only a ledger that was found and could not be read is a fault.
+ */
+function diagnoseUnreadableLedger(coverage, rows, days) {
+  if (coverage.scanned === 0) return undefined
+  const tables = coverage.sources.join(' and ')
+  if (coverage.assistantRows === 0) {
+    return (
+      `no readable token source: ${coverage.scanned} row(s) were scanned in the last ${days} day(s) ` +
+      `but none matched the assistant discriminator of ${tables}, so no usage can be attributed. ` +
+      'This is an unrecognised ledger layout, not an empty one; point PANTHEON_COST_DB at a ' +
+      'database whose token layout this build understands.'
+    )
+  }
+  if (!rows.some((row) => row.tokensInput !== 0 || row.tokensOutput !== 0)) {
+    return (
+      `no readable token source: ${coverage.assistantRows} assistant row(s) were found in ${tables} ` +
+      'but none carried both an attributable agent and a token payload, so no usage can be ' +
+      'attributed. This is an unrecognised ledger layout, not an empty one.'
+    )
+  }
+  return undefined
+}
+
 function main() {
   const argv = process.argv.slice(2)
   const valueFor = (name) => {
@@ -56,44 +258,45 @@ function main() {
 
   try {
     const db = new DatabaseSync(dbPath, { readOnly: true })
-    assertCompatibleSchema(db)
+    const sources = resolveSources(db)
+    for (const source of sources) assertColumns(db, source)
     const since = Date.now() - days * 86_400_000
-    const rows = db.prepare('SELECT data FROM message WHERE time_created >= ?').all(since)
-    db.close()
 
     const byAgent = new Map()
-    for (const row of rows) {
-      try {
-        const outer = JSON.parse(String(row.data))
-        const info = typeof outer.data === 'string' ? JSON.parse(outer.data) : outer
-        if (info.role !== 'assistant') continue
-        const agent = typeof info.agent === 'string' && info.agent !== '' ? info.agent : null
-        if (!agent) continue
-        const metadata = info.metadata
-        const phase =
-          (typeof info.phase === 'string' && info.phase) ||
-          (typeof metadata?.phase === 'string' && metadata.phase) ||
-          'unknown'
-        const tokensInput = Number(info.tokens?.input) || 0
-        const tokensOutput = Number(info.tokens?.output) || 0
-        const key = `${agent}\u0000${phase}`
+    const coverage = { sources: [], scanned: 0, assistantRows: 0 }
+    const note = (key, value) => {
+      if (value === undefined) return
+      const current = coverage[key]
+      coverage[key] = current === undefined ? value : Math.min(current, value)
+    }
+    for (const source of sources) {
+      const read = readSource(db, source, since)
+      for (const row of read.groups.values()) {
+        const key = rowKey(row.agent, row.phase)
         const acc = byAgent.get(key) ?? {
-          agent,
-          phase,
+          agent: row.agent,
+          phase: row.phase,
           tokensInput: 0,
           tokensOutput: 0,
           tokensTotal: 0,
         }
-        acc.tokensInput += tokensInput
-        acc.tokensOutput += tokensOutput
-        acc.tokensTotal += tokensInput + tokensOutput
+        acc.tokensInput += row.tokensInput
+        acc.tokensOutput += row.tokensOutput
+        acc.tokensTotal += row.tokensInput + row.tokensOutput
         byAgent.set(key, acc)
-      } catch {
-        // Skip malformed rows — a partial ledger never breaks the report.
       }
+      coverage.sources.push(source)
+      coverage.scanned += read.scanned
+      coverage.assistantRows += read.assistantRows
+      note('windowStart', read.windowStart)
+      note('databaseStart', read.databaseStart)
     }
+    db.close()
+
     const result = [...byAgent.values()].sort((a, b) => b.tokensTotal - a.tokensTotal)
-    process.stdout.write(JSON.stringify({ ok: true, days, rows: result }))
+    const unreadable = diagnoseUnreadableLedger(coverage, result, days)
+    if (unreadable !== undefined) return fail(unreadable)
+    process.stdout.write(JSON.stringify({ ok: true, days, rows: result, coverage }))
   } catch (err) {
     fail(err instanceof Error ? err.message : String(err))
   }

package/scripts/install/agents-md.mjs

@@ -21,7 +21,7 @@
 import { existsSync, readdirSync, readFileSync, writeFileSync } from 'node:fs'
 import { dirname, join } from 'node:path'
 import { fileURLToPath } from 'node:url'
-import yaml from 'js-yaml'
+import { load } from 'js-yaml'
 
 // ---------------------------------------------------------------------------
 // Paths
@@ -47,7 +47,7 @@ function parseAgentFrontmatter(content) {
   if (!match) return null
 
   try {
-    const fm = yaml.load(match[1])
+    const fm = load(match[1])
     if (!fm || typeof fm !== 'object' || !fm.name) return null
     return {
       name: String(fm.name),

package/scripts/install/health-check.mjs

@@ -173,13 +173,20 @@ export function healthCheck(target, { dryRun = false, pythonTarget = target } =
   if (python) {
     const mcpResult = spawnSync(
       python,
-      ['-c', 'from mcp.server.fastmcp import FastMCP; print(FastMCP.__module__)'],
+      [
+        '-c',
+        'import importlib.metadata as metadata; from fastmcp import FastMCP; ' +
+          'print(f\'mcp={metadata.version("mcp")} fastmcp={metadata.version("fastmcp")} {FastMCP.__module__}\')',
+      ],
       { stdio: 'pipe' },
     )
     if (mcpResult.status === 0) {
       results.passed.push({ check: 'mcp-sdk', detail: mcpResult.stdout.toString().trim() })
     } else {
-      results.failed.push({ check: 'mcp-sdk', detail: 'NOT INSTALLED — all MCP servers will fail' })
+      results.failed.push({
+        check: 'mcp-sdk',
+        detail: 'MCP SDK or standalone FastMCP is unavailable — all MCP servers will fail',
+      })
     }
   }
   // Check 4: the memory server's only search backend is the stdlib's own

package/scripts/install/shared.mjs

@@ -24,7 +24,7 @@ if (parseInt(process.versions.node.split('.')[0], 10) < REQUIRED_NODE_MAJOR) {
   process.exit(1)
 }
 
-import yaml from 'js-yaml'
+import { dump, load } from 'js-yaml'
 
 export const __dirname = dirname(fileURLToPath(import.meta.url))
 export const ROOT = join(__dirname, '..', '..')
@@ -326,10 +326,10 @@ export function collectSkillNames() {
  * Returns { fm: object, body: string } or null if no frontmatter.
  */
 export function parseFrontmatter(content) {
-  const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n?([\s\S]*)$/)
+  const match = content.match(/^---\r?\n([\s\S]*?)(?:\r?\n)?---\r?\n?([\s\S]*)$/)
   if (!match) return null
   return {
-    fm: yaml.load(match[1]) ?? {},
+    fm: match[1].trim() ? (load(match[1]) ?? {}) : {},
     body: match[2],
   }
 }
@@ -339,9 +339,9 @@ export function parseFrontmatter(content) {
  * Uses long-line mode and avoids unnecessary quoting.
  */
 export function serializeFm(fm) {
-  return yaml.dump(fm, {
+  return dump(fm, {
     lineWidth: -1,
-    quotingType: '"',
+    quoteStyle: 'double',
     forceQuotes: false,
     noRefs: true,
   })

package/scripts/mcp_persistence_server.py

@@ -25,7 +25,7 @@ from pathlib import Path
 from typing import Annotated
 
 from _pantheon_paths import pantheon_home, pantheon_project
-from mcp.server.fastmcp import FastMCP
+from fastmcp import FastMCP
 from pydantic import Field
 
 # ── Schema ──────────────────────────────────────────────────────────────────────

package/scripts/mcp_resources_server.py

@@ -19,7 +19,7 @@ from typing import Any
 import yaml
 from _pantheon_paths import has_symlink_component, pantheon_home, pantheon_project
 from eval_store import get_latest_eval, list_evals
-from mcp.server.fastmcp import FastMCP
+from fastmcp import FastMCP
 
 # ── Path Resolution ──────────────────────────────────────────────────────────
 _PANTHEON_HOME: Path = pantheon_home()

package/scripts/memory_mcp_server.py

@@ -33,7 +33,7 @@ from pathlib import Path
 from typing import Annotated, Any
 
 from _pantheon_paths import pantheon_home
-from mcp.server.fastmcp import FastMCP
+from fastmcp import FastMCP
 from pydantic import BeforeValidator, Field
 
 # ── Paths ─────────────────────────────────────────────────────────────────────

package/scripts/probe-context-rehydrate.mjs

@@ -63,6 +63,12 @@ spec.loader.exec_module(module)
 
 async def call(name, arguments):
     result = await module.mcp.call_tool(name, arguments)
+    structured = getattr(result, "structured_content", None)
+    if structured is not None:
+        metadata = getattr(result, "meta", None) or {}
+        if metadata.get("fastmcp", {}).get("wrap_result"):
+            return structured.get("result") if isinstance(structured, dict) else structured
+        return structured
     if isinstance(result, tuple):
         structured = result[1]
         if structured is not None:

package/scripts/probe-pantheon-cost.mjs

@@ -56,11 +56,48 @@ function setEnvironment(values) {
   }
 }
 
+/**
+ * Write the rows in the family the requested host generation actually uses.
+ *
+ * Call sites describe an assistant turn the way they always have
+ * (`role: 'assistant'`, plus a `phase`). This function translates that
+ * descriptor into what is on disk: for V2, `role` becomes the `type` COLUMN and
+ * `phase` is dropped, because a real V2 payload carries neither — measured, the
+ * `role` key is absent from 40052 of 40052 rows on a host 2.0.22 database.
+ *
+ * Keeping that translation in ONE place is the point: a fixture that wrote
+ * `role` into a session_message row asserts the V1 format under a V2 name, which
+ * is exactly how the stale V1 predicate in the CLI mirror stayed green — the
+ * synthetic database agreed with the bug instead of contradicting it.
+ */
 function createSyntheticDb(DatabaseSync, path, rows) {
   const db = new DatabaseSync(path)
-  db.exec('CREATE TABLE message (data TEXT NOT NULL, time_created INTEGER NOT NULL)')
-  const insert = db.prepare('INSERT INTO message (data, time_created) VALUES (?, ?)')
-  for (const data of rows) insert.run(JSON.stringify(data), Date.now())
+  if (requestedVersion === 'v2') {
+    db.exec(
+      'CREATE TABLE session_message (id TEXT, session_id TEXT, type TEXT, seq INTEGER, time_created INTEGER, time_updated INTEGER, data TEXT)',
+    )
+    const insert = db.prepare(
+      'INSERT INTO session_message (id, session_id, type, seq, time_created, time_updated, data) VALUES (?,?,?,?,?,?,?)',
+    )
+    rows.forEach((row, index) => {
+      const payload = { ...row }
+      delete payload.role
+      delete payload.phase
+      insert.run(
+        `msg_${index + 1}`,
+        'ses_probe',
+        row.role,
+        index + 1,
+        Date.now(),
+        Date.now(),
+        JSON.stringify(payload),
+      )
+    })
+  } else {
+    db.exec('CREATE TABLE message (data TEXT NOT NULL, time_created INTEGER NOT NULL)')
+    const insert = db.prepare('INSERT INTO message (data, time_created) VALUES (?, ?)')
+    for (const data of rows) insert.run(JSON.stringify(data), Date.now())
+  }
   db.close()
 }
 
@@ -129,7 +166,10 @@ async function main() {
   const root = mkdtempSync(join(tmpdir(), 'pantheon-cost-probe-'))
   const dataHome = join(root, 'data')
   const opencodeDir = join(dataHome, 'opencode')
-  const selectedDb = join(opencodeDir, requestedVersion === 'v2' ? 'opencode-v2.db' : 'opencode.db')
+  // Every real installation writes `opencode.db`; the v1/v2 distinction is the
+  // SCHEMA of that file, never its name. Deriving `opencode-v2.db` here made the
+  // probe assert a path that exists on no real machine.
+  const selectedDb = join(opencodeDir, 'opencode.db')
   const otherDb = join(root, 'other.db')
   const envDb = join(root, 'env.db')
   const explicitDb = join(root, 'explicit.db')
@@ -139,7 +179,9 @@ async function main() {
   try {
     mkdirSync(opencodeDir, { recursive: true })
     // These rows intentionally include a monetary-looking field to prove it is
-    // ignored; only input/output/total tokens may appear in the report.
+    // ignored; only input/output/total tokens may appear in the report. Every
+    // V2 phase reads back as `unknown` because the V2 reader emits that literal:
+    // a V2 payload carries no phase field for a reader to recover.
     const selectedRows =
       requestedVersion === 'v2'
         ? [
@@ -224,8 +266,8 @@ async function main() {
         selectedOutput,
         requestedVersion === 'v2'
           ? [
-              { agent: 'aphrodite', phase: 'plan', input: 8, output: 10, total: 18 },
-              { agent: 'zeus', phase: 'audit', input: 4, output: 6, total: 10 },
+              { agent: 'aphrodite', phase: 'unknown', input: 8, output: 10, total: 18 },
+              { agent: 'zeus', phase: 'unknown', input: 4, output: 6, total: 10 },
             ]
           : [
               { agent: 'hermes', phase: 'plan', input: 13, output: 24, total: 37 },
@@ -250,7 +292,12 @@ async function main() {
         {},
         { sessionID: 'cost-probe' },
       )
-      requireOutput(output, '| env-agent | override | 1 | 2 | 3 |', 'PANTHEON_COST_DB')
+      // V2 has no phase to report, so the same row reads back as 'unknown'.
+      requireOutput(
+        output,
+        `| env-agent | ${requestedVersion === 'v2' ? 'unknown' : 'override'} | 1 | 2 | 3 |`,
+        'PANTHEON_COST_DB',
+      )
       if (output.includes('wrong-version') || output.includes('env-agent') === false) {
         throw new Error('PANTHEON_COST_DB did not override the version-selected database')
       }
@@ -269,7 +316,11 @@ async function main() {
         {},
         { sessionID: 'cost-probe' },
       )
-      requireOutput(output, '| explicit-agent | explicit | 9 | 1 | 10 |', 'explicit dbPath')
+      requireOutput(
+        output,
+        `| explicit-agent | ${requestedVersion === 'v2' ? 'unknown' : 'explicit'} | 9 | 1 | 10 |`,
+        'explicit dbPath',
+      )
       if (output.includes('env-agent')) throw new Error('explicit dbPath did not take precedence')
       checks.push('explicit dbPath precedence: PASS')
     } finally {
@@ -287,11 +338,7 @@ async function main() {
         { sessionID: 'cost-probe' },
       )
       requireOutput(output, 'not found', 'missing DB')
-      requireOutput(
-        output,
-        requestedVersion === 'v2' ? 'opencode-v2.db' : 'opencode.db',
-        'selected missing DB path',
-      )
+      requireOutput(output, 'opencode.db', 'selected missing DB path')
       checks.push('missing DB: PASS')
     } finally {
       missingRestore()
@@ -302,7 +349,16 @@ async function main() {
     }).pantheon_cost.execute({}, { sessionID: 'cost-probe' })
     requireOutput(incompatibleOutput, 'incompatible opencode.db schema', 'incompatible schema')
     checks.push('incompatible schema: PASS')
-    emit('PASS', 'offline synthetic DB checks passed; no real DB was read', checks)
+    // Names the synthetic fixture that was built, because the previous wording
+    // ("no real DB was read") was true of every run and therefore true of the run
+    // in which the V2 ledger was read as V1 and silently dropped. A verdict that
+    // cannot fail is not a verdict: it stayed green across the exact defect it
+    // was meant to cover.
+    emit(
+      'PASS',
+      `${checks.length} synthetic ${requestedVersion} checks passed against a temporary opencode.db (no user database was read)`,
+      checks,
+    )
   } catch (error) {
     emit('FAIL', diagnostic(error), checks)
   } finally {

package/scripts/validate-routing.mjs

@@ -19,7 +19,7 @@ import { createHash } from 'node:crypto'
 import { existsSync, readdirSync, readFileSync } from 'node:fs'
 import { dirname, join } from 'node:path'
 import { fileURLToPath } from 'node:url'
-import yaml from 'js-yaml'
+import { load } from 'js-yaml'
 import { validatePresetDefs } from '../src/pantheon/presets.mjs'
 
 const __dirname = dirname(fileURLToPath(import.meta.url))
@@ -52,7 +52,7 @@ if (!existsSync(routingPath)) {
   console.error('❌ routing.yml not found')
   process.exit(1)
 }
-const routing = yaml.load(readFileSync(routingPath, 'utf8'))
+const routing = load(readFileSync(routingPath, 'utf8'))
 
 console.log('🔍 Validating routing.yml...\n')
 
@@ -253,7 +253,7 @@ function parseFrontmatter(filePath) {
   const match = content.match(FRONTMATTER_RE)
   if (!match) return null
   try {
-    return yaml.load(match[1])
+    return load(match[1])
   } catch {
     return null
   }

package/src/mcp/code_mode_server.py

@@ -36,7 +36,7 @@ from typing import Any
 
 import yaml
 from _pantheon_paths import pantheon_home, pantheon_project
-from mcp.server.fastmcp import FastMCP
+from fastmcp import FastMCP
 
 # ── Constants ─────────────────────────────────────────────────────────────────
 ALLOWED_EXTENSIONS: frozenset[str] = frozenset({".sh", ".py"})

package/src/mcp/mcp_persistence_server.py

@@ -25,7 +25,7 @@ from pathlib import Path
 from typing import Annotated
 
 from _pantheon_paths import pantheon_home, pantheon_project
-from mcp.server.fastmcp import FastMCP
+from fastmcp import FastMCP
 from pydantic import Field
 
 # ── Schema ──────────────────────────────────────────────────────────────────────

package/src/mcp/mcp_resources_server.py

@@ -19,7 +19,7 @@ from typing import Any
 import yaml
 from _pantheon_paths import has_symlink_component, pantheon_home, pantheon_project
 from eval_store import get_latest_eval, list_evals
-from mcp.server.fastmcp import FastMCP
+from fastmcp import FastMCP
 
 # ── Path Resolution ──────────────────────────────────────────────────────────
 _PANTHEON_HOME: Path = pantheon_home()

package/src/mcp/memory_mcp_server.py

@@ -34,7 +34,7 @@ from pathlib import Path
 from typing import Annotated, Any
 
 from _pantheon_paths import pantheon_home
-from mcp.server.fastmcp import FastMCP
+from fastmcp import FastMCP
 from pydantic import BeforeValidator, Field
 
 try: