Update app.py

Polish missing field logic

app.py

@@ -79,9 +79,13 @@ Return only valid JSON with this exact shape:
 Rules:
 - Use only facts from this one message.
 - Do not invent details.
-- Put pickup or delivery time in pickup_time.
-- Put pickup place or delivery address in delivery_address.
+- Put dates and times in pickup_time, such as "tomorrow", "Saturday morning", or "Friday 5pm".
+- Put pickup places or delivery addresses in delivery_address, such as "farmers market".
+- "pickup at the farmers market" means delivery_address is "farmers market", not pickup_time.
+- "paid already" means payment_status is "paid".
+- "I can pay Venmo" means payment_status is "can pay Venmo".
 - If unknown, use an empty string.
+- Do not ask for flavor unless the product clearly needs a flavor choice.
 - missing_fields can only contain: quantity, flavor, pickup_time, delivery_address, payment_status.
 """
 
@@ -146,13 +150,30 @@ def missing_list(order):
     else:
         fields = [str(part).strip() for part in raw if str(part).strip()]
 
+    allowed = {"quantity", "flavor", "pickup_time", "delivery_address", "payment_status"}
+    fields = [field for field in fields if field in allowed]
+
     item = text_value(order.get("item"))
-    if item and not text_value(order.get("quantity")):
+    quantity = text_value(order.get("quantity"))
+    flavor = text_value(order.get("flavor"))
+    pickup_time = text_value(order.get("pickup_time"))
+    delivery_address = text_value(order.get("delivery_address"))
+    payment_status = text_value(order.get("payment_status"))
+
+    if item and not quantity:
         fields.append("quantity")
-    if item and not text_value(order.get("pickup_time")) and not text_value(order.get("delivery_address")):
-        fields.append("pickup_time")
-    if item and not text_value(order.get("payment_status")):
-        fields.append("payment_status")
+
+    if pickup_time:
+        fields = [field for field in fields if field != "pickup_time"]
+    if delivery_address:
+        fields = [field for field in fields if field != "delivery_address"]
+    if payment_status:
+        fields = [field for field in fields if field != "payment_status"]
+    if flavor:
+        fields = [field for field in fields if field != "flavor"]
+
+    if "flavor" in fields and item.lower() not in ["cake", "birthday cake", "cupcakes"]:
+        fields = [field for field in fields if field != "flavor"]
 
     return sorted(set(fields))