Autofix event name camelcase

docs/rules.md

@@ -38,7 +38,7 @@ title:       "Rule Index of Solhint"
 | Rule Id                                                                              | Error                                                                                                         | Recommended  | Deprecated  |
 | ------------------------------------------------------------------------------------ | ------------------------------------------------------------------------------------------------------------- | ------------ | ----------- |
 | [const-name-snakecase](./rules/naming/const-name-snakecase.md)                       | Constant name must be in capitalized SNAKE_CASE. (Does not check IMMUTABLES, use immutable-vars-naming)       | $~~~~~~~~$✔️ |             |
-| [contract-name-camelcase](./rules/naming/contract-name-camelcase.md)                 | Contract name must be in CamelCase.                                                                           | $~~~~~~~~$✔️ |             |
+| [contract-name-camelcase](./rules/naming/contract-name-camelcase.md)                 | Contract, Structs and Enums should be in CamelCase.                                                           | $~~~~~~~~$✔️ |             |
 | [event-name-camelcase](./rules/naming/event-name-camelcase.md)                       | Event name must be in CamelCase.                                                                              | $~~~~~~~~$✔️ |             |
 | [foundry-test-functions](./rules/naming/foundry-test-functions.md)                   | Enforce naming convention on functions for Foundry test cases                                                 |              |             |
 | [func-name-mixedcase](./rules/naming/func-name-mixedcase.md)                         | Function name must be in mixedCase.                                                                           | $~~~~~~~~$✔️ |             |

docs/rules/naming/contract-name-camelcase.md

@@ -12,7 +12,7 @@ title:       "contract-name-camelcase | Solhint"
 
 
 ## Description
-Contract name must be in CamelCase.
+Contract, Structs and Enums should be in CamelCase.
 
 ## Options
 This rule accepts a string option of rule severity. Must be one of "error", "warn", "off". Default to warn.
@@ -26,6 +26,9 @@ This rule accepts a string option of rule severity. Must be one of "error", "war
 }
 ```
 
+### Notes
+- Solhint allows this rule to automatically fix the code with `--fix` option
+- The FIX will only change first letter and remove underscores
 
 ## Examples
 This rule does not have examples.

docs/rules/naming/event-name-camelcase.md

@@ -26,6 +26,9 @@ This rule accepts a string option of rule severity. Must be one of "error", "war
 }
 ```
 
+### Notes
+- Solhint allows this rule to automatically fix the code with `--fix` option
+- The FIX will only change first letter and remove underscores
 
 ## Examples
 This rule does not have examples.

e2e/08-autofix/contract-name-camelcase/.solhint.json

@@ -0,0 +1,5 @@
+{
+  "rules": {
+    "contract-name-camelcase": "error"
+  }
+}

e2e/08-autofix/contract-name-camelcase/Foo1.sol

