@signalridge/pi-workflows 1.7.1 → 1.7.3

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 1.7.3
+### Patch Changes
+
+- e04f729: Remove repeated copies of the workflow arming reminder while preserving other input hooks.
+
+## 1.7.2
+### Patch Changes
+
+- 919655f: Yield before live agent dispatch so synchronous managed-spawn setup cannot be interrupted by the script bootstrap watchdog before promise tracking is attached. Preserve lexical call identity and reservations, check cancellation after yielding, and retain the 1000ms script watchdog. Add real Node process regressions for slow setup, runaway scripts, and pending-spawn cleanup.
+
 ## 1.7.1
 ### Patch Changes
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@signalridge/pi-workflows",
-  "version": "1.7.1",
+  "version": "1.7.3",
   "description": "Deterministic JavaScript orchestration workflows backed by pi-subagents managed spawning.",
   "type": "module",
   "license": "MIT",

package/skills/workflow-authoring/SKILL.md

@@ -2,7 +2,7 @@
 name: workflow-authoring
 description: Design, write, review, and debug Pi JavaScript workflows that coordinate multiple subagents. Use when a request needs named dependencies, bounded fan-out, per-item pipelines, quality gates, saved or nested workflows, pause/resume, or workflow progress; do not use it merely to run an existing named workflow.
 metadata:
-  version: "1.7.1"
+  version: "1.7.3"
 ---
 
 # Workflow authoring

package/skills/workflow-authoring/references/capabilities.md

@@ -2,7 +2,7 @@
 # Workflow capability index
 
 Contract format: `2.0.0`<br>
-Contract content / skill / extension: `1.7.1`
+Contract content / skill / extension: `1.7.3`
 
 This compact generated index covers supported runtime globals and workflow-tool inputs. For constraints, compatibility behavior, internal boundaries, and dynamic-reference ownership, follow the [exhaustive generated facts](capability-details.md).
 

package/skills/workflow-authoring/references/capability-details.md

@@ -2,7 +2,7 @@
 # Exhaustive workflow capability facts
 
 Contract format: `2.0.0`<br>
-Contract content / skill / extension: `1.7.1`
+Contract content / skill / extension: `1.7.3`
 
 Every exact fact below is projected from the installed extension's capability contract. Explanatory judgment belongs in the hand-written references next to this file.
 

package/skills/workflow-patterns/SKILL.md

@@ -2,7 +2,7 @@
 name: workflow-patterns
 description: Choose and run one of Pi's built-in multi-agent workflows for research, adversarial review, code review, multi-perspective analysis, or codebase auditing. Use when the request matches a named pattern; use workflow-authoring instead for a new script or custom topology.
 metadata:
-  version: "1.7.1"
+  version: "1.7.3"
 ---
 
 # Built-in workflow patterns

package/skills/workflow-review/SKILL.md

@@ -2,7 +2,7 @@
 name: workflow-review
 description: Audit an existing Pi JavaScript workflow for topology, prompt/data flow, limits, failure handling, cancellation, resume safety, and publication compatibility. Use when reviewing or debugging a workflow script or saved run; do not use to run a standard named workflow.
 metadata:
-  version: "1.7.1"
+  version: "1.7.3"
 ---
 
 # Workflow review

package/src/arming.ts

@@ -29,6 +29,16 @@ export const DEFAULT_TRIGGER_WORD = "workflow";
 export const WORKFLOW_ARMED_DIRECTIVE =
   "<system-reminder>You typed the workflow trigger word, which counts as an explicit opt-in to multi-agent orchestration: the `workflow` tool is authorized for this turn. It runs in the background by default — the turn ends and the result is delivered back into the conversation when it finishes, which is expected rather than a stall, so you need not stay and block. Pass background:false only when the user is waiting for the result inline. This is permission, not an instruction: if the request is conversational, trivial, or better answered directly, answer it directly and do not run a workflow.</system-reminder>";
 
+/** Keep one copy when an earlier input transform already added this directive. */
+export function deduplicateWorkflowDirective(text: string): string {
+  let seen = false;
+  return text.replaceAll(WORKFLOW_ARMED_DIRECTIVE, (directive) => {
+    if (seen) return "";
+    seen = true;
+    return directive;
+  });
+}
+
 function escapeRegExp(text: string): string {
   return text.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 }

