Fix pr #5644: Fix issue #5637: [Bug]: Incorrectly interpreting a Pyth…

…on error as a success message
All-Hands-AI · Dec 17, 2024 · 39ce6cd · 39ce6cd
1 parent 087d34c
commit 39ce6cd
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/frontend/src/state/chat-slice.ts b/frontend/src/state/chat-slice.ts
@@ -149,9 +149,8 @@ export const chatSlice = createSlice({
       } else if (observationID === "run_ipython") {
         // For IPython, we consider it successful if there's no error message
         const ipythonObs = observation.payload as IPythonObservation;
-        causeMessage.success = !ipythonObs.message
-          .toLowerCase()
-          .includes("error");
+        // Check for error in the message field which contains error information
+        causeMessage.success = !ipythonObs.message;
       }
 
       if (observationID === "run" || observationID === "run_ipython") {