improve error messaging

Author: sethbern

bases/rsptx/web2py_server/applications/runestone/controllers/peer.py

@@ -1079,7 +1079,7 @@ def _call_openai(messages):
         "temperature": 0.4,
         "max_tokens": 300,
     }
-    resp = requests.post(url, headers=headers, data=json.dumps(payload), timeout=30)
+    resp = requests.post(url, headers=headers, json=payload, timeout=30)
     logger.warning(f"PEER LLM CALL | provider=openai-course-token | model={model}")
     resp.raise_for_status()
     data = resp.json()

bases/rsptx/web2py_server/applications/runestone/views/peer/peer_async.html

@@ -222,8 +222,6 @@ <h3>Congratulations, you have completed this assignment!</h3>
     };
   })();
 </script>
-
-<!-- <script src="/runestone/static/js/llmtest.js"></script> -->
 
 <script>
 (function () {
@@ -354,21 +352,27 @@ <h3>Congratulations, you have completed this assignment!</h3>
     ];
 
 
-    const resp = await fetch("/runestone/peer/get_async_llm_reflection", {
-      method: "POST",
-      headers: { "Content-Type": "application/json" },
-      body: JSON.stringify({
-        div_id: currentQuestion,
-        selected_answer: selected,
-        messages: window._llmMessages
-      })
-    });
+    let resp;
+    try {
+      resp = await fetchWithRetry("/runestone/peer/get_async_llm_reflection", {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({
+          div_id: currentQuestion,
+          selected_answer: selected,
+          messages: window._llmMessages
+        })
+      });
+    } catch (e) {
+      chat.innerHTML = "<p><em>LLM error. Please try again.</em></p>";
+      return;
+    }
 
     const data = await resp.json();
     chat.innerHTML = "";
 
     if (!data.ok) {
-      chat.innerHTML = "<p><em>Error talking to peer.</em></p>";
+      chat.innerHTML = "<p><em>LLM error. Please try again.</em></p>";
       return;
     }
 
@@ -461,23 +465,40 @@ <h3>Congratulations, you have completed this assignment!</h3>
       content: msg
     });
 
-    const resp = await fetch("/runestone/peer/get_async_llm_reflection", {
-      method: "POST",
-      headers: { "Content-Type": "application/json" },
-      body: JSON.stringify({
-        div_id: currentQuestion,
-        selected_answer: selected,
-        messages: window._llmMessages
-      })
-    });
+    btn.disabled = true;
+    input.disabled = true;
+
+    await new Promise(resolve => setTimeout(resolve, 1000));
+
+    let resp;
+    try {
+      resp = await fetchWithRetry("/runestone/peer/get_async_llm_reflection", {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({
+          div_id: currentQuestion,
+          selected_answer: selected,
+          messages: window._llmMessages
+        })
+      });
+    } catch (e) {
+      appendMsg("assistant", "LLM error. Please try again.");
+      btn.disabled = false;
+      input.disabled = false;
+      return;
+    }
 
     const data = await resp.json();
     if (!data.ok) {
-      appendMsg("assistant", "(error talking to peer)");
+      appendMsg("assistant", "LLM error. Please try again.");
+      btn.disabled = false;
+      input.disabled = false;
       return;
     }
 
     appendMsg("assistant", data.reply);
+    btn.disabled = false;
+    input.disabled = false;
     if (typeof logPeerEvent === "function") {
       window._llmTurnIndex = (window._llmTurnIndex || 0) + 1;
       logPeerEvent({