ultimate-pi 0.11.0 → 0.12.0

package/.pi/scripts/validate-plan-dag.mjs

@@ -0,0 +1,258 @@
+#!/usr/bin/env node
+/**
+ * validate-plan-dag — deterministic ExecutionPlan DAG checks (YAML packet in).
+ */
+
+import { access } from "node:fs/promises";
+import { constants } from "node:fs";
+import { dirname, join, resolve } from "node:path";
+import { fileURLToPath } from "node:url";
+import { readYamlFile, writeYamlFile } from "../lib/harness-yaml.mjs";
+
+const ROOT = join(dirname(fileURLToPath(import.meta.url)), "..", "..");
+
+const MINIMUMS = {
+	low: { phases: 2, work_items: 5, acceptance_checks: 3, risks: 0 },
+	med: { phases: 3, work_items: 8, acceptance_checks: 5, risks: 3 },
+	high: { phases: 4, work_items: 12, acceptance_checks: 8, risks: 3 },
+};
+
+function fail(msg) {
+	console.error(`validate-plan-dag: FAIL: ${msg}`);
+	process.exit(1);
+}
+
+function ok(msg) {
+	console.log(`  ✓ ${msg}`);
+}
+
+function topoSort(workItems) {
+	const ids = new Set(workItems.map((w) => w.work_item_id));
+	const adj = new Map();
+	for (const w of workItems) {
+		adj.set(w.work_item_id, (w.depends_on ?? []).filter((d) => ids.has(d)));
+	}
+	const visited = new Set();
+	const stack = new Set();
+	const order = [];
+	const cycles = [];
+
+	function dfs(n, path) {
+		if (stack.has(n)) {
+			cycles.push([...path, n]);
+			return;
+		}
+		if (visited.has(n)) return;
+		visited.add(n);
+		stack.add(n);
+		for (const d of adj.get(n) ?? []) dfs(d, [...path, n]);
+		stack.delete(n);
+		order.push(n);
+	}
+
+	for (const id of ids) dfs(id, []);
+	order.reverse();
+	return { order, cycles };
+}
+
+function computeCriticalPath(workItems) {
+	const ids = new Set(workItems.map((w) => w.work_item_id));
+	const len = new Map();
+	for (const w of workItems) len.set(w.work_item_id, 0);
+	let changed = true;
+	while (changed) {
+		changed = false;
+		for (const w of workItems) {
+			const deps = (w.depends_on ?? []).filter((d) => ids.has(d));
+			const base = deps.length === 0 ? 0 : Math.max(...deps.map((d) => len.get(d) ?? 0)) + 1;
+			if (base > (len.get(w.work_item_id) ?? 0)) {
+				len.set(w.work_item_id, base);
+				changed = true;
+			}
+		}
+	}
+	const maxLen = Math.max(0, ...len.values());
+	const end = workItems.filter((w) => len.get(w.work_item_id) === maxLen).map((w) => w.work_item_id);
+	// Backtrack one longest path
+	const path = [];
+	let cur = end[0];
+	if (!cur) return [];
+	const byId = new Map(workItems.map((w) => [w.work_item_id, w]));
+	while (cur) {
+		path.unshift(cur);
+		const w = byId.get(cur);
+		const deps = (w?.depends_on ?? []).filter((d) => ids.has(d));
+		if (deps.length === 0) break;
+		cur = deps.reduce((a, b) => ((len.get(a) ?? 0) >= (len.get(b) ?? 0) ? a : b));
+	}
+	return path;
+}
+
+export function validateExecutionPlan(packet, projectRoot = ROOT) {
+	const errors = [];
+	const ep = packet.execution_plan;
+	if (!ep) {
+		errors.push("execution_plan required");
+		return { status: "fail", errors, report: null };
+	}
+
+	const risk = packet.risk_level ?? "med";
+	const min = MINIMUMS[risk] ?? MINIMUMS.med;
+	const phases = ep.phases ?? [];
+	const workItems = ep.work_items ?? [];
+	const conflicts = [];
+
+	if (phases.length < min.phases) {
+		errors.push(`need >= ${min.phases} phases for risk ${risk}`);
+	}
+	if (workItems.length < min.work_items) {
+		errors.push(`need >= ${min.work_items} work_items for risk ${risk}`);
+	}
+	const ac = packet.acceptance_checks ?? [];
+	if (ac.length < min.acceptance_checks) {
+		errors.push(`need >= ${min.acceptance_checks} acceptance_checks`);
+	}
+	if ((ep.risk_register ?? []).length < min.risks) {
+		errors.push(`need >= ${min.risks} risks for risk ${risk}`);
+	}
+
+	const phaseIds = new Set(phases.map((p) => p.phase_id));
+	const phaseIndex = new Map(phases.map((p, i) => [p.phase_id, i]));
+	const wiIds = new Set(workItems.map((w) => w.work_item_id));
+
+	for (const p of phases) {
+		if (!p.exit_criteria?.length) errors.push(`phase ${p.phase_id} missing exit_criteria`);
+		if (!p.work_item_ids?.length) errors.push(`phase ${p.phase_id} has no work items`);
+	}
+
+	const wiInPhase = new Set();
+	for (const w of workItems) {
+		if (!phaseIds.has(w.phase_id)) {
+			errors.push(`work_item ${w.work_item_id} unknown phase_id`);
+		}
+		wiInPhase.add(w.work_item_id);
+		for (const d of w.depends_on ?? []) {
+			if (!wiIds.has(d)) errors.push(`work_item ${w.work_item_id} depends_on missing ${d}`);
+		}
+		if (!w.non_code && (!w.files || w.files.length === 0)) {
+			errors.push(`work_item ${w.work_item_id} needs files[] or non_code: true`);
+		}
+	}
+
+	for (const p of phases) {
+		for (const wid of p.work_item_ids ?? []) {
+			if (!wiIds.has(wid)) errors.push(`phase ${p.phase_id} references missing ${wid}`);
+		}
+	}
+
+	const { order, cycles } = topoSort(workItems);
+	if (cycles.length) errors.push(`cycle detected: ${JSON.stringify(cycles[0])}`);
+
+	// File conflicts
+	for (let i = 0; i < workItems.length; i++) {
+		for (let j = i + 1; j < workItems.length; j++) {
+			const a = workItems[i];
+			const b = workItems[j];
+			const filesA = new Set(a.files ?? []);
+			const overlap = (b.files ?? []).filter((f) => filesA.has(f));
+			if (overlap.length === 0) continue;
+			const reachable = (from, to, seen = new Set()) => {
+				if (from === to) return true;
+				if (seen.has(from)) return false;
+				seen.add(from);
+				const w = workItems.find((x) => x.work_item_id === from);
+				for (const d of w?.depends_on ?? []) {
+					if (reachable(d, to, seen)) return true;
+				}
+				return false;
+			};
+			if (!reachable(a.work_item_id, b.work_item_id) && !reachable(b.work_item_id, a.work_item_id)) {
+				if ((phaseIndex.get(a.phase_id) ?? 0) === (phaseIndex.get(b.phase_id) ?? 0)) {
+					conflicts.push(
+						`file overlap ${overlap.join(",")} between ${a.work_item_id} and ${b.work_item_id} without dependency`,
+					);
+				}
+			}
+		}
+	}
+
+	const computedCp = computeCriticalPath(workItems);
+	const authorCp = ep.schedule_metadata?.critical_path_work_item_ids ?? [];
+	if (computedCp.length >= 3 && authorCp.length) {
+		const same =
+			authorCp.length === computedCp.length &&
+			authorCp.every((id, i) => id === computedCp[i]);
+		if (!same) {
+			errors.push(
+				`critical_path mismatch author=${authorCp.join("→")} computed=${computedCp.join("→")}`,
+			);
+		}
+	}
+
+	const acIds = new Set(
+		ac.map((c) => (typeof c === "string" ? c : c.id)).filter(Boolean),
+	);
+	for (const w of workItems) {
+		for (const acid of w.acceptance_check_ids ?? []) {
+			if (!acIds.has(acid)) errors.push(`${w.work_item_id} references orphan ${acid}`);
+		}
+	}
+	for (const acid of acIds) {
+		const used = workItems.some((w) => (w.acceptance_check_ids ?? []).includes(acid));
+		if (!used) errors.push(`orphan acceptance check ${acid}`);
+	}
+
+	const status = errors.length === 0 && conflicts.length === 0 ? "pass" : "fail";
+	const report = {
+		status,
+		topological_order: order,
+		cycles,
+		conflicts: [...conflicts, ...errors],
+	};
+	return { status, errors: [...errors, ...conflicts], report };
+}
+
+async function main() {
+	const args = process.argv.slice(2);
+	let packetPath = null;
+	let writeBack = false;
+	for (let i = 0; i < args.length; i++) {
+		if (args[i] === "--packet" && args[i + 1]) packetPath = args[++i];
+		else if (args[i] === "--write") writeBack = true;
+	}
+
+	if (!packetPath) {
+		console.error("Usage: validate-plan-dag.mjs --packet <plan-packet.yaml> [--write]");
+		process.exit(2);
+	}
+
+	const abs = resolve(packetPath);
+	try {
+		await access(abs, constants.R_OK);
+	} catch {
+		fail(`cannot read ${abs}`);
+	}
+
+	const packet = await readYamlFile(abs);
+	const { status, errors, report } = validateExecutionPlan(packet, dirname(abs));
+
+	if (writeBack && report && packet.execution_plan) {
+		packet.execution_plan.dag_validation = {
+			status: report.status,
+			topological_order: report.topological_order,
+			cycles: report.cycles,
+			conflicts: report.conflicts,
+		};
+		await writeYamlFile(abs, packet);
+	}
+
+	if (status !== "pass") {
+		for (const e of errors) console.error(`  - ${e}`);
+		fail("validation failed");
+	}
+	ok(`DAG validation pass (${report.topological_order.length} work items)`);
+}
+
+if (process.argv[1] && fileURLToPath(import.meta.url) === resolve(process.argv[1])) {
+	main();
+}

