code-quality-lib 2.5.0 → 2.6.0

package/README.md

@@ -75,10 +75,6 @@ Let's configure your quality checks!
 📦 Detected package manager: npm
 Use npm? (Y/n):
 
-⚙️  Use project config files (.eslintrc, .prettierrc, etc.)?
-Answer "No" to use bundled configs from code-quality-lib
-Use project configs? (y/N):
-
 🔧 Select tools to run (default = all checked):
 [✓] TypeScript? (Y/n):
 [✓] ESLint? (Y/n):
@@ -86,15 +82,16 @@ Use project configs? (y/N):
 [✓] Knip? (Y/n):
 [✓] Snyk? (Y/n):
 
-🌍 Load .env file before running checks?
-Load .env? (Y/n):
+🌍 Set up environment-specific tool sets?
+This allows different tools for development vs CI/CD
+Configure environments? (y/N):
 
 📋 Configuration Summary:
 ──────────────────────────────────────────────────
 📦 Package Manager: npm
-⚙️  Config: Bundled configs
+⚙️  Config: Project configs (detected)
 🔧 Tools: TypeScript, ESLint, Prettier, Knip, Snyk
-🌍 Load .env: Yes
+🌍 Load .env: Yes (always)
 ──────────────────────────────────────────────────
 Run checks with these settings? (Y/n):
 ```
@@ -104,7 +101,8 @@ Run checks with these settings? (Y/n):
 - **Remember settings** — First run creates `.code-quality.json`, future runs skip questions
 - **Yes/No questions** — Simple Y/n prompts with sensible defaults
 - **Checkbox-style tools** — Each tool can be individually enabled/disabled
-- **Bundled configs default** — Uses library's built-in configs by default
+- **Always uses project configs** — Automatically detects and uses your existing ESLint/Prettier configs
+- **Always loads .env** — Environment variables are always available for your tools
 
 After confirmation, it runs the quality checks with your selected settings.
 
@@ -185,7 +183,6 @@ Or configure environments in `.code-quality/config.json`:
 
 ```json
 {
-  "version": "1.0.0",
   "environments": {
     "development": {
       "tools": ["ESLint", "TypeScript", "Prettier"]
@@ -197,9 +194,7 @@ Or configure environments in `.code-quality/config.json`:
       "tools": ["ESLint", "TypeScript", "Prettier", "Knip", "Snyk"]
     }
   },
-  "packageManager": "npm",
-  "useProjectConfig": true,
-  "loadEnv": true
+  "packageManager": "npm"
 }
 ```
 
@@ -234,52 +229,32 @@ const checker = new CodeQualityChecker(config)
 await checker.run()
 ```
 
-### Config Modes
+### Configuration
 
-**Use Project Configs (Default)**
+The library automatically detects and uses your project's existing configuration files (`.eslintrc`, `.prettierrc`, `tsconfig.json`, etc.) if they exist. If no project configs are found, it uses bundled configurations.
 
