source-academy / plugins / 30344199234

97%

feat/add-init-infra: 67%

push

github

Add Data Visualizer plugin for Conductor (#71)

* Add common-data-visualizer wire-format package

Shared, language-agnostic protocol package for the data visualizer plugin pair (Conductor migration of draw_data), mirroring common-stepper's structure. Classification/cycle-detection logic is intentionally kept out of this package and out of per-language runners — it lives once in the host plugin (web-data-visualizer, added in a later PR), since pair/list/tree shapes are shared across languages unlike ASTs.

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

* Add runner-data-visualizer base plugin

BaseDataVisualizerRunnerPlugin<TValue>: the language-agnostic runner half of the data visualizer. Owns channel wiring, row accumulation across a run, and request/replay — mirrors BaseStepperRunnerPlugin's structure, but sendDrawing() fires once per draw_data(...) call (not once per run) since that's a builtin invoked from arbitrary points in student code, not something the evaluator drives top-down like stepping.

The single abstract method (toNode) is a purely mechanical value-to-node conversion; RefIdAllocator gives it row-scoped identity tracking (WeakMap-by-reference, same technique as py-slang's PyCseMachinePlugin) without requiring any cycle-detection or classification logic in the adapter itself — that stays host-side, once, shared across languages.

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

* Add data-visualizer classification logic

classify.ts: cycle-detection and pair/list/tree classification for one SerializedDataVisualizerNode, ported from the old pre-Conductor frontend's DataVisualizer (hasCycle/isBinaryTree/isGeneralTree/initializeTreeMetaData), generalized from a fixed 2-element-pair representation to the N-ary "array" node (needed for native Python lists, which aren't fixed at len... (continued)

46 of 50 branches covered (92.0%)

Branch coverage included in aggregate %.

26 of 26 new or added lines in 2 files covered. (100.0%)

129 of 129 relevant lines covered (100.0%)

5.03 hits per line