ultimate-pi 0.7.0 → 0.8.0

package/README.md

@@ -45,7 +45,7 @@ If something blocks, inspect status (no run id needed):
 | `/harness-auto "<task>"` | End-to-end pipeline (recommended) |
 | `/harness-plan "<task>"` | Create or **revise** the active plan in context (no plan path to copy) |
 | `/harness-run` | Execute the active plan from context (**no `--plan`** on happy path) |
-| `/harness-eval` | Eval for active run (optional `--run`; **new session** after execute) |
+| `/harness-eval` | Eval for active run (optional `--run`; spawns isolated `harness/evaluator`) |
 | `/harness-review` | Independent review (optional `--run`) |
 | `/harness-critic` | Adversarial review (optional `--run`) |
 | `/harness-trace` | Trace summary (optional `--run`) |
@@ -63,7 +63,6 @@ Use this when you want each step separate:
 ```text
 /harness-plan "your task"
 /harness-run
-# New Pi session (review isolation):
 /harness-eval
 /harness-review
 /harness-critic
@@ -78,7 +77,7 @@ Recovery: `--run` and `--plan` remain for scripts; `/harness-use-run` and `/harn
 - **System prompt** — [`.pi/extensions/00-ultimate-pi-system-prompt.ts`](.pi/extensions/00-ultimate-pi-system-prompt.ts) sets the base prompt from packaged [`.pi/SYSTEM.md`](.pi/SYSTEM.md), or from your workspace override **`.pi/system.md`** (lowercase) if you create one. Nothing is copied into your project by default. After upgrading the package or editing either file, run **`/reload`**.
 - **Model routing (vendored + gated)** — [`pi-model-router`](https://github.com/yeliu84/pi-model-router) ships inside this package (`vendor/pi-model-router/`). [`.pi/extensions/pi-model-router-harness.ts`](.pi/extensions/pi-model-router-harness.ts) activates it **only after** `.pi/model-router.json` exists (generation: `/harness-setup` Step 3.5), so **`router/auto` does not appear** beforehand. See [THIRD_PARTY_NOTICES.md](THIRD_PARTY_NOTICES.md). [`.pi/scripts/harness-sync-model-router.mjs`](.pi/scripts/harness-sync-model-router.mjs) may set **`defaultProvider`/`defaultModel`** to **`router`/`auto`** when the project sets no default — run **`/reload`** afterward. Do **not** add `npm:@yeliu84/pi-model-router` to `.pi/settings.json`; it duplicates the fork. Maintainer refresh: **`npm run vendor:sync-router`**.
 - **Active run + plan context** — PlanPacket lives at a fixed path per run; the extension injects it for `/harness-plan` (revise) and `/harness-run` (execute). Session state plus `.pi/harness/active-run.json`; no run ids or plan paths to copy.
-- **Review isolation** — run evaluate/review/critic in a **new session** after execute (see troubleshooting).
+- **Review isolation** — `/harness-eval`, `/harness-review`, and `/harness-critic` spawn isolated subagents (`inherit_context: false`); stay in the same session (see ADR 0032).
 - **Concurrent plans** — a second `/harness-plan` while a run is active is blocked until `/harness-abort` or `/harness-new-run` (except drift replan / amend after `needs_clarification`).
 - **Plan before mutate** — write/edit/shell that changes the repo is blocked until execute phase.
 - **No auto-merge** — you decide when to open or merge a PR.
@@ -91,10 +90,10 @@ Optional: copy [`.env.example`](.env.example) to `.env` if you use PostHog or ot
 | Problem | Try |
 |---------|-----|
 | Setup fails | `node --version` (need 18+), rerun `/harness-setup` |
-| "No active run" on eval | Finish plan+run first, or `/harness-run-status`; open a new session for eval |
+| "No active run" on eval | Finish plan+run first, or `/harness-run-status` |
 | Forgot where you left off | `/harness-run-status` |
 | Second plan rejected | `/harness-abort` or `/harness-new-run` |
-| Blocked in evaluate/review | Run review in a fresh session (isolation from execute) |
+| Blocked in evaluate/review | Spawn review via Agent (`harness/evaluator` / `harness/adversary`); do not run review tools inline in execute phase |
 | High plan drift | `harness-drift-replan` or abort then replan (ADR 0007) |
 | Budget / scope stop | `/harness-budget-status`, narrow the task or split the plan |
 | Test integrity warning | `/harness-test-integrity-last`, fix or justify test changes |

package/THIRD_PARTY_NOTICES.md

@@ -5,7 +5,7 @@
 - **Project:** https://github.com/yeliu84/pi-model-router  
 - **License:** MIT ([vendor/pi-model-router/LICENSE](vendor/pi-model-router/LICENSE))  
 - **Pinned revision:** See [vendor/pi-model-router/UPSTREAM_PIN.md](vendor/pi-model-router/UPSTREAM_PIN.md)  
-- ultimate-pi loads it from [`vendor/pi-model-router`](vendor/pi-model-router); import specifiers were adapted for `@mariozechner/pi-coding-agent` and related Pi packages.
+- ultimate-pi loads it from [`vendor/pi-model-router`](vendor/pi-model-router); import specifiers were adapted for `@earendil-works/pi-coding-agent` and related Pi packages.
 
 ## pi-vcc (vendored)
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ultimate-pi",
-	"version": "0.7.0",
+	"version": "0.8.0",
 	"description": "Ultimate AI coding harness for pi.dev — extensible skills, Obsidian wiki knowledge layer, compressed context, deterministic output",
 	"keywords": [
 		"pi-package",
@@ -70,7 +70,7 @@
 		"vendor/pi-vcc"
 	],
 	"peerDependencies": {
-		"@mariozechner/pi-coding-agent": "*"
+		"@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/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/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",
@@ -82,7 +82,7 @@
 		"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/sentrux-rules-sync.test.mjs && npx -y tsx --test test/harness-vcc-settings.test.ts",
+		"test": "node --test test/harness-verify.test.mjs test/harness-ask-user.test.mjs test/harness-subagents-loader.test.mjs test/sentrux-rules-sync.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: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",
@@ -90,20 +90,25 @@
 	},
 	"devDependencies": {
 		"@biomejs/biome": "2.4.14",
-		"@mariozechner/pi-agent-core": "0.73.0",
-		"@mariozechner/pi-ai": "0.73.0",
-		"@mariozechner/pi-coding-agent": "0.73.0",
-		"@mariozechner/pi-tui": "0.73.0",
+		"@earendil-works/pi-agent-core": "0.74.1",
+		"@earendil-works/pi-ai": "0.74.1",
+		"@earendil-works/pi-coding-agent": "0.74.1",
+		"@earendil-works/pi-tui": "0.74.1",
 		"@sinclair/typebox": "^0.34.49",
 		"lefthook": "2.1.6",
 		"typescript": "^6.0.3"
 	},
 	"dependencies": {
 		"@posthog/pi": "latest",
-		"asciify-image": "^0.1.10",
 		"croner": "^9.0.0",
 		"jimp": "^1.6.1",
 		"nanoid": "^5.1.5",
 		"posthog-node": "^5.30.6"
+	},
+	"overrides": {
+		"@mariozechner/pi-agent-core": "npm:@earendil-works/pi-agent-core@0.74.1",
+		"@mariozechner/pi-ai": "npm:@earendil-works/pi-ai@0.74.1",
+		"@mariozechner/pi-coding-agent": "npm:@earendil-works/pi-coding-agent@0.74.1",
+		"@mariozechner/pi-tui": "npm:@earendil-works/pi-tui@0.74.1"
 	}
 }

