@xynogen/pix-optimizer 1.1.18 → 1.1.20

package/src/caveman.test.ts

@@ -1,203 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import {
-	buildHelp,
-	buildPrompt,
-	LEVEL_NUMBERS,
-	LEVELS,
-	type Level,
-	resolveLevel,
-	STATUS_LABELS,
-	STOP_ALIASES,
-	toggleLevel,
-} from "./caveman.ts";
-
-// ── LEVELS ────────────────────────────────────────────────────────────────────
-
-describe("LEVELS", () => {
-	it("contains off as first entry", () => {
-		expect(LEVELS[0]).toBe("off");
-	});
-
-	it("contains all expected levels", () => {
-		const expected: Level[] = ["off", "lite", "full", "ultra", "micro"];
-		for (const l of expected) expect(LEVELS).toContain(l);
-	});
-});
-
-// ── STOP_ALIASES ──────────────────────────────────────────────────────────────
-
-describe("STOP_ALIASES", () => {
-	it("includes off, stop, quit", () => {
-		expect(STOP_ALIASES.has("off")).toBe(true);
-		expect(STOP_ALIASES.has("stop")).toBe(true);
-		expect(STOP_ALIASES.has("quit")).toBe(true);
-	});
-
-	it("does not include active levels", () => {
-		expect(STOP_ALIASES.has("full")).toBe(false);
-		expect(STOP_ALIASES.has("ultra")).toBe(false);
-	});
-});
-
-// ── STATUS_LABELS ─────────────────────────────────────────────────────────────
-
-describe("STATUS_LABELS", () => {
-	it("has a label for every non-off level", () => {
-		const nonOff = LEVELS.filter((l) => l !== "off") as Exclude<Level, "off">[];
-		for (const l of nonOff) {
-			expect(STATUS_LABELS[l]).toBeTruthy();
-		}
-	});
-
-	it("latin levels are uppercase", () => {
-		expect(STATUS_LABELS.lite).toBe("LITE");
-		expect(STATUS_LABELS.full).toBe("FULL");
-		expect(STATUS_LABELS.ultra).toBe("ULTRA");
-		expect(STATUS_LABELS.micro).toBe("MICRO");
-	});
-});
-
-// ── resolveLevel ──────────────────────────────────────────────────────────────
-
-describe("resolveLevel", () => {
-	it("resolves valid levels", () => {
-		expect(resolveLevel("lite")).toBe("lite");
-		expect(resolveLevel("full")).toBe("full");
-		expect(resolveLevel("ultra")).toBe("ultra");
-		expect(resolveLevel("micro")).toBe("micro");
-		expect(resolveLevel("off")).toBe("off");
-	});
-
-	it("maps stop aliases to off", () => {
-		expect(resolveLevel("stop")).toBe("off");
-		expect(resolveLevel("quit")).toBe("off");
-	});
-
-	it("is case-insensitive", () => {
-		expect(resolveLevel("FULL")).toBe("full");
-		expect(resolveLevel("Ultra")).toBe("ultra");
-		expect(resolveLevel("STOP")).toBe("off");
-	});
-
-	it("trims whitespace", () => {
-		expect(resolveLevel("  full  ")).toBe("full");
-	});
-
-	it("returns null for unknown input", () => {
-		expect(resolveLevel("unknown")).toBeNull();
-		expect(resolveLevel("")).toBeNull();
-		expect(resolveLevel("config")).toBeNull(); // config handled separately
-	});
-});
-
-// ── numeric levels ────────────────────────────────────────────────────────────
-
-describe("numeric levels", () => {
-	it("maps 1/2/3 to lite/full/ultra", () => {
-		expect(resolveLevel("1")).toBe("lite");
-		expect(resolveLevel("2")).toBe("full");
-		expect(resolveLevel("3")).toBe("ultra");
-	});
-
-	it("maps 0 to off", () => {
-		expect(resolveLevel("0")).toBe("off");
-	});
-
-	it("LEVEL_NUMBERS only covers 1-3", () => {
-		expect(Object.keys(LEVEL_NUMBERS).sort()).toEqual(["1", "2", "3"]);
-	});
-
-	it("rejects out-of-range numbers", () => {
-		expect(resolveLevel("4")).toBeNull();
-		expect(resolveLevel("9")).toBeNull();
-	});
-
-	it("trims whitespace around numbers", () => {
-		expect(resolveLevel("  2  ")).toBe("full");
-	});
-});
-
-// ── buildHelp ─────────────────────────────────────────────────────────────────
-
-describe("buildHelp", () => {
-	it("lists numeric shortcuts", () => {
-		const help = buildHelp("off");
-		expect(help).toContain("1");
-		expect(help).toContain("lite");
-		expect(help).toContain("2");
-		expect(help).toContain("full");
-		expect(help).toContain("3");
-		expect(help).toContain("ultra");
-	});
-
-	it("shows current level when active", () => {
-		expect(buildHelp("ultra")).toContain("ULTRA");
-	});
-
-	it("shows off when disabled", () => {
-		expect(buildHelp("off")).toContain("off");
-	});
-
-	it("mentions config", () => {
-		expect(buildHelp("off")).toContain("config");
-	});
-
-	it("does not mention wenyan", () => {
-		expect(buildHelp("off").toLowerCase()).not.toContain("wenyan");
-	});
-});
-
-// ── toggleLevel ───────────────────────────────────────────────────────────────
-
-describe("toggleLevel", () => {
-	it("off → full", () => {
-		expect(toggleLevel("off")).toBe("full");
-	});
-
-	it("full → off", () => {
-		expect(toggleLevel("full")).toBe("off");
-	});
-
-	it("any non-off level → off", () => {
-		const nonOff = LEVELS.filter((l) => l !== "off") as Level[];
-		for (const l of nonOff) {
-			expect(toggleLevel(l)).toBe("off");
-		}
-	});
-});
-
-// ── buildPrompt ───────────────────────────────────────────────────────────────
-
-describe("buildPrompt", () => {
-	it("returns empty string for off", () => {
-		expect(buildPrompt("off")).toBe("");
-	});
-
-	it("returns MICRO_PROMPT for micro (no BASE)", () => {
-		const p = buildPrompt("micro");
-		expect(p).toContain("Token efficiency");
-		expect(p).not.toContain("CAVEMAN MODE");
-	});
-
-	it("includes BASE for all standard levels", () => {
-		const standard = ["lite", "full", "ultra"] as Level[];
-		for (const l of standard) {
-			expect(buildPrompt(l)).toContain("CAVEMAN MODE");
-		}
-	});
-
-	it("includes SAFETY clause for all standard levels", () => {
-		const standard = ["lite", "full", "ultra"] as Level[];
-		for (const l of standard) {
-			expect(buildPrompt(l)).toContain("Auto-clarity");
-		}
-	});
-
-	it("each level has distinct intensity instructions", () => {
-		const lite = buildPrompt("lite");
-		const ultra = buildPrompt("ultra");
-		expect(lite).toContain("Keep articles");
-		expect(ultra).toContain("Abbreviate");
-		expect(lite).not.toContain("Abbreviate");
-	});
-});

