spets 0.1.76 → 0.1.78

package/dist/index.js

@@ -272,9 +272,9 @@ type="checkpoint", checkpoint="clarify":
   - GOTO step 2
 
 type="checkpoint", checkpoint="approve":
-  - READ \`specPath\`, summarize to user
-  - ASK user: approve / revise / reject / stop
-  - RUN matching \`onComplete\`
+  - READ \`specPath\`, summarize key points to user
+  - ASK user using AskUserQuestion with options: Approve / Revise / Reject / Stop
+  - RUN matching \`onComplete\` based on user's choice
   - GOTO step 2
 
 type="complete" or "error":
@@ -320,9 +320,9 @@ type="checkpoint", checkpoint="clarify":
   - GOTO step 2
 
 type="checkpoint", checkpoint="approve":
-  - READ \`specPath\`, summarize to user
-  - ASK user: approve / revise / reject / stop
-  - RUN matching \`onComplete\`
+  - READ \`specPath\`, summarize key points to user
+  - ASK user using AskUserQuestion with options: Approve / Revise / Reject / Stop
+  - RUN matching \`onComplete\` based on user's choice
   - GOTO step 2
 
 type="complete" or "error":
@@ -363,9 +363,9 @@ type="checkpoint", checkpoint="clarify":
   - GOTO step 2
 
 type="checkpoint", checkpoint="approve":
-  - READ \`specPath\`, summarize to user
-  - ASK user: approve / revise / reject / stop
-  - RUN matching \`onComplete\`
+  - READ \`specPath\`, summarize key points to user
+  - ASK user using AskUserQuestion with options: Approve / Revise / Reject / Stop
+  - RUN matching \`onComplete\` based on user's choice
   - GOTO step 2
 
 type="complete" or "error":
@@ -945,36 +945,55 @@ function buildClarifyPrompt(params) {
   parts.push("");
   const hasPreviousDecisions = params.previousDecisions && params.previousDecisions.length > 0;
   if (hasPreviousDecisions) {
-    parts.push("## Previous Decisions to Review");
+    parts.push("## Previous Decisions (All Resolved)");
     parts.push("");
-    parts.push("The user has responded to previous decisions. Review each one:");
+    parts.push("The user has responded to all previous decisions. Each response below is **already resolved** unless the user explicitly asked for clarification.");
     parts.push("");
     for (const entry of params.previousDecisions) {
       const selectedOption = entry.decision.options.find((o) => o.id === entry.answer.selectedOptionId);
       let label;
+      let description = "";
       if (entry.answer.selectedOptionId === "ai") {
-        label = `Selected: AI recommendation`;
+        const aiOption = entry.decision.options.find((o) => o.id === "ai");
+        label = "AI recommendation";
+        if (aiOption?.recommendation) {
+          const recommended = entry.decision.options.find((o) => o.id === aiOption.recommendation);
+          description = recommended ? ` (${recommended.label})` : "";
+        }
       } else if (selectedOption) {
-        label = `Selected: ${selectedOption.label}`;
+        label = selectedOption.label;
+        description = selectedOption.description ? ` - ${selectedOption.description}` : "";
       } else {
-        label = `User's response: "${entry.answer.selectedOptionId}"`;
+        label = entry.answer.selectedOptionId;
+      }
+      const hasCustomInput = entry.answer.customInput && entry.answer.customInput.trim().length > 0;
+      parts.push(`### ${entry.decision.decision} [RESOLVED]`);
+      parts.push(`**Selected:** ${label}${description}`);
+      if (hasCustomInput) {
+        parts.push(`**User comment:** "${entry.answer.customInput}"`);
       }
-      parts.push(`**Q: ${entry.decision.decision}**`);
-      parts.push(`\u2192 ${label}`);
       parts.push("");
     }
     parts.push("## Your Task");
     parts.push("");
-    parts.push("Review each response above and determine:");
+    parts.push("All decisions above are marked [RESOLVED] by default. Analyze each user response semantically:");
     parts.push("");
-    parts.push('1. **RESOLVED**: User gave a clear preference (selected option, said "good/great/fine/ok", or gave specific direction)');
-    parts.push(`2. **NEEDS CLARIFICATION**: User asked a question ("what does X mean?", "can you explain?", "I don't understand")`);
+    parts.push("**RESOLVED (proceed with this choice):**");
+    parts.push("- User selected an option (any option selection = resolved)");
+    parts.push('- Approval expressions: "ok", "okay", "good", "great", "fine", "sure", "yes", "\uB3D9\uC758", "\uC88B\uC544", "\uADF8\uB807\uAC8C \uD574", "\u3147\u3147"');
+    parts.push("- Short affirmations or no comment at all");
     parts.push("");
-    parts.push("Then output:");
-    parts.push('- If ALL resolved \u2192 `{"ready": true, "decisions": [], "summary": "..."}`');
-    parts.push("- If ANY needs clarification \u2192 Re-ask ONLY those specific questions with better explanations");
+    parts.push("**NEEDS CLARIFICATION (re-ask with better explanation):**");
+    parts.push('- Explicit questions: "\uBB50\uC57C?", "what does X mean?", "\uBB34\uC2A8 \uB73B\uC774\uC57C?", "can you explain?"');
+    parts.push(`- Confusion expressions: "\uC774\uD574 \uC548 \uB428", "I don't understand", "\uC124\uBA85\uD574\uC918", "\uC798 \uBAA8\uB974\uACA0\uC5B4"`);
+    parts.push('- Requests for more info: "\uB354 \uC54C\uB824\uC918", "tell me more", "\uC608\uC2DC \uC788\uC5B4?"');
     parts.push("");
-    parts.push("**IMPORTANT**: Do NOT ask new questions about the same topics. Only re-ask if user explicitly asked for clarification.");
+    parts.push("**CRITICAL RULE: When in doubt, treat as RESOLVED.**");
+    parts.push("Do NOT re-ask unless user EXPLICITLY expressed confusion or asked a question.");
+    parts.push("");
+    parts.push("Then output:");
+    parts.push('- If ALL resolved \u2192 `{"ready": true, "decisions": [], "summary": "Proceeding with: [list chosen options]"}`');
+    parts.push("- If ANY needs clarification \u2192 Re-ask ONLY those with clearer explanations and examples");
     parts.push("");
   } else {
     parts.push("## Your Task");

package/package.json

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