🐛 Guard outputs and fix embed children (#202)

See https://github.com/orgs/executablebooks/discussions/1062#discussioncomment-6682400

.changeset/lazy-crabs-walk.md

@@ -0,0 +1,5 @@
+---
+'@myst-theme/jupyter': patch
+---
+
+Fix embed node to render all children.

.changeset/tiny-ghosts-notice.md

@@ -0,0 +1,5 @@
+---
+'@myst-theme/jupyter': patch
+---
+
+Guard output if null or undefined

packages/jupyter/src/embed.tsx

@@ -1,5 +1,4 @@
 import { type GenericNode } from 'myst-common';
-import { JupyterOutput } from './output';
 import { OutputDecoration } from './decoration';
 import { MyST } from 'myst-to-react';
 
@@ -13,12 +12,7 @@ export function Embed({ node }: { node: GenericNode }) {
       title={node.source?.title}
       url={node.source?.url}
     >
-      <JupyterOutput
-        outputId={output.id}
-        identifier={output.identifier}
-        align={node.align}
-        data={node.data}
-      />
+      <MyST ast={node.children} />
     </OutputDecoration>
   );
 }

packages/jupyter/src/output.tsx

@@ -24,6 +24,7 @@ export function allOutputsAreSafe(
   directOutputTypes: Set<string>,
   directMimeTypes: Set<string>,
 ) {
+  if (!outputs || outputs.length === 0) return true;
   return outputs.reduce((flag, output) => {
     if (directOutputTypes.has(output.output_type)) return flag && true;
     const data = (output as MinifiedMimeOutput).data;

packages/jupyter/src/safe.tsx

@@ -72,6 +72,7 @@ function SafeOutput({ output }: { output: MinifiedOutput }) {
 }
 
 export function SafeOutputs({ keyStub, outputs }: { keyStub: string; outputs: MinifiedOutput[] }) {
+  if (!outputs) return null;
   // TODO better key - add keys during content creation?
   const components = outputs.map((output, idx) => (
     <SafeOutput key={`${keyStub}-${idx}`} output={output} />