spets 0.1.67 → 0.1.68

package/dist/index.js

@@ -930,8 +930,8 @@ function buildClarifyPrompt(params) {
   const parts = [];
   parts.push("# Clarify Phase");
   parts.push("");
-  parts.push("Your task is to identify any questions that need user input before generating the document.");
-  parts.push("Do NOT generate the document yet - only generate questions if needed.");
+  parts.push("Your task is to identify decisions that need user input before proceeding.");
+  parts.push("Do NOT generate the document yet - only identify decision points.");
   parts.push("");
   parts.push("## Task Information");
   parts.push("");
@@ -943,52 +943,63 @@ function buildClarifyPrompt(params) {
   parts.push("");
   parts.push(params.gatheredContext);
   parts.push("");
-  const hasQAHistory = params.previousQA && params.previousQA.length > 0;
-  if (hasQAHistory) {
-    parts.push("## Previous Q&A");
+  const hasPreviousDecisions = params.previousDecisions && params.previousDecisions.length > 0;
+  if (hasPreviousDecisions) {
+    parts.push("## Previous Decisions");
     parts.push("");
-    for (const entry of params.previousQA) {
-      parts.push(`**Q:** ${entry.question.question}`);
-      parts.push(`**A:** ${entry.answer.answer}`);
+    for (const entry of params.previousDecisions) {
+      const selectedOption = entry.decision.options.find((o) => o.id === entry.answer.selectedOptionId);
+      const label = entry.answer.selectedOptionId === "ai" ? `AI recommendation (${entry.decision.aiRecommendation?.optionId})` : selectedOption?.label || entry.answer.selectedOptionId;
+      parts.push(`**${entry.decision.decision}**: ${label}`);
+      if (entry.answer.customInput) {
+        parts.push(`  Custom input: ${entry.answer.customInput}`);
+      }
       parts.push("");
     }
-    parts.push("## Your Task");
-    parts.push("");
-    parts.push("For each Q&A above, judge if the question is resolved:");
-    parts.push("- Clear answer \u2192 resolved, do NOT re-ask");
-    parts.push('- "you decide" / "\uC54C\uC544\uC11C \uD574" \u2192 resolved, use your judgment');
-    parts.push('- "give options" / "\uC635\uC158 \uC918" \u2192 NOT resolved, re-ask WITH concrete options');
-    parts.push("- Confused / unclear \u2192 NOT resolved, re-ask simpler");
-    parts.push("");
-    parts.push("Only output questions that are NOT resolved. Do NOT ask new questions.");
+    parts.push("Based on these decisions, identify any remaining decision points.");
+    parts.push("Do NOT re-ask about decisions already made.");
     parts.push("");
   } else {
-    parts.push("## Consider");
+    parts.push("## Your Task");
     parts.push("");
-    parts.push("1. Is the task clear enough to proceed?");
-    parts.push("2. Are there ambiguities that need user decision?");
+    parts.push("Identify decision points where:");
+    parts.push("1. Multiple valid approaches exist");
+    parts.push("2. User preference matters");
+    parts.push("3. Tradeoffs need to be considered");
     parts.push("");
   }
   parts.push("## Output Format");
   parts.push("");
-  parts.push("Output a JSON object with questions array:");
+  parts.push("For each decision, provide options with descriptions and tradeoffs.");
+  parts.push('Always include "ai" as the last option with your recommendation.');
   parts.push("");
   parts.push("```json");
   parts.push("{");
-  parts.push('  "questions": [');
+  parts.push('  "ready": false,');
+  parts.push('  "decisions": [');
   parts.push("    {");
-  parts.push('      "id": "q1",');
-  parts.push('      "question": "Your specific question?",');
-  parts.push('      "context": "Why this matters for the document",');
-  parts.push('      "options": ["Option A", "Option B"]  // optional');
+  parts.push('      "id": "d1",');
+  parts.push('      "decision": "What needs to be decided",');
+  parts.push('      "why": "Why this decision matters",');
+  parts.push('      "options": [');
+  parts.push('        { "id": "opt1", "label": "Option A", "description": "...", "tradeoffs": "..." },');
+  parts.push('        { "id": "opt2", "label": "Option B", "description": "...", "tradeoffs": "..." },');
+  parts.push('        { "id": "ai", "label": "Let AI decide", "description": "AI chooses based on context", "recommendation": "opt1", "reason": "..." }');
+  parts.push("      ]");
   parts.push("    }");
   parts.push("  ]");
   parts.push("}");
   parts.push("```");
   parts.push("");
-  parts.push('If no questions are needed, return: `{"questions": []}`');
+  parts.push("If all clear to proceed:");
+  parts.push("```json");
+  parts.push('{ "ready": true, "decisions": [], "summary": "How we will proceed" }');
+  parts.push("```");
   parts.push("");
-  parts.push("**Important:** Output ONLY the JSON, no other text.");
+  parts.push("**Important:**");
+  parts.push("- Output ONLY the JSON, no other text.");
+  parts.push("- Every decision MUST have at least 2 concrete options with descriptions and tradeoffs.");
+  parts.push('- Every decision MUST include "ai" option as the last choice.');
   parts.push("");
   return parts.join("\n");
 }
@@ -1387,7 +1398,7 @@ var Orchestrator = class {
     };
   }
   /**
-   * Phase 2: Question generation
+   * Phase 2: Decision identification
    */
   responsePhaseClarify(state) {
     const prompt = buildClarifyPrompt({
@@ -1395,7 +1406,9 @@ var Orchestrator = class {
       step: state.currentStep,
       description: state.description,
       gatheredContext: state.exploreOutput ? JSON.stringify(state.exploreOutput) : "",
+      previousDecisions: state.decisionHistory,
       previousQA: state.qaHistory,
+      // Legacy support
       cwd: this.cwd
     });
     return {
@@ -1406,7 +1419,9 @@ var Orchestrator = class {
       description: state.description,
       prompt,
       gatheredContext: state.exploreOutput ? JSON.stringify(state.exploreOutput) : "",
+      previousDecisions: state.decisionHistory,
       previousQA: state.qaHistory,
+      // Legacy support
       onComplete: `npx spets orchestrate clarify-done ${state.taskId}`
     };
   }
@@ -1510,7 +1525,7 @@ ${issues}`;
     };
   }
   /**
-   * Checkpoint: Questions need answers
+   * Checkpoint: Decisions need user input
    */
   responseCheckpointClarify(state) {
     return {
@@ -1518,7 +1533,9 @@ ${issues}`;
       checkpoint: "clarify",
       taskId: state.taskId,
       step: state.currentStep,
+      decisions: state.decisions || [],
       questions: state.questions || [],
+      // Legacy support
       onComplete: `npx spets orchestrate clarified ${state.taskId} '<answers_json>'`
     };
   }
@@ -1614,11 +1631,34 @@ ${issues}`;
     return this.responsePhaseClarify(state);
   }
   /**
-   * Phase 2 complete: Questions generated (or none)
-   * If questions exist → checkpoint
-   * If no questions → move to execute phase with accumulated answers
+   * Phase 2 complete: Decisions identified (or ready to proceed)
+   * If decisions exist → checkpoint for user input
+   * If ready → move to execute phase
+   */
+  cmdClarifyDone(taskId, clarifyOutput) {
+    const state = this.loadState(taskId);
+    if (!state) {
+      return this.responseError(`No workflow found: ${taskId}`, taskId);
+    }
+    if (!clarifyOutput.ready && clarifyOutput.decisions.length > 0) {
+      state.decisions = clarifyOutput.decisions;
+      state.status = "clarify_pending";
+      state.phase = "clarify";
+      this.saveState(state);
+      return this.responseCheckpointClarify(state);
+    }
+    state.answers = state.decisionHistory?.map((entry) => entry.answer);
+    state.status = "phase_execute";
+    state.phase = "execute";
+    state.verifyAttempts = 0;
+    this.saveState(state);
+    return this.responsePhaseExecute(state);
+  }
+  /**
+   * Legacy support: Questions-based clarify done
+   * @deprecated Use decision-based cmdClarifyDone instead
    */
-  cmdClarifyDone(taskId, questions) {
+  cmdClarifyDoneLegacy(taskId, questions) {
     const state = this.loadState(taskId);
     if (!state) {
       return this.responseError(`No workflow found: ${taskId}`, taskId);
@@ -1638,22 +1678,22 @@ ${issues}`;
     return this.responsePhaseExecute(state);
   }
   /**
-   * Human answered questions
-   * - Store Q&A in history
-   * - Loop back to clarify phase - AI will judge if answers resolved the questions
+   * Human made decisions
+   * - Store decisions in history
+   * - Loop back to clarify phase to check for remaining decisions
    */
   cmdClarified(taskId, answers) {
     const state = this.loadState(taskId);
     if (!state) {
       return this.responseError(`No workflow found: ${taskId}`, taskId);
     }
-    const questions = state.questions || [];
-    const newEntries = questions.map((q, i) => ({
-      question: q,
-      answer: answers[i] || { questionId: q.id, answer: "" }
+    const decisions = state.decisions || [];
+    const newEntries = decisions.map((d, i) => ({
+      decision: d,
+      answer: answers[i] || { decisionId: d.id, selectedOptionId: "ai" }
     }));
-    state.qaHistory = [...state.qaHistory || [], ...newEntries];
-    state.questions = void 0;
+    state.decisionHistory = [...state.decisionHistory || [], ...newEntries];
+    state.decisions = void 0;
     state.answers = void 0;
     state.status = "phase_clarify";
     state.phase = "clarify";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spets",
-  "version": "0.1.67",
+  "version": "0.1.68",
   "description": "Spec Driven Development Execution Framework",
   "type": "module",
   "main": "dist/index.js",