package/.pi/scripts/vendor-sync-pi-subagents.sh

@@ -0,0 +1,19 @@
+#!/usr/bin/env bash
+# Re-fetch upstream pi-subagents from narumiruna/pi-extensions.
+set -euo pipefail
+ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/../.." && pwd)"
+VEND="$ROOT/vendor/pi-subagents"
+BASE="https://raw.githubusercontent.com/narumiruna/pi-extensions/main/extensions/pi-subagents"
+
+mkdir -p "$VEND/src"
+curl -fsSL "$BASE/LICENSE" -o "$VEND/LICENSE"
+curl -fsSL "$BASE/src/subagents.ts" -o "$VEND/src/subagents.upstream.ts"
+
+# Preserve ultimate-pi harness extensions (agents.ts, harness patches applied to subagents.ts manually or via merge).
+if [[ ! -f "$VEND/src/agents.ts" ]]; then
+	curl -fsSL "$BASE/src/agents.ts" -o "$VEND/src/agents.ts"
+fi
+
+sed -i 's/from "typebox"/from "@sinclair\/typebox"/g' "$VEND/src/subagents.upstream.ts" 2>/dev/null || true
+
+echo "Fetched upstream into $VEND/src/subagents.upstream.ts — merge harness changes into subagents.ts before commit."

