@kontourai/flow-agents 0.1.2 → 0.2.0

package/evals/integration/test_utterance_check.sh

@@ -22,17 +22,17 @@ HOOK="$ROOT/scripts/hooks/utterance-check.js"
 RUN_HOOK="$ROOT/scripts/hooks/run-hook.js"
 
 # ---------------------------------------------------------------------------
-# Hook: pass-through when disabled (default)
+# Hook: pass-through when disabled by default (no config, no env var)
 # ---------------------------------------------------------------------------
 
 echo ""
-echo "--- hook: disabled by default ---"
+echo "--- hook: disabled by default (no config, no env var) ---"
 
 INPUT_JSON='{"hook_event_name":"PostToolUse","tool_response":"The coverage is 92% and all tests pass."}'
 
 if node "$HOOK" >"$TMPDIR_EVAL/disabled.out" 2>"$TMPDIR_EVAL/disabled.err" <<< "$INPUT_JSON"; then
   if grep -qF '"hook_event_name"' "$TMPDIR_EVAL/disabled.out"; then
-    _pass "utterance check hook passes through when FLOW_AGENTS_UTTERANCE_CHECK_ENABLED is unset"
+    _pass "utterance check hook passes through when no config and FLOW_AGENTS_UTTERANCE_CHECK_ENABLED is unset"
   else
     _fail "utterance check hook pass-through output was not the raw input"
   fi
@@ -40,6 +40,63 @@ else
   _fail "utterance check hook should exit 0 when disabled"
 fi
 
+# ---------------------------------------------------------------------------
+# Hook: env var force-off overrides a config that would enable
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: env var force-off overrides config ---"
+
+# Create a temp repo dir with a config that has enabled:true
+FAKE_REPO="$TMPDIR_EVAL/fake-repo"
+mkdir -p "$FAKE_REPO/context/settings"
+cat > "$FAKE_REPO/AGENTS.md" <<'AGENTS_EOF'
+# Fake repo for testing
+AGENTS_EOF
+cat > "$FAKE_REPO/context/settings/flow-agents-settings.json" <<'CONFIG_EOF'
+{"schema_version":"1.0","utteranceCheck":{"enabled":true,"mode":"report","extractor":"reference"}}
+CONFIG_EOF
+
+INPUT_WITH_CWD="{\"hook_event_name\":\"PostToolUse\",\"tool_response\":\"text\",\"cwd\":\"$FAKE_REPO\"}"
+
+if FLOW_AGENTS_UTTERANCE_CHECK_ENABLED=false \
+   node "$HOOK" >"$TMPDIR_EVAL/forceoff.out" 2>"$TMPDIR_EVAL/forceoff.err" <<< "$INPUT_WITH_CWD"; then
+  if grep -qF '"hook_event_name"' "$TMPDIR_EVAL/forceoff.out"; then
+    _pass "env var FLOW_AGENTS_UTTERANCE_CHECK_ENABLED=false forces hook off even when config has enabled:true"
+  else
+    _fail "force-off pass-through output did not match raw input"
+  fi
+else
+  _fail "hook should exit 0 when force-off via env var"
+fi
+
+# ---------------------------------------------------------------------------
+# Hook: config-based enable (no env var override) passes through to CLI
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: config-based enable reaches CLI (fail-open on missing CLI is acceptable) ---"
+
+if node "$HOOK" >"$TMPDIR_EVAL/config-enable.out" 2>"$TMPDIR_EVAL/config-enable.err" <<< "$INPUT_WITH_CWD"; then
+  _pass "hook with config enabled exits 0 (fails open when CLI or survey is unavailable)"
+else
+  _fail "hook with config enabled should exit 0 (fail-open)"
+fi
+
+# ---------------------------------------------------------------------------
+# Hook: env var force-on (legacy behavior still works)
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: env var force-on still works ---"
+
+if FLOW_AGENTS_UTTERANCE_CHECK_ENABLED=true \
+   node "$HOOK" >"$TMPDIR_EVAL/forceon.out" 2>"$TMPDIR_EVAL/forceon.err" <<< "$INPUT_JSON"; then
+  _pass "FLOW_AGENTS_UTTERANCE_CHECK_ENABLED=true still enables the hook (legacy env var override)"
+else
+  _fail "hook with force-on env var should exit 0"
+fi
+
 # ---------------------------------------------------------------------------
 # Hook: pass-through with empty input
 # ---------------------------------------------------------------------------
