@mcpc-tech/cli 0.1.8-beta.1 → 0.1.8-beta.3

package/bin/mcpc.mjs

@@ -1039,7 +1039,7 @@ var SystemPrompts = {
 4. **Collect feedback** from each action result
 5. **Decide** next step:
    - **proceed**: More work needed
-   - **complete**: Question answered
+   - **complete**: Question answered (do NOT provide action field)
    - **retry**: Current action failed
 6. **Provide** parameter object matching action name
 7. **Continue** until complete
@@ -1048,9 +1048,16 @@ var SystemPrompts = {
 \`\`\`json
 {
   "action": "tool_name",
-  "decision": "proceed|retry|complete", 
+  "decision": "proceed|retry", 
   "tool_name": { /* tool parameters */ }
 }
+\`\`\`
+
+**When task is complete:**
+\`\`\`json
+{
+  "decision": "complete"
+}
 \`\`\``,
   /**
    * Workflow execution system prompt
@@ -1090,14 +1097,22 @@ var SystemPrompts = {
 - Continue until question answered
 
 ## JSON Response Format
+**During execution:**
 \`\`\`json
 {
   "action": "tool_name",
-  "decision": "proceed|retry|complete",
+  "decision": "proceed|retry",
   "tool_name": { /* tool parameters */ }
 }
 \`\`\`
 
+**When task is complete (do NOT include action):**
+\`\`\`json
+{
+  "decision": "complete"
+}
+\`\`\`
+
 ## Available Tools
 {toolList}`,
   /**
@@ -1132,9 +1147,12 @@ You MUST respond with a JSON object for workflow execution:
 
 **For Step Execution (Subsequent Calls):**
 - action: "{toolName}"
-- decision: "proceed" (advance), "retry" (retry), or "complete" (finish - sampling mode only)
+- decision: "proceed" (advance), "retry" (retry)
 - [step parameters]: Tool-specific parameters you autonomously determine for current step
 
+**When Workflow is Complete (do NOT include action):**
+- decision: "complete"
+
 **\u{1F3AF} AGENTIC WORKFLOW CONSTRAINTS:**
 - Response must be pure JSON demonstrating autonomous decision-making within workflow structure
 - Invalid JSON indicates failure in agentic workflow reasoning
@@ -2077,7 +2095,7 @@ var BaseSamplingExecutor = class {
         role: "user",
         content: {
           type: "text",
-          text: 'Return ONE AND ONLY ONE raw JSON object (no code fences, explanations, or multiple objects). The JSON MUST include action and decision. Example: {"action":"<tool>","decision":"proceed|complete","<tool>":{}}'
+          text: 'Return ONE AND ONLY ONE raw JSON object (no code fences, explanations, or multiple objects). During execution provide: {"action":"<tool>","decision":"proceed|retry","<tool>":{}}. When complete provide: {"decision":"complete"}'
         }
       }
     ];
@@ -2121,7 +2139,41 @@ var BaseSamplingExecutor = class {
             }
           });
           const action = parsedData["action"];
-          const actionStr = action && typeof action === "string" ? String(action) : "unknown_action";
+          const decision = parsedData["decision"];
+          if (typeof decision !== "string") {
+            this.conversationHistory.push({
+              role: "user",
+              content: {
+                type: "text",
+                text: 'Missing required field "decision". Provide: {"action":"<tool>","decision":"proceed|retry","<tool>":{}} or {"decision":"complete"}'
+              }
+            });
+            if (iterationSpan) endSpan(iterationSpan);
+            continue;
+          }
+          if (decision === "complete" && action) {
+            this.conversationHistory.push({
+              role: "user",
+              content: {
+                type: "text",
+                text: 'Invalid: Cannot have both "decision":"complete" and "action" field. When complete, only provide {"decision":"complete"}.'
+              }
+            });
+            if (iterationSpan) endSpan(iterationSpan);
+            continue;
+          }
+          if (decision !== "complete" && !action) {
+            this.conversationHistory.push({
+              role: "user",
+              content: {
+                type: "text",
+                text: 'Missing required field "action". When executing, provide: {"action":"<tool>","decision":"proceed|retry","<tool>":{}}'
+              }
+            });
+            if (iterationSpan) endSpan(iterationSpan);
+            continue;
+          }
+          const actionStr = decision === "complete" ? "completion" : action && typeof action === "string" ? String(action) : "unknown_action";
           const spanName = `mcpc.sampling_iteration.${actionStr}`;
           iterationSpan = this.tracingEnabled ? startSpan(spanName, {
             iteration: this.currentIteration + 1,
@@ -2132,17 +2184,6 @@ var BaseSamplingExecutor = class {
             maxIterations: this.maxIterations,
             messages: JSON.stringify(this.conversationHistory)
           }, loopSpan ?? void 0) : null;
-          if (!action || typeof parsedData["decision"] !== "string") {
-            this.conversationHistory.push({
-              role: "user",
-              content: {
-                type: "text",
-                text: 'Required fields missing: action or decision. Return ONLY raw JSON, no code fences or explanations. Example: {"action":"<tool>","decision":"proceed|complete","<tool>":{}}'
-              }
-            });
-            if (iterationSpan) endSpan(iterationSpan);
-            continue;
-          }
           const result = await this.processAction(parsedData, schema, state, iterationSpan);
           this.logIterationProgress(parsedData, result, model, stopReason, role);
           if (iterationSpan) {
@@ -2158,6 +2199,7 @@ var BaseSamplingExecutor = class {
               maxIterations: this.maxIterations,
               parsed: rawJson,
               action: typeof action === "string" ? action : String(action),
+              decision: typeof decision === "string" ? decision : String(decision),
               samplingResponse: responseContent,
               toolResult: JSON.stringify(result),
               model,
@@ -2423,15 +2465,6 @@ var SamplingExecutor = class extends BaseSamplingExecutor {
   }
   async processAction(parsedData, schema, _state, parentSpan) {
     const toolCallData = parsedData;
-    const isComplete = toolCallData.decision === "complete";
-    const actionName = toolCallData.action;
-    if (isComplete && actionName && actionName !== "complete") {
-      this.logger.debug({
-        message: "Decision is 'complete' with action present, treating as 'proceed'",
-        action: actionName
-      });
-      toolCallData.decision = "proceed";
-    }
     if (toolCallData.decision === "complete") {
       return await this.createCompletionResult("Task completed", parentSpan);
     }
@@ -2479,8 +2512,7 @@ ${JSON.stringify(context2, null, 2)}`;
 ## Current Task
 You will now use agentic sampling to complete the following task: "${userRequest}"${contextInfo}
 
-When you need to use a tool, specify the tool name in 'action' and provide tool-specific parameters as additional properties.
-When the task is complete, use "action": "complete".`;
+When you need to use a tool, specify the tool name in 'action' and provide tool-specific parameters as additional properties.`;
     return this.injectJsonInstruction({
       prompt: basePrompt + taskPrompt,
       schema: agenticSchema
@@ -3052,15 +3084,6 @@ var WorkflowSamplingExecutor = class extends BaseSamplingExecutor {
       throw new Error("WorkflowState is required for workflow");
     }
     const toolCallData = parsedData;
-    const isComplete = toolCallData.decision === "complete";
-    const actionName = toolCallData.action;
-    if (isComplete && actionName && actionName !== "complete") {
-      this.logger.debug({
-        message: "Decision is 'complete' with action present, treating as 'proceed'",
-        action: actionName
-      });
-      toolCallData.decision = "proceed";
-    }
     if (toolCallData.decision === "complete") {
       return await this.createCompletionResult("Task completed", parentSpan);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mcpc-tech/cli",
-  "version": "0.1.8-beta.1",
+  "version": "0.1.8-beta.3",
   "homepage": "https://jsr.io/@mcpc/cli",
   "type": "module",
   "dependencies": {