Skip to content

fix(wasm): pass source to compile_ast — playground build drift from #541#581

Merged
InauguralPhysicist merged 1 commit into
mainfrom
fix/wasm-compile-ast-src-drift
Jul 12, 2026
Merged

fix(wasm): pass source to compile_ast — playground build drift from #541#581
InauguralPhysicist merged 1 commit into
mainfrom
fix/wasm-compile-ast-src-drift

Conversation

@InauguralPhysicist

Copy link
Copy Markdown
Collaborator

Restores the Docs site workflow (red on every main push since 2026-07-11): PR #541 added a third src parameter to compile_ast for runtime caret spans, and web/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's source, 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



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>
@InauguralPhysicist InauguralPhysicist merged commit 9f72e31 into main Jul 12, 2026
17 checks passed
@InauguralPhysicist InauguralPhysicist deleted the fix/wasm-compile-ast-src-drift branch July 12, 2026 19:31
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