@pi-unipi/unipi 2.4.0 → 2.4.2

package/packages/updater/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@pi-unipi/updater",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "Auto-updater, changelog browser, and readme browser for Unipi — checks npm registry, renders CHANGELOG.md and README.md files in TUI overlays",
   "type": "module",
-  "main": "src/index.ts",
+  "main": "index.ts",
   "license": "MIT",
   "author": "Neuron Mr White",
   "repository": {
@@ -22,6 +22,7 @@
   ],
   "files": [
     "index.ts",
+    "types.ts",
     "src/**/*.ts",
     "skills/**/*",
     "README.md"
@@ -33,7 +34,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@pi-unipi/core": "2.4.0"
+    "@pi-unipi/core": "2.4.1"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "^0.80.0",

package/packages/utility/README.md

@@ -10,7 +10,7 @@ The diff rendering is the standout feature — Shiki-powered syntax-highlighted
 |---------|-------------|
 | `/unipi:env` | Show environment info (Node, Pi, OS, paths) |
 | `/unipi:doctor` | Run diagnostics across all modules |
-| `/unipi:status` | Show status of all unipi modules |
+| `/unipi:status` | Show immediate guidance to the live `/unipi:info` dashboard and `/unipi:doctor` diagnostics |
 | `/unipi:cleanup` | Clean stale DBs, temp files, old sessions |
 | `/unipi:reload` | Explain how to reload extensions |
 | `/unipi:name-badge` | Toggle name badge overlay |

package/packages/utility/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/utility",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "Utility commands and tools for Pi coding agent — lifecycle, diagnostics, cache, analytics, display, batch execution",
   "type": "module",
   "main": "src/index.ts",
@@ -28,21 +28,24 @@
     "src/types.ts",
     "src/**/*.ts",
     "README.md",
-    "skills/**/*.md"
+    "skills/**/*.md",
+    "!src/__tests__/**",
+    "!src/**/*.test.ts"
   ],
   "publishConfig": {
     "access": "public"
   },
   "dependencies": {
-    "@pi-unipi/core": "2.4.0",
+    "@pi-unipi/core": "2.4.1",
     "diff": "^7.0.0",
-    "@shikijs/cli": "^4.0.2"
+    "shiki": "^4.0.2"
   },
   "devDependencies": {
     "@types/diff": "^7.0.2"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "^0.80.0",
+    "@earendil-works/pi-tui": "^0.80.0",
     "typebox": "^1.1.38"
   },
   "scripts": {

package/packages/utility/src/commands.ts

@@ -234,19 +234,11 @@ export function registerUtilityCommands(pi: ExtensionAPI): void {
         return;
       }
 
-      // Request status from all modules — responses will be logged
-      const requestId = `status-${Date.now()}`;
-      emitEvent(pi, UNIPI_EVENTS.MODULE_STATUS_REQUEST, { requestId });
-
-      // Give modules a moment to respond, then show what we know
-      await new Promise((resolve) => setTimeout(resolve, 500));
-
       sendResponse(
         pi,
         "## 📊 Module Status\n\n" +
-          "Status request broadcast to all modules.\n" +
-          "Modules that support status reporting will respond via events.\n\n" +
-          `*Request ID: \`${requestId}\`*`,
+          "Use `/unipi:info` for the live module and tool dashboard.\n" +
+          "Use `/unipi:doctor` to run configuration and runtime diagnostics.",
       );
     },
   });

package/packages/utility/src/diff/theme.ts

@@ -101,9 +101,6 @@ const PRESETS: Record<string, DiffPreset> = {
   },
 };
 
-/** Hardcoded fallback colors (last resort) */
-const HARDCODED_FALLBACK: DiffColors = PRESETS.default.colors;
-
 // ─── Preset Access ──────────────────────────────────────────────────────────────
 
 /**

package/packages/utility/src/tui/name-badge.ts

@@ -13,15 +13,6 @@ import type { Theme } from "@earendil-works/pi-coding-agent";
 /** Placeholder text when no session name is set */
 const PLACEHOLDER = "Set a name";
 
-/**
- * Pad content to exact visible width.
- */
-function padVisible(content: string, targetWidth: number): string {
-  const vw = visibleWidth(content);
-  const pad = Math.max(0, targetWidth - vw);
-  return content + " ".repeat(pad);
-}
-
 /**
  * NameBadgeComponent — bordered box HUD overlay showing session name.
  *

package/packages/web-api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/web-api",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "Web search, read, and summarize tools with provider-based backend selection for Pi coding agent",
   "type": "module",
   "main": "src/index.ts",
@@ -31,7 +31,7 @@
     "README.md"
   ],
   "dependencies": {
-    "@pi-unipi/core": "2.4.0",
+    "@pi-unipi/core": "2.4.1",
     "defuddle": "^0.18.1",
     "linkedom": "^0.18.12",
     "lodash": "^4.17.21",

package/packages/web-api/src/engine/extract.ts

@@ -35,7 +35,6 @@ import { truncateContent, formatContent } from "./format.js";
 const MAX_REDIRECTS = 5;
 
 /** Maximum alternate link fallbacks to try */
