pi-subagents-lite 1.2.0 → 1.4.0

package/src/prompts.ts

@@ -1,94 +0,0 @@
-/**
- * prompts.ts — System prompt builder for agents.
- *
- * Every agent gets a fresh context — no inherited parent identity.
- * EnvInfo is imported from types.ts — branch is a string (empty when unknown).
- */
-
-import type { AgentConfig, EnvInfo } from "./types.js";
-import type { SkillMeta } from "./skill-loader.js";
-
-/** Extra sections to inject into the system prompt (skills). */
-export interface PromptExtras {
-  /** Preloaded skill contents to inject (full content). */
-  skillBlocks?: { name: string; content: string }[];
-  /** Skill metadata for whitelist display (name, description, location only). */
-  skillMetas?: SkillMeta[];
-}
-
-/**
- * Build the system prompt for an agent from its config.
- *
- * Always uses fresh-context mode: env header + config.systemPrompt.
- * Prepends an `<active_agent name=""/>` tag so downstream extensions
- * (e.g. permission/policy systems) can resolve per-agent policy.
- *
- * @param extras  Optional extra sections to inject (preloaded skills).
- */
-export function buildAgentPrompt(
-  config: AgentConfig,
-  cwd: string,
-  env: EnvInfo,
-  extras?: PromptExtras,
-): string {
-  const activeAgentTag = `<active_agent name="${config.name}"/>\n\n`;
-
-  const envLines = [
-    "# Environment",
-    `Working directory: ${cwd}`,
-    env.isGitRepo ? "Git repository: yes" : "Not a git repository",
-  ];
-  if (env.isGitRepo && env.branch) {
-    envLines.push(`Branch: ${env.branch}`);
-  }
-  envLines.push(`Platform: ${env.platform}`);
-  const envBlock = envLines.join("\n");
-
-  // Build optional extras suffix (skills)
-  const extraSections: string[] = [];
-
-  // Skill metadata whitelist (like Pi's available_skills format)
-  if (extras?.skillMetas?.length) {
-    const lines = [
-      "The following skills provide specialized instructions for specific tasks.",
-      "Use the read tool to load a skill's file when the task matches its description.",
-      "When a skill file references a relative path, resolve it against the skill directory (parent of SKILL.md / dirname of the path) and use that absolute path in tool commands.",
-      "",
-      "<available_skills>",
-    ];
-    for (const skill of extras.skillMetas) {
-      lines.push("  <skill>");
-      lines.push(`    <name>${escapeXml(skill.name)}</name>`);
-      lines.push(`    <description>${escapeXml(skill.description)}</description>`);
-      lines.push(`    <location>${escapeXml(skill.location)}</location>`);
-      lines.push("  </skill>");
-    }
-    lines.push("</available_skills>");
-    extraSections.push(lines.join("\n"));
-  }
-
-  // Preloaded skill contents (full dump into system prompt)
-  if (extras?.skillBlocks?.length) {
-    for (const skill of extras.skillBlocks) {
-      extraSections.push(`\n# Preloaded Skill: ${skill.name}\n${skill.content}`);
-    }
-  }
-
-  const extrasSuffix = extraSections.length > 0 ? `\n\n${extraSections.join("\n")}` : "";
-
-  const header = `You are a pi coding agent sub-agent.
-You have been invoked to handle a specific task autonomously.
-
-${envBlock}`;
-
-  return `${activeAgentTag}${header}\n\n${config.systemPrompt}${extrasSuffix}`;
-}
-
-function escapeXml(value: string): string {
-  // Only escape < and > — enough for XML-like tags, keeps text readable for LLMs
-  return value
-    .replace(/</g, "&lt;")
-    .replace(/>/g, "&gt;");
-}
-
-

package/src/skill-loader.ts

