Skip to content

release: upgrade cairo, native, proving-utils#14200

Open
dorimedini-starkware wants to merge 1 commit into
mainfrom
05-26-release_upgrade_cairo_native_proving-utils
Open

release: upgrade cairo, native, proving-utils#14200
dorimedini-starkware wants to merge 1 commit into
mainfrom
05-26-release_upgrade_cairo_native_proving-utils

Conversation

@dorimedini-starkware
Copy link
Copy Markdown
Collaborator

No description provided.

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

Copy link
Copy Markdown
Collaborator Author

dorimedini-starkware commented May 26, 2026

@dorimedini-starkware dorimedini-starkware self-assigned this May 26, 2026
@github-actions
Copy link
Copy Markdown

Artifacts upload workflows:

@dorimedini-starkware dorimedini-starkware marked this pull request as ready for review May 26, 2026 10:21
@cursor
Copy link
Copy Markdown

cursor Bot commented May 26, 2026

PR Summary

Medium Risk
Wide toolchain bump changes compiled contract gas, bouncer migration costs, and OS hint coverage; native sha512_process_block is still todo!(), which is a footgun for Cairo Native execution.

Overview
This PR bumps the Cairo / native / proving toolchain and refreshes dependent snapshots across the sequencer.

Dependency pins in Cargo.toml move cairo-lang crates from 2.17.0-rc.42.19.0-rc.0, cairo-native to 0.9.0-rc.7, and proving-utils (privacy-prove, privacy-circuit-verify-v1) to git rev 1e0f8fc. Matching version files are updated (cairo_compiler_version.txt, native_compiler_version.txt). Cargo.lock reflects a large transitive refresh (e.g. indexmap, crypto/hash crates, stwo-circuits git rev c99a181).

Blockifier / execution: syscall and transaction tests are re-baselined for lower gas_consumed, fewer range-check builtins, and updated redeposit amounts. CASM hash migration regression gas in bouncer_test drops (110M→104M Sierra, 240M→225M proving). Native SyscallHandler gains a sha512_process_block stub that **todo!()**s. Stack-trace golden files shift PCs after recompilation.

OS / flow tests: many hint_coverage JSON fixtures add EnterScopeNewNode. Fuzz contract addresses and experimental-libfunc builtin counts (poseidon/blake) are updated.

Risk note: Native Cairo1 paths that hit SHA-512 syscalls will panic until the stub is implemented; gas/bouncer numbers changing with the compiler can affect fee limits and block capacity if production constants are tied to these tests.

Reviewed by Cursor Bugbot for commit f293605. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit fbc7569. Configure here.

Comment thread crates/blockifier/src/execution/native/syscall_handler.rs
Copy link
Copy Markdown
Collaborator

@Yoni-Starkware Yoni-Starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Yoni-Starkware reviewed 22 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on dorimedini-starkware).


Cargo.toml line 338 at r1 (raw file):

primitive-types = "0.12.1"
privacy-circuit-verify-v0 = { package = "privacy-circuit-verify", git = "https://github.com/starkware-libs/proving-utils", rev = "580135e" }
privacy-circuit-verify-v1 = { package = "privacy-circuit-verify", git = "https://github.com/starkware-libs/proving-utils", rev = "039e52c" }

v1 (latest) should be aligned with privacy-prove

Suggestion:

privacy-circuit-verify-v1 = { package = "privacy-circuit-verify", git = "https://github.com/starkware-libs/proving-utils", rev = "1e0f8fc" }

@dorimedini-starkware dorimedini-starkware force-pushed the 05-26-release_upgrade_cairo_native_proving-utils branch from fbc7569 to 1e7cdc6 Compare May 26, 2026 10:42
Copy link
Copy Markdown
Collaborator Author

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dorimedini-starkware made 1 comment.
Reviewable status: 20 of 53 files reviewed, 2 unresolved discussions (waiting on Yoni-Starkware).


Cargo.toml line 338 at r1 (raw file):

Previously, Yoni-Starkware (Yoni) wrote…

v1 (latest) should be aligned with privacy-prove

Done.

Copy link
Copy Markdown
Collaborator

@Yoni-Starkware Yoni-Starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@Yoni-Starkware reviewed 33 files and all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on dorimedini-starkware).

Copy link
Copy Markdown
Collaborator Author

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dorimedini-starkware resolved 1 discussion.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on dorimedini-starkware).

@dorimedini-starkware dorimedini-starkware force-pushed the 05-26-release_upgrade_cairo_native_proving-utils branch from 1e7cdc6 to 6ba9319 Compare May 26, 2026 10:50
@dorimedini-starkware dorimedini-starkware force-pushed the 05-26-release_upgrade_cairo_native_proving-utils branch from 6ba9319 to f293605 Compare May 26, 2026 11:19
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.

3 participants