-const MAX_ALTERNATE_LINKS = 3;
 
 /**
  * Validate a URL for fetching.
@@ -193,31 +192,6 @@ function findMetaRefresh(document: Document): string | null {
   return match[1];
 }
 
-/**
- * Check for alternate JSON content links.
- *
- * @param document - DOM document
- * @returns Array of alternate URLs
- */
-function findAlternateLinks(document: Document): string[] {
-  const alternates: string[] = [];
-
-  // Look for JSON feeds, oEmbed, etc.
-  const links = document.querySelectorAll(
-    'link[rel="alternate"][type="application/json"], ' +
-    'link[rel="alternate"][type="application/ld+json"]'
-  );
-
-  for (const link of Array.from(links)) {
-    const href = link.getAttribute("href");
-    if (href) {
-      alternates.push(href);
-    }
-  }
-
-  return alternates.slice(0, MAX_ALTERNATE_LINKS);
-}
-
 /**
  * Detect content type from response.
  */

package/packages/web-api/src/tools.ts

@@ -62,19 +62,6 @@ function getAvailableProviders(capability: WebCapability): WebProvider[] {
   });
 }
 
-/**
- * Select provider for a capability.
- * If sourceRank is specified, use that rank.
- * Otherwise, use the lowest-ranked available provider.
- */
-function selectProvider(
-  capability: WebCapability,
-  sourceRank?: number
-): WebProvider {
-  const candidates = selectProviderChain(capability, sourceRank);
-  return candidates[0];
-}
-
 /**
  * Build the ordered list of providers to try for a capability.
  *

package/packages/workflow/commands.ts

@@ -8,7 +8,7 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { readFileSync, readdirSync, existsSync, statSync } from "fs";
 import { join, basename } from "path";
-import { UNIPI_PREFIX, WORKFLOW_COMMANDS, getToolsForCommand, getSandboxLevel, type SandboxLevel } from "@pi-unipi/core";
+import { UNIPI_PREFIX, WORKFLOW_COMMANDS, getToolsForCommand, getSandboxLevel, type SandboxLevel, type UnipiWorkflowEvent } from "@pi-unipi/core";
 
 type CompletionItem = { value: string; label: string; description: string };
 
@@ -22,6 +22,8 @@ export interface WorkflowCommandOptions {
   setActiveTools: (tools: string[], level: SandboxLevel) => void;
   /** Save tools for later restore */
   saveTools: (tools: string[]) => void;
+  /** Begin one workflow lifecycle; false means another workflow is active. */
+  startWorkflow: (event: UnipiWorkflowEvent) => boolean;
 }
 
 /** Command definition */
