Revert "Refactored case sniffs to allow choosing snake or camel case."

This reverts commit 6e7350e.

Author: AlexSkrypnyk

CLAUDE.md

@@ -4,12 +4,11 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 
 ## Project Overview
 
-This is a PHP_CodeSniffer (PHPCS) standard package (`drevops/phpcs-standard`) that enforces custom coding conventions, specifically focused on configurable naming (snakeCase or camelCase) for local variables and parameters.
+This is a PHP_CodeSniffer (PHPCS) standard package (`drevops/phpcs-standard`) that enforces custom coding conventions, specifically focused on snake_case naming for local variables and parameters.
 
 **Key Goals:**
-- Enforce consistent naming conventions for local variables and function/method parameters
-- Support configurable formats: snakeCase (default) or camelCase
-- Exclude class properties from naming enforcement (properties follow different conventions)
+- Enforce snake_case for local variables and function/method parameters
+- Exclude class properties from snake_case enforcement (properties follow different conventions)
 - Preserve inherited parameter names from interfaces and parent classes
 - Provide auto-fixing support via `phpcbf`
 - Provide a standalone, reusable PHPCS standard for the DrevOps ecosystem
@@ -50,8 +49,8 @@ Coverage reports are generated in:
 Run a single test file:
 ```bash
 ./vendor/bin/phpunit tests/Unit/AbstractVariableSnakeCaseSniffTest.php
-./vendor/bin/phpunit tests/Unit/LocalVariableNamingSniffTest.php
-./vendor/bin/phpunit tests/Unit/ParameterNamingSniffTest.php
+./vendor/bin/phpunit tests/Unit/LocalVariableSnakeCaseSniffTest.php
+./vendor/bin/phpunit tests/Unit/ParameterSnakeCaseSniffTest.php
 ```
 
 Run only unit tests or functional tests:
@@ -87,15 +86,15 @@ composer normalize --dry-run
 ### Directory Structure
 - `src/DrevOps/` - Source code for the PHPCS standard
   - `Sniffs/NamingConventions/`
-    - `AbstractVariableNamingSniff.php` - Base class with shared functionality
-    - `LocalVariableNamingSniff.php` - Enforces snake_case for local variables
-    - `ParameterNamingSniff.php` - Enforces snake_case for parameters
+    - `AbstractSnakeCaseSniff.php` - Base class with shared functionality
+    - `LocalVariableSnakeCaseSniff.php` - Enforces snake_case for local variables
+    - `ParameterSnakeCaseSniff.php` - Enforces snake_case for parameters
   - `ruleset.xml` - DrevOps standard definition
 - `tests/` - PHPUnit tests organized by type:
   - `Unit/` - Unit tests for individual sniff methods (using reflection)
     - `AbstractVariableSnakeCaseSniffTest.php` - Tests shared base class methods
-    - `LocalVariableNamingSniffTest.php` - Tests local variable sniff
-    - `ParameterNamingSniffTest.php` - Tests parameter sniff
+    - `LocalVariableSnakeCaseSniffTest.php` - Tests local variable sniff
+    - `ParameterSnakeCaseSniffTest.php` - Tests parameter sniff
     - `UnitTestCase.php` - Base test class with helper methods
   - `Functional/` - Integration tests that run actual phpcs commands
   - `Fixtures/` - Test fixture files with intentional violations
@@ -106,22 +105,15 @@ composer normalize --dry-run
 
 The standard uses an **abstract base class pattern** with two concrete implementations:
 
-#### AbstractVariableNamingSniff
+#### AbstractSnakeCaseSniff
 
-Base class (src/DrevOps/Sniffs/NamingConventions/AbstractVariableNamingSniff.php) containing shared functionality:
-
-**Public property:**
-- `$format` - Configurable naming convention ('snakeCase' or 'camelCase', default: 'snakeCase')
+Base class (src/DrevOps/Sniffs/NamingConventions/AbstractSnakeCaseSniff.php) containing shared functionality:
 
 **Core methods:**
 - `register()` - Registers T_VARIABLE token for processing
 - `isReserved()` - Identifies PHP reserved variables ($this, $_GET, etc.)
 - `isSnakeCase()` - Validates snake_case format using regex
