starknet_os: os resources test - add send_message_to_l1#14143
Conversation
PR SummaryMedium Risk Overview
Reviewed by Cursor Bugbot for commit a359fa6. Bugbot is set up for automated code reviews on this repo. Configure here. |
191b6ac to
9534bec
Compare
48a7191 to
39edd2e
Compare
9534bec to
ed548b5
Compare
39edd2e to
b4dc505
Compare
ed548b5 to
8be4f98
Compare
b4dc505 to
6d69bf0
Compare
6d69bf0 to
a359fa6
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 a359fa6. Configure here.
| @@ -1,4 +1,4 @@ | |||
| + /os_constants/allowed_virtual_os_program_hashes/2: "0x66a46bb88553c4dafd12959575033660780d689c99659ab3620e2f524f71701" | |||
| + /os_constants/allowed_virtual_os_program_hashes/2: "0x471dd673f38125ba94e98145e3579d700ad13bd0e177687b67bab6221da4fa0" | |||
There was a problem hiding this comment.
Regression diff file missing SendMessageToL1 n_steps change
Medium Severity
The SendMessageToL1 n_steps was changed from 144 to 124 in blockifier_versioned_constants_0_14_4.json, but the corresponding entry ~ /os_resources/execute_syscalls/SendMessageToL1/n_steps: 124 is missing from the 0.14.3_0.14.4.txt regression diff file. The test_vc_diffs_regression test uses expect_file! to compare the computed JSON diff against this file, so it will fail without this entry.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit a359fa6. Configure here.



No description provided.