@@ -402,6 +404,16 @@ export function registerWorkflowCommands(
         return null;
       },
       handler: async (args, ctx) => {
+        const workflowEvent: UnipiWorkflowEvent = {
+          command: cmd.name,
+          fullCommand: `/${fullCommand}`,
+          args: args?.trim() ?? "",
+        };
+        if (!options.startWorkflow(workflowEvent)) {
+          if (ctx.hasUI) ctx.ui.notify("Another UniPi workflow is still active", "warning");
+          return;
+        }
+
         // Apply sandbox — save current tools, set command's tools
         const currentTools = options.getActiveTools();
         options.saveTools(currentTools);

package/packages/workflow/index.ts

@@ -21,6 +21,7 @@ import {
   getBlockedToolsForLevel,
 } from "@pi-unipi/core";
 import { registerWorkflowCommands } from "./commands.js";
+import { WorkflowLifecycle } from "./lifecycle.js";
 
 /** Package version (read from package.json at load time) */
 const VERSION = getPackageVersion(dirname(fileURLToPath(import.meta.url)));
@@ -41,6 +42,7 @@ let currentSandboxLevel: SandboxLevel | null = null;
 let currentSandboxTools: string[] | null = null;
 
 export default function (pi: ExtensionAPI) {
+  const workflowLifecycle = new WorkflowLifecycle();
 
   // Register all workflow commands
   registerWorkflowCommands(pi, {
@@ -55,6 +57,11 @@ export default function (pi: ExtensionAPI) {
     saveTools: (tools: string[]) => {
       savedTools = tools;
     },
+    startWorkflow: (event) => {
+      if (!workflowLifecycle.start(event)) return false;
+      emitEvent(pi, UNIPI_EVENTS.WORKFLOW_START, event);
+      return true;
+    },
   });
 
   // Block tool calls that violate sandbox
@@ -109,7 +116,7 @@ export default function (pi: ExtensionAPI) {
   });
 
   // Restore tools when agent finishes
-  pi.on("agent_end", async (_event, _ctx) => {
+  pi.on("agent_end", async (event, _ctx) => {
     if (sandboxActive && savedTools) {
       pi.setActiveTools(savedTools);
       savedTools = null;
@@ -117,6 +124,9 @@ export default function (pi: ExtensionAPI) {
       currentSandboxLevel = null;
       currentSandboxTools = null;
     }
+
+    const completedWorkflow = workflowLifecycle.complete(event.messages);
+    if (completedWorkflow) emitEvent(pi, UNIPI_EVENTS.WORKFLOW_END, completedWorkflow);
   });
 
   // Announce module presence on session start
@@ -163,5 +173,6 @@ export default function (pi: ExtensionAPI) {
     ralphDetected = false;
     savedTools = null;
     sandboxActive = false;
+    workflowLifecycle.reset();
   });
 }

package/packages/workflow/lifecycle.ts

@@ -0,0 +1,43 @@
+import type { UnipiWorkflowEvent } from "@pi-unipi/core";
+
+interface WorkflowMessage {
+  role: string;
+  stopReason?: string;
+}
+
+export interface CompletedWorkflowEvent extends UnipiWorkflowEvent {
+  success: boolean;
+  durationMs: number;
+}
+
+/** Single-active workflow lifecycle state shared by slash handlers and agent_end. */
+export class WorkflowLifecycle {
+  private active: (UnipiWorkflowEvent & { startedAt: number }) | null = null;
+
+  constructor(private readonly now: () => number = Date.now) {}
+
+  start(event: UnipiWorkflowEvent): boolean {
+    if (this.active) return false;
+    this.active = { ...event, startedAt: this.now() };
+    return true;
+  }
+
+  complete(messages: WorkflowMessage[]): CompletedWorkflowEvent | undefined {
+    if (!this.active) return undefined;
+    const workflow = this.active;
+    this.active = null;
+    const finalAssistant = [...messages].reverse().find((message) => message.role === "assistant");
+    const success = finalAssistant?.stopReason !== "error" && finalAssistant?.stopReason !== "aborted";
+    return {
+      command: workflow.command,
+      fullCommand: workflow.fullCommand,
+      args: workflow.args,
+      success,
+      durationMs: this.now() - workflow.startedAt,
+    };
+  }
+
+  reset(): void {
+    this.active = null;
+  }
+}

package/packages/workflow/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/workflow",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "Structured development workflow commands for Pi coding agent",
   "type": "module",
   "main": "index.ts",
@@ -19,6 +19,7 @@
   ],
   "files": [
     "*.ts",
+    "!*.test.ts",
     "skills/**/*",
     "README.md"
   ],
@@ -26,7 +27,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@pi-unipi/core": "2.4.0"
+    "@pi-unipi/core": "2.4.1"
   },
   "peerDependencies": {
     "@earendil-works/pi-ai": "^0.80.0",

package/packages/autocomplete/index.ts

@@ -1,7 +0,0 @@
-/**
- * @pi-unipi/command-enchantment — Re-export
- *
- * Re-exports from src/ for module resolution.
- */
-
-export { default } from "./src/index.js";

package/packages/autocomplete/src/__tests__/command-registry.audit.test.ts

@@ -1,120 +0,0 @@
-import { describe, expect, it } from "vitest";
-import { existsSync, globSync, readFileSync } from "node:fs";
-import { dirname, join } from "node:path";
-
-function findRepoRoot(start: string): string {
-  let dir = start;
-  while (dir !== dirname(dir)) {
-    if (existsSync(join(dir, "packages", "autocomplete", "src", "constants.ts"))) return dir;
-    dir = dirname(dir);
-  }
-  return start;
-}
-
-const root = findRepoRoot(process.cwd());
-
-function read(path: string): string {
-  return readFileSync(join(root, path), "utf-8");
-}
-
-function collectConstants(): Map<string, string> {
-  const constants = new Map<string, string>([["UNIPI_PREFIX", "unipi:"]]);
-
-  for (const path of globSync("packages/**/*.ts", { cwd: root })) {
-    const text = read(path);
-    for (const obj of text.matchAll(/(?:export\s+)?const\s+(\w+)\s*=\s*\{([\s\S]*?)\}\s*as\s+const/g)) {
-      const [, name, body] = obj;
-      for (const item of body.matchAll(/(\w+):\s*"([^"]+)"/g)) {
-        constants.set(`${name}.${item[1]}`, item[2]);
-      }
-    }
-  }
-
-  return constants;
-}
-
-function evaluateCommandExpression(expr: string, constants: Map<string, string>): string | null {
-  const trimmed = expr.trim();
-
-  if (trimmed.startsWith('"') || trimmed.startsWith("'")) {
-    const quote = trimmed[0];
-    const end = trimmed.indexOf(quote, 1);
-    return end > 0 ? trimmed.slice(1, end) : null;
-  }
-
-  if (trimmed.startsWith("`")) {
-    const end = trimmed.lastIndexOf("`");
-    if (end <= 0) return null;
-    return trimmed.slice(1, end).replace(/\$\{([^}]+)\}/g, (_match, key: string) => {
-      const resolved = constants.get(key.trim());
-      return resolved ?? `\${${key}}`;
-    });
-  }
-
-  return constants.get(trimmed) ?? null;
-}
-
-function registeredCommands(): { commands: Set<string>; nonUnipi: string[]; unresolved: string[] } {
-  const constants = collectConstants();
-  const commands = new Set<string>();
-  const nonUnipi: string[] = [];
-  const unresolved: string[] = [];
-
-  for (const path of globSync("packages/**/*.ts", { cwd: root }).sort()) {
-    const text = read(path);
-    if (!text.includes("registerCommand")) continue;
-
-    for (const match of text.matchAll(/\.registerCommand\(\s*([^,\n]+)/g)) {
-      const expr = match[1].trim();
-
-      // Workflow registers a loop over WORKFLOW_COMMANDS via local `fullCommand`.
-      if (path === "packages/workflow/commands.ts" && expr === "fullCommand") {
-        for (const [key, value] of constants) {
-          if (key.startsWith("WORKFLOW_COMMANDS.")) commands.add(`unipi:${value}`);
-        }
-        continue;
-      }
-
-      const command = evaluateCommandExpression(expr, constants);
-      if (!command || command.includes("${")) {
-        unresolved.push(`${path}: ${expr}`);
-        continue;
-      }
-
-      if (command.startsWith("unipi:")) {
-        commands.add(command);
-      } else {
-        nonUnipi.push(`${command} (${path})`);
-      }
-    }
-  }
-
-  return { commands, nonUnipi, unresolved };
-}
-
-function autocompleteRegistry(): { registry: Set<string>; descriptions: Set<string>; registryPackages: Set<string>; labels: Set<string> } {
-  const text = read("packages/autocomplete/src/constants.ts");
-  const registryBody = text.match(/export const COMMAND_REGISTRY[^=]*= \{([\s\S]*?)\n\};/)?.[1] ?? "";
-  const descriptionsBody = text.match(/export const COMMAND_DESCRIPTIONS[^=]*= \{([\s\S]*?)\n\};/)?.[1] ?? "";
-  const labelsBody = text.match(/export const PACKAGE_LABELS[^=]*= \{([\s\S]*?)\n\};/)?.[1] ?? "";
-
-  const registry = new Set([...registryBody.matchAll(/"(unipi:[^"]+)"\s*:/g)].map((m) => m[1]));
-  const descriptions = new Set([...descriptionsBody.matchAll(/"(unipi:[^"]+)"\s*:/g)].map((m) => m[1]));
-  const registryPackages = new Set([...registryBody.matchAll(/"unipi:[^"]+"\s*:\s*"([^"]+)"/g)].map((m) => m[1]));
-  const labels = new Set([...labelsBody.matchAll(/^\s*"?([a-z][a-z0-9-]*)"?:\s*"/gm)].map((m) => m[1]));
-
-  return { registry, descriptions, registryPackages, labels };
-}
-
-describe("autocomplete command registry audit", () => {
-  it("mirrors every registered /unipi:* command and has no non-unipi package commands", () => {
-    const registered = registeredCommands();
-    const autocomplete = autocompleteRegistry();
-
-    expect(registered.unresolved, "unresolved registerCommand expressions").toEqual([]);
-    expect(registered.nonUnipi, "package commands must use the unipi: prefix").toEqual([]);
-    expect([...registered.commands].sort()).toEqual([...autocomplete.registry].sort());
-    expect([...autocomplete.registry].sort()).toEqual([...autocomplete.descriptions].sort());
-    expect([...autocomplete.registryPackages].filter((pkg) => !autocomplete.labels.has(pkg)).sort()).toEqual([]);
-  });
-});