package/src/json.test.ts

@@ -1,167 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import {
-	adviseToon,
-	isUniformObjectArray,
-	JSON_SYSTEM_PROMPT,
-	mentionsJson,
-	objectDepth,
-} from "./json.ts";
-
-describe("JSON_SYSTEM_PROMPT", () => {
-	it("mentions the jq + toon pipeline", () => {
-		expect(JSON_SYSTEM_PROMPT).toContain("jq");
-		expect(JSON_SYSTEM_PROMPT).toContain("toon");
-	});
-
-	it("teaches both encode and decode directions", () => {
-		expect(JSON_SYSTEM_PROMPT).toContain("toon -d");
-	});
-
-	it("warns to keep JSON for API contracts", () => {
-		expect(JSON_SYSTEM_PROMPT.toLowerCase()).toContain("contract");
-	});
-
-	it("references the toon-json skill", () => {
-		expect(JSON_SYSTEM_PROMPT).toContain("toon-json");
-	});
-});
-
-describe("mentionsJson (injection gate)", () => {
-	it("matches json in any case", () => {
-		expect(mentionsJson("parse this JSON")).toBe(true);
-		expect(mentionsJson("a Json file")).toBe(true);
-		expect(mentionsJson("json blob")).toBe(true);
-	});
-
-	it("matches related tokens", () => {
-		expect(mentionsJson("pipe through jq")).toBe(true);
-		expect(mentionsJson("convert to toon")).toBe(true);
-		expect(mentionsJson("a .jsonl dataset")).toBe(true);
-		expect(mentionsJson("the openapi spec")).toBe(true);
-		expect(mentionsJson("swagger doc")).toBe(true);
-		expect(mentionsJson("some js object")).toBe(true);
-	});
-
-	it("does not match unrelated prompts", () => {
-		expect(mentionsJson("refactor the auth module")).toBe(false);
-		expect(mentionsJson("fix the bug")).toBe(false);
-	});
-
-	it("is word-bounded (no false positives inside words)", () => {
-		expect(mentionsJson("adjust the layout")).toBe(false); // contains 'js'
-		expect(mentionsJson("a jsx component")).toBe(false); // jsx != js
-		expect(mentionsJson("jsonify is a typo")).toBe(false); // jsonify != json
-	});
-
-	it("handles empty / nullish input", () => {
-		expect(mentionsJson("")).toBe(false);
-		expect(mentionsJson(undefined)).toBe(false);
-		expect(mentionsJson(null)).toBe(false);
-	});
-});
-
-describe("isUniformObjectArray", () => {
-	it("true for same-keyed flat objects", () => {
-		expect(
-			isUniformObjectArray([
-				{ id: 1, name: "a" },
-				{ id: 2, name: "b" },
-			]),
-		).toBe(true);
-	});
-
-	it("false for differing keys", () => {
-		expect(isUniformObjectArray([{ id: 1 }, { id: 2, extra: true }])).toBe(false);
-	});
-
-	it("false when a value is itself an object", () => {
-		expect(
-			isUniformObjectArray([
-				{ id: 1, meta: { x: 1 } },
-				{ id: 2, meta: { x: 2 } },
-			]),
-		).toBe(false);
-	});
-
-	it("false for empty array", () => {
-		expect(isUniformObjectArray([])).toBe(false);
-	});
-
-	it("false for array of primitives", () => {
-		expect(isUniformObjectArray([1, 2, 3])).toBe(false);
-	});
-});
-
-describe("objectDepth", () => {
-	it("primitive is depth 0", () => {
-		expect(objectDepth(42)).toBe(0);
-		expect(objectDepth("x")).toBe(0);
-	});
-
-	it("flat object is depth 1", () => {
-		expect(objectDepth({ a: 1, b: 2 })).toBe(1);
-	});
-
-	it("nested object counts each level", () => {
-		expect(objectDepth({ a: { b: { c: 1 } } })).toBe(3);
-	});
-
-	it("array nesting counts too", () => {
-		expect(objectDepth([{ a: [1, 2] }])).toBe(3);
-	});
-});
-
-describe("adviseToon", () => {
-	it("recommends TOON for uniform object arrays (sweet spot)", () => {
-		const advice = adviseToon([
-			{ id: 1, role: "admin" },
-			{ id: 2, role: "user" },
-		]);
-		expect(advice.useToon).toBe(true);
-		expect(advice.reason).toContain("tabular");
-	});
-
-	it("recommends TOON for primitive arrays", () => {
-		expect(adviseToon([1, 2, 3]).useToon).toBe(true);
-	});
-
-	it("recommends TOON for shallow objects", () => {
-		expect(adviseToon({ a: 1, b: 2 }).useToon).toBe(true);
-	});
-
-	it("rejects array of arrays (TOON's worse case)", () => {
-		const advice = adviseToon([
-			[1, 2],
-			[3, 4],
-		]);
-		expect(advice.useToon).toBe(false);
-		expect(advice.reason).toContain("array of arrays");
-	});
-
-	it("rejects empty array", () => {
-		expect(adviseToon([]).useToon).toBe(false);
-	});
-
-	it("rejects deeply nested objects beyond maxDepth", () => {
-		const deep = { a: { b: { c: { d: { e: 1 } } } } }; // depth 5
-		const advice = adviseToon(deep, 4);
-		expect(advice.useToon).toBe(false);
-		expect(advice.reason).toContain("depth");
-	});
-
-	it("rejects non-uniform object arrays", () => {
-		const advice = adviseToon([{ id: 1 }, { id: 2, extra: true }]);
-		expect(advice.useToon).toBe(false);
-	});
-
-	it("rejects primitive scalars", () => {
-		expect(adviseToon(42).useToon).toBe(false);
-		expect(adviseToon("hi").useToon).toBe(false);
-	});
-
-	it("honours a custom maxDepth", () => {
-		const d2 = { a: { b: 1 } }; // depth 2
-		expect(adviseToon(d2, 1).useToon).toBe(false);
-		expect(adviseToon(d2, 2).useToon).toBe(true);
-	});
-});

