@jaybeeuu/agent-cortex 2.3.0 → 2.4.1

package/extensions/auto-name.ts

@@ -1,30 +1,29 @@
 /**
- * Auto-name sessions using a tiny side-agent LLM.
+ * Auto-name sessions from the first user prompt.
  *
- * On the first user prompt of a new session, calls a cheap configured model
- * (default: opencode/mimo-v2.5-free) to generate a concise session title,
- * then sets it via pi.setSessionName(). Falls back to simple truncation if
- * the LLM call fails.
+ * On the first prompt of a new session, names the session with a trimmed
+ * version of that prompt, then sets it via pi.setSessionName(). Existing names
+ * (set via --name or /name) are never overwritten.
+ *
+ * The name used to come from a cheap side-model, which needed pi-ai's
+ * `complete()` — a value import that stopped this extension loading outside the
+ * agent-cortex checkout (see test/pi-extensions-load.test.mjs). The truncation
+ * that already served as its fallback is now the only path; `tinyModel.enabled`
+ * still turns naming off entirely.
  *
  * Config (in settings.json under `tinyModel`):
  * ```json
  * {
  *   "tinyModel": {
- *     "model": "opencode/mimo-v2.5-free",
+ *     "enabled": true,
  *     "maxNameLength": 50
  *   }
  * }
  * ```
- *
- * Existing names (set via --name or /name) are never overwritten.
  */
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import {
-  loadTinyModelConfig,
-  parseModelString,
-  generateNameFromPrompt,
-} from "./lib/tiny-model.js";
+import { loadTinyModelConfig } from "./lib/tiny-model.js";
 import type { TinyModelConfig } from "./lib/tiny-model.js";
 
 export default function (pi: ExtensionAPI) {
@@ -41,7 +40,7 @@ export default function (pi: ExtensionAPI) {
     }
   });
 