@@ -0,0 +1,61 @@
+// SPDX-License-Identifier: Apache-2.0
+pragma solidity ^0.8.0;
+
+struct not_Used_Struct1 {
+        uint256 three;
+        uint256 four;
+}
+
+enum __status1_ {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+struct NotUsedStruct2 {
+        uint256 three;
+        uint256 four;
+}
+
+enum Status2 {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+contract __generic {
+
+    struct not_UsedStruct3__ {
+        uint256 three;
+        uint256 four;
+    }
+
+    enum status3     {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+    struct NotUsedStruct4 {
+        uint256 three;
+        uint256 four;
+    }
+
+    enum Status4 {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+    constructor() {}
+
+    function function1() pure external {
+        uint af1 = 0;
+        af1;
+    }
+
+    function function2() external pure {
+        uint b;
+        b;
+    }
+}

e2e/08-autofix/contract-name-camelcase/Foo1AfterFix.sol

@@ -0,0 +1,61 @@
+// SPDX-License-Identifier: Apache-2.0
+pragma solidity ^0.8.0;
+
+struct NotUsedStruct1 {
+        uint256 three;
+        uint256 four;
+}
+
+enum Status1 {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+struct NotUsedStruct2 {
+        uint256 three;
+        uint256 four;
+}
+
+enum Status2 {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+contract Generic {
+
+    struct NotUsedStruct3 {
+        uint256 three;
+        uint256 four;
+    }
+
+    enum Status3     {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+    struct NotUsedStruct4 {
+        uint256 three;
+        uint256 four;
+    }
+
+    enum Status4 {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+    constructor() {}
+
+    function function1() pure external {
+        uint af1 = 0;
+        af1;
+    }
+
+    function function2() external pure {
+        uint b;
+        b;
+    }
+}

e2e/08-autofix/contract-name-camelcase/Foo1BeforeFix.sol

@@ -0,0 +1,61 @@
+// SPDX-License-Identifier: Apache-2.0
+pragma solidity ^0.8.0;
+
+struct not_Used_Struct1 {
+        uint256 three;
+        uint256 four;
+}
+
+enum __status1_ {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+struct NotUsedStruct2 {
+        uint256 three;
+        uint256 four;
+}
+
+enum Status2 {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+contract __generic {
+
+    struct not_UsedStruct3__ {
+        uint256 three;
+        uint256 four;
+    }
+
+    enum status3     {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+    struct NotUsedStruct4 {
+        uint256 three;
+        uint256 four;
+    }
+
+    enum Status4 {
+        Pending,
+        Approved,
+        Rejected
+    }
+
+    constructor() {}
+
+    function function1() pure external {
+        uint af1 = 0;
+        af1;
+    }
+
+    function function2() external pure {
+        uint b;
+        b;
+    }
+}

e2e/08-autofix/event-name-camelcase/.solhint.json

@@ -0,0 +1,5 @@
+{
+  "rules": {
+    "event-name-camelcase": "error"
+  }
+}

e2e/08-autofix/event-name-camelcase/Foo1.sol

@@ -0,0 +1,33 @@
+// SPDX-License-Identifier: Apache-2.0
+pragma solidity ^0.8.0;
+
+contract Generic {
+
+    event not_UsedEvent1(uint256 indexed amount, address account);
+
+    event __not_UsedEvent___2(uint256 indexed amount, address indexed account);
+
+    event not___UsedEvent3(uint256 amount, address account);
+
+    event notUsedEvent4(uint256 indexed amount, address indexed account);
+
+    event notUsedEvent5____(uint256 indexed amount, address account);
+
+    event __not_UsedEvent6__(uint256 indexed amount, address account);
+
+    event OkEvent1(uint256 indexed amount, address account);
+
+    event OkEventOk2(uint256 indexed amount, address indexed account);
+
+    constructor() {}
+
+    function function1() external pure {
+        uint af1 = 0;
+        af1;
+    }
+
+    function function2() external pure {
+        uint b;
+        b;
+    }
+}

e2e/08-autofix/event-name-camelcase/Foo1AfterFix.sol

@@ -0,0 +1,33 @@
+// SPDX-License-Identifier: Apache-2.0
+pragma solidity ^0.8.0;
+
+contract Generic {
+
+    event NotUsedEvent1(uint256 indexed amount, address account);
+
+    event NotUsedEvent2(uint256 indexed amount, address indexed account);
+
+    event NotUsedEvent3(uint256 amount, address account);
+
+    event NotUsedEvent4(uint256 indexed amount, address indexed account);
+
+    event NotUsedEvent5(uint256 indexed amount, address account);
+
+    event NotUsedEvent6(uint256 indexed amount, address account);
+
+    event OkEvent1(uint256 indexed amount, address account);
+
+    event OkEventOk2(uint256 indexed amount, address indexed account);
+
+    constructor() {}
+
+    function function1() external pure {
+        uint af1 = 0;
+        af1;
+    }
+
+    function function2() external pure {
+        uint b;
+        b;
+    }
+}

e2e/08-autofix/event-name-camelcase/Foo1BeforeFix.sol

@@ -0,0 +1,33 @@
+// SPDX-License-Identifier: Apache-2.0
+pragma solidity ^0.8.0;
+
+contract Generic {
+
+    event not_UsedEvent1(uint256 indexed amount, address account);
+
+    event __not_UsedEvent___2(uint256 indexed amount, address indexed account);
+
+    event not___UsedEvent3(uint256 amount, address account);
+
+    event notUsedEvent4(uint256 indexed amount, address indexed account);
+
+    event notUsedEvent5____(uint256 indexed amount, address account);
+
+    event __not_UsedEvent6__(uint256 indexed amount, address account);
+
+    event OkEvent1(uint256 indexed amount, address account);
+
+    event OkEventOk2(uint256 indexed amount, address indexed account);
+
+    constructor() {}
+
+    function function1() external pure {
+        uint af1 = 0;
+        af1;
+    }
+
+    function function2() external pure {
+        uint b;
+        b;
+    }
+}