@xynogen/pix-core 0.5.33 → 0.5.34

package/README.md

@@ -53,7 +53,7 @@ Pi activates extensions per installed package via each package's `pi.extensions`
 | `pix-runtime` | Unified `~/.pi/agent/pix.json` config runtime, `/pix` settings command, once guard, and auto-collapse policy |
 | `pix-optimizer` | Caveman mode + RTK tool rewriting + ponytail lazy-dev mode (`/optimizer`) |
 | `pix-gate` | Permission gate for dangerous bash commands |
-| `pix-subagent` | `agent` / `agent_result` / `agent_steer` — planner-driven sub-agents with live widget |
+| `pix-subagent` | `agent` / `agent_control` — planner-driven sub-agents with live widget |
 | `compaction` | Built into pix-core: replaces pi's built-in context compaction with two levers. **Summary prompt is always pix's** — every compaction (manual `/compact`, threshold, overflow) generates the summary with the current conversation model (no silent routing) from an editable source prompt. **Trigger is pix's when `compaction.triggerPercent > 0`** — after each settled turn pix reads live context usage and compacts at `max(contextWindow × triggerPercent, compaction.minimumTokens)`. The default 100K floor prevents low percentages from compacting too early (for example, 10% of a 300K model waits for 100K rather than 30K); `0` disables pix's trigger and lets pi decide, while pix's summary prompt still applies. After a pix-triggered compaction pix sends a short, visible "resume" user message so the agent continues on its own. Every trigger, resume, and summary run emits a notify line (reason, model, token counts). |
 
 ## Install

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-core",
-	"version": "0.5.33",
+	"version": "0.5.34",
 	"description": "Pi extension bundle — installs and activates all core pix-* extensions",
 	"type": "module",
 	"main": "src/extension.ts",
@@ -18,6 +18,7 @@
 	},
 	"files": [
 		"src",
+		"!src/**/*.test.*",
 		"README.md",
 		"LICENSE"
 	],
@@ -42,31 +43,31 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@xynogen/pix-runtime": "^0.8.1",
-		"@xynogen/pix-data": "^0.4.6",
-		"@xynogen/pix-pretty": "^1.19.0",
-		"@xynogen/pix-welcome": "^0.2.1",
-		"@xynogen/pix-footer": "^0.2.2",
-		"@xynogen/pix-diagnostics": "^0.2.2",
-		"@xynogen/pix-display": "^0.1.16",
-		"@xynogen/pix-prompts": "^0.4.4",
-		"@xynogen/pix-skills": "^0.8.3",
-		"@xynogen/pix-models": "^0.2.4",
-		"@xynogen/pix-update": "^0.2.0",
-		"@xynogen/pix-commands": "^0.6.2",
-		"@xynogen/pix-nudge": "^0.2.3",
-		"@xynogen/pix-read": "^0.2.3",
-		"@xynogen/pix-write": "^0.2.3",
-		"@xynogen/pix-edit": "^0.2.3",
-		"@xynogen/pix-find": "^0.2.3",
-		"@xynogen/pix-grep": "^0.2.4",
-		"@xynogen/pix-ls": "^0.2.3",
-		"@xynogen/pix-bash": "^0.2.3",
-		"@xynogen/pix-todo": "^0.3.2",
-		"@xynogen/pix-ask": "^0.2.26",
-		"@xynogen/pix-optimizer": "^1.1.28",
-		"@xynogen/pix-gate": "^0.2.11",
-		"@xynogen/pix-subagent": "^0.7.3"
+		"@xynogen/pix-runtime": "^0.8.2",
+		"@xynogen/pix-data": "^0.4.7",
+		"@xynogen/pix-pretty": "^1.20.0",
+		"@xynogen/pix-welcome": "^0.2.2",
+		"@xynogen/pix-footer": "^0.2.3",
+		"@xynogen/pix-diagnostics": "^0.2.3",
+		"@xynogen/pix-display": "^0.1.17",
+		"@xynogen/pix-prompts": "^0.4.5",
+		"@xynogen/pix-skills": "^0.8.4",
+		"@xynogen/pix-models": "^0.2.5",
+		"@xynogen/pix-update": "^0.2.1",
+		"@xynogen/pix-commands": "^0.6.3",
+		"@xynogen/pix-nudge": "^0.2.4",
+		"@xynogen/pix-read": "^0.2.4",
+		"@xynogen/pix-write": "^0.2.4",
+		"@xynogen/pix-edit": "^0.2.4",
+		"@xynogen/pix-find": "^0.2.4",
+		"@xynogen/pix-grep": "^0.2.5",
+		"@xynogen/pix-ls": "^0.2.4",
+		"@xynogen/pix-bash": "^0.2.4",
+		"@xynogen/pix-todo": "^0.3.3",
+		"@xynogen/pix-ask": "^0.2.27",
+		"@xynogen/pix-optimizer": "^1.1.29",
+		"@xynogen/pix-gate": "^0.2.12",
+		"@xynogen/pix-subagent": "^0.8.0"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-ai": "*",

package/src/compaction.test.ts

