@jameslovespancakes/pi-plus 1.0.13 → 1.0.14

package/README.md

@@ -129,7 +129,9 @@ the one line to run, and verifies.
 `remote_test` snapshots your working tree, admission-checks CPU/GPU/disk,
 reserves a slot, and **deletes the uploaded source the moment the run ends**,
 keeping only `test.log` and `result.json`. Cleanup runs on the worker, so it
-still happens if your laptop sleeps.
+still happens if your laptop sleeps. Worker capacity is queried only by
+`remote_status` or `remote_test`; Pi Plus never injects SSH status into ordinary
+chat turns.
 
 ### Coordinate multiple agents
 
@@ -152,26 +154,9 @@ across every running pi agent. `/board` opens the messaging view:
 
 `/board setup` installs the server locally (pi starts it each session) or onto
 any Mac or Linux host over SSH, where launchd or systemd brings it back after a
-reboot.
-
-### Compact without deleting the source
-
-Better Compact replaces Pi's normal compaction with a reversible local archive,
-deterministic protection and extractive compression. Jev mode adds six-signal
-routing through OpenRouter's decisions API; Jev ranks compression but cannot
-override protected facts or authorize source deletion.
-
-```
-/compact better on    # switch to local deterministic routing and compact now
-/compact better jev   # switch to Jev routing and compact now
-/compact better off   # restore Pi compaction and compact now
-```
-
-The selected mode persists for later manual and automatic compactions. Jev mode
-requires `OPENROUTER_API_KEY` (or OpenRouter auth configured in Pi). Original
-chunks stay under `~/.pi/agent/super-context/archives/`; the
-`super_context_recall` tool performs bounded retrieval from the current branch's
-checkpoint.
+reboot. Board state is returned only when `agent_board` is called or a real
+board message is delivered; background presence snapshots are not added to
+model context.
 
 ### Orchestrate repeatable workflows
 
@@ -213,12 +198,11 @@ with workflow options when a task needs them.
 | `/remote add \| rename \| remove` | jump to one step |
 | `/board` | live agent board UI |
 | `/board setup \| restart \| clear \| status` | manage the board server |
-| `/compact better on \| off \| jev` | select reversible compaction and compact now |
 | `/workflow` | open the running workflow agent board |
 | `/workflow <name> [args]` | run a bundled workflow |
 
 **Tools available to the agent:** `workflow`, `list_models`, `agent_board`,
-`remote_status`, `remote_test`, `super_context_recall`.
+`remote_status`, `remote_test`.
 
 ---
 