@@ -1,178 +0,0 @@
-/**
- * skill-loader.ts — Preload named skills.
- *
- * Roots, in precedence order:
- *   - <cwd>/.pi/skills           (project, Pi's standard)
- *   - <cwd>/.agents/skills       (project, cross-tool Agent Skills spec — https://agentskills.io)
- *   - getAgentDir()/skills       (user, default ~/.pi/agent/skills — Pi's standard)
- *   - ~/.agents/skills           (user, cross-tool Agent Skills spec)
- *   - ~/.pi/skills               (legacy global, pre-Pi)
- *
- * Layout per root:
- *   - <root>/<name>.md            (flat file at the top level)
- *   - <root>/.../<name>/SKILL.md  (directory skill, may be nested — Pi's standard)
- *
- * Recursion skips dotfile entries and node_modules. A directory that itself contains
- * SKILL.md is a skill — we don't descend into it (Pi: skills don't nest).
- *
- * Symlinks are rejected for security (deviation from Pi, which follows them).
- */
-
-import type { Dirent } from "node:fs";
-import { existsSync, readdirSync } from "node:fs";
-import { homedir } from "node:os";
-import { join } from "node:path";
-import { getAgentDir } from "@earendil-works/pi-coding-agent";
-import { isSymlink, isUnsafeName, safeReadFile } from "./utils.js";
-
-interface PreloadedSkill {
-  name: string;
-  content: string;
-}
-
-export interface SkillMeta {
-  name: string;
-  description: string;
-  location: string;
-}
-
-/**
- * Skill search roots in precedence order (project → user → legacy).
- * Shared by preloadSkills and loadSkillMeta.
- */
-function getSkillRoots(cwd: string): string[] {
-  return [
-    join(cwd, ".pi", "skills"),           // project — Pi standard
-    join(cwd, ".agents", "skills"),       // project — Agent Skills spec
-    join(getAgentDir(), "skills"),         // user — Pi standard
-    join(homedir(), ".agents", "skills"),  // user — Agent Skills spec
-    join(homedir(), ".pi", "skills"),      // legacy global, pre-Pi
-  ];
-}
-
-export function preloadSkills(skillNames: string[], cwd: string): PreloadedSkill[] {
-  return skillNames.map((name) => ({ name, content: loadSkillContent(name, cwd) }));
-}
-
-/**
- * Load skill metadata only (name, description, location) without full content.
- * Used for the skills whitelist — agent can read full content on-demand.
- */
-export function loadSkillMeta(skillNames: string[], cwd: string): SkillMeta[] {
-  return skillNames.map((name) => {
-    const location = findSkillLocation(name, cwd);
-    if (!location) {
-      return { name, description: `(Skill "${name}" not found)`, location: "" };
-    }
-    const description = extractDescription(location);
-    return { name, description, location };
-  });
-}
-
-function loadSkillContent(name: string, cwd: string): string {
-  if (isUnsafeName(name)) {
-    return `(Skill "${name}" skipped: name contains path traversal characters)`;
-  }
-  for (const root of getSkillRoots(cwd)) {
-    const content = findInRoot(root, name, "content");
-    if (content !== undefined) return content;
-  }
-  return `(Skill "${name}" not found in .pi/skills/, .agents/skills/, or global skill locations)`;
-}
-
-function findInRoot(root: string, name: string, mode: "content" | "location"): string | undefined {
-  if (isSymlink(root)) return undefined;
-  const flatPath = join(root, `${name}.md`);
-  if (mode === "location") {
-    if (existsSync(flatPath)) return flatPath;
-  } else {
-    const content = safeReadFile(flatPath)?.trim();
-    if (content !== undefined) return content;
-  }
-  return findSkillDirectory(root, name, mode);
-}
-
-/**
- * BFS under `root` for a directory named `name` containing `SKILL.md`.
- * Pi-conforming filters. Returns either the file content or the file path.
- */
-function findSkillDirectory(root: string, name: string, mode: "content" | "location"): string | undefined {
-  if (!existsSync(root)) return undefined;
-  const queue: string[] = [root];
-
-  while (queue.length > 0) {
-    const current = queue.shift();
-    if (current === undefined) continue;
-
-    let entries: Dirent[];
-    try {
-      entries = readdirSync(current, { withFileTypes: true });
-    } catch {
-      continue;
-    }
-
-    // Deterministic byte-order traversal — locale-independent.
-    entries.sort((a, b) => (a.name < b.name ? -1 : a.name > b.name ? 1 : 0));
-
-    for (const entry of entries) {
-      if (!entry.isDirectory()) continue;
-      if (entry.name.startsWith(".") || entry.name === "node_modules") continue;
-
-      // Symlinked dirs already filtered by entry.isDirectory() — Dirent uses lstat semantics.
-      const path = join(current, entry.name);
-      const skillMd = join(path, "SKILL.md");
-      const isSkillDir = existsSync(skillMd);
-
-      if (isSkillDir) {
-        if (entry.name === name) {
-          if (mode === "location") return skillMd;
-          const content = safeReadFile(skillMd)?.trim();
-          if (content !== undefined) return content;
-        }
-        continue; // Pi rule: skills don't nest — don't descend into a skill dir
-      }
-
-      queue.push(path);
-    }
-  }
-  return undefined;
-}
-
-/**
- * Find skill file location without reading content.
- * Returns the full path to the SKILL.md or .md file, or undefined if not found.
- */
-function findSkillLocation(name: string, cwd: string): string | undefined {
-  if (isUnsafeName(name)) return undefined;
-  for (const root of getSkillRoots(cwd)) {
-    const location = findInRoot(root, name, "location");
-    if (location !== undefined) return location;
-  }
-  return undefined;
-}
-
-/** Extract description from SKILL.md frontmatter. */
-function extractDescription(filePath: string): string {
-  try {
-    const content = safeReadFile(filePath);
-    if (!content) return "(no description)";
-
-    // Simple frontmatter extraction
-    const normalized = content.replace(/\r\n/g, "\n").replace(/\r/g, "\n");
-    if (!normalized.startsWith("---\n")) return "(no description)";
-    const endIndex = normalized.indexOf("\n---\n", 4);
-    if (endIndex === -1) return "(no description)";
-
-    const yamlString = normalized.slice(4, endIndex);
-    // Simple extraction of description field
-    const descMatch = yamlString.match(/^description:\s*["']?(.+?)["']?\s*$/m);
-    if (descMatch && descMatch[1]) {
-      // Truncate long descriptions
-      const desc = descMatch[1].trim();
-      return desc.length > 200 ? desc.slice(0, 197) + "..." : desc;
-    }
-    return "(no description)";
-  } catch {
-    return "(error reading description)";
-  }
-}

package/src/state.ts

@@ -1,83 +0,0 @@
-/**
- * state.ts — Shared module state. Extracted from index.ts to break circular deps.
- *
- * manager and widget use holders because they're reassigned after import and the
- * PI runtime doesn't propagate ESM live binding reassignments.
- */
-
-import type { ExtensionContext, ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import type { SessionModelOverrides, SubagentsConfig } from "./model-precedence.js";
-import { DEFAULT_CONFIG } from "./config-io.js";
-import { AgentManager } from "./agent-manager.js";
-import { AgentWidget, type AgentActivity } from "./ui/agent-widget.js";
-
-export let sessionOverrides: SessionModelOverrides = { default: null };
-export let __config: SubagentsConfig = { ...DEFAULT_CONFIG, agent: { ...DEFAULT_CONFIG.agent }, concurrency: { ...DEFAULT_CONFIG.concurrency } };
-export const agentActivity = new Map<string, AgentActivity>();
-export let piInstance: ExtensionAPI;
-/** Stored ExtensionContext from session_start — used by menu spawn flow. */
-export let sessionCtx: ExtensionContext;
-
-// Holder objects — PI runtime doesn't propagate ESM live binding reassignments
-const managerHolder: { current?: AgentManager } = {};
-const widgetHolder: { current?: AgentWidget } = {};
-
-export function setConfig(config: SubagentsConfig): void { __config = config; }
-export function resetSessionOverrides(): void { sessionOverrides = { default: null }; }
-export function setManager(m: AgentManager): void { managerHolder.current = m; }
-export function clearManager(): void { managerHolder.current = undefined; }
-export function setWidget(w: AgentWidget | undefined): void { widgetHolder.current = w; }
-export function setPiInstance(pi: ExtensionAPI): void { piInstance = pi; }
-export function setSessionCtx(ctx: ExtensionContext): void { sessionCtx = ctx; }
-export function getManager(): AgentManager { return managerHolder.current!; }
-export function getWidget(): AgentWidget | undefined { return widgetHolder.current; }
-
-// State mutation helpers
-
-/** Update the cost display toggle in config and sync to widget. */
-export function setShowCostEnabled(enabled: boolean): void {
-  __config.agent.showCost = enabled;
-  getWidget()?.setShowCost(enabled);
-}
-
-/** Sync widget display settings from config to the widget instance. */
-export function syncWidgetSettings(): void {
-  const w = getWidget();
-  if (!w) return;
-  w.setForceCompact(__config.agent.widgetCompact === true);
-  w.setWidgetShortcut(__config.agent.widgetShortcut === true);
-  w.setMaxLines(__config.agent.widgetMaxLines ?? 12);
-  w.setMaxLinesCompact(
-    __config.agent.widgetMaxLinesCompact ?? Math.floor((__config.agent.widgetMaxLines ?? 12) / 2),
-  );
-}
-
-/** Track previous tool expansion state to detect ctrl+o toggle. */
-let lastToolsExpanded: boolean | undefined;
-
-/** Reset lastToolsExpanded (called at session_start). */
-export function resetLastToolsExpanded(): void {
-  lastToolsExpanded = undefined;
-}
-
-/** Sync compact mode with the tool expansion state (ctrl+o toggle).
- *  Only syncs when widgetShortcut is enabled in config (opt-in behavior).
- *  Only triggers on state change (not every tool_execution_start).
- *  When forceCompact (widgetCompact) is ON, ignores ctrl+o state changes.
- */
-export function syncCompactFromToolsExpanded(expanded: boolean): void {
-  if (__config.agent.widgetShortcut !== true) {
-    lastToolsExpanded = expanded;
-    return;
-  }
-  // When forceCompact is ON, ignore ctrl+o state changes
-  if (__config.agent.widgetCompact === true) {
-    lastToolsExpanded = expanded;
-    return;
-  }
-  // Tools expanded → widget full, tools collapsed → widget compact
-  if (lastToolsExpanded !== undefined && lastToolsExpanded !== expanded) {
-    getWidget()?.setCompactMode(!expanded);
-  }
-  lastToolsExpanded = expanded;
-}