@pi-unipi/workflow 2.4.0 → 2.4.1

package/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/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/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/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",