@haoyiyin/workflow 0.2.7 → 0.2.9

package/dist/src/pi-extension.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pi-extension.d.ts","sourceRoot":"","sources":["../../src/pi-extension.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAMH,MAAM,CAAC,OAAO,WAAW,EAAE,EAAE,GAAG,QAwD/B"}
+{"version":3,"file":"pi-extension.d.ts","sourceRoot":"","sources":["../../src/pi-extension.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAMH,MAAM,CAAC,OAAO,WAAW,EAAE,EAAE,GAAG,QAwF/B"}

package/dist/src/pi-extension.js

@@ -16,35 +16,72 @@ export default function (pi) {
         }
         // Classify intent
         const intent = classifyIntent(event.text);
-        // If intent matches a skill, dispatch
-        if (intent.skill) {
-            ctx.ui.notify(`[yi-workflow] Dispatching to /skill:${intent.skill} (${intent.reason})`, "info");
-            // Transform input to skill command
-            return {
-                action: "transform",
-                text: `/skill:${intent.skill} ${event.text}`,
-            };
+        const targetSkill = intent.skill || "quick-task";
+        // Determine reason for dispatch
+        let reason = intent.reason;
+        if (!intent.skill) {
+            const needsExternal = /^(what|how|why|when|where|who|search|find|look|weather|time|price|stock|translate)/i.test(event.text);
+            if (needsExternal) {
+                reason = "External query - needs subagent";
+            }
+            else {
+                reason = "Default dispatch - no direct execution allowed";
+            }
         }
-        // NO DIRECT EXECUTION: Block all non-skill work
-        // If no yi-workflow skill matches, check if it's a question that needs external data
-        const needsExternal = /^(what|how|why|when|where|who|search|find|look|weather|time|price|stock|translate)/i.test(event.text);
-        if (needsExternal) {
-            ctx.ui.notify(`[yi-workflow] External query detected - dispatching to quick-task for handling`, "info");
-            return {
-                action: "transform",
-                text: `/skill:quick-task ${event.text}`,
-            };
-        }
-        // Block direct execution with explanation
-        ctx.ui.notify(`[yi-workflow] ⚠️ THIN DISPATCHER: I cannot execute directly. Please use /skill:name or rephrase your request.`, "warning");
+        ctx.ui.notify(`[yi-workflow] Dispatching to /skill:${targetSkill} (${reason})`, "info");
+        // Transform to skill command - main agent will receive "/skill:X ..."
+        // The before_agent_start hook injects constraints forcing main agent to call skill
         return {
             action: "transform",
-            text: `/skill:quick-task ${event.text}`,
+            text: `/skill:${targetSkill} ${event.text}`,
         };
     });
     // Log when session starts
     pi.on("session_start", async (event, ctx) => {
         ctx.ui.notify("yi-workflow extension loaded", "info");
     });
+    // CRITICAL: Inject THIN DISPATCHER rules into system prompt
+    // This ensures main agent NEVER executes work directly
+    pi.on("before_agent_start", async (event, ctx) => {
+        const systemPrompt = event.systemPrompt || "";
+        const userInput = event.userInput || "";
+        // Detect if this input was transformed to a skill command
+        const isSkillCommand = userInput.startsWith("/skill:");
+        // Prepend THIN DISPATCHER constraints
+        const thinDispatcherRules = `
+╔══════════════════════════════════════════════════════════════════╗
+║              THIN DISPATCHER PROTOCOL [ABSOLUTE]                 ║
+╚══════════════════════════════════════════════════════════════════╝
+
+YOU ARE A THIN DISPATCHER. YOUR ONLY JOB IS TO DISPATCH TO SKILLS.
+
+⚠️ ABSOLUTE RULE #1: If user input starts with "/skill:", you MUST call that skill.
+⚠️ ABSOLUTE RULE #2: You NEVER execute work yourself - only dispatch.
+⚠️ ABSOLUTE RULE #3: Tools (Read, Edit, Bash, etc.) are FORBIDDEN unless in subagent.
+
+FORBIDDEN ACTIONS (VIOLATION = PROTOCOL BREACH):
+- Reading files (use /skill:review-diff)
+- Writing/editing files (use /skill:execute-plan)
+- Running commands (use /skill:quick-task)
+- Writing code directly (use /skill:tdd or /skill:execute-plan)
+- Using any tool directly (use /skill:quick-task to dispatch subagent)
+
+CURRENT INPUT: "${userInput}"
+IS_SKILL_COMMAND: ${isSkillCommand ? "YES - MUST CALL SKILL" : "NO"}
+
+${isSkillCommand ? `
+╔══════════════════════════════════════════════════════════════════╗
+║ ⚠️  MANDATORY: CALL THE SKILL BELOW  ⚠️                          ║
+╚══════════════════════════════════════════════════════════════════╝
+The input starts with "/skill:". You MUST call that skill.
+DO NOT process this input as a regular conversation.
+DO NOT use tools directly.
+CALL THE SKILL: "${userInput.split(" ")[0]}"
+` : ""}
+`;
+        // Prepend to system prompt
+        event.systemPrompt = thinDispatcherRules + "\n\n" + systemPrompt;
+        return { action: "continue" };
+    });
 }
 //# sourceMappingURL=pi-extension.js.map

