protofire · dbale-altoros · Aug 22, 2025 · Jul 29, 2025 · Jul 29, 2025 · Aug 5, 2025
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -1,6 +1,6 @@
 module.exports = {
   parserOptions: {
-    ecmaVersion: 2020,
+    ecmaVersion: 2022,
   },
   env: {
     browser: false,

diff --git a/.github/workflows/DOCKER.yml b/.github/workflows/DOCKER.yml
@@ -17,7 +17,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v5
 
     - name: Login to Docker Hub
       run: docker login -u ${{ secrets.DOCKER_USERNAME }} -p ${{ secrets.DOCKER_ACCESS_TOKEN }}

diff --git a/.github/workflows/E2E.yml b/.github/workflows/E2E.yml
@@ -17,8 +17,8 @@ jobs:
         node: [16, 18, 20]
 
     steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v5
+      - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node }}
           cache: 'npm'
@@ -52,8 +52,8 @@ jobs:
       - name: Enable Debugging
         run: |
           echo "::debug::Debugging enabled"
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v5
+      - uses: actions/setup-node@v4
         with:
           node-version: 18
           cache: 'npm'
@@ -90,8 +90,8 @@ jobs:
     name: Test on MacOS
 
     steps:
-    - uses: actions/checkout@v4
-    - uses: actions/setup-node@v3
+    - uses: actions/checkout@v5
+    - uses: actions/setup-node@v4
       with:
         node-version: 18
         cache: 'npm'

diff --git a/.github/workflows/PRE-COMMIT-HOOK-TEST.yml b/.github/workflows/PRE-COMMIT-HOOK-TEST.yml
@@ -19,7 +19,7 @@ jobs:
     runs-on: ubuntu-latest
     name: Test Hook
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
       - uses: actions/setup-node@v4
         with:
           node-version: 20

diff --git a/.github/workflows/TESTS.yml b/.github/workflows/TESTS.yml
@@ -19,8 +19,8 @@ jobs:
         node-version: [16, 18, 20]
 
     steps:
-    - uses: actions/checkout@v4
-    - uses: actions/setup-node@v3
+    - uses: actions/checkout@v5
+    - uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node-version }}
         cache: 'npm'
@@ -38,8 +38,8 @@ jobs:
     runs-on: windows-latest
 
     steps:
-    - uses: actions/checkout@v4
-    - uses: actions/setup-node@v3
+    - uses: actions/checkout@v5
+    - uses: actions/setup-node@v4
       with:
         node-version: 18
         cache: 'npm'
@@ -57,8 +57,8 @@ jobs:
     runs-on: macos-latest
 
     steps:
-    - uses: actions/checkout@v4
-    - uses: actions/setup-node@v3
+    - uses: actions/checkout@v5
+    - uses: actions/setup-node@v4
       with:
         node-version: 18
         cache: 'npm'

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,47 @@
+## [6.0.1] - 2025-08-22
+
+🛠️ `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
+<br><br>
+
+🧰 `Infra`: Update GitHub Actions in CI workflows. Keeps CI up to date and consistent across jobs
+
+🧰 `Infra`: Ignore enhancement to mimic as regular linters
+<br><br>
+
+🧹 `Chore`: Remove fs-extra dependency. Simplifies deps
+
+🧹 `Chore`: Remove strip-ansi. Cleans unused/legacy deps
+
+🧹 `Chore`: Changed changelog format
+<br><br>
+
+🧱 `Enhancement`: added config for no-unused-vars to validate parameters or not
+<br><br>
+
+🧱 `Validation`: Improve AJV schema validation and related caching approach. More robust/faster config validation
+<br><br>
+
+✨🛡️ Kudos to our contributors! 🛡️✨
+- [@smol-ninja](https://github.com/smol-ninja) 🏅
+- [@talentlessguy](https://github.com/talentlessguy) 🏅
+- [@loki-sama](https://github.com/loki-sama)
+- [@gonzaotc](https://github.com/gonzaotc)
+- [@kilavvy](https://github.com/kilavvy)
+- [@jmendiola222](https://github.com/jmendiola222)
+- [@Daulox92](https://github.com/Daulox92)
+- [@RidaMichofi](https://github.com/RidaMichofi)
+- [@ddnexus](https://github.com/ddnexus)
+- [@zeevick10](https://github.com/zeevick10)
+- [@RyanRHall](https://github.com/RyanRHall)
+- [@fulldecent](https://github.com/fulldecent)
+- [@ncjones](https://github.com/ncjones)
+
 ## [6.0.0] - 2025-07-04
 
 ### BREAKING CHANGE
@@ -27,7 +71,7 @@ Rules revision - [#692](https://github.com/protofire/solhint/pull/692)
 ### Fixed
 - `gas-struct-packing` false positives - [#690](https://github.com/protofire/solhint/pull/690)
 - `gas-increment-by-one` backward expression - [#691](https://github.com/protofire/solhint/pull/691)
-- Typo (thanks to @[MarkFizz77](https://github.com/MarkFizz77))
+- Typo (thanks to [@MarkFizz77](https://github.com/MarkFizz77))
 
 ### Added
  - `use-natspec`: Promote the use of natspec and make several checks to enforce it - [#689](https://github.com/protofire/solhint/pull/689)

diff --git a/README.md b/README.md
@@ -9,8 +9,6 @@
 [![](https://img.shields.io/badge/Join%20Our%20Discord-magenta)](https://discord.gg/4TYGq3zpjs)
 [![Donate with Ethereum](https://img.shields.io/badge/Donate-ETH-blue)](https://etherscan.io/address/0xA81705c8C247C413a19A244938ae7f4A0393944e)
 [![NPM version](https://badge.fury.io/js/solhint.svg)](https://npmjs.org/package/solhint)
-[![Coverage Status](https://coveralls.io/repos/github/protofire/solhint/badge.svg?branch=master)](
-https://coveralls.io/github/protofire/solhint?branch=master)
 [![MIT licensed](https://img.shields.io/badge/license-MIT-blue.svg)](https://raw.githubusercontent.com/protofire/solhint/master/LICENSE)
 
 This is an open source project for linting [Solidity](http://solidity.readthedocs.io/en/develop/) code. This project
@@ -80,7 +78,7 @@ Commands:
   stdin [options]                         linting of source code data provided to STDIN
   list-rules                              display covered rules of current .solhint.json
 ```
