Add kernel error log handler (#119)

* Add kernel log handler

* Pin Lab in UI test

Author: trungleduc

.github/workflows/build.yml

@@ -108,7 +108,7 @@ jobs:
       - name: Install the extension
         run: |
           set -eux
-          python -m pip install "jupyterlab>=4.0.0,<5" glue_jupyterlab*.whl
+          python -m pip install "jupyterlab==4.0.2" glue_jupyterlab*.whl
 
       - name: Install dependencies
         working-directory: ui-tests

src/document/plugin.ts

@@ -117,6 +117,7 @@ export const newFilePlugin: JupyterFrontEndPlugin<void> = {
     commandPalette?: ICommandPalette
   ) => {
     const { commands } = app;
+
     commands.addCommand(CommandIDs.createNew, {
       label: args => 'New Glue Session',
       caption: 'Create a new Glue Session',

src/leftPanel/data/dataPanel.ts

@@ -8,6 +8,7 @@ import { IDocumentManager } from '@jupyterlab/docmanager';
 import { DatasetsWidget } from './datasetsWidget';
 import { SubsetsWidget } from './subsetsWidget';
 import { IControlPanelModel } from '../../types';
+import { logKernelError } from '../../tools';
 
 export class DataPanel extends SidePanel {
   constructor(options: {
@@ -53,6 +54,7 @@ export class DataPanel extends SidePanel {
             `;
 
             const future = kernel.requestExecute({ code }, false);
+            future.onReply = logKernelError;
             await future.done;
           }
         }

src/leftPanel/plugin.ts

@@ -1,6 +1,6 @@
 import { ControlPanelModel } from './model';
 import { ControlPanelWidget } from './widget';
-import { glueIcon } from '../tools';
+import { glueIcon, logKernelError } from '../tools';
 import {
   ILayoutRestorer,
   JupyterFrontEnd,
@@ -243,6 +243,7 @@ function addCommands(
       `;
 
       const future = kernel.requestExecute({ code }, false);
+      future.onReply = logKernelError;
       await future.done;
     }
   });

src/tools.ts

@@ -4,6 +4,7 @@ import glueIconStr from '../style/icons/glue-icon.svg';
 import { DocumentRegistry } from '@jupyterlab/docregistry';
 import { IRenderMimeRegistry } from '@jupyterlab/rendermime';
 import { Signal } from '@lumino/signaling';
+import { KernelMessage } from '@jupyterlab/services';
 
 export const glueIcon = new LabIcon({
   name: 'gluelab:glue-icon',
@@ -33,3 +34,16 @@ export function mockNotebook(
   };
   return panel;
 }
+
+/**
+ * Log the kernel error message to the JS console.
+ *
+ * @param {KernelMessage.IExecuteReplyMsg} msg
+ */
+export const logKernelError = (msg: KernelMessage.IExecuteReplyMsg): void => {
+  const { content } = msg;
+  if (content.status === 'error') {
+    const { ename, evalue, traceback } = content;
+    console.error('[Kernel Execution Error]', { ename, evalue, traceback });
+  }
+};

src/viewPanel/sessionWidget.ts

@@ -13,7 +13,7 @@ import { CommandIDs } from '../commands';
 import { HTabPanel } from '../common/tabPanel';
 import { GlueSessionModel } from '../document/docModel';
 import { LinkEditor } from '../linkPanel/linkEditor';
-import { mockNotebook } from '../tools';
+import { logKernelError, mockNotebook } from '../tools';
 import { DATASET_MIME, IDict, IGlueSessionSharedModel } from '../types';
 import { TabView } from './tabView';
 
@@ -163,8 +163,8 @@ export class SessionWidget extends BoxPanel {
     from glue_jupyterlab.glue_session import SharedGlueSession
     GLUE_SESSION = SharedGlueSession("${this._context.localPath}")
     `;
-
     const future = kernel.requestExecute({ code }, false);
+    future.onReply = logKernelError;
     await future.done;
     this._pythonSessionCreated.resolve();