Skip to content

Comments

Enhance Solidity Code Formatting for code consistency and beautification#306

Open
pankajjagtapp wants to merge 3 commits intomasterfrom
pankaj/feat/enhance-code-formatting
Open

Enhance Solidity Code Formatting for code consistency and beautification#306
pankajjagtapp wants to merge 3 commits intomasterfrom
pankaj/feat/enhance-code-formatting

Conversation

@pankajjagtapp
Copy link
Contributor

@pankajjagtapp pankajjagtapp commented Oct 30, 2025

Note

Low Risk
Low risk: changes are limited to Foundry configuration and a new developer script; main risk is CI/dev friction if formatter settings or lint exclusions are too strict/lenient.

Overview
Standardizes Solidity tooling configuration. Updates foundry.toml to add Foundry lint settings (ignores test/** and src/archive/**, disables several naming/import lints) and a detailed fmt section to enforce consistent forge fmt output (e.g., high line_length, import sorting, quote/underscore styles).

Adds a safety check script. Introduces script/verify-forge-fmt-bytecode.sh, which builds contracts before/after running forge fmt, extracts deployed bytecode from artifacts, and fails if any compiled bytecode differs, writing per-contract comparisons under cache/bytecodeComparison/.

Written by Cursor Bugbot for commit af170ce. This will update automatically on new commits. Configure here.

@pankajjagtapp pankajjagtapp self-assigned this Oct 30, 2025
@pankajjagtapp pankajjagtapp force-pushed the pankaj/feat/enhance-code-formatting branch from 77a5f3e to 06b611b Compare November 13, 2025 19:36
@seongyun-ko
Copy link
Contributor

Let's do this once we merge #301

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

📊 Forge Coverage Report

forge coverage failed; see logs above (tests step is the gate).

Generated by workflow run #618

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

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.

3 participants