Skip to content

fix(vm): wire B64Dec into tree_bridge_returns_result#579

Merged
danieljohnmorris merged 1 commit into
mainfrom
fix/b64dec-bridge-list
May 21, 2026
Merged

fix(vm): wire B64Dec into tree_bridge_returns_result#579
danieljohnmorris merged 1 commit into
mainfrom
fix/b64dec-bridge-list

Conversation

@danieljohnmorris
Copy link
Copy Markdown
Collaborator

Summary

One-line fix: #560 added the standard base64 cluster but missed wiring B64Dec into the returns_result list in src/vm/mod.rs. b64-dec! (with bang) tripped the debug_assert!(tree_bridge_returns_result(builtin)) at line 1887 because the bridge thought it returned a plain value.

Why this matters

examples/crypto-primitives.@ has been failing on main since #560 merged. Every PR's coverage job has inherited the failure. This one-line edit unblocks the whole queue.

Verification

  • Build + clippy clean.
  • cargo test --release --features cranelift --test examples_engines passes (was panicking).

#560 added the standard base64 cluster but missed wiring B64Dec into
the returns-Result list. b64-dec! tripped the debug_assert at
src/vm/mod.rs:1887 because the bridge thought it returned a plain
value. crypto-primitives.@ has been red on main since #560 landed.
@danieljohnmorris danieljohnmorris merged commit 59411e1 into main May 21, 2026
1 of 4 checks passed
@danieljohnmorris danieljohnmorris deleted the fix/b64dec-bridge-list branch May 21, 2026 20:01
@codecov
Copy link
Copy Markdown

codecov Bot commented May 21, 2026

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
7544 1 7543 0
View the top 1 failed test(s) by shortest run time
ilo::skill_md::body_is_thin_bootstrap
Stack Traces | 0.004s run time
thread 'body_is_thin_bootstrap' (64405) panicked at tests/skill_md.rs:254:5:
SKILL.md body is 10925 bytes; bootstrap shape should stay well under 8 KB
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

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