Fix #159, make parameters const (#164)

* Fix #159, make parameters `const`

* Resolve breakage on newer `quarto` versions

src/server.jl

@@ -872,7 +872,7 @@ function evaluate_params!(f, params::Dict{String})
     end
 
     exprs = map(collect(pairs(params))) do (key, value)
-        :(@eval getfield(Main, :Notebook) $(Symbol(key::String)) = $value)
+        :(@eval getfield(Main, :Notebook) const $(Symbol(key::String)) = $value)
     end
     expr = Expr(:block, exprs...)
     Malt.remote_eval_fetch(f.worker, expr)

test/Project.toml

@@ -6,3 +6,6 @@ NodeJS_18_jll = "c1e1d063-8311-5f52-a749-c7b05e91ae37"
 Sockets = "6462fe0b-24de-5631-8697-dd941f90decc"
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
 quarto_jll = "b7163347-bfae-5fd9-aba4-19f139889d78"
+
+[compat]
+quarto_jll = "= 1.4.549"

test/examples/parameters.qmd

@@ -41,3 +41,6 @@ g["a"]
 g["b"]
 ```
 
+```{julia}
+a = nothing
+```

test/testsets/non_standard_mimetypes.jl

@@ -41,7 +41,7 @@ include("../utilities/prelude.jl")
                     # (pdf). All we are doing here at the moment is ensuring quarto doesn't
                     # break on our notebook outputs.
                     if success(`$quarto_bin --version`)
-                        @test success(`$quarto_bin render $ipynb --to $format`)
+                        @test success(`$quarto_bin render $ipynb --no-execute --to $format`)
                     else
                         @error "quarto not found, skipping smoke test."
                     end

test/testsets/parameters.jl

@@ -11,6 +11,11 @@ test_example(joinpath(@__DIR__, "../examples/parameters.qmd")) do json
     @test cells[12]["outputs"][1]["text"] == "[1, 2, 3]"
     @test cells[14]["outputs"][1]["data"]["text/plain"] == "1"
     @test cells[16]["outputs"][1]["data"]["text/plain"] == "2"
+    @test cells[18]["outputs"][1]["output_type"] == "error"
+    @test contains(
+        cells[18]["outputs"][1]["traceback"][1],
+        "invalid redefinition of constant",
+    )
 end
 
 @testset "parameters via options" begin

test/utilities/prelude.jl

@@ -58,7 +58,7 @@ if !@isdefined(SCHEMA)
                 # (pdf). All we are doing here at the moment is ensuring quarto doesn't
                 # break on our notebook outputs.
                 if success(`$quarto_bin --version`)
-                    @test success(`$quarto_bin render $ipynb --to docx`)
+                    @test success(`$quarto_bin render $ipynb --no-execute --to docx`)
                 else
                     @error "quarto not found, skipping smoke test."
                 end