-### Notes
+### New Versions
 - Solhint checks if there are newer versions. The `--disc`  option avoids that check.
 - `--save` option will create a file named as `YYYYMMDDHHMMSS_solhintReport.txt` on current folder with default or specified format 
 
@@ -95,10 +93,9 @@ This option currently works on:
 - quotes
 - contract-name-capwords
 - avoid-suicide
-
 <br><br>
-## Configuration
 
+## Configuration
 You can use a `.solhint.json` file to configure Solhint for the whole project.
 
 To generate a new sample `.solhint.json` file in current folder you can do:
@@ -114,12 +111,12 @@ This file has the following format:
   "extends": "solhint:recommended"
 }
 ```
-### Note 1
+
 The `solhint:default` configuration contains only two rules: max-line-length & no-console
 It is now deprecated since version 5.1.0
 <br>
 
-### Note 2
+### Multiple Configs
 Multiple configs files can be used at once. All config files should be named `.solhint.json`.
 If not done like this, multiple hierarchy configuration will not work.
 Solhint will go though all config files automatically.
@@ -148,7 +145,7 @@ Project ROOT =>
 <br><br>
 
 
-### Sample
+### Sample of simple config with recommended rules
 ```json
   {
     "extends": "solhint:recommended",
@@ -160,15 +157,28 @@ Project ROOT =>
   }
 ```
 A full list of all supported rules can be found [here](docs/rules.md).
+<br><br>
+
+### Ignore Configuration
+You can exclude files from linting using a `.solhintignore` file (name by default) or `--ignore-path` followed by a custom name. 
+It uses the same syntax as `.gitignore`, including support for negation with !.
 
-To ignore files that do not require validation you can use a `.solhintignore` file. It supports rules in
-the `.gitignore` format.
+Example:
 
 ```
