fix: align isCodeBlock with old PR after rebase

Author: fwkoch

packages/myst-execute/src/execute.ts

@@ -9,7 +9,7 @@ import assert from 'node:assert';
 import type { CodeBlock, ExecutionResult, ExecutableNode } from './types.js';
 import { executeCodeCell, evaluateInlineExpression } from './kernel.js';
 import {
-  isCellBlock,
+  isCodeBlock,
   codeBlockRaisesException,
   codeBlockSkipsExecution,
   isInlineExpression,
@@ -35,7 +35,7 @@ export async function computeExecutableNodes(
 
   const results: ExecutionResult[] = [];
   for (const matchedNode of nodes) {
-    if (isCellBlock(matchedNode)) {
+    if (isCodeBlock(matchedNode)) {
       // Pull out code to execute
       const code = select('code', matchedNode) as Code;
       const { status, outputs } = await executeCodeCell(kernel, code.value);
@@ -102,7 +102,7 @@ export function applyComputedOutputsToNodes(
     // Pull out the result for this node
     const thisResult = computedResult.shift();
 
-    if (isCellBlock(matchedNode)) {
+    if (isCodeBlock(matchedNode)) {
       const rawOutputData = (thisResult as IOutput[]) ?? [];
       // Pull out outputs to set data
       const outputs = select('outputs', matchedNode) as Outputs;
@@ -132,6 +132,6 @@ export function getExecutableNodes(tree: GenericParent) {
       )[]
     )
       // Filter out nodes that skip execution
-      .filter((node) => !(isCellBlock(node) && codeBlockSkipsExecution(node)))
+      .filter((node) => !(isCodeBlock(node) && codeBlockSkipsExecution(node)))
   );
 }

packages/myst-execute/src/transform.ts

@@ -11,7 +11,7 @@ import type { Plugin } from 'unified';
 import { createHash } from 'node:crypto';
 import type { ICache } from './cache.js';
 import { createKernelConnection } from './kernel.js';
-import { isCellBlock, isInlineExpression, codeBlockRaisesException } from './utils.js';
+import { isCodeBlock, isInlineExpression, codeBlockRaisesException } from './utils.js';
 import type { ExecutableNode, ExecutionResult } from './types.js';
 import {
   getExecutableNodes,
@@ -35,7 +35,7 @@ function buildCacheKey(
     raisesException: boolean;
   }[] = [];
   for (const node of nodes) {
-    if (isCellBlock(node)) {
+    if (isCodeBlock(node)) {
       hashableItems.push({
         kind: node.type,
         content: (select('code', node) as Code).value,

packages/myst-execute/src/utils.ts

@@ -1,15 +1,14 @@
-import { select } from 'unist-util-select';
 import type { InlineExpression } from 'myst-spec-ext';
 import type { GenericNode } from 'myst-common';
-import { NotebookCellTags } from 'myst-common';
+import { NotebookCell, NotebookCellTags } from 'myst-common';
 import type { CodeBlock } from './types.js';
 /**
  * Return true if the given node is a block over a code node and output node
  *
  * @param node node to test
  */
-export function isCellBlock(node: GenericNode): node is CodeBlock {
-  return node.type === 'block' && select('code', node) !== null && select('outputs', node) !== null;
+export function isCodeBlock(node: GenericNode): node is CodeBlock {
+  return node.type === 'block' && node.kind === NotebookCell.code;
 }
 
 /**