From 293c0d3b8b6e9668c2e2b82d5903ee6bdb1a7b26 Mon Sep 17 00:00:00 2001
From: Alan Fleming <>
Date: Sun, 11 Feb 2024 22:31:05 +1100
Subject: [PATCH] injectCode now checks for an error and returns
 content.payload

---
 src/widgets/utils.ts | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/widgets/utils.ts b/src/widgets/utils.ts
index 65b90b3a..e2bf1458 100644
--- a/src/widgets/utils.ts
+++ b/src/widgets/utils.ts
@@ -118,10 +118,20 @@ export async function injectCode({
   });
   const future = connection.requestExecute({
     code: code,
-    store_history: false
+    store_history: false,
+    stop_on_error: true,
+    silent: true,
+    allow_stdin: false
   });
   // TODO: Is there a better result to return?
-  return (await future.done) as any;
+  const result = (await future.done) as any;
+  if (result.content.status === 'ok') {
+    return result.content.payload;
+  } else {
+    throw new Error(
+      `Execution status = ${result.status} not 'ok' traceback=${result.content.traceback}`
+    );
+  }
 }
 
 /**