pi-soly 1.2.0 → 1.4.0

package/switch/tests/core.test.ts

@@ -1,210 +0,0 @@
-// =============================================================================
-// tests/core.test.ts — Tests for pi-switch core
-// =============================================================================
-
-/// <reference types="bun-types" />
-import { describe, test, expect, beforeAll, afterAll } from "bun:test";
-import * as fs from "node:fs";
-import * as os from "node:os";
-import * as path from "node:path";
-import {
-	DEFAULT_ROTOR,
-	BUILTIN_ROTORS,
-	ROTOR_META,
-	getRotorMeta,
-	isValidRotorName,
-	discoverUserRotors,
-	availableAgents,
-	nextAgent,
-	parseRotorName,
-	formatAgentBadge,
-	formatRotorSwitchNotify,
-	formatHeaderLine,
-	groupedAvailableRotors,
-	agentFilePath,
-	loadAgent,
-	saveAgent,
-} from "../core.js";
-
-describe("DEFAULT_ROTOR", () => {
-	test("is 'worker'", () => {
-		expect(DEFAULT_ROTOR).toBe("worker");
-	});
-});
-
-describe("isValidRotorName", () => {
-	test("accepts simple names", () => {
-		expect(isValidRotorName("worker")).toBe(true);
-		expect(isValidRotorName("my_agent")).toBe(true);
-	});
-	test("rejects invalid", () => {
-		expect(isValidRotorName("with space")).toBe(false);
-		expect(isValidRotorName("")).toBe(false);
-		expect(isValidRotorName("a".repeat(65))).toBe(false);
-	});
-});
-
-describe("ROTOR_META", () => {
-	test("every built-in has metadata", () => {
-		for (const a of BUILTIN_ROTORS) {
-			expect(ROTOR_META[a]).toBeDefined();
-			expect(ROTOR_META[a]!.emoji.length).toBeGreaterThan(0);
-		}
-	});
-	test("meta has writesFiles flag", () => {
-		expect(ROTOR_META.worker!.writesFiles).toBe(true);
-		expect(ROTOR_META.oracle!.writesFiles).toBe(false);
-	});
-});
-
-describe("getRotorMeta", () => {
-	test("returns fallback for unknown", () => {
-		const m = getRotorMeta("zzz");
-		expect(m.emoji.length).toBeGreaterThan(0);
-	});
-});
-
-describe("nextAgent", () => {
-	test("cycles forward", () => {
-		expect(nextAgent("a", ["a", "b", "c"])).toBe("b");
-		expect(nextAgent("c", ["a", "b", "c"])).toBe("a");
-	});
-	test("returns first if current not in cycle", () => {
-		expect(nextAgent("zzz", ["a", "b"])).toBe("a");
-	});
-});
-
-describe("parseRotorName", () => {
-	test("trims and validates", () => {
-		expect(parseRotorName("  oracle  ")).toBe("oracle");
-		expect(parseRotorName("with space")).toBeNull();
-	});
-});
-
-describe("formatAgentBadge", () => {
-	test("null for default", () => {
-		expect(formatAgentBadge(DEFAULT_ROTOR)).toBeNull();
-	});
-	test("emoji + name for non-default", () => {
-		const b = formatAgentBadge("oracle");
-		expect(b).toContain("oracle");
-	});
-});
-
-describe("formatRotorSwitchNotify", () => {
-	test("multi-line: old → new + capability", () => {
-		const out = formatRotorSwitchNotify("worker", "oracle");
-		expect(out).toContain("pi-switch agent changed");
-		expect(out).toContain("worker");
-		expect(out).toContain("oracle");
-		expect(out).toContain("writes files: no");
-	});
-});
-
-describe("formatHeaderLine", () => {
-	test("always non-empty (even for default)", () => {
-		const h = formatHeaderLine("worker");
-		expect(h).toContain("worker");
-		expect(h).toContain("Ctrl+Tab");
-	});
-	test("includes read-only tag when applicable", () => {
-		const h = formatHeaderLine("oracle");
-		expect(h).toContain("read-only");
-	});
-	test("omits read-only tag when agent writes", () => {
-		const h = formatHeaderLine("worker");
-		expect(h).not.toContain("read-only");
-	});
-});
-
-describe("groupedAvailableRotors", () => {
-	test("includes built-in group", () => {
-		const groups = groupedAvailableRotors("/nonexistent");
-		expect(groups[0]?.header).toBe("built-in");
-	});
-	test("includes user group when present", () => {
-		// Use HOME override so the new ~.agents/ scan picks up our fixture
-		const home = process.env.HOME || process.env.USERPROFILE || os.homedir();
-		const prevHome = process.env.HOME;
-		const prevUserProfile = process.env.USERPROFILE;
-		const tmp = fs.mkdtempSync(path.join(os.tmpdir(), "pis-home-"));
-		process.env.HOME = tmp;
-		process.env.USERPROFILE = tmp;
-		const agentsDir = path.join(tmp, ".agents");
-		fs.mkdirSync(agentsDir, { recursive: true });
-		fs.writeFileSync(path.join(agentsDir, "my.md"), "---\nname: my-helper\n---\n# body\n");
-		const groups = groupedAvailableRotors();
-		const userGroup = groups.find((g) => g.header === "user-defined");
-		expect(userGroup?.agents).toContain("my-helper");
-		// restore
-		process.env.HOME = prevHome ?? home;
-		process.env.USERPROFILE = prevUserProfile ?? home;
-		fs.rmSync(tmp, { recursive: true, force: true });
-	});
-
-	test("includes project agent when present (cwd scope)", () => {
-		const projectDir = fs.mkdtempSync(path.join(os.tmpdir(), "pis-proj-"));
-		const agentsDir = path.join(projectDir, ".agents");
-		fs.mkdirSync(agentsDir, { recursive: true });
-		fs.writeFileSync(path.join(agentsDir, "proj.md"), "---\nname: project-helper\n---\n# body\n");
-		const groups = groupedAvailableRotors(projectDir);
-		const userGroup = groups.find((g) => g.header === "user-defined");
-		expect(userGroup?.agents).toContain("project-helper");
-		fs.rmSync(projectDir, { recursive: true, force: true });
-	});
-});
-
-// ---------------------------------------------------------------------------
-// Persistence
-// ---------------------------------------------------------------------------
-
-describe("agentFilePath", () => {
-	test("prefers .soly/agent when soly dir exists", () => {
-		const cwd = fs.mkdtempSync(path.join(os.tmpdir(), "pis-path-"));
-		fs.mkdirSync(path.join(cwd, ".soly"), { recursive: true });
-		expect(agentFilePath(cwd)).toBe(path.join(cwd, ".soly", "agent"));
-		fs.rmSync(cwd, { recursive: true, force: true });
-	});
-	test("falls back to ~/.pi-switch/agent when no soly", () => {
-		const cwd = fs.mkdtempSync(path.join(os.tmpdir(), "pis-fb-"));
-		// Ensure no .soly in cwd
-		expect(agentFilePath(cwd)).toContain(".pi-switch");
-		fs.rmSync(cwd, { recursive: true, force: true });
-	});
-});
-
-describe("loadAgent / saveAgent", () => {
-	let tmpCwd: string;
-	let origHome: string | undefined;
-	beforeAll(() => {
-		tmpCwd = fs.mkdtempSync(path.join(os.tmpdir(), "pis-rt-"));
-		origHome = process.env.HOME;
-		process.env.HOME = tmpCwd;
-	});
-	afterAll(() => {
-		if (origHome) process.env.HOME = origHome;
-		fs.rmSync(tmpCwd, { recursive: true, force: true });
-	});
-
-	test("round-trip", () => {
-		saveAgent(tmpCwd, "oracle");
-		expect(loadAgent(tmpCwd)).toBe("oracle");
-	});
-	test("returns null when file missing", () => {
-		const freshHome = fs.mkdtempSync(path.join(os.tmpdir(), "pis-fresh-"));
-		fs.mkdirSync(path.join(freshHome, ".pi-switch"), { recursive: true });
-		const prevHome = process.env.HOME;
-		process.env.HOME = freshHome;
-		try {
-			expect(loadAgent("/anywhere")).toBeNull();
-		} finally {
-			process.env.HOME = prevHome;
-		}
-		fs.rmSync(freshHome, { recursive: true, force: true });
-	});
-	test("rejects invalid name on load", () => {
-		const file = agentFilePath(tmpCwd);
-		fs.writeFileSync(file, "with space\n", "utf-8");
-		expect(loadAgent(tmpCwd)).toBeNull();
-	});
-});

