Skip to content

blockifier: rename field to cairo native mode to match type name#12868

Merged
ArniStarkware merged 1 commit intomain-v0.14.2from
arni/blockifier/rename_field_to_cairo_native_mode
Mar 1, 2026
Merged

blockifier: rename field to cairo native mode to match type name#12868
ArniStarkware merged 1 commit intomain-v0.14.2from
arni/blockifier/rename_field_to_cairo_native_mode

Conversation

@ArniStarkware
Copy link
Copy Markdown
Contributor

@ArniStarkware ArniStarkware commented Feb 25, 2026

Note

Medium Risk
Mostly a mechanical rename, but it changes serialized config keys and runtime config wiring; misaligned config files or external tooling still using the old key could break native compilation behavior or startup.

Overview
Renames the Cairo Native execution-mode field in CairoNativeRunConfig from cairo_native_run_mode to cairo_native_mode across the blockifier crate (including serialization via SerializeConfig) and all call sites (native/trivial class managers, tests, integration tests, and blockifier_reexecution).

Updates config surface area accordingly by changing the JSON schema key and deployment app configs (batcher/gateway and their replacers) to emit/expect ...cairo_native_run_config.cairo_native_mode instead of ...cairo_native_run_config.cairo_native_run_mode, plus aligns the Python bridge (native_blockifier) conversion to populate the renamed field.

Written by Cursor Bugbot for commit 8b19a17. This will update automatically on new commits. Configure here.

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Copy Markdown

Artifacts upload workflows:

@ArniStarkware ArniStarkware marked this pull request as ready for review February 25, 2026 19:03
Copy link
Copy Markdown
Collaborator

@avi-starkware avi-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:

@avi-starkware reviewed 16 files and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on ArniStarkware).

@ArniStarkware ArniStarkware added this pull request to the merge queue Mar 1, 2026
Merged via the queue into main-v0.14.2 with commit 051a6d9 Mar 1, 2026
30 of 35 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants