Skip to content

starknet_os: os resources test - add replace_class#14142

Open
dorimedini-starkware wants to merge 1 commit into
05-23-starknet_os_os_resources_test_-_support_virtual_builtins_add_sha256from
05-23-starknet_os_os_resources_test_-_add_replace_class
Open

starknet_os: os resources test - add replace_class#14142
dorimedini-starkware wants to merge 1 commit into
05-23-starknet_os_os_resources_test_-_support_virtual_builtins_add_sha256from
05-23-starknet_os_os_resources_test_-_add_replace_class

Conversation

@dorimedini-starkware
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Collaborator Author

dorimedini-starkware commented May 24, 2026

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@cursor
Copy link
Copy Markdown

cursor Bot commented May 24, 2026

PR Summary

Low Risk
Test harness and fee/resource constant tuning only; no runtime auth or execution-path logic changes beyond updated OS hash and syscall cost tables.

Overview
replace_class is now included in the per-syscall OS resource regression flow instead of being skipped as unmeasurable.

Costs are updated to match measurement: Cairo OS REPLACE_CLASS_GAS_COST drops from 10670 to 10000, blockifier ReplaceClass n_steps from 106 to 92, and program_hash.json reflects the rebuilt OS. The Cairo1 OsResourcesTestContract invokes replace_class_syscall in __execute__, and ReplaceClass is removed from UNMEASURABLE_SYSCALLS in the flow test.

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

@dorimedini-starkware dorimedini-starkware force-pushed the 05-23-starknet_os_os_resources_test_-_add_replace_class branch from 9534bec to ed548b5 Compare May 24, 2026 20:08
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