Fix kwargs extraction, plotly args, add Base.vect for array literals

- NamedTuple{(:x,:y)} is UnionAll not DataType — handle both
- plotly(divid, traces, layout) positional arg order fixed
- Base.vect([1,2,3]) → JS array literal [1, 2, 3]

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Dale-Black

src/compiler/codegen.jl

@@ -574,10 +574,13 @@ function _extract_kwargs(ctx::JSCompilationContext, kwargs_ssa)
     field_names = Symbol[]
     if type_ssa isa Core.SSAValue
         type_type = ctx.code_info.ssavaluetypes[type_ssa.id]
-        if type_type isa Core.Const && type_type.val isa DataType
+        if type_type isa Core.Const
             nt_type = type_type.val
-            if nt_type <: NamedTuple
+            if nt_type isa DataType && nt_type <: NamedTuple
                 field_names = collect(nt_type.parameters[1])
+            elseif nt_type isa UnionAll && nt_type <: NamedTuple
+                # NamedTuple{(:x,:y)} is UnionAll — field names in .body.parameters[1]
+                field_names = collect(nt_type.body.parameters[1])
             end
         end
     end
@@ -818,6 +821,11 @@ function compile_call(ctx::JSCompilationContext, expr::Expr)
                 end
             end
 
+            # Array literal: [1.0, 2.0, 3.0] → Base.vect(1.0, 2.0, 3.0) → [1.0, 2.0, 3.0]
+            if resolved_fn === Base.vect
+                return "[$(join(call_args, ", "))]"
+            end
+
             # Array creation: Float64[] → getindex(Float64) → []
             # Also handles array indexing: arr[i] → arr[i-1]
             if resolved_fn === Base.getindex

src/compiler/packages_plotly.jl

@@ -35,13 +35,29 @@ end
 # ─── Plot function: creates/updates a Plotly chart ───
 
 function _plotly_plot_compiler(ctx, kwargs, pos_args)
-    # pos_args[1] = traces (array), pos_args[2] = layout (optional)
-    # kwargs may include: divid (element ID)
-    traces_js = length(pos_args) >= 1 ? pos_args[1] : "[]"
-    layout_js = length(pos_args) >= 2 ? pos_args[2] : "{}"
+    # Positional: plotly(divid, traces, layout) OR plot(traces, layout)
+    # Detect by checking if first arg looks like a string (divid)
+    el_id = "\"therapy-plot\""
+    traces_js = "[]"
+    layout_js = "{}"
 
-    # Get target element ID from kwargs or default
-    el_id = get(kwargs, :divid, "\"therapy-plot\"")
+    if length(pos_args) >= 3
+        # plotly(divid, traces, layout)
+        el_id = pos_args[1]
+        traces_js = pos_args[2]
+        layout_js = pos_args[3]
+    elseif length(pos_args) >= 2
+        # plot(traces, layout) or plotly(divid, traces)
+        traces_js = pos_args[1]
+        layout_js = pos_args[2]
+    elseif length(pos_args) >= 1
+        traces_js = pos_args[1]
+    end
+
+    # Override with kwargs if present
+    if haskey(kwargs, :divid)
+        el_id = kwargs[:divid]
+    end
 
     return "(function() { var _el = document.getElementById($(el_id)); if (_el && typeof Plotly !== 'undefined') { Plotly.react(_el, $(traces_js), $(layout_js), {responsive: true, displayModeBar: false}); } else if (_el) { var _s = document.createElement('script'); _s.src = 'https://cdn.plot.ly/plotly-2.35.2.min.js'; _s.onload = function() { Plotly.newPlot(_el, $(traces_js), $(layout_js), {responsive: true, displayModeBar: false}); }; document.head.appendChild(_s); } }())"
 end