@@ -1,82 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { compactionThresholdTokens, resumeDecisionAfterCompaction } from "./compaction.ts";
-
-describe("compactionThresholdTokens", () => {
-	it("uses the 100k floor when the percentage threshold is lower", () => {
-		expect(compactionThresholdTokens(300_000, 10, 100_000)).toBe(100_000);
-	});
-
-	it("uses the percentage threshold when it is higher than the floor", () => {
-		expect(compactionThresholdTokens(1_000_000, 60, 100_000)).toBe(600_000);
-	});
-});
-
-describe("resumeDecisionAfterCompaction", () => {
-	const threshold = 100_000;
-
-	it("resumes when idle, no pending messages, and under threshold", () => {
-		expect(
-			resumeDecisionAfterCompaction({
-				estimatedTokensAfter: 40_000,
-				threshold,
-				idle: true,
-				hasPending: false,
-			}),
-		).toBe("resume");
-	});
-
-	it("skips when the agent is busy with the user's in-flight prompt", () => {
-		expect(
-			resumeDecisionAfterCompaction({
-				estimatedTokensAfter: 40_000,
-				threshold,
-				idle: false,
-				hasPending: false,
-			}),
-		).toBe("skip");
-	});
-
-	it("skips when the user has a prompt queued during compaction", () => {
-		expect(
-			resumeDecisionAfterCompaction({
-				estimatedTokensAfter: 40_000,
-				threshold,
-				idle: true,
-				hasPending: true,
-			}),
-		).toBe("skip");
-	});
-
-	it("latches off (loop) when still at/above threshold after compaction", () => {
-		expect(
-			resumeDecisionAfterCompaction({
-				estimatedTokensAfter: threshold,
-				threshold,
-				idle: true,
-				hasPending: false,
-			}),
-		).toBe("loop");
-	});
-
-	it("treats a missing estimate as 0 tokens (does not latch)", () => {
-		expect(
-			resumeDecisionAfterCompaction({
-				estimatedTokensAfter: undefined,
-				threshold,
-				idle: true,
-				hasPending: false,
-			}),
-		).toBe("resume");
-	});
-
-	it("loop takes priority over a busy agent", () => {
-		expect(
-			resumeDecisionAfterCompaction({
-				estimatedTokensAfter: 200_000,
-				threshold,
-				idle: false,
-				hasPending: true,
-			}),
-		).toBe("loop");
-	});
-});

package/src/once.test.ts

