Skip to content

Derive fuzzable trait#4240

Merged
piotmag769 merged 1 commit into
masterfrom
claude
May 15, 2026
Merged

Derive fuzzable trait#4240
piotmag769 merged 1 commit into
masterfrom
claude

Conversation

@piotmag769

@piotmag769 piotmag769 commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

Closes #2968

Introduced changes

  • self-explanatory

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added relevant tests
  • Performed self-review of the code TODO
  • Added changes to CHANGELOG.md

@piotmag769 piotmag769 requested review from a team, DelevoXDG and franciszekjob March 31, 2026 11:28
Comment thread crates/snforge-scarb-plugin/src/lib.rs
Comment thread crates/snforge-scarb-plugin/tests/integration/fuzzable.rs Outdated
@ddoktorski ddoktorski requested review from ddoktorski and removed request for franciszekjob April 2, 2026 12:34
@piotmag769 piotmag769 force-pushed the claude branch 2 times, most recently from 331e948 to 39ffa74 Compare May 6, 2026 12:46
@piotmag769 piotmag769 requested a review from a team as a code owner May 6, 2026 12:46
@piotmag769 piotmag769 force-pushed the claude branch 4 times, most recently from c1925c2 to 22d74e2 Compare May 7, 2026 18:10
Comment thread crates/forge/tests/data/fuzzing/tests/derive_fuzzable.cairo
Comment thread docs/listings/fuzz_testing/src/derive_fuzzable.cairo
Comment thread crates/snforge-scarb-plugin/src/derives/fuzzable.rs Outdated
@piotmag769 piotmag769 force-pushed the claude branch 2 times, most recently from d5fb527 to b736c3c Compare May 8, 2026 14:55
Comment thread crates/snforge-scarb-plugin/src/derives/fuzzable.rs Outdated
Comment thread crates/snforge-scarb-plugin/src/derives/fuzzable.rs Outdated
@piotmag769 piotmag769 force-pushed the claude branch 2 times, most recently from 64b5513 to 617b8ce Compare May 8, 2026 23:44
@piotmag769 piotmag769 requested a review from ddoktorski May 8, 2026 23:45
Comment thread crates/snforge-scarb-plugin/tests/integration/fuzzable.rs Outdated
Comment thread docs/src/appendix/snforge-library/fuzzable.md
Comment thread crates/forge/tests/e2e/fuzzing.rs
Comment thread crates/snforge-scarb-plugin/src/derives/fuzzable.rs
Comment thread crates/snforge-scarb-plugin/src/derives/fuzzable.rs
@piotmag769 piotmag769 requested a review from DelevoXDG May 13, 2026 10:21
@piotmag769 piotmag769 force-pushed the claude branch 2 times, most recently from d0153a4 to 851cdc6 Compare May 13, 2026 10:44
Comment thread snforge_std/src/fuzzable.cairo

@DelevoXDG DelevoXDG left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's add that issue though 🙏🏻

#4240 (comment)

@piotmag769 piotmag769 added this pull request to the merge queue May 15, 2026
Merged via the queue into master with commit 3a0f0f7 May 15, 2026
27 checks passed
@piotmag769 piotmag769 deleted the claude branch May 15, 2026 10:53
franciszekjob pushed a commit that referenced this pull request May 20, 2026
Closes #2968

- self-explanatory

- [x] Linked relevant issue
- [x] Updated relevant documentation
- [x] Added relevant tests
- [x] Performed self-review of the code TODO
- [x] Added changes to `CHANGELOG.md`
die-herdplatte pushed a commit to die-herdplatte/starknet-foundry that referenced this pull request May 21, 2026
## Introduced changes

- `check_error_free` instead of `is_empty` -
foundry-rs#4240 (comment)
- stopped returning additional diags when parser error happen - this
makes the UX better while writing the code. Now:

<img width="200" height="105" alt="image"
src="https://github.com/user-attachments/assets/e41247ab-c945-4378-ab36-eafe09c8b48a"
/>

vs before: 

<img width="189" height="101" alt="image"
src="https://github.com/user-attachments/assets/e6b375db-4cb9-4921-8572-525cd748bb15"
/>


## Checklist

- [ ] Linked relevant issue
- [ ] Updated relevant documentation
- [ ] Added relevant tests
- [ ] Performed self-review of the code
- [ ] Added changes to `CHANGELOG.md`
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.

Add derive macro for Fuzzable trait

3 participants