package/dist/src/pi-extension.js.map

@@ -1 +1 @@
-{"version":3,"file":"pi-extension.js","sourceRoot":"","sources":["../../src/pi-extension.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAE3D,6CAA6C;AAC7C,gDAAgD;AAChD,MAAM,CAAC,OAAO,WAAW,EAAO;IAC9B,yCAAyC;IACzC,EAAE,CAAC,EAAE,CAAC,OAAO,EAAE,KAAK,EAAE,KAAU,EAAE,GAAQ,EAAE,EAAE;QAC5C,4CAA4C;QAC5C,IAAI,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YAC/B,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC;QAChC,CAAC;QAED,kBAAkB;QAClB,MAAM,MAAM,GAAG,cAAc,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAE1C,sCAAsC;QACtC,IAAI,MAAM,CAAC,KAAK,EAAE,CAAC;YACjB,GAAG,CAAC,EAAE,CAAC,MAAM,CACX,uCAAuC,MAAM,CAAC,KAAK,KAAK,MAAM,CAAC,MAAM,GAAG,EACxE,MAAM,CACP,CAAC;YAEF,mCAAmC;YACnC,OAAO;gBACL,MAAM,EAAE,WAAW;gBACnB,IAAI,EAAE,UAAU,MAAM,CAAC,KAAK,IAAI,KAAK,CAAC,IAAI,EAAE;aAC7C,CAAC;QACJ,CAAC;QAED,gDAAgD;QAChD,qFAAqF;QACrF,MAAM,aAAa,GAAG,qFAAqF,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAE7H,IAAI,aAAa,EAAE,CAAC;YAClB,GAAG,CAAC,EAAE,CAAC,MAAM,CACX,gFAAgF,EAChF,MAAM,CACP,CAAC;YACF,OAAO;gBACL,MAAM,EAAE,WAAW;gBACnB,IAAI,EAAE,qBAAqB,KAAK,CAAC,IAAI,EAAE;aACxC,CAAC;QACJ,CAAC;QAED,0CAA0C;QAC1C,GAAG,CAAC,EAAE,CAAC,MAAM,CACX,+GAA+G,EAC/G,SAAS,CACV,CAAC;QAEF,OAAO;YACL,MAAM,EAAE,WAAW;YACnB,IAAI,EAAE,qBAAqB,KAAK,CAAC,IAAI,EAAE;SACxC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,0BAA0B;IAC1B,EAAE,CAAC,EAAE,CAAC,eAAe,EAAE,KAAK,EAAE,KAAU,EAAE,GAAQ,EAAE,EAAE;QACpD,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,8BAA8B,EAAE,MAAM,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;AACL,CAAC"}
+{"version":3,"file":"pi-extension.js","sourceRoot":"","sources":["../../src/pi-extension.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAE3D,6CAA6C;AAC7C,gDAAgD;AAChD,MAAM,CAAC,OAAO,WAAW,EAAO;IAC9B,yCAAyC;IACzC,EAAE,CAAC,EAAE,CAAC,OAAO,EAAE,KAAK,EAAE,KAAU,EAAE,GAAQ,EAAE,EAAE;QAC5C,4CAA4C;QAC5C,IAAI,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YAC/B,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC;QAChC,CAAC;QAED,kBAAkB;QAClB,MAAM,MAAM,GAAG,cAAc,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAC1C,MAAM,WAAW,GAAG,MAAM,CAAC,KAAK,IAAI,YAAY,CAAC;QAEjD,gCAAgC;QAChC,IAAI,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC;QAC3B,IAAI,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC;YAClB,MAAM,aAAa,GAAG,qFAAqF,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;YAC7H,IAAI,aAAa,EAAE,CAAC;gBAClB,MAAM,GAAG,iCAAiC,CAAC;YAC7C,CAAC;iBAAM,CAAC;gBACN,MAAM,GAAG,gDAAgD,CAAC;YAC5D,CAAC;QACH,CAAC;QAED,GAAG,CAAC,EAAE,CAAC,MAAM,CACX,uCAAuC,WAAW,KAAK,MAAM,GAAG,EAChE,MAAM,CACP,CAAC;QAEF,sEAAsE;QACtE,mFAAmF;QACnF,OAAO;YACL,MAAM,EAAE,WAAW;YACnB,IAAI,EAAE,UAAU,WAAW,IAAI,KAAK,CAAC,IAAI,EAAE;SAC5C,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,0BAA0B;IAC1B,EAAE,CAAC,EAAE,CAAC,eAAe,EAAE,KAAK,EAAE,KAAU,EAAE,GAAQ,EAAE,EAAE;QACpD,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,8BAA8B,EAAE,MAAM,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;IAEH,4DAA4D;IAC5D,uDAAuD;IACvD,EAAE,CAAC,EAAE,CAAC,oBAAoB,EAAE,KAAK,EAAE,KAAU,EAAE,GAAQ,EAAE,EAAE;QACzD,MAAM,YAAY,GAAG,KAAK,CAAC,YAAY,IAAI,EAAE,CAAC;QAC9C,MAAM,SAAS,GAAG,KAAK,CAAC,SAAS,IAAI,EAAE,CAAC;QAExC,0DAA0D;QAC1D,MAAM,cAAc,GAAG,SAAS,CAAC,UAAU,CAAC,SAAS,CAAC,CAAC;QAEvD,sCAAsC;QACtC,MAAM,mBAAmB,GAAG;;;;;;;;;;;;;;;;;;kBAkBd,SAAS;oBACP,cAAc,CAAC,CAAC,CAAC,uBAAuB,CAAC,CAAC,CAAC,IAAI;;EAEjE,cAAc,CAAC,CAAC,CAAC;;;;;;;mBAOA,SAAS,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;CACzC,CAAC,CAAC,CAAC,EAAE;CACL,CAAC;QAEE,2BAA2B;QAC3B,KAAK,CAAC,YAAY,GAAG,mBAAmB,GAAG,MAAM,GAAG,YAAY,CAAC;QAEjE,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC;IAChC,CAAC,CAAC,CAAC;AACL,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@haoyiyin/workflow",
-  "version": "0.2.7",
+  "version": "0.2.9",
   "type": "module",
   "description": "Production workflow skills for planning, executing, reviewing, and finishing coding-agent work",
   "main": "./dist/index.js",

package/src/pi-extension.ts

@@ -19,45 +19,29 @@ export default function (pi: any) {
 
     // Classify intent
     const intent = classifyIntent(event.text);
+    const targetSkill = intent.skill || "quick-task";
 
-    // If intent matches a skill, dispatch
-    if (intent.skill) {
-      ctx.ui.notify(
-        `[yi-workflow] Dispatching to /skill:${intent.skill} (${intent.reason})`,
-        "info"
-      );
-
-      // Transform input to skill command
-      return {
-        action: "transform",
-        text: `/skill:${intent.skill} ${event.text}`,
-      };
+    // Determine reason for dispatch
+    let reason = intent.reason;
+    if (!intent.skill) {
+      const needsExternal = /^(what|how|why|when|where|who|search|find|look|weather|time|price|stock|translate)/i.test(event.text);
+      if (needsExternal) {
+        reason = "External query - needs subagent";
+      } else {
+        reason = "Default dispatch - no direct execution allowed";
+      }
     }
 
-    // NO DIRECT EXECUTION: Block all non-skill work
-    // If no yi-workflow skill matches, check if it's a question that needs external data
-    const needsExternal = /^(what|how|why|when|where|who|search|find|look|weather|time|price|stock|translate)/i.test(event.text);
-
-    if (needsExternal) {
-      ctx.ui.notify(
-        `[yi-workflow] External query detected - dispatching to quick-task for handling`,
-        "info"
-      );
-      return {
-        action: "transform",
-        text: `/skill:quick-task ${event.text}`,
-      };
-    }
-
-    // Block direct execution with explanation
     ctx.ui.notify(
-      `[yi-workflow] ⚠️ THIN DISPATCHER: I cannot execute directly. Please use /skill:name or rephrase your request.`,
-      "warning"
+      `[yi-workflow] Dispatching to /skill:${targetSkill} (${reason})`,
+      "info"
     );
 
+    // Transform to skill command - main agent will receive "/skill:X ..."
+    // The before_agent_start hook injects constraints forcing main agent to call skill
     return {
       action: "transform",
-      text: `/skill:quick-task ${event.text}`,
+      text: `/skill:${targetSkill} ${event.text}`,
     };
   });
 
@@ -65,4 +49,52 @@ export default function (pi: any) {
   pi.on("session_start", async (event: any, ctx: any) => {
     ctx.ui.notify("yi-workflow extension loaded", "info");
   });
+
+  // CRITICAL: Inject THIN DISPATCHER rules into system prompt
+  // This ensures main agent NEVER executes work directly
+  pi.on("before_agent_start", async (event: any, ctx: any) => {
+    const systemPrompt = event.systemPrompt || "";
+    const userInput = event.userInput || "";
+
+    // Detect if this input was transformed to a skill command
+    const isSkillCommand = userInput.startsWith("/skill:");
+
+    // Prepend THIN DISPATCHER constraints
+    const thinDispatcherRules = `
+╔══════════════════════════════════════════════════════════════════╗
+║              THIN DISPATCHER PROTOCOL [ABSOLUTE]                 ║
+╚══════════════════════════════════════════════════════════════════╝
+
+YOU ARE A THIN DISPATCHER. YOUR ONLY JOB IS TO DISPATCH TO SKILLS.
+
+⚠️ ABSOLUTE RULE #1: If user input starts with "/skill:", you MUST call that skill.
+⚠️ ABSOLUTE RULE #2: You NEVER execute work yourself - only dispatch.
+⚠️ ABSOLUTE RULE #3: Tools (Read, Edit, Bash, etc.) are FORBIDDEN unless in subagent.
+
+FORBIDDEN ACTIONS (VIOLATION = PROTOCOL BREACH):
+- Reading files (use /skill:review-diff)
+- Writing/editing files (use /skill:execute-plan)
+- Running commands (use /skill:quick-task)
+- Writing code directly (use /skill:tdd or /skill:execute-plan)
+- Using any tool directly (use /skill:quick-task to dispatch subagent)
+
+CURRENT INPUT: "${userInput}"
+IS_SKILL_COMMAND: ${isSkillCommand ? "YES - MUST CALL SKILL" : "NO"}
+
+${isSkillCommand ? `
+╔══════════════════════════════════════════════════════════════════╗
+║ ⚠️  MANDATORY: CALL THE SKILL BELOW  ⚠️                          ║
+╚══════════════════════════════════════════════════════════════════╝
+The input starts with "/skill:". You MUST call that skill.
+DO NOT process this input as a regular conversation.
+DO NOT use tools directly.
+CALL THE SKILL: "${userInput.split(" ")[0]}"
+` : ""}
+`;
+
+    // Prepend to system prompt
+    event.systemPrompt = thinDispatcherRules + "\n\n" + systemPrompt;
+
+    return { action: "continue" };
+  });
 }