@@ -1,213 +0,0 @@
-import { afterEach, describe, expect, it } from "bun:test";
-import registerCommands from "@xynogen/pix-commands/extension";
-import registerSubagent from "@xynogen/pix-subagent/extension";
-import registerTodo from "@xynogen/pix-todo";
-
-// Mirror of the per-instance guard. pix-core does not own once.ts (each member
-// duplicates it to stay cross-dep-free), so we re-declare the contract here and
-// assert the dedupe semantics that the aggregator relies on.
-function once(pi: object, key: string, fn: () => void): void {
-	const g = globalThis as { __pixOnce?: WeakMap<object, Set<string>> };
-	if (!g.__pixOnce) g.__pixOnce = new WeakMap<object, Set<string>>();
-	const registry = g.__pixOnce;
-	let loaded = registry.get(pi);
-	if (!loaded) {
-		loaded = new Set<string>();
-		registry.set(pi, loaded);
-	}
-	if (loaded.has(key)) return;
-	loaded.add(key);
-	fn();
-}
-
-afterEach(() => {
-	delete (globalThis as { __pixOnce?: WeakMap<object, Set<string>> }).__pixOnce;
-});
-
-describe("once", () => {
-	it("runs the factory on first invocation", () => {
-		const pi = {};
-		let calls = 0;
-		once(pi, "pix-footer", () => {
-			calls++;
-		});
-		expect(calls).toBe(1);
-	});
-
-	it("skips repeated invocations of the same key on the same pi", () => {
-		const pi = {};
-		let calls = 0;
-		const reg = () => {
-			calls++;
-		};
-		once(pi, "pix-footer", reg);
-		once(pi, "pix-footer", reg);
-		once(pi, "pix-footer", reg);
-		expect(calls).toBe(1);
-	});
-
-	it("isolates distinct keys on the same pi", () => {
-		const pi = {};
-		const seen: string[] = [];
-		once(pi, "pix-footer", () => seen.push("footer"));
-		once(pi, "pix-welcome", () => seen.push("welcome"));
-		expect(seen).toEqual(["footer", "welcome"]);
-	});
-
-	it("shares the registry across calls via globalThis (same pi)", () => {
-		const pi = {};
-		let calls = 0;
-		once(pi, "pix-skills", () => {
-			calls++;
-		});
-		// A second loader pass (e.g. standalone install after pix-core) reuses
-		// the same globalThis registry for the same pi and must not re-run.
-		once(pi, "pix-skills", () => {
-			calls++;
-		});
-		expect(calls).toBe(1);
-	});
-
-	it("re-runs for a new pi instance with the same key (/new rehydration)", () => {
-		const pi1 = {};
-		const pi2 = {};
-		let calls = 0;
-		once(pi1, "pix-footer", () => {
-			calls++;
-		});
-		// pi2 is a fresh instance (as built by Pi on /new, /resume, /fork, /reload).
-		// The factory must run again — this is the rehydration guarantee.
-		once(pi2, "pix-footer", () => {
-			calls++;
-		});
-		expect(calls).toBe(2);
-	});
-
-	it("same key on two different pi instances dedupes within each instance", () => {
-		const pi1 = {};
-		const pi2 = {};
-		let calls1 = 0;
-		let calls2 = 0;
-		once(pi1, "pix-welcome", () => {
-			calls1++;
-		});
-		once(pi1, "pix-welcome", () => {
-			calls1++;
-		}); // second on pi1 → skip
-		once(pi2, "pix-welcome", () => {
-			calls2++;
-		});
-		once(pi2, "pix-welcome", () => {
-			calls2++;
-		}); // second on pi2 → skip
-		expect(calls1).toBe(1); // deduped within pi1
-		expect(calls2).toBe(1); // deduped within pi2
-	});
-});
-
-// Behavior pin: prove a REAL guarded member dedupes when its factory runs
-// twice against the SAME pi — the exact scenario the aggregator creates when a
-// tool is installed both via pix-core (this boots it) and standalone (Pi boots
-// it again). The member must register its tool only once or Pi reports a
-// tool conflict.
-describe("member factory dedupe (pix-todo)", () => {
-	function makeHost() {
-		const toolNames: string[] = [];
-		const pi = {
-			registerTool(def: { name: string }) {
-				toolNames.push(def.name);
-			},
-			appendEntry() {},
-			on() {},
-		} as never;
-		return { pi, toolNames };
-	}
-
-	it("registers the tool once across core + standalone activation (same pi)", () => {
-		const { pi, toolNames } = makeHost();
-		// First pass: pix-core's aggregator invokes the member factory.
-		registerTodo(pi);
-		// Second pass: standalone install loads the same module and invokes it
-		// again. The globalThis once() registry must suppress re-registration.
-		registerTodo(pi);
-		expect(toolNames).toEqual(["todo"]);
-	});
-
-	it("registers the tool again for a fresh pi instance (/new rehydration)", () => {
-		const { pi: pi1, toolNames: tools1 } = makeHost();
-		const { pi: pi2, toolNames: tools2 } = makeHost();
-		// First session.
-		registerTodo(pi1);
-		// /new: a fresh pi is built; factory must run again.
-		registerTodo(pi2);
-		expect(tools1).toEqual(["todo"]);
-		expect(tools2).toEqual(["todo"]);
-	});
-});
-
-describe("member factory dedupe (pix-commands)", () => {
-	function makeHost() {
-		const commandNames: string[] = [];
-		const pi = {
-			registerCommand(name: string) {
-				commandNames.push(name);
-			},
-			registerEntryRenderer() {},
-			on() {},
-		} as never;
-		return { pi, commandNames };
-	}
-
-	it("registers /clear, /btw, /afk, and /yolo once across core + standalone activation", () => {
-		const { pi, commandNames } = makeHost();
-		registerCommands(pi);
-		registerCommands(pi);
-		expect(commandNames).toEqual(["clear", "btw", "afk", "yolo"]);
-	});
-
-	it("registers commands again for a fresh pi instance", () => {
-		const first = makeHost();
-		const second = makeHost();
-		registerCommands(first.pi);
-		registerCommands(second.pi);
-		expect(first.commandNames).toEqual(["clear", "btw", "afk", "yolo"]);
-		expect(second.commandNames).toEqual(["clear", "btw", "afk", "yolo"]);
-	});
-});
-
-describe("member factory dedupe (pix-subagent)", () => {
-	function makeHost() {
-		const toolNames: string[] = [];
-		const pi = {
-			registerTool(def: { name: string }) {
-				toolNames.push(def.name);
-			},
-			appendEntry() {},
-			on() {},
-			getAvailableAgentTypes() {
-				return [];
-			},
-			getAvailableModels() {
-				return [];
-			},
-			registerMessageRenderer() {},
-		} as never;
-		return { pi, toolNames };
-	}
-
-	it("registers agent tools once across core + standalone activation (same pi)", () => {
-		const { pi, toolNames } = makeHost();
-		registerSubagent(pi);
-		registerSubagent(pi);
-		expect(toolNames).toEqual(["agent_info", "agent", "agent_result", "agent_steer"]);
-	});
-
-	it("registers agent tools again for a fresh pi instance (/new rehydration)", () => {
-		const { pi: pi1, toolNames: tools1 } = makeHost();
-		const { pi: pi2, toolNames: tools2 } = makeHost();
-		registerSubagent(pi1);
-		registerSubagent(pi2);
-		expect(tools1).toEqual(["agent_info", "agent", "agent_result", "agent_steer"]);
-		expect(tools2).toEqual(["agent_info", "agent", "agent_result", "agent_steer"]);
-	});
-});