@pi-unipi/subagents 2.3.0 → 2.4.1

package/src/conversation-viewer.ts

@@ -1,299 +0,0 @@
-/**
- * @pi-unipi/subagents — Conversation Viewer
- *
- * Live-scrolling overlay for viewing agent conversations.
- * Subscribes to session events for real-time streaming updates.
- * Supports keyboard navigation: ↑↓, PgUp/PgDn, Home/End, Esc/q to close.
- */
-
-import type { AgentSession } from "@earendil-works/pi-coding-agent";
-import {
-  type Component,
-  matchesKey,
-  type TUI,
-  truncateToWidth,
-  visibleWidth,
-  wrapTextWithAnsi,
-} from "@earendil-works/pi-tui";
-import type { AgentActivity } from "./types.js";
-
-/** Lines consumed by chrome: top border + header + header sep + footer sep + footer + bottom border. */
-const CHROME_LINES = 6;
-const MIN_VIEWPORT = 3;
-
-/** Extract text from content array. */
-function extractText(content: string | Array<{ type: string; text?: string }>): string {
-  if (typeof content === "string") return content;
-  return content
-    .filter((p): p is { type: "text"; text: string } => p.type === "text" && typeof p.text === "string")
-    .map((p) => p.text)
-    .join("");
-}
-
-/** Format duration. */
-function formatMs(ms: number): string {
-  if (ms >= 60_000) return `${(ms / 60_000).toFixed(1)}m`;
-  if (ms >= 1_000) return `${(ms / 1_000).toFixed(1)}s`;
-  return `${ms}ms`;
-}
-
-/** Format tokens compactly. */
-function formatTokens(count: number): string {
-  if (count >= 1_000_000) return `${(count / 1_000_000).toFixed(1)}M token`;
-  if (count >= 1_000) return `${(count / 1_000).toFixed(1)}k token`;
-  return `${count} token`;
-}
-
-/** Describe current activity from active tools. */
-function describeActivity(activeTools: Map<string, string>, responseText?: string): string {
-  if (activeTools.size > 0) {
-    const names = [...new Set(activeTools.values())];
-    return names.join(", ") + "…";
-  }
-  if (responseText && responseText.trim().length > 0) {
-    const lastLine = responseText.split("\n").find((l) => l.trim())?.trim() ?? "";
-    if (lastLine.length > 60) return lastLine.slice(0, 60) + "…";
-    if (lastLine.length > 0) return lastLine;
-  }
-  return "thinking…";
-}
-
-interface ViewerRecord {
-  type: string;
-  description: string;
-  status: string;
-  toolUses: number;
-  startedAt: number;
-  completedAt?: number;
-}
-
-export class ConversationViewer implements Component {
-  private scrollOffset = 0;
-  private autoScroll = true;
-  private unsubscribe: (() => void) | undefined;
-  private lastInnerW = 0;
-  private closed = false;
-
-  constructor(
-    private tui: TUI,
-    private session: AgentSession,
-    private record: ViewerRecord,
-    private activity: AgentActivity | undefined,
-    private theme: any,
-    private done: (result: undefined) => void,
-  ) {
-    this.unsubscribe = session.subscribe(() => {
-      if (this.closed) return;
-      this.tui.requestRender();
-    });
-  }
-
-  handleInput(data: string): void {
-    if (matchesKey(data, "escape") || matchesKey(data, "q")) {
-      this.closed = true;
-      this.done(undefined);
-      return;
-    }
-
-    const totalLines = this.buildContentLines(this.lastInnerW).length;
-    const viewportHeight = this.viewportHeight();
-    const maxScroll = Math.max(0, totalLines - viewportHeight);
-
-    if (matchesKey(data, "up") || matchesKey(data, "k")) {
-      this.scrollOffset = Math.max(0, this.scrollOffset - 1);
-      this.autoScroll = this.scrollOffset >= maxScroll;
-    } else if (matchesKey(data, "down") || matchesKey(data, "j")) {
-      this.scrollOffset = Math.min(maxScroll, this.scrollOffset + 1);
-      this.autoScroll = this.scrollOffset >= maxScroll;
-    } else if (matchesKey(data, "pageUp")) {
-      this.scrollOffset = Math.max(0, this.scrollOffset - viewportHeight);
-      this.autoScroll = false;
-    } else if (matchesKey(data, "pageDown")) {
-      this.scrollOffset = Math.min(maxScroll, this.scrollOffset + viewportHeight);
-      this.autoScroll = this.scrollOffset >= maxScroll;
-    } else if (matchesKey(data, "home")) {
-      this.scrollOffset = 0;
-      this.autoScroll = false;
-    } else if (matchesKey(data, "end")) {
-      this.scrollOffset = maxScroll;
-      this.autoScroll = true;
-    }
-  }
-
-  render(width: number): string[] {
-    if (width < 6) return [];
-    const th = this.theme;
-    const innerW = width - 4; // border + padding
-    this.lastInnerW = innerW;
-    const lines: string[] = [];
-
-    const pad = (s: string, len: number) => {
-      const vis = visibleWidth(s);
-      return s + " ".repeat(Math.max(0, len - vis));
-    };
-    const row = (content: string) =>
-      th.fg("border", "│") + " " + truncateToWidth(pad(content, innerW), innerW) + " " + th.fg("border", "│");
-    const hrTop = th.fg("border", `╭${"─".repeat(width - 2)}╮`);
-    const hrBot = th.fg("border", `╰${"─".repeat(width - 2)}╯`);
-    const hrMid = row(th.fg("dim", "─".repeat(innerW)));
-
-    // Header
-    lines.push(hrTop);
-    const name = this.record.type;
-    const statusIcon =
-      this.record.status === "running"
-        ? th.fg("accent", "●")
-        : this.record.status === "completed"
-          ? th.fg("success", "✓")
-          : this.record.status === "error"
-            ? th.fg("error", "✗")
-            : th.fg("dim", "○");
-
-    const duration = this.record.completedAt
-      ? formatMs(this.record.completedAt - this.record.startedAt)
-      : `${formatMs(Date.now() - this.record.startedAt)} (running)`;
-
-    const headerParts: string[] = [duration];
-    const toolUses = this.activity?.toolUses ?? this.record.toolUses;
-    if (toolUses > 0) headerParts.unshift(`${toolUses} tool${toolUses === 1 ? "" : "s"}`);
-    if (this.activity?.session) {
-      try {
-        const tokens = (this.activity.session as any).getSessionStats().tokens.total;
-        if (tokens > 0) headerParts.push(formatTokens(tokens));
-      } catch {
-        /* */
-      }
-    }
-
-    lines.push(
-      row(
-        `${statusIcon} ${th.bold(name)}  ${th.fg("muted", this.record.description)} ${th.fg("dim", "·")} ${th.fg("dim", headerParts.join(" · "))}`,
-      ),
-    );
-    lines.push(hrMid);
-
-    // Content area
-    const contentLines = this.buildContentLines(innerW);
-    const viewportHeight = this.viewportHeight();
-    const maxScroll = Math.max(0, contentLines.length - viewportHeight);
-
-    if (this.autoScroll) {
-      this.scrollOffset = maxScroll;
-    }
-
-    const visibleStart = Math.min(this.scrollOffset, maxScroll);
-    const visible = contentLines.slice(visibleStart, visibleStart + viewportHeight);
-
-    for (let i = 0; i < viewportHeight; i++) {
-      lines.push(row(visible[i] ?? ""));
-    }
-
-    // Footer
-    lines.push(hrMid);
-    const scrollPct =
-      contentLines.length <= viewportHeight
-        ? "100%"
-        : `${Math.round(((visibleStart + viewportHeight) / contentLines.length) * 100)}%`;
-    const footerLeft = th.fg("dim", `${contentLines.length} lines · ${scrollPct}`);
-    const footerRight = th.fg("dim", "↑↓ scroll · PgUp/PgDn · Esc close");
-    const footerGap = Math.max(1, innerW - visibleWidth(footerLeft) - visibleWidth(footerRight));
-    lines.push(row(footerLeft + " ".repeat(footerGap) + footerRight));
-    lines.push(hrBot);
-
-    return lines;
-  }
-
-  invalidate(): void {
-    /* no cached state to clear */
-  }
-
-  dispose(): void {
-    this.closed = true;
-    if (this.unsubscribe) {
-      this.unsubscribe();
-      this.unsubscribe = undefined;
-    }
-  }
-
-  // ---- Private ----
-
-  private viewportHeight(): number {
-    return Math.max(MIN_VIEWPORT, this.tui.terminal.rows - CHROME_LINES);
-  }
-
-  private buildContentLines(width: number): string[] {
-    if (width <= 0) return [];
-
-    const th = this.theme;
-    const messages = (this.session as any).messages;
-    const lines: string[] = [];
-
-    if (!messages || messages.length === 0) {
-      lines.push(th.fg("dim", "(waiting for first message...)"));
-      return lines;
-    }
-
-    let needsSeparator = false;
-    for (const msg of messages) {
-      if (msg.role === "user") {
-        const text = typeof msg.content === "string" ? msg.content : extractText(msg.content);
-        if (!text.trim()) continue;
-        if (needsSeparator) lines.push(th.fg("dim", "───"));
-        lines.push(th.fg("accent", "[User]"));
-        for (const line of wrapTextWithAnsi(text.trim(), width)) {
-          lines.push(line);
-        }
-      } else if (msg.role === "assistant") {
-        const textParts: string[] = [];
-        const toolCalls: string[] = [];
-        for (const c of msg.content) {
-          if (c.type === "text" && c.text) textParts.push(c.text);
-          else if (c.type === "tool_use" || c.type === "toolCall") {
-            toolCalls.push((c as any).name ?? (c as any).toolName ?? "unknown");
-          }
-        }
-        if (needsSeparator) lines.push(th.fg("dim", "───"));
-        lines.push(th.bold("[Assistant]"));
-        if (textParts.length > 0) {
-          for (const line of wrapTextWithAnsi(textParts.join("\n").trim(), width)) {
-            lines.push(line);
-          }
-        }
-        for (const name of toolCalls) {
-          lines.push(truncateToWidth(th.fg("muted", `  [Tool: ${name}]`), width));
-        }
-      } else if (msg.role === "toolResult") {
-        const text = extractText(msg.content);
-        const truncated = text.length > 500 ? text.slice(0, 500) + "... (truncated)" : text;
-        if (!truncated.trim()) continue;
-        if (needsSeparator) lines.push(th.fg("dim", "───"));
-        lines.push(th.fg("dim", "[Result]"));
-        for (const line of wrapTextWithAnsi(truncated.trim(), width)) {
-          lines.push(th.fg("dim", line));
-        }
-      } else if ((msg as any).role === "bashExecution") {
-        const bash = msg as any;
-        if (needsSeparator) lines.push(th.fg("dim", "───"));
-        lines.push(truncateToWidth(th.fg("muted", `  $ ${bash.command}`), width));
-        if (bash.output?.trim()) {
-          const out = bash.output.length > 500 ? bash.output.slice(0, 500) + "... (truncated)" : bash.output;
-          for (const line of wrapTextWithAnsi(out.trim(), width)) {
-            lines.push(th.fg("dim", line));
-          }
-        }
-      } else {
-        continue;
-      }
-      needsSeparator = true;
-    }
-
-    // Streaming indicator for running agents
-    if (this.record.status === "running" && this.activity) {
-      const act = describeActivity(this.activity.activeTools, this.activity.responseText);
-      lines.push("");
-      lines.push(truncateToWidth(th.fg("accent", "▍ ") + th.fg("dim", act), width));
-    }
-
-    return lines.map((l) => truncateToWidth(l, width));
-  }
-}