-  pi.on("input", async (event, ctx) => {
+  pi.on("input", async (event, _ctx) => {
     if (!config.enabled || hasNamed) return;
     if (event.source === "rpc") return;
     if (pi.getSessionName()) {
@@ -51,46 +50,14 @@ export default function (pi: ExtensionAPI) {
     if (!event.text?.trim()) return;
 
     hasNamed = true;
-
-    // Fire the LLM call asynchronously — don't block the agent
-    (async () => {
-      try {
-        const { provider, id } = parseModelString(config.model);
-        const model = ctx.modelRegistry.find(provider, id);
-        if (!model) {
-          // Model not found; fall back to truncation
-          fallbackName(event.text, config.maxNameLength, pi);
-          return;
-        }
-
-        const { apiKey, headers } =
-          await ctx.modelRegistry.getApiKeyAndHeaders(model);
-
-        const name = await generateNameFromPrompt(
-          model,
-          apiKey,
-          headers,
-          event.text,
-          config.maxNameLength,
-        );
-
-        if (name) {
-          pi.setSessionName(name);
-        } else {
-          fallbackName(event.text, config.maxNameLength, pi);
-        }
-      } catch (err) {
-        console.warn("[auto-name] LLM call failed, falling back:", err);
-        fallbackName(event.text, config.maxNameLength, pi);
-      }
-    })();
+    nameFromPrompt(event.text, config.maxNameLength, pi);
   });
 }
 
 /**
- * Fallback: truncate the first line of the prompt to use as a session name.
+ * Name the session with the first line of the prompt, truncated to `maxLength`.
  */
-function fallbackName(
+function nameFromPrompt(
   text: string,
   maxLength: number,
   pi: ExtensionAPI,

package/extensions/lib/tiny-model.ts

@@ -1,43 +1,45 @@
 /**
- * Shared helpers for making cheap LLM calls from extensions.
+ * The `tinyModel` config block shared by the extensions that used to call a
+ * cheap side-model: session naming (auto-name) and response summaries (notify).
  *
- * Reads a `tinyModel` config key from settings.json (global + project merged)
- * and provides functions for session naming and response summarization using
- * the configured model — typically the tiniest free model available.
- *
- * Config (in ~/.pi/agent/settings.json or .pi/settings.json):
+ * Reads the key from settings.json (global + project merged):
  * ```json
  * {
  *   "tinyModel": {
- *     "model": "opencode/mimo-v2.5-free",
+ *     "enabled": true,
  *     "maxNameLength": 50,
  *     "maxSummaryLength": 100
  *   }
  * }
  * ```
+ *
+ * The model call itself is gone. It went through pi-ai's `complete()`, a value
+ * import of pi's runtime, and that import stopped the extension loading outside
+ * the agent-cortex checkout (see test/pi-extensions-load.test.mjs). The
+ * ExtensionAPI promises no completion call. Newer hosts (pi 1.4.0) expose
+ * `ModelRegistry.complete`, but the installed host (0.73.1) does not. Calling it
+ * would need an untyped cast to a method that exists on one version only.
+ * Extensions also make no internal LLM calls (AGENTS.md), so this feature stays
+ * off. Both callers fall back to the truncation they already used when the model
+ * failed. The `model` key is no longer read and is ignored if present.
  */
 
-import { complete } from "@earendil-works/pi-ai";
 import { homedir } from "node:os";
 import { join } from "node:path";
 import { readFileSync, existsSync } from "node:fs";
-import type { Model, Api } from "@earendil-works/pi-ai";
 
 /* ── Types ── */
 
 export interface TinyModelConfig {
-  /** Provider/id string, e.g. "opencode/mimo-v2.5-free" */
-  model: string;
   /** Global on/off toggle */
   enabled: boolean;
-  /** Max characters for auto-generated session names */
+  /** Max characters for generated session names */
   maxNameLength: number;
   /** Max characters for response summaries in notifications */
   maxSummaryLength: number;
 }
 
 const DEFAULT_CONFIG: TinyModelConfig = {
-  model: "opencode/mimo-v2.5-free",
   enabled: true,
   maxNameLength: 40,
   maxSummaryLength: 100,
@@ -72,141 +74,3 @@ export function loadTinyModelConfig(cwd?: string): TinyModelConfig {
 
   return { ...DEFAULT_CONFIG, ...merged } as TinyModelConfig;
 }
-
-/**
- * Parse a "provider/id" string into its parts.
- */
-export function parseModelString(modelStr: string): {
-  provider: string;
-  id: string;
-} {
-  const slash = modelStr.indexOf("/");
-  if (slash === -1) return { provider: "opencode", id: modelStr };
-  return {
-    provider: modelStr.slice(0, slash),
-    id: modelStr.slice(slash + 1),
-  };
-}
-
-/* ── LLM helpers ── */
-
-type ModelAny = Model<Api>;
-type CompleteOptions = Record<string, any>;
-
-/**
- * Generate a concise session title from the user's first prompt.
- *
- * Calls the configured tiny model with a system prompt tuned for naming.
- * Returns null on failure (caller should fall back to truncation).
- */
-export async function generateNameFromPrompt(
-  model: ModelAny,
-  apiKey: string | undefined,
-  headers: Record<string, string> | undefined,
-  prompt: string,
-  maxLength: number,
-): Promise<string | null> {
-  const systemPrompt = [
-    "You are a session naming assistant. Generate a concise title for a coding session.",
-    "",
-    "Rules:",
-    "- Output ONLY the title, no quotes, no prefix, no explanation",
-    `- Maximum ${maxLength} characters`,
-    "- Summarize intent, do not copy verbatim",
-    "- Keep it short (3\u20135 words), capture the general theme",
-    "- If the message mentions files or modules, keep the key names brief",
-  ].join("\n");
-
-  const truncated = prompt.length > 2000 ? prompt.slice(0, 2000) + "..." : prompt;
-
-  try {
-    const options: CompleteOptions = { maxTokens: 64 };
-    if (apiKey) options.apiKey = apiKey;
-    if (headers) options.headers = headers;
-
-    const result = await complete(model, {
-      systemPrompt,
-      messages: [{ role: "user", content: truncated, timestamp: Date.now() }],
-    }, options);
-
-    const text = result.content
-      ?.filter((b: any): b is { type: "text"; text: string } => b.type === "text")
-      ?.map((b) => b.text)
-      ?.join("")
-      ?.trim() ?? "";
-
-    return cleanTitle(text, maxLength);
-  } catch {
-    return null;
-  }
-}
-
-/**
- * Summarize an assistant response into a one-line summary.
- *
- * Calls the configured tiny model. Returns null on failure.
- */
-export async function summarizeResponseText(
-  model: ModelAny,
-  apiKey: string | undefined,
-  headers: Record<string, string> | undefined,
-  responseText: string,
-  maxLength: number,
-): Promise<string | null> {
-  const systemPrompt = [
-    "Summarize what the AI assistant did in this coding session response.",
-    "Be concise — one short sentence or phrase.",
-    `Maximum ${maxLength} characters.`,
-    "Focus on what changed or was accomplished.",
-    "Output ONLY the summary, no quotes, no prefix, no explanation.",
-  ].join("\n");
-
-  const truncated =
-    responseText.length > 2000 ? responseText.slice(0, 2000) + "..." : responseText;
-
-  try {
-    const options: CompleteOptions = { maxTokens: 64 };
-    if (apiKey) options.apiKey = apiKey;
-    if (headers) options.headers = headers;
-
-    const result = await complete(model, {
-      systemPrompt,
-      messages: [{ role: "user", content: truncated, timestamp: Date.now() }],
-    }, options);
-
-    const text = result.content
-      ?.filter((b: any): b is { type: "text"; text: string } => b.type === "text")
-      ?.map((b) => b.text)
-      ?.join("")
-      ?.trim() ?? "";
-
-    return text || null;
-  } catch {
-    return null;
-  }
-}
-
-/* ── Internal helpers ── */
-
-function cleanTitle(raw: string, maxLength: number): string | null {
-  let name = raw.trim();
-  if (!name) return null;
-
-  // Strip surrounding quotes if present
-  if (
-    (name.startsWith('"') && name.endsWith('"')) ||
-    (name.startsWith("'") && name.endsWith("'"))
-  ) {
-    name = name.slice(1, -1);
-  }
-
-  // Collapse newlines
-  name = name.replace(/\n/g, " ").trim();
-
-  // Truncate
-  if (name.length > maxLength) {
-    name = name.slice(0, maxLength - 3) + "...";
-  }
-
-  return name || null;
-}

package/extensions/notify/index.ts

@@ -5,13 +5,16 @@
  * multiline notification showing:
  *   Line 1: working directory path
  *   Line 2: session name (if set)
- *   Line 3: LLM-generated summary of what the agent did
+ *   Line 3: truncated summary of what the agent did
  *
  * The notification title is the tmux session:window.pane identifier (when
  * available) or "pi".
  *
- * Response summarization uses the tiny model configured in settings.json
- * under the `tinyModel` key.
+ * The summary used to come from a cheap side-model, which needed pi-ai's
+ * `complete()` — a value import that stopped this extension loading outside the
+ * agent-cortex checkout (see test/pi-extensions-load.test.mjs). The truncation
+ * that already served as its fallback is now the only path; `maxSummaryLength`
+ * still sets its length.
  */
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
@@ -19,11 +22,7 @@ import type { AgentMessage } from "@earendil-works/pi-agent-core";
 import { execSync } from "node:child_process";
 import { homedir } from "node:os";
 
-import {
-  loadTinyModelConfig,
-  parseModelString,
-  summarizeResponseText,
-} from "../lib/tiny-model.js";
+import { loadTinyModelConfig } from "../lib/tiny-model.js";
 import type { TinyModelConfig } from "../lib/tiny-model.js";
 
 /* ── Context helpers ── */
@@ -215,20 +214,10 @@ export default function (pi: ExtensionAPI) {
     const sessionName = pi.getSessionName();
     if (sessionName) sections.push(sessionName);
 
-    // Section 2: LLM-generated summary of the assistant's response
+    // Section 2: truncated summary of the assistant's response
     const responseText = extractAssistantText(event.messages);
     if (responseText) {
-      try {
-        const summary = await summarizeResponseWithFallback(
-          ctx,
-          responseText,
-          config,
-        );
-        sections.push(summary);
-      } catch {
-        // If summary fails (e.g. config not loaded), just show raw text
-        sections.push(responseText.slice(0, config?.maxSummaryLength ?? 100).replace(/\s+/g, ' ').trim());
-      }
+      sections.push(truncateToOneLine(responseText, config?.maxSummaryLength ?? 100));
     }
 
     // Desktop notification
@@ -239,45 +228,6 @@ export default function (pi: ExtensionAPI) {
   });
 }
 
-/**
- * Summarize response text using the configured tiny model, falling back
- * to simple truncation if the LLM call fails or is disabled.
- */
-async function summarizeResponseWithFallback(
-  ctx: any,
-  responseText: string,
-  config: TinyModelConfig,
-): Promise<string> {
-  if (!config.enabled) {
-    return truncateToOneLine(responseText, config.maxSummaryLength);
-  }
-
-  try {
-    const { provider, id } = parseModelString(config.model);
-    const model = ctx.modelRegistry.find(provider, id);
-    if (!model) {
-      return truncateToOneLine(responseText, config.maxSummaryLength);
-    }
-
-    const { apiKey, headers } =
-      await ctx.modelRegistry.getApiKeyAndHeaders(model);
-
-    const summary = await summarizeResponseText(
-      model,
-      apiKey,
-      headers,
-      responseText,
-      config.maxSummaryLength,
-    );
-
-    if (summary) return summary;
-  } catch (err) {
-    console.warn("[notify] LLM summary failed, falling back:", err);
-  }
-
-  return truncateToOneLine(responseText, config.maxSummaryLength);
-}
-
 /**
  * Truncate text to roughly `maxLen` characters on a single line,
  * breaking at a word boundary.

package/extensions/output-style/package.json

@@ -12,8 +12,5 @@
     "@types/node": "^22.0.0",
     "tsx": "^4.0.0",
     "typescript": "^5.0.0"
-  },
-  "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.78.1"
   }
 }

package/extensions/session-start/package.json

@@ -12,8 +12,5 @@
     "@types/node": "^22.0.0",
     "tsx": "^4.0.0",
     "typescript": "^5.0.0"
-  },
-  "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.78.1"
   }
 }

package/extensions/skill-stats/package.json

@@ -12,8 +12,5 @@
     "@types/node": "^22.0.0",
     "tsx": "^4.0.0",
     "typescript": "^5.0.0"
-  },
-  "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.78.1"
   }
 }

package/extensions/subagent/README.md

@@ -7,7 +7,7 @@ Delegate tasks to specialized subagents with isolated context windows.
 - **Isolated context**: Each subagent runs in a separate `pi` process
 - **Streaming output**: See tool calls and progress as they happen
 - **Parallel streaming**: All parallel tasks stream updates simultaneously
-- **Markdown rendering**: Final output rendered with proper formatting (expanded view)
+- **Plain-text output**: Final output wrapped to the terminal width (expanded view)
 - **Usage tracking**: Shows turns, tokens, cost, and context usage per agent
 - **Abort support**: Ctrl+C propagates to kill subagent processes
 
@@ -125,7 +125,7 @@ pending `wait_for_agents` returns immediately with whatever is known so far.
 **Expanded view** (Ctrl+O):
 - Full task text
 - All tool calls with formatted arguments
-- Final output rendered as Markdown
+- Final output rendered as width-wrapped plain text
 - Per-task usage (for chain/parallel)
 
 **Parallel mode streaming**:

package/extensions/subagent/agents.ts

@@ -3,9 +3,10 @@
  */
 
 import * as fs from "node:fs";
+import * as os from "node:os";
 import * as path from "node:path";
 import { fileURLToPath } from "node:url";
-import { getAgentDir, parseFrontmatter } from "@earendil-works/pi-coding-agent";
+import { parseFrontmatter } from "./frontmatter.ts";
 
 export type AgentScope = "user" | "project" | "both";
 
@@ -145,6 +146,21 @@ function isRenderedAgentsTree(agentsDir: string): boolean {
 	return !entries.some((entry) => entry.isDirectory() && fs.existsSync(path.join(agentsDir, entry.name, "agent.md")));
 }
 
+/**
+ * pi's user config directory, which holds pi's own `agents/` directory.
+ *
+ * Read here rather than through `getAgentDir()` from
+ * `@earendil-works/pi-coding-agent`: a value import of pi's runtime makes this
+ * extension fail to load outside the agent-cortex checkout
+ * (test/pi-extensions-load.test.mjs). Same contract — `PI_CODING_AGENT_DIR`
+ * wins when set, otherwise `~/.pi/agent`.
+ */
+function getAgentDir(): string {
+	const override = process.env.PI_CODING_AGENT_DIR;
+	if (override) return path.resolve(override.replace(/^~(?=$|[\\/])/, os.homedir()));
+	return path.join(os.homedir(), ".pi", "agent");
+}
+
 /**
  * Directories the "user" scope discovers agents from, in precedence order.
  *

package/extensions/subagent/components.test.ts

@@ -0,0 +1,111 @@
+/**
+ * The local transcript components pi's renderers draw for the subagent tool.
+ *
+ * pi-tui's own components are unreachable from an installed extension, so these
+ * implement pi's `Component` contract directly. The contract that matters is the
+ * width: pi-tui reports a rendered line wider than the terminal as an error, so
+ * every line these return must fit the width they are given, with SGR colour
+ * sequences still intact.
+ */
+
+import { describe, it } from "node:test";
+import assert from "node:assert/strict";
+
+import { Container, Spacer, Text } from "./components.ts";
+
+const RESET = "\x1b[0m";
+const GREEN = "\x1b[32m";
+
+/** Visible column count, ignoring SGR sequences. */
+function visibleWidth(line: string): number {
+	return line.replace(/\x1b\[[0-9;]*m/g, "").length;
+}
+
+/**
+ * Visible terminal-cell count, ignoring SGR sequences and counting a wide or
+ * astral character as two cells. `String.length` counts a wide character as one
+ * unit, so a width assertion built on it misses the overflow this guards.
+ */
+function cellWidth(line: string): number {
+	return [...line.replace(/\x1b\[[0-9;]*m/g, "")].reduce((cells, character) => {
+		const codePoint = character.codePointAt(0) ?? 0;
+		const wide =
+			codePoint > 0xffff ||
+			(codePoint >= 0x1100 &&
+				(codePoint <= 0x115f ||
+					(codePoint >= 0x2e80 && codePoint <= 0xa4cf) ||
+					(codePoint >= 0xac00 && codePoint <= 0xd7a3) ||
+					(codePoint >= 0xf900 && codePoint <= 0xfaff) ||
+					(codePoint >= 0xfe30 && codePoint <= 0xfe6f) ||
+					(codePoint >= 0xff00 && codePoint <= 0xff60) ||
+					(codePoint >= 0xffe0 && codePoint <= 0xffe6)));
+		return cells + (wide ? 2 : 1);
+	}, 0);
+}
+
+describe("Text", () => {
+	it("wraps a long line to the width it is given", () => {
+		const lines = new Text("alpha beta gamma delta").render(11);
+
+		assert.deepEqual(lines, ["alpha beta", "gamma delta"]);
+		assert.ok(lines.every((line) => visibleWidth(line) <= 11));
+	});
+
+	it("hard-wraps a word longer than the width instead of overflowing", () => {
+		const lines = new Text("abcdefghij").render(4);
+
+		assert.deepEqual(lines, ["abcd", "efgh", "ij"]);
+	});
+
+	it("keeps a colour sequence intact and carries it onto the wrapped line", () => {
+		const lines = new Text(`${GREEN}alpha beta gamma${RESET}`).render(11);
+
+		assert.equal(lines.length, 2);
+		for (const line of lines) assert.ok(visibleWidth(line) <= 11, `width of ${JSON.stringify(line)}`);
+		assert.ok(lines[0].endsWith(RESET), "a line cut inside a colour span closes it");
+		assert.ok(lines[1].startsWith(GREEN), "the continuation reopens the colour");
+		assert.equal(lines.map((line) => line.replace(/\x1b\[[0-9;]*m/g, "")).join(" "), "alpha beta gamma");
+	});
+
+	it("pads and separates lines with the padding it is given", () => {
+		const lines = new Text("body", 1, 1).render(10);
+
+		assert.deepEqual(lines, ["  ", " body ", "  "]);
+	});
+
+	it("keeps an empty line and renders blank text as one empty line", () => {
+		assert.deepEqual(new Text("a\n\nb").render(10), ["a", "", "b"]);
+		assert.deepEqual(new Text().render(10), [""]);
+	});
+
+	it("counts a wide character as two cells so it cannot overflow the width", () => {
+		// Each of these characters occupies two terminal cells but one string unit.
+		assert.deepEqual(new Text("日本語").render(4), ["日本", "語"]);
+
+		const lines = new Text("日本語のテキストは全角文字です。これは折り返しのテストです。").render(20);
+		assert.ok(lines.length > 1, "wide text must wrap");
+		for (const line of lines) assert.ok(cellWidth(line) <= 20, `width of ${JSON.stringify(line)}`);
+	});
+
+	it("counts an astral character as two cells", () => {
+		assert.deepEqual(new Text("🚀🚀🚀").render(4), ["🚀🚀", "🚀"]);
+	});
+});
+
+describe("Spacer", () => {
+	it("renders the blank lines it was asked for", () => {
+		assert.deepEqual(new Spacer(2).render(10), ["", ""]);
+		assert.deepEqual(new Spacer().render(10), [""]);
+	});
+});
+
+describe("Container", () => {
+	it("draws its children in order, each at the given width", () => {
+		const container = new Container();
+		container.addChild(new Text("alpha beta"));
+		container.addChild(new Spacer(1));
+		container.addChild(new Text("gamma"));
+
+		assert.deepEqual(container.render(5), ["alpha", "beta", "", "gamma"]);
+	});
+});