jupyterlab-ipyflow 0.0.155 → 0.0.157

package/lib/index.js

@@ -10,7 +10,7 @@ const readyMakingInputClass = 'ready-making-input-cell';
 const linkedWaitingClass = 'linked-waiting';
 const linkedReadyMakerClass = 'linked-ready-maker';
 // ipyflow frontend state
-let dirtyCells = new Set();
+const dirtyCells = new Set();
 let waitingCells = new Set();
 let readyCells = new Set();
 let waiterLinks = {};
@@ -23,6 +23,7 @@ let orderIdxById = {};
 let cellPendingExecution = null;
 let lastExecutionMode = null;
 let isReactivelyExecuting = false;
+let isAltModeExecuting = false;
 let lastExecutionHighlights = null;
 let executedReactiveReadyCells = new Set();
 let newReadyCells = new Set();
@@ -42,19 +43,22 @@ const extension = {
             isVisible: () => true,
             isToggled: () => false,
             execute: () => {
-                if (notebooks.activeCell.model.type === 'code') {
+                if (!isAltModeExecuting && notebooks.activeCell.model.type === 'code') {
+                    isAltModeExecuting = true;
                     const session = notebooks.currentWidget.sessionContext;
                     if (session.isReady && notebooks.activeCell.model.type === 'code') {
-                        session.session.kernel.requestExecute({
+                        session.session.kernel
+                            .requestExecute({
                             code: '%flow toggle-reactivity-until-next-reset',
                             silent: true,
                             store_history: false,
-                        }).done.then(() => {
+                        })
+                            .done.then(() => {
                             CodeCell.execute(notebooks.activeCell, session);
                         });
                     }
                 }
-            }
+            },
         });
         app.commands.addKeyBinding({
             command: 'alt-mode-execute',
@@ -77,15 +81,20 @@ const extension = {
                 clearCellState(nbPanel.content);
                 activeCell = nbPanel.content.activeCell;
                 activeCellId = nbPanel.content.activeCell.model.id;
-                let commDisconnectHandler = () => { };
+                let commDisconnectHandler = function () {
+                    // do nothing if not connected.
+                };
                 if (session.session.kernel.name === IPYFLOW_KERNEL_NAME) {
                     commDisconnectHandler = connectToComm(session, nbPanel.content);
                 }
                 session.kernelChanged.connect((_, args) => {
                     clearCellState(nbPanel.content);
                     commDisconnectHandler();
-                    commDisconnectHandler = () => { };
-                    if (args.newValue !== null && args.newValue.name === IPYFLOW_KERNEL_NAME) {
+                    commDisconnectHandler = function () {
+                        // do nothing if not connected.
+                    };
+                    if (args.newValue !== null &&
+                        args.newValue.name === IPYFLOW_KERNEL_NAME) {
                         commDisconnectHandler = connectToComm(session, nbPanel.content);
                     }
                 });
@@ -99,7 +108,9 @@ const extension = {
                         session.ready.then(() => {
                             clearCellState(nbPanel.content);
                             commDisconnectHandler();
-                            commDisconnectHandler = () => { };
+                            commDisconnectHandler = function () {
+                                // do nothing if not connected.
+                            };
                             if (session.session.kernel.name === IPYFLOW_KERNEL_NAME) {
                                 commDisconnectHandler = connectToComm(session, nbPanel.content);
                             }
@@ -108,7 +119,7 @@ const extension = {
                 });
             });
         });
-    }
+    },
 };
 const getJpInputCollapser = (elem) => {
     if (elem === null || elem === undefined) {
@@ -262,7 +273,7 @@ const connectToComm = (session, notebook) => {
         const payload = {
             type: 'change_active_cell',
             active_cell_id: newActiveCell.id,
-            active_cell_order_idx: newActiveCellOrderIdx
+            active_cell_order_idx: newActiveCellOrderIdx,
         };
         comm.send(payload);
     };
@@ -286,7 +297,9 @@ const connectToComm = (session, notebook) => {
         }
         activeCell = cell;
         activeCellId = cell.model.id;
-        if (activeCell === null || activeCell.model === null || activeCell.model.type !== "code") {
+        if (activeCell === null ||
+            activeCell.model === null ||
+            activeCell.model.type !== 'code') {
             return;
         }
         activeCell.model.stateChanged.connect(onExecution);
@@ -305,10 +318,11 @@ const connectToComm = (session, notebook) => {
         {
             action: 'mouseover',
             update: 'add',
-        }, {
+        },
+        {
             action: 'mouseout',
             update: 'remove',
-        }
+        },
     ];
     const updateOneCellUI = (id) => {
         const model = cellModelsById[id];
@@ -334,13 +348,13 @@ const connectToComm = (session, notebook) => {
         if (lastExecutionMode === 'reactive') {
             return;
         }
-        if (waiterLinks.hasOwnProperty(id)) {
+        if (Object.prototype.hasOwnProperty.call(waiterLinks, id)) {
             actionUpdatePairs.forEach(({ action, update }) => {
                 addUnsafeCellInteraction(getJpInputCollapser(elem), waiterLinks[id], cellsById, getJpInputCollapser, action, update, waitingCells);
                 addUnsafeCellInteraction(getJpOutputCollapser(elem), waiterLinks[id], cellsById, getJpInputCollapser, action, update, waitingCells);
             });
         }
-        if (readyMakerLinks.hasOwnProperty(id)) {
+        if (Object.prototype.hasOwnProperty.call(readyMakerLinks, id)) {
             if (!waitingCells.has(id)) {
                 elem.classList.add(readyMakingClass);
                 elem.classList.add(readyClass);
@@ -388,7 +402,7 @@ const connectToComm = (session, notebook) => {
             readyMakerLinks = payload.ready_maker_links;
             cellPendingExecution = null;
             const exec_mode = payload.exec_mode;
-            isReactivelyExecuting = isReactivelyExecuting || (exec_mode === 'reactive');
+            isReactivelyExecuting = isReactivelyExecuting || exec_mode === 'reactive';
             const flow_order = payload.flow_order;
             const exec_schedule = payload.exec_schedule;
             lastExecutionMode = exec_mode;
@@ -400,18 +414,20 @@ const connectToComm = (session, notebook) => {
                 let lastExecutedCellIdSeen = false;
                 for (const cell of notebook.widgets) {
                     if (!lastExecutedCellIdSeen) {
-                        lastExecutedCellIdSeen = (cell.model.id == lastExecutedCellId);
+                        lastExecutedCellIdSeen = cell.model.id === lastExecutedCellId;
                         if (flow_order === 'in_order' || exec_schedule === 'strict') {
                             continue;
                         }
                     }
-                    if (cell.model.type !== 'code' || executedReactiveReadyCells.has(cell.model.id)) {
+                    if (cell.model.type !== 'code' ||
+                        executedReactiveReadyCells.has(cell.model.id)) {
                         continue;
                     }
                     if (!newReadyCells.has(cell.model.id)) {
                         continue;
                     }
-                    if (!forcedReactiveCells.has(cell.model.id) && exec_mode !== 'reactive') {
+                    if (!forcedReactiveCells.has(cell.model.id) &&
+                        exec_mode !== 'reactive') {
                         continue;
                     }
                     const codeCell = cell;
@@ -425,7 +441,8 @@ const connectToComm = (session, notebook) => {
                     else if (codeCell.model.executionCount == null) {
                         // pass
                     }
-                    else if (codeCell.model.executionCount < cellPendingExecution.model.executionCount) {
+                    else if (codeCell.model.executionCount <
+                        cellPendingExecution.model.executionCount) {
                         // otherwise, execute in order of earliest execution counter
                         cellPendingExecution = codeCell;
                     }
@@ -445,6 +462,7 @@ const connectToComm = (session, notebook) => {
                 executedReactiveReadyCells = new Set();
                 updateUI(notebook);
                 isReactivelyExecuting = false;
+                isAltModeExecuting = false;
             }
             else {
                 isReactivelyExecuting = true;

package/package.json

@@ -24,17 +24,19 @@
     "url": "https://github.com/ipyflow/ipyflow.git"
   },
   "scripts": {
-    "build": "jlpm run build:lib && jlpm run build:labextension:dev",
-    "build:prod": "jlpm run build:lib && jlpm run build:labextension",
+    "build": "npm run build:lib && npm run build:labextension",
+    "build:dev": "npm run build:lib && npm run build:labextension:dev",
     "build:labextension": "jupyter labextension build .",
     "build:labextension:dev": "jupyter labextension build --development True .",
     "build:lib": "tsc",
-    "clean": "jlpm run clean:lib",
+    "clean": "npm run clean:lib",
     "clean:lib": "rimraf lib tsconfig.tsbuildinfo",
     "eslint": "eslint . --ext .ts,.tsx --fix",
     "eslint:check": "eslint . --ext .ts,.tsx",
+    "format": "prettier --write 'src/*.ts'",
+    "lint": "prettier --check 'src/*.ts'",
     "install:extension": "jupyter labextension develop --overwrite .",
-    "prepare": "jlpm run clean && jlpm run build:prod",
+    "prepare": "npm run clean && npm run build",
     "watch": "run-p watch:src watch:labextension",
     "watch:src": "tsc -w",
     "watch:labextension": "jupyter labextension watch ."
@@ -64,5 +66,5 @@
     "extension": true,
     "outputDir": "../../core/ipyflow/resources/labextension/"
   },
-  "version": "0.0.155"
+  "version": "0.0.157"
 }