-```json
-{
-  "useProjectConfig": true
-}
-```
-
-Uses your project's existing config files (`.eslintrc.js`, `.prettierrc`, `tsconfig.json`, etc.)
-
-**Use Reference Configs**
-
-```json
-{
-  "useProjectConfig": false
-}
-```
-
-Uses reference configs from `.code-quality/` directory as starting point for new projects
+**Environment variables** from `.env` files are always loaded automatically.
 
 ## Library Usage
 
 ```javascript
 const { CodeQualityChecker, runQualityCheck } = require('code-quality-lib')
 
-// Quick — run all checks with defaults (uses project's config files)
+// Quick — run all checks with defaults (auto-detects project configs)
 const result = await runQualityCheck()
 console.log(result.success ? 'All passed' : 'Some failed')
 
-// Use bundled configs instead of project's configs
-const checker = new CodeQualityChecker({
-  useProjectConfig: false, // use library's bundled .eslintrc, .prettierrc, etc.
-})
-await checker.run()
-
-// Custom — select tools, override commands
+// Advanced — custom configuration
 const customChecker = new CodeQualityChecker({
-  tools: ['TypeScript', 'ESLint'],
-  packageManager: 'pnpm',
+  environments: {
+    development: { tools: ['ESLint', 'TypeScript'] },
+    ci: { tools: ['ESLint', 'TypeScript', 'Prettier', 'Knip', 'Snyk'] },
+  },
+  packageManager: 'npm',
   commands: {
     TypeScript: 'tsc --noEmit',
     ESLint: 'eslint src/ --ext .ts,.tsx',
   },
-  loadEnv: false,
 })
 
 const result = await customChecker.run({ showLogs: true })
@@ -290,12 +265,10 @@ console.log(result.results) // per-tool results array
 
 | Option             | Type                                 | Default       | Description                                                                                 |
 | ------------------ | ------------------------------------ | ------------- | ------------------------------------------------------------------------------------------- |
-| `useProjectConfig` | `boolean`                            | `true`        | Use project's config files (`.eslintrc.js`, `.prettierrc`, etc.) instead of bundled configs |
-| `tools`            | `string[]`                           | All 5 tools   | Which tools to run                                                                          |
+| `tools`            | `string[]`                           | All 5 tools   | Which tools to run (deprecated - use environments instead)                                    |
 | `packageManager`   | `'npm' \| 'bun' \| 'pnpm' \| 'yarn'` | auto-detected | Force a specific package manager                                                            |
 | `commands`         | `Record<string, string>`             | bundled paths | Custom commands per tool                                                                    |
 | `descriptions`     | `Record<string, string>`             | built-in      | Custom descriptions per tool                                                                |
-| `loadEnv`          | `boolean`                            | `true`        | Load `.env` file                                                                            |
 | `environment`      | `string`                             | auto-detected | Override environment (development, ci, production)                                          |
 | `environments`     | `Record<string, EnvironmentConfig>`  | -             | Environment-specific tool configurations                                                    |
 

package/{.code-quality → config}/config.json

@@ -1,9 +1,5 @@
 {
-  "version": "1.0.0",
-  "packageManager": "npm",
-  "useProjectConfig": false,
-  "loadEnv": true,
-  "generated": "2026-03-11T16:57:59.790Z",
+  "packageManager": "bun",
   "environments": {
     "development": {
       "tools": [
@@ -31,4 +27,4 @@
       ]
     }
   }
-}
+}

package/{.code-quality → config}/tsconfig.json

@@ -23,10 +23,6 @@
       "@/*": ["src/*"]
     }
   },
-  "include": [
-    "**/*.ts",
-    "**/*.tsx",
-    "**/*.mts"
-  ],
+  "include": ["**/*.ts", "**/*.tsx", "**/*.mts"],
   "exclude": ["node_modules", "dist", "build", "coverage", "examples"]
 }

package/index.js

@@ -183,8 +183,6 @@ class CodeQualityChecker {
       options.environment || process.env.NODE_ENV || process.env.CODE_QUALITY_ENV || 'development'
 
     this.options = {
-      loadEnv: options.loadEnv !== false,
-      useProjectConfig: options.useProjectConfig !== false,
       tools: this._resolveToolsForEnvironment(options.tools, options.environments, env),
       commands: options.commands || {},
       descriptions: options.descriptions || {},
@@ -193,10 +191,11 @@ class CodeQualityChecker {
       environments: options.environments || {},
     }
 
-    if (this.options.loadEnv) loadEnvFile()
+    // Always load .env file
+    loadEnvFile()
 
-    // Detect project configs if useProjectConfig is enabled
-    this.projectConfigs = this.options.useProjectConfig ? detectProjectConfigs() : {}
+    // Always detect and use project configs if they exist
+    this.projectConfigs = detectProjectConfigs()
   }
 
   _resolveToolsForEnvironment(tools, environments, env) {
@@ -239,17 +238,15 @@ class CodeQualityChecker {
         const binPath = path.join(binDir, defaultTool.bin)
         let args = defaultTool.args
 
-        // Add config flags if project configs exist and useProjectConfig is true
-        if (this.options.useProjectConfig) {
-          if (toolName === 'ESLint' && this.projectConfigs.eslint) {
-            args = `${args} --config ${this.projectConfigs.eslint}`
-          } else if (toolName === 'Prettier' && this.projectConfigs.prettier) {
-            args = `${args} --config ${this.projectConfigs.prettier}`
-          } else if (toolName === 'TypeScript' && this.projectConfigs.typescript) {
-            args = `--project ${this.projectConfigs.typescript} --noEmit`
-          } else if (toolName === 'Knip' && this.projectConfigs.knip) {
-            args = `--config ${this.projectConfigs.knip}`
-          }
+        // Add config flags if project configs exist
+        if (toolName === 'ESLint' && this.projectConfigs.eslint) {
+          args = `${args} --config ${this.projectConfigs.eslint}`
+        } else if (toolName === 'Prettier' && this.projectConfigs.prettier) {
+          args = `${args} --config ${this.projectConfigs.prettier}`
+        } else if (toolName === 'TypeScript' && this.projectConfigs.typescript) {
+          args = `--project ${this.projectConfigs.typescript} --noEmit`
+        } else if (toolName === 'Knip' && this.projectConfigs.knip) {
+          args = `--config ${this.projectConfigs.knip}`
         }
 
         cmd = `${binPath}${args ? ' ' + args : ''}`
@@ -552,28 +549,7 @@ class CodeQualityChecker {
     const showLogs = options.showLogs || false
     const useFix = options.useFix || false
 
-    // Load .env file if enabled
-    if (this.options.loadEnv) {
-      try {
-        const dotenvPath = path.join(process.cwd(), '.env')
-        if (fs.existsSync(dotenvPath)) {
-          // Try to load dotenv as optional dependency
-          let dotenv
-          try {
-            dotenv = require('dotenv')
-          } catch (_dotenvError) {
-            // dotenv not installed, skip .env loading
-            console.log('⚠️  dotenv not installed. Install with: npm install dotenv')
-          }
-
-          if (dotenv) {
-            dotenv.config({ path: dotenvPath })
-          }
-        }
-      } catch (_error) {
-        // .env file missing or other error, continue without it
-      }
-    }
+    // .env file is already loaded in constructor
 
     const checks = this._getChecks()
     const pm = this.options.packageManager
@@ -587,9 +563,7 @@ class CodeQualityChecker {
     console.log('─'.repeat(50))
     console.log(`📦 Package Manager: ${pm}`)
     console.log(`🌍 Environment: ${this.options.environment}`)
-    console.log(
-      `⚙️  Config: ${this.options.useProjectConfig ? 'Project configs' : 'Bundled configs'}`
-    )
+    console.log(`⚙️  Config: Project configs (detected)`)
     console.log(`🔧 Tools: ${checks.length} quality checks\n`)
 
     if (showLogs) {
@@ -839,15 +813,52 @@ async function runQualityCheck(options = {}) {
 
 function initConfigFiles() {
   const rootDir = process.cwd()
-  const libConfigDir = path.join(__dirname, 'config')
+  
+  // Try multiple approaches to find the config directory
+  let libConfigDir
+  
+  // Method 1: Use __filename if it points to the actual script
+  if (__filename !== '[eval]' && !__filename.includes('bunx')) {
+    const scriptPath = path.resolve(__filename)
+    const packageDir = path.dirname(scriptPath)
+    libConfigDir = path.join(packageDir, 'config')
+  }
+  
+  // Method 2: Try to find the package via require.resolve
+  if (!libConfigDir || !fs.existsSync(libConfigDir)) {
+    try {
+      const packagePath = path.dirname(require.resolve('code-quality-lib/package.json'))
+      libConfigDir = path.join(packagePath, 'config')
+    } catch (e) {
+      // Continue to next method
+    }
+  }
+  
+  // Method 3: Use global npm path
+  if (!libConfigDir || !fs.existsSync(libConfigDir)) {
+    const npmPrefix = process.env.npm_config_prefix || '/opt/homebrew'
+    libConfigDir = path.join(npmPrefix, 'lib', 'node_modules', 'code-quality-lib', 'config')
+  }
+  
+  // Method 4: Fallback to local config directory
+  if (!libConfigDir || !fs.existsSync(libConfigDir)) {
+    libConfigDir = path.join(__dirname, 'config')
+  }
 
   console.log('\n🚀 Initializing config files in root directory...\n')
 
+  // Create .code-quality directory if it doesn't exist
+  const codeQualityDir = path.join(rootDir, '.code-quality')
+  if (!fs.existsSync(codeQualityDir)) {
+    fs.mkdirSync(codeQualityDir, { recursive: true })
+  }
+
   const configFiles = [
     { src: 'eslint.config.mjs', dest: 'eslint.config.mjs', desc: 'ESLint configuration' },
     { src: 'tsconfig.json', dest: 'tsconfig.json', desc: 'TypeScript configuration' },
     { src: '.prettierrc', dest: '.prettierrc', desc: 'Prettier configuration' },
     { src: '.prettierignore', dest: '.prettierignore', desc: 'Prettier ignore patterns' },
+    { src: 'config.json', dest: '.code-quality/config.json', desc: 'Code quality configuration' },
   ]
 
   let copied = 0
@@ -894,8 +905,6 @@ function generateConfigFile() {
   }
 
   const config = {
-    version: '1.0.0',
-    tools: undefined, // Use environment-based tools
     environments: {
       development: {
         tools: ['ESLint', 'TypeScript', 'Prettier'],
@@ -908,23 +917,20 @@ function generateConfigFile() {
       },
     },
     packageManager: detectPackageManager(),
-    useProjectConfig: true,
-    loadEnv: true,
     commands: {
       ESLint: '. --ext .js,.jsx,.ts,.tsx',
       TypeScript: 'tsc --noEmit',
-      Prettier: '--check .',
-      Knip: '',
-      Snyk: 'test --severity-threshold=high',
+      Prettier: '. --check',
+      Knip: 'check',
+      Snyk: 'test',
     },
     descriptions: {
-      ESLint: 'Code linting and style checking',
-      TypeScript: 'Type checking and compilation',
-      Prettier: 'Code formatting validation',
+      ESLint: 'JavaScript/TypeScript linting',
+      TypeScript: 'TypeScript type checking',
+      Prettier: 'Code formatting',
       Knip: 'Dead code detection',
       Snyk: 'Security vulnerability scanning',
     },
-    generated: new Date().toISOString(),
   }
 
   try {
@@ -1064,13 +1070,8 @@ async function runWizard() {
   if (existingConfig) {
     console.log('\n📋 Found existing configuration:')
     console.log(`📦 Package Manager: ${existingConfig.packageManager || 'auto-detected'}`)
-    console.log(
-      `⚙️  Config: ${existingConfig.useProjectConfig !== false ? 'Project configs' : 'Bundled configs'}`
-    )
-    console.log(
-      `🔧 Tools: ${(existingConfig.tools || ['ESLint', 'TypeScript', 'Prettier', 'Knip', 'Snyk']).join(', ')}`
-    )
-    console.log(`🌍 Load .env: ${existingConfig.loadEnv !== false ? 'Yes' : 'No'}`)
+    console.log(`⚙️  Config: Project configs (detected)`)
+    console.log(`🔧 Tools: ${(existingConfig.tools || ['ESLint', 'TypeScript', 'Prettier', 'Knip', 'Snyk']).join(', ')}`)
 
     const rerun = await askQuestion(rl, '\nReconfigure? (y/N): ')
     if (!rerun.toLowerCase().startsWith('y')) {
@@ -1094,13 +1095,7 @@ async function runWizard() {
     ? await askQuestion(rl, 'Enter package manager (npm/bun/pnpm/yarn): ')
     : pm
 
-  // Step 2: Config Type
-  console.log('\n⚙️  Use project config files (.eslintrc, .prettierrc, etc.)?')
-  console.log('Answer "No" to use bundled configs from code-quality-lib')
-  const configAnswer = await askQuestion(rl, 'Use project configs? (y/N): ')
-  const useProjectConfig = configAnswer.toLowerCase().startsWith('y')
-
-  // Step 3: Tools Selection (Checkbox style)
+  // Step 2: Tools Selection (Checkbox style)
   console.log('\n🔧 Select tools to run (default = all checked):')
   const allTools = ['ESLint', 'TypeScript', 'Prettier', 'Knip', 'Snyk']
   const selectedTools = []
@@ -1124,37 +1119,35 @@ async function runWizard() {
     }
   }
 
-  // Step 4: Environment Configuration
+  // Always create environments configuration
+  let environments = {
+    development: { tools: selectedTools },
+    ci: { tools: [...selectedTools, 'Knip', 'Snyk'] },
+    production: { tools: [...selectedTools, 'Knip', 'Snyk'] },
+  }
+
+  // Step 4: Optional environment customization
   console.log('\n🌍 Set up environment-specific tool sets?')
   console.log('This allows different tools for development vs CI/CD')
   const envConfigAnswer = await askQuestion(rl, 'Configure environments? (y/N): ')
   const configureEnvironments = envConfigAnswer.toLowerCase().startsWith('y')
 
-  let environments = {}
   if (configureEnvironments) {
     console.log('\n🔧 Configure development tools (default: ESLint, TypeScript, Prettier):')
     const devTools = []
     for (const tool of allTools) {
-      const isDefaultYes = ['ESLint', 'TypeScript', 'Prettier'].includes(tool)
-      const prompt = isDefaultYes ? `[✓] ${tool}? (Y/n): ` : `[ ] ${tool}? (y/N): `
-      const answer = await askQuestion(rl, prompt)
-
-      if (isDefaultYes) {
-        if (!answer.toLowerCase().startsWith('n')) {
-          devTools.push(tool)
-        }
-      } else {
-        if (answer.toLowerCase().startsWith('y')) {
-          devTools.push(tool)
-        }
+      const answer = await askQuestion(rl, `[${selectedTools.includes(tool) ? '✓' : ' '}] ${tool}? (Y/n): `)
+      if (!answer.toLowerCase().startsWith('n')) {
+        devTools.push(tool)
       }
     }
 
-    console.log('\n🚀 Configure CI/CD tools (default: all tools):')
+    console.log('\n🚀 Configure CI/CD tools (default: ESLint, TypeScript, Prettier, Knip, Snyk):')
     const ciTools = []
     for (const tool of allTools) {
-      const answer = await askQuestion(rl, `[✓] ${tool}? (Y/n): `)
-      if (!answer.toLowerCase().startsWith('n')) {
+      const defaultChecked = ['Knip', 'Snyk'].includes(tool) || selectedTools.includes(tool)
+      const answer = await askQuestion(rl, `[${defaultChecked ? '✓' : ' '}] ${tool}? (${defaultChecked ? 'Y/n' : 'y/N'}): `)
+      if ((defaultChecked && !answer.toLowerCase().startsWith('n')) || (!defaultChecked && answer.toLowerCase().startsWith('y'))) {
         ciTools.push(tool)
       }
     }
@@ -1166,16 +1159,11 @@ async function runWizard() {
     }
   }
 
-  // Step 5: Load .env
-  console.log('\n🌍 Load .env file before running checks?')
-  const envAnswer = await askQuestion(rl, 'Load .env? (Y/n): ')
-  const loadEnv = !envAnswer.toLowerCase().startsWith('n')
-
-  // Step 6: Show summary and confirm
+  // Step 5: Show summary and confirm
   console.log('\n📋 Configuration Summary:')
   console.log('─'.repeat(50))
   console.log(`📦 Package Manager: ${selectedPm}`)
-  console.log(`⚙️  Config: ${useProjectConfig ? 'Project configs' : 'Bundled configs'}`)
+  console.log(`⚙️  Config: Project configs (detected)`)
 
   if (configureEnvironments) {
     console.log(`🌍 Environment Config: Enabled`)
@@ -1185,7 +1173,7 @@ async function runWizard() {
     console.log(`🔧 Tools: ${selectedTools.join(', ')}`)
   }
 
-  console.log(`🌍 Load .env: ${loadEnv ? 'Yes' : 'No'}`)
+  console.log(`🌍 Load .env: Yes (always)`)
   console.log('─'.repeat(50))
 
   const confirm = await askQuestion(rl, 'Run checks with these settings? (Y/n): ')
@@ -1199,13 +1187,8 @@ async function runWizard() {
 
   // Save config for next time in .code-quality/ directory
   const config = {
-    version: '1.0.0',
     packageManager: selectedPm,
-    useProjectConfig,
-    tools: configureEnvironments ? undefined : selectedTools,
-    environments: configureEnvironments ? environments : undefined,
-    loadEnv,
-    generated: new Date().toISOString(),
+    environments: environments, // Always include environments
   }
 
   try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "code-quality-lib",
-  "version": "2.5.0",
+  "version": "2.6.0",
   "description": "A configurable code quality checker library for Node.js projects",
   "main": "index.js",
   "types": "index.d.ts",
@@ -51,6 +51,7 @@
   "files": [
     "index.js",
     "index.d.ts",
+    "config/",
     ".code-quality/",
     "README.md",
     "LICENSE"