chore(deps): update dependency solhint to v6

[renovate]

This PR contains the following updates:

Package	Change	Age	Confidence
solhint (source)	5.0.5 → 6.0.3

---

Release Notes

protofire/solhint (solhint)

v6.0.3

Compare Source

🛠️ Fix: removed unused files, normalized schema for validation, load-rules, base-checker and validator improvements

🛠️ Fix: removing console log from use-natspec rule (thanks brossetti1!!)

🛠️ Fix: misc minor issues and typos. General polish and stability

✨🛡️ Kudos to our contributors! 🛡️✨

• brossetti1
• maradini77

v6.0.2

Compare Source

🛠️ Fix: corrected use-natspec enforcement behavior on internal functions

🛠️ Fix: misc minor issues and typos. General polish and stability

🆕 Rule: added foundry-no-block to detect usage of block.timestamp and block.number in Solidity tests

🆕 Rule: added no-unused-private-functions to prevent unused private function declarations

🆕 Rule: added no-immutable-before-declaration to enforce correct immutable declaration order

🧱 Enhancement: improvements to foundry-test-function rule, including naming and configuration updates

🧰 Infra: update GitHub Actions in CI workflows to v5 and v6. Keeps CI aligned with latest ecosystem changes

🧰 Infra: bump js-yaml dependency following security recommendations

🧹 Chore: stop publishing the test directory to npm packages. Reduces package size and noise

🧹 Chore: foundry-test-functions is deprecated and will be removed in v7.0.0. Please rename to foundry-test-function-naming.
WILL BE REPLACED IN v7

✨🛡️ Kudos to our contributors! 🛡️✨

• MamunC0der
• @​MarkFizz77
• RidaMichofi
• Daulox92

v6.0.1

Compare Source

🛠️ Fix: no-unused-vars for modifiers

🛠️ Fix: no-unused-imports handled for multiline @​inheritdoc. Eliminates missed detections with multiline inheritance docs

🛠️ Fix: func-name-mixedcase improvement, added snake case exception in interface (naming). Aligns interface naming edge cases

🛠️ Fix: misc minor issues; docs link and typos. General polish and stability

🧰 Infra: Update GitHub Actions in CI workflows. Keeps CI up to date and consistent across jobs

🧰 Infra: Ignore enhancement to mimic as regular linters

🧹 Chore: Remove fs-extra dependency. Simplifies deps

🧹 Chore: Remove strip-ansi. Cleans unused/legacy deps

🧹 Chore: Changed changelog format

🧱 Enhancement: added config for no-unused-vars to validate parameters or not

🧱 Validation: Improve AJV schema validation and related caching approach. More robust/faster config validation

✨🛡️ Kudos to our contributors! 🛡️✨

• @​smol-ninja 🏅
• @​talentlessguy 🏅
• @​loki-sama
• @​gonzaotc
• @​kilavvy
• @​jmendiola222
• @​Daulox92
• @​RidaMichofi
• @​ddnexus
• @​zeevick10
• @​RyanRHall
• @​fulldecent
• @​ncjones

v6.0.0

Compare Source

BREAKING CHANGE

Some rules were removed from the recommended ruleset and other were promoted. See below:
Rules revision - #​692

🟥 DEMOTED

• payable-fallback: Outdated due to the introduction of receive(); rarely needed in modern Solidity.
  
  

✅ PROMOTED

• interface-starts-with-i: Promotes clarity by clearly distinguishing interfaces from contracts.
• gas-calldata-parameters: Encourages optimal memory usage for external functions.
• gas-increment-by-one: Suggests using ++i style for gas-efficient increments.
• gas-indexed-events: Improves event indexing for off-chain systems and reduces gas usage.
• gas-small-strings: Recommends cheaper encoding using bytes32 for short strings.
• gas-strict-inequalities: Helps avoid subtle off-by-one bugs in range conditions.
• gas-struct-packing: Maximizes storage efficiency by suggesting tight struct packing.
• duplicated-imports: Prevents redundant imports that bloat the AST and confuse code structure.
• import-path-check: Ensures all imported files exist and resolve correctly, avoiding runtime errors.
• function-max-lines: Encourages smaller, more readable and maintainable functions.
• constructor-syntax: Enforces modern constructor syntax for consistency and clarity.
• use-natspec: Promotes proper documentation with NatSpec for better audits and readability.

Fixed

• gas-struct-packing false positives - #​690
• gas-increment-by-one backward expression - #​691
• Typo (thanks to @​MarkFizz77)

Added

• use-natspec: Promote the use of natspec and make several checks to enforce it - #​689

v5.2.0

Compare Source

Fixed

• imports-order interference with no-unused-imports - #​684

Added

