@mcpc-tech/cli 0.1.8-beta.2 → 0.1.9

package/bin/mcpc.mjs

@@ -2139,29 +2139,19 @@ var BaseSamplingExecutor = class {
             }
           });
           const action = parsedData["action"];
-          const actionStr = action && typeof action === "string" ? String(action) : "unknown_action";
-          const spanName = `mcpc.sampling_iteration.${actionStr}`;
-          iterationSpan = this.tracingEnabled ? startSpan(spanName, {
-            iteration: this.currentIteration + 1,
-            agent: this.name,
-            action: actionStr,
-            systemPrompt: systemPrompt(),
-            maxTokens: String(Number.MAX_SAFE_INTEGER),
-            maxIterations: this.maxIterations,
-            messages: JSON.stringify(this.conversationHistory)
-          }, loopSpan ?? void 0) : null;
-          if (!action || typeof parsedData["decision"] !== "string") {
+          const decision = parsedData["decision"];
+          if (typeof decision !== "string") {
             this.conversationHistory.push({
               role: "user",
               content: {
                 type: "text",
-                text: 'Required fields missing. During execution provide: {"action":"<tool>","decision":"proceed|retry","<tool>":{}}. When complete provide: {"decision":"complete"}'
+                text: 'Missing required field "decision". Provide: {"action":"<tool>","decision":"proceed|retry","<tool>":{}} or {"decision":"complete"}'
               }
             });
             if (iterationSpan) endSpan(iterationSpan);
             continue;
           }
-          if (parsedData["decision"] === "complete" && action) {
+          if (decision === "complete" && action) {
             this.conversationHistory.push({
               role: "user",
               content: {
@@ -2172,6 +2162,28 @@ var BaseSamplingExecutor = class {
             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,
+            agent: this.name,
+            action: actionStr,
+            systemPrompt: systemPrompt(),
+            maxTokens: String(Number.MAX_SAFE_INTEGER),
+            maxIterations: this.maxIterations,
+            messages: JSON.stringify(this.conversationHistory)
+          }, loopSpan ?? void 0) : null;
           const result = await this.processAction(parsedData, schema, state, iterationSpan);
           this.logIterationProgress(parsedData, result, model, stopReason, role);
           if (iterationSpan) {
@@ -2187,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,
@@ -2452,19 +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) {
-      return {
-        content: [
-          {
-            type: "text",
-            text: 'Invalid: Cannot have both "decision":"complete" and "action" field. When complete, only provide {"decision":"complete"}. When executing, provide {"action":"<tool>","decision":"proceed|retry","<tool>":{}}.'
-          }
-        ],
-        isError: true
-      };
-    }
     if (toolCallData.decision === "complete") {
       return await this.createCompletionResult("Task completed", parentSpan);
     }
@@ -3084,19 +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) {
-      return {
-        content: [
-          {
-            type: "text",
-            text: 'Invalid: Cannot have both "decision":"complete" and "action" field. When complete, only provide {"decision":"complete"}. When executing, provide {"action":"<tool>","decision":"proceed|retry","<tool>":{}}.'
-          }
-        ],
-        isError: true
-      };
-    }
     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.2",
+  "version": "0.1.9",
   "homepage": "https://jsr.io/@mcpc/cli",
   "type": "module",
   "dependencies": {

package/types/src/app.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"app.d.ts","sources":["../../src/app.ts"],"names":[],"mappings":"AAAA,SAAS,WAAW,4BAAwC;AAI5D,cAAc,mBAAmB,0BAA0D;AAC3F,cAAc,UAAU,6BAA6B;AAErD,OAAO,cAAM,eACX,SAAS,eACR,QAAQ,qBA8BT;AAEF,OAAO,cAAM,iBAAgB,YAO3B"}
+{"version":3,"file":"app.d.ts","sources":["../../src/app.ts"],"names":[],"mappings":"AAAA,SAAS,WAAW,4BAAwC;AAI5D,cAAc,mBAAmB,0BAAmD;AACpF,cAAc,UAAU,6BAA6B;AAErD,OAAO,cAAM,eACX,SAAS,eACR,QAAQ,qBA8BT;AAEF,OAAO,cAAM,iBAAgB,YAO3B"}

package/types/src/config/loader.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"loader.d.ts","sources":["../../../src/config/loader.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA2CC,GAED,cAAc,iBAAiB,0BAAuC;AAKtE,iBAAiB;EACf;;GAEC,GACD,OAAO,MAAM;EACb;;GAEC,GACD,UAAU,MAAM;EAChB;;GAEC,GACD;IACE,QAAQ,OAAO,MAAM,EAAE,OAAO;IAC9B,WAAW,OAAO,MAAM,EAAE,OAAO;;EAEnC;;GAEC,GACD,QAAQ;;AAwHV;;;CAGC,GACD,OAAO,iBAAe,cAAc,QAAQ,aAAa,IAAI;AAoK7D;;CAEC,GACD,OAAO,iBAAS,eAAe,QAAQ,UAAU,GAAG,IAAI"}
+{"version":3,"file":"loader.d.ts","sources":["../../../src/config/loader.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA2CC,GAED,cAAc,iBAAiB,0BAAgC;AAK/D,iBAAiB;EACf;;GAEC,GACD,OAAO,MAAM;EACb;;GAEC,GACD,UAAU,MAAM;EAChB;;GAEC,GACD;IACE,QAAQ,OAAO,MAAM,EAAE,OAAO;IAC9B,WAAW,OAAO,MAAM,EAAE,OAAO;;EAEnC;;GAEC,GACD,QAAQ;;AAwHV;;;CAGC,GACD,OAAO,iBAAe,cAAc,QAAQ,aAAa,IAAI;AAoK7D;;CAEC,GACD,OAAO,iBAAS,eAAe,QAAQ,UAAU,GAAG,IAAI"}