source-academy / py-slang / 30065565222

86%

push

github

Fix Pynter WASM debug trailer leaking into the REPL; surface fault names properly (#344)

* Stop leaking Pynter WASM's debug trailer into the REPL, surface fault names properly

siwasm_run (Pynter's devices/wasm/wasm/lib.c) printf's a debugging trailer to
stdout on every run, meant for its standalone WASM demo. PyPvmlPynterEvaluator
wired that stdout straight to conductor.sendOutput, so users saw a stray
"Program exited with result type undefined: undefined" line after every
program, and on a fault, the fault name only surfaced disguised as program
output while sendError reported a useless "Unknown return type: 0" (faults
leave the WASM return pointer zeroed, which is the only other place the
result is read from).

The print callback now drops the redundant success trailer, and captures the
fault trailer's fault name instead of discarding it, using it to replace the
generic throw with a proper "Pynter fault: <name>" error. No changes to
Pynter itself — same fix as documented in source-academy/pynter's README
(new fault-code table) and its WASM npm package README (fault-reporting
guidance for browser embedders).

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

* Fix prettier formatting in PyPvmlPynterEvaluator.test.ts

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

---------

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

4405 of 5504 branches covered (80.03%)

Branch coverage included in aggregate %.

9 of 15 new or added lines in 1 file covered. (60.0%)

9544 of 10690 relevant lines covered (89.28%)

178377.0 hits per line