• Added cache flag allowing to skip already evaluated files if no error was thrown and file did not change - #​685
• Added multi-directory config support by hierarchy - #​678
• Added better error handling on invalid configs - #​683

Thanks to @​smol-ninja - @​PaulRBerg @​lechpzn, @​otc-png, @​MamunC0der, @​kks-code, @​RidaMichofi

For making the time of proposing rules, pointing out typos, broken links, unused code, copyright, small reviews, etc. We really appreciate your contributions!

v5.1.0

Compare Source

Added

• Argument to avoid discord poster - #​646
• Auto-fix for no-unused-import - #​648 (Thanks to @​DenisGorbachev)
• New Rule: import-path-check - #​657

Updated

• yml and parser update - #​668 (Thanks to @​zeevick10)
• cleanup deprecated code - #​655
• Improved ordering rule - #​656

BREAKING CHANGE

• Removed code for solhint:default ruleset - #​654

Thanks to @​fanqiaojun, @​maximevtush, @​MaxweLL22-22, @​PixelPil0t1, @​wgyt, @​operagxsasha, @​Hopium21, @​leopardracer, @​omahs, @​zeevick10, @​gap-editor, @​Olexandr88, @​jirutka

For making the time of pointing out typos, license update, broken links, unused code, copyright, small reviews, etc. We really appreciate your contributions!

---

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR was generated by Mend Renovate. View the repository job log.

[pubkey]

@copilot please analyze and propose a fix or improvement for this issue/PR.

CI error:

Run npm run lint
npm warn Unknown project config "unsafe-perm". This will stop working in the next major version of npm.

> eth-crypto@4.0.0 lint
> eslint src test config && solhint "contracts/**/*.sol"

ConfigMissingError: Failed to load config "solhint:default" to extend from.
    at /home/runner/work/eth-crypto/eth-crypto/node_modules/solhint/lib/config/config-file.js:85:13
    at Array.reduceRight (<anonymous>)
    at applyExtends (/home/runner/work/eth-crypto/eth-crypto/node_modules/solhint/lib/config/config-file.js:80:25)
    at processStr (/home/runner/work/eth-crypto/eth-crypto/node_modules/solhint/lib/index.js:24:12)
    at processAndCache (/home/runner/work/eth-crypto/eth-crypto/node_modules/solhint/lib/index.js:71:18)
    at Object.processFile (/home/runner/work/eth-crypto/eth-crypto/node_modules/solhint/lib/index.js:100:36)
    at /home/runner/work/eth-crypto/eth-crypto/node_modules/solhint/solhint.js:203:45
    at Array.map (<anonymous>)
    at executeMainActionLogic (/home/runner/work/eth-crypto/eth-crypto/node_modules/solhint/solhint.js:198:27)
    at /home/runner/work/eth-crypto/eth-crypto/node_modules/solhint/solhint.js:153:9
Error: Process completed with exit code 255.

Details

Important Reference (RxDB docs for LLMs)

Use the official RxDB docs here to align with architecture, plugins, storage abstractions, replication patterns, typings, and coding conventions:
https://rxdb.info/llms.txt

---

What to do (pick the right action based on what’s present)

Bug report with no test case

• Create a minimal failing test that reproduces the bug.
• Ensure the test fails on current main (or before your changes) and passes after the fix.
• Keep the test small and focused (one behavior).

Bug report with a test case already present but no fix

• Implement the minimal fix to make the existing test pass.
• Avoid unrelated refactors.
• Add additional assertions only if they clarify the intended behavior.

Regression (something used to work)

• Add/confirm a regression test that captures the previous correct behavior.
• Fix with the smallest change that restores compatibility.

Dependency version bump required

• Update the dependency versions as requested.
• Adjust code/types/tests to satisfy the new versions.
• If the bump requires breaking changes, document the reason and propose the smallest migration.

Types/TS errors

• Fix types in a way that preserves runtime behavior.
• Prefer stricter/correct typings over any or broad casts.
• Add a type-level or runtime test if the repo pattern supports it.

Docs-only or README mismatch

• Update docs/examples to match actual behavior.
• If docs reveal a bug, add a test and fix accordingly.

---

Output requirements

Please provide:

• A short root-cause explanation
• A unified diff patch (code + tests when applicable)
• Notes on edge-cases / compatibility
• If relevant: any migration notes

---

🚨 Author mention rule

If your solution:

• modifies the original author’s implementation, OR
• adds additional logic on top of the author's work, OR
• adds/changes tests related to the author's contribution,

then mention the original issue/PR author at the end of your response:

@ please review the proposed changes.

[Copilot]

@pubkey I've opened a new pull request, #987, to work on those changes. Once the pull request is ready, I'll request review from you.

[renovate]

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.