@@ -230,8 +214,7 @@ Everything lives in one file, `~/.pi/agent/pi-plus.json`, created on first use:
 {
   "env":    { "ARTIFICIAL_ANALYSIS_API_KEY": "aa_…", "AGENT_BOARD_URL": "ws://…" },
   "policy": { "autoApprove": [], "requireApproval": [], "deny": [] },
-  "remote": { "workers": [] },
-  "compact": { "better": "off" }
+  "remote": { "workers": [] }
 }
 ```
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jameslovespancakes/pi-plus",
-  "version": "1.0.13",
+  "version": "1.0.14",
   "type": "module",
   "description": "pi and more",
   "license": "MIT",
@@ -37,7 +37,6 @@
       "./src/domains/setup/index.ts",
       "./src/domains/subscriptions/index.ts",
       "./src/domains/models/index.ts",
-      "./src/domains/compact/index.ts",
       "./src/domains/workflows/index.ts",
       "./src/domains/agents/index.ts",
       "./src/domains/remote/index.ts"
@@ -50,10 +49,10 @@
     "ws": "^8.18.3"
   },
   "peerDependencies": {
-    "@earendil-works/pi-agent-core": "*",
-    "@earendil-works/pi-ai": "*",
-    "@earendil-works/pi-coding-agent": "*",
-    "@earendil-works/pi-tui": "*",
+    "@earendil-works/pi-agent-core": "^0.86.1",
+    "@earendil-works/pi-ai": "^0.86.1",
+    "@earendil-works/pi-coding-agent": "^0.86.1",
+    "@earendil-works/pi-tui": "^0.86.1",
     "typebox": "*"
   },
   "scripts": {

package/src/core/config.ts

@@ -15,22 +15,14 @@ export interface PolicySection {
 }
 
 export interface RemoteSection {
-  injectStatus?: boolean;
   defaults?: Record<string, unknown>;
   workers: Record<string, unknown>[];
 }
 
-export type BetterCompactMode = "off" | "on" | "jev";
-
-export interface CompactSection {
-  better: BetterCompactMode;
-}
-
 export interface PiPlusConfig {
   env: Record<string, string>;
   policy: PolicySection;
   remote: RemoteSection;
-  compact: CompactSection;
 }
 
 const DEFAULTS: PiPlusConfig = {
@@ -40,8 +32,7 @@ const DEFAULTS: PiPlusConfig = {
     requireApproval: ["openrouter/*", "google/*", "openai/*"],
     deny: [],
   },
-  remote: { injectStatus: true, workers: [] },
-  compact: { better: "off" },
+  remote: { workers: [] },
 };
 
 /** Legacy file -> section, applied only when that section is still absent. */
@@ -90,7 +81,7 @@ const MIGRATIONS: { file: string; apply: (raw: any, into: PiPlusConfig) => boole
     file: "remote.json",
     apply: (raw, into) => {
       if (into.remote.workers.length > 0 || !Array.isArray(raw?.workers)) return false;
-      into.remote = { injectStatus: raw.injectStatus !== false, defaults: raw.defaults, workers: raw.workers };
+      into.remote = { defaults: raw.defaults, workers: raw.workers };
       return true;
     },
   },
@@ -98,7 +89,7 @@ const MIGRATIONS: { file: string; apply: (raw: any, into: PiPlusConfig) => boole
     file: "remote-workers.json",
     apply: (raw, into) => {
       if (into.remote.workers.length > 0 || !Array.isArray(raw?.workers)) return false;
-      into.remote = { injectStatus: raw.injectStatus !== false, defaults: raw.defaults, workers: raw.workers };
+      into.remote = { defaults: raw.defaults, workers: raw.workers };
       return true;
     },
   },
@@ -111,7 +102,6 @@ export function configPath(): string {
 }
 
 function normalize(raw: Partial<PiPlusConfig> | undefined): PiPlusConfig {
-  const better = raw?.compact?.better;
   return {
     env: raw?.env && typeof raw.env === "object" ? { ...raw.env } : {},
     policy: {
@@ -122,13 +112,9 @@ function normalize(raw: Partial<PiPlusConfig> | undefined): PiPlusConfig {
       deny: Array.isArray(raw?.policy?.deny) ? raw.policy.deny : [],
     },
     remote: {
-      injectStatus: raw?.remote?.injectStatus !== false,
       defaults: raw?.remote?.defaults,
       workers: Array.isArray(raw?.remote?.workers) ? raw.remote.workers : [],
     },
-    compact: {
-      better: better === "on" || better === "jev" ? better : "off",
-    },
   };
 }
 
@@ -137,16 +123,16 @@ export function readConfig(): PiPlusConfig {
 
   const path = configPath();
   const exists = existsSync(path);
-  const config = normalize(exists ? readJson<Partial<PiPlusConfig>>(path, {}) : undefined);
+  const raw = exists ? readJson<Partial<PiPlusConfig>>(path, {}) : undefined;
+  const config = normalize(raw);
 
-  // Only consider legacy sources when the unified file is absent or partial.
   let migrated = false;
   for (const migration of MIGRATIONS) {
     const legacyPath = agentPath(migration.file);
     if (!existsSync(legacyPath)) continue;
-    const raw = readJson<any>(legacyPath, undefined);
-    if (raw === undefined) continue;
-    if (migration.apply(raw, config)) migrated = true;
+    const legacy = readJson<any>(legacyPath, undefined);
+    if (legacy === undefined) continue;
+    if (migration.apply(legacy, config)) migrated = true;
   }
 
   if (!exists || migrated) writeJson(path, config, true);

package/src/domains/agents/format.ts

@@ -1,30 +1,5 @@
 const DELIVERY_TEXT_MAX = 2_000;
 
-export interface SnapshotAgent {
-  sessionId: string;
-  alias?: string;
-  host: string;
-  branch?: string;
-  repo?: string;
-  commit?: string;
-  state: "idle" | "thinking" | "tool";
-  lastTool?: string;
-  coordinator?: string | null;
-  reports?: string[];
-}
-
-export interface SnapshotSelf {
-  branch?: string;
-  repo?: string;
-  commit?: string;
-}
-
-export interface SnapshotCoordination {
-  coordinator?: string | null;
-  reports?: string[];
-  repoThread?: string;
-}
-
 export interface DeliveryMessage {
   senderId: string;
   senderAlias?: string;
@@ -47,39 +22,6 @@ function cleanLine(value: unknown, max: number): string {
   return text.length <= max ? text : `${text.slice(0, max - 1)}…`;
 }
 
-function compactTool(lastTool?: string): string | undefined {
-  if (!lastTool) return undefined;
-  const separator = lastTool.indexOf(":");
-  return cleanLine(separator < 0 ? lastTool : lastTool.slice(0, separator), 32);
-}
-
-export function formatBoardSnapshot(
-  agents: readonly SnapshotAgent[],
-  self: SnapshotSelf,
-  coordination: SnapshotCoordination,
-): string {
-  const commit = self.commit?.slice(0, 12) || "no-commit";
-  const meta = [
-    `${self.branch || "-"}@${commit}`,
-    `${agents.length} peer${agents.length === 1 ? "" : "s"}`,
-    coordination.repoThread ? `room=${coordination.repoThread}` : undefined,
-    coordination.coordinator ? `reports-to=${coordination.coordinator}` : undefined,
-    coordination.reports?.length ? `reports=${coordination.reports.join(",")}` : undefined,
-  ].filter(Boolean).join(" ");
-  const lines = [`[board ${meta}]`];
-  for (const agent of agents.slice(0, 12)) {
-    const agentCommit = agent.commit?.slice(0, 12) || "no-commit";
-    const sameLine = agent.repo && self.repo === agent.repo && agent.branch === self.branch;
-    const drift = sameLine && agent.commit && self.commit && agent.commit !== self.commit ? " !commit" : "";
-    const tool = agent.state === "tool" ? compactTool(agent.lastTool) : undefined;
-    const activity = tool ? `tool:${tool}` : agent.state;
-    const role = agent.reports?.length ? " coord" : agent.coordinator ? ` ->${agent.coordinator}` : "";
-    lines.push(`- ${agent.alias || agent.sessionId.slice(0, 8)}@${agent.host} ${activity} ${agent.branch || "-"}@${agentCommit}${drift}${role}`);
-  }
-  if (agents.length > 12) lines.push(`- +${agents.length - 12} more; use agent_board agents`);
-  return lines.join("\n");
-}
-
 function compactDeliveryText(text: string): string {
   const clean = text.replace(/\r\n?/g, "\n").replace(/[\u0000-\u0008\u000b\u000c\u000e-\u001f\u007f]/g, " ").trim();
   if (clean.length <= DELIVERY_TEXT_MAX) return clean;

package/src/domains/agents/index.ts

@@ -11,11 +11,10 @@ import { Type } from "typebox";
 import WebSocket from "ws";
 import { env } from "../../core/env.ts";
 import { handleBoardAdmin, registerBoardLifecycle } from "./board-setup.ts";
-import { formatBoardDeliveries, formatBoardSnapshot, type BoardDelivery } from "./format.ts";
+import { formatBoardDeliveries, type BoardDelivery } from "./format.ts";
 
 const HEARTBEAT_MS = 8_000;
 const REQUEST_TIMEOUT_MS = 2_000;
-const SNAPSHOT_CACHE_MS = 1_500;
 const MESSAGE_MAX = 8_000;
 const DELIVERY_DEBOUNCE_MS = 250;
 const DELIVERY_BATCH_MAX = 8;
@@ -304,8 +303,6 @@ export default function (pi: ExtensionAPI) {
   let ctx: ExtensionContext | undefined;
   let self: AgentInfo | undefined;
   let gitInfo: GitInfo = {};
-  let snapshot: { at: number; agents: AgentInfo[] } = { at: 0, agents: [] };
-  let lastInjectedSnapshot: string | undefined;
   let pendingDeliveries: BoardDelivery[] = [];
   let deliveryTimer: ReturnType<typeof setTimeout> | undefined;
 
@@ -321,12 +318,6 @@ export default function (pi: ExtensionAPI) {
     self = { ...self, ...gitInfo };
     client.presence(gitInfo);
   };
-  const snapshotAgents = async (): Promise<AgentInfo[]> => {
-    if (Date.now() - snapshot.at > SNAPSHOT_CACHE_MS) {
-      snapshot = { at: Date.now(), agents: await client.request("agents", {}, 500) };
-    }
-    return snapshot.agents;
-  };
   const flushDeliveries = () => {
     if (deliveryTimer) clearTimeout(deliveryTimer);
     deliveryTimer = undefined;
@@ -354,7 +345,6 @@ export default function (pi: ExtensionAPI) {
 
   client.on((event) => {
     if (event.t === "connection") {
-      if (event.connected) { snapshot.at = 0; lastInjectedSnapshot = undefined; }
       updateStatus();
       return;
     }
@@ -374,19 +364,9 @@ export default function (pi: ExtensionAPI) {
   });
   pi.on("session_info_changed", async (event) => { if (self) { self.alias = event.name; client.presence({ alias: event.name }); } });
   pi.on("model_select", async (event) => client.presence({ model: `${event.model.provider}/${event.model.id}` }));
-  pi.on("before_agent_start", async (event) => {
+  pi.on("before_agent_start", (event) => {
     client.presence({ state: "thinking", lastPrompt: cleanLine(event.prompt) });
     client.activity("prompt", event.prompt);
-    if (!client.connected) return;
-    try {
-      const content = formatBoardSnapshot(await snapshotAgents(), gitInfo, client.coordination);
-      if (content === lastInjectedSnapshot) return;
-      lastInjectedSnapshot = content;
-      // Persist one compact snapshot at a user-turn boundary. A transient message
-      // appended in `context` becomes Anthropic's final cache breakpoint but is
-      // absent from the next transcript, forcing a conversation-cache miss.
-      return { message: { customType: "agent-board-snapshot", content, display: false, details: {} } };
-    } catch { return; }
   });
   pi.on("tool_execution_start", async (event) => {
     const lastTool = cleanLine(`${event.toolName}: ${JSON.stringify(event.args)}`, 100);
@@ -394,8 +374,6 @@ export default function (pi: ExtensionAPI) {
   });
   pi.on("tool_execution_end", async () => client.presence({ state: "thinking" }));
   pi.on("agent_settled", async () => { client.presence({ state: "idle", lastTool: undefined }); refreshGit(); });
-  pi.on("session_compact", async () => { lastInjectedSnapshot = undefined; });
-  pi.on("session_tree", async () => { lastInjectedSnapshot = undefined; });
   pi.on("session_shutdown", async (_event, eventCtx) => {
     if (deliveryTimer) clearTimeout(deliveryTimer);
     deliveryTimer = undefined; pendingDeliveries = [];

package/src/domains/remote/config-path.ts

@@ -20,7 +20,6 @@ export interface RemoteWorkerRecord {
 }
 
 export interface RemoteSettings {
-  injectStatus: boolean;
   defaults: Record<string, unknown>;
   workers: RemoteWorkerRecord[];
 }
@@ -28,7 +27,6 @@ export interface RemoteSettings {
 export function readRemote(): RemoteSettings {
   const section = readConfig().remote;
   return {
-    injectStatus: section.injectStatus !== false,
     defaults: (section.defaults as Record<string, unknown>) ?? {},
     workers: (section.workers as RemoteWorkerRecord[]) ?? [],
   };

package/src/domains/remote/index.ts

@@ -31,7 +31,6 @@ interface Limits {
 	gpuMemoryBlockPercent: number;
 	memoryBlockPercent: number;
 	minimumFreeDiskGB: number;
-	statusCacheSeconds: number;
 	retentionHours: number;
 }
 
@@ -58,7 +57,6 @@ interface Worker extends Omit<WorkerInput, keyof Limits | "enabled">, Limits {
 }
 
 interface Config {
-	injectStatus: boolean;
 	workers: Worker[];
 }
 
@@ -100,7 +98,6 @@ const DEFAULT_LIMITS: Limits = {
 	gpuMemoryBlockPercent: 90,
 	memoryBlockPercent: 90,
 	minimumFreeDiskGB: 10,
-	statusCacheSeconds: 30,
 	retentionHours: 24,
 };
 
@@ -117,8 +114,6 @@ const HARD_WALK_EXCLUDES = new Set([
 	"remote_tests",
 ]);
 
-let statusCache: { expiresAt: number; key: string; statuses: WorkerStatus[] } | undefined;
-
 function finiteNumber(value: unknown, fallback: number, min: number, max: number): number {
 	return typeof value === "number" && Number.isFinite(value) ? Math.min(max, Math.max(min, value)) : fallback;
 }
@@ -139,7 +134,6 @@ async function loadConfig(): Promise<Config> {
 		),
 		memoryBlockPercent: finiteNumber(defaults.memoryBlockPercent, DEFAULT_LIMITS.memoryBlockPercent, 1, 100),
 		minimumFreeDiskGB: finiteNumber(defaults.minimumFreeDiskGB, DEFAULT_LIMITS.minimumFreeDiskGB, 0, 100000),
-		statusCacheSeconds: finiteNumber(defaults.statusCacheSeconds, DEFAULT_LIMITS.statusCacheSeconds, 1, 600),
 		retentionHours: finiteNumber(defaults.retentionHours, DEFAULT_LIMITS.retentionHours, 1, 24 * 365),
 	};
 	const names = new Set<string>();
@@ -172,11 +166,10 @@ async function loadConfig(): Promise<Config> {
 			),
 			memoryBlockPercent: finiteNumber(raw.memoryBlockPercent, defaultLimits.memoryBlockPercent, 1, 100),
 			minimumFreeDiskGB: finiteNumber(raw.minimumFreeDiskGB, defaultLimits.minimumFreeDiskGB, 0, 100000),
-			statusCacheSeconds: finiteNumber(raw.statusCacheSeconds, defaultLimits.statusCacheSeconds, 1, 600),
 			retentionHours: finiteNumber(raw.retentionHours, defaultLimits.retentionHours, 1, 24 * 365),
 		};
 	});
-	return { injectStatus: parsed.injectStatus !== false, workers };
+	return { workers };
 }
 
 /** `-i`/`-p` only when the worker was added without an ~/.ssh/config entry. */
@@ -331,27 +324,15 @@ async function probeWorker(worker: Worker): Promise<WorkerStatus> {
 	}
 }
 
-async function probeWorkers(config: Config, force = false): Promise<WorkerStatus[]> {
+async function probeWorkers(config: Config): Promise<WorkerStatus[]> {
 	const active = config.workers.filter((worker) => worker.enabled);
-	if (active.length === 0) return [];
-	const key = JSON.stringify(active.map((worker) => [worker.name, worker.ssh, worker.root]));
-	const ttlSeconds = Math.min(...active.map((worker) => worker.statusCacheSeconds));
-	if (!force && statusCache && statusCache.key === key && statusCache.expiresAt > Date.now()) return statusCache.statuses;
-	const statuses = await Promise.all(active.map(probeWorker));
-	statusCache = { key, statuses, expiresAt: Date.now() + ttlSeconds * 1000 };
-	return statuses;
+	return Promise.all(active.map(probeWorker));
 }
 
 function metric(value: number | undefined): string {
 	return value === undefined ? "?" : `${Math.round(value)}%`;
 }
 
-function compactStatus(status: WorkerStatus): string {
-	const gpu = status.gpuPercent === undefined ? "GPU ?" : `GPU ${metric(status.gpuPercent)}`;
-	const reason = status.reasons.length ? ` (${status.reasons.join(", ")})` : "";
-	return `${status.name} ${status.state.toUpperCase()} CPU ${metric(status.cpuPercent)} MEM ${metric(status.memoryPercent)} ${gpu} active-jobs ${status.jobs ?? "?"}${reason}`;
-}
-
 function detailedStatuses(statuses: WorkerStatus[]): string {
 	return statuses
 		.map((status) => {
@@ -696,7 +677,6 @@ export default function remoteJobsExtension(pi: ExtensionAPI) {
 			const selected = params.host ? enabled.filter((worker) => worker.name === params.host) : enabled;
 			if (params.host && selected.length === 0) throw new Error(`Unknown remote worker: ${params.host}`);
 			const statuses = await Promise.all(selected.map(probeWorker));
-			statusCache = undefined;
 			return {
 				content: [{ type: "text", text: detailedStatuses(statuses) }],
 				details: { statuses },
@@ -745,7 +725,7 @@ export default function remoteJobsExtension(pi: ExtensionAPI) {
 			if (requestedHost !== "auto" && !enabledWorkers.some((worker) => worker.name === requestedHost)) {
 				throw new Error(`Unknown remote worker: ${requestedHost}. Available: ${enabledWorkers.map((w) => w.name).join(", ")}`);
 			}
-			let statuses = await probeWorkers(config, true);
+			const statuses = await probeWorkers(config);
 			let worker = pickWorker(config, statuses, requestedHost, params.requiresGpu ?? false);
 			if (!worker) return blockedResult(undefined, undefined, detailedStatuses(statuses));
 			let workerStatus = statuses.find((status) => status.name === worker!.name);
@@ -818,7 +798,6 @@ export default function remoteJobsExtension(pi: ExtensionAPI) {
 					text += `\n\n[Output truncated; full log: ${remoteJob}/test.log]`;
 				}
 				void runSsh(worker, "bash -s", { input: cleanupScript(worker, snapshot.repoName), timeoutSeconds: 15 }).catch(() => {});
-				statusCache = undefined;
 				return {
 					content: [{ type: "text", text }],
 					details: {
@@ -847,20 +826,4 @@ export default function remoteJobsExtension(pi: ExtensionAPI) {
 		},
 	});
 
-	pi.on("before_agent_start", async (event) => {
-		try {
-			const config = await loadConfig();
-			if (!config.injectStatus) return;
-			const statuses = await probeWorkers(config);
-			if (statuses.length === 0) return;
-			const line = statuses.map(compactStatus).join("; ");
-			return {
-				systemPrompt:
-					event.systemPrompt +
-					`\n\nRemote worker capacity (recent sample): ${line}. remote_test always performs a fresh hard admission check. Never retry a BLOCKED worker immediately; choose another READY worker or run locally.`,
-			};
-		} catch {
-			return;
-		}
-	});
 }

package/src/domains/workflows/runtime/agent-retry.ts

@@ -8,6 +8,7 @@ import {
 export const AGENT_RETRY_BASE_DELAY_MS = 1_000;
 export const AGENT_RETRY_MAX_DELAY_MS = 30_000;
 export const WORKFLOW_PROVIDER_ERROR_CODE = "WORKFLOW_PROVIDER_ERROR";
+const CODEX_ACCESS_VERIFICATION_ERROR = /Unable to verify\s+[^.\r\n]{1,120}\s+access\.\s*Please try again\.?/i;
 
 export interface AgentRetryScheduler {
   sleep(delayMs: number, signal: AbortSignal | undefined): Promise<void>;
@@ -25,12 +26,11 @@ export interface ProviderErrorDetails {
 export class WorkflowProviderError extends Error {
   override readonly name = "WorkflowProviderError";
   readonly code = WORKFLOW_PROVIDER_ERROR_CODE;
+  readonly details: ProviderErrorDetails;
 
-  constructor(
-    message: string,
-    readonly details: ProviderErrorDetails,
-  ) {
+  constructor(message: string, details: ProviderErrorDetails) {
     super(message);
+    this.details = details;
   }
 
   get retryable(): boolean {
@@ -79,13 +79,18 @@ export function providerErrorFromMessages(
   if (usageLimit && options.pauseOnUsageLimit) return usageLimit;
   const message = messages.findLast(isAssistantMessage);
   if (!message || message.stopReason !== "error") return undefined;
-  const errorMessage = typeof message.errorMessage === "string" && message.errorMessage.length > 0
+  const providerMessage = typeof message.errorMessage === "string" && message.errorMessage.length > 0
     ? message.errorMessage
     : "Provider session ended with an unspecified error.";
-  const retryable = isRetryableAssistantError(message as AssistantMessage);
   const provider = stringDetail(message.provider);
   const model = stringDetail(message.model);
   const api = stringDetail(message.api);
+  const codexAccessVerification = provider === "openai-codex"
+    && CODEX_ACCESS_VERIFICATION_ERROR.test(providerMessage);
+  const retryable = isRetryableAssistantError(message as AssistantMessage) || codexAccessVerification;
+  const errorMessage = codexAccessVerification && model
+    ? `Codex temporarily could not verify access for the selected model ${provider}/${model}. No alternate model was requested. Provider response: ${providerMessage}`
+    : providerMessage;
   return new WorkflowProviderError(errorMessage, {
     stopReason: "error",
     retryable,

package/src/domains/compact/archive.ts

@@ -1,140 +0,0 @@
-import { createHash } from "node:crypto";
-import { existsSync, readFileSync } from "node:fs";
-import { basename } from "node:path";
-import { agentPath, writeJson } from "../../core/store.ts";
-import type { ArchiveRecord, SemanticChunk, SuperContextArchive } from "./types.ts";
-
-function safeSessionId(sessionId: string): string {
-  const readable = sessionId.replace(/[^A-Za-z0-9._-]/g, "-").slice(0, 80) || "session";
-  const suffix = createHash("sha256").update(sessionId).digest("hex").slice(0, 10);
-  return `${readable}-${suffix}`;
-}
-
-export function archivePath(sessionId: string): string {
-  return agentPath("super-context", "archives", `${safeSessionId(sessionId)}.json`);
-}
-
-function emptyArchive(sessionId: string): SuperContextArchive {
-  return { version: 1, sessionId, records: [], checkpoints: {} };
-}
-
-function isArchiveRecord(value: unknown): value is ArchiveRecord {
-  if (!value || typeof value !== "object") return false;
-  const record = value as Partial<ArchiveRecord>;
-  return typeof record.id === "string"
-    && typeof record.hash === "string"
-    && typeof record.text === "string"
-    && typeof record.role === "string"
-    && typeof record.ordinal === "number"
-    && typeof record.archivedAt === "string";
-}
-
-/** Refuse to overwrite a corrupt archive: source recoverability wins over convenience. */
-export function loadArchive(sessionId: string): SuperContextArchive {
-  const path = archivePath(sessionId);
-  if (!existsSync(path)) return emptyArchive(sessionId);
-
-  let parsed: unknown;
-  try {
-    parsed = JSON.parse(readFileSync(path, "utf8"));
-  } catch (error) {
-    throw new Error(`Super Context archive is unreadable; refusing to overwrite ${path}: ${error instanceof Error ? error.message : String(error)}`, { cause: error });
-  }
-  if (!parsed || typeof parsed !== "object") throw new Error(`Invalid Super Context archive: ${path}`);
-  const candidate = parsed as Partial<SuperContextArchive>;
-  if (candidate.version !== 1 || candidate.sessionId !== sessionId || !Array.isArray(candidate.records)) {
-    throw new Error(`Unsupported Super Context archive format: ${path}`);
-  }
-  if (!candidate.records.every(isArchiveRecord)) throw new Error(`Invalid Super Context archive records: ${path}`);
-  const checkpoints = candidate.checkpoints;
-  if (!checkpoints || typeof checkpoints !== "object" || Array.isArray(checkpoints)) {
-    throw new Error(`Invalid Super Context archive checkpoints: ${path}`);
-  }
-  for (const ids of Object.values(checkpoints)) {
-    if (!Array.isArray(ids) || ids.some((id) => typeof id !== "string")) {
-      throw new Error(`Invalid Super Context archive checkpoint: ${path}`);
-    }
-  }
-  return candidate as SuperContextArchive;
-}
-
-function addChunks(archive: SuperContextArchive, chunks: readonly SemanticChunk[]): void {
-  const byId = new Map(archive.records.map((record) => [record.id, record]));
-  let ordinal = archive.records.reduce((maximum, record) => Math.max(maximum, record.ordinal), -1) + 1;
-  const archivedAt = new Date().toISOString();
-  for (const chunk of chunks) {
-    const existing = byId.get(chunk.id);
-    if (existing) {
-      if (existing.hash !== chunk.hash || existing.text !== chunk.text) {
-        throw new Error(`Super Context archive ID collision for ${chunk.id}`);
-      }
-      continue;
-    }
-    const record: ArchiveRecord = { ...chunk, ordinal, archivedAt };
-    ordinal += 1;
-    archive.records.push(record);
-    byId.set(record.id, record);
-  }
-}
-
-function checkpointHash(recordIds: readonly string[]): string {
-  const digest = createHash("sha256").update(recordIds.join("\0")).digest("hex");
-  return `SCC-${digest.slice(0, 20)}`;
-}
-
-export interface PersistedCheckpoint {
-  archive: SuperContextArchive;
-  checkpointId: string;
-  records: ArchiveRecord[];
-  duplicateChunks: number;
-  file: string;
-}
-
-/** Add-only persistence for immutable records and content-addressed checkpoints. */
-export function persistCheckpoint(
-  sessionId: string,
-  chunks: readonly SemanticChunk[],
-  priorRecordIds: readonly string[],
-): PersistedCheckpoint {
-  const archive = loadArchive(sessionId);
-  const currentIds = chunks.map((chunk) => chunk.id);
-  const allRecordIds = [...priorRecordIds, ...currentIds];
-  const duplicateChunks = allRecordIds.length - new Set(allRecordIds).size;
-  addChunks(archive, chunks);
-
-  const knownIds = new Set(archive.records.map((record) => record.id));
-  const recordIds = [...new Set(allRecordIds)];
-  const missing = recordIds.filter((id) => !knownIds.has(id));
-  if (missing.length > 0) throw new Error(`Super Context archive is missing ${missing.length} source record(s)`);
-
-  const checkpointId = checkpointHash(recordIds);
-  const existing = archive.checkpoints[checkpointId];
-  if (existing && JSON.stringify(existing) !== JSON.stringify(recordIds)) {
-    throw new Error(`Super Context checkpoint collision for ${checkpointId}`);
-  }
-  archive.checkpoints[checkpointId] ??= recordIds;
-
-  const file = archivePath(sessionId);
-  if (!writeJson(file, archive, false, 0o600)) throw new Error(`Failed to persist Super Context archive: ${file}`);
-  const byId = new Map(archive.records.map((record) => [record.id, record]));
-  return {
-    archive,
-    checkpointId,
-    records: recordIds.map((id) => byId.get(id)!),
-    duplicateChunks,
-    file,
-  };
-}
-
-export function checkpointRecords(archive: SuperContextArchive, checkpointId: string | undefined): ArchiveRecord[] {
-  if (!checkpointId) return [];
-  const ids = archive.checkpoints[checkpointId];
-  if (!ids) return [];
-  const byId = new Map(archive.records.map((record) => [record.id, record]));
-  const records = ids.map((id) => byId.get(id)).filter((record): record is ArchiveRecord => record !== undefined);
-  return records.length === ids.length ? records : [];
-}
-
-export function archiveDisplayName(file: string): string {
-  return basename(file);
-}