Skip to content

Fix debugger template corpus snapshots and preview regressions - #3

Merged
synalysis merged 3 commits into
mainfrom
debugger-template-corpus-snapshots
Jun 2, 2026
Merged

Fix debugger template corpus snapshots and preview regressions#3
synalysis merged 3 commits into
mainfrom
debugger-template-corpus-snapshots

Conversation

@synalysis

Copy link
Copy Markdown
Owner

Summary

  • Refresh all 30 debugger_template_corpus golden snapshots for current runtime/debugger behavior (runtime_view_output_model_sha256, preview pipeline, new watch-demo-drawing-showcase fixture).
  • Fix elmx regressions that broke preview on several templates:
    • String.toInt returns Maybe (not Result) so Maybe.withDefault works in updates (games with storage best-score load).
    • Curried emission for let-bound multi-arg helpers and nested lambdas (companion demo views).
  • Stabilize corpus comparisons: atom/string normalization in render trees, timeline RandomGenerated placeholder, deterministic corpus random via :corpus_fixed_random_int, float rounding in preview ops.

Test plan

  • mix test test/ide/mcp/debugger_template_corpus_test.exs --only template_corpus --max-cases 1 (31/31, run twice)
  • mix test in elmx (307 tests)
  • CI Debugger Strict Coverage on this PR

Made with Cursor

synalysis and others added 3 commits June 3, 2026 01:07
String.toInt now returns Maybe, curried let-bound calls emit correctly, and
corpus bootstrap uses deterministic random plus stable snapshot normalization
so all 30 template fixtures match current runtime preview behavior.

Co-authored-by: Cursor <cursoragent@cursor.com>
Drop revision and other machine-specific watch_model fields from snapshot
comparison, keep corpus random override visible to async runtime tasks, and
run template corpus ExUnit steps serially in CI.

Co-authored-by: Cursor <cursoragent@cursor.com>
SHA256 fields depend on compile artifact layout and differ between CI and
local runs even when preview output is equivalent.

Co-authored-by: Cursor <cursoragent@cursor.com>
@synalysis
synalysis merged commit 2467e2f into main Jun 2, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant