@dmitryrechkin/eslint-standard 1.1.2 → 1.1.3

package/package.json

@@ -1,7 +1,7 @@
 {
 	"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.1.2",
+	"version": "1.1.3",
 	"main": "eslint.config.mjs",
 	"bin": {
 	  "eslint-standard": "./src/cli/index.mjs"
@@ -15,8 +15,10 @@
 	"scripts": {
 	  "postinstall": "node src/cli/postinstall.mjs",
 	  "package:publish": "npm publish --access public",
-	  "test": "npm run test:formatting",
-	  "test:formatting": "node tests/test-runner.js"
+	  "test": "npm run test:formatting && npm run test:cli",
+	  "test:formatting": "node tests/test-runner.js",
+	  "test:cli": "node tests/test-cli.js",
+	  "test:install": "node tests/test-install-simulation.js"
 	},
 	"keywords": [],
 	"author": "",

package/src/cli/check-deps.mjs

@@ -3,20 +3,32 @@
 import { readFileSync } from 'fs';
 import { fileURLToPath } from 'url';
 import { dirname, join, resolve } from 'path';
+import { spawn } from 'child_process';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
 
+// Check for --install flag
+const shouldInstall = process.argv.includes('--install');
+
 // Get peer dependencies
 const packageJsonPath = join(__dirname, '../../package.json');
 const packageJson = JSON.parse(readFileSync(packageJsonPath, 'utf8'));
 const peerDeps = packageJson.peerDependencies || {};
 
-// Check if running from node_modules
+// Check if running from node_modules and find project root
 const isInNodeModules = __dirname.includes('node_modules');
-const projectRoot = isInNodeModules 
-	? resolve(__dirname, '../../../../') 
-	: process.cwd();
+let projectRoot;
+
+if (isInNodeModules) {
+	// Handle different package manager structures
+	// pnpm: .pnpm/@org+pkg@version_deps/node_modules/@org/pkg
+	// npm/yarn: node_modules/@org/pkg
+	const parts = __dirname.split('node_modules');
+	projectRoot = parts[0].replace(/[\\/]$/, ''); // Remove trailing slash
+} else {
+	projectRoot = process.cwd();
+}
 
 // Read project's package.json
 let projectPackageJson;
@@ -56,13 +68,33 @@ if (missingDeps.length === 0 && outdatedDeps.length === 0) {
 } else {
 	if (missingDeps.length > 0) {
 		console.log(`\n❌ Missing ${missingDeps.length} dependencies:`);
-		console.log('Run: npx @dmitryrechkin/eslint-standard install-deps');
+		missingDeps.forEach(dep => console.log(`  - ${dep}`));
+		
+		if (shouldInstall) {
+			console.log('\n🔧 Auto-installing missing dependencies...\n');
+			
+			// Import and run the install-deps script
+			try {
+				const installDepsModule = await import('./install-deps.mjs');
+				// The install-deps script will handle the installation
+			} catch (error) {
+				console.error('❌ Failed to auto-install dependencies:', error.message);
+				process.exit(1);
+			}
+		} else {
+			console.log('\n💡 To install missing dependencies:');
+			console.log('   npx @dmitryrechkin/eslint-standard install-deps');
+			console.log('   or run this command with --install flag');
+		}
 	}
 	if (outdatedDeps.length > 0) {
 		console.log(`\n⚠️  ${outdatedDeps.length} dependencies may be outdated:`);
 		outdatedDeps.forEach(dep => console.log(`  - ${dep}`));
 	}
-	process.exit(1);
+	
+	if (!shouldInstall && missingDeps.length > 0) {
+		process.exit(1);
+	}
 }
 
 export default function checkDeps() {

package/src/cli/index.mjs

@@ -20,13 +20,15 @@ Usage:
   npx @dmitryrechkin/eslint-standard <command>
 
 Commands:
-  install-deps    Install all peer dependencies
-  check-deps      Check if all peer dependencies are installed
-  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
   npx @dmitryrechkin/eslint-standard check-deps
+  npx @dmitryrechkin/eslint-standard check-deps --install
 		`);
 		break;
 	default:

package/src/cli/postinstall-check.mjs

@@ -0,0 +1,54 @@
+#!/usr/bin/env node
+
+import { readFileSync, existsSync } from 'fs';
+import { fileURLToPath } from 'url';
+import { dirname, join } from 'path';
+import { execSync } from 'child_process';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+
+// Don't run during local development
+if (!__dirname.includes('node_modules')) {
+	process.exit(0);
+}
+
+// Check if we're in CI environment
+const isCI = process.env.CI || process.env.CONTINUOUS_INTEGRATION || process.env.GITHUB_ACTIONS;
+
+// Check if scripts are disabled
+const npmConfigIgnoreScripts = process.env.npm_config_ignore_scripts === 'true';
+
+if (isCI || npmConfigIgnoreScripts) {
+	// Silent exit in CI or when scripts are disabled
+	process.exit(0);
+}
+
+// Simple check for missing dependencies
+try {
+	const packageJsonPath = join(__dirname, '../../package.json');
+	const packageJson = JSON.parse(readFileSync(packageJsonPath, 'utf8'));
+	const peerDeps = Object.keys(packageJson.peerDependencies || {});
+	
+	// Quick check if any peer deps are missing
+	let missingCount = 0;
+	for (const dep of peerDeps) {
+		try {
+			// Try to resolve the dependency
+			require.resolve(dep, { paths: [process.cwd()] });
+		} catch {
+			missingCount++;
+		}
+	}
+	
+	if (missingCount > 0) {
+		console.log(`
+⚠️  @dmitryrechkin/eslint-standard: ${missingCount} peer dependencies are missing!
+
+Run this command to check and install them:
+👉 ${'\x1b[36m'}npx @dmitryrechkin/eslint-standard check-deps --install${'\x1b[0m'}
+`);
+	}
+} catch (error) {
+	// Silent fail - don't break installation
+}

package/src/cli/postinstall.mjs

@@ -11,6 +11,17 @@ if (!__dirname.includes('node_modules')) {
 	process.exit(0);
 }
 
+// Check environment variables for auto-install preference
+const autoInstall = process.env.ESLINT_STANDARD_AUTO_INSTALL === 'true';
+const skipInstall = process.env.ESLINT_STANDARD_SKIP_INSTALL === 'true';
+
+// Check if we're in CI environment
+const isCI = process.env.CI || process.env.CONTINUOUS_INTEGRATION || process.env.GITHUB_ACTIONS;
+
+if (skipInstall || isCI) {
+	process.exit(0);
+}
+
 console.log(`
 ╔════════════════════════════════════════════════════════════════╗
 ║                                                                ║
@@ -21,9 +32,9 @@ console.log(`
 📦 This package requires peer dependencies to work properly.
 
 🚀 Quick install (auto-detects your package manager):
-   ${'\x1b[36m'}npx @dmitryrechkin/eslint-standard install-deps${'\x1b[0m'}
+   ${'\x1b[36m'}npx @dmitryrechkin/eslint-standard check-deps --install${'\x1b[0m'}
 
-📋 Or install manually:
+📋 Or install all dependencies directly:
    ${'\x1b[90m'}npm install -D eslint @typescript-eslint/parser @typescript-eslint/eslint-plugin eslint-plugin-unused-imports @stylistic/eslint-plugin eslint-plugin-jsdoc eslint-plugin-simple-import-sort eslint-plugin-perfectionist${'\x1b[0m'}
 
 🔍 To check if all dependencies are installed:
@@ -32,9 +43,22 @@ console.log(`
 📚 Documentation: https://github.com/dmitryrechkin/eslint-standard
 `);
 
-// Run check-deps to show current status
-try {
-	await import('./check-deps.mjs');
-} catch (error) {
-	// Silent fail - don't break installation
+// Don't run check-deps in postinstall to avoid errors
+// Users can run it manually with: npx @dmitryrechkin/eslint-standard check-deps
+
+if (autoInstall) {
+	console.log('\n🤖 Auto-install enabled via ESLINT_STANDARD_AUTO_INSTALL=true');
+	console.log('📦 Checking and installing peer dependencies...\n');
+	
+	try {
+		// Run check-deps with --install flag
+		const { execSync } = await import('child_process');
+		execSync('node ' + join(__dirname, 'index.mjs') + ' check-deps --install', {
+			stdio: 'inherit',
+			cwd: process.cwd()
+		});
+	} catch (error) {
+		console.error('❌ Auto-install failed. Please run manually:');
+		console.error('   npx @dmitryrechkin/eslint-standard check-deps --install');
+	}
 }