source-academy / py-slang / 30367171085 / 1

86%

main: 86%

push

github

Add draw_data builtin (Data Visualizer support) (#365)

* Wire data-visualizer Context plumbing and Python adapter

Adds context.dataVisualizer (typed against the external BaseDataVisualizerRunnerPlugin<Value> abstract base, not the local concrete subclass — matches how conductor/evaluator are already typed by interface). PyCseEvaluatorBase registers PythonDataVisualizerRunnerPlugin and loads its host bundle in the constructor, guarded by variant >= 2 (draw_data doesn't exist as a builtin below that), next to the existing csePlugin registration. evaluateChunk stashes the plugin onto context alongside conductor/evaluator, and resetRun() is called in the same per-run reset block that clears breakpointSteps/changepointSteps.

toDataVisualizerNode.ts is the actual thin adapter: dispatches on Value.type, recurses into list elements, reuses toPythonString (the same stringifier print/repr already use) for leaf/function display values. No cycle-detection or classification here — that's entirely host-side per the shared-logic decision.

Note: @sourceacademy/common-data-visualizer and @sourceacademy/runner-data-visualizer aren't published yet (this PR's sibling work in the plugins repo), so yarn.lock won't actually resolve them until that lands — same bootstrapping situation every cross-repo Conductor dependency in this epic goes through before its own upstream PR merges. Verified locally via a temporary portal: link (not committed).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011JmkCBZfioraGKf1tR6K3n

* Add draw_data builtin

DataVisualizerBuiltins.draw_data: @Validate(2, null, ...) matching python_2_specs.md's own documented signature (value1, value2, *values — diverges from js-slang's minimum of 1), calls context.dataVisualizer?.sendDrawing(args) and returns None. Registered as its own GroupName.DATA_VISUALIZER group (mirrors how list/pairmutator/stream are each their own group despite all being lis... (continued)

4428 of 5532 branches covered (80.04%)

Branch coverage included in aggregate %.

9590 of 10738 relevant lines covered (89.31%)

179915.99 hits per line