package/src/opt.test.ts

@@ -1,56 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { buildOptHelp, levelBar } from "./opt.ts";
-import type { OptimizerHandle, OptimizerTool } from "./status.ts";
-
-/** Build a handle set with fixed current values + value lists. */
-function fakeHandles(): Record<OptimizerTool, OptimizerHandle> {
-	const mk = (name: OptimizerTool, current: string, values: string[]): OptimizerHandle => ({
-		name,
-		help: `${name} — ${name} help`,
-		values,
-		current: () => current,
-		run: () => {},
-	});
-	return {
-		caveman: mk("caveman", "full", ["off", "lite", "full", "ultra", "micro"]),
-		rtk: mk("rtk", "on", ["off", "on"]),
-		toon: mk("toon", "off", ["off", "on"]),
-		ponytail: mk("ponytail", "off", ["off", "lite", "full", "ultra"]),
-	};
-}
-
-describe("levelBar", () => {
-	const PONYTAIL = ["off", "lite", "full", "ultra"];
-	const BINARY = ["off", "on"];
-
-	it("fills segments by value position in a multi-step ladder", () => {
-		expect(levelBar("off", PONYTAIL)).toBe("▱▱▱");
-		expect(levelBar("lite", PONYTAIL)).toBe("▰▱▱");
-		expect(levelBar("full", PONYTAIL)).toBe("▰▰▱");
-		expect(levelBar("ultra", PONYTAIL)).toBe("▰▰▰");
-	});
-
-	it("renders a single segment for binary tools", () => {
-		expect(levelBar("off", BINARY)).toBe("▱");
-		expect(levelBar("on", BINARY)).toBe("▰");
-	});
-
-	it("treats an unknown value as off (no fill)", () => {
-		expect(levelBar("???", PONYTAIL)).toBe("▱▱▱");
-	});
-
-	it("returns empty string when there are no non-off steps", () => {
-		expect(levelBar("off", ["off"])).toBe("");
-	});
-});
-
-describe("buildOptHelp", () => {
-	it("lists every tool with its current value", () => {
-		const help = buildOptHelp(fakeHandles());
-		expect(help).toContain("caveman: full");
-		expect(help).toContain("rtk: on");
-		expect(help).toContain("toon: off");
-		expect(help).toContain("ponytail: off");
-		expect(help).toContain("caveman help");
-	});
-});

