killeros 1.5.7 → 2.0.0

package/killeros/commands.ts

@@ -1,76 +1,5 @@
-import { type ExtensionAPI, type ExtensionCommandContext, type ExtensionContext } from "@earendil-works/pi-coding-agent";
+import { type ExtensionAPI, type ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import type { AutocompleteItem } from "@earendil-works/pi-tui";
-import { formatThreadControls, type ThreadStatus } from "./subagent-ui.ts";
-
-export type SubagentControlAction = "list" | "inspect" | "wait" | "steer" | "interrupt" | "collect" | "resume" | "close";
-
-export interface SubagentControlRequest {
-  action: SubagentControlAction;
-  threadId?: string;
-  all?: true;
-  message?: string;
-  task?: string;
-  timeoutMs?: number;
-}
-
-interface SubagentControlThread {
-  id: string;
-  displayName?: string;
-  name?: string;
-  agent?: string;
-  role?: string;
-  task?: string;
-  prompt?: string;
-  status?: string;
-  state?: string;
-}
-
-export interface SubagentControlDetails {
-  results?: readonly SubagentControlThread[];
-  threads?: readonly SubagentControlThread[];
-}
-
-export interface SubagentControlResult {
-  text: string;
-  details?: SubagentControlDetails;
-  usage?: unknown;
-}
-
-export interface SubagentControlApi {
-  execute(request: SubagentControlRequest, ctx: ExtensionContext): Promise<SubagentControlResult>;
-}
-
-export interface SubagentToolLike {
-  name: string;
-  execute(
-    toolCallId: string,
-    params: unknown,
-    signal: AbortSignal | undefined,
-    onUpdate: undefined,
-    ctx: ExtensionContext,
-  ): Promise<{
-    content?: readonly { type: string; text?: string }[];
-    details?: unknown;
-    usage?: unknown;
-  }>;
-}
-
-export function createSubagentControlApi(tool: SubagentToolLike): SubagentControlApi {
-  return {
-    async execute(request, ctx) {
-      const toolRequest = request.action === "interrupt" && request.threadId === "all"
-        ? { action: "interrupt", all: true }
-        : request;
-      const result = await tool.execute("subagents-command", toolRequest, ctx.signal, undefined, ctx);
-      const text = result.content?.find((item) => item.type === "text")?.text ?? "";
-      return {
-        text,
-        details: result.details as SubagentControlDetails | undefined,
-        usage: result.usage,
-      };
-    },
-  };
-}
 
 async function confirmNewSession(ctx: ExtensionCommandContext): Promise<boolean> {
   if (!ctx.hasUI) return true;
@@ -128,7 +57,6 @@ const BUILTIN_COMMANDS: ReadonlyArray<{ name: string; description: string }> = [
 const COMMAND_SYNTAX_HINTS: Readonly<Record<string, string>> = {
   goal: "/goal [objective|clear|edit|pause|resume]",
   variants: "/variants [level]",
-  subagents: "/subagents [list|inspect|wait|steer|interrupt|collect|resume|close] [thread]",
   model: "/model [provider/model]",
   "scoped-models": "/scoped-models",
   login: "/login [provider]",
@@ -151,162 +79,6 @@ function scoreCommandMatch(name: string, prefix: string): number {
   return 0;
 }
 
-const SUBAGENT_COMMAND_USAGE = "/subagents [list|inspect|wait|steer|interrupt|collect|resume|close] [thread]";
-
-function subagentCommandError(message: string): Error {
-  return new Error(`${message} Usage: ${SUBAGENT_COMMAND_USAGE}`);
-}
-
-function parseThreadReference(action: SubagentControlAction, tail: string): string {
-  const reference = tail.match(/^(\S+)(?:\s+([\s\S]*))?$/u)?.[1];
-  if (!reference) throw subagentCommandError(`/subagents ${action} requires a thread reference.`);
-  return reference;
-}
-
-function parseExplicitSubagentCommand(args: string): SubagentControlRequest {
-  const trimmed = args.trim();
-  const match = trimmed.match(/^(\S+)(?:\s+([\s\S]*))?$/u);
-  if (!match) throw subagentCommandError("/subagents requires an explicit verb outside TUI.");
-  const action = match[1]!.toLocaleLowerCase() as SubagentControlAction;
-  const tail = match[2]?.trim() ?? "";
-
-  if (action === "list") {
-    if (tail) throw subagentCommandError("/subagents list does not accept arguments.");
-    return { action };
-  }
-  if (action === "wait") {
-    if (!tail) return { action };
-    const parts = tail.split(/\s+/u);
-    if (parts.length > 2) throw subagentCommandError("/subagents wait accepts one thread reference and one timeout-ms value.");
-    if (parts.length === 1 && /^\d+$/u.test(parts[0]!)) {
-      return { action, timeoutMs: parseTimeout(parts[0]!) };
-    }
-    const request: SubagentControlRequest = { action, threadId: parts[0] };
-    if (parts[1] !== undefined) request.timeoutMs = parseTimeout(parts[1]);
-    return request;
-  }
-  if (action === "steer") {
-    const referenceAndMessage = tail.match(/^(\S+)(?:\s+([\s\S]+))?$/u);
-    if (!referenceAndMessage?.[1]) throw subagentCommandError("/subagents steer requires a thread reference.");
-    if (!referenceAndMessage[2]?.trim()) throw subagentCommandError("/subagents steer requires a message.");
-    return { action, threadId: referenceAndMessage[1], message: referenceAndMessage[2] };
-  }
-  if (action === "resume") {
-    const referenceAndTask = tail.match(/^(\S+)(?:\s+([\s\S]+))?$/u);
-    if (!referenceAndTask?.[1]) throw subagentCommandError("/subagents resume requires a thread reference.");
-    return {
-      action,
-      threadId: referenceAndTask[1],
-      ...(referenceAndTask[2] ? { task: referenceAndTask[2] } : {}),
-    };
-  }
-  if (action === "inspect" || action === "interrupt" || action === "collect" || action === "close") {
-    const threadId = parseThreadReference(action, tail);
-    if (tail.slice(threadId.length).trim()) throw subagentCommandError(`/subagents ${action} accepts one thread reference.`);
-    return { action, threadId };
-  }
-  throw subagentCommandError(`Unknown /subagents action ${JSON.stringify(match[1])}.`);
-}
-
-function parseTimeout(value: string): number {
-  const timeoutMs = Number(value);
-  if (!Number.isSafeInteger(timeoutMs) || timeoutMs < 0) {
-    throw subagentCommandError("/subagents wait timeout-ms must be a non-negative integer.");
-  }
-  return timeoutMs;
-}
-
-function controlThreads(result: SubagentControlResult): SubagentControlThread[] {
-  const details = result.details;
-  if (!details) return [];
-  const results = [...(details.results ?? [])];
-  const threads = [...(details.threads ?? [])];
-  const candidates = results.length ? results : threads;
-  return candidates.filter((thread) => thread && typeof thread.id === "string" && thread.state !== "closed" && thread.status !== "closed");
-}
-
-function threadStatus(thread: SubagentControlThread): ThreadStatus {
-  const status = (thread.status ?? thread.state ?? "queued").toLocaleLowerCase();
-  if (status === "active" || status === "running") return "running";
-  if (status === "done" || status === "complete" || status === "closed") return "complete";
-  if (status === "stopped" || status === "cancelled") return "cancelled";
-  if (status === "limited") return "limited";
-  if (status === "orphaned") return "orphaned";
-  if (status === "failed") return "failed";
-  return "queued";
-}
-
-function threadLabel(thread: SubagentControlThread): string {
-  const name = thread.displayName ?? thread.name ?? thread.agent ?? thread.role ?? thread.id;
-  return `${name} · ${thread.id} · ${threadStatus(thread)}`;
-}
-
-function selectedThread(threads: readonly SubagentControlThread[], labels: readonly string[], choice: string): SubagentControlThread | undefined {
-  const index = labels.indexOf(choice);
-  if (index >= 0) return threads[index];
-  return threads.find((thread) => thread.id === choice || thread.displayName === choice || thread.name === choice);
-}
-
-async function executeSubagentControl(
-  control: SubagentControlApi | undefined,
-  request: SubagentControlRequest,
-  ctx: ExtensionCommandContext,
-): Promise<void> {
-  if (!control) throw new Error("Subagent control API is not available.");
-  const result = await control.execute(request, ctx);
-  if (result?.text) ctx.ui.notify(result.text, "info");
-}
-
-async function runTuiSubagentCommand(control: SubagentControlApi | undefined, ctx: ExtensionCommandContext): Promise<void> {
-  if (!control) throw new Error("Subagent control API is not available.");
-  const listed = await control.execute({ action: "list" }, ctx);
-  const threads = controlThreads(listed);
-  if (!threads.length) {
-    ctx.ui.notify("No child threads.", "info");
-    return;
-  }
-
-  const labels = threads.map(threadLabel);
-  const selected = await ctx.ui.select("Select a thread", labels);
-  if (selected === undefined) return;
-  const thread = selectedThread(threads, labels, selected);
-  if (!thread) return;
-
-  const controls = formatThreadControls(threadStatus(thread)).filter((item) => item.enabled);
-  const controlLabels = controls.map((item) => item.label);
-  const selectedControl = await ctx.ui.select("Select a control", controlLabels);
-  if (selectedControl === undefined) return;
-  const chosen = controls.find((item) => item.label === selectedControl || item.id === selectedControl);
-  if (!chosen) return;
-
-  const request: SubagentControlRequest = { action: chosen.id, threadId: thread.id };
-  if (chosen.id === "steer") {
-    const message = await ctx.ui.input("Steer child thread", "Message");
-    if (message === undefined || !message.trim()) return;
-    request.message = message;
-  } else if (chosen.id === "resume") {
-    const task = await ctx.ui.input("Resume child thread", "Optional task");
-    if (task === undefined) return;
-    if (task) request.task = task;
-  }
-  await executeSubagentControl(control, request, ctx);
-}
-
-export function registerSubagentCommand(pi: ExtensionAPI, control?: SubagentControlApi | void): void {
-  const api = control && typeof control.execute === "function" ? control : undefined;
-  pi.registerCommand("subagents", {
-    description: "Inspect and control child threads",
-    handler: async (args, ctx) => {
-      if (!args.trim()) {
-        if (ctx.mode !== "tui") throw subagentCommandError("/subagents requires an explicit verb outside TUI.");
-        await runTuiSubagentCommand(api, ctx);
-        return;
-      }
-      await executeSubagentControl(api, parseExplicitSubagentCommand(args), ctx);
-    },
-  });
-}
-
 export function registerSlashAutocomplete(pi: ExtensionAPI): void {
   const usage = new Map<string, number>();
   pi.on("session_start", (_event, ctx) => {
@@ -349,15 +121,6 @@ export function registerSlashAutocomplete(pi: ExtensionAPI): void {
           }
         }
 
-        if (!commands.has("subagents")) {
-          commands.set("subagents", {
-            name: "subagents",
-            description: "Inspect and control child threads",
-            category: "Extension",
-            syntaxHint: COMMAND_SYNTAX_HINTS.subagents,
-          });
-        }
-
         const ranked = [...commands.values()]
           .map((command) => ({
             command,

package/killeros/display.ts

@@ -6,9 +6,11 @@ export function formatCwd(cwd: string): string {
   if (!home) return cwd;
   const normalizedHome = home.replace(/[\\/]+$/, "");
   const normalizedCwd = cwd.replace(/[\\/]+$/, "");
-  if (normalizedCwd === normalizedHome) return "~";
+  const comparedHome = process.platform === "win32" ? normalizedHome.toLocaleLowerCase() : normalizedHome;
+  const comparedCwd = process.platform === "win32" ? normalizedCwd.toLocaleLowerCase() : normalizedCwd;
+  if (comparedCwd === comparedHome) return "~";
   const separator = normalizedCwd.slice(normalizedHome.length, normalizedHome.length + 1);
-  return normalizedCwd.startsWith(normalizedHome) && (separator === "/" || separator === "\\")
+  return comparedCwd.startsWith(comparedHome) && (separator === "/" || separator === "\\")
     ? `~${normalizedCwd.slice(normalizedHome.length)}`
     : cwd;
 }
@@ -20,6 +22,7 @@ export function padRight(text: string, width: number): string {
 }
 
 export function formatTime(milliseconds: number): string {
+  if (!Number.isFinite(milliseconds)) return "0s";
   const totalSeconds = Math.max(0, Math.floor(milliseconds / 1_000));
   if (totalSeconds < 60) return `${totalSeconds}s`;
   const minutes = Math.floor(totalSeconds / 60);
@@ -28,6 +31,7 @@ export function formatTime(milliseconds: number): string {
 }
 
 export function formatTokens(value: number): string {
+  if (!Number.isFinite(value)) return "0";
   const amount = Math.max(0, value);
   if (amount < 1_000) return `${Math.round(amount)}`;
   if (amount >= 1_000_000) {

package/killeros/footer.ts

@@ -13,8 +13,8 @@ export function formatCost(usd: number): string {
 }
 
 export function formatContextProgress(tokensUsed: number | null, contextWindow: number, theme: Theme): string {
-  if (tokensUsed === null) return theme.fg("dim", "—% left (—)");
-  const windowSize = contextWindow > 0 ? contextWindow : 128_000;
+  if (tokensUsed === null || !Number.isFinite(tokensUsed)) return theme.fg("dim", "—% left (—)");
+  const windowSize = Number.isFinite(contextWindow) && contextWindow > 0 ? contextWindow : 128_000;
   const remaining = Math.max(0, Math.min(windowSize, windowSize - Math.max(0, tokensUsed)));
   const percentLeft = Math.max(0, Math.min(100, Math.round((remaining / windowSize) * 100)));
   const color: ThemeColor = percentLeft < 20 ? "error" : percentLeft <= 50 ? "warning" : "success";
@@ -150,7 +150,12 @@ export function registerFooter(pi: ExtensionAPI, goalRuntime: GoalRuntime): void
           const level = model?.reasoning === false
             ? theme.fg("thinkingOff", "no reasoning")
             : theme.fg(LEVEL_COLORS[thinkingLevel], thinkingLevel);
-          const usage = ctx.getContextUsage();
+          let usage: ReturnType<ExtensionContext["getContextUsage"]>;
+          try {
+            usage = ctx.getContextUsage();
+          } catch {
+            usage = undefined;
+          }
           const contextWindow = usage?.contextWindow ?? model?.contextWindow ?? 128_000;
           const context = formatContextProgress(usage?.tokens ?? null, contextWindow, theme);
           const branch = footerData.getGitBranch();

package/killeros/goals.ts

@@ -1,7 +1,7 @@
 import { type ExtensionAPI, type ExtensionContext, type ThemeColor } from "@earendil-works/pi-coding-agent";
 import { Text } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
-import { MAX_NODE_TIMER_MS } from "./subagent-process.ts";
+import { MAX_NODE_TIMER_MS } from "./limits.ts";
 import { CONCISE_SYSTEM_PROMPT } from "./concise.ts";
 import { formatTime, formatTokens } from "./display.ts";
 import { reportError } from "./errors.ts";
@@ -209,7 +209,7 @@ function scheduleGoalContinuation(
   runtime: GoalRuntime,
   initState: InitRuntime,
   ctx: ExtensionContext,
-): void {
+): boolean {
   if (!isGoalModeSupported(ctx)
     || !isSavedSession(ctx)
     || runtime.state?.status !== "active"
@@ -217,7 +217,7 @@ function scheduleGoalContinuation(
     || runtime.continuationHeld
     || runtime.goalTurnInFlight
     || initState.active
-    || ctx.hasPendingMessages()) return;
+    || ctx.hasPendingMessages()) return false;
   const current = runtime.state;
   runtime.continuationScheduled = true;
   runtime.goalTurnInFlight = false;
@@ -230,10 +230,12 @@ function scheduleGoalContinuation(
       content: goalContinuationMessage(current, ctx),
       display: false,
     }, { triggerTurn: true, deliverAs: "followUp" });
+    return true;
   } catch (error) {
     runtime.continuationScheduled = false;
     runtime.goalTurnInFlight = false;
     pauseGoalAfterFailure(pi, runtime, ctx, `continuation could not start: ${error instanceof Error ? error.message : String(error)}`);
+    return false;
   }
 }
 
@@ -551,8 +553,7 @@ export function registerGoal(
         try {
           transitionGoal(pi, runtime, "resume", "active", undefined, true);
           runtime.continuationScheduled = false;
-          scheduleGoalContinuation(pi, runtime, initState, ctx);
-          ctx.ui.notify("Goal resumed", "info");
+          if (scheduleGoalContinuation(pi, runtime, initState, ctx)) ctx.ui.notify("Goal resumed", "info");
         } catch (error) {
           reportError(ctx, "Goal could not be resumed", error);
         }
@@ -612,8 +613,7 @@ export function registerGoal(
         try {
           persistGoalState(pi, runtime, "edit", next);
           runtime.continuationScheduled = false;
-          scheduleGoalContinuation(pi, runtime, initState, ctx);
-          ctx.ui.notify("Goal updated and active", "info");
+          if (scheduleGoalContinuation(pi, runtime, initState, ctx)) ctx.ui.notify("Goal updated and active", "info");
         } catch (error) {
           pauseGoalAfterFailure(
             pi,
@@ -677,8 +677,9 @@ export function registerGoal(
       };
       try {
         persistGoalState(pi, runtime, unfinished ? "replace" : "set", state);
-        scheduleGoalContinuation(pi, runtime, initState, ctx);
-        ctx.ui.notify("Goal active. KillerOS will continue until completion, a repeated blocker, or pause.", "info");
+        if (scheduleGoalContinuation(pi, runtime, initState, ctx)) {
+          ctx.ui.notify("Goal active. KillerOS will continue until completion, a repeated blocker, or pause.", "info");
+        }
       } catch (error) {
         reportError(ctx, "Goal could not be started", error);
         scheduleGoalContinuation(pi, runtime, initState, ctx);

package/killeros/hooks.ts

@@ -3,7 +3,7 @@ import { existsSync, readFileSync } from "node:fs";
 import path from "node:path";
 import { CONFIG_DIR_NAME, type ExtensionAPI, type ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { reportError } from "./errors.ts";
-import { MAX_NODE_TIMER_MS } from "./subagent-process.ts";
+import { MAX_NODE_TIMER_MS } from "./limits.ts";
 
 type KillerosHookEvent = "tool_call" | "tool_result" | "agent_settled";
 

package/killeros/limits.ts

@@ -0,0 +1 @@
+export const MAX_NODE_TIMER_MS = 2_147_483_647;

package/killeros/personal-instructions.ts

@@ -1,6 +1,7 @@
 import { closeSync, existsSync, openSync, readFileSync, readSync } from "node:fs";
 import os from "node:os";
 import path from "node:path";
+import { StringDecoder } from "node:string_decoder";
 import { fileURLToPath } from "node:url";
 import { CONFIG_DIR_NAME, type ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import type { InitRuntime } from "./runtime.ts";
@@ -14,7 +15,8 @@ function readBoundedText(filePath: string, limit = PERSONAL_INSTRUCTIONS_LIMIT):
     descriptor = openSync(filePath, "r");
     const buffer = Buffer.alloc(limit + 1);
     const bytesRead = readSync(descriptor, buffer, 0, buffer.length, 0);
-    const content = buffer.toString("utf8", 0, Math.min(bytesRead, limit));
+    const decoder = new StringDecoder("utf8");
+    const content = decoder.write(buffer.subarray(0, Math.min(bytesRead, limit)));
     if (!content.trim()) return undefined;
     return bytesRead > limit
       ? `${content}\n\n[Personal instructions truncated by KillerOS]`

package/killeros/question.ts

@@ -56,6 +56,8 @@ type QuestionSelection =
 const CUSTOM_INPUT_MAX_CHARACTERS = 4_000;
 const CUSTOM_INPUT_HISTORY_LIMIT = 100;
 const CUSTOM_INPUT_HISTORY_BYTES = 64 * 1024;
+const FILTER_QUERY_MAX_CHARACTERS = 4_000;
+const FILTER_QUERY_MAX_BYTES = 16_000;
 
 function isPrintableInput(data: string): boolean {
   return data.length > 0 && !/[\u0000-\u001F\u007F-\u009F]/u.test(data);
@@ -123,7 +125,7 @@ export function registerQuestionTool(pi: ExtensionAPI): void {
   pi.registerTool<typeof QuestionParams, QuestionDetails>({
     name: "question",
     label: "Question",
-    description: "Ask one interactive multiple-choice question. Provide 1-9 concise options. The user can filter options or type a custom answer.",
+    description: `Ask one interactive multiple-choice question. Provide 1-9 concise options. The user can filter options or type a custom answer. Filter queries are limited to ${FILTER_QUERY_MAX_CHARACTERS.toLocaleString()} characters and ${FILTER_QUERY_MAX_BYTES.toLocaleString()} bytes.`,
     promptSnippet: "Ask the user one multiple-choice question when a decision is required to proceed",
     promptGuidelines: [
       "Use question only when user input is required to choose between concrete alternatives; do not use question for rhetorical or optional follow-up prompts.",
@@ -198,6 +200,21 @@ export function registerQuestionTool(pi: ExtensionAPI): void {
           tui.requestRender();
         };
 
+        const appendFilterInput = (value: string): void => {
+          const nextCharacters = inputCharacterCount(filterQuery) + inputCharacterCount(value);
+          const nextBytes = Buffer.byteLength(filterQuery, "utf8") + Buffer.byteLength(value, "utf8");
+          if (nextCharacters > FILTER_QUERY_MAX_CHARACTERS || nextBytes > FILTER_QUERY_MAX_BYTES) {
+            ctx.ui.notify(
+              `Question filters are limited to ${FILTER_QUERY_MAX_CHARACTERS.toLocaleString()} characters and ${FILTER_QUERY_MAX_BYTES.toLocaleString()} bytes`,
+              "error",
+            );
+            return;
+          }
+          filterQuery += value;
+          optionIndex = 0;
+          refresh();
+        };
+
         editor.onSubmit = (value) => {
           const answer = value.trim();
           if (answer) {
@@ -288,9 +305,7 @@ export function registerQuestionTool(pi: ExtensionAPI): void {
             return;
           }
           if (printableInput) {
-            filterQuery += printableInput;
-            optionIndex = 0;
-            refresh();
+            appendFilterInput(printableInput);
           }
         };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "killeros",
-  "version": "1.5.7",
+  "version": "2.0.0",
   "description": "A production-hardened TUI and workflow extension for the Pi coding agent.",
   "type": "module",
   "keywords": [
@@ -22,11 +22,6 @@
   "files": [
     "Killeros.ts",
     "killeros/*.ts",
-    "subagents.ts",
-    "subagent-lifecycle.ts",
-    "subagent-process.ts",
-    "subagent-ui.ts",
-    "agents/*.md",
     "themes/killeros.json",
     "README.md",
     "CHANGELOG.md"
@@ -36,7 +31,7 @@
   },
   "scripts": {
     "check": "tsc --noEmit",
-    "test": "node --test --experimental-strip-types test/*.test.js"
+    "test": "node --test --experimental-strip-types test/*.test.ts"
   },
   "pi": {
     "extensions": [
@@ -50,7 +45,6 @@
     "@earendil-works/pi-ai": ">=0.82.1",
     "@earendil-works/pi-coding-agent": ">=0.82.1",
     "@earendil-works/pi-tui": ">=0.82.1",
-    "pi-web-access": ">=0.17.1",
     "typebox": ">=1.1.38 <2"
   },
   "devDependencies": {

package/agents/debugger.md

@@ -1,50 +0,0 @@
----
-name: debugger
-description: debugger — reproduce failures, eliminate competing root-cause hypotheses, fix the shared cause, and prove the regression is gone
-access: write
-tools: read, grep, find, ls, edit, write, bash, web_search, source_check, fetch_content, get_search_content
-# Replace inherit with provider/model to pin this role; set thinking separately when needed.
-model: inherit
-thinking: inherit
----
-
-# Role
-
-You are the `debugger` role, a calm incident investigator who treats a symptom as a clue, never as a diagnosis. You may repair the code, but only after the failure and its cause are understood well enough to avoid a plausible-looking patch.
-
-## Diagnostic gate
-
-Require a concrete error, failing test, reproduction step, expected result, or observable mismatch. If the report is too vague, state the missing evidence and the cheapest way to obtain it instead of guessing or editing around the symptom.
-
-## Investigation
-
-1. **Reproduce.** Use the smallest existing command or test and preserve the actual output and conditions that matter.
-2. **Trace.** Follow entry point to state transition to failure. Inspect every relevant caller, boundary, shared helper, cleanup path, and error transformation.
-3. **Classify.** Decide whether the failure is runtime, logic, integration, configuration, dependency, timing, or data-flow related.
-4. **Compete.** Keep two or three plausible hypotheses when the cause is not proven. Test the cheapest falsifier first. Use targeted history or blame only when current code leaves competing explanations.
-5. **Prove.** Create a minimal reproduction or regression test before the fix when practical, then make the smallest root-cause change without unrelated refactoring.
-
-## Verification
-
-Rerun the original proof and the nearest regression checks. A passing unrelated test is not evidence that the reported bug is fixed. If the failure cannot be reproduced or the repair cannot be verified, say so plainly and do not claim completion.
-
-## Report
-
-Return the diagnosis, evidence chain, competing hypotheses that were eliminated, changed paths, commands and results, residual risk, and the next missing proof. Keep the repair narrow and leave broader cleanup to a separate request.
-
-## Skills and web research
-
-Before doing task work, always inspect the available skill list and load the most relevant skill with `read` from its `SKILL.md`. If no relevant skill exists, say so instead of inventing one.
-
-When the task depends on current facts, external documentation, standards, package behavior, or a user-requested web lookup, use `web_search` to find sources and `fetch_content` to read the strongest pages. Use `source_check` when a claim needs exact passage evidence and `get_search_content` to retrieve bounded slices from stored results. Prefer primary sources, vary research queries when the question is broad, and cite URLs in the report. Do not claim to have searched or loaded a skill unless the tool call succeeded.
-
-## Communication
-
-Use these six rules in every response:
-
-1. Never use a metaphor, simile, or other figure of speech which you are used to seeing in print.
-2. Never use a long word where a short one will do.
-3. If it is possible to cut a word out, always cut it out.
-4. Never use the passive where you can use the active.
-5. Never use a foreign phrase, a scientific word, or a jargon word if you can think of an everyday English equivalent.
-6. Break any of these rules sooner than say anything outright barbarous.

package/agents/documenter.md

@@ -1,49 +0,0 @@
----
-name: documenter
-description: documenter — make source-backed docs answer the reader’s next question while preserving exact code, command, and behavior parity
-access: write
-tools: read, grep, find, ls, edit, write, bash, web_search, source_check, fetch_content, get_search_content
-# Replace inherit with provider/model to pin this role; set thinking separately when needed.
-model: inherit
-thinking: inherit
----
-
-# Role
-
-You are the `documenter` role, a technical writer who treats documentation as part of the product’s interface, not a place to decorate guesses. The reader should finish knowing what to do, what will happen, and what to do when it does not.
-
-## Reader contract
-
-Before editing, identify the audience, their existing knowledge, the job they are trying to complete, the document’s scope, and its non-scope. Put the essential answer first; a busy reader may only see the opening paragraph.
-
-## Source of truth
-
-Read the implementation, tests, manifests, configuration, and existing documentation that establish the behavior. Source and runnable checks outrank stale prose. Derive every command, option, example, guarantee, version, and limitation from repository evidence. Never invent a feature, benchmark, workflow, or user outcome.
-
-## Writing workflow
-
-1. Map the reader’s goal to the smallest useful path: prerequisites, exact action, expected result, failure recovery, and useful depth.
-2. Preserve project terminology and voice. Explain unfamiliar concepts by relating them to behavior the reader already knows.
-3. Reuse verified examples and update the smallest relevant documentation surface; do not rewrite unrelated prose for style.
-4. Keep README, API, configuration, and code claims in parity. Distinguish source files from generated output and call out ambiguity instead of laundering it into confident text.
-
-## Verification and report
-
-Check links, headings, code fences, examples, paths, versions, and cross-references when practical. Report the audience and evidence used, changed paths, checks performed, and any behavior that still needs an authoritative decision. Do not modify production code unless the request explicitly includes it.
-
-## Skills and web research
-
-Before doing task work, always inspect the available skill list and load the most relevant skill with `read` from its `SKILL.md`. If no relevant skill exists, say so instead of inventing one.
-
-When the task depends on current facts, external documentation, standards, package behavior, or a user-requested web lookup, use `web_search` to find sources and `fetch_content` to read the strongest pages. Use `source_check` when a claim needs exact passage evidence and `get_search_content` to retrieve bounded slices from stored results. Prefer primary sources, vary research queries when the question is broad, and cite URLs in the report. Do not claim to have searched or loaded a skill unless the tool call succeeded.
-
-## Communication
-
-Use these six rules in every response:
-
-1. Never use a metaphor, simile, or other figure of speech which you are used to seeing in print.
-2. Never use a long word where a short one will do.
-3. If it is possible to cut a word out, always cut it out.
-4. Never use the passive where you can use the active.
-5. Never use a foreign phrase, a scientific word, or a jargon word if you can think of an everyday English equivalent.
-6. Break any of these rules sooner than say anything outright barbarous.

package/agents/planner.md

@@ -1,53 +0,0 @@
----
-name: planner
-description: planner — turn an ambiguous request into the smallest buildable route with explicit evidence, contracts, decisions, and proof
-access: read
-tools: read, grep, find, ls, web_search, source_check, fetch_content, get_search_content
-# Replace inherit with provider/model to pin this role; set thinking separately when needed.
-model: inherit
-thinking: inherit
----
-
-# Role
-
-You are the `planner` role, a skeptical implementation strategist rather than a code generator. A good plan is not a paraphrase of the request: it is a verified route from the current repository to an observable result.
-
-## Workflow
-
-1. **Frame the request.** Translate it into an outcome, acceptance criteria, explicit non-goals, and decisions that still need the user’s answer. Treat explicit requirements as binding; challenge only speculative expansion.
-2. **Read the repository.** Inspect manifests, entry points, callers, tests, conventions, and current behavior. Treat source and runnable checks as stronger evidence than filenames or assumptions.
-3. **Map the change.** Name the exact files and symbols involved. Trace relevant data flow, control flow, boundaries, reuse points, dependencies, and compatibility risks.
-4. **Choose the smallest route.** Prefer an existing pattern, then the standard library or native behavior, then an installed dependency, and only then new code or an abstraction. Explain why a new file or dependency is necessary.
-5. **Make proof executable.** Pair each implementation step with focused checks, meaningful edge cases, and a clear success condition. Include rollback or containment concerns when the change has operational risk.
-6. **Separate certainty levels.** Label verified facts, inferences, assumptions, and unknowns. Never turn an unverified guess into a contract for the worker.
-
-## Deliverable
-
-Return a compact plan with:
-
-- the goal and non-goals;
-- evidence and affected paths or symbols;
-- the ordered implementation route and contracts between steps;
-- focused tests or commands that will prove the result;
-- risks, alternatives, and unresolved decisions.
-
-## Boundaries
-
-Do not modify files. Do not produce speculative architecture, a feature tour, or implementation code disguised as a plan. Stop exploring once the plan is supported by repository evidence.
-
-## Skills and web research
-
-Before doing task work, always inspect the available skill list and load the most relevant skill with `read` from its `SKILL.md`. If no relevant skill exists, say so instead of inventing one.
-
-When the task depends on current facts, external documentation, standards, package behavior, or a user-requested web lookup, use `web_search` to find sources and `fetch_content` to read the strongest pages. Use `source_check` when a claim needs exact passage evidence and `get_search_content` to retrieve bounded slices from stored results. Prefer primary sources, vary research queries when the question is broad, and cite URLs in the report. Do not claim to have searched or loaded a skill unless the tool call succeeded.
-
-## Communication
-
-Use these six rules in every response:
-
-1. Never use a metaphor, simile, or other figure of speech which you are used to seeing in print.
-2. Never use a long word where a short one will do.
-3. If it is possible to cut a word out, always cut it out.
-4. Never use the passive where you can use the active.
-5. Never use a foreign phrase, a scientific word, or a jargon word if you can think of an everyday English equivalent.
-6. Break any of these rules sooner than say anything outright barbarous.