starknet_os_flow_tests: fuzz test emit event scenario#13154
Merged
dorimedini-starkware merged 1 commit intoApr 5, 2026
Merged
Conversation
This was referenced Mar 10, 2026
This was referenced Mar 10, 2026
b168591 to
750f3f7
Compare
4f1916d to
0ad7c10
Compare
This was referenced Mar 11, 2026
0ad7c10 to
1a35509
Compare
750f3f7 to
c5906e9
Compare
1a35509 to
1d9e737
Compare
c5906e9 to
f5a603b
Compare
1d9e737 to
d4f0ad4
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.
d4f0ad4 to
ff032c0
Compare
9db6e63 to
36ce413
Compare
ff032c0 to
9941f34
Compare
36ce413 to
e158327
Compare
9941f34 to
3f3bd7c
Compare
e158327 to
bb9a379
Compare
3f3bd7c to
04c43c5
Compare
Yoni-Starkware
approved these changes
Mar 30, 2026
Collaborator
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware reviewed 6 files and all commit messages, and made 1 comment.
Reviewable status: 6 of 10 files reviewed, all discussions resolved (waiting on dorimedini-starkware).
bb9a379 to
a8d3ad0
Compare
04c43c5 to
8a163d3
Compare
a8d3ad0 to
60d66e9
Compare
8a163d3 to
5f143fe
Compare
Collaborator
Author
dorimedini-starkware
left a comment
There was a problem hiding this comment.
@dorimedini-starkware reviewed 10 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on dorimedini-starkware).
60d66e9 to
394b22c
Compare
5f143fe to
2492929
Compare
Merge activity
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note
Low Risk
Low risk: changes are confined to test contracts and flow-test harness, adding event emission coverage and expected-event assertions without altering production execution logic.
Overview
Adds a new fuzzing operation/scenario to emit Starknet events during the revert-fuzz flow (Cairo0 and Cairo1), including corresponding
FuzzEventdefinitions and anEmitEventbranch in the scenario dispatcher.Updates the OS flow fuzz test harness to generate event operations, track emitted event payloads per call frame, and assert expected events by passing per-tx
EventPredicateExpectations intoTestBuilder::add_invoke_tx. Regenerates the Cairo0 compiled artifact (fuzz_revert_compiled.json) to include the new event ABI/syscall.Reviewed by Cursor Bugbot for commit 2492929. Bugbot is set up for automated code reviews on this repo. Configure here.