@signalridge/pi-workflows 1.7.4 → 1.7.5

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 1.7.5
+### Patch Changes
+
+- 3364cc0: Declare `@sinclair/typebox` as a host-provided peer dependency with a `*` range instead of a runtime dependency, avoiding duplicate runtime modules and Pi extension-loader warnings.
+- 3364cc0: Adopt Pi 1.0.4's host-provided dependency contract: declare imported Pi and TypeBox modules as `*` peers rather than installing duplicate runtime modules, and align Pi development pins with the repository's tested 1.0.4 baseline. The wildcard identifies host module ownership; it does not promise compatibility with every Pi version.
+- 3364cc0: Bind code-review discovery and workflow executions (including resumes and nested scripts) to the session working directory. Preserve structured user content when deduplicating the workflow directive.
+- Updated dependencies [3364cc0]
+- Updated dependencies [3364cc0]
+  - @signalridge/pi-subagents-protocol@1.5.1
+
 ## 1.7.4
 ### Patch Changes
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@signalridge/pi-workflows",
-  "version": "1.7.4",
+  "version": "1.7.5",
   "description": "Deterministic JavaScript orchestration workflows backed by pi-subagents managed spawning.",
   "type": "module",
   "license": "MIT",
@@ -15,13 +15,13 @@
     "lifecycle": "stable"
   },
   "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.84.0 || ^0.85.0 || ^0.86.0 || ^0.87.0 || ^0.99.1",
-    "@earendil-works/pi-tui": "^0.84.0 || ^0.85.0 || ^0.86.0 || ^0.87.0 || ^0.99.1",
-    "@signalridge/pi-subagents": ">=1.9.0"
+    "@earendil-works/pi-coding-agent": "*",
+    "@earendil-works/pi-tui": "*",
+    "@signalridge/pi-subagents": ">=1.9.0",
+    "@sinclair/typebox": "*"
   },
   "dependencies": {
-    "@signalridge/pi-subagents-protocol": "^1.5.0",
-    "@sinclair/typebox": "^0.34.50",
+    "@signalridge/pi-subagents-protocol": "^1.5.1",
     "acorn": "^8.16.0"
   },
   "pi": {

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.4"
+  version: "1.7.5"
 ---
 
 # 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.4`
+Contract content / skill / extension: `1.7.5`
 
 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.4`
+Contract content / skill / extension: `1.7.5`
 
 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.4"
+  version: "1.7.5"
 ---
 
 # 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.4"
+  version: "1.7.5"
 ---
 
 # Workflow review

package/src/engine.ts

@@ -351,6 +351,7 @@ export class WorkflowEngine {
     private readonly journal: JournalWriter,
     private readonly readEntries: () => readonly SessionEntryLike[] = () => [],
     protocolGate?: () => Promise<ManagedProtocolCheck>,
+    private readonly cwd: string = process.cwd(),
   ) {
     this.protocolGate =
       protocolGate ??
@@ -1110,6 +1111,7 @@ export class WorkflowEngine {
 
     try {
       const result = await runWorkflow(script, {
+        cwd: this.cwd,
         args: options.args,
         agent: runner,
         mainModel: options.mainModel,

package/src/index.ts

@@ -118,9 +118,10 @@ function textResult(text: string, details: unknown = {}) {
  * Run one command for diff discovery. argv array, never a shell string — a
  * branch name or path reaches this from user input.
  */
-function runCommand(file: string, args: string[]): CommandResult {
+function runCommand(file: string, args: string[], cwd: string): CommandResult {
   try {
     const stdout = execFileSync(file, args, {
+      cwd,
       encoding: "utf8",
       maxBuffer: 32 * 1024 * 1024,
       stdio: ["ignore", "pipe", "ignore"],
@@ -1048,11 +1049,29 @@ export default function piWorkflows(pi: ExtensionAPI): void {
         }
       }
       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;
+      if (lastUserIndex < 0 || !message || !("content" in message)) return;
+      let seen = false;
+      const deduplicate = (text: string): string =>
+        text.replaceAll(WORKFLOW_ARMED_DIRECTIVE, (directive) => {
+          if (seen) return "";
+          seen = true;
+          return directive;
+        });
+      const content =
+        typeof message.content === "string"
+          ? deduplicate(message.content)
+          : message.content.map((part) => {
+              if (part.type !== "text") return part;
+              const text = deduplicate(part.text);
+              return text === part.text ? part : { ...part, text };
+            });
+      if (
+        content === message.content ||
+        (Array.isArray(content) && content.every((part, index) => part === message.content[index]))
+      )
+        return;
       const messages = event.messages.slice();
-      messages[lastUserIndex] = { ...message, content: deduplicatedText };
+      messages[lastUserIndex] = { ...message, content };
       return { messages };
     });
 
@@ -1115,7 +1134,8 @@ export default function piWorkflows(pi: ExtensionAPI): void {
           // takes a diff rather than something typed, so it resolves one first.
           let scriptArgs: Record<string, unknown> | undefined;
           if (name === "code-review") {
-            const scope = resolveCodeReviewScope(text, runCommand);
+            const cwd = ctx.cwd;
+            const scope = resolveCodeReviewScope(text, (file, argv) => runCommand(file, argv, cwd));
             for (const notice of scope.notices) ctx.ui.notify(notice, "warning");
             if (!scope.diff.trim()) return;
             scriptArgs = { diff: scope.diff, diffSource: scope.diffSource };
@@ -1357,6 +1377,7 @@ export default function piWorkflows(pi: ExtensionAPI): void {
           }
         },
         awaitProtocol,
+        sessionCwd,
       );
       try {
         const entries =