package/CHANGELOG.md

@@ -4,6 +4,18 @@ All notable changes to this project are documented in this file.
 
 ## [Unreleased]
 
+## [v0.12.0] — 2026-05-18
+
+### ✨ Features
+
+- **Harness subagents:** vendor `pi-subagents`; spawn budget, precheck, and policy bridge.
+- **Router spawn auth:** forward concrete provider/model + API key to subprocesses using `--no-extensions` (fixes planning scout 401 with `router/auto`).
+- **Plan review gate:** debate orchestration, execution-plan schemas, `write_harness_yaml`, plan-phase smoke fixture (ADR 0035).
+
+### 🔧 Chores
+
+- Stop tracking harness runtime and local graphify artifacts in git.
+
 ## [v0.11.0] — 2026-05-17
 
 ### ✨ Features

package/THIRD_PARTY_NOTICES.md

@@ -14,3 +14,11 @@
 - **License:** MIT (see upstream repository)  
 - **Pinned revision:** See [vendor/pi-vcc/UPSTREAM_PIN.md](vendor/pi-vcc/UPSTREAM_PIN.md)  
 - ultimate-pi loads it from [`vendor/pi-vcc`](vendor/pi-vcc) via [`.pi/extensions/ultimate-pi-vcc.ts`](.pi/extensions/ultimate-pi-vcc.ts). Harness configuration is env-only: `HARNESS_VCC_COMPACTION`, `HARNESS_VCC_DEBUG` ([`.pi/extensions/lib/harness-vcc-settings.ts`](.pi/extensions/lib/harness-vcc-settings.ts)). Maintainer refresh: `npm run vendor:sync-vcc`.