package/src/persist.test.ts

@@ -1,56 +0,0 @@
-/**
- * Regression test for the cross-session reset bug: optimizer tool states
- * (caveman/ponytail/rtk/toon) were lost on a full quit/restart because they
- * only persisted to the session log. persist.ts adds disk persistence so a
- * value written in one session is readable in the next.
- */
-
-import { afterAll, beforeAll, describe, expect, test } from "bun:test";
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { loadOptValue, saveOptValue } from "./persist.ts";
-
-let tmpAgentDir: string;
-
-beforeAll(() => {
-	tmpAgentDir = mkdtempSync(join(tmpdir(), "optimizer-persist-test-"));
-	process.env.PI_CODING_AGENT_DIR = tmpAgentDir;
-});
-
-afterAll(() => {
-	delete process.env.PI_CODING_AGENT_DIR;
-	try {
-		rmSync(tmpAgentDir, { recursive: true });
-	} catch {
-		// temp dir may already be gone — safe to ignore
-	}
-});
-
-describe("optimizer persistence", () => {
-	test("returns undefined before anything is saved", () => {
-		expect(loadOptValue("caveman")).toBeUndefined();
-	});
-
-	test("round-trips a single tool value across save/load (new-session sim)", () => {
-		saveOptValue("caveman", "lite");
-		// A fresh load (as a new session would do) sees the persisted value.
-		expect(loadOptValue("caveman")).toBe("lite");
-	});
-
-	test("persists each tool independently in one shared file", () => {
-		saveOptValue("ponytail", "full");
-		saveOptValue("rtk", "off");
-		saveOptValue("toon", "on");
-		expect(loadOptValue("caveman")).toBe("lite");
-		expect(loadOptValue("ponytail")).toBe("full");
-		expect(loadOptValue("rtk")).toBe("off");
-		expect(loadOptValue("toon")).toBe("on");
-	});
-
-	test("overwriting one tool leaves the others intact", () => {
-		saveOptValue("caveman", "ultra");
-		expect(loadOptValue("caveman")).toBe("ultra");
-		expect(loadOptValue("ponytail")).toBe("full");
-	});
-});

