starknet_os: os resources test - add meta tx#14135
Conversation
PR SummaryMedium Risk Overview The OS resources test contract now invokes Versioned constants and Cairo OS constants bump MetaTxV0 gas and VM resources (steps and Pedersen builtins). The pinned deployable measurement account takes variable-length calldata on Reviewed by Cursor Bugbot for commit 7c9637b. Bugbot is set up for automated code reviews on this repo. Configure here. |
08dbf97 to
f7fd3a3
Compare
311e125 to
a775a63
Compare
f7fd3a3 to
3ca8f12
Compare
a775a63 to
268457e
Compare
dafb7a8 to
ce6eb29
Compare
268457e to
8b293fb
Compare
ce6eb29 to
f48908d
Compare
8b293fb to
bd9c3be
Compare
f48908d to
f30d4c1
Compare
bd9c3be to
9711ff8
Compare
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: 1 of 5 files reviewed, 1 unresolved discussion (waiting on dorimedini-starkware).
crates/blockifier_test_utils/resources/feature_contracts/cairo1/os_resources_test_contract.cairo line 69 at r1 (raw file):
calldata: array![0, 0, 0, 0].span(), signature: array![].span(), )
Call the const contract instead, so you could have a base cost without arguments (same comment as the deploy syscall)
Code quote:
// meta_tx_v0 syscall - base.
meta_tx_v0_syscall(
address: self_address,
entry_point_selector: EXECUTE_FUNCTION_SELECTOR,
// class hash, address, deployable class hash, extra args len.
calldata: array![0, 0, 0, 0].span(),
signature: array![].span(),
)f30d4c1 to
ed74ad5
Compare
9711ff8 to
cfcc21d
Compare
cfcc21d to
cca3d02
Compare
ed74ad5 to
7c9637b
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ 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 7c9637b. Configure here.
| ~ /os_resources/execute_syscalls/Deploy/constant/n_steps: 1183 | ||
| ~ /os_resources/execute_syscalls/LibraryCall/n_steps: 874 | ||
| ~ /os_resources/execute_syscalls/MetaTxV0/constant/builtin_instance_counter/pedersen_builtin: 13 | ||
| ~ /os_resources/execute_syscalls/MetaTxV0/constant/n_steps: 1333 |
There was a problem hiding this comment.
Regression diff has wrong MetaTxV0 values
Medium Severity
The regression diff file claims MetaTxV0/constant/n_steps: 1333 and pedersen_builtin: 13, but the actual blockifier_versioned_constants_0_14_4.json has n_steps: 1315 and pedersen_builtin: 10. The 0.14.3 file has n_steps: 1301 and pedersen_builtin: 9. The regression diff format shows the 0.14.4 value (confirmed by the CallContract/n_steps: 901 entry matching the 0.14.4 file), so these values are inconsistent and will likely fail regression tests.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 7c9637b. Configure here.



No description provided.