@pi-unipi/unipi 2.4.1 → 2.4.2

package/packages/subagents/src/__tests__/workflow-integration.test.ts

@@ -1,334 +0,0 @@
-/**
- * Test: Workflow integration — `/unipi:work` with subagent support
- *
- * Verifies:
- * - spawn_helper and get_helper_result tools are properly defined
- * - Agent types (explore, work) are correctly configured
- * - Concurrency limit is respected
- * - Custom agent type loading works
- * - System prompt builder generates correct prompts
- */
-
-import { describe, it } from "node:test";
-import assert from "node:assert/strict";
-
-// Test type definitions
-const BUILTIN_TYPES = ["explore", "work"] as const;
-
-interface AgentConfig {
-  name: string;
-  displayName?: string;
-  description: string;
-  builtinToolNames?: string[];
-  disallowedTools?: string[];
-  extensions: true | string[] | false;
-  skills: true | string[] | false;
-  model?: string;
-  thinking?: string;
-  maxTurns?: number;
-  systemPrompt: string;
-  promptMode: "replace" | "append";
-  inheritContext?: boolean;
-  runInBackground?: boolean;
-  isolated?: boolean;
-  memory?: string;
-  isDefault?: boolean;
-  enabled?: boolean;
-  source?: "builtin" | "project" | "global";
-}
-
-// Test prompt builder
-function buildAgentPrompt(
-  config: AgentConfig,
-  cwd: string,
-  env: { isGitRepo: boolean; branch: string; platform: string },
-  parentSystemPrompt: string,
-): string {
-  if (config.promptMode === "append") {
-    return [
-      parentSystemPrompt,
-      "",
-      "---",
-      "",
-      `## Agent Role: ${config.displayName ?? config.name}`,
-      config.systemPrompt,
-    ].join("\n");
-  }
-
-  return [
-    `# ${config.displayName ?? config.name}`,
-    "",
-    config.systemPrompt,
-    "",
-    "---",
-    "",
-    `Working directory: ${cwd}`,
-    `Git: ${env.isGitRepo ? `${env.branch} on ${env.platform}` : "not a git repo"}`,
-  ].join("\n");
-}
-
-// Test concurrency manager
-class ConcurrencyManager {
-  private maxConcurrent: number;
-  private running: number = 0;
-  private queue: Array<{ id: string; resolve: () => void }> = [];
-
-  constructor(maxConcurrent: number) {
-    this.maxConcurrent = maxConcurrent;
-  }
-
-  async acquire(id: string): Promise<() => void> {
-    if (this.running >= this.maxConcurrent) {
-      await new Promise<void>((resolve) => {
-        this.queue.push({ id, resolve });
-      });
-    }
-
-    this.running++;
-
-    let released = false;
-    return () => {
-      if (released) return;
-      released = true;
-      this.running--;
-
-      // Start next in queue
-      if (this.queue.length > 0) {
-        const next = this.queue.shift()!;
-        next.resolve();
-      }
-    };
-  }
-
-  getRunning(): number {
-    return this.running;
-  }
-
-  getQueueLength(): number {
-    return this.queue.length;
-  }
-}
-
-describe("Workflow Integration", () => {
-  describe("Tool Definitions", () => {
-    it("should define spawn_helper tool with correct parameters", () => {
-      const toolDef = {
-        name: "spawn_helper",
-        description: "Launch a sub-agent for parallel work.",
-        parameters: {
-          type: "object",
-          required: ["type", "prompt", "description"],
-          properties: {
-            type: { type: "string" },
-            prompt: { type: "string" },
-            description: { type: "string" },
-            run_in_background: { type: "boolean" },
-            max_turns: { type: "number" },
-            model: { type: "string" },
-            thinking: { type: "string" },
-          },
-        },
-      };
-
-      assert.equal(toolDef.name, "spawn_helper");
-      assert.equal(toolDef.parameters.required.length, 3);
-      assert.ok(toolDef.parameters.properties.type);
-      assert.ok(toolDef.parameters.properties.prompt);
-      assert.ok(toolDef.parameters.properties.description);
-    });
-
-    it("should define get_helper_result tool with correct parameters", () => {
-      const toolDef = {
-        name: "get_helper_result",
-        description: "Check status and retrieve results from a background agent.",
-        parameters: {
-          type: "object",
-          required: ["agent_id"],
-          properties: {
-            agent_id: { type: "string" },
-            wait: { type: "boolean" },
-          },
-        },
-      };
-
-      assert.equal(toolDef.name, "get_helper_result");
-      assert.equal(toolDef.parameters.required.length, 1);
-      assert.ok(toolDef.parameters.properties.agent_id);
-    });
-  });
-
-  describe("Agent Types", () => {
-    it("should have explore and work as builtin types", () => {
-      assert.deepEqual(BUILTIN_TYPES, ["explore", "work"]);
-    });
-
-    it("should define explore agent with read-only tools", () => {
-      const exploreConfig: AgentConfig = {
-        name: "explore",
-        description: "Fast parallel codebase exploration",
-        builtinToolNames: ["read", "bash", "grep", "find", "ls"],
-        systemPrompt: "You are a read-only exploration agent.",
-        promptMode: "replace",
-        extensions: true,
-        skills: true,
-      };
-
-      assert.ok(!exploreConfig.builtinToolNames?.includes("write"));
-      assert.ok(!exploreConfig.builtinToolNames?.includes("edit"));
-      assert.ok(exploreConfig.builtinToolNames?.includes("read"));
-    });
-
-    it("should define work agent with read-write tools", () => {
-      const workConfig: AgentConfig = {
-        name: "work",
-        description: "Parallel file writes with transparent locking",
-        builtinToolNames: ["read", "write", "edit", "bash", "grep", "find", "ls"],
-        systemPrompt: "You are a read-write work agent.",
-        promptMode: "replace",
-        extensions: true,
-        skills: true,
-      };
-
-      assert.ok(workConfig.builtinToolNames?.includes("write"));
-      assert.ok(workConfig.builtinToolNames?.includes("edit"));
-      assert.ok(workConfig.builtinToolNames?.includes("read"));
-    });
-  });
-
-  describe("System Prompt Builder", () => {
-    const env = {
-      isGitRepo: true,
-      branch: "main",
-      platform: "GitHub",
-    };
-
-    it("should build replace mode prompt", () => {
-      const config: AgentConfig = {
-        name: "explore",
-        displayName: "Explorer",
-        description: "Test",
-        systemPrompt: "Find all authentication files.",
-        promptMode: "replace",
-        extensions: true,
-        skills: true,
-      };
-
-      const prompt = buildAgentPrompt(config, "/workspace", env, "Parent prompt");
-
-      assert.ok(prompt.startsWith("# Explorer"));
-      assert.ok(prompt.includes("Find all authentication files."));
-      assert.ok(prompt.includes("Working directory: /workspace"));
-      assert.ok(!prompt.includes("Parent prompt"));
-    });
-
-    it("should build append mode prompt", () => {
-      const config: AgentConfig = {
-        name: "work",
-        displayName: "Worker",
-        description: "Test",
-        systemPrompt: "Refactor the auth module.",
-        promptMode: "append",
-        extensions: true,
-        skills: true,
-      };
-
-      const prompt = buildAgentPrompt(config, "/workspace", env, "Parent prompt");
-
-      assert.ok(prompt.startsWith("Parent prompt"));
-      assert.ok(prompt.includes("## Agent Role: Worker"));
-      assert.ok(prompt.includes("Refactor the auth module."));
-    });
-  });
-
-  describe("Concurrency Limit", () => {
-    it("should respect max concurrent limit", async () => {
-      const manager = new ConcurrencyManager(2);
-      const events: string[] = [];
-
-      // Start 3 agents, only 2 should run immediately
-      const release1 = await manager.acquire("agent-1");
-      events.push("agent-1-started");
-
-      const release2 = await manager.acquire("agent-2");
-      events.push("agent-2-started");
-
-      assert.equal(manager.getRunning(), 2);
-      assert.deepEqual(events, ["agent-1-started", "agent-2-started"]);
-
-      // Third agent should queue
-      const acquire3Promise = manager.acquire("agent-3").then((release) => {
-        events.push("agent-3-started");
-        return release;
-      });
-
-      assert.equal(manager.getQueueLength(), 1);
-
-      // Release first agent
-      release1();
-      const release3 = await acquire3Promise;
-
-      assert.equal(manager.getRunning(), 2);
-      assert.deepEqual(events, ["agent-1-started", "agent-2-started", "agent-3-started"]);
-
-      release2();
-      release3();
-    });
-
-    it("should queue agents in order", async () => {
-      const manager = new ConcurrencyManager(1);
-      const events: string[] = [];
-
-      const release1 = await manager.acquire("agent-1");
-      events.push("1");
-
-      const p2 = manager.acquire("agent-2").then(r => { events.push("2"); return r; });
-      const p3 = manager.acquire("agent-3").then(r => { events.push("3"); return r; });
-
-      release1();
-      const release2 = await p2;
-      release2();
-      const release3 = await p3;
-
-      assert.deepEqual(events, ["1", "2", "3"]);
-      release3();
-    });
-  });
-
-  describe("Custom Agent Loading", () => {
-    it("should parse agent markdown frontmatter", () => {
-      const markdown = `---
-name: code-checker
-description: Code quality checker
-tools: read, grep, find, bash
-thinking: high
----
-You are a code quality checker. Review code for issues.`;
-
-      // Simple frontmatter parser
-      const match = markdown.match(/^---\n([\s\S]*?)\n---\n([\s\S]*)$/);
-      assert.ok(match);
-
-      const frontmatter = match![1];
-      const body = match![2];
-
-      assert.ok(frontmatter.includes("name: code-checker"));
-      assert.ok(frontmatter.includes("tools: read, grep, find, bash"));
-      assert.ok(body.includes("You are a code quality checker."));
-    });
-
-    it("should validate required fields", () => {
-      const validAgent = {
-        name: "test-agent",
-        description: "Test agent",
-        systemPrompt: "Do something.",
-        promptMode: "replace" as const,
-        extensions: true as const,
-        skills: true as const,
-      };
-
-      assert.ok(validAgent.name);
-      assert.ok(validAgent.description);
-      assert.ok(validAgent.systemPrompt);
-    });
-  });
-});