package/src/ponytail.test.ts

@@ -1,195 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import {
-	buildHelp,
-	buildPrompt,
-	LEVEL_NUMBERS,
-	LEVELS,
-	type Level,
-	resolveLevel,
-	STATUS_LABELS,
-	STOP_ALIASES,
-	toggleLevel,
-} from "./ponytail.ts";
-
-// ── LEVELS ────────────────────────────────────────────────────────────────────
-
-describe("LEVELS", () => {
-	it("contains off as first entry", () => {
-		expect(LEVELS[0]).toBe("off");
-	});
-
-	it("contains all expected levels", () => {
-		const expected: Level[] = ["off", "lite", "full", "ultra"];
-		for (const l of expected) expect(LEVELS).toContain(l);
-	});
-
-	it("has no micro level (caveman-only)", () => {
-		expect(LEVELS).not.toContain("micro" as Level);
-	});
-});
-
-// ── STOP_ALIASES ──────────────────────────────────────────────────────────────
-
-describe("STOP_ALIASES", () => {
-	it("includes off, stop, quit", () => {
-		expect(STOP_ALIASES.has("off")).toBe(true);
-		expect(STOP_ALIASES.has("stop")).toBe(true);
-		expect(STOP_ALIASES.has("quit")).toBe(true);
-	});
-
-	it("does not include active levels", () => {
-		expect(STOP_ALIASES.has("full")).toBe(false);
-		expect(STOP_ALIASES.has("ultra")).toBe(false);
-	});
-});
-
-// ── STATUS_LABELS ─────────────────────────────────────────────────────────────
-
-describe("STATUS_LABELS", () => {
-	it("has a label for every non-off level", () => {
-		const nonOff = LEVELS.filter((l) => l !== "off") as Exclude<Level, "off">[];
-		for (const l of nonOff) {
-			expect(STATUS_LABELS[l]).toBeTruthy();
-		}
-	});
-
-	it("levels are uppercase", () => {
-		expect(STATUS_LABELS.lite).toBe("LITE");
-		expect(STATUS_LABELS.full).toBe("FULL");
-		expect(STATUS_LABELS.ultra).toBe("ULTRA");
-	});
-});
-
-// ── resolveLevel ──────────────────────────────────────────────────────────────
-
-describe("resolveLevel", () => {
-	it("resolves valid levels", () => {
-		expect(resolveLevel("lite")).toBe("lite");
-		expect(resolveLevel("full")).toBe("full");
-		expect(resolveLevel("ultra")).toBe("ultra");
-		expect(resolveLevel("off")).toBe("off");
-	});
-
-	it("maps stop aliases to off", () => {
-		expect(resolveLevel("stop")).toBe("off");
-		expect(resolveLevel("quit")).toBe("off");
-	});
-
-	it("is case-insensitive", () => {
-		expect(resolveLevel("FULL")).toBe("full");
-		expect(resolveLevel("Ultra")).toBe("ultra");
-		expect(resolveLevel("STOP")).toBe("off");
-	});
-
-	it("trims whitespace", () => {
-		expect(resolveLevel("  full  ")).toBe("full");
-	});
-
-	it("returns null for unknown input", () => {
-		expect(resolveLevel("unknown")).toBeNull();
-		expect(resolveLevel("")).toBeNull();
-		expect(resolveLevel("config")).toBeNull(); // config handled separately
-		expect(resolveLevel("micro")).toBeNull(); // no micro level
-	});
-});
-
-// ── numeric levels ────────────────────────────────────────────────────────────
-
-describe("numeric levels", () => {
-	it("maps 1/2/3 to lite/full/ultra", () => {
-		expect(resolveLevel("1")).toBe("lite");
-		expect(resolveLevel("2")).toBe("full");
-		expect(resolveLevel("3")).toBe("ultra");
-	});
-
-	it("maps 0 to off", () => {
-		expect(resolveLevel("0")).toBe("off");
-	});
-
-	it("LEVEL_NUMBERS only covers 1-3", () => {
-		expect(Object.keys(LEVEL_NUMBERS).sort()).toEqual(["1", "2", "3"]);
-	});
-
-	it("rejects out-of-range numbers", () => {
-		expect(resolveLevel("4")).toBeNull();
-		expect(resolveLevel("9")).toBeNull();
-	});
-
-	it("trims whitespace around numbers", () => {
-		expect(resolveLevel("  2  ")).toBe("full");
-	});
-});
-
-// ── buildHelp ─────────────────────────────────────────────────────────────────
-
-describe("buildHelp", () => {
-	it("lists numeric shortcuts", () => {
-		const help = buildHelp("off");
-		expect(help).toContain("1");
-		expect(help).toContain("lite");
-		expect(help).toContain("2");
-		expect(help).toContain("full");
-		expect(help).toContain("3");
-		expect(help).toContain("ultra");
-	});
-
-	it("shows current level when active", () => {
-		expect(buildHelp("ultra")).toContain("ULTRA");
-	});
-
-	it("shows off when disabled", () => {
-		expect(buildHelp("off")).toContain("off");
-	});
-
-	it("mentions config", () => {
-		expect(buildHelp("off")).toContain("config");
-	});
-});
-
-// ── toggleLevel ───────────────────────────────────────────────────────────────
-
-describe("toggleLevel", () => {
-	it("off → full", () => {
-		expect(toggleLevel("off")).toBe("full");
-	});
-
-	it("full → off", () => {
-		expect(toggleLevel("full")).toBe("off");
-	});
-
-	it("any non-off level → off", () => {
-		const nonOff = LEVELS.filter((l) => l !== "off") as Level[];
-		for (const l of nonOff) {
-			expect(toggleLevel(l)).toBe("off");
-		}
-	});
-});
-
-// ── buildPrompt ───────────────────────────────────────────────────────────────
-
-describe("buildPrompt", () => {
-	it("returns empty string for off", () => {
-		expect(buildPrompt("off")).toBe("");
-	});
-
-	it("includes BASE ladder for all active levels", () => {
-		for (const l of ["lite", "full", "ultra"] as Level[]) {
-			expect(buildPrompt(l)).toContain("PONYTAIL MODE ACTIVE");
-			expect(buildPrompt(l)).toContain("first rung that holds");
-		}
-	});
-
-	it("includes SAFETY clause for all active levels", () => {
-		for (const l of ["lite", "full", "ultra"] as Level[]) {
-			expect(buildPrompt(l)).toContain("When NOT to be lazy");
-		}
-	});
-
-	it("each level has distinct intensity instructions", () => {
-		const lite = buildPrompt("lite");
-		const ultra = buildPrompt("ultra");
-		expect(lite).toContain("name the lazier alternative");
-		expect(ultra).toContain("YAGNI extremist");
-		expect(lite).not.toContain("YAGNI extremist");
-	});
-});