source-academy / py-slang / 31112109178

86%

push

github

Wire up stepper thumbnails for opaque values (DrRacket style) (#402)

* Wire up stepper thumbnails for opaque values (DrRacket style)

moduleInterop.ts now calls a module's RENDER_THUMBNAIL_SYMBOL hook (see
source-academy/modules docs/src/modules/5-advanced/conductor-interop/6-opaque-thumbnails.md)
when converting an opaque return value for the stepper, populating the
dataUrl field ast.ts's opaqueValue already reserved for it. syntaxProfile.ts's
Opaque template renders that thumbnail inline via the new image/unless
SyntaxTemplatePart kinds (source-academy/plugins, companion PR), falling back
to <label> text when no thumbnail is available.

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

* Address CodeRabbit review: catch throwing hook-property reads; split non-string test case

renderThumbnail now reads RENDER_THUMBNAIL_SYMBOL off the payload inside the
try block, so a throwing getter or Proxy trap degrades to the <label>
fallback like any other broken hook, rather than rejecting. The combined
"throws or resolves to a non-string" test is split into two so both branches
of renderThumbnail are actually exercised, matching the test names.

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

---------

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

4755 of 5967 branches covered (79.69%)

Branch coverage included in aggregate %.

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

10620 of 11904 relevant lines covered (89.21%)

165544.55 hits per line