package/packages/subagents/src/prompts.ts

@@ -1,39 +0,0 @@
-/**
- * @pi-unipi/subagents — System prompt builder
- */
-
-import type { AgentConfig } from "./types.js";
-
-/**
- * Build system prompt for an agent.
- */
-export function buildAgentPrompt(
-  config: AgentConfig,
-  cwd: string,
-  env: { isGitRepo: boolean; branch: string; platform: string },
-  parentSystemPrompt: string,
-): string {
-  if (config.promptMode === "append") {
-    // Append mode: parent prompt + agent additions
-    return [
-      parentSystemPrompt,
-      "",
-      "---",
-      "",
-      `## Agent Role: ${config.displayName ?? config.name}`,
-      config.systemPrompt,
-    ].join("\n");
-  }
-
-  // Replace mode: standalone prompt
-  return [
-    `# ${config.displayName ?? config.name}`,
-    "",
-    config.systemPrompt,
-    "",
-    "---",
-    "",
-    `Working directory: ${cwd}`,
-    `Git: ${env.isGitRepo ? `${env.branch} on ${env.platform}` : "not a git repo"}`,
-  ].join("\n");
-}

package/packages/utility/index.ts

@@ -1 +0,0 @@
-export { default } from "./src/index.js";

package/packages/web-api/src/tui/progress.ts

