From 9b904da0468c72eae6c2fea5f01998e520672a8f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?T=C3=82CHES?= <afromanguy@me.com>
Date: Sat, 14 Mar 2026 21:23:14 -0600
Subject: [PATCH] fix: add empty-answer validation guards to discuss-phase
 (closes #912) (#1048)

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 get-shit-done/workflows/discuss-phase.md | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/get-shit-done/workflows/discuss-phase.md b/get-shit-done/workflows/discuss-phase.md
index 20be5ece2..35a4513c2 100644
--- a/get-shit-done/workflows/discuss-phase.md
+++ b/get-shit-done/workflows/discuss-phase.md
@@ -105,6 +105,13 @@ Phase: "API documentation"
 - Scope (roadmap defines this)
 </gray_area_identification>
 
+<answer_validation>
+**IMPORTANT: Answer validation** — After every AskUserQuestion call, check if the response is empty or whitespace-only. If so:
+1. Retry the question once with the same parameters
+2. If still empty, present the options as a plain-text numbered list and ask the user to type their choice number
+Never proceed with an empty answer.
+</answer_validation>
+
 <process>
 
 **Express path available:** If you already have a PRD or acceptance criteria document, use `/gsd:plan-phase {phase} --prd path/to/prd.md` to skip this discussion and go straight to planning.