@@ -84,8 +141,7 @@ if SA_DISABLED_HOOKS=post:utterance-check \
    node "$RUN_HOOK" post:utterance-check utterance-check.js standard,strict \
    >"$TMPDIR_EVAL/disabled-runner.out" 2>"$TMPDIR_EVAL/disabled-runner.err" <<< "$HOOK_INPUT"
 then
-  if cmp -s "$TMPDIR_EVAL/disabled-runner.out" <(printf '%s
-' "$HOOK_INPUT"); then
+  if cmp -s "$TMPDIR_EVAL/disabled-runner.out" <(printf '%s\n' "$HOOK_INPUT"); then
     _pass "run-hook.js passes input through when hook id is in SA_DISABLED_HOOKS"
   else
     _fail "run-hook.js disabled hook output did not match raw input"
@@ -94,6 +150,184 @@ else
   _fail "run-hook.js with disabled hook should exit 0"
 fi
 
+# ---------------------------------------------------------------------------
+# Hook: module.exports shape
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: module.exports contract ---"
+
+if node -e '
+  const h = require(process.argv[1]);
+  if (typeof h.run !== "function") { console.error("run missing"); process.exit(1); }
+  if (typeof h.extractUtteranceText !== "function") { console.error("extractUtteranceText missing"); process.exit(2); }
+  if (typeof h.findPackageRoot !== "function") { console.error("findPackageRoot missing"); process.exit(3); }
+  if (typeof h.findRepoRoot !== "function") { console.error("findRepoRoot missing"); process.exit(4); }
+  if (typeof h.loadRepoConfig !== "function") { console.error("loadRepoConfig missing"); process.exit(5); }
+  if (typeof h.resolvePolicy !== "function") { console.error("resolvePolicy missing"); process.exit(6); }
+' "$HOOK"; then
+  _pass "utterance-check hook exports run, extractUtteranceText, findPackageRoot, findRepoRoot, loadRepoConfig, resolvePolicy"
+else
+  _fail "utterance-check hook module.exports is missing expected functions"
+fi
+
+# ---------------------------------------------------------------------------
+# Hook: loadRepoConfig reads utteranceCheck from settings file
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: loadRepoConfig reads from context/settings/flow-agents-settings.json ---"
+
+if node -e '
+  const { loadRepoConfig } = require(process.argv[1]);
+  const fakeRepo = process.argv[2];
+  const cfg = loadRepoConfig(fakeRepo);
+  if (!cfg) { console.error("loadRepoConfig returned null for a repo with settings"); process.exit(1); }
+  if (cfg.enabled !== true) { console.error("expected enabled:true, got:", cfg.enabled); process.exit(2); }
+  if (cfg.mode !== "report") { console.error("expected mode:report, got:", cfg.mode); process.exit(3); }
+  if (cfg.extractor !== "reference") { console.error("expected extractor:reference, got:", cfg.extractor); process.exit(4); }
+' "$HOOK" "$FAKE_REPO"; then
+  _pass "loadRepoConfig correctly reads utteranceCheck fields from settings file"
+else
+  _fail "loadRepoConfig did not return expected config from settings file"
+fi
+
+# ---------------------------------------------------------------------------
+# Hook: loadRepoConfig returns null when settings file is absent
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: loadRepoConfig returns null when file is absent ---"
+
+MISSING_REPO="$TMPDIR_EVAL/no-settings-repo"
+mkdir -p "$MISSING_REPO"
+touch "$MISSING_REPO/AGENTS.md"
+
+if node -e '
+  const { loadRepoConfig } = require(process.argv[1]);
+  const cfg = loadRepoConfig(process.argv[2]);
+  if (cfg !== null) { console.error("expected null, got:", JSON.stringify(cfg)); process.exit(1); }
+' "$HOOK" "$MISSING_REPO"; then
+  _pass "loadRepoConfig returns null when context/settings/flow-agents-settings.json is absent"
+else
+  _fail "loadRepoConfig should return null for a repo without the settings file"
+fi
+
+# ---------------------------------------------------------------------------
+# Hook: resolvePolicy respects config enabled:false as default-off
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: resolvePolicy returns disabled when config has enabled:false ---"
+
+mkdir -p "$TMPDIR_EVAL/disabled-repo/context/settings"
+touch "$TMPDIR_EVAL/disabled-repo/AGENTS.md"
+cat > "$TMPDIR_EVAL/disabled-repo/context/settings/flow-agents-settings.json" <<'DCFG_EOF'
+{"schema_version":"1.0","utteranceCheck":{"enabled":false}}
+DCFG_EOF
+
+if node -e '
+  const { resolvePolicy } = require(process.argv[1]);
+  const policy = resolvePolicy(process.argv[2]);
+  if (policy.enabled !== false) { console.error("expected enabled:false, got:", policy.enabled); process.exit(1); }
+' "$HOOK" "$TMPDIR_EVAL/disabled-repo"; then
+  _pass "resolvePolicy returns {enabled:false} when config has enabled:false"
+else
+  _fail "resolvePolicy should return disabled policy when config has enabled:false"
+fi
+
+# ---------------------------------------------------------------------------
+# Hook: resolvePolicy applies strict mode from config
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: resolvePolicy applies mode:strict from config ---"
+
+mkdir -p "$TMPDIR_EVAL/strict-repo/context/settings"
+touch "$TMPDIR_EVAL/strict-repo/AGENTS.md"
+cat > "$TMPDIR_EVAL/strict-repo/context/settings/flow-agents-settings.json" <<'SCFG_EOF'
+{"schema_version":"1.0","utteranceCheck":{"enabled":true,"mode":"strict","extractor":"reference"}}
+SCFG_EOF
+
+if node -e '
+  const { resolvePolicy } = require(process.argv[1]);
+  const policy = resolvePolicy(process.argv[2]);
+  if (policy.enabled !== true) { console.error("expected enabled:true, got:", policy.enabled); process.exit(1); }
+  if (policy.mode !== "strict") { console.error("expected mode:strict, got:", policy.mode); process.exit(2); }
+' "$HOOK" "$TMPDIR_EVAL/strict-repo"; then
+  _pass "resolvePolicy applies mode:strict from config"
+else
+  _fail "resolvePolicy did not apply strict mode from config"
+fi
+
+# ---------------------------------------------------------------------------
+# Hook: resolvePolicy applies anthropic extractor from config
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: resolvePolicy applies extractor:anthropic from config ---"
+
+mkdir -p "$TMPDIR_EVAL/anthropic-repo/context/settings"
+touch "$TMPDIR_EVAL/anthropic-repo/AGENTS.md"
+cat > "$TMPDIR_EVAL/anthropic-repo/context/settings/flow-agents-settings.json" <<'ACFG_EOF'
+{"schema_version":"1.0","utteranceCheck":{"enabled":true,"mode":"report","extractor":"anthropic","model":"claude-haiku-4-5"}}
+ACFG_EOF
+
+if node -e '
+  const { resolvePolicy } = require(process.argv[1]);
+  const policy = resolvePolicy(process.argv[2]);
+  if (policy.extractor !== "anthropic") { console.error("expected extractor:anthropic, got:", policy.extractor); process.exit(1); }
+  if (policy.model !== "claude-haiku-4-5") { console.error("expected model:claude-haiku-4-5, got:", policy.model); process.exit(2); }
+' "$HOOK" "$TMPDIR_EVAL/anthropic-repo"; then
+  _pass "resolvePolicy applies extractor:anthropic and model from config"
+else
+  _fail "resolvePolicy did not apply anthropic extractor from config"
+fi
+
+# ---------------------------------------------------------------------------
+# Hook: resolvePolicy env var STRICT overrides report mode from config
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: env var STRICT overrides report mode in config ---"
+
+if node -e '
+  const { resolvePolicy } = require(process.argv[1]);
+  // Set env var before requiring resolvePolicy
+  process.env.FLOW_AGENTS_UTTERANCE_CHECK_STRICT = "true";
+  const policy = resolvePolicy(process.argv[2]);
+  delete process.env.FLOW_AGENTS_UTTERANCE_CHECK_STRICT;
+  if (policy.mode !== "strict") { console.error("expected mode:strict from env var, got:", policy.mode); process.exit(1); }
+' "$HOOK" "$FAKE_REPO"; then
+  _pass "FLOW_AGENTS_UTTERANCE_CHECK_STRICT=true env var overrides report mode in config"
+else
+  _fail "env var STRICT did not override report mode from config"
+fi
+
+# ---------------------------------------------------------------------------
+# Hook: extractUtteranceText extracts from PostToolUse and Stop events
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- hook: extractUtteranceText ---"
+
+if node -e '
+  const { extractUtteranceText } = require(process.argv[1]);
+  const postToolUse = { hook_event_name: "PostToolUse", tool_response: "The answer is 42." };
+  const text = extractUtteranceText(postToolUse);
+  if (text !== "The answer is 42.") { console.error("PostToolUse extract failed:", text); process.exit(1); }
+  const stopWithContent = { hook_event_name: "Stop", content: [{ type: "text", text: "Done!" }] };
+  const text2 = extractUtteranceText(stopWithContent);
+  if (text2 !== "Done!") { console.error("Stop content extract failed:", text2); process.exit(2); }
+  const emptyEvent = { hook_event_name: "PostToolUse" };
+  const text3 = extractUtteranceText(emptyEvent);
+  if (text3 !== null) { console.error("Empty event should return null, got:", text3); process.exit(3); }
+' "$HOOK"; then
+  _pass "extractUtteranceText handles PostToolUse, Stop content, and empty events"
+else
+  _fail "extractUtteranceText behavior was unexpected"
+fi
+
 # ---------------------------------------------------------------------------
 # CLI: build and test --not-configured
 # ---------------------------------------------------------------------------
@@ -148,6 +382,24 @@ else
   _fail "CLI --help should exit 0"
 fi
 
+# ---------------------------------------------------------------------------
+# CLI: --extractor flag appears in help
+# ---------------------------------------------------------------------------
+
+echo ""
+echo "--- cli: --extractor flag in help ---"
+
+if node "$ROOT/build/src/cli.js" utterance-check --help \
+   >"$TMPDIR_EVAL/help2.out" 2>"$TMPDIR_EVAL/help2.err"; then
+  if grep -q '\-\-extractor' "$TMPDIR_EVAL/help2.err"; then
+    _pass "CLI --help mentions --extractor flag"
+  else
+    _fail "CLI --help does not mention --extractor flag"
+  fi
+else
+  _fail "CLI --help should exit 0"
+fi
+
 # ---------------------------------------------------------------------------
 # CLI: missing --utterance exits non-zero
 # ---------------------------------------------------------------------------
@@ -203,58 +455,53 @@ else
 fi
 
 # ---------------------------------------------------------------------------
-# CLI: utterance check registers as a valid flow-agents command
-# ---------------------------------------------------------------------------
-
-echo ""
-echo "--- cli: command registration ---"
-
-if node "$ROOT/build/src/cli.js" commands 2>/dev/null | grep -q 'utterance-check'; then
-  _pass "utterance-check is registered as a flow-agents CLI command"
-else
-  _fail "utterance-check is not registered in flow-agents CLI commands"
-fi
-
-# ---------------------------------------------------------------------------
-# Hook: module.exports shape
+# CLI: --extractor anthropic without ANTHROPIC_API_KEY fails open (exit 0)
 # ---------------------------------------------------------------------------
 
 echo ""
-echo "--- hook: module.exports contract ---"
+echo "--- cli: anthropic extractor without API key fails open ---"
 
-if node -e '
-  const h = require(process.argv[1]);
-  if (typeof h.run !== "function") { console.error("run missing"); process.exit(1); }
-  if (typeof h.extractUtteranceText !== "function") { console.error("extractUtteranceText missing"); process.exit(2); }
-  if (typeof h.findPackageRoot !== "function") { console.error("findPackageRoot missing"); process.exit(3); }
-' "$HOOK"; then
-  _pass "utterance-check hook exports run, extractUtteranceText, findPackageRoot"
+# Run without ANTHROPIC_API_KEY set.
+# The CLI should emit not_configured JSON and exit 0 (fail open).
+if env -u ANTHROPIC_API_KEY \
+   node "$ROOT/build/src/cli.js" utterance-check check \
+   --utterance "The test coverage is 92%." \
+   --extractor anthropic \
+   >"$TMPDIR_EVAL/no-apikey.out" 2>"$TMPDIR_EVAL/no-apikey.err"
+then
+  status_val=$(node -e '
+    const r = JSON.parse(require("fs").readFileSync(process.argv[1],"utf8"));
+    console.log(r.status);
+  ' "$TMPDIR_EVAL/no-apikey.out" 2>/dev/null || echo "parse-error")
+  if [[ "$status_val" == "not_configured" ]]; then
+    _pass "CLI --extractor anthropic without ANTHROPIC_API_KEY emits not_configured and exits 0 (fail open)"
+  elif [[ "$status_val" == "ok" || "$status_val" == "error" ]]; then
+    # If survey is installed and somehow proceeded (shouldn't happen without key), still accept
+    _pass "CLI --extractor anthropic produced a valid report (status: $status_val)"
+  else
+    _fail "CLI --extractor anthropic without API key produced unexpected output (status: $status_val)"
+  fi
 else
-  _fail "utterance-check hook module.exports is missing expected functions"
+  exit_code=$?
+  # Exit 1 means survey not installed — that's a different fail-open path, acceptable
+  if [[ "$exit_code" -eq 1 ]]; then
+    _pass "CLI --extractor anthropic: survey not installed, exits 1 (not_configured)"
+  else
+    _fail "CLI --extractor anthropic without API key should exit 0 or 1 (fail open), got: $exit_code"
+  fi
 fi
 
 # ---------------------------------------------------------------------------
-# Hook: extractUtteranceText extracts from PostToolUse and Stop events
+# CLI: utterance check registers as a valid flow-agents command
 # ---------------------------------------------------------------------------
 
 echo ""
-echo "--- hook: extractUtteranceText ---"
+echo "--- cli: command registration ---"
 
-if node -e '
-  const { extractUtteranceText } = require(process.argv[1]);
-  const postToolUse = { hook_event_name: "PostToolUse", tool_response: "The answer is 42." };
-  const text = extractUtteranceText(postToolUse);
-  if (text !== "The answer is 42.") { console.error("PostToolUse extract failed:", text); process.exit(1); }
-  const stopWithContent = { hook_event_name: "Stop", content: [{ type: "text", text: "Done!" }] };
-  const text2 = extractUtteranceText(stopWithContent);
-  if (text2 !== "Done!") { console.error("Stop content extract failed:", text2); process.exit(2); }
-  const emptyEvent = { hook_event_name: "PostToolUse" };
-  const text3 = extractUtteranceText(emptyEvent);
-  if (text3 !== null) { console.error("Empty event should return null, got:", text3); process.exit(3); }
-' "$HOOK"; then
-  _pass "extractUtteranceText handles PostToolUse, Stop content, and empty events"
+if node "$ROOT/build/src/cli.js" commands 2>/dev/null | grep -q 'utterance-check'; then
+  _pass "utterance-check is registered as a flow-agents CLI command"
 else
-  _fail "extractUtteranceText behavior was unexpected"
+  _fail "utterance-check is not registered in flow-agents CLI commands"
 fi
 
 # ---------------------------------------------------------------------------

package/evals/run.sh

@@ -190,6 +190,8 @@ run_integration() {
   bash "$EVAL_DIR/integration/test_runtime_adapter_activation.sh" || result=1
   echo ""
   bash "$EVAL_DIR/integration/test_bundle_install.sh" || result=1
+  echo ""
+  bash "$EVAL_DIR/integration/test_bundle_lifecycle.sh" || result=1
   return $result
 }
 

package/evals/static/test_universal_bundles.sh

@@ -57,7 +57,7 @@ fi
 
 echo ""
 echo "--- Bundle Layout ---"
-for dir in "$DIST_DIR/kiro" "$DIST_DIR/claude-code" "$DIST_DIR/codex"; do
+for dir in "$DIST_DIR/kiro" "$DIST_DIR/claude-code" "$DIST_DIR/codex" "$DIST_DIR/opencode" "$DIST_DIR/pi"; do
   if [[ -d "$dir" ]]; then
     _pass "$(basename "$dir") bundle exists"
   else
@@ -80,6 +80,8 @@ codex_agents=$(find "$DIST_DIR/codex/.codex/agents" -maxdepth 1 -name '*.toml' 2
 [[ "$kiro_agents" == "$source_agents" ]] && _pass "Kiro agent count matches source ($kiro_agents)" || _fail "Kiro agent count mismatch: source=$source_agents dist=$kiro_agents"
 [[ "$claude_agents" == "$source_agents" ]] && _pass "Claude agent count matches source ($claude_agents)" || _fail "Claude agent count mismatch: source=$source_agents dist=$claude_agents"
 [[ "$codex_agents" == "$expected_codex_agents" ]] && _pass "Codex agent count matches source minus manifest exclusions ($codex_agents)" || _fail "Codex agent count mismatch: expected=$expected_codex_agents dist=$codex_agents"
+opencode_agents=$(find "$DIST_DIR/opencode/.opencode/agents" -maxdepth 1 -name '*.md' 2>/dev/null | wc -l | tr -d ' ')
+[[ "$opencode_agents" == "$source_agents" ]] && _pass "opencode agent count matches source ($opencode_agents)" || _fail "opencode agent count mismatch: source=$source_agents dist=$opencode_agents"
 
 echo ""
 echo "--- Kiro JSON ---"
@@ -226,9 +228,142 @@ else
   _fail "Codex hooks missing telemetry/policy lifecycle coverage or CODEX_HOME root resolution"
 fi
 
+echo ""
+echo "--- opencode Export Shape ---"
+if node - "$DIST_DIR/opencode/.opencode/agents" <<'NODE'
+const fs = require("node:fs");
+const path = require("node:path");
+const required = new Set(["description", "mode", "model"]);
+const validModes = new Set(["subagent", "primary", "all"]);
+for (const name of fs.readdirSync(process.argv[2]).filter((file) => file.endsWith(".md"))) {
+  const text = fs.readFileSync(path.join(process.argv[2], name), "utf8");
+  if (!text.startsWith("---\n")) throw new Error(`${name}: missing frontmatter start`);
+  const parts = text.split("\n---\n");
+  if (parts.length < 2) throw new Error(`${name}: missing frontmatter end`);
+  const fmLines = parts[0].replace("---\n", "").split(/\r?\n/).filter((line) => line.includes(":"));
+  const keys = new Set(fmLines.map((line) => line.split(":", 1)[0].trim()));
+  const missing = [...required].filter((key) => !keys.has(key));
+  if (missing.length) throw new Error(`${name}: missing frontmatter keys ${missing.join(", ")}`);
+  const modeMatch = fmLines.find((line) => line.trim().startsWith("mode:"));
+  if (modeMatch) {
+    const mode = modeMatch.split(":", 2)[1].trim();
+    if (!validModes.has(mode)) throw new Error(`${name}: invalid mode value: ${mode}`);
+  }
+  if (!parts.slice(1).join("\n---\n").trim()) throw new Error(`${name}: empty body`);
+}
+console.log("ok");
+NODE
+then
+  _pass "opencode agent markdown has valid YAML frontmatter with description, mode, model"
+else
+  _fail "opencode agent markdown frontmatter/shape check failed"
+fi
+
+if [[ -f "$DIST_DIR/opencode/.opencode/plugins/flow-agents.js" ]]; then
+  _pass "opencode bundle includes Flow Agents plugin"
+else
+  _fail "opencode bundle missing .opencode/plugins/flow-agents.js"
+fi
+
+if [[ -f "$DIST_DIR/opencode/opencode.json" ]]; then
+  if node - "$DIST_DIR/opencode/opencode.json" <<'NODE'
+const fs = require("node:fs");
+const data = JSON.parse(fs.readFileSync(process.argv[2], "utf8"));
+if (!data || typeof data !== "object") throw new Error("opencode.json must be an object");
+// opencode's config schema rejects non-array `instructions` and aborts
+// startup (caught by live acceptance smoke 2026-06-11). Pin the constraint.
+if ("instructions" in data && !Array.isArray(data.instructions)) {
+  throw new Error("opencode.json instructions must be an array of file paths when present");
+}
+console.log("ok");
+NODE
+  then
+    _pass "opencode.json is valid JSON and schema-safe (instructions array-or-absent)"
+  else
+    _fail "opencode.json is invalid or violates opencode config schema"
+  fi
+else
+  _fail "opencode bundle missing opencode.json"
+fi
+
+# Generated hook artifacts must PARSE in their host language. The pi live
+# smoke (2026-06-11) caught the generator emitting an unterminated string
+# (template-literal escaping) that pi's loader rejected at startup.
+if node --check "$DIST_DIR/opencode/.opencode/plugins/flow-agents.js" 2>/dev/null; then
+  _pass "generated opencode plugin parses as JavaScript"
+else
+  _fail "generated opencode plugin has a JavaScript syntax error"
+fi
+
+# Semantic errors (TS2xxx: unresolved modules/types) are expected without the
+# host's node_modules; only syntax-class errors (TS1xxx) mean a broken artifact.
+PI_TS_SYNTAX_ERRORS=$(npx tsc --ignoreConfig --noEmit --noResolve --skipLibCheck --target esnext --module esnext \
+    "$DIST_DIR/pi/.pi/extensions/flow-agents.ts" 2>&1 | grep -c "error TS1" || true)
+if [[ "$PI_TS_SYNTAX_ERRORS" -eq 0 ]]; then
+  _pass "generated pi extension parses as TypeScript (no TS1xxx syntax errors)"
+else
+  _fail "generated pi extension has $PI_TS_SYNTAX_ERRORS TypeScript syntax errors"
+fi
+
+if [[ -d "$DIST_DIR/opencode/.opencode/skills" ]] && [[ $(find "$DIST_DIR/opencode/.opencode/skills" -name "SKILL.md" | wc -l | tr -d ' ') -gt 0 ]]; then
+  _pass "opencode bundle includes skills in .opencode/skills/"
+else
+  _fail "opencode bundle missing skills in .opencode/skills/"
+fi
+
+echo ""
+echo "--- pi Export Shape ---"
+if [[ -f "$DIST_DIR/pi/.pi/extensions/flow-agents.ts" ]]; then
+  _pass "pi bundle includes Flow Agents extension"
+else
+  _fail "pi bundle missing .pi/extensions/flow-agents.ts"
+fi
+
+if [[ -d "$DIST_DIR/pi/.pi/skills" ]] && [[ $(find "$DIST_DIR/pi/.pi/skills" -name "SKILL.md" | wc -l | tr -d ' ') -gt 0 ]]; then
+  _pass "pi bundle includes skills in .pi/skills/"
+else
+  _fail "pi bundle missing skills in .pi/skills/"
+fi
+
+if node - "$DIST_DIR/pi/.pi/extensions/flow-agents.ts" <<'NODE'
+const fs = require("node:fs");
+const text = fs.readFileSync(process.argv[2], "utf8");
+if (!text.includes("pi-hook-adapter.js")) throw new Error("pi extension does not reference pi-hook-adapter.js");
+if (!text.includes("pi-telemetry-hook.js")) throw new Error("pi extension does not reference pi-telemetry-hook.js");
+if (!text.includes("workflow-steering.js")) throw new Error("pi extension missing workflow-steering.js reference");
+if (!text.includes("config-protection.js")) throw new Error("pi extension missing config-protection.js reference");
+if (!text.includes("stop-goal-fit.js")) throw new Error("pi extension missing stop-goal-fit.js reference");
+if (!text.includes("before_agent_start")) throw new Error("pi extension missing before_agent_start event handler");
+if (!text.includes("tool_call")) throw new Error("pi extension missing tool_call event handler");
+console.log("ok");
+NODE
+then
+  _pass "pi extension references correct hook adapters and event handlers"
+else
+  _fail "pi extension is missing required hook adapter or event handler references"
+fi
+
+if node - "$DIST_DIR/opencode/.opencode/plugins/flow-agents.js" <<'NODE'
+const fs = require("node:fs");
+const text = fs.readFileSync(process.argv[2], "utf8");
+if (!text.includes("opencode-hook-adapter.js")) throw new Error("opencode plugin does not reference opencode-hook-adapter.js");
+if (!text.includes("opencode-telemetry-hook.js")) throw new Error("opencode plugin does not reference opencode-telemetry-hook.js");
+if (!text.includes("workflow-steering.js")) throw new Error("opencode plugin missing workflow-steering.js reference");
+if (!text.includes("config-protection.js")) throw new Error("opencode plugin missing config-protection.js reference");
+if (!text.includes("stop-goal-fit.js")) throw new Error("opencode plugin missing stop-goal-fit.js reference");
+if (!text.includes("session.created")) throw new Error("opencode plugin missing session.created event handler");
+if (!text.includes("tool.execute.before")) throw new Error("opencode plugin missing tool.execute.before event handler");
+console.log("ok");
+NODE
+then
+  _pass "opencode plugin references correct hook adapters and event handlers"
+else
+  _fail "opencode plugin is missing required hook adapter or event handler references"
+fi
+
 echo ""
 echo "--- Shared Task Dirs ---"
-for dir in "$DIST_DIR/claude-code/.flow-agents" "$DIST_DIR/codex/.flow-agents"; do
+for dir in "$DIST_DIR/claude-code/.flow-agents" "$DIST_DIR/codex/.flow-agents" "$DIST_DIR/opencode/.flow-agents" "$DIST_DIR/pi/.flow-agents"; do
   if [[ -d "$dir" ]]; then
     _pass "$(realpath "$dir" 2>/dev/null || echo "$dir") exists"
   else