@dimensional-innovations/tool-config 2.0.0 → 3.0.1

package/README.md

@@ -5,7 +5,7 @@
 [![npm version](https://img.shields.io/npm/v/@dimensional-innovations/tool-config.svg)](https://www.npmjs.com/package/@dimensional-innovations/tool-config)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Node Version](https://img.shields.io/badge/node-%3E%3D22.12.0-brightgreen.svg)](https://nodejs.org/)
-[![Test Coverage](https://img.shields.io/badge/coverage-100%25-brightgreen.svg)](./tests/)
+[![Test Coverage](https://img.shields.io/badge/coverage-96%25-brightgreen.svg)](./tests/)
 
 **One package. Five tools. Zero configuration.**
 
@@ -35,11 +35,13 @@ npm install --save-dev @dimensional-innovations/tool-config
 
 - 🎯 **Zero Configuration** - Auto-detects your framework, environment, and TypeScript
 - 🧰 **Multi-Tool Support** - ESLint, Prettier, Stylelint, TypeScript, semantic-release in one package
-- ⚛️ **8+ Frameworks** - React, Vue, Svelte, Solid, Astro, Angular, Vanilla JS, Node.js
+- ⚛️ **9 Frameworks** - React, Vue, Svelte, Solid, Astro, Angular, Vanilla JS, Node.js, Electron
 - 📦 **All-In-One** - All plugins and parsers included as dependencies
 - 🔧 **Customizable** - Override any setting while keeping smart defaults
 - 🚀 **Modern** - ESLint 9+ flat config, TypeScript with tsgo support (10x faster)
-- ✅ **Battle-Tested** - 535 tests with 100% coverage
+- 🔄 **Uninstall Support** - Clean removal with safety checks for modified files
+- 🎬 **Smart Scripts** - Auto-generated `check-all` script runs tools in optimal order
+- ✅ **Battle-Tested** - 556 tests with 96% coverage
 
 ## Quick Start
 
@@ -118,14 +120,17 @@ That's it! The configs will automatically detect your framework and TypeScript s
     "lint:fix": "eslint --fix .",
     "prettier": "prettier --check .",
     "prettier:fix": "prettier --write .",
-    "style": "stylelint '**/*.css'",
-    "style:fix": "stylelint '**/*.css' --fix",
+    "style": "stylelint '**/*.css' --allow-empty-input",
+    "style:fix": "stylelint '**/*.css' --fix --allow-empty-input",
     "typecheck": "tsc --noEmit",
-    "typecheck:watch": "tsc --noEmit --watch"
+    "typecheck:watch": "tsc --noEmit --watch",
+    "check-all": "npm run prettier && npm run style && npm run lint && npm run typecheck"
   }
 }
 ```
 
+**Tip:** The CLI automatically adds a `check-all` script when you setup multiple tools. This runs all validation tools in the optimal order: formatting → styles → linting → type checking.
+
 ## Supported Frameworks
 
 | Framework | ESLint | Prettier | Stylelint           | TypeScript      | Auto-Detect |
@@ -138,6 +143,7 @@ That's it! The configs will automatically detect your framework and TypeScript s
 | Angular   | ✅     | ✅       | ✅                  | ✅ decorators   | ✅          |
 | Vanilla   | ✅     | ✅       | ✅                  | ✅ browser      | ✅          |
 | Node.js   | ✅     | ✅       | N/A                 | ✅ NodeNext     | ✅          |
+| Electron  | ✅     | ✅       | N/A                 | ✅ multi-config | ✅          |
 
 **Meta-frameworks**: Next.js (→ React), Nuxt (→ Vue), SvelteKit (→ Svelte)
 
@@ -174,23 +180,11 @@ await createConfig('eslint', {
 })
 ```
 
-**Example - Custom Rules:**
+**Custom rules:**
 
 ```javascript
 export default await createConfig('eslint', {
-  rules: {
-    'no-console': 'off',
-    'react/prop-types': 'warn'
-  }
-})
-```
-
-**Example - Explicit Framework:**
-
-```javascript
-export default await createConfig('eslint', {
-  framework: 'vue',
-  typescript: true
+  rules: { 'no-console': 'off' }
 })
 ```
 
@@ -199,20 +193,9 @@ export default await createConfig('eslint', {
 ```javascript
 createConfig('prettier', {
   framework: 'auto',
-  // Override any Prettier option
+  printWidth: 100, // Override any Prettier option
   semi: false,
-  singleQuote: true,
-  printWidth: 100
-})
-```
-
-**Example - Custom Settings:**
-
-```javascript
-export default createConfig('prettier', {
-  printWidth: 120,
-  tabWidth: 4,
-  trailingComma: 'all'
+  singleQuote: true
 })
 ```
 
@@ -221,57 +204,29 @@ export default createConfig('prettier', {
 ```javascript
 createConfig('stylelint', {
   framework: 'auto',
-  cssType: 'auto', // 'auto' | 'scss' | 'css' | { preprocessor, tailwind, modules }
+  cssType: 'auto', // 'auto' | 'scss' | 'css'
+  cssType: { preprocessor: 'scss', tailwind: true }, // Or object
   rules: {} // Custom rule overrides
 })
 ```
 
-**Example - SCSS + Tailwind:**
-
-```javascript
-export default createConfig('stylelint', {
-  cssType: {
-    preprocessor: 'scss',
-    tailwind: true
-  }
-})
-```
-
 ### TypeScript Options
 
 ```javascript
 await createConfig('typescript', {
   framework: 'auto', // Auto-detect or specify framework
   environment: 'auto', // 'auto' | 'browser' | 'node' | 'universal'
-  checker: 'auto', // 'auto' | 'modern' (tsgo) | 'legacy' (tsc/vue-tsc)
+  checker: 'auto', // 'auto' | 'modern' (tsgo) | 'legacy' (tsc)
   strict: true, // Enable strict type checking
-  compilerOptions: {} // Custom TypeScript compiler options
-})
-```
-
-**Example - React with Custom Options:**
-
-```javascript
-export default await createConfig('typescript', {
-  framework: 'react',
   compilerOptions: {
+    // Custom compiler options
     baseUrl: '.',
-    paths: {
-      '@/*': ['src/*']
-    }
+    paths: { '@/*': ['src/*'] }
   }
 })
 ```
 
-**Example - Use tsgo for 10x Faster Type Checking:**
-
-```javascript
-export default await createConfig('typescript', {
-  checker: 'modern' // Forces tsgo when installed
-})
-```
-
-**tsgo Support:** Install `@typescript/native-preview` for 10x faster type checking. The package automatically detects and uses tsgo when available.
+**tsgo Support:** Install `@typescript/native-preview` for 10x faster type checking. Auto-detected or set `checker: 'modern'` to force it.
 
 ### semantic-release Options
 
@@ -282,15 +237,6 @@ createConfig('semantic-release', {
 })
 ```
 
-**Example - Library Release:**
-
-```javascript
-export default createConfig('semantic-release', {
-  preset: 'library',
-  gitProvider: 'github'
-})
-```
-
 ## Framework-Specific Guides
 
 ### React
@@ -419,22 +365,68 @@ npx @dimensional-innovations/tool-config
 - Choose which tools to configure
 - Auto-detects framework and TypeScript
 - Creates config files automatically
-- Adds npm scripts to package.json
-- **NEW:** Automated CI/CD pipeline setup
+- Adds npm scripts to package.json (including `check-all` for multiple tools)
+- Automated CI/CD pipeline setup
+- **Uninstall support** - Clean removal of configs and scripts
 - Supports dry-run mode
 
-**Options:**
+**Setup Options:**
 
 ```bash
+# Interactive mode
+npx @dimensional-innovations/tool-config                 # Choose tools interactively
+
+# Single tool setup
 npx @dimensional-innovations/tool-config eslint          # Setup ESLint only
-npx @dimensional-innovations/tool-config --all           # Setup all tools
-npx @dimensional-innovations/tool-config --dry-run       # Preview without creating files
+npx @dimensional-innovations/tool-config prettier        # Setup Prettier only
+npx @dimensional-innovations/tool-config stylelint       # Setup Stylelint only
+
+# Setup all tools
+npx @dimensional-innovations/tool-config --all           # Setup all tools at once
+
+# CI/CD setup
 npx @dimensional-innovations/tool-config --ci gitlab     # Setup GitLab CI/CD
 npx @dimensional-innovations/tool-config --ci github     # Setup GitHub Actions
 npx @dimensional-innovations/tool-config --setup-ci      # Interactive CI setup
+
+# Preview mode
+npx @dimensional-innovations/tool-config --dry-run       # Preview without creating files
+
+# Help
 npx @dimensional-innovations/tool-config --help          # Show all options
 ```
 
+**Uninstall Options:**
+
+```bash
+# Interactive uninstall
+npx @dimensional-innovations/tool-config --uninstall     # Choose which tools to remove
+
+# Uninstall specific tool
+npx @dimensional-innovations/tool-config --uninstall eslint
+
+# Uninstall all detected tools
+npx @dimensional-innovations/tool-config --uninstall --all
+
+# Remove CI/CD configuration
+npx @dimensional-innovations/tool-config --uninstall --ci
+
+# Preview uninstall
+npx @dimensional-innovations/tool-config --uninstall --dry-run
+```
+
+**Safety Features:**
+
+The uninstall command includes safety features to protect your customizations:
+
+- Only removes files that match the original generated templates
+- Skips modified configuration files (warns you instead)
+- Only removes auto-generated package.json scripts
+- Preserves custom scripts with the same names
+- Supports dry-run to preview what will be removed
+- Cleans up empty directories (e.g., `.github/workflows`)
+- Idempotent - safe to run multiple times
+
 **CI/CD Integration:**
 
 When you select `semantic-release` in interactive mode, the CLI will automatically prompt you to setup CI/CD for automated releases. Or use the `--ci` flag to setup CI/CD directly:
@@ -535,64 +527,43 @@ See the [`examples/`](./examples/) directory for complete working examples:
 - [Node Backend](./examples/node-backend/) - Express, TypeScript
 - [Vanilla JS](./examples/vanilla-js/) - Pure JavaScript, no framework
 
-Each example includes all 4 tool configurations and npm scripts.
+Each example includes all 5 tool configurations and npm scripts.
 
 ## Advanced Usage
 
-### Monorepo Support
-
-Use different configs per workspace:
+**Monorepo Support:**
 
 ```javascript
 // apps/frontend/eslint.config.js
-export default await createConfig('eslint', {
-  framework: 'react',
-  typescript: true
-})
+export default await createConfig('eslint', { framework: 'react' })
 
 // apps/backend/eslint.config.js
-export default await createConfig('eslint', {
-  framework: 'node',
-  environment: 'node'
-})
+export default await createConfig('eslint', { framework: 'node' })
 ```
 
-### Custom Ignore Patterns
+**Custom Ignore Patterns:**
 
 ```javascript
 export default await createConfig('eslint', {
-  ignore: ['dist/**', 'build/**', '**/*.generated.ts']
+  ignore: ['dist/**', '**/*.generated.ts']
 })
 ```
 
-### Environment-Specific Rules
+**Environment-Specific Rules:**
 
 ```javascript
-const isDev = process.env.NODE_ENV === 'development'
-
 export default await createConfig('eslint', {
   rules: {
-    'no-console': isDev ? 'off' : 'error',
-    'no-debugger': isDev ? 'warn' : 'error'
+    'no-console': process.env.NODE_ENV === 'development' ? 'off' : 'error'
   }
 })
 ```
 
-### Combining with Other Configs
+**Combining with Other Configs:**
 
 ```javascript
-import { createConfig } from '@dimensional-innovations/tool-config'
-import customRules from './custom-rules.js'
-
 const baseConfig = await createConfig('eslint')
-
-export default [
-  ...baseConfig,
-  {
-    files: ['**/*.js'],
-    rules: customRules
-  }
-]
+export default [...baseConfig, { files: ['**/*.js'], rules: customRules }]
 ```
 
 ## Ignoring Files
@@ -647,8 +618,7 @@ export default createConfig('stylelint', {
 
 **Prettier** - Edit `.prettierignore` file:
 
-```
-# Custom ignores
+```text
 generated/
 legacy/
 ```
@@ -670,77 +640,20 @@ npx prettier --check . --debug-check
 
 ## Troubleshooting
 
-### Tools Processing Build Outputs
+**ESLint not detecting framework:** Ensure framework is in `package.json` dependencies, or explicitly set `framework: 'react'`.
 
-If you see errors about linting/formatting files in `dist/`, `coverage/`, or `out/`:
+**TypeScript parsing errors:** Install TypeScript: `npm install --save-dev typescript`
 
-1. **ESLint**: Ignore patterns are applied automatically ✅
-2. **Stylelint**: Ignore patterns are applied automatically ✅
-3. **Prettier**: Check that `.prettierignore` was created. If not, re-run config creation or manually create the file.
-
-**Solution**: If `.prettierignore` is missing, it will be auto-created next time you use the config. Or create it manually:
+**Prettier plugin not found:** Install required peer dependency:
 
 ```bash
-# Let Prettier generate it
-node -e "import('@dimensional-innovations/tool-config').then(m => m.createConfig('prettier'))"
+npm install --save-dev prettier-plugin-svelte  # Svelte
+npm install --save-dev prettier-plugin-astro   # Astro
 ```
 
-### ESLint Not Detecting Framework
-
-Ensure your `package.json` includes the framework dependency:
+**Stylelint not linting .vue files:** Update script: `"style": "stylelint '**/*.css' '**/*.vue' --allow-empty-input"`
 
-```json
-{
-  "dependencies": {
-    "react": "^18.0.0"
-  }
-}
-```
-
-Or explicitly specify:
-
-```javascript
-export default await createConfig('eslint', {
-  framework: 'react'
-})
-```
-
-### TypeScript Parsing Errors
-
-Make sure `typescript` is installed:
-
-```bash
-npm install --save-dev typescript
-```
-
-### Prettier Plugin Not Found
-
-Install the required peer dependency:
-
-```bash
-npm install --save-dev prettier-plugin-svelte  # For Svelte
-npm install --save-dev prettier-plugin-astro   # For Astro
-```
-
-### Stylelint Not Linting .vue Files
-
-Update your npm script to include `.vue` files:
-
-```json
-{
-  "scripts": {
-    "style": "stylelint '**/*.css' '**/*.vue'"
-  }
-}
-```
-
-### Config Not Updating After Framework Change
-
-Delete `node_modules/.cache` and restart your editor:
-
-```bash
-rm -rf node_modules/.cache
-```
+**Config not updating:** Delete `node_modules/.cache` and restart editor.
 
 ## Requirements
 
@@ -757,20 +670,18 @@ This package follows these principles:
 - **Fail Loudly** - Errors for critical issues, warnings for quality improvements
 - **Framework Agnostic** - No opinions about which framework you use
 - **Modern Standards** - Uses latest tooling and ESLint flat config
-- **Comprehensive Coverage** - 100% test coverage, battle-tested in production
+- **Comprehensive Coverage** - 96% test coverage, battle-tested in production
 
 ## Contributing
 
-Contributions welcome! This package is built to be extensible.
-
-**To add a new framework:**
+Contributions welcome! To add a new framework:
 
 1. Add detection logic in `src/detectors.js`
 2. Create presets in `src/tools/{tool}/presets/frameworks/`
 3. Add tests with 100% coverage
 4. Update documentation
 
-See [IMPLEMENTATION_PLAN.md](./IMPLEMENTATION_PLAN.md) for architecture details.
+See [CLAUDE.md](./CLAUDE.md) for architecture details.
 
 ## License
 

package/bin/lib/formatting.js

@@ -0,0 +1,103 @@
+/**
+ * Formatting utilities for CLI output
+ * Standardized box drawing with consistent width
+ */
+
+import pc from 'picocolors'
+
+const BOX_WIDTH = 70 // Standard width for all boxes
+
+export const BOX = {
+  topLeft: '┌',
+  topRight: '┐',
+  bottomLeft: '└',
+  bottomRight: '┘',
+  vertical: '│',
+  horizontal: '─',
+  separator: '━',
+  width: BOX_WIDTH
+}
+
+/**
+ * Create a box with standard width (70 chars)
+ * @param {string[]} lines - Array of content lines
+ * @param {string} title - Optional title for top border
+ * @returns {string} Formatted box
+ */
+export function createBox(lines, title = '') {
+  const width = BOX.width
+  const contentWidth = width - 4 // Remove: │ + space + space + │
+
+  // Create top border
+  let top
+  if (title) {
+    const titlePart = `─ ${title} `
+    const remaining = width - 2 - titlePart.length // -2 for corners
+    top = BOX.topLeft + titlePart + BOX.horizontal.repeat(remaining) + BOX.topRight
+  } else {
+    top = BOX.topLeft + BOX.horizontal.repeat(width - 2) + BOX.topRight
+  }
+
+  // Create content lines - pad each to exact width
+  const content = lines.map(line => {
+    const padded = line.slice(0, contentWidth).padEnd(contentWidth)
+    return `${BOX.vertical} ${padded} ${BOX.vertical}`
+  })
+
+  // Create bottom border
+  const bottom = BOX.bottomLeft + BOX.horizontal.repeat(width - 2) + BOX.bottomRight
+
+  return [top, ...content, bottom].join('\n')
+}
+
+/**
+ * Create a separator line
+ * @returns {string} Separator line
+ */
+export function createSeparator() {
+  return BOX.separator.repeat(BOX.width)
+}
+
+/**
+ * Create a simple indented list
+ * @param {string[]} items - Array of items
+ * @param {string} bullet - Bullet character (default: '•')
+ * @returns {string} Formatted list
+ */
+export function createList(items, bullet = '•') {
+  return items.map(item => `   ${bullet} ${item}`).join('\n')
+}
+
+/**
+ * Create a progress indicator
+ * @param {number} current - Current step
+ * @param {number} total - Total steps
+ * @param {string} label - Label for this step
+ * @returns {string} Progress line
+ */
+export function createProgress(current, total, label) {
+  return `  [${current}/${total}] ${label}`
+}
+
+/**
+ * Color symbols for CLI output
+ */
+export const SYMBOLS = {
+  success: pc.green('✓'),
+  warning: pc.yellow('⚠'),
+  error: pc.red('✗'),
+  info: pc.blue('ℹ')
+}
+
+/**
+ * Color helper functions
+ */
+export const colors = {
+  success: pc.green,
+  warning: pc.yellow,
+  error: pc.red,
+  info: pc.blue,
+  dim: pc.dim,
+  bold: pc.bold,
+  cyan: pc.cyan
+}

package/bin/lib/handlers/stylelint.js

@@ -51,8 +51,8 @@ export function getScripts(detected) {
     extensions.length === 1 ? `**/*.${extensions[0]}` : `**/*.{${extensions.join(',')}}`
 
   return {
-    style: `stylelint "${pattern}"`,
-    'style:fix': `stylelint "${pattern}" --fix`
+    style: `stylelint "${pattern}" --allow-empty-input`,
+    'style:fix': `stylelint "${pattern}" --fix --allow-empty-input`
   }
 }
 

package/bin/lib/package-manager.js

@@ -112,3 +112,90 @@ export function updatePackageJsonScripts(tools, getScriptsFn, context) {
     console.error('  ❌ Failed to update package.json:', error.message)
   }
 }
+
+/**
+ * Remove scripts from package.json for tools
+ * @param {string[]} tools - Array of tool names to remove scripts for
+ * @param {Function} getScriptsFn - Function that returns scripts for a tool: (tool, detected) => object
+ * @param {Object} context - Context object with {detected, cwd, dryRun}
+ */
+export function removePackageJsonScripts(tools, getScriptsFn, context) {
+  const { detected, cwd, dryRun = false } = context
+  const packageJsonPath = join(cwd, 'package.json')
+
+  if (!existsSync(packageJsonPath)) {
+    console.log('  ℹ️  No package.json found - skipping script removal')
+    return
+  }
+
+  try {
+    const packageJson = JSON.parse(readFileSync(packageJsonPath, 'utf8'))
+
+    if (!packageJson.scripts) {
+      console.log('  ℹ️  No scripts section in package.json')
+      return
+    }
+
+    const scriptsToRemove = []
+
+    // Collect script names to remove based on tools
+    for (const tool of tools) {
+      const toolScripts = getScriptsFn(tool, detected)
+      for (const scriptName of Object.keys(toolScripts)) {
+        const expectedCommand = toolScripts[scriptName]
+        const actualCommand = packageJson.scripts[scriptName]
+
+        // Only remove if the script exists and matches our expected command
+        if (actualCommand === expectedCommand) {
+          scriptsToRemove.push(scriptName)
+        } else if (actualCommand) {
+          console.log(`  ⚠️  Script "${scriptName}" has been modified - skipping for safety`)
+        }
+      }
+    }
+
+    // Handle check-all script separately
+    const checkAllScript = packageJson.scripts['check-all']
+    if (checkAllScript) {
+      // Only remove if it's auto-generated (follows npm run X && npm run Y pattern)
+      const isAutoGenerated =
+        checkAllScript.startsWith('npm run') && checkAllScript.includes(' && npm run')
+
+      if (isAutoGenerated) {
+        scriptsToRemove.push('check-all')
+      }
+    }
+
+    if (scriptsToRemove.length === 0) {
+      console.log('  ℹ️  No scripts to remove from package.json')
+      return
+    }
+
+    if (dryRun) {
+      console.log('  📝 Would remove scripts from package.json:')
+      for (const name of scriptsToRemove) {
+        console.log(`     - ${name}`)
+      }
+      return
+    }
+
+    // Remove scripts
+    for (const scriptName of scriptsToRemove) {
+      delete packageJson.scripts[scriptName]
+    }
+
+    // Remove scripts section if empty
+    if (Object.keys(packageJson.scripts).length === 0) {
+      delete packageJson.scripts
+    }
+
+    writeFileSync(packageJsonPath, `${JSON.stringify(packageJson, null, 2)}\n`, 'utf8')
+
+    console.log('  ✅ Removed scripts from package.json:')
+    for (const name of scriptsToRemove) {
+      console.log(`     - ${name}`)
+    }
+  } catch (error) {
+    console.error('  ❌ Failed to update package.json:', error.message)
+  }
+}