source-academy / plugins / 29019816342

98%

feat/add-init-infra: 67%

push

github

Add breakpointSteps to the CSE snapshot protocol (#55)

* feat(cse-machine): carry breakpoint step indices over the __cse channel

Adds breakpointSteps: number[] to CseSnapshotMessage — a run-level array
of 0-based step indices where a breakpoint (e.g. Python's breakpoint())
sits on top of the control. CseMachinePlugin.sendSnapshots takes it as an
optional second argument; CseMachineHostPlugin.receiveSnapshots now
receives it as a second parameter.

Lets host apps wire up breakpoint-navigation controls for CSE-machine
evaluators, matching the stepper's existing redexNodeType contract.
Paired with a py-slang change that detects breakpoint() in the CSE
machine and populates this array (source-academy/py-slang issue #189).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* fix(cse-machine): address Gemini review on #55

Make CseSnapshotMessage.breakpointSteps optional for backward compatibility
with runners that predate this field, and defensively check
Array.isArray(message.breakpointSteps) in the web plugin before passing it
to receiveSnapshots, since messages cross a conduit channel boundary and
could come from an older or malformed runner.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* chore: formatting

---------

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

14 of 16 branches covered (87.5%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

38 of 38 relevant lines covered (100.0%)

13.34 hits per line