+
+## pi-subagents (vendored)
+
+- **Project:** https://github.com/narumiruna/pi-extensions (`extensions/pi-subagents`)  
+- **npm:** `@narumitw/pi-subagents@0.1.26`  
+- **License:** MIT ([vendor/pi-subagents/LICENSE](vendor/pi-subagents/LICENSE))  
+- **Pinned revision:** See [vendor/pi-subagents/UPSTREAM_PIN.md](vendor/pi-subagents/UPSTREAM_PIN.md)  
+- ultimate-pi loads it from [`vendor/pi-subagents`](vendor/pi-subagents) via [`.pi/extensions/harness-subagents.ts`](.pi/extensions/harness-subagents.ts) with harness discovery, spawn gates, and subprocess env. Maintainer refresh: `npm run vendor:sync-subagents`.

package/biome.json

@@ -14,8 +14,8 @@
 			"!graphify-books-out/**/*",
 			"!vendor/**/*",
 			"!.pi/harness/active-run.json",
-			"!.pi/harness/runs/**/run-context.json",
-			"!.pi/harness/runs/**/plan-packet.json"
+			"!.pi/harness/runs/**/run-context.yaml",
+			"!.pi/harness/runs/**/plan-packet.yaml"
 		]
 	},
 	"formatter": {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ultimate-pi",
-	"version": "0.11.0",
+	"version": "0.12.0",
 	"description": "Ultimate AI coding harness for pi.dev — extensible skills, Obsidian wiki knowledge layer, compressed context, deterministic output",
 	"keywords": [
 		"pi-package",
@@ -73,16 +73,17 @@
 		"@earendil-works/pi-coding-agent": "*"
 	},
 	"scripts": {
-		"check:ts": "tsc --noEmit --target ES2023 --lib ES2023 --moduleResolution nodenext --module nodenext --skipLibCheck .pi/extensions/00-ultimate-pi-system-prompt.ts .pi/lib/harness-run-context.ts .pi/lib/harness-ui-state.ts .pi/extensions/harness-run-context.ts .pi/extensions/lib/harness-vcc-settings.ts .pi/extensions/dotenv-loader.ts .pi/extensions/lib/posthog-node.d.ts .pi/extensions/lib/harness-posthog.ts .pi/extensions/lib/harness-paths.ts .pi/extensions/pi-model-router-harness.ts .pi/extensions/provider-payload-sanitize.ts .pi/extensions/harness-telemetry.ts .pi/extensions/harness-ask-user.ts .pi/extensions/harness-plan-approval.ts .pi/extensions/lib/ask-user/schema.ts .pi/extensions/lib/ask-user/types.ts .pi/extensions/lib/ask-user/validate.ts .pi/extensions/lib/ask-user/dialog.ts .pi/extensions/lib/ask-user/fallback.ts .pi/extensions/lib/ask-user/render.ts .pi/extensions/lib/plan-approval/types.ts .pi/extensions/lib/plan-approval/schema.ts .pi/extensions/lib/plan-approval/validate.ts .pi/extensions/lib/plan-approval/format-plan.ts .pi/extensions/lib/plan-approval/dialog.ts .pi/extensions/lib/plan-approval/fallback.ts .pi/extensions/lib/plan-approval/render.ts .pi/extensions/lib/plan-approval/create-plan.ts .pi/extensions/lib/harness-subagents/parent-harness-ui-bridge.ts .pi/extensions/lib/harness-subagents/parent-harness-ui-hooks.ts .pi/extensions/lib/harness-subagents/parent-ask-user-bridge.ts .pi/extensions/trace-recorder.ts .pi/extensions/observation-bus.ts .pi/extensions/drift-monitor.ts .pi/extensions/policy-gate.ts .pi/extensions/budget-guard.ts .pi/extensions/debate-orchestrator.ts .pi/extensions/harness-live-widget.ts .pi/extensions/sentrux-rules-sync.ts .pi/extensions/custom-header.ts .pi/extensions/lib/harness-subagents/agent-loader.ts .pi/extensions/lib/harness-subagents/agent-parser.ts .pi/extensions/lib/harness-subagents/agent-manifest.ts .pi/extensions/lib/harness-subagents/blackboard.ts .pi/extensions/lib/harness-subagents/blackboard-tool.ts .pi/extensions/lib/harness-subagents/spawn-policy.ts .pi/extensions/lib/harness-subagents/types-blackboard.ts .pi/extensions/harness-web-tools.ts .pi/extensions/harness-web-guard.ts .pi/extensions/lib/harness-web/run-cli.ts",
+		"check:ts": "tsc --noEmit --target ES2023 --lib ES2023 --moduleResolution nodenext --module nodenext --skipLibCheck .pi/extensions/custom-system-prompt.ts .pi/lib/harness-run-context.ts .pi/lib/harness-ui-state.ts .pi/extensions/harness-run-context.ts .pi/extensions/lib/harness-vcc-settings.ts .pi/extensions/dotenv-loader.ts .pi/extensions/lib/posthog-node.d.ts .pi/extensions/lib/harness-posthog.ts .pi/extensions/lib/harness-paths.ts .pi/extensions/pi-model-router-harness.ts .pi/extensions/provider-payload-sanitize.ts .pi/extensions/harness-telemetry.ts .pi/extensions/harness-ask-user.ts .pi/extensions/harness-plan-approval.ts .pi/extensions/lib/ask-user/schema.ts .pi/extensions/lib/ask-user/types.ts .pi/extensions/lib/ask-user/validate.ts .pi/extensions/lib/ask-user/dialog.ts .pi/extensions/lib/ask-user/fallback.ts .pi/extensions/lib/ask-user/render.ts .pi/extensions/lib/plan-approval/types.ts .pi/extensions/lib/plan-approval/schema.ts .pi/extensions/lib/plan-approval/validate.ts .pi/extensions/lib/plan-approval/format-plan.ts .pi/extensions/lib/plan-approval/dialog.ts .pi/extensions/lib/plan-approval/fallback.ts .pi/extensions/lib/plan-approval/render.ts .pi/extensions/lib/plan-approval/create-plan.ts .pi/extensions/harness-subagents.ts .pi/extensions/lib/harness-subagents-bridge.ts .pi/extensions/lib/harness-subagent-auth.ts .pi/extensions/lib/harness-subagent-policy.ts .pi/extensions/lib/harness-subagent-precheck.ts .pi/extensions/lib/harness-spawn-budget.ts .pi/extensions/lib/spawn-policy.ts vendor/pi-subagents/src/agents.ts vendor/pi-subagents/src/subagents.ts .pi/extensions/trace-recorder.ts .pi/extensions/observation-bus.ts .pi/extensions/drift-monitor.ts .pi/extensions/policy-gate.ts .pi/extensions/budget-guard.ts .pi/extensions/debate-orchestrator.ts .pi/extensions/harness-live-widget.ts .pi/extensions/sentrux-rules-sync.ts .pi/extensions/custom-header.ts .pi/extensions/harness-web-tools.ts .pi/extensions/harness-web-guard.ts .pi/extensions/lib/harness-web/run-cli.ts",
 		"vendor:sync-router": "bash .pi/scripts/vendor-sync-pi-model-router.sh",
 		"vendor:sync-vcc": "bash .pi/scripts/vendor-sync-pi-vcc.sh",
+		"vendor:sync-subagents": "bash .pi/scripts/vendor-sync-pi-subagents.sh",
 		"release": "bash .pi/scripts/release.sh",
 		"lint": "biome check",
 		"lint:fix": "biome check --fix",
 		"format": "biome format --write",
 		"format:check": "biome format",
 		"prepare": "lefthook install",
-		"test": "node --test test/harness-verify.test.mjs test/harness-ask-user.test.mjs test/harness-subagents-loader.test.mjs test/harness-subagents-import-path.test.mjs test/sentrux-rules-sync.test.mjs test/harness-budget-guard.test.mjs && npx -y tsx --test test/harness-vcc-settings.test.ts test/harness-plan-phase-policy.test.mjs test/harness-subagent-policy.test.mjs test/harness-turn-routing.test.mjs test/plan-approval-format.test.mjs test/plan-approval-dialog.test.mjs test/plan-approval-sync.test.mjs test/plan-create-plan.test.mjs test/plan-review-format.test.mjs",
+		"test": "node --test test/harness-verify.test.mjs test/harness-ask-user.test.mjs test/harness-subagents-loader.test.mjs test/harness-subagent-precheck.test.mjs test/sentrux-rules-sync.test.mjs test/harness-budget-guard.test.mjs && node .pi/harness/evals/smoke/smoke-harness-plan.mjs --fixture && npx -y tsx --test test/harness-vcc-settings.test.ts test/harness-plan-phase-policy.test.mjs test/harness-subagent-policy.test.mjs test/harness-spawn-budget.test.mjs test/harness-turn-routing.test.mjs test/plan-approval-format.test.mjs test/plan-approval-dialog.test.mjs test/plan-approval-sync.test.mjs test/plan-create-plan.test.mjs test/plan-review-format.test.mjs test/debate-plan-phase.test.mjs",
 		"test:vcc": "npx -y tsx --test vendor/pi-vcc/tests/*.test.ts",
 		"harness:sentrux-bootstrap": "node .pi/scripts/harness-sentrux-bootstrap.mjs",
 		"harness:sentrux-sync": "node .pi/scripts/sentrux-rules-sync.mjs --force",
@@ -96,7 +97,8 @@
 		"@earendil-works/pi-tui": "0.74.1",
 		"@sinclair/typebox": "^0.34.49",
 		"lefthook": "2.1.6",
-		"typescript": "^6.0.3"
+		"typescript": "^6.0.3",
+		"yaml": "^2.8.0"
 	},
 	"dependencies": {
 		"@posthog/pi": "latest",

package/.pi/agents/harness/planner.md

@@ -1,13 +0,0 @@
----
-description: "DEPRECATED — relocated to harness/planning/. Do not spawn harness/planner."
-tools: read
-extensions: false
-max_turns: 1
-inherit_context: false
----
-
-**Relocated:** plan-phase agents live under `harness/planning/` (scouts, plan-adversary).
-
-Use `/harness-plan` in the parent session — do **not** spawn `harness/planner` or `harness/planning/planner`.
-
-See `.pi/agents/harness/planning/` and `.pi/prompts/harness-plan.md`.

package/.pi/agents/harness/planning/hypothesis-eval.md

@@ -1,59 +0,0 @@
----
-description: Plan-phase blind hypothesis self-evaluation (read-only).
-tools: read, grep, find, ls
-disallowed_tools: write, edit, bash, ask_user, approve_plan, create_plan, Agent
-extensions: false
-thinking: medium
-max_turns: 12
-inherit_context: false
----
-
-You are the **Harness hypothesis evaluator** — blind self-evaluation only.
-
-## Mission
-
-Score the hypothesis brief on research quality dimensions. You do **not** revise the hypothesis, build PlanPacket, or mutate anything.
-
-## Input (strict)
-
-You receive **only**:
-
-- Original task statement
-- `PlanHypothesisBrief` JSON
-
-You must **not** use decomposition, scout findings, PlanPacket, or adversary output even if present in the prompt — ignore them.
-
-## Scoring rubric
-
-| Dimension | 90+ | 70–89 | <50 |
-|-----------|-----|-------|--------|
-| Novelty | Reframes problem | Novel combo | Known approach |
-| Coherence | Implementation-ready | Minor gaps | Vague |
-| Testability | Fully specified experiment | Clear direction | Unfalsifiable |
-| Impact | Field-changing | Meaningful | Incremental |
-
-**Relevance**: Does the primary hypothesis address the original task? (`passes` true/false + rationale).
-
-Set `revision_recommended: true` when **testability** score < 70 or **relevance.passes** is false.
-
-## Output (required JSON block)
-
-```json
-{
-  "schema_version": "1.0.0",
-  "dimensions": {
-    "novelty": { "score": 75, "rationale": "…" },
-    "coherence": { "score": 80, "rationale": "…" },
-    "testability": { "score": 85, "rationale": "…" },
-    "impact": { "score": 70, "rationale": "…" }
-  },
-  "relevance": {
-    "passes": true,
-    "rationale": "…"
-  },
-  "revision_recommended": false,
-  "human_summary": "…"
-}
-```
-
-Match `PlanHypothesisEval` (`.pi/harness/specs/plan-hypothesis-eval.schema.json`).

package/.pi/agents/harness/planning/planner.md

@@ -1,20 +0,0 @@
----
-description: "DEPRECATED — do not spawn. Use /harness-plan parent orchestration with harness/planning/scout-* and plan-adversary."
-tools: read
-extensions: false
-max_turns: 1
-inherit_context: false
----
-
-This agent is **deprecated**. `/harness-plan` no longer spawns `harness/planning/planner`.
-
-The parent orchestrator runs:
-
-- `harness/planning/scout-graphify`
-- `harness/planning/scout-structure`
-- `harness/planning/scout-semantic` (skipped when `--quick`)
-- `harness/planning/plan-adversary`
-
-Then the parent calls `ask_user`, `approve_plan`, and `create_plan` in the main session.
-
-Do not use this file except for manifest compatibility or project overrides.

package/.pi/extensions/lib/harness-subagents/agent-loader.ts

@@ -1,126 +0,0 @@
-/**
- * Recursive discovery: $UP_PKG/.pi/agents/** + project .pi/agents/** overrides.
- */
-
-import { createHash } from "node:crypto";
-import { type Dirent, existsSync, readdirSync, readFileSync } from "node:fs";
-import { join, relative } from "node:path";
-import { getAgentDir } from "@earendil-works/pi-coding-agent";
-import { parseAgentMarkdown } from "./agent-parser.js";
-import type { AgentConfig } from "./vendored/types.js";
-
-export type AgentSource = "package" | "project" | "global";
-
-export interface DiscoveredAgentFile {
-	id: string;
-	path: string;
-	source: AgentSource;
-	content: string;
-}
-
-/** Reject path traversal and unsafe ids. */
-export function isSafeAgentId(id: string): boolean {
-	if (!id || id.includes("..") || id.startsWith("/") || id.includes("\\")) {
-		return false;
-	}
-	return /^[a-zA-Z0-9][a-zA-Z0-9/_-]*$/.test(id);
-}
-
-function walkAgentsDir(
-	rootDir: string,
-	source: AgentSource,
-	out: Map<string, DiscoveredAgentFile>,
-): void {
-	if (!existsSync(rootDir)) return;
-
-	const stack: string[] = [rootDir];
-	while (stack.length > 0) {
-		const dir = stack.pop()!;
-		let entries: Dirent[];
-		try {
-			entries = readdirSync(dir, { withFileTypes: true });
-		} catch {
-			continue;
-		}
-
-		for (const entry of entries) {
-			const full = join(dir, entry.name);
-			if (entry.isDirectory()) {
-				stack.push(full);
-				continue;
-			}
-			if (!entry.name.endsWith(".md")) continue;
-
-			const rel = relative(rootDir, full).replace(/\\/g, "/");
-			const id = rel.replace(/\.md$/i, "");
-			if (!isSafeAgentId(id)) continue;
-
-			let content: string;
-			try {
-				content = readFileSync(full, "utf-8");
-			} catch {
-				continue;
-			}
-
-			out.set(id, { id, path: full, source, content });
-		}
-	}
-}
-
-/**
- * Discover agent files from package, global, and project (low → high priority).
- */
-export function discoverAgentFiles(
-	cwd: string,
-	packageRoot: string,
-): Map<string, DiscoveredAgentFile> {
-	const files = new Map<string, DiscoveredAgentFile>();
-
-	const packageAgents = join(packageRoot, ".pi", "agents");
-	const globalDir = join(getAgentDir(), "agents");
-	const projectDir = join(cwd, ".pi", "agents");
-
-	walkAgentsDir(packageAgents, "package", files);
-	walkAgentsDir(globalDir, "global", files);
-	walkAgentsDir(projectDir, "project", files);
-
-	return files;
-}
-
-/** Load merged AgentConfig map (project overrides package for same id). */
-export function loadHarnessAgents(
-	cwd: string,
-	packageRoot: string,
-): Map<string, AgentConfig> {
-	const agents = new Map<string, AgentConfig>();
-	for (const file of discoverAgentFiles(cwd, packageRoot).values()) {
-		agents.set(file.id, parseAgentMarkdown(file.id, file.content, file.source));
-	}
-	return agents;
-}
-
-export function sha256Content(content: string): string {
-	return createHash("sha256").update(content, "utf8").digest("hex");
-}
-
-/** Package-only manifest entries (path → hash). */
-export function loadPackageAgentHashes(
-	packageRoot: string,
-): Map<string, { path: string; sha256: string }> {
-	const packageAgents = join(packageRoot, ".pi", "agents");
-	const out = new Map<string, { path: string; sha256: string }>();
-	const files = new Map<string, DiscoveredAgentFile>();
-	walkAgentsDir(packageAgents, "package", files);
-	for (const f of files.values()) {
-		out.set(f.id, { path: f.path, sha256: sha256Content(f.content) });
-	}
-	return out;
-}
-
-/** Legacy hook used by pi-subagents custom-agents.ts replacement. */
-export function loadCustomAgents(
-	cwd: string,
-	packageRoot: string,
-): Map<string, AgentConfig> {
-	return loadHarnessAgents(cwd, packageRoot);
-}