Merge pull request #732 from protofire/develop

MERGE Develop into Master - v6.0.2

Author: dbale-altoros

.github/workflows/E2E.yml

@@ -18,7 +18,7 @@ jobs:
 
     steps:
       - uses: actions/checkout@v5
-      - uses: actions/setup-node@v4
+      - uses: actions/setup-node@v6
         with:
           node-version: ${{ matrix.node }}
           cache: 'npm'
@@ -53,7 +53,7 @@ jobs:
         run: |
           echo "::debug::Debugging enabled"
       - uses: actions/checkout@v5
-      - uses: actions/setup-node@v4
+      - uses: actions/setup-node@v6
         with:
           node-version: 18
           cache: 'npm'
@@ -91,7 +91,7 @@ jobs:
 
     steps:
     - uses: actions/checkout@v5
-    - uses: actions/setup-node@v4
+    - uses: actions/setup-node@v6
       with:
         node-version: 18
         cache: 'npm'

.github/workflows/PRE-COMMIT-HOOK-TEST.yml

@@ -20,7 +20,7 @@ jobs:
     name: Test Hook
     steps:
       - uses: actions/checkout@v5
-      - uses: actions/setup-node@v4
+      - uses: actions/setup-node@v6
         with:
           node-version: 20
       - uses: actions/setup-python@v5

.github/workflows/TESTS.yml

@@ -20,7 +20,7 @@ jobs:
 
     steps:
     - uses: actions/checkout@v5
-    - uses: actions/setup-node@v4
+    - uses: actions/setup-node@v6
       with:
         node-version: ${{ matrix.node-version }}
         cache: 'npm'
@@ -39,7 +39,7 @@ jobs:
 
     steps:
     - uses: actions/checkout@v5
-    - uses: actions/setup-node@v4
+    - uses: actions/setup-node@v6
       with:
         node-version: 18
         cache: 'npm'
@@ -58,7 +58,7 @@ jobs:
 
     steps:
     - uses: actions/checkout@v5
-    - uses: actions/setup-node@v4
+    - uses: actions/setup-node@v6
       with:
         node-version: 18
         cache: 'npm'

CHANGELOG.md

@@ -1,3 +1,36 @@
+## [6.0.2] - 2025-12-17
+🛠️ `Fix`: corrected use-natspec enforcement behavior on internal functions
+
+🛠️ `Fix`: misc minor issues and typos. General polish and stability
+<br><br>
+
+🆕 `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
+<br><br>
+
+🧱 `Enhancement`: improvements to `foundry-test-function` rule, including naming and configuration updates
+<br><br>
+
+🧰 `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
+<br><br>
+
+🧹 `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
+<br><br>
+
+✨🛡️ Kudos to our contributors! 🛡️✨
+- [MamunC0der](https://github.com/MamunC0der)
+- [@MarkFizz77](https://github.com/MarkFizz77)
+- [RidaMichofi](https://github.com/RidaMichofi)
+- [Daulox92](https://github.com/Daulox92)
+
 ## [6.0.1] - 2025-08-22
 
 🛠️ `Fix`: `no-unused-vars` for modifiers

conf/rulesets/solhint-all.js

@@ -15,6 +15,7 @@ module.exports = Object.freeze({
     'no-empty-blocks': 'warn',
     'no-global-import': 'warn',
     'no-unused-import': 'warn',
+    'no-unused-private-funcs': 'warn',
     'no-unused-vars': [
       'warn',
       {
@@ -67,11 +68,13 @@ module.exports = Object.freeze({
     'gas-struct-packing': 'warn',
     'comprehensive-interface': 'warn',
     'duplicated-imports': 'warn',
+    'foundry-no-block-time-number': ['warn', ['test', 'tests']],
     'import-path-check': ['warn', ['[~dependenciesPath]']],
     quotes: ['error', 'double'],
     'const-name-snakecase': 'warn',
     'contract-name-capwords': 'warn',
     'event-name-capwords': 'warn',
+    'foundry-test-function-naming': ['warn', ['setUp']],
     'foundry-test-functions': ['warn', ['setUp']],
     'func-name-mixedcase': 'warn',
     'func-named-parameters': ['warn', 4],
@@ -117,6 +120,7 @@ module.exports = Object.freeze({
     ],
     'multiple-sends': 'warn',
     'no-complex-fallback': 'warn',
+    'no-immutable-before-declaration': 'warn',
     'no-inline-assembly': 'warn',
     'not-rely-on-block-hash': 'warn',
     'not-rely-on-time': 'warn',

conf/rulesets/solhint-recommended.js

@@ -97,6 +97,7 @@ module.exports = Object.freeze({
     ],
     'multiple-sends': 'warn',
     'no-complex-fallback': 'warn',
+    'no-immutable-before-declaration': 'warn',
     'no-inline-assembly': 'warn',
     'not-rely-on-block-hash': 'warn',
     reentrancy: 'warn',

docker/Dockerfile

@@ -1,5 +1,5 @@
 FROM node:20-alpine
 LABEL maintainer="diego.bale@protofire.io" 
-ENV VERSION=6.0.1
+ENV VERSION=6.0.2
 
 RUN npm install -g solhint@"$VERSION"