package/vendor/pi-model-router/UPSTREAM_PIN.md

@@ -3,6 +3,6 @@
 - **Repository:** https://github.com/yeliu84/pi-model-router
 - **License:** MIT (`LICENSE` in this tree)
 - **Pinned upstream commit:** `8c60095da0e753c242c4be9bb617b85f4dd3255c`
-- **Local changes:** TypeScript imports in `extensions/*.ts` use `@mariozechner/*`; relative imports end in `.js` for NodeNext; `package.json` peerDependencies list `@mariozechner/*`.
+- **Local changes:** TypeScript imports in `extensions/*.ts` use `@earendil-works/*`; relative imports end in `.js` for NodeNext; `package.json` peerDependencies list `@earendil-works/*`.
 
 **Refresh upstream:** run `npm run vendor:sync-router` from ultimate-pi root (updates this file with the latest commit SHA).

package/vendor/pi-model-router/extensions/commands.ts

@@ -1,8 +1,8 @@
 import type {
   ExtensionAPI,
   ExtensionContext,
-} from '@mariozechner/pi-coding-agent';
-import type { AutocompleteItem } from '@mariozechner/pi-tui';
+} from '@earendil-works/pi-coding-agent';
+import type { AutocompleteItem } from '@earendil-works/pi-tui';
 import type {
   RouterConfig,
   RouterPinByProfile,

package/vendor/pi-model-router/extensions/config.ts

@@ -1,7 +1,7 @@
 import { existsSync, readFileSync } from 'node:fs';
 import { join } from 'node:path';
-import { getAgentDir } from '@mariozechner/pi-coding-agent';
-import type { ThinkingLevel } from '@mariozechner/pi-agent-core';
+import { getAgentDir } from '@earendil-works/pi-coding-agent';
+import type { ThinkingLevel } from '@earendil-works/pi-agent-core';
 import type {
   RouterConfig,
   RouterProfile,

package/vendor/pi-model-router/extensions/index.ts

@@ -1,7 +1,7 @@
 import type {
   ExtensionAPI,
   ExtensionContext,
-} from '@mariozechner/pi-coding-agent';
+} from '@earendil-works/pi-coding-agent';
 import {
   type RouterConfig,
   type RouterPersistedState,

package/vendor/pi-model-router/extensions/provider.ts

@@ -8,11 +8,11 @@ import {
   type Model,
   type SimpleStreamOptions,
   type Message,
-} from '@mariozechner/pi-ai';
+} from '@earendil-works/pi-ai';
 import type {
   ExtensionAPI,
   ExtensionContext,
-} from '@mariozechner/pi-coding-agent';
+} from '@earendil-works/pi-coding-agent';
 import type {
   RouterConfig,
   RoutingDecision,

package/vendor/pi-model-router/extensions/routing.ts

@@ -1,5 +1,5 @@
-import { streamSimple, type Context, type Message } from '@mariozechner/pi-ai';
-import type { ExtensionContext } from '@mariozechner/pi-coding-agent';
+import { streamSimple, type Context, type Message } from '@earendil-works/pi-ai';
+import type { ExtensionContext } from '@earendil-works/pi-coding-agent';
 import type {
   RouterTier,
   RouterPhase,

package/vendor/pi-model-router/extensions/types.ts

@@ -1,4 +1,4 @@
-import type { ThinkingLevel } from '@mariozechner/pi-agent-core';
+import type { ThinkingLevel } from '@earendil-works/pi-agent-core';
 
 export type RouterTier = 'high' | 'medium' | 'low';
 export type RouterPin = RouterTier | 'auto';

package/vendor/pi-model-router/extensions/ui.ts

@@ -1,4 +1,4 @@
-import type { ExtensionContext } from '@mariozechner/pi-coding-agent';
+import type { ExtensionContext } from '@earendil-works/pi-coding-agent';
 import type {
   RoutingDecision,
   RouterConfig,

package/vendor/pi-model-router/package.json

@@ -35,10 +35,10 @@
     "prepublishOnly": "npm run tsc"
   },
   "peerDependencies": {
-    "@mariozechner/pi-agent-core": "*",
-    "@mariozechner/pi-ai": "*",
-    "@mariozechner/pi-coding-agent": "*",
-    "@mariozechner/pi-tui": "*",
+    "@earendil-works/pi-agent-core": "*",
+    "@earendil-works/pi-ai": "*",
+    "@earendil-works/pi-coding-agent": "*",
+    "@earendil-works/pi-tui": "*",
     "@sinclair/typebox": "*"
   },
   "devDependencies": {

package/vendor/pi-vcc/index.ts

@@ -1,4 +1,4 @@
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { registerBeforeCompactHook } from "./src/hooks/before-compact";
 import { registerPiVccCommand } from "./src/commands/pi-vcc";
 import { registerVccRecallCommand } from "./src/commands/vcc-recall";

package/vendor/pi-vcc/package.json

@@ -15,7 +15,7 @@
     "url": "git+https://github.com/sting8k/pi-vcc.git"
   },
   "peerDependencies": {
-    "@mariozechner/pi-coding-agent": "*",
+    "@earendil-works/pi-coding-agent": "*",
     "@sinclair/typebox": "*"
   },
   "pi": {

package/vendor/pi-vcc/src/commands/pi-vcc.ts

@@ -1,4 +1,4 @@
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { getLastCompactionStats, PI_VCC_COMPACT_INSTRUCTION } from "../hooks/before-compact";
 
 const formatTokens = (n: number): string => {

package/vendor/pi-vcc/src/commands/vcc-recall.ts

@@ -1,4 +1,4 @@
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { loadAllMessages } from "../core/load-messages";
 import { searchEntries } from "../core/search-entries";
 import { formatRecallOutput } from "../core/format-recall";

package/vendor/pi-vcc/src/core/content.ts

@@ -1,4 +1,4 @@
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 
 export const clip = (text: string, max = 200): string => {
   if (text.length <= max) return text;

package/vendor/pi-vcc/src/core/load-messages.ts

@@ -1,5 +1,5 @@
 import { readFileSync } from "fs";
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 import { renderMessage, type RenderedEntry } from "./render-entries";
 
 export interface LoadedMessages {

package/vendor/pi-vcc/src/core/normalize.ts

@@ -1,4 +1,4 @@
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 import type { NormalizedBlock } from "../types";
 import { textOf } from "./content";
 import { sanitize } from "./sanitize";

package/vendor/pi-vcc/src/core/render-entries.ts

@@ -1,4 +1,4 @@
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 import { clip, textOf } from "./content";
 import { summarizeToolArgs } from "./tool-args";
 import { extractPath } from "./tool-args";

package/vendor/pi-vcc/src/core/report.ts

@@ -1,4 +1,4 @@
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 import { buildSections } from "./build-sections";
 import { clip } from "./content";
 import { normalize } from "./normalize";

package/vendor/pi-vcc/src/core/search-entries.ts

@@ -1,4 +1,4 @@
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 import type { RenderedEntry } from "./render-entries";
 import { textOf } from "./content";
 

package/vendor/pi-vcc/src/core/summarize.ts

@@ -1,4 +1,4 @@
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 import type { FileOps } from "../types";
 import { normalize } from "./normalize";
 import { filterNoise } from "./filter-noise";

package/vendor/pi-vcc/src/hooks/before-compact.ts

@@ -1,5 +1,5 @@
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
-import { convertToLlm } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import { convertToLlm } from "@earendil-works/pi-coding-agent";
 import { writeFileSync } from "fs";
 import { compile } from "../core/summarize";
 import { loadSettings, type PiVccSettings } from "../core/settings";

package/vendor/pi-vcc/src/tools/recall.ts

@@ -1,5 +1,5 @@
 import { Type } from "@sinclair/typebox";
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { loadAllMessages } from "../core/load-messages";
 import { searchEntries } from "../core/search-entries";
 import { formatRecallOutput } from "../core/format-recall";

package/vendor/pi-vcc/src/types.ts

@@ -1,4 +1,4 @@
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 
 export interface FileOps {
   readFiles?: string[];

package/vendor/pi-vcc/tests/fixtures.ts

@@ -1,4 +1,4 @@
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 
 const ts = Date.now();
 const assistBase = {

package/vendor/pi-vcc/tests/render-entries.test.ts

@@ -1,6 +1,6 @@
 import { describe, it, expect } from "bun:test";
 import { renderMessage } from "../src/core/render-entries";
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 import { userMsg, assistantText, assistantWithToolCall, toolResult } from "./fixtures";
 
 describe("renderMessage", () => {

package/vendor/pi-vcc/tests/search-entries.test.ts

@@ -1,7 +1,7 @@
 import { describe, it, expect } from "bun:test";
 import { searchEntries } from "../src/core/search-entries";
 import type { RenderedEntry } from "../src/core/render-entries";
-import type { Message } from "@mariozechner/pi-ai";
+import type { Message } from "@earendil-works/pi-ai";
 
 const entries: RenderedEntry[] = [
   { index: 0, role: "user", summary: "Fix login bug" },

package/vendor/pi-vcc/tests/support/load-session.ts

@@ -1,5 +1,5 @@
-import { buildSessionContext, loadEntriesFromFile } from "../../node_modules/@mariozechner/pi-coding-agent/dist/core/session-manager.js";
-import type { Message } from "@mariozechner/pi-ai";
+import { buildSessionContext, loadEntriesFromFile } from "../../node_modules/@earendil-works/pi-coding-agent/dist/core/session-manager.js";
+import type { Message } from "@earendil-works/pi-ai";
 
 export interface LoadedSession {
   messageCount: number;