package/switch/tests/index.test.ts

@@ -1,48 +0,0 @@
-// =============================================================================
-// tests/index.test.ts — Regression tests for index.ts handlers
-// =============================================================================
-//
-// These exercise the agent-command dispatch without spinning up a real
-// pi session. We test the parse logic by feeding the same input strings
-// through a hand-rolled mock and inspecting the notify/output side-effects.
-
-/// <reference types="bun-types" />
-import { describe, test, expect } from "bun:test";
-import { availableAgents } from "../core.js";
-
-describe("/agent handler parse logic (regression for `/agent researcher` bug)", () => {
-	// The original bug: `/agent researcher` was interpreted as "show list"
-	// instead of "set agent to researcher" because the parser only checked
-	// the SECOND token, not the first. (After cycle reduction, `researcher`
-	// is no longer a built-in, so we use `oracle` as the example agent.)
-	test("single-arg '/agent <name>' is a set, not a list", () => {
-		const input = "oracle";
-		const parts = input.trim().split(/\s+/);
-		const subcommand = parts[0]?.toLowerCase();
-		const cycle = availableAgents();
-		// Single known agent → set, not show
-		expect(parts.length).toBe(1);
-		expect(cycle.includes(subcommand ?? "")).toBe(true);
-	});
-	test("'/agent create <name>' → create subcommand", () => {
-		const input = "create my-debugger";
-		const subcommand = input.trim().split(/\s+/)[0];
-		expect(subcommand).toBe("create");
-	});
-	test("'/agent doctor' → doctor subcommand", () => {
-		const input = "doctor";
-		const subcommand = input.trim().split(/\s+/)[0];
-		expect(subcommand).toBe("doctor");
-	});
-	test("'/agent recommend <task>' → recommend subcommand", () => {
-		const input = "recommend investigate the bug";
-		const subcommand = input.trim().split(/\s+/)[0];
-		expect(subcommand).toBe("recommend");
-	});
-	test("'/agent <unknown>' falls through to error (NOT set)", () => {
-		const input = "nonexistent-agent-xyz";
-		const subcommand = input.trim().split(/\s+/)[0];
-		const cycle = availableAgents();
-		expect(cycle.includes(subcommand ?? "")).toBe(false);
-	});
-});