@@ -1,168 +0,0 @@
-/**
- * @unipi/web-api — TUI Progress Renderer
- *
- * Renders batch fetch progress for TUI display.
- */
-
-import type { FetchProgress, FetchProgressStatus } from "../engine/types.js";
-
-/** Spinner frames for animation */
-const SPINNER_FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
-
-/** Status glyphs */
-const STATUS_GLYPHS: Record<FetchProgressStatus, string> = {
-  queued: "○",
-  connecting: SPINNER_FRAMES[0],
-  waiting: SPINNER_FRAMES[0],
-  loading: SPINNER_FRAMES[0],
-  processing: SPINNER_FRAMES[0],
-  done: "✓",
-  error: "✗",
-};
-
-/**
- * Get a spinner frame for the given index.
- * Cycles through spinner frames for animation.
- *
- * @param index - Animation frame index
- * @returns Spinner character
- */
-export function getSpinnerFrame(index: number): string {
-  return SPINNER_FRAMES[index % SPINNER_FRAMES.length];
-}
-
-/**
- * Render a progress bar.
- *
- * @param percent - Progress percentage (0-100)
- * @param width - Bar width in characters
- * @returns Progress bar string
- */
-export function renderProgressBar(percent: number, width: number = 10): string {
-  const filled = Math.round((percent / 100) * width);
-  const empty = width - filled;
-  return "█".repeat(filled) + "░".repeat(empty);
-}
-
-/**
- * Truncate a URL for display.
- *
- * @param url - URL to truncate
- * @param maxLength - Maximum length
- * @returns Truncated URL
- */
-function truncateUrl(url: string, maxLength: number): string {
-  if (url.length <= maxLength) {
-    return url;
-  }
-
-  // Try to keep the domain
-  try {
-    const parsed = new URL(url);
-    const domain = parsed.host;
-    const path = parsed.pathname + parsed.search;
-
-    if (domain.length + 3 >= maxLength) {
-      return url.slice(0, maxLength - 1) + "…";
-    }
-
-    const remaining = maxLength - domain.length - 3;
-    if (path.length <= remaining) {
-      return domain + path;
-    }
-
-    return domain + path.slice(0, remaining - 1) + "…";
-  } catch {
-    return url.slice(0, maxLength - 1) + "…";
-  }
-}
-
-/**
- * Render a single progress item line.
- *
- * @param progress - Progress object
- * @param width - Available width
- * @param spinnerIndex - Animation frame index
- * @returns Formatted line
- */
-export function renderProgressLine(
-  progress: FetchProgress,
-  width: number = 80,
-  spinnerIndex: number = 0
-): string {
-  // Status glyph
-  let glyph = STATUS_GLYPHS[progress.status];
-  if (["connecting", "waiting", "loading", "processing"].includes(progress.status)) {
-    glyph = getSpinnerFrame(spinnerIndex);
-  }
-
-  // Truncate URL
-  const urlMax = Math.min(40, width - 30);
-  const url = truncateUrl(progress.url, urlMax);
-
-  // Progress bar
-  const bar = renderProgressBar(progress.percent, 8);
-
-  // Status text
-  const statusText = progress.phase || progress.status;
-
-  // Format line
-  return `${glyph} ${url.padEnd(urlMax)}  ${statusText.padEnd(12)} [${bar}]`;
-}
-
-/**
- * Render batch progress header.
- *
- * @param progress - All progress items
- * @param concurrency - Current concurrency
- * @returns Header line
- */
-export function renderBatchProgressHeader(
-  progress: FetchProgress[],
-  concurrency: number
-): string {
-  const total = progress.length;
-  const done = progress.filter((p) => p.status === "done").length;
-  const error = progress.filter((p) => p.status === "error").length;
-  const active = progress.filter(
-    (p) => !["queued", "done", "error"].includes(p.status)
-  ).length;
-
-  return `batch_web_content_read ${done}/${total} done · ok ${done - error} · err ${error} · concurrency ${concurrency}`;
-}
-
-/**
- * Render full batch progress display.
- *
- * @param progress - All progress items
- * @param concurrency - Current concurrency
- * @param width - Available width
- * @param spinnerIndex - Animation frame index
- * @returns Formatted string
- */
-export function renderBatchProgress(
-  progress: FetchProgress[],
-  concurrency: number = 8,
-  width: number = 80,
-  spinnerIndex: number = 0
-): string {
-  const lines: string[] = [];
-
-  // Header
-  lines.push(renderBatchProgressHeader(progress, concurrency));
-  lines.push("");
-
-  // Progress items (show up to 10)
-  const maxItems = 10;
-  const itemsToShow = progress.slice(0, maxItems);
-
-  for (const item of itemsToShow) {
-    lines.push(renderProgressLine(item, width, spinnerIndex));
-  }
-
-  if (progress.length > maxItems) {
-    lines.push(`  ... and ${progress.length - maxItems} more`);
-  }
-
-  return lines.join("\n");
-}