-node_modules/
-additional-tests.sol
+contracts/**
+!contracts/utils/
+!contracts/utils/SafeMath.sol
 ```
 
+This will:
+  - Ignore everything inside contracts/
+  - Except the folder contracts/utils/
+  - And the file SafeMath.sol inside it
+
+Tip: To unignore a file, you must also unignore its parent folders.
+<br><br>
+
 ### Cache
 Solhint supports a caching mechanism using the `--cache` flag to avoid re-linting files that haven't changed. 
 When enabled, Solhint stores a hash of each file's content and effective configuration, skipping analysis if neither has changed. 
@@ -185,8 +195,7 @@ Example:
 solhint contracts/**/*.sol --cache
 solhint Foo.sol --cache --cache-location tmp/my-cache.json
 ```
-
-
+<br><br>
 
 ### Extendable rulesets
 

diff --git a/_config.yml b/_config.yml
diff --git a/conf/rulesets/solhint-all.js b/conf/rulesets/solhint-all.js
@@ -15,7 +15,12 @@ module.exports = Object.freeze({
     'no-empty-blocks': 'warn',
     'no-global-import': 'warn',
     'no-unused-import': 'warn',
-    'no-unused-vars': 'warn',
+    'no-unused-vars': [
+      'warn',
+      {
+        validateParameters: true,
+      },
+    ],
     'one-contract-per-file': 'warn',
     'payable-fallback': 'warn',
     'reason-string': [

diff --git a/conf/rulesets/solhint-recommended.js b/conf/rulesets/solhint-recommended.js
@@ -13,7 +13,12 @@ module.exports = Object.freeze({
     'no-empty-blocks': 'warn',
     'no-global-import': 'warn',
     'no-unused-import': 'warn',
-    'no-unused-vars': 'warn',
+    'no-unused-vars': [
+      'warn',
+      {
+        validateParameters: true,
+      },
+    ],
     'one-contract-per-file': 'warn',
     'reason-string': [
       'warn',

diff --git a/docker/Dockerfile b/docker/Dockerfile
@@ -1,5 +1,5 @@
 FROM node:20-alpine
 LABEL maintainer="diego.bale@protofire.io" 
-ENV VERSION=6.0.0
+ENV VERSION=6.0.1
 
 RUN npm install -g solhint@"$VERSION"
diff --git a/docs/_config.yml b/docs/_config.yml
diff --git a/docs/rules.md b/docs/rules.md
@@ -45,7 +45,7 @@ title:       "Rule Index of Solhint"
 | [use-forbidden-name](./rules/naming/use-forbidden-name.md)                           | Avoid to use letters 'I', 'l', 'O' as identifiers.                                                            | $~~~~~~~~$✔️ |            |
 | [var-name-mixedcase](./rules/naming/var-name-mixedcase.md)                           | Variable names must be in mixedCase. (Does not check IMMUTABLES nor CONSTANTS (use inherent rules for that)   | $~~~~~~~~$✔️ |            |
 | [imports-on-top](./rules/order/imports-on-top.md)                                    | Import statements must be on top.                                                                             | $~~~~~~~~$✔️ |            |
-| [imports-order](./rules/naming/imports-order.md)                                     | Order the imports of the contract to follow a certain hierarchy (read "Notes section")                        |              |            |
+| [imports-order](./rules/order/imports-order.md)                                      | Order the imports of the contract to follow a certain hierarchy (read "Notes section")                        |              |            |
 | [ordering](./rules/order/ordering.md)                                                | Check order of elements in file and inside each contract, according to the style guide                        |              |            |
 | [visibility-modifier-order](./rules/order/visibility-modifier-order.md)              | Visibility modifier must be first in list of modifiers.                                                       | $~~~~~~~~$✔️ |            |
 

diff --git a/docs/rules/best-practices/no-unused-vars.md b/docs/rules/best-practices/no-unused-vars.md
@@ -15,13 +15,24 @@ title:       "no-unused-vars | Solhint"
 Variable "name" is unused.
 
 ## Options
-This rule accepts a string option for rule severity. Must be one of "error", "warn", "off". Defaults to warn.
+This rule accepts an array of options:
+
+| Index | Description                                                                                                                                       | Default Value               |
+| ----- | ------------------------------------------------------------------------------------------------------------------------------------------------- | --------------------------- |
+| 0     | Rule severity. Must be one of "error", "warn", "off".                                                                                             | warn                        |
+| 1     | A JSON object with a single property "validateParameters" as boolean specifying whether to report unused parameters from functions and modifiers. | {"validateParameters":true} |
+
 
 ### Example Config
 ```json
 {
   "rules": {
-    "no-unused-vars": "warn"
+    "no-unused-vars": [
+      "warn",
+      {
+        "validateParameters": true
+      }
+    ]
   }
 }
 ```

diff --git a/docs/rules/best-practices/use-natspec.md b/docs/rules/best-practices/use-natspec.md
@@ -133,7 +133,7 @@ This rule accepts an array of options:
 ```
 
 ## Version
-This rule was introduced in the latest version.
+This rule was introduced in [Solhint 6.0.0](https://github.com/protofire/solhint/blob/v6.0.0)
 
 ## Resources
 - [Rule source](https://github.com/protofire/solhint/blob/master/lib/rules/best-practices/use-natspec.js)

diff --git a/docs/rules/miscellaneous/quotes.md b/docs/rules/miscellaneous/quotes.md
@@ -36,7 +36,7 @@ This rule accepts an array of options:
 ```
 
 ### Notes
-- This rule allows to put a double quote inside single quote string and viceversa
+- This rule allows to put a double quote inside single quote string and vice versa
 
 ## Examples
 ### 👍 Examples of **correct** code for this rule

diff --git a/docs/rules/naming/func-name-mixedcase.md b/docs/rules/naming/func-name-mixedcase.md
@@ -26,6 +26,9 @@ This rule accepts a string option for rule severity. Must be one of "error", "wa
 }
 ```
 
+### Notes
+- SNAKE_CASE allowed only in interfaces when matching constant/immutable getter signatures.
+- Return must be elementary, enum, UDVT, or contract/interface.
 
 ## Examples
 This rule does not have examples.

diff --git a/docs/rules/naming/imports-order.md → docs/rules/order/imports-order.md b/docs/rules/naming/imports-order.md → docs/rules/order/imports-order.md