bigpowers 2.88.16 → 2.88.18

package/scripts/lib/story-verify-memo.sh

@@ -0,0 +1,56 @@
+#!/usr/bin/env bash
+# story: BUG-2026-10-30-compliance-akita-masked
+# Memoized verify execution for run-story-verify.sh. The caller defines
+# run_with_timeout and may set MEMO_DIR (one per gate_all run).
+
+if [ -n "${STORY_VERIFY_MEMO_LOADED:-}" ]; then return 0; fi
+STORY_VERIFY_MEMO_LOADED=1
+
+# Many stories share one verify (e61s02…e76s02 all run the same installer
+# test); same command at the same commit gives the same result, so run it once
+# per gate_all and reuse the verdict for every story that declares it.
+# Sets RUN_RC, RUN_OUT, and RUN_REUSED (1 when nothing new ran).
+run_memoized() {
+  local cmd="$1" key
+  key=$(printf '%s' "$cmd" | cksum | tr -d ' ')
+  if [ -n "${MEMO_DIR:-}" ] && [ -f "$MEMO_DIR/$key.rc" ]; then
+    RUN_RC=$(cat "$MEMO_DIR/$key.rc")
+    RUN_OUT=$(cat "$MEMO_DIR/$key.out")
+    return 0
+  fi
+  RUN_REUSED=0
+  RUN_OUT=$(run_with_timeout "$cmd") && RUN_RC=0 || RUN_RC=$?
+  if [ -n "${MEMO_DIR:-}" ]; then
+    printf '%s' "$RUN_RC" > "$MEMO_DIR/$key.rc"
+    printf '%s' "$RUN_OUT" > "$MEMO_DIR/$key.out"
+  fi
+}
+
+# A plain `a && b && c` chain shares no shell state between segments when it
+# has no quoting, expansion, grouping, pipes, `;`, cd, export or assignment —
+# so each segment can be cached on its own (test-adapter-render.sh prefixes ten
+# different story verifies). Anything else is cached as one whole command.
+is_plain_and_chain() {
+  case "$1" in
+    *\'*|*\"*|*\$*|*\`*|*\(*|*\)*|*\{*|*\}*|*\|*|*\;*|*=*|*\<*) return 1 ;;
+  esac
+  printf '%s' "$1" | grep -qE '(^|&&)[[:space:]]*(cd|export|source|\.)[[:space:]]' && return 1
+  return 0
+}
+
+run_memoized_chain() {
+  local cmd="$1" seg
+  RUN_REUSED=1
+  if ! is_plain_and_chain "$cmd"; then
+    run_memoized "$cmd"
+    return
+  fi
+  local rest="$cmd"
+  while :; do
+    seg=${rest%%&&*}
+    run_memoized "$(printf '%s' "$seg" | sed 's/^[[:space:]]*//;s/[[:space:]]*$//')"
+    [ "$RUN_RC" -eq 0 ] || return
+    [ "$seg" = "$rest" ] && return
+    rest=${rest#*&&}
+  done
+}

package/scripts/lib/test-assertions.sh

@@ -23,3 +23,9 @@ ta_fail() { echo "  FAIL $1"; TA_FAIL=$((TA_FAIL + 1)); }
 # Strip ANSI color codes — used by scripts that grep the colored output of a
 # gate/skill they're testing, where the escape codes would break the match.
 ta_strip_ansi() { sed -E 's/\x1b\[[0-9;]*m//g'; }
+
+# Validator style for validate-*.sh gates: "FAIL: msg" / "ok  : msg", counting
+# into the caller's ERRORS so its summary can report "N error(s)".
+ta_err() { echo "FAIL: $*"; ERRORS=$((ERRORS + 1)); }
+
+ta_ok() { echo "ok  : $*"; }

package/scripts/mcp-smoke.ts

@@ -1,11 +1,12 @@
 // story: e32s06
-// Simulates OMP spawning bigpowers-mcp from the installed plugin root (#123).
-// Verifies Node resolves @modelcontextprotocol/sdk from the root package graph
-// and the server reaches its startup log line without module-not-found.
+// Spawns bigpowers-mcp from the shipped .mcp.json the way each host expands it
+// (#123, #168). Verifies Node resolves @modelcontextprotocol/sdk from the root
+// package graph and the server reaches its startup log line without
+// module-not-found, whether or not the host injects OMP_PLUGIN_ROOT.
 
 import { readFileSync } from "node:fs";
 import { spawn } from "node:child_process";
-import { dirname, join } from "node:path";
+import { dirname, join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 
 const repoRoot = dirname(dirname(fileURLToPath(import.meta.url)));
@@ -15,53 +16,57 @@ const manifest = JSON.parse(readFileSync(join(repoRoot, ".mcp.json"), "utf8")) a
 const server = manifest.mcpServers?.["bigpowers-mcp"];
 if (!server) throw new Error("bigpowers-mcp entry missing from .mcp.json");
 
-const entry = join(repoRoot, "bigpowers-mcp/build/index.js");
-const started = await new Promise<{ ok: boolean; detail: string }>((resolve) => {
-  const child = spawn("node", [entry], {
-    cwd: repoRoot,
-    stdio: ["ignore", "pipe", "pipe"],
-    env: { ...process.env, OMP_PLUGIN_ROOT: repoRoot },
+// POSIX ${VAR} / ${VAR:-default}; an unset ${VAR} stays literal (omp and Claude Code both do this).
+function expand(value: string, env: Record<string, string | undefined>): string {
+  return value.replace(/\$\{([^}:]+)(?::-([^}]*))?\}/g, (whole, name: string, fallback?: string) => {
+    const v = env[name];
+    if (v !== undefined && (fallback === undefined || v !== "")) return v;
+    return fallback ?? whole;
   });
+}
 
-  let stderr = "";
-  const timer = setTimeout(() => {
-    child.kill("SIGTERM");
-    resolve({ ok: false, detail: `timeout waiting for startup (stderr: ${stderr.trim()})` });
-  }, 5000);
+// omp injects OMP_PLUGIN_ROOT and honours cwd; Claude Code project scope sets
+// neither and spawns from the project root.
+const { OMP_PLUGIN_ROOT: _ambient, ...baseEnv } = process.env;
+const hosts = [
+  { name: "omp", env: { ...baseEnv, OMP_PLUGIN_ROOT: repoRoot }, honoursCwd: true },
+  { name: "claude-code", env: baseEnv, honoursCwd: false },
+];
 
-  child.stderr.on("data", (chunk: Buffer) => {
-    stderr += chunk.toString();
-    if (stderr.includes("bigpowers-mcp started")) {
-      clearTimeout(timer);
+function startServer(entry: string, cwd: string, env: NodeJS.ProcessEnv): Promise<{ ok: boolean; detail: string }> {
+  return new Promise((done) => {
+    const child = spawn("node", [entry], { cwd, stdio: ["ignore", "pipe", "pipe"], env });
+
+    let stderr = "";
+    const timer = setTimeout(() => {
       child.kill("SIGTERM");
-      resolve({ ok: true, detail: "bigpowers-mcp started" });
-    }
-  });
+      done({ ok: false, detail: `timeout waiting for startup (stderr: ${stderr.trim()})` });
+    }, 5000);
 
-  child.on("exit", (code, signal) => {
-    if (stderr.includes("ERR_MODULE_NOT_FOUND")) {
+    child.stderr.on("data", (chunk: Buffer) => {
+      stderr += chunk.toString();
+      if (stderr.includes("bigpowers-mcp started")) {
+        clearTimeout(timer);
+        child.kill("SIGTERM");
+        done({ ok: true, detail: "bigpowers-mcp started" });
+      }
+    });
+
+    child.on("exit", (code, signal) => {
+      if (signal === "SIGTERM" && stderr.includes("bigpowers-mcp started")) return;
       clearTimeout(timer);
-      resolve({ ok: false, detail: stderr.trim() });
-      return;
-    }
-    if (signal === "SIGTERM" && stderr.includes("bigpowers-mcp started")) return;
-    clearTimeout(timer);
-    resolve({
-      ok: false,
-      detail: `exit code=${code ?? "null"} signal=${signal ?? "null"} stderr=${stderr.trim()}`,
+      done({ ok: false, detail: `exit code=${code ?? "null"} signal=${signal ?? "null"} stderr=${stderr.trim()}` });
     });
   });
-});
+}
 
-if (!started.ok) {
-  throw new Error(started.detail);
+const results = [];
+for (const host of hosts) {
+  const cwd = host.honoursCwd && server.cwd ? resolve(repoRoot, expand(server.cwd, host.env)) : repoRoot;
+  const entry = resolve(cwd, expand(server.args?.[0] ?? "", host.env));
+  const started = await startServer(entry, cwd, host.env);
+  if (!started.ok) throw new Error(`[${host.name}] ${entry}: ${started.detail}`);
+  results.push({ host: host.name, entry, startup: started.detail });
 }
 
-console.log(
-  JSON.stringify({
-    pluginRoot: repoRoot,
-    entry,
-    ompPluginRootInManifest: server.cwd?.includes("OMP_PLUGIN_ROOT") ?? false,
-    startup: started.detail,
-  }),
-);
+console.log(JSON.stringify({ pluginRoot: repoRoot, results }));

package/scripts/run-story-verify.sh

@@ -20,6 +20,8 @@ cd "$REPO_ROOT"
 source "$REPO_ROOT/scripts/lib/fail-open-detect.sh"
 # shellcheck source=lib/python-env.sh
 source "$REPO_ROOT/scripts/lib/python-env.sh"
+# shellcheck source=lib/story-verify-memo.sh
+source "$REPO_ROOT/scripts/lib/story-verify-memo.sh"
 resolve_python || exit 1
 
 VERIFY_TIMEOUT_SECONDS=120
@@ -58,55 +60,6 @@ reject_non_executable() {
   FAIL=$((FAIL + 1))
 }
 
-# Many stories share one verify (e61s02…e76s02 all run the same installer
-# test); same command at the same commit gives the same result, so run it once
-# per gate_all and reuse the verdict for every story that declares it.
-# Sets RUN_RC, RUN_OUT, and RUN_REUSED (1 when nothing new ran).
-run_memoized() {
-  local cmd="$1" key
-  key=$(printf '%s' "$cmd" | cksum | tr -d ' ')
-  if [ -n "${MEMO_DIR:-}" ] && [ -f "$MEMO_DIR/$key.rc" ]; then
-    RUN_RC=$(cat "$MEMO_DIR/$key.rc")
-    RUN_OUT=$(cat "$MEMO_DIR/$key.out")
-    return 0
-  fi
-  RUN_REUSED=0
-  RUN_OUT=$(run_with_timeout "$cmd") && RUN_RC=0 || RUN_RC=$?
-  if [ -n "${MEMO_DIR:-}" ]; then
-    printf '%s' "$RUN_RC" > "$MEMO_DIR/$key.rc"
-    printf '%s' "$RUN_OUT" > "$MEMO_DIR/$key.out"
-  fi
-}
-
-# A plain `a && b && c` chain shares no shell state between segments when it
-# has no quoting, expansion, grouping, pipes, `;`, cd, export or assignment —
-# so each segment can be cached on its own (test-adapter-render.sh prefixes ten
-# different story verifies). Anything else is cached as one whole command.
-is_plain_and_chain() {
-  case "$1" in
-    *\'*|*\"*|*\$*|*\`*|*\(*|*\)*|*\{*|*\}*|*\|*|*\;*|*=*|*\<*) return 1 ;;
-  esac
-  printf '%s' "$1" | grep -qE '(^|&&)[[:space:]]*(cd|export|source|\.)[[:space:]]' && return 1
-  return 0
-}
-
-run_verify_cmd() {
-  local cmd="$1" seg
-  RUN_REUSED=1
-  if ! is_plain_and_chain "$cmd"; then
-    run_memoized "$cmd"
-    return
-  fi
-  local rest="$cmd"
-  while :; do
-    seg=${rest%%&&*}
-    run_memoized "$(printf '%s' "$seg" | sed 's/^[[:space:]]*//;s/[[:space:]]*$//')"
-    [ "$RUN_RC" -eq 0 ] || return
-    [ "$seg" = "$rest" ] && return
-    rest=${rest#*&&}
-  done
-}
-
 run_story_verify() {
   local sid="$1" src="$2" cmd="$3"
 
@@ -119,7 +72,7 @@ run_story_verify() {
     return 1
   fi
 
-  run_verify_cmd "$cmd"
+  run_memoized_chain "$cmd"
   local note=""
   [ "$RUN_REUSED" -eq 1 ] && note=" (shared result)"
   if [ "$RUN_RC" -eq 0 ]; then

package/scripts/test-golden-gate-runner.sh

@@ -16,13 +16,13 @@ source "$(dirname "${BASH_SOURCE[0]}")/lib/skill-common.sh"
 resolve_repo_root
 cd "$REPO_ROOT"
 
-FAILURES=0
-pass() { echo "PASS $1"; }
-fail() { echo "FAIL $1"; FAILURES=$((FAILURES + 1)); }
+source scripts/lib/test-assertions.sh
+TA_PASS=0
+TA_FAIL=0
 
 # Run the runner in a subshell with a fixture gate list; prints its stdout and
 # the final counters as "COUNTS pass fail skip".
-run_fixture() {
+ggr_run_fixture() {
   (
     source scripts/lib/golden-suite-gates.sh
     GOLDEN_GATES=(
@@ -35,25 +35,25 @@ run_fixture() {
   )
 }
 
-out=$(GOLDEN_SKIP_GATES="" run_fixture)
+out=$(GOLDEN_SKIP_GATES="" ggr_run_fixture)
 echo "$out" | grep -q 'marker-line-from-failing-gate' \
-  && pass "(a) failing gate output is printed" \
-  || fail "(a) failing gate output is missing"
+  && ta_pass "(a) failing gate output is printed" \
+  || ta_fail "(a) failing gate output is missing"
 echo "$out" | grep -q '^COUNTS 2 1 0$' \
-  && pass "(d) no skip list runs every gate" \
-  || fail "(d) default run counts wrong: $(echo "$out" | grep COUNTS)"
+  && ta_pass "(d) no skip list runs every gate" \
+  || ta_fail "(d) default run counts wrong: $(echo "$out" | grep COUNTS)"
 
-out=$(GOLDEN_SKIP_GATES="covered-gate" GOLDEN_SKIP_REASON="runs in its own CI job" run_fixture)
+out=$(GOLDEN_SKIP_GATES="covered-gate" GOLDEN_SKIP_REASON="runs in its own CI job" ggr_run_fixture)
 echo "$out" | grep -q '^\[covered-gate\] .*SKIP.*runs in its own CI job' \
-  && pass "(b) skipped gate reported with reason" \
-  || fail "(b) skipped gate not reported with reason"
+  && ta_pass "(b) skipped gate reported with reason" \
+  || ta_fail "(b) skipped gate not reported with reason"
 echo "$out" | grep -q '^COUNTS 1 1 1$' \
-  && pass "(b) skipped gate counted as skip, not pass" \
-  || fail "(b) skip counts wrong: $(echo "$out" | grep COUNTS)"
+  && ta_pass "(b) skipped gate counted as skip, not pass" \
+  || ta_fail "(b) skip counts wrong: $(echo "$out" | grep COUNTS)"
 
-out=$(GOLDEN_SKIP_GATES="no-such-gate" run_fixture)
+out=$(GOLDEN_SKIP_GATES="no-such-gate" ggr_run_fixture)
 echo "$out" | grep -q 'no-such-gate' && echo "$out" | grep -q '^COUNTS 2 2 0$' \
-  && pass "(c) unknown skip name fails the run" \
-  || fail "(c) unknown skip name accepted: $(echo "$out" | grep COUNTS)"
+  && ta_pass "(c) unknown skip name fails the run" \
+  || ta_fail "(c) unknown skip name accepted: $(echo "$out" | grep COUNTS)"
 
-[ "$FAILURES" -eq 0 ] && echo "OK: golden gate runner" || { echo "$FAILURES check(s) failed"; exit 1; }
+[ "$TA_FAIL" -eq 0 ] && echo "OK: golden gate runner" || { echo "$TA_FAIL check(s) failed"; exit 1; }

package/scripts/test-project-runtime.sh

@@ -6,27 +6,18 @@ ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
 # shellcheck disable=SC1091
 source "$ROOT/scripts/lib/project-runtime.sh"
 
-PASS=0
-FAIL=0
+source "$ROOT/scripts/lib/test-assertions.sh"
+TA_PASS=0
+TA_FAIL=0
 TMPDIR_ROOT="$(mktemp -d)"
 trap 'rm -rf "$TMPDIR_ROOT"' EXIT
 
-pass() {
-  echo "  PASS $1"
-  PASS=$((PASS + 1))
-}
-
-fail() {
-  echo "  FAIL $1" >&2
-  FAIL=$((FAIL + 1))
-}
-
 assert_eq() {
   local name="$1" expected="$2" actual="$3"
   if [[ "$actual" == "$expected" ]]; then
-    pass "$name"
+    ta_pass "$name"
   else
-    fail "$name — expected '$expected', got '$actual'"
+    ta_fail "$name — expected '$expected', got '$actual'"
   fi
 }
 
@@ -35,11 +26,11 @@ assert_fails_with() {
   shift 2
   local output
   if output="$("$@" 2>&1)"; then
-    fail "$name — command unexpectedly succeeded"
+    ta_fail "$name — command unexpectedly succeeded"
   elif [[ "$output" == *"$expected"* ]]; then
-    pass "$name"
+    ta_pass "$name"
   else
-    fail "$name — expected error containing '$expected', got '$output'"
+    ta_fail "$name — expected error containing '$expected', got '$output'"
   fi
 }
 
@@ -87,5 +78,5 @@ assert_fails_with "empty specs directory is rejected" \
   "specs_dir must not be empty" bp_specs_dir "$EMPTY_PROJECT"
 
 echo "---"
-echo "test-project-runtime: $PASS passed, $FAIL failed"
-[[ "$FAIL" -eq 0 ]]
+echo "test-project-runtime: $TA_PASS passed, $TA_FAIL failed"
+[[ "$TA_FAIL" -eq 0 ]]

package/scripts/validate-docs-base-path.sh

@@ -9,29 +9,22 @@ WEBSITE="$REPO_ROOT/website"
 SITE_BASE="/bigpowers"
 ERRORS=0
 
-fail() {
-  echo "FAIL: $*"
-  ERRORS=$((ERRORS + 1))
-}
-
-pass() {
-  echo "ok  : $*"
-}
+source "$REPO_ROOT/scripts/lib/test-assertions.sh"
 
 ensure_website_deps() {
   if [[ -d "$WEBSITE/node_modules/astro" ]]; then
     return 0
   fi
   if ! command -v npm >/dev/null 2>&1; then
-    fail "npm unavailable and website/node_modules missing — cannot run build check"
+    ta_err "npm unavailable and website/node_modules missing — cannot run build check"
     return 1
   fi
   echo "info: installing website deps (golden-suite only installs repo root npm ci)"
   if ! (cd "$WEBSITE" && npm ci >/dev/null 2>&1); then
-    fail "website npm ci failed — cannot run build check"
+    ta_err "website npm ci failed — cannot run build check"
     return 1
   fi
-  pass "website deps installed for build check"
+  ta_ok "website deps installed for build check"
 }
 
 INDEX_MDX="$WEBSITE/src/content/docs/index.mdx"
@@ -39,27 +32,27 @@ LLMS_TXT="$WEBSITE/public/llms.txt"
 
 echo "--- [docs] generated landing hero link ---"
 if [[ ! -f "$INDEX_MDX" ]]; then
-  fail "missing $INDEX_MDX — run cd website && npm run generate"
+  ta_err "missing $INDEX_MDX — run cd website && npm run generate"
 elif grep -q "link: ${SITE_BASE}/skills/" "$INDEX_MDX"; then
-  pass "hero Browse skills link includes SITE_BASE"
+  ta_ok "hero Browse skills link includes SITE_BASE"
 else
-  fail "hero link missing ${SITE_BASE}/skills/ prefix in index.mdx"
+  ta_err "hero link missing ${SITE_BASE}/skills/ prefix in index.mdx"
 fi
 
 echo "--- [docs] no root-absolute skill links in landing page ---"
 if [[ -f "$INDEX_MDX" ]] && grep -E '\]\(/skills/' "$INDEX_MDX" >/dev/null 2>&1; then
-  fail "index.mdx still contains root-absolute /skills/ links"
+  ta_err "index.mdx still contains root-absolute /skills/ links"
 else
-  pass "index.mdx skill links are base-prefixed or absent"
+  ta_ok "index.mdx skill links are base-prefixed or absent"
 fi
 
 echo "--- [docs] llms.txt URL shape ---"
 if [[ ! -f "$LLMS_TXT" ]]; then
-  fail "missing $LLMS_TXT — run cd website && npm run generate"
+  ta_err "missing $LLMS_TXT — run cd website && npm run generate"
 elif grep -q '/bigpowers/bigpowers/' "$LLMS_TXT"; then
-  fail "llms.txt double-prefixes SITE_BASE in URLs"
+  ta_err "llms.txt double-prefixes SITE_BASE in URLs"
 else
-  pass "llms.txt has no /bigpowers/bigpowers/ double prefix"
+  ta_ok "llms.txt has no /bigpowers/bigpowers/ double prefix"
 fi
 
 echo "--- [docs] production build hero href ---"
@@ -69,12 +62,12 @@ elif ensure_website_deps; then
   BUILD_ERR=$(mktemp)
   if (cd "$WEBSITE" && npm run build >/dev/null 2>"$BUILD_ERR"); then
     if grep -q 'href="/bigpowers/skills/"' "$WEBSITE/dist/index.html" 2>/dev/null; then
-      pass "dist/index.html hero href includes /bigpowers/skills/"
+      ta_ok "dist/index.html hero href includes /bigpowers/skills/"
     else
-      fail "dist/index.html missing href=\"/bigpowers/skills/\" for Browse skills"
+      ta_err "dist/index.html missing href=\"/bigpowers/skills/\" for Browse skills"
     fi
   else
-    fail "website build failed — $(head -3 "$BUILD_ERR" 2>/dev/null | tr '\n' ' ')"
+    ta_err "website build failed — $(head -3 "$BUILD_ERR" 2>/dev/null | tr '\n' ' ')"
   fi
   rm -f "$BUILD_ERR"
 fi

package/scripts/validate-mcp-package.sh

@@ -9,59 +9,57 @@ cd "$REPO_ROOT"
 
 ERRORS=0
 
-fail() {
-  echo "FAIL: $*"
-  ERRORS=$((ERRORS + 1))
-}
-
-pass() {
-  echo "ok  : $*"
-}
+source "$REPO_ROOT/scripts/lib/test-assertions.sh"
 
 MCP_JSON="$REPO_ROOT/.mcp.json"
 echo "--- [MCP] shipped .mcp.json OMP compatibility ---"
 if [[ ! -f "$MCP_JSON" ]]; then
-  fail ".mcp.json missing at repo root"
+  ta_err ".mcp.json missing at repo root"
 else
   if grep -q 'workspaceFolder' "$MCP_JSON" 2>/dev/null; then
-    fail ".mcp.json uses workspaceFolder — not valid in OMP env expansion (#123)"
+    ta_err ".mcp.json uses workspaceFolder — not valid in OMP env expansion (#123)"
   else
-    pass ".mcp.json does not reference workspaceFolder"
+    ta_ok ".mcp.json does not reference workspaceFolder"
   fi
   if grep -q 'OMP_PLUGIN_ROOT' "$MCP_JSON" 2>/dev/null; then
-    pass ".mcp.json references OMP_PLUGIN_ROOT for plugin installs"
+    ta_ok ".mcp.json references OMP_PLUGIN_ROOT for plugin installs"
+  else
+    ta_err ".mcp.json must reference OMP_PLUGIN_ROOT for OMP plugin cwd/args"
+  fi
+  if grep -q '\${OMP_PLUGIN_ROOT}' "$MCP_JSON" 2>/dev/null; then
+    ta_err ".mcp.json uses \${OMP_PLUGIN_ROOT} without a :- default — hosts that do not set it spawn a literal path (#168)"
   else
-    fail ".mcp.json must reference OMP_PLUGIN_ROOT for OMP plugin cwd/args"
+    ta_ok ".mcp.json OMP_PLUGIN_ROOT references carry a :- default"
   fi
 fi
 
 echo "--- [MCP] root runtime dependencies ---"
 for dep in "@modelcontextprotocol/sdk" "unified" "remark-parse" "zod"; do
   if node -e "const p=require('./package.json'); process.exit(p.dependencies?.['$dep'] ? 0 : 1)"; then
-    pass "root dependencies include $dep"
+    ta_ok "root dependencies include $dep"
   else
-    fail "root dependencies missing $dep (bigpowers-mcp runtime)"
+    ta_err "root dependencies missing $dep (bigpowers-mcp runtime)"
   fi
 done
 
 echo "--- [MCP] built server entry ---"
 if [[ -f "$REPO_ROOT/bigpowers-mcp/build/index.js" ]]; then
-  pass "bigpowers-mcp/build/index.js exists"
+  ta_ok "bigpowers-mcp/build/index.js exists"
 else
-  fail "bigpowers-mcp/build/index.js missing — run cd bigpowers-mcp && npm run build"
+  ta_err "bigpowers-mcp/build/index.js missing — run cd bigpowers-mcp && npm run build"
 fi
 
 echo "--- [MCP] runtime load smoke (scripts/mcp-smoke.ts) ---"
 if ! command -v node >/dev/null 2>&1; then
   echo "warn: node unavailable — skipping runtime load smoke"
 elif [[ ! -f "$REPO_ROOT/scripts/mcp-smoke.ts" ]]; then
-  fail "scripts/mcp-smoke.ts not found"
+  ta_err "scripts/mcp-smoke.ts not found"
 else
   SMOKE_ERR=$(mktemp)
   if node "$REPO_ROOT/scripts/mcp-smoke.ts" >/dev/null 2>"$SMOKE_ERR"; then
-    pass "MCP server loads from simulated OMP plugin root"
+    ta_ok "MCP server loads from simulated OMP plugin root"
   else
-    fail "MCP smoke failed — $(head -3 "$SMOKE_ERR" 2>/dev/null | tr '\n' ' ')"
+    ta_err "MCP smoke failed — $(head -3 "$SMOKE_ERR" 2>/dev/null | tr '\n' ' ')"
   fi
   rm -f "$SMOKE_ERR"
 fi