Skip to content

feat: bulk_memory capability testdata for metered CosmWasm VM - #2

Merged
hard-nett merged 4 commits into
v3.0.7-zkfrom
feat/bulk-memory-engine
Aug 20, 2026
Merged

feat: bulk_memory capability testdata for metered CosmWasm VM#2
hard-nett merged 4 commits into
v3.0.7-zkfrom
feat/bulk-memory-engine

Conversation

@hard-nett

Copy link
Copy Markdown
Member

Why

terp v6 default must accept rustc ≥ 1.87 CosmWasm guests. This branch sits on v3.0.7-zk (includes the store-ID / muslc checksum commits already on that line) and adds bulk_memory to wasmvm test capabilities plus rustc 1.87+ testdata.

Not in this PR

Makefile / libwasmvm/Cargo.toml remaps to a cosmwasm-bulk-memory directory name. After merge, sibling ../../cosmwasm on v3.1.0-zk.0 is the metered VM.

Stack

Depends on permissionlessweb/cosmwasm feat/bulk-memory-meteredv3.1.0-zk.0.

hard-nett added 4 commits August 14, 2026 13:55
Persist combined store now runs the same static Wasm checks as StoreCode.
Circuit identity is CircuitFooter::to_circuit_key (72), not a SHA-256 checksum.
Combined FFI blob is wasm checksum (32) || circuit key (72). Tests use norick_vk.bin
and assert those lengths; invalid wasm is rejected by check_wasm.
[skip ci] Host release dylib + cbindgen headers after check_wasm / 32+72 IDs.
Linux muslc/glibc artifacts still need make release-build-alpine and
release-build-linux on a Docker host.
[skip ci] Alpine muslc .a rebuilt with terpnetwork/zk-alpine-builder:1.88.
glibc .so still needs a rustc 1.88 debian builder.
Include rustc 1.87+ bulk-memory testdata. Local path remaps to
cosmwasm-bulk-memory stay uncommitted (worktree layout only).
@hard-nett
hard-nett merged commit f0d0c1f into v3.0.7-zk Aug 20, 2026
4 of 14 checks passed
@autofix-troubleshooter

Copy link
Copy Markdown

Hi! I'm the autofix logoautofix.ci troubleshooter bot.

It looks like you correctly set up a CI job that uses the autofix.ci GitHub Action, but the autofix.ci GitHub App has not been installed for this repository. This means that autofix.ci unfortunately does not have the permissions to fix this pull request. If you are the repository owner, please install the app and then restart the CI workflow! 😃

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.

2 participants