fix(wasm): pass source to compile_ast — playground build drift from #541#581
Merged
Merged
Conversation
PR #541 (#407 runtime caret spans) added a third `src` parameter to compile_ast; web/eigs_wasm.c (built only by the Docs site workflow's emcc job) was not updated, so every main push since 2026-07-11 failed the WASM playground build. Pass the run's source, matching main.c, repl.c, and eigs_embed.c. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Restores the Docs site workflow (red on every main push since 2026-07-11): PR #541 added a third
srcparameter tocompile_astfor runtime caret spans, andweb/eigs_wasm.c— compiled only by the Docs site workflow's emcc job, so no local/PR gate caught it — still passed two arguments. Pass the run'ssource, matching the three in-tree call sites (main.c, repl.c, eigs_embed.c). Syntax-checked against src headers with a stub emscripten.h; the workflow's emcc build is the real gate.Found while verifying main CI after #580 merged.
🤖 Generated with Claude Code