Enhance Solidity Code Formatting for code consistency and beautification#306
Enhance Solidity Code Formatting for code consistency and beautification#306pankajjagtapp wants to merge 4 commits intomasterfrom
Conversation
77a5f3e to
06b611b
Compare
…; add bytecode verification script
|
Let's do this once we merge #301 |
📊 Forge Coverage ReportGenerated by workflow run #618 |
…nhance bytecode verification script with working tree check
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Fuzz configuration accidentally removed in formatting PR
- Restored the [fuzz] section with max_shrink_iters = 100 to foundry.toml, which was accidentally deleted during formatting changes.
Or push these changes by commenting:
@cursor push b2b671e953
Preview (b2b671e953)
diff --git a/foundry.toml b/foundry.toml
--- a/foundry.toml
+++ b/foundry.toml
@@ -18,6 +18,9 @@
solc-version = '0.8.27'
dynamic_test_linking = true
+[fuzz]
+max_shrink_iters = 100
+
ignored_warnings_from = [
"src/archive/",
"src/libraries/",This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.
| "test/" | ||
| ] | ||
|
|
||
| [fuzz] |
There was a problem hiding this comment.
Fuzz configuration accidentally removed in formatting PR
High Severity
The [fuzz] section with max_shrink_iters = 100 was deleted as part of this formatting-only PR. This is unrelated to code formatting and changes fuzz test behavior — without this setting, Foundry uses its default (significantly higher, likely ~5000), which can drastically slow down CI fuzz test runs and alter shrinking behavior. This appears to be an accidental deletion.



Note
Low Risk
Low risk: changes are limited to Foundry config and a developer script; no Solidity contract logic is modified. Main risk is CI/dev workflow friction if
forge fmtsettings or the new script assumptions (git/jq/artifacts) don’t match environments.Overview
Adds Foundry
lintconfiguration and a detailedfmtconfiguration infoundry.tomlto standardize Solidity formatting (e.g., very longline_length, import sorting, numeric underscore style).Introduces
script/verify-forge-fmt-bytecode.sh, which compiles contracts before/afterforge fmt, extracts deployed bytecode from artifacts, and fails if formatting results in bytecode differences, saving per-contract comparisons undercache/bytecodeComparison/.Written by Cursor Bugbot for commit d2aefed. This will update automatically on new commits. Configure here.