source-academy / py-slang / 28852666182

78%

push

github

feat(stepper): report cumulative print output per step (#203)

* feat(stepper): report cumulative print output per step

Each serialized step now carries the program's cumulative textual output
(everything `print` has written) up to and including that step, so the host can
render a running output panel that grows as the slider advances.

- builtins: `formatPrintOutput(args)` — CPython default formatting (`str()` of
  each arg, space-joined, trailing newline; `print()` is a blank line).
- reduce: `ReduceResult.output`; `contractCall` records a `print` call's text;
  threaded through `stepHead`/`reduceProgram`/`reduceBlock`. `print` still
  evaluates to `None`.
- getSteps: `drive()` accumulates output across contractions and stamps every
  step; a print's text first appears on its "Ran print" step and persists on
  every step after it. Emitted only when non-empty.

The `output` field is emitted via a small local type-widening until this package
depends on `@sourceacademy/common-stepper` >= 0.0.2 (which adds `output` to
`SerializedStepperStep`); it crosses the runner->host channel as plain JSON
regardless. Rendering the panel is the host (plugins) side.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* fix(stepper): address review feedback on cumulative-output PR

- Hoist a single `isTruthy` predicate into ast.ts, shared by reduce.ts
  (if/ternary conditions) and builtins.ts (`bool()`), replacing two
  independent copies of the same logic that could silently drift apart.
- getSteps.ts: replace the 7 call sites that each manually repeated
  `steps.push({..., output})` with a `pushStep(ast, markers)` closure
  that captures `output` by reference, so a future push site can't
  forget to stamp it.

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

---------

Co-authored-by: Shrey Jain <“shreyjain5132@email.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Co-authored-by: Martin Henz <henz@nus.edu.sg>

2559 of 3549 branches covered (72.1%)

Branch coverage included in aggregate %.

26 of 31 new or added lines in 4 files covered. (83.87%)

6042 of 7486 relevant lines covered (80.71%)

14253.65 hits per line