claude: Move executeResult functions from engine to core utilities

Move executeResultIncludes and executeResultEngineDependencies from
src/execute/jupyter/jupyter.ts to src/core/jupyter/jupyter.ts.

These functions are shared utilities used by multiple engines, not
engine-specific logic. Moving them to core breaks a circular dependency:
- src/core/quarto-api.ts was importing from src/execute/jupyter/jupyter.ts
- src/execute/jupyter/jupyter.ts was importing quartoAPI
- This created a cycle: core → execute → core

With this change, the dependency flow is now one-way:
- Engines (execute/) import from quartoAPI (core/)
- quartoAPI imports from core utilities (core/jupyter/)
- Core utilities have no dependency on engines

This is a prerequisite for implementing true dependency inversion via
the registry pattern, which will eliminate remaining circular dependencies.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: gordonwoodhull

src/core/jupyter/jupyter.ts

@@ -61,7 +61,10 @@ import {
   isCaptionableData,
   isDisplayData,
 } from "./display-data.ts";
-import { extractJupyterWidgetDependencies } from "./widgets.ts";
+import {
+  extractJupyterWidgetDependencies,
+  includesForJupyterWidgetDependencies,
+} from "./widgets.ts";
 import { removeAndPreserveHtml } from "./preserve.ts";
 import { pandocAsciify, pandocAutoIdentifier } from "../pandoc/pandoc-id.ts";
 import { Metadata } from "../../config/types.ts";
@@ -112,6 +115,8 @@ import {
   kFigCapLoc,
   kHtmlTableProcessing,
   kInclude,
+  kIncludeAfterBody,
+  kIncludeInHeader,
   kLayout,
   kLayoutAlign,
   kLayoutNcol,
@@ -142,6 +147,7 @@ import {
   JupyterOutputStream,
   JupyterToMarkdownOptions,
   JupyterToMarkdownResult,
+  JupyterWidgetDependencies,
 } from "./types.ts";
 import { figuresDir, inputFilesDir } from "../render.ts";
 import { lines, trimEmptyLines } from "../lib/text.ts";
@@ -153,8 +159,9 @@ import {
   removeIfEmptyDir,
 } from "../path.ts";
 import { convertToHtmlSpans, hasAnsiEscapeCodes } from "../ansi-colors.ts";
-import { kProjectType, EngineProjectContext } from "../../project/types.ts";
+import { EngineProjectContext, kProjectType } from "../../project/types.ts";
 import { mergeConfigs } from "../config.ts";
+import type { PandocIncludes } from "../../execute/types.ts";
 import { encodeBase64 } from "encoding/base64";
 import {
   isHtmlOutput,
@@ -2140,3 +2147,37 @@ function outputTypeCssClass(output_type: string) {
   }
   return `cell-output-${output_type}`;
 }
+
+// Engine helper functions for processing execute results
+// These are used by multiple engines (Jupyter, etc.) to handle widget dependencies
+export function executeResultIncludes(
+  tempDir: string,
+  widgetDependencies?: JupyterWidgetDependencies,
+): PandocIncludes | undefined {
+  if (widgetDependencies) {
+    const includes: PandocIncludes = {};
+    const includeFiles = includesForJupyterWidgetDependencies(
+      [widgetDependencies],
+      tempDir,
+    );
+    if (includeFiles.inHeader) {
+      includes[kIncludeInHeader] = [includeFiles.inHeader];
+    }
+    if (includeFiles.afterBody) {
+      includes[kIncludeAfterBody] = [includeFiles.afterBody];
+    }
+    return includes;
+  } else {
+    return undefined;
+  }
+}
+
+export function executeResultEngineDependencies(
+  widgetDependencies?: JupyterWidgetDependencies,
+): Array<unknown> | undefined {
+  if (widgetDependencies) {
+    return [widgetDependencies];
+  } else {
+    return undefined;
+  }
+}

src/core/quarto-api.ts

@@ -54,7 +54,7 @@ import { quartoDataDir } from "./appdirs.ts";
 import {
   executeResultEngineDependencies,
   executeResultIncludes,
-} from "../execute/jupyter/jupyter.ts";
+} from "./jupyter/jupyter.ts";
 import { completeMessage, withSpinner } from "./console.ts";
 import { checkRender } from "../command/check/check-render.ts";
 import type { RenderServiceWithLifetime } from "../command/render/types.ts";

src/execute/jupyter/jupyter.ts

@@ -13,7 +13,6 @@ import { error, info } from "../../deno_ral/log.ts";
 
 import * as ld from "../../core/lodash.ts";
 
