killeros 2.0.9 → 2.0.11

package/killeros/init-evidence.ts

@@ -35,11 +35,11 @@ export interface InitEvidenceBuildResult {
 
 function evidenceKey(relativePath: string): string {
   const normalized = relativePath.replaceAll("\\", "/");
-  return process.platform === "win32" ? normalized.toLocaleLowerCase() : normalized;
+  return process.platform === "win32" ? normalized.toLowerCase() : normalized;
 }
 
 function sensitiveEvidencePath(relativePath: string): boolean {
-  const normalized = relativePath.replaceAll("\\", "/").toLocaleLowerCase();
+  const normalized = relativePath.replaceAll("\\", "/").toLowerCase();
   const name = path.posix.basename(normalized);
   return /^\.env(?:\.|$)/u.test(name)
     || [".npmrc", ".pypirc", ".netrc", "id_rsa", "id_ed25519", "credentials.json"].includes(name)
@@ -50,8 +50,8 @@ function sensitiveEvidencePath(relativePath: string): boolean {
 function excludedPath(relativePath: string): boolean {
   const segments = relativePath.replaceAll("\\", "/").split("/");
   return segments.some((segment, index) =>
-    (index < segments.length - 1 && EXCLUDED_DIRS.has(segment.toLocaleLowerCase()))
-    || EXCLUDED_GUIDANCE.has(segment.toLocaleLowerCase()))
+    (index < segments.length - 1 && EXCLUDED_DIRS.has(segment.toLowerCase()))
+    || EXCLUDED_GUIDANCE.has(segment.toLowerCase()))
     || sensitiveEvidencePath(relativePath);
 }
 
@@ -74,7 +74,7 @@ async function collectCandidates(projectRoot: string): Promise<string[]> {
       if (files.length >= PATH_LIMIT) break;
       const relativePath = path.posix.join(current.relativePath.replaceAll("\\", "/"), entry.name);
       if (entry.isDirectory()) {
-        if (current.depth < DEPTH_LIMIT && !EXCLUDED_DIRS.has(entry.name.toLocaleLowerCase())) {
+        if (current.depth < DEPTH_LIMIT && !EXCLUDED_DIRS.has(entry.name.toLowerCase())) {
           queue.push({ relativePath, depth: current.depth + 1 });
         }
       } else if (entry.isFile() && !excludedPath(relativePath)) {

package/killeros/init.ts

@@ -76,7 +76,7 @@ export function registerInitCommand(pi: ExtensionAPI, initState: InitRuntime, go
       if (!initState.active || !initState.evidence || !initState.targetPath || !initState.projectRoot) {
         throw new Error("killeros_init_read is available only during /init");
       }
-      const generatedTarget = initState.outcome.kind === "written" && requestedPath.replaceAll("\\", "/").toLocaleLowerCase() === "agents.md";
+      const generatedTarget = initState.outcome.kind === "written" && requestedPath.replaceAll("\\", "/").toLowerCase() === "agents.md";
       const text = generatedTarget
         ? await readGeneratedInitTarget(initState.projectRoot, initState.targetPath)
         : await readInitEvidence(initState.evidence, requestedPath);

package/killeros/notifications.ts

@@ -1,12 +1,10 @@
-import { randomUUID } from "node:crypto";
-import { mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from "node:fs";
-import { basename, dirname, join } from "node:path";
+import { basename } from "node:path";
 import type { StopReason } from "@earendil-works/pi-ai";
 import {
-  getAgentDir,
   type ExtensionAPI,
   type ExtensionContext,
 } from "@earendil-works/pi-coding-agent";
+import { createKillerosSettingsStore } from "./settings.ts";
 
 export interface NotificationPreferenceStore {
   load(): boolean;
@@ -20,42 +18,14 @@ export interface CompletionNotificationDependencies {
 
 export const COMPLETION_BELL_GLYPH = "󰂚";
 
-const defaultSettingsPath = (): string => join(getAgentDir(), "killeros.json");
-
-type StoredSettings = Record<string, unknown>;
-
-function readStoredSettings(settingsPath: string): StoredSettings {
-  try {
-    const parsed: unknown = JSON.parse(readFileSync(settingsPath, "utf8"));
-    if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
-      throw new Error("KillerOS settings must contain a JSON object");
-    }
-    return parsed as StoredSettings;
-  } catch (error) {
-    if ((error as NodeJS.ErrnoException).code === "ENOENT") return {};
-    throw error;
-  }
-}
-
 export function createNotificationPreferenceStore(
-  settingsPath = defaultSettingsPath(),
+  settingsPath?: string,
 ): NotificationPreferenceStore {
+  const settings = createKillerosSettingsStore(settingsPath);
   return {
-    load: () => readStoredSettings(settingsPath).completionSound === true,
+    load: () => settings.load().completionSound === true,
     save: (enabled) => {
-      const current = readStoredSettings(settingsPath);
-      mkdirSync(dirname(settingsPath), { recursive: true });
-      const temporaryPath = `${settingsPath}.${process.pid}.${randomUUID()}.tmp`;
-      try {
-        writeFileSync(
-          temporaryPath,
-          `${JSON.stringify({ ...current, completionSound: enabled }, null, 2)}\n`,
-          { encoding: "utf8", mode: 0o600 },
-        );
-        renameSync(temporaryPath, settingsPath);
-      } finally {
-        rmSync(temporaryPath, { force: true });
-      }
+      settings.update({ completionSound: enabled });
     },
   };
 }

package/killeros/question.ts

@@ -45,7 +45,7 @@ const QuestionParams = Type.Object({
   })),
 });
 
-export type QuestionParamsValue = Static<typeof QuestionParams>;
+type QuestionParamsValue = Static<typeof QuestionParams>;
 
 type NormalizedQuestionSelection =
   | { mode: "single"; minSelections: 1; maxSelections: 1 }
@@ -100,11 +100,7 @@ interface MultipleQuestionDetails {
   cancelled?: boolean;
 }
 
-export type QuestionDetails = SingleQuestionDetails | MultipleQuestionDetails;
-
-export interface QuestionRunner {
-  ask(params: QuestionParamsValue, signal: AbortSignal | undefined, ctx: ExtensionContext): Promise<QuestionDetails>;
-}
+type QuestionDetails = SingleQuestionDetails | MultipleQuestionDetails;
 
 type QuestionSelection =
   | { kind: "selected"; answer: string; originalIndex: number }
@@ -219,7 +215,7 @@ class MultipleResultText {
   invalidate(): void {}
 }
 
-export function registerQuestionTool(pi: ExtensionAPI): QuestionRunner {
+export function registerQuestionTool(pi: ExtensionAPI): void {
   const customInputHistory: string[] = [];
   let customInputHistoryBytes = 0;
   const clearCustomInputHistory = (): void => {
@@ -303,7 +299,7 @@ export function registerQuestionTool(pi: ExtensionAPI): QuestionRunner {
           const keyText = keybindings.getKeys(keybinding)
             .join("/")
             .split("/")
-            .map((key) => key.split("+").map((part) => process.platform === "darwin" && part.toLocaleLowerCase() === "alt" ? "option" : part).join("+"))
+            .map((key) => key.split("+").map((part) => process.platform === "darwin" && part.toLowerCase() === "alt" ? "option" : part).join("+"))
             .join("/");
           return theme.fg("dim", keyText) + theme.fg("muted", ` ${description}`);
         };
@@ -322,11 +318,11 @@ export function registerQuestionTool(pi: ExtensionAPI): QuestionRunner {
         customInputHistory.forEach((value) => editor.addToHistory(value));
 
         const filteredOptions = (): DisplayOption[] => {
-          const query = filterQuery.trim().toLocaleLowerCase();
+          const query = filterQuery.trim().toLowerCase();
           return options.filter((option) => option.isOther
             || query.length === 0
-            || option.label.toLocaleLowerCase().includes(query)
-            || option.description?.toLocaleLowerCase().includes(query));
+            || option.label.toLowerCase().includes(query)
+            || option.description?.toLowerCase().includes(query));
         };
         const selectedCount = (): number => selectedOriginalIndices.size + (customAnswer === undefined ? 0 : 1);
         const orderedMultipleSelection = () => {
@@ -763,12 +759,4 @@ export function registerQuestionTool(pi: ExtensionAPI): QuestionRunner {
     },
   };
   pi.registerTool(questionTool);
-
-  return {
-    async ask(params, signal, ctx): Promise<QuestionDetails> {
-      const result = await questionTool.execute("killeros-workflow-gate", params, signal, undefined, ctx);
-      if (!result.details) throw new Error("Question did not return a structured result");
-      return result.details;
-    },
-  };
 }

package/killeros/runtime.ts

@@ -57,6 +57,7 @@ export interface GoalRuntime {
   continuationHeld: boolean;
   goalTurnInFlight: boolean;
   agentEndObserved: boolean;
+  automaticCompaction?: "pending" | "completed";
   persistenceRetryNeeded: boolean;
   lastStopReason?: string;
   lastError?: string;
@@ -73,6 +74,7 @@ export function createGoalRuntime(): GoalRuntime {
     continuationHeld: false,
     goalTurnInFlight: false,
     agentEndObserved: false,
+    automaticCompaction: undefined,
     persistenceRetryNeeded: false,
   };
 }

package/killeros/settings.ts

@@ -0,0 +1,47 @@
+import { randomUUID } from "node:crypto";
+import { mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from "node:fs";
+import { dirname, join } from "node:path";
+import { getAgentDir } from "@earendil-works/pi-coding-agent";
+
+export type KillerosSettings = Record<string, unknown>;
+
+export interface KillerosSettingsStore {
+  load(): KillerosSettings;
+  update(patch: Readonly<Record<string, unknown>>): void;
+}
+
+function readStoredSettings(settingsPath: string): KillerosSettings {
+  try {
+    const parsed: unknown = JSON.parse(readFileSync(settingsPath, "utf8"));
+    if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
+      throw new Error("KillerOS settings must contain a JSON object");
+    }
+    return parsed as KillerosSettings;
+  } catch (error) {
+    if ((error as NodeJS.ErrnoException).code === "ENOENT") return {};
+    throw error;
+  }
+}
+
+export function createKillerosSettingsStore(
+  settingsPath = join(getAgentDir(), "killeros.json"),
+): KillerosSettingsStore {
+  return {
+    load: () => readStoredSettings(settingsPath),
+    update: (patch) => {
+      const current = readStoredSettings(settingsPath);
+      mkdirSync(dirname(settingsPath), { recursive: true });
+      const temporaryPath = `${settingsPath}.${process.pid}.${randomUUID()}.tmp`;
+      try {
+        writeFileSync(
+          temporaryPath,
+          `${JSON.stringify({ ...current, ...patch }, null, 2)}\n`,
+          { encoding: "utf8", mode: 0o600 },
+        );
+        renameSync(temporaryPath, settingsPath);
+      } finally {
+        rmSync(temporaryPath, { force: true });
+      }
+    },
+  };
+}

package/killeros/variants.ts

@@ -46,7 +46,7 @@ function isThinkingLevel(value: string): value is ThinkingLevel {
 }
 
 function resolveThinkingLevel(input: string): ThinkingLevel | undefined {
-  const normalized = input.trim().toLocaleLowerCase();
+  const normalized = input.trim().toLowerCase();
   return isThinkingLevel(normalized) ? normalized : LEVEL_ALIASES[normalized];
 }
 
@@ -144,7 +144,7 @@ export function registerVariants(pi: ExtensionAPI): void {
           const keyText = keybindings.getKeys(keybinding)
             .join("/")
             .split("/")
-            .map((key) => key.split("+").map((part) => process.platform === "darwin" && part.toLocaleLowerCase() === "alt" ? "option" : part).join("+"))
+            .map((key) => key.split("+").map((part) => process.platform === "darwin" && part.toLowerCase() === "alt" ? "option" : part).join("+"))
             .join("/");
           return theme.fg("dim", keyText) + theme.fg("muted", ` ${description}`);
         };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "killeros",
-  "version": "2.0.9",
+  "version": "2.0.11",
   "description": "TUI, goals, and workflow automation for the Pi coding agent",
   "type": "module",
   "keywords": [
@@ -31,7 +31,7 @@
   },
   "scripts": {
     "check": "tsc --noEmit",
-    "test": "node --test --experimental-strip-types test/*.test.ts"
+    "test": "node --test --experimental-strip-types test/*.test.*"
   },
   "pi": {
     "extensions": [
@@ -42,9 +42,9 @@
     ]
   },
   "peerDependencies": {
-    "@earendil-works/pi-ai": ">=0.84.1",
-    "@earendil-works/pi-coding-agent": ">=0.84.1",
-    "@earendil-works/pi-tui": ">=0.84.1",
+    "@earendil-works/pi-ai": ">=0.84.2",
+    "@earendil-works/pi-coding-agent": ">=0.84.2",
+    "@earendil-works/pi-tui": ">=0.84.2",
     "typebox": ">=1.1.38 <2"
   },
   "devDependencies": {

package/killeros/decision-gated-workflow.ts

@@ -1,76 +0,0 @@
-import path from "node:path";
-import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
-import type { QuestionDetails } from "./question.ts";
-import type { WorkflowAdapter, WorkflowPolicy, WorkflowToolAuthorization } from "./workflow-gate.ts";
-
-const READ_ONLY_TOOLS = ["read", "grep", "find", "ls", "question"] as const;
-
-const DOCUMENTATION_PATHS: readonly RegExp[] = [
-  /^(?:docs\/)?(?:glossary|context-map)(?:\.md|\/|$)/u,
-  /^docs\/adr\/[^/]+\.md$/u,
-];
-
-function relativePath(input: Readonly<Record<string, unknown>>, ctx: ExtensionContext): string | undefined {
-  if (typeof input.path !== "string" || input.path.trim().length === 0) return;
-  const absolute = path.resolve(ctx.cwd, input.path);
-  const relative = path.relative(ctx.cwd, absolute);
-  if (!relative || relative.startsWith(`..${path.sep}`) || path.isAbsolute(relative)) return;
-  return relative.replaceAll(path.sep, "/").toLocaleLowerCase();
-}
-
-function authorizeDocumentationTool(
-  toolName: string,
-  input: Readonly<Record<string, unknown>>,
-  ctx: ExtensionContext,
-): WorkflowToolAuthorization {
-  if (toolName !== "edit" && toolName !== "write") return true;
-  const target = relativePath(input, ctx);
-  if (target && DOCUMENTATION_PATHS.some((pattern) => pattern.test(target))) return true;
-  return "With docs policy permits writes only to the agreed glossary, context-map, and ADR paths";
-}
-
-const NORMAL_POLICY: WorkflowPolicy = {
-  id: "normal",
-  allowedTools: READ_ONLY_TOOLS,
-};
-
-const WITH_DOCS_POLICY: WorkflowPolicy = {
-  id: "with-docs",
-  allowedTools: [...READ_ONLY_TOOLS, "edit", "write"],
-  authorizeTool: authorizeDocumentationTool,
-};
-
-const POLICIES = [NORMAL_POLICY, WITH_DOCS_POLICY] as const;
-
-function selectedAnswer(details: QuestionDetails): string | undefined {
-  if (!("answer" in details) || details.answer === null) return;
-  return details.answer;
-}
-
-export function createDecisionGatedWorkflowAdapter(): WorkflowAdapter {
-  return {
-    id: "decision-gated-workflow",
-    activation: "decision-gated-workflow",
-    question: {
-      question: "Choose the policy for this workflow before the model starts",
-      options: [
-        {
-          label: "Normal",
-          description: "Interview and read-only work; implementation files stay protected.",
-        },
-        {
-          label: "With docs",
-          description: "Allow only agreed glossary, context-map, or ADR documentation writes.",
-        },
-      ],
-    },
-    policies: POLICIES,
-    selectPolicy(details) {
-      switch (selectedAnswer(details)) {
-        case "Normal": return NORMAL_POLICY;
-        case "With docs": return WITH_DOCS_POLICY;
-        default: return undefined;
-      }
-    },
-  };
-}