Squashed 'src/resources/extension-subtrees/julia-engine/' changes from f591fc4..cb74379

Author: MichaelHatherly

src/resources/extension-subtrees/julia-engine/AGENTS.md

@@ -59,3 +59,7 @@ CI runs on all three platforms (Linux, macOS, Windows) against a pinned quarto-c
 3. Runs the full test suite
 
 When bumping `QUARTO_CLI_REV`, use the full commit hash annotated with the version tag for clarity (e.g. `abc123 # v1.9.35`).
+
+## Changelog
+
+Every PR with user-facing or otherwise meaningful changes must include an update to `CHANGELOG.md` (enforced by CI). Add entries under the `## Unreleased` section. Use the `skip-changelog` label to bypass the check for PRs that don't need an entry (e.g. internal cleanups, CI, or docs changes).

src/resources/extension-subtrees/julia-engine/_extensions/julia-engine/Project.toml

@@ -2,4 +2,4 @@
 QuartoNotebookRunner = "4c0109c6-14e9-4c88-93f0-2b974d3468f4"
 
 [compat]
-QuartoNotebookRunner = "=0.17.4"
+QuartoNotebookRunner = "=0.18.1"

src/resources/extension-subtrees/julia-engine/_extensions/julia-engine/_extension.yml

@@ -1,5 +1,5 @@
 title: Quarto Julia Engine Extension
-version: 0.1.0
+version: 0.2.0
 quarto-required: ">=1.9.0"
 contributes:
   engines:

src/resources/extension-subtrees/julia-engine/_extensions/julia-engine/julia-engine.js

@@ -631,6 +631,7 @@ var kFigFormat = "fig-format";
 var kFigPos = "fig-pos";
 var kIpynbProduceSourceNotebook = "produce-source-notebook";
 var kKeepHidden = "keep-hidden";
+var kKeepIpynb = "keep-ipynb";
 
 // src/julia-engine.ts
 var isWindows2 = Deno.build.os === "windows";
@@ -749,6 +750,11 @@ var juliaEngineDiscovery = {
           language: "julia"
         };
         const assets = quarto.jupyter.assets(options.target.input, options.format.pandoc.to);
+        if (options.format.execute[kKeepIpynb]) {
+          const stem = options.target.source.replace(/\.[^.]+$/, "");
+          const ipynbPath = stem + ".ipynb";
+          Deno.writeTextFileSync(ipynbPath, JSON.stringify(nb, null, 2));
+        }
         const result = await quarto.jupyter.toMarkdown(nb, {
           executeOptions: options,
           language: nb.metadata.kernelspec.language.toLowerCase(),

src/resources/extension-subtrees/julia-engine/src/constants.ts

@@ -13,3 +13,4 @@ export const kFigFormat = "fig-format";
 export const kFigPos = "fig-pos";
 export const kIpynbProduceSourceNotebook = "produce-source-notebook";
 export const kKeepHidden = "keep-hidden";
+export const kKeepIpynb = "keep-ipynb";

src/resources/extension-subtrees/julia-engine/src/julia-engine.ts

@@ -37,6 +37,7 @@ import {
   kIpynbProduceSourceNotebook,
   kJuliaEngine,
   kKeepHidden,
+  kKeepIpynb,
 } from "./constants.ts";
 
 // Platform detection
@@ -220,6 +221,14 @@ export const juliaEngineDiscovery: ExecutionEngineDiscovery = {
           options.format.pandoc.to,
         );
 
+        // Write notebook to file if keep-ipynb is set (must happen before
+        // toMarkdown which mutates nb in place)
+        if (options.format.execute[kKeepIpynb]) {
+          const stem = options.target.source.replace(/\.[^.]+$/, "");
+          const ipynbPath = stem + ".ipynb";
+          Deno.writeTextFileSync(ipynbPath, JSON.stringify(nb, null, 2));
+        }
+
         // NOTE: for perforance reasons the 'nb' is mutated in place
         // by jupyterToMarkdown (we don't want to make a copy of a
         // potentially very large notebook) so should not be relied

src/resources/extension-subtrees/julia-engine/tests/docs/julia-engine/.gitignore

@@ -4,3 +4,4 @@
 # Allow only source files
 !*/*.qmd
 !*/_quarto.yml
+!*/_metadata.yml

src/resources/extension-subtrees/julia-engine/tests/docs/julia-engine/engine-reordering/_metadata.yml

@@ -0,0 +1 @@
+engines: ["julia"]

src/resources/extension-subtrees/julia-engine/tests/docs/julia-engine/engine-reordering/_quarto.yml

@@ -44,6 +44,40 @@ Deno.test("source ranges with includes", async () => {
   try { Deno.removeSync(outputFile); } catch { /* ok */ }
 });
 
+Deno.test("keep-ipynb", async () => {
+  const dir = docs("julia-engine/keep-ipynb");
+  const input = join(dir, "keep-ipynb.qmd");
+  renderQmd(input, ["--to", "html"]);
+
+  const outputHtml = join(dir, "keep-ipynb.html");
+  assert(existsSync(outputHtml), `Output file ${outputHtml} should exist`);
+
+  const ipynbFile = join(dir, "keep-ipynb.ipynb");
+  assert(existsSync(ipynbFile), `keep-ipynb file ${ipynbFile} should exist`);
+
+  const content = await Deno.readTextFile(ipynbFile);
+  const nb = JSON.parse(content);
+  assert(nb.cells, "Notebook should have cells");
+  assert(nb.metadata, "Notebook should have metadata");
+
+  try { Deno.removeSync(outputHtml); } catch { /* ok */ }
+  try { Deno.removeSync(ipynbFile); } catch { /* ok */ }
+});
+
+Deno.test("no keep-ipynb by default", () => {
+  const dir = docs("julia-engine/keep-ipynb");
+  const input = join(dir, "no-keep-ipynb.qmd");
+  renderQmd(input, ["--to", "html"]);
+
+  const outputHtml = join(dir, "no-keep-ipynb.html");
+  assert(existsSync(outputHtml), `Output file ${outputHtml} should exist`);
+
+  const ipynbFile = join(dir, "no-keep-ipynb.ipynb");
+  assert(!existsSync(ipynbFile), `ipynb file ${ipynbFile} should NOT exist without keep-ipynb`);
+
+  try { Deno.removeSync(outputHtml); } catch { /* ok */ }
+});
+
 Deno.test("engine reordering", () => {
   const dir = docs("julia-engine/engine-reordering");
   renderQmd("notebook.qmd", ["--to", "html"], dir);