-
 import {
   kBaseFormat,
   kExecuteDaemon,
@@ -186,7 +185,9 @@ title: "Title"
     if (caps) {
       checkCompleteMessage(kMessage + "OK");
       if (conf.jsonResult) {
-        jupyterJson["capabilities"] = await quarto.jupyter.capabilitiesJson(caps);
+        jupyterJson["capabilities"] = await quarto.jupyter.capabilitiesJson(
+          caps,
+        );
       } else {
         checkInfoMsg(await quarto.jupyter.capabilitiesMessage(caps, kIndent));
       }
@@ -212,7 +213,10 @@ title: "Title"
           checkInfoMsg("");
         }
       } else {
-        const installMessage = quarto.jupyter.installationMessage(caps, kIndent);
+        const installMessage = quarto.jupyter.installationMessage(
+          caps,
+          kIndent,
+        );
         checkInfoMsg(installMessage);
         checkInfoMsg("");
         jupyterJson["installed"] = false;
@@ -492,7 +496,9 @@ title: "Title"
         // that is coming from a non-qmd source
         const preserveCellMetadata =
           options.format.render[kNotebookPreserveCells] === true ||
-          (quarto.format.isHtmlDashboardOutput(options.format.identifier[kBaseFormat]) &&
+          (quarto.format.isHtmlDashboardOutput(
+            options.format.identifier[kBaseFormat],
+          ) &&
             !quarto.path.isQmdFile(options.target.source));
 
         // NOTE: for perforance reasons the 'nb' is mutated in place
@@ -511,7 +517,9 @@ title: "Title"
             toLatex: quarto.format.isLatexOutput(options.format.pandoc),
             toMarkdown: quarto.format.isMarkdownOutput(options.format),
             toIpynb: quarto.format.isIpynbOutput(options.format.pandoc),
-            toPresentation: quarto.format.isPresentationOutput(options.format.pandoc),
+            toPresentation: quarto.format.isPresentationOutput(
+              options.format.pandoc,
+            ),
             figFormat: options.format.execute[kFigFormat],
             figDpi: options.format.execute[kFigDpi],
             figPos: options.format.render[kFigPos],
@@ -747,7 +755,9 @@ async function ensureYamlKernelspec(
   const markdown = target.markdown.value;
   const yamlJupyter = quarto.markdownRegex.extractYaml(markdown)?.jupyter;
   if (yamlJupyter && typeof yamlJupyter !== "boolean") {
-    const [yamlKernelspec, _] = await quarto.jupyter.kernelspecFromMarkdown(markdown);
+    const [yamlKernelspec, _] = await quarto.jupyter.kernelspecFromMarkdown(
+      markdown,
+    );
     if (yamlKernelspec.name !== kernelspec?.name) {
       const nb = quarto.jupyter.fromJSON(Deno.readTextFileSync(target.source));
       nb.metadata.kernelspec = yamlKernelspec;
@@ -781,7 +791,11 @@ async function createNotebookforTarget(
   target: ExecutionTarget,
   project?: EngineProjectContext,
 ) {
-  const nb = await quarto.jupyter.quartoMdToJupyter(target.markdown.value, true, project);
+  const nb = await quarto.jupyter.quartoMdToJupyter(
+    target.markdown.value,
+    true,
+    project,
+  );
   Deno.writeTextFileSync(target.input, JSON.stringify(nb, null, 2));
   return nb;
 }
@@ -842,35 +856,3 @@ interface JupyterTargetData {
   transient: boolean;
   kernelspec: JupyterKernelspec;
 }
-
-export function executeResultIncludes(
-  tempDir: string,
-  widgetDependencies?: JupyterWidgetDependencies,
-): PandocIncludes | undefined {
-  if (widgetDependencies) {
-    const includes: PandocIncludes = {};
-    const includeFiles = quarto.jupyter.widgetDependencyIncludes(
-      [widgetDependencies],
-      tempDir,
-    );
-    if (includeFiles.inHeader) {
-      includes[kIncludeInHeader] = [includeFiles.inHeader];
-    }
-    if (includeFiles.afterBody) {
-      includes[kIncludeAfterBody] = [includeFiles.afterBody];
-    }
-    return includes;
-  } else {
-    return undefined;
-  }
-}
-
-export function executeResultEngineDependencies(
-  widgetDependencies?: JupyterWidgetDependencies,
-): Array<unknown> | undefined {
-  if (widgetDependencies) {
-    return [widgetDependencies];
-  } else {
-    return undefined;
-  }
-}