package/src/index.ts

@@ -3,7 +3,12 @@ import type { ExtensionAPI, ExtensionCommandContext, ExtensionContext } from "@e
 import { defineTool } from "@earendil-works/pi-coding-agent";
 import { Text } from "@earendil-works/pi-tui";
 import { Type } from "@sinclair/typebox";
-import { DEFAULT_TRIGGER_WORD, hasTriggerWord, WORKFLOW_ARMED_DIRECTIVE } from "./arming.js";
+import {
+  DEFAULT_TRIGGER_WORD,
+  deduplicateWorkflowDirective,
+  hasTriggerWord,
+  WORKFLOW_ARMED_DIRECTIVE,
+} from "./arming.js";
 import { BUILTIN_WORKFLOWS, validateBuiltinArgs } from "./builtins.js";
 import { type CommandResult, resolveCodeReviewScope } from "./code-review-scope.js";
 import { type ScriptStartOptions, type ScriptStartResult, WorkflowEngine, WorkflowWaitAbortedError } from "./engine.js";
@@ -952,6 +957,14 @@ export default function piWorkflows(pi: ExtensionAPI): void {
     // question. Nothing is swallowed and no UI opens.
     pi.on("input", (event) => {
       if (event.source === "extension") return { action: "continue" as const };
+      const deduplicatedText = deduplicateWorkflowDirective(event.text);
+      if (deduplicatedText !== event.text) {
+        return {
+          action: "transform" as const,
+          text: deduplicatedText,
+          ...(event.images ? { images: event.images } : {}),
+        };
+      }
       const explicitTrigger = keywordTriggerEnabled && hasTriggerWord(event.text, triggerKeyword);
       const substantive = effortLevel !== "off" && event.text.trim().length >= 16 && !event.text.trim().startsWith("/");
       if (!explicitTrigger && !substantive) return { action: "continue" as const };
@@ -969,6 +982,29 @@ export default function piWorkflows(pi: ExtensionAPI): void {
       };
     });
 
+    // Input transforms can be chained with handlers loaded after this package.
+    // Normalize only our own repeated marker again at the final context boundary;
+    // unrelated plugin content and hooks remain untouched.
+    pi.on("context", (event) => {
+      type UserMessage = Extract<(typeof event.messages)[number], { role: "user" }>;
+      let lastUserIndex = -1;
+      let lastUserMessage: UserMessage | undefined;
+      for (let index = 0; index < event.messages.length; index += 1) {
+        const message = event.messages[index];
+        if (message && "role" in message && message.role === "user") {
+          lastUserIndex = index;
+          lastUserMessage = message;
+        }
+      }
+      const message = lastUserMessage;
+      if (lastUserIndex < 0 || !message || !("content" in message) || typeof message.content !== "string") return;
+      const deduplicatedText = deduplicateWorkflowDirective(message.content);
+      if (deduplicatedText === message.content) return;
+      const messages = event.messages.slice();
+      messages[lastUserIndex] = { ...message, content: deduplicatedText };
+      return { messages };
+    });
+
     // Standing effort mode mirrors the reference: it auto-arms substantive
     // interactive messages but remains guidance, not an inferred spend ceiling.
     pi.registerCommand("effort", {

package/src/runtime.ts

@@ -1606,6 +1606,16 @@ export async function runWorkflow<T = unknown>(
     shared.physicalAgentCount++;
 
     return limiter(async () => {
+      // Keep lexical identity, scope, thread ownership and cap reservations above
+      // synchronous, but leave the timed VM bootstrap before invoking host code.
+      // Even an immediately available limiter slot must yield: synchronous spawn
+      // setup can outlast the script watchdog and interrupt RPC promise creation
+      // before its callers attach rejection/cleanup handlers. The returned call
+      // is now tracked before dispatch, and a bootstrap timeout cancels it here.
+      await Promise.resolve();
+      throwIfAborted();
+      if (batch?.cancelled) throw agentLimitError();
+
       const timeout = agentOptions.timeoutMs !== undefined ? agentOptions.timeoutMs : agentTimeoutMs;
       const retryAttempts = normalizeAgentRetries(agentOptions.retries ?? options.agentRetries ?? 0);
       const maxAttempts = retryAttempts + 1;