@dmitryrechkin/eslint-standard 1.4.2 → 1.4.3

package/eslint.config.mjs

@@ -18,7 +18,10 @@ import unicornPlugin from 'eslint-plugin-unicorn';
 import noSecretsPlugin from 'eslint-plugin-no-secrets';
 import regexpPlugin from 'eslint-plugin-regexp';
 import functionalPlugin from 'eslint-plugin-functional';
-
+// eslint-disable-next-line import/no-extraneous-dependencies
+import prettierConfig from 'eslint-config-prettier';
+// eslint-disable-next-line import/no-extraneous-dependencies
+import prettierPlugin from 'eslint-plugin-prettier';
 export default function ({
 	tsconfigPath = './tsconfig.json',
 	ignores = [],
@@ -30,8 +33,10 @@ export default function ({
 		{
 			ignores: ['node_modules/**', 'dist/**', ...ignores],
 		},
+		// Add Prettier integration
+		prettierConfig,
 		{
-			files: ['**/*.{js,jsx,ts,tsx}', ...files],
+			files: ['**/*.{js,ts}', ...files], // Exclude jsx, tsx, and astro files from main config
 			languageOptions: {
 				parser: tsParser,
 				parserOptions: {
@@ -59,6 +64,7 @@ export default function ({
 				'no-secrets': noSecretsPlugin,
 				'regexp': regexpPlugin,
 				'functional': functionalPlugin,
+				'prettier': prettierPlugin,
 				...plugins,
 			},
 			settings: {
@@ -73,6 +79,11 @@ export default function ({
 				'import/extensions': ['.js', '.jsx', '.ts', '.tsx'],
 			},
 			rules: {
+				// Prettier integration
+				'prettier/prettier': 'error',
+				// Disable rules that conflict with prettier
+				'switch-case-brace/switch-case-brace-style': 'off',
+				
 				// Original @dmitryrechkin/eslint-standard rules
 				'@typescript-eslint/explicit-function-return-type': 'error',
 				'@typescript-eslint/no-explicit-any': 'error', // Ban 'any' type for type safety
@@ -540,7 +551,10 @@ export default function ({
 					devDependencies: ['**/*.test.{js,jsx,ts,tsx}', '**/*.spec.{js,jsx,ts,tsx}', '**/test/**', '**/tests/**', '**/__tests__/**']
 				}],
 				'import/no-mutable-exports': 'error',
-				'import/no-unused-modules': 'off', // Disabled due to .eslintrc requirement in flat config
+				'import/no-unused-modules': ['error', {
+					missingExports: true, // Find files with no exports
+					unusedExports: true   // Find unused exports
+				}],
 				'import/unambiguous': 'off',
 				'import/no-commonjs': 'off',
 				'import/no-amd': 'error',
@@ -780,12 +794,39 @@ export default function ({
 				'unicorn/prevent-abbreviations': 'off', // Allow abbreviations for domain-specific terms
 				
 				// Allow custom rules to be added
-				...rules,
+				...rules
 			},
 		},
+		// Prettier config for TypeScript/JavaScript files with Allman brace style
+		{
+			files: ['**/*.{tsx,jsx,ts,js}'],
+			rules: {
+				'prettier/prettier': [
+					'error',
+					{
+						parser: 'typescript',
+						plugins: ['prettier-plugin-brace-style'],
+						braceStyle: 'allman'
+					}
+				]
+			}
+		},
+		// Prettier config for Astro files
+		{
+			files: ['**/*.astro'],
+			rules: {
+				'prettier/prettier': [
+					'error',
+					{
+						parser: 'astro',
+						plugins: ['prettier-plugin-astro']
+					}
+				]
+			}
+		},
 		// Test file specific overrides
 		{
-			files: ['**/*.test.{js,jsx,ts,tsx}', '**/*.spec.{js,jsx,ts,tsx}', '**/tests/**/*.{js,jsx,ts,tsx}'],
+			files: ['**/*.test.{js,ts}', '**/*.spec.{js,ts}', '**/tests/**/*.{js,ts}'],
 			ignores: ['**/tests/fixtures/**/*'], // Don't apply test overrides to fixture files
 			rules: {
 				// Disable function scoping rule for test helpers

package/package.json

@@ -1,32 +1,36 @@
 {
 	"name": "@dmitryrechkin/eslint-standard",
 	"description": "This package provides a shared ESLint configuration which includes TypeScript support and a set of specific linting rules designed to ensure high-quality and consistent code style across projects.",
-	"version": "1.4.2",
+	"version": "1.4.3",
 	"main": "eslint.config.mjs",
+	"exports": {
+		".": "./eslint.config.mjs"
+	},
 	"bin": {
 		"eslint-standard": "./src/cli/index.mjs"
 	},
 	"files": [
 		"eslint.config.mjs",
 		"src/",
-		"docs/",
 		"README.md",
 		"LICENSE"
 	],
 	"scripts": {
 		"postinstall": "node src/cli/postinstall.mjs",
 		"package:publish": "npm publish --access public",
-		"test": "node tests/test-all-rules.js",
-		"test:all": "node tests/test-all-rules.js",
-		"test:formatting": "node tests/test-runner.js",
-		"test:complexity": "node tests/test-complexity-rules.js",
-		"test:safety": "node tests/test-safety-rules.js",
-		"test:security": "node tests/test-security-rules.js",
-		"test:sonarjs": "node tests/test-sonarjs-rules.js",
-		"test:unicorn": "node tests/test-unicorn-rules.js",
-		"test:switch-case": "node tests/test-switch-case-simple.js",
-		"test:cli": "node tests/test-cli.js",
-		"test:install": "node tests/test-install-simulation.js"
+		"test": "node tests/test-all-rules.mjs",
+		"test:all": "node tests/test-all-rules.mjs",
+		"test:formatting": "node tests/test-runner.mjs",
+		"test:simple": "node tests/simple-test.mjs",
+		"test:debug": "node tests/debug-test.mjs",
+		"test:complexity": "node tests/test-complexity-rules.mjs",
+		"test:safety": "node tests/test-safety-rules.mjs",
+		"test:security": "node tests/test-security-rules.mjs",
+		"test:sonarjs": "node tests/test-sonarjs-rules.mjs",
+		"test:unicorn": "node tests/test-unicorn-rules.mjs",
+		"test:switch-case": "node tests/test-switch-case-simple.mjs",
+		"test:cli": "node tests/test-cli.mjs",
+		"test:install": "node tests/test-install-simulation.mjs"
 	},
 	"keywords": [],
 	"author": "",
@@ -36,6 +40,7 @@
 		"@typescript-eslint/eslint-plugin": "^8.0.0",
 		"@typescript-eslint/parser": "^8.0.0",
 		"eslint": "^9.0.0",
+		"eslint-config-prettier": "^10.0.0",
 		"eslint-plugin-functional": "^7.0.0",
 		"eslint-plugin-import": "^2.0.0",
 		"eslint-import-resolver-typescript": "^3.0.0",
@@ -43,19 +48,23 @@
 		"eslint-plugin-jsx-a11y": "^6.0.0",
 		"eslint-plugin-no-secrets": "^1.0.0",
 		"eslint-plugin-perfectionist": "^4.0.0",
+		"eslint-plugin-prettier": "^5.0.0",
 		"eslint-plugin-promise": "^7.0.0",
 		"eslint-plugin-regexp": "^2.0.0",
 		"eslint-plugin-security": "^3.0.0",
 		"eslint-plugin-simple-import-sort": "^12.0.0",
 		"eslint-plugin-sonarjs": "^3.0.0",
 		"eslint-plugin-unicorn": "^56.0.0",
-		"eslint-plugin-unused-imports": "^4.0.0"
+		"eslint-plugin-unused-imports": "^4.0.0",
+		"prettier-plugin-astro": "^0.14.0",
+		"prettier-plugin-brace-style": "^0.8.0"
 	},
 	"devDependencies": {
 		"@stylistic/eslint-plugin": "^5.2.2",
 		"@typescript-eslint/eslint-plugin": "^8.38.0",
 		"@typescript-eslint/parser": "^8.38.0",
 		"eslint": "^9.32.0",
+		"eslint-config-prettier": "^10.1.8",
 		"eslint-plugin-functional": "^7.3.0",
 		"eslint-plugin-import": "^2.32.0",
 		"eslint-import-resolver-typescript": "^3.6.3",
@@ -63,12 +72,15 @@
 		"eslint-plugin-jsx-a11y": "^6.10.2",
 		"eslint-plugin-no-secrets": "^1.1.2",
 		"eslint-plugin-perfectionist": "^4.15.0",
+		"eslint-plugin-prettier": "^5.5.3",
 		"eslint-plugin-promise": "^7.2.1",
 		"eslint-plugin-regexp": "^2.9.0",
 		"eslint-plugin-security": "^3.0.1",
 		"eslint-plugin-simple-import-sort": "^12.1.1",
 		"eslint-plugin-sonarjs": "^3.0.4",
 		"eslint-plugin-unicorn": "^56.0.1",
-		"eslint-plugin-unused-imports": "^4.1.4"
+		"eslint-plugin-unused-imports": "^4.1.4",
+		"prettier-plugin-astro": "^0.14.1",
+		"prettier-plugin-brace-style": "^0.8.1"
 	}
 }

package/src/cli/index.mjs

@@ -20,10 +20,10 @@ Usage:
   npx @dmitryrechkin/eslint-standard <command>
 
 Commands:
-  install-deps         Install all peer dependencies
-  check-deps           Check if all peer dependencies are installed
-  check-deps --install Auto-install missing dependencies if any
-  help                 Show this help message
+  install-deps                  Install all peer dependencies
+  check-deps                    Check if all peer dependencies are installed
+  check-deps --install          Auto-install missing dependencies if any
+  help                          Show this help message
 
 Examples:
   npx @dmitryrechkin/eslint-standard install-deps

package/src/cli/setup-aggressive-cleanup.mjs

@@ -0,0 +1,97 @@
+#!/usr/bin/env node
+
+/**
+ * @file CLI Setup for Aggressive Unused Code Detection
+ * 
+ * This CLI tool helps set up aggressive unused code detection and cleanup
+ * in your project with external tools and optimal configurations.
+ * 
+ * @author PageFast Team
+ * @version 1.0.0
+ */
+
+import { setupAggressiveCleanup } from '../configs/external-tools-setup.mjs';
+
+// Simple command line argument parsing (avoiding external dependencies)
+const args = process.argv.slice(2);
+const options = {
+	global: args.includes('--global') || args.includes('-g'),
+	packageManager: args.find((arg, i) => (args[i-1] === '--package-manager' || args[i-1] === '-p')) || 'npm',
+	dryRun: args.includes('--dry-run'),
+	help: args.includes('--help') || args.includes('-h')
+};
+
+/**
+ * Main CLI program for setting up aggressive cleanup
+ */
+async function main()
+{
+	try
+	{
+		if (options.help)
+		{
+			console.log(`
+@dmitryrechkin/eslint-standard - Aggressive Cleanup Setup
+
+Usage:
+  npx @dmitryrechkin/eslint-standard setup-aggressive-cleanup [options]
+
+Options:
+  -g, --global                 Install cleanup tools globally
+  -p, --package-manager <type> Package manager to use (npm, pnpm, yarn)
+  --dry-run                   Show what would be done without making changes
+  -h, --help                  Show this help message
+
+Examples:
+  npx @dmitryrechkin/eslint-standard setup-aggressive-cleanup
+  npx @dmitryrechkin/eslint-standard setup-aggressive-cleanup --global
+  npx @dmitryrechkin/eslint-standard setup-aggressive-cleanup -p pnpm
+  npx @dmitryrechkin/eslint-standard setup-aggressive-cleanup --dry-run
+			`);
+			return;
+		}
+
+		if (options.dryRun)
+		{
+			console.log('🔍 DRY RUN - Would perform the following actions:\n');
+			console.log('1. Install external cleanup tools:');
+			console.log('   - ts-prune (find unused exports)');
+			console.log('   - unimported (find unused files)'); 
+			console.log('   - knip (advanced dead code elimination)');
+			console.log('   - depcheck (find unused dependencies)');
+			console.log('   - ts-remove-unused (remove unused imports)');
+			console.log('\n2. Add cleanup scripts to package.json');
+			console.log('\n3. Create optimal tsconfig.json for unused code detection');
+			console.log('\n4. Create knip.json configuration');
+			console.log('\nRun without --dry-run to actually perform setup.');
+			return;
+		}
+
+		await setupAggressiveCleanup({
+			packageManager: options.packageManager,
+			globalTools: options.global
+		});
+
+		console.log('\n🎉 Setup complete! Next steps:');
+		console.log('\n1. Enable aggressive cleanup in your ESLint config:');
+		console.log('   ```javascript');
+		console.log('   import baseConfig from "@dmitryrechkin/eslint-standard";');
+		console.log('   export default baseConfig({');
+		console.log('     aggressiveCleanup: true // 🔥 Enable aggressive mode');
+		console.log('   });');
+		console.log('   ```');
+		console.log('\n2. Run cleanup check:');
+		console.log('   npm run cleanup:check');
+		console.log('\n3. Review and fix issues:');
+		console.log('   npm run cleanup:fix');
+
+	}
+	catch (error)
+	{
+		console.error('\n❌ Setup failed:', error.message);
+		process.exit(1);
+	}
+}
+
+// Run the main function
+main();

package/docs/AUTO_INSTALL.md

@@ -1,50 +0,0 @@
-# Auto-Installation Options
-
-## Manual Installation (Default - Recommended)
-
-By default, peer dependencies are NOT auto-installed. After installing the package, run:
-
-```bash
-npx @dmitryrechkin/eslint-standard check-deps --install
-```
-
-## Environment Variable Control
-
-You can control the installation behavior using environment variables:
-
-### Enable Auto-Install
-
-```bash
-# One-time auto-install
-ESLINT_STANDARD_AUTO_INSTALL=true npm install @dmitryrechkin/eslint-standard
-
-# Or add to .npmrc for project
-echo "ESLINT_STANDARD_AUTO_INSTALL=true" >> .npmrc
-```
-
-### Disable All Postinstall Messages
-
-```bash
-# Skip all postinstall scripts
-ESLINT_STANDARD_SKIP_INSTALL=true npm install @dmitryrechkin/eslint-standard
-
-# Or globally
-npm install --ignore-scripts
-```
-
-## CI/CD Environments
-
-The postinstall script automatically detects CI environments and skips execution to avoid issues.
-
-## Security Considerations
-
-- Auto-installation is **opt-in only** via environment variable
-- Respects npm's `--ignore-scripts` flag
-- Skips in CI environments
-- Never modifies files without user consent
-
-## Best Practices
-
-1. **Development**: Use `check-deps --install` for quick setup
-2. **Production**: Explicitly install peer dependencies in package.json
-3. **CI/CD**: Add peer dependencies to your package.json for reproducible builds

package/docs/COMPLEXITY_RULES.md

@@ -1,200 +0,0 @@
-# Built-in Complexity Rules
-
-`@dmitryrechkin/eslint-standard` includes industry-standard complexity rules by default to ensure code maintainability and readability. These rules help detect code quality issues like deep nesting, large functions, and violations of SOLID principles.
-
-## Available Rules
-
-### 1. Cyclomatic Complexity
-```javascript
-'complexity': ['error', 10]
-```
-- Limits the number of linearly independent paths through a function
-- Default: 10 (strict mode: 5)
-- Helps identify functions that are doing too much
-
-### 2. Function Length
-```javascript
-'max-lines-per-function': ['error', {
-    max: 100,
-    skipBlankLines: true,
-    skipComments: true
-}]
-```
-- Limits function length to 100 lines (pragmatic for complex business logic)
-- Encourages reasonably sized, focused functions
-- Balances Single Responsibility with real-world needs
-
-### 3. Maximum Statements
-```javascript
-'max-statements': ['error', 20]
-```
-- Limits the number of statements in a function
-- Default: 20 (strict mode: 10)
-- Forces decomposition of complex logic
-
-### 4. Nesting Depth
-```javascript
-'max-depth': ['error', { max: 3 }]
-```
-- Limits block nesting to 3 levels (strict mode: 2)
-- Prevents deeply nested if/else chains
-- Encourages early returns and guard clauses
-
-### 5. Callback Nesting
-```javascript
-'max-nested-callbacks': ['error', 3]
-```
-- Limits callback nesting depth
-- Encourages use of async/await or promises
-- Prevents "callback hell"
-
-### 6. Parameter Count
-```javascript
-'max-params': ['error', 4]
-```
-- Limits function parameters to 4 (strict mode: 3)
-- Encourages parameter objects for complex functions
-- Improves function signatures
-
-### 7. File Size
-```javascript
-'max-lines': ['warn', {
-    max: 300,
-    skipBlankLines: true,
-    skipComments: true
-}]
-```
-- Warns when files exceed 300 lines
-- Encourages modular code organization
-- Helps maintain Single Responsibility for modules
-
-## Customizing Rules
-
-The complexity rules are included by default. You can customize them by overriding specific rules:
-
-```javascript
-import eslintStandard from '@dmitryrechkin/eslint-standard';
-
-export default eslintStandard({
-    tsconfigPath: './tsconfig.json',
-    rules: {
-        // Make complexity stricter
-        'complexity': ['error', 8],
-        'max-lines-per-function': ['error', { max: 40 }],
-        'max-depth': ['error', { max: 2 }],
-        
-        // Or relax for legacy code
-        'complexity': ['warn', 15],
-        'max-lines-per-function': ['warn', { max: 100 }],
-        
-        // Or disable specific rules
-        'max-lines': 'off'
-    }
-});
-```
-
-## Examples of Violations
-
-### 1. High Cyclomatic Complexity
-```typescript
-// ❌ BAD: Complexity > 10
-function processOrder(order: Order): Result {
-    if (order.status === 'pending') {
-        if (order.payment) {
-            if (order.payment.method === 'credit') {
-                if (order.payment.verified) {
-                    // ... more conditions
-                }
-            } else if (order.payment.method === 'debit') {
-                // ... more branches
-            }
-        }
-    } else if (order.status === 'processing') {
-        // ... more branches
-    }
-    // Total complexity: 15+
-}
-
-// ✅ GOOD: Break into smaller functions
-function processOrder(order: Order): Result {
-    if (!isOrderReady(order)) {
-        return { error: 'Order not ready' };
-    }
-    
-    const payment = processPayment(order.payment);
-    if (!payment.success) {
-        return { error: payment.error };
-    }
-    
-    return completeOrder(order, payment);
-}
-```
-
-### 2. Deep Nesting
-```typescript
-// ❌ BAD: Nesting depth > 3
-function validateData(data: any): boolean {
-    if (data) {
-        if (data.user) {
-            if (data.user.profile) {
-                if (data.user.profile.email) {
-                    return validateEmail(data.user.profile.email);
-                }
-            }
-        }
-    }
-    return false;
-}
-
-// ✅ GOOD: Use early returns
-function validateData(data: any): boolean {
-    if (!data?.user?.profile?.email) {
-        return false;
-    }
-    return validateEmail(data.user.profile.email);
-}
-```
-
-### 3. Long Functions
-```typescript
-// ❌ BAD: Function > 50 lines
-function generateReport(data: Data): Report {
-    // 100+ lines of code doing multiple things:
-    // - Data validation
-    // - Data transformation
-    // - Calculations
-    // - Formatting
-    // - File generation
-}
-
-// ✅ GOOD: Split into focused functions
-function generateReport(data: Data): Report {
-    const validatedData = validateReportData(data);
-    const transformedData = transformReportData(validatedData);
-    const calculations = calculateReportMetrics(transformedData);
-    const formatted = formatReportData(calculations);
-    return createReportFile(formatted);
-}
-```
-
-## Benefits
-
-1. **Improved Readability**: Smaller functions are easier to understand
-2. **Better Testability**: Simple functions are easier to test
-3. **Reduced Bugs**: Less complex code has fewer places for bugs to hide
-4. **Easier Maintenance**: Changes are localized to smaller units
-5. **Team Collaboration**: Consistent complexity limits across the codebase
-
-## Migration Strategy
-
-1. **Start with Warnings**: Use `warn` instead of `error` initially
-2. **Fix Incrementally**: Address the worst violations first
-3. **Set Realistic Goals**: Gradually tighten limits over time
-4. **Document Exceptions**: Use `eslint-disable` comments sparingly with explanations
-
-## Related Tools
-
-- **SonarQube**: For comprehensive code quality metrics
-- **CodeClimate**: For tracking technical debt
-- **Lizard**: For cyclomatic complexity analysis
-- **JSComplexity**: Visual complexity reports