-- `isCamelCase()` - Validates camelCase format using regex
-- `isValidFormat()` - Validates variable name against configured format
-- `toSnakeCase()` - Converts variable name to snake_case
-- `toCamelCase()` - Converts variable name to camelCase
-- `toFormat()` - Converts variable name to the configured format
+- `toSnakeCase()` - Converts camelCase to snake_case for suggestions
 
 **Helper methods:**
 - `getParameterNames()` - Extracts parameter names from function signature
@@ -132,33 +124,29 @@ Base class (src/DrevOps/Sniffs/NamingConventions/AbstractVariableNamingSniff.php
 - `isProperty()` - Distinguishes class properties from local variables
 - `isInheritedParameter()` - Detects parameters from interfaces/parent classes
 
-#### LocalVariableNamingSniff
+#### LocalVariableSnakeCaseSniff
 
-Enforces configurable naming convention for **local variables** inside functions/methods.
+Enforces snake_case for **local variables** inside functions/methods.
 
 **What gets checked:**
 - ✅ Local variables inside function/method bodies
-- ❌ Function/method parameters (handled by ParameterNaming)
+- ❌ Function/method parameters (handled by ParameterSnakeCase)
 - ❌ Class properties (not enforced)
 - ❌ Reserved PHP variables ($this, superglobals, etc.)
 
-**Error codes:**
-- `DrevOps.NamingConventions.LocalVariableNaming.NotSnakeCase` (when format='snakeCase')
-- `DrevOps.NamingConventions.LocalVariableNaming.NotCamelCase` (when format='camelCase')
+**Error code:** `DrevOps.NamingConventions.LocalVariableSnakeCase.NotSnakeCase`
 
-#### ParameterNamingSniff
+#### ParameterSnakeCaseSniff
 
-Enforces configurable naming convention for **function/method parameters**.
+Enforces snake_case for **function/method parameters**.
 
 **What gets checked:**
 - ✅ Function and method parameters (in signature only)
-- ❌ Local variables (handled by LocalVariableNaming)
+- ❌ Local variables (handled by LocalVariableSnakeCase)
 - ❌ Parameters inherited from interfaces/parent classes/abstract methods
 - ❌ Promoted constructor properties
 
-**Error codes:**
-- `DrevOps.NamingConventions.ParameterNaming.NotSnakeCase` (when format='snakeCase')
-- `DrevOps.NamingConventions.ParameterNaming.NotCamelCase` (when format='camelCase')
+**Error code:** `DrevOps.NamingConventions.ParameterSnakeCase.NotSnakeCase`
 
 ### PHPCS Standard Registration
 
@@ -181,16 +169,16 @@ Tests are organized by class hierarchy:
 **AbstractVariableSnakeCaseSniffTest.php**
 - Tests all shared base class methods using reflection
 - Tests: `isSnakeCase()`, `toSnakeCase()`, `isReserved()`, `register()`, `getParameterNames()`, `isProperty()`, `isPromotedProperty()`, `isInheritedParameter()`
-- Each test uses concrete sniff instances (LocalVariableNamingSniff or ParameterNamingSniff) to access protected methods
+- Each test uses concrete sniff instances (LocalVariableSnakeCaseSniff or ParameterSnakeCaseSniff) to access protected methods
 
-**LocalVariableNamingSniffTest.php**
+**LocalVariableSnakeCaseSniffTest.php**
 - Tests sniff-specific logic: error code constant and `process()` method
-- Configured to run only LocalVariableNaming sniff in isolation
+- Configured to run only LocalVariableSnakeCase sniff in isolation
 - Validates that local variables are checked and parameters are skipped
 
-**ParameterNamingSniffTest.php**
+**ParameterSnakeCaseSniffTest.php**
 - Tests sniff-specific logic: error code constant, `register()`, and `process()` method
-- Configured to run only ParameterNaming sniff in isolation
+- Configured to run only ParameterSnakeCase sniff in isolation
 - Validates that parameters are checked and local variables are skipped
 - Includes tests for inherited parameter detection
 
@@ -202,15 +190,15 @@ Tests are organized by class hierarchy:
 
 #### 2. Functional Tests
 
-**LocalVariableNamingSniffFunctionalTest.php**
+**LocalVariableSnakeCaseSniffFunctionalTest.php**
 - Run actual `phpcs` commands as external processes
 - Test complete PHPCS integration with JSON output parsing
-- Verify LocalVariableNaming sniff detection and error codes
+- Verify LocalVariableSnakeCase sniff detection and error codes
 
-**ParameterNamingSniffFunctionalTest.php**
+**ParameterSnakeCaseSniffFunctionalTest.php**
 - Run actual `phpcs` commands as external processes
 - Test complete PHPCS integration with JSON output parsing
-- Verify ParameterNaming sniff detection and error codes
+- Verify ParameterSnakeCase sniff detection and error codes
 
 Tests include:
 - Confirms violations are detected with correct error codes
@@ -251,7 +239,7 @@ When implementing or modifying sniffs:
 
 1. Place sniff classes in `src/DrevOps/Sniffs/` following PHPCS naming conventions
    - Format: `CategoryName/SniffNameSniff.php`
-   - Example: `NamingConventions/LocalVariableNamingSniff.php`
+   - Example: `NamingConventions/LocalVariableSnakeCaseSniff.php`
 2. Consider using abstract base classes for shared functionality across related sniffs
 3. Implement the `Sniff` interface from `PHP_CodeSniffer\Sniffs\Sniff`
 4. Use `declare(strict_types=1);` at the top of all PHP files
@@ -266,10 +254,8 @@ When implementing or modifying sniffs:
 10. Create fixture files in `tests/Fixtures/` with intentional violations
 11. Follow error code naming: `StandardName.Category.SniffName.ErrorName`
     - Examples:
-      - `DrevOps.NamingConventions.LocalVariableNaming.NotSnakeCase`
-      - `DrevOps.NamingConventions.LocalVariableNaming.NotCamelCase`
-      - `DrevOps.NamingConventions.ParameterNaming.NotSnakeCase`
-      - `DrevOps.NamingConventions.ParameterNaming.NotCamelCase`
+      - `DrevOps.NamingConventions.LocalVariableSnakeCase.NotSnakeCase`
+      - `DrevOps.NamingConventions.ParameterSnakeCase.NotSnakeCase`
 
 ## CI/CD
 

README.md

@@ -19,7 +19,7 @@
 
 ---
 PHP_CodeSniffer standard enforcing:
-- Consistent naming conventions for local variables and function/method parameters (configurable: `snakeCase` or `camelCase`)
+- `snake_case` naming for local variables and function/method parameters
 - PHPUnit data provider naming conventions and organization
 
 ## Installation
@@ -60,8 +60,8 @@ Use individual sniffs:
 ```xml
 <ruleset name="Custom Standards">
   <!-- Naming Conventions -->
-  <rule ref="DrevOps.NamingConventions.LocalVariableNaming"/>
-  <rule ref="DrevOps.NamingConventions.ParameterNaming"/>
+  <rule ref="DrevOps.NamingConventions.LocalVariableSnakeCase"/>
+  <rule ref="DrevOps.NamingConventions.ParameterSnakeCase"/>
 
   <!-- Testing Practices -->
   <rule ref="DrevOps.TestingPractices.DataProviderPrefix"/>
@@ -70,93 +70,55 @@ Use individual sniffs:
 </ruleset>
 ```
 
-### Configure naming convention
+## `LocalVariableSnakeCase`
 
-By default, both sniffs enforce `snakeCase`. Configure to use `camelCase`:
+Enforces `snake_case` for local variables inside functions/methods.
 
-```xml
-<ruleset name="Custom Standards">
-  <rule ref="DrevOps.NamingConventions.LocalVariableNaming">
-    <properties>
-      <property name="format" value="camelCase"/>
-    </properties>
-  </rule>
-
-  <rule ref="DrevOps.NamingConventions.ParameterNaming">
-    <properties>
-      <property name="format" value="camelCase"/>
-    </properties>
-  </rule>
-</ruleset>
-```
-
-## `LocalVariableNaming`
-
-Enforces consistent naming convention for local variables inside functions/methods.
-
-**With `snakeCase` (default):**
 ```php
 function processOrder() {
     $order_id = 1;        // ✓ Valid
-    $orderId = 1;         // ✗ Error: NotSnakeCase
-}
-```
-
-**With `camelCase`:**
-```php
-function processOrder() {
-    $orderId = 1;         // ✓ Valid
-    $order_id = 1;        // ✗ Error: NotCamelCase
+    $orderId = 1;         // ✗ Error: VariableNotSnakeCase
 }
 ```
 
 Excludes:
-- Function/method parameters (handled by `ParameterNaming`)
+- Function/method parameters (handled by `ParameterSnakeCase`)
 - Class properties (not enforced)
 - Reserved variables (`$this`, `$_GET`, `$_POST`, etc.)
 
-### Error codes
+### Error code
 
-- `DrevOps.NamingConventions.LocalVariableNaming.NotSnakeCase` (when `format="snakeCase"`)
-- `DrevOps.NamingConventions.LocalVariableNaming.NotCamelCase` (when `format="camelCase"`)
+`DrevOps.NamingConventions.LocalVariableSnakeCase.NotSnakeCase`
 
 ### Ignore
 
 ```php
-// phpcs:ignore DrevOps.NamingConventions.LocalVariableNaming.NotSnakeCase
+// phpcs:ignore DrevOps.NamingConventions.LocalVariableSnakeCase.NotSnakeCase
 $myVariable = 'value';
 ```
 
-## `ParameterNaming`
+## `ParameterSnakeCase`
 
-Enforces consistent naming convention for function/method parameters.
+Enforces `snake_case` for function/method parameters.
 
-**With `snakeCase` (default):**
 ```php
 function processOrder($order_id, $user_data) {  // ✓ Valid
-function processOrder($orderId, $userData) {    // ✗ Error: NotSnakeCase
-```
-
-**With `camelCase`:**
-```php
-function processOrder($orderId, $userData) {    // ✓ Valid
-function processOrder($order_id, $user_data) {  // ✗ Error: NotCamelCase
+function processOrder($orderId, $userData) {    // ✗ Error: ParameterNotSnakeCase
 ```
 
 Excludes:
 - Parameters inherited from interfaces/parent classes
 - Parameters in interface/abstract method declarations
 - Class properties (including promoted constructor properties)
 
-### Error codes
+### Error code
 
-- `DrevOps.NamingConventions.ParameterNaming.NotSnakeCase` (when `format="snakeCase"`)
-- `DrevOps.NamingConventions.ParameterNaming.NotCamelCase` (when `format="camelCase"`)
+`DrevOps.NamingConventions.ParameterSnakeCase.NotSnakeCase`
 
 ### Ignore
 
 ```php
-// phpcs:ignore DrevOps.NamingConventions.ParameterNaming.NotSnakeCase
+// phpcs:ignore DrevOps.NamingConventions.ParameterSnakeCase.NotSnakeCase
 function process($legacyParam) {}
 ```
 

composer.json

@@ -1,19 +1,8 @@
 {
     "name": "drevops/phpcs-standard",
-    "description": "DrevOps PHP_CodeSniffer rules: enforce consistent naming (snakeCase or camelCase) for variables and parameters, PHPUnit data provider conventions.",
+    "description": "DrevOps PHP_CodeSniffer rules: enforce snake_case for variables and parameters.",
     "license": "GPL-2.0-or-later",
     "type": "phpcodesniffer-standard",
-    "keywords": [
-        "phpcs",
-        "phpcodesniffer",
-        "standards",
-        "snakeCase",
-        "camelCase",
-        "naming-conventions",
-        "code-quality",
-        "phpunit",
-        "data-providers"
-    ],
     "authors": [
         {
             "name": "Alex Skrypnyk",