package/packages/web-api/src/tui/result.ts

@@ -1,173 +0,0 @@
-/**
- * @unipi/web-api — TUI Result Renderer
- *
- * Renders single and batch results for TUI display.
- */
-
-import type { FetchResult, BatchFetchResult, FetchError } from "../engine/types.js";
-
-/** Maximum preview lines */
-const PREVIEW_LINES = 7;
-
-/**
- * Truncate content for preview.
- *
- * @param content - Content to preview
- * @param maxLines - Maximum lines
- * @returns Preview string
- */
-function truncatePreview(content: string, maxLines: number = PREVIEW_LINES): string {
-  const lines = content.split("\n").slice(0, maxLines);
-  return lines.join("\n");
-}
-
-/**
- * Render a single result for display.
- *
- * @param result - Fetch result
- * @param verbose - Include metadata header
- * @returns Formatted string
- */
-export function renderSingleResult(
-  result: FetchResult,
-  verbose: boolean = true
-): string {
-  const lines: string[] = [];
-
-  if (verbose) {
-    // Title
-    lines.push(`# ${result.title || "Untitled"}`);
-    lines.push("");
-
-    // Metadata
-    const meta: string[] = [];
-    if (result.author) {
-      meta.push(`Author: ${result.author}`);
-    }
-    if (result.published) {
-      meta.push(`Published: ${result.published}`);
-    }
-    if (result.site) {
-      meta.push(`Site: ${result.site}`);
-    }
-    if (result.language) {
-      meta.push(`Language: ${result.language}`);
-    }
-    if (result.wordCount) {
-      meta.push(`Words: ${result.wordCount}`);
-    }
-
-    if (meta.length > 0) {
-      lines.push(meta.join(" · "));
-    }
-
-    // URL
-    lines.push(`URL: ${result.url}`);
-    if (result.finalUrl !== result.url) {
-      lines.push(`Final URL: ${result.finalUrl}`);
-    }
-
-    lines.push("");
-    lines.push("---");
-    lines.push("");
-  }
-
-  // Content preview
-  const preview = truncatePreview(result.content);
-  lines.push(preview);
-
-  // Expand hint
-  if (result.content.split("\n").length > PREVIEW_LINES) {
-    lines.push("");
-    lines.push(`... [${result.wordCount} words total · Ctrl+O to expand]`);
-  }
-
-  return lines.join("\n");
-}
-
-/**
- * Render a batch result for display.
- *
- * @param result - Batch fetch result
- * @returns Formatted string
- */
-export function renderBatchResult(result: BatchFetchResult): string {
-  const lines: string[] = [];
-
-  // Summary header
-  lines.push(`# Batch Read Results`);
-  lines.push("");
-  lines.push(
-    `Total: ${result.total} · Succeeded: ${result.succeeded} · Failed: ${result.failed}`
-  );
-  lines.push("");
-
-  // Per-item results
-  for (let i = 0; i < result.items.length; i++) {
-    const item = result.items[i];
-    const status = item.status === "done" ? "✓" : "✗";
-
-    lines.push(`## [${i + 1}/${result.total}] ${status}`);
-
-    if (item.status === "done") {
-      lines.push(`**${item.result.title}**`);
-      lines.push(`URL: ${item.result.url}`);
-      lines.push(`Words: ${item.result.wordCount}`);
-      lines.push("");
-
-      // Content preview
-      const preview = truncatePreview(item.result.content);
-      lines.push(preview);
-
-      if (item.result.content.split("\n").length > PREVIEW_LINES) {
-        lines.push("...");
-      }
-    } else {
-      lines.push(`URL: ${item.error.url || "unknown"}`);
-      lines.push(`Error: ${item.error.error}`);
-    }
-
-    lines.push("");
-  }
-
-  return lines.join("\n");
-}
-
-/**
- * Render an error result for display.
- *
- * @param error - Fetch error
- * @returns Formatted string
- */
-export function renderErrorResult(error: FetchError): string {
-  const lines: string[] = [];
-
-  lines.push(`# Fetch Error`);
-  lines.push("");
-  lines.push(`**${error.error}**`);
-  lines.push("");
-  lines.push(`Code: \`${error.code}\``);
-  lines.push(`Phase: \`${error.phase}\``);
-
-  if (error.url) {
-    lines.push("");
-    lines.push(`URL: ${error.url}`);
-    if (error.finalUrl && error.finalUrl !== error.url) {
-      lines.push(`Final URL: ${error.finalUrl}`);
-    }
-  }
-
-  if (error.statusCode) {
-    lines.push("");
-    lines.push(
-      `HTTP Status: ${error.statusCode}${error.statusText ? ` ${error.statusText}` : ""}`
-    );
-  }
-
-  if (error.retryable) {
-    lines.push("");
-    lines.push(`*This error may be retried.*`);
-  }
-
-  return lines.join("\n");
-}