package/switch/tests/prompt.test.ts

@@ -1,108 +0,0 @@
-// =============================================================================
-// tests/prompt.test.ts — Tests for the system-prompt section + recommendAgent
-// =============================================================================
-
-/// <reference types="bun-types" />
-import { describe, test, expect } from "bun:test";
-import { buildPiSwitchSection, recommendAgent, TASK_AGENT_HINTS } from "../prompt.js";
-
-describe("buildPiSwitchSection", () => {
-	const s = buildPiSwitchSection();
-
-	test("starts with header", () => {
-		expect(s.trim().startsWith("## pi-switch")).toBe(true);
-	});
-	test("mentions /agent command and Ctrl+Tab", () => {
-		expect(s).toContain("/agent");
-		expect(s).toContain("Ctrl+Tab");
-	});
-	test("explains built-in categories", () => {
-		expect(s).toContain("oracle");
-		expect(s).toContain("scout");
-		expect(s).toContain("worker");
-	});
-	test("mentions soly-manager as the single subagent", () => {
-		expect(s).toContain("soly-manager");
-	});
-	test("explains user-defined", () => {
-		expect(s).toMatch(/user[- ]?defined/i);
-		expect(s).toContain("~/.pi/agent/agents/");
-	});
-	test("has anti-patterns", () => {
-		expect(s).toContain("DON");
-	});
-	test("includes task→agent heuristics table", () => {
-		expect(s).toContain("debug");
-		expect(s).toContain("refactor");
-	});
-	test("is reasonably short (< 5KB)", () => {
-		expect(s.length).toBeLessThan(5000);
-	});
-});
-
-describe("TASK_AGENT_HINTS", () => {
-	test("every hint has required fields", () => {
-		for (const h of TASK_AGENT_HINTS) {
-			expect(h.agent.length).toBeGreaterThan(0);
-			expect(h.emoji.length).toBeGreaterThan(0);
-			expect(h.why.length).toBeGreaterThan(5);
-			expect(h.pattern).toBeInstanceOf(RegExp);
-		}
-	});
-});
-
-describe("recommendAgent", () => {
-	test("research keywords → researcher (English)", () => {
-		expect(recommendAgent("look up the latest pi-subagents API")?.agent).toBe("researcher");
-		expect(recommendAgent("what is the best lib for X?")?.agent).toBe("researcher");
-	});
-	test("research keywords → researcher (Russian)", () => {
-		expect(recommendAgent("Изучи React Server Components")?.agent).toBe("researcher");
-		expect(recommendAgent("Найди инфу про Zustand")?.agent).toBe("researcher");
-	});
-	test("debug keywords → soly-manager", () => {
-		expect(recommendAgent("fix this bug")?.agent).toBe("soly-manager");
-		expect(recommendAgent("why is this crash happening")?.agent).toBe("soly-manager");
-		expect(recommendAgent("repro the failing test")?.agent).toBe("soly-manager");
-		expect(recommendAgent("Почему падает тест?")?.agent).toBe("soly-manager");
-	});
-	test("refactor keywords → soly-manager", () => {
-		expect(recommendAgent("refactor this function")?.agent).toBe("soly-manager");
-		expect(recommendAgent("simplify the auth flow")?.agent).toBe("soly-manager");
-		expect(recommendAgent("Упрости эту функцию")?.agent).toBe("soly-manager");
-	});
-	test("test keywords → soly-manager", () => {
-		expect(recommendAgent("write tests for the parser")?.agent).toBe("soly-manager");
-		expect(recommendAgent("improve coverage")?.agent).toBe("soly-manager");
-		expect(recommendAgent("Напиши тесты для парсера")?.agent).toBe("soly-manager");
-	});
-	test("review keywords → reviewer", () => {
-		expect(recommendAgent("review this PR")?.agent).toBe("reviewer");
-		expect(recommendAgent("audit the security")?.agent).toBe("reviewer");
-		expect(recommendAgent("Проверь этот код")?.agent).toBe("reviewer");
-	});
-	test("docs keywords → soly-manager", () => {
-		expect(recommendAgent("update the readme")?.agent).toBe("soly-manager");
-		expect(recommendAgent("add jsdoc to the function")?.agent).toBe("soly-manager");
-		expect(recommendAgent("Обнови документацию")?.agent).toBe("soly-manager");
-	});
-	test("plan keywords → soly-manager", () => {
-		expect(recommendAgent("plan the migration")?.agent).toBe("soly-manager");
-		expect(recommendAgent("design the API")?.agent).toBe("soly-manager");
-		expect(recommendAgent("Спланируй миграцию")?.agent).toBe("soly-manager");
-	});
-	test("implement keywords → worker", () => {
-		expect(recommendAgent("implement the feature")?.agent).toBe("worker");
-		expect(recommendAgent("build the auth module")?.agent).toBe("worker");
-		expect(recommendAgent("Сделай эту фичу")?.agent).toBe("worker");
-	});
-	test("no match → null", () => {
-		expect(recommendAgent("hello world")).toBeNull();
-		expect(recommendAgent("")).toBeNull();
-	});
-	test("returns emoji and why", () => {
-		const r = recommendAgent("fix this bug");
-		expect(r?.emoji).toBeTruthy();
-		expect(r?.why).toBeTruthy();
-	});
-});

package/switch/tsconfig.json

@@ -1,28 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "esnext",
-    "module": "nodenext",
-    "moduleResolution": "nodenext",
-    "lib": [
-      "esnext"
-    ],
-    "types": [
-      "node"
-    ],
-    "skipLibCheck": true,
-    "noEmit": true,
-    "strict": true,
-    "esModuleInterop": true,
-    "allowSyntheticDefaultImports": true,
-    "resolveJsonModule": true,
-    "forceConsistentCasingInFileNames": true,
-    "noUncheckedIndexedAccess": false,
-    "allowImportingTsExtensions": true
-  },
-  "include": [
-    "**/*.ts"
-  ],
-  "exclude": [
-    "node_modules"
-  ]
-}