package/src/custom-agents.ts

@@ -1,118 +0,0 @@
-/**
- * @pi-unipi/subagents — Custom agent loader
- *
- * Discovers agent types from:
- * - <workspace>/.unipi/config/agents/*.md (project, highest priority)
- * - ~/.unipi/config/agents/*.md (global)
- */
-
-import { existsSync, readdirSync, readFileSync, renameSync } from "node:fs";
-import { join } from "node:path";
-import { homedir } from "node:os";
-import { parseFrontmatter } from "@earendil-works/pi-coding-agent";
-import type { AgentConfig } from "./types.js";
-
-/** Backup a corrupted file by renaming to .bak */
-function backupCorrupted(filePath: string): void {
-  const backupPath = filePath + ".bak";
-  try {
-    renameSync(filePath, backupPath);
-  } catch {
-    // If backup fails, just leave it
-  }
-}
-
-/** Get project agents directory. */
-function getProjectAgentsDir(cwd: string): string {
-  return join(cwd, ".unipi", "config", "agents");
-}
-
-/** Get global agents directory. */
-function getGlobalAgentsDir(): string {
-  return join(homedir(), ".unipi", "config", "agents");
-}
-
-/** All known built-in tool names. */
-const BUILTIN_TOOL_NAMES = ["read", "bash", "edit", "write", "grep", "find", "ls"];
-
-/**
- * Load a single agent from a .md file.
- */
-function loadAgentFromFile(filePath: string, source: "project" | "global"): AgentConfig | null {
-  try {
-    const content = readFileSync(filePath, "utf-8");
-    const { frontmatter, body } = parseFrontmatter(content);
-
-    if (!frontmatter || typeof frontmatter !== "object") {
-      return null;
-    }
-
-    const name = filePath.split("/").pop()?.replace(/\.md$/, "") ?? "unknown";
-
-    // Parse tools from comma-separated string
-    const toolsStr = (frontmatter as any).tools as string | undefined;
-    const builtinToolNames = toolsStr
-      ? toolsStr.split(",").map((t) => t.trim()).filter((t) => BUILTIN_TOOL_NAMES.includes(t))
-      : [...BUILTIN_TOOL_NAMES];
-
-    return {
-      name,
-      displayName: (frontmatter as any).display_name as string | undefined,
-      description: ((frontmatter as any).description as string) ?? `${name} agent`,
-      builtinToolNames,
-      disallowedTools: ((frontmatter as any).disallowed_tools as string | undefined)
-        ?.split(",")
-        .map((t) => t.trim()),
-      extensions: (frontmatter as any).extensions !== false,
-      skills: (frontmatter as any).skills !== false,
-      model: (frontmatter as any).model as string | undefined,
-      thinking: (frontmatter as any).thinking as any,
-      maxTurns: (frontmatter as any).max_turns as number | undefined,
-      systemPrompt: body.trim(),
-      promptMode: ((frontmatter as any).prompt_mode as "replace" | "append") ?? "replace",
-      inheritContext: (frontmatter as any).inherit_context as boolean | undefined,
-      runInBackground: (frontmatter as any).run_in_background as boolean | undefined,
-      isolated: (frontmatter as any).isolated as boolean | undefined,
-      enabled: (frontmatter as any).enabled !== false,
-      source,
-    };
-  } catch {
-    // Corrupted file — backup and skip
-    backupCorrupted(filePath);
-    return null;
-  }
-}
-
-/**
- * Load all custom agents from project and global directories.
- * Project agents override global agents with the same name.
- */
-export function loadCustomAgents(cwd: string): Map<string, AgentConfig> {
-  const agents = new Map<string, AgentConfig>();
-
-  // Load global agents first
-  const globalDir = getGlobalAgentsDir();
-  if (existsSync(globalDir)) {
-    const files = readdirSync(globalDir).filter((f) => f.endsWith(".md"));
-    for (const file of files) {
-      const agent = loadAgentFromFile(join(globalDir, file), "global");
-      if (agent) {
-        agents.set(agent.name, agent);
-      }
-    }
-  }
-
-  // Load project agents (overrides global)
-  const projectDir = getProjectAgentsDir(cwd);
-  if (existsSync(projectDir)) {
-    const files = readdirSync(projectDir).filter((f) => f.endsWith(".md"));
-    for (const file of files) {
-      const agent = loadAgentFromFile(join(projectDir, file), "project");
-      if (agent) {
-        agents.set(agent.name, agent);
-      }
-    }
-  }
-
-  return agents;
-}

package/src/file-lock.ts

@@ -1,102 +0,0 @@
-/**
- * @pi-unipi/subagents — Per-file transparent locking
- *
- * Agents never see lock errors. Write tool queues internally.
- * Per-file granularity: locking src/auth.ts doesn't block src/login.ts.
- */
-
-import type { FileLockEntry } from "./types.js";
-
-export class FileLock {
-  /** Active locks by file path. */
-  private locks = new Map<string, FileLockEntry>();
-  /** Queue of waiting acquires per file path. */
-  private queues = new Map<string, Array<() => void>>();
-
-  /**
-   * Acquire a lock on a file. Blocks until available.
-   * Returns a release function.
-   *
-   * @param filePath - Absolute path to the file
-   * @param agentId - ID of the agent requesting the lock
-   * @returns Release function — call when done writing
-   */
-  async acquire(filePath: string, agentId: string): Promise<() => void> {
-    // Wait for existing lock
-    while (this.locks.has(filePath)) {
-      await new Promise<void>((resolve) => {
-        const queue = this.queues.get(filePath) ?? [];
-        queue.push(resolve);
-        this.queues.set(filePath, queue);
-      });
-    }
-
-    // Create lock entry
-    let releaseFn: () => void;
-    const promise = new Promise<void>((resolve) => {
-      releaseFn = () => {
-        this.locks.delete(filePath);
-        resolve();
-        // Wake next waiter
-        const queue = this.queues.get(filePath);
-        if (queue && queue.length > 0) {
-          const next = queue.shift()!;
-          next();
-        }
-      };
-    });
-
-    const entry: FileLockEntry = {
-      agentId,
-      filePath,
-      promise,
-      release: releaseFn!,
-    };
-
-    this.locks.set(filePath, entry);
-    return releaseFn!;
-  }
-
-  /**
-   * Check if a file is currently locked.
-   */
-  isLocked(filePath: string): boolean {
-    return this.locks.has(filePath);
-  }
-
-  /**
-   * Get the agent that holds a lock on a file.
-   */
-  getHolder(filePath: string): string | undefined {
-    return this.locks.get(filePath)?.agentId;
-  }
-
-  /**
-   * Get count of locked files.
-   */
-  get lockCount(): number {
-    return this.locks.size;
-  }
-
-  /**
-   * Release all locks held by an agent (on abort).
-   */
-  releaseAll(agentId: string): void {
-    for (const [filePath, entry] of this.locks) {
-      if (entry.agentId === agentId) {
-        entry.release();
-      }
-    }
-  }
-
-  /**
-   * Clear all locks (on shutdown).
-   */
-  clear(): void {
-    for (const entry of this.locks.values()) {
-      entry.release();
-    }
-    this.locks.clear();
-    this.queues.clear();
-  }
-}