@jmruthers/pace-core 0.6.6 → 0.6.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jmruthers/pace-core",
-  "version": "0.6.6",
+  "version": "0.6.8",
   "description": "React component library with Tailwind v4",
   "private": false,
   "publishConfig": {
@@ -79,8 +79,6 @@
     "./core-usage-manifest.json": "./core-usage-manifest.json",
     "./cursor-rules": "./cursor-rules",
     "./scripts/install-cursor-rules": "./scripts/install-cursor-rules.cjs",
-    "./scripts/audit-dependencies": "./scripts/audit/audit-dependencies.cjs",
-    "./scripts/audit": "./scripts/audit/index.cjs",
     "./source": {
       "import": "./src/index.ts",
       "default": "./src/index.ts"
@@ -153,7 +151,8 @@
     "core-usage-manifest.json",
     "eslint-config-pace-core.cjs",
     "scripts",
-    "cursor-rules"
+    "cursor-rules",
+    "audit-tool"
   ],
   "scripts": {
     "_comment_build": "Build package and documentation",
@@ -167,7 +166,8 @@
     "link": "npm run build && npm link",
     "unlink": "npm unlink",
     "validate:theme": "node scripts/validate-theme.js",
-    "prepublishOnly": "npm run build",
+    "prepublishOnly": "npm run validate:dependencies && npm run build",
+    "validate:dependencies": "node scripts/validate-dependencies.cjs",
     "clean": "rimraf dist",
     "_comment_test": "Test suite with various configurations",
     "test": "cross-env NODE_OPTIONS=\"--max-old-space-size=8192 --max-semi-space-size=128\" vitest run --config ../../vitest.config.ts",
@@ -198,7 +198,9 @@
     "docs:generate": "node scripts/generate-docs.js generate",
     "docs:update-index": "node scripts/generate-docs.js update-index",
     "docs:generate-api": "node scripts/generate-docs.js generate-api",
-    "docs:all": "node scripts/generate-docs.js all"
+    "docs:all": "node scripts/generate-docs.js all",
+    "_comment_audit": "Audit tool for consuming apps",
+    "audit:pace-core": "node packages/core/audit-tool/index.cjs"
   },
   "keywords": [
     "react",
@@ -217,38 +219,67 @@
     "clsx": "^2.0.0",
     "date-fns": "^3.0.0",
     "date-fns-tz": "^3.0.0",
-    "react": "^19.0.0",
+    "react": "^19.2.3",
     "react-day-picker": "^9.0.0",
-    "react-dom": "^19.0.0",
+    "react-dom": "^19.2.3",
     "react-hook-form": "^7.0.0",
     "react-router-dom": "^6.0.0",
     "tailwind-merge": "^2.0.0",
-    "tailwindcss": "^4.0.0",
+    "tailwindcss": "^4.1.16",
     "zod": "^3.20.0"
   },
+  "peerDependenciesMeta": {
+    "@tanstack/react-query": {
+      "optional": true
+    },
+    "@tanstack/react-table": {
+      "optional": true
+    },
+    "clsx": {
+      "optional": true
+    },
+    "date-fns": {
+      "optional": true
+    },
+    "date-fns-tz": {
+      "optional": true
+    },
+    "react-hook-form": {
+      "optional": true
+    },
+    "react-day-picker": {
+      "optional": true
+    },
+    "zod": {
+      "optional": true
+    },
+    "tailwind-merge": {
+      "optional": true
+    }
+  },
   "devDependencies": {
     "@testing-library/jest-dom": "^6.6.3",
-    "@testing-library/react": "^16.1.0",
-    "@testing-library/user-event": "^14.5.2",
+    "@testing-library/react": "^16.3.0",
+    "@testing-library/user-event": "^14.6.1",
     "@types/papaparse": "^5.3.16",
-    "@types/react": "^19.0.0",
-    "@types/react-dom": "^19.0.0",
+    "@types/react": "^19.2.7",
+    "@types/react-dom": "^19.2.3",
     "@types/react-window": "^1.8.8",
-    "@vitejs/plugin-react": "^4.5.1",
-    "@vitest/coverage-v8": "^3.2.3",
+    "@vitejs/plugin-react": "^5.1.2",
+    "@vitest/coverage-v8": "^4.0.16",
     "esbuild": "^0.20.0",
     "eslint-plugin-react-hooks": "^5.2.0",
     "eslint-plugin-react-refresh": "^0.4.20",
     "globals": "^16.3.0",
     "jsdom": "^25.0.1",
     "react-router-dom": "^6.26.2",
-    "tsup": "^8.5.0",
-    "typedoc": "^0.26.11",
+    "tsup": "^8.5.1",
+    "typedoc": "^0.27.0",
     "typedoc-plugin-markdown": "^3.17.1",
     "typedoc-plugin-merge-modules": "^5.1.0",
-    "typescript": "^5.4.0",
+    "typescript": "^5.9.3",
     "typescript-eslint": "^8.39.0",
-    "vite": "^6.0.3"
+    "vite": "^7.2.7"
   },
   "dependencies": {
     "@hookform/resolvers": "^3.9.0",

package/scripts/eslint-audit.cjs

@@ -0,0 +1,123 @@
+#!/usr/bin/env node
+
+/**
+ * ESLint Audit Script
+ * @package pace-core
+ * @module Scripts/eslint-audit
+ * 
+ * Runs ESLint across the entire repository and outputs results to a timestamped file
+ * in the audit directory.
+ * 
+ * Usage:
+ *   npm run audit:eslint
+ */
+
+const { execSync } = require('child_process');
+const fs = require('fs');
+const path = require('path');
+
+// ANSI color codes for terminal output
+const colors = {
+  reset: '\x1b[0m',
+  green: '\x1b[32m',
+  yellow: '\x1b[33m',
+  blue: '\x1b[34m',
+  cyan: '\x1b[36m',
+  bold: '\x1b[1m',
+  red: '\x1b[31m'
+};
+
+// Generate timestamp in format yyyymmddHHMM
+function generateTimestamp() {
+  const now = new Date();
+  const year = now.getFullYear();
+  const month = String(now.getMonth() + 1).padStart(2, '0');
+  const day = String(now.getDate()).padStart(2, '0');
+  const hours = String(now.getHours()).padStart(2, '0');
+  const minutes = String(now.getMinutes()).padStart(2, '0');
+  return `${year}${month}${day}${hours}${minutes}`;
+}
+
+// Main execution
+function main() {
+  const cwd = process.cwd();
+  const auditDir = path.join(cwd, 'audit');
+  const timestamp = generateTimestamp();
+  const outputFile = path.join(auditDir, `${timestamp}-eslint-report.txt`);
+  
+  // Create audit directory if it doesn't exist
+  if (!fs.existsSync(auditDir)) {
+    fs.mkdirSync(auditDir, { recursive: true });
+    console.log(`${colors.green}Created audit/ directory${colors.reset}`);
+  }
+  
+  console.log(`${colors.cyan}Running ESLint audit across the repository...${colors.reset}\n`);
+  
+  try {
+    // Run ESLint and capture output
+    const eslintCommand = 'npm run lint 2>&1';
+    const output = execSync(eslintCommand, { 
+      encoding: 'utf8',
+      cwd,
+      stdio: 'pipe'
+    });
+    
+    // Write output to file
+    const reportContent = `ESLint Audit Report
+Generated: ${new Date().toISOString()}
+Timestamp: ${timestamp}
+
+${output}
+`;
+    
+    fs.writeFileSync(outputFile, reportContent, 'utf8');
+    
+    // Check if there are any errors
+    const hasErrors = output.includes('error') || output.includes('✖');
+    const hasWarnings = output.includes('warning') || output.includes('⚠');
+    
+    console.log(`${colors.bold}Audit Summary:${colors.reset}`);
+    if (hasErrors) {
+      console.log(`  ${colors.red}Errors found${colors.reset} - See report for details`);
+    } else if (hasWarnings) {
+      console.log(`  ${colors.yellow}Warnings found${colors.reset} - See report for details`);
+    } else {
+      console.log(`  ${colors.green}No errors or warnings${colors.reset}`);
+    }
+    
+    console.log(`\n${colors.cyan}Report saved to:${colors.reset} ${outputFile}`);
+    console.log(`\n${colors.cyan}To view the report:${colors.reset}`);
+    console.log(`  ${colors.bold}cat ${outputFile}${colors.reset}`);
+    
+  } catch (error) {
+    // ESLint may exit with non-zero code if errors are found
+    // Capture the output anyway
+    const output = error.stdout || error.message || String(error);
+    
+    const reportContent = `ESLint Audit Report
+Generated: ${new Date().toISOString()}
+Timestamp: ${timestamp}
+
+${output}
+
+Exit Code: ${error.status || 1}
+`;
+    
+    fs.writeFileSync(outputFile, reportContent, 'utf8');
+    
+    console.log(`${colors.yellow}ESLint found issues (exit code: ${error.status || 1})${colors.reset}`);
+    console.log(`\n${colors.cyan}Report saved to:${colors.reset} ${outputFile}`);
+    console.log(`\n${colors.cyan}To view the report:${colors.reset}`);
+    console.log(`  ${colors.bold}cat ${outputFile}${colors.reset}`);
+    
+    // Don't exit with error - we've captured the report
+    process.exit(0);
+  }
+}
+
+// Run if called directly
+if (require.main === module) {
+  main();
+}
+
+module.exports = { main, generateTimestamp };

package/scripts/install-cursor-rules.cjs

@@ -1,21 +1,20 @@
 #!/usr/bin/env node
 
 /**
- * Installation Script for pace-core Cursor Rules and ESLint Config
+ * Installation Script for pace-core Cursor Rules
  * @package @jmruthers/pace-core
  * @module Scripts/install-cursor-rules
  * 
- * Sets up both:
- * 1. Cursor rules - Copies cursor rules from pace-core to consuming app's .cursor/rules/ directory
- * 2. ESLint config - Adds pace-core ESLint config to consuming app's ESLint configuration
+ * Sets up cursor rules:
+ * - Copies cursor rules from pace-core to consuming app's .cursor/rules/ directory
  * 
  * This is an opt-in script - it does NOT run automatically via postinstall.
  * 
+ * For ESLint setup, use install-eslint-config.cjs separately.
+ * 
  * Usage:
- *   node install-cursor-rules.cjs              # Install both cursor rules and ESLint
- *   node install-cursor-rules.cjs --skip-eslint # Only install cursor rules
- *   node install-cursor-rules.cjs --skip-cursor # Only setup ESLint
- *   node install-cursor-rules.cjs --force        # Force update even if already configured
+ *   node install-cursor-rules.cjs        # Install cursor rules
+ *   node install-cursor-rules.cjs --force # Force update even if already configured
  */
 
 const fs = require('fs');
@@ -219,250 +218,21 @@ function installCursorRules(force = false) {
   console.log(`${colors.cyan}Restart Cursor to load the new rules.${colors.reset}`);
 }
 
-// Find existing ESLint config file
-function findESLintConfig() {
-  const cwd = process.cwd();
-  const possibleConfigs = [
-    'eslint.config.js',
-    'eslint.config.cjs',
-    'eslint.config.mjs',
-    '.eslintrc.js',
-    '.eslintrc.cjs',
-    '.eslintrc.json',
-    '.eslintrc.yaml',
-    '.eslintrc.yml',
-  ];
-  
-  for (const configFile of possibleConfigs) {
-    const configPath = path.join(cwd, configFile);
-    if (fs.existsSync(configPath)) {
-      return { path: configPath, name: configFile, isESM: configFile.endsWith('.js') || configFile.endsWith('.mjs') };
-    }
-  }
-  
-  return null;
-}
-
-// Check if pace-core config is already included
-function hasPaceCoreConfig(configContent) {
-  // Check for pace-core config import/require
-  const paceCorePatterns = [
-    /@jmruthers\/pace-core\/eslint-config/,
-    /pace-core\/eslint-config/,
-    /paceCoreConfig/,
-    /pace-core-compliance/,
-  ];
-  
-  return paceCorePatterns.some(pattern => pattern.test(configContent));
-}
-
-// Detect if file is ES module
-function isESModule(filePath, content) {
-  if (filePath.endsWith('.mjs')) return true;
-  if (filePath.endsWith('.cjs')) return false;
-  if (filePath.endsWith('.js')) {
-    // Check for ES module indicators
-    return /^import\s+.*from|^export\s+default/.test(content.trim());
-  }
-  return false;
-}
-
-// Backup file before modification
-function backupFile(filePath) {
-  const backupPath = `${filePath}.backup.${Date.now()}`;
-  fs.copyFileSync(filePath, backupPath);
-  return backupPath;
-}
-
-// Setup ESLint configuration
-function setupESLintConfig(force = false, skipIfExists = false) {
-  const cwd = process.cwd();
-  const existingConfig = findESLintConfig();
-  
-  if (existingConfig) {
-    // Read existing config
-    let content = fs.readFileSync(existingConfig.path, 'utf8');
-    const isESM = isESModule(existingConfig.path, content);
-    const format = isESM ? 'ES Module' : 'CommonJS';
-    
-    console.log(`${colors.cyan}Found ESLint config:${colors.reset} ${existingConfig.name} (${format})`);
-    
-    // Check if already configured
-    if (hasPaceCoreConfig(content)) {
-      if (skipIfExists && !force) {
-        console.log(`${colors.blue}○${colors.reset} ${existingConfig.name} already includes pace-core rules`);
-        return { action: 'skipped', file: existingConfig.name, format };
-      }
-      
-      if (force) {
-        console.log(`${colors.yellow}Updating existing ESLint config...${colors.reset}`);
-      } else {
-        console.log(`${colors.blue}○${colors.reset} ${existingConfig.name} already configured. Use --force to update.`);
-        return { action: 'skipped', file: existingConfig.name, format };
-      }
-    } else {
-      console.log(`${colors.cyan}Adding pace-core config to ${existingConfig.name}...${colors.reset}`);
-    }
-    
-    // Backup before modification
-    const backupPath = backupFile(existingConfig.path);
-    console.log(`${colors.cyan}  Backed up to ${path.basename(backupPath)}${colors.reset}`);
-    
-    // Add pace-core config
-    if (isESM) {
-      // ES Module format
-      if (!content.includes('import paceCoreConfig')) {
-        // Add import at top (after other imports if they exist)
-        const importLines = content.match(/^(import\s+[^;]+;?\s*\n)+/m);
-        if (importLines) {
-          // Add after existing imports
-          content = content.replace(
-            importLines[0],
-            `${importLines[0]}import paceCoreConfig from '@jmruthers/pace-core/eslint-config';\n`
-          );
-        } else {
-          // Add at the beginning
-          content = `import paceCoreConfig from '@jmruthers/pace-core/eslint-config';\n${content}`;
-        }
-      }
-      
-      // Add to export default array
-      if (content.includes('export default [')) {
-        // Already an array, add paceCoreConfig at the beginning
-        if (!content.includes('...paceCoreConfig')) {
-          content = content.replace(
-            /(export\s+default\s*\[)\s*/,
-            '$1\n  ...paceCoreConfig,'
-          );
-        }
-      } else if (content.includes('export default')) {
-        // Not an array, wrap it
-        const exportMatch = content.match(/(export\s+default\s+)(.+?)(;?\s*$)/s);
-        if (exportMatch) {
-          content = content.replace(
-            exportMatch[0],
-            `${exportMatch[1]}[\n  ...paceCoreConfig,\n  ${exportMatch[2]}\n];`
-          );
-        }
-      }
-    } else {
-      // CommonJS format
-      if (!content.includes('require(\'@jmruthers/pace-core/eslint-config\')')) {
-        // Add require at top (after other requires if they exist)
-        const requireLines = content.match(/^(const\s+\w+\s*=\s*require\([^)]+\);\s*\n)+/m);
-        if (requireLines) {
-          content = content.replace(
-            requireLines[0],
-            `${requireLines[0]}const paceCoreConfig = require('@jmruthers/pace-core/eslint-config');\n`
-          );
-        } else {
-          content = `const paceCoreConfig = require('@jmruthers/pace-core/eslint-config');\n${content}`;
-        }
-      }
-      
-      // Add to module.exports
-      if (content.includes('module.exports = [')) {
-        // Already an array, add paceCoreConfig at the beginning
-        if (!content.includes('...paceCoreConfig')) {
-          content = content.replace(
-            /(module\.exports\s*=\s*\[)\s*/,
-            '$1\n  ...paceCoreConfig,'
-          );
-        }
-      } else if (content.includes('module.exports =')) {
-        // Not an array, wrap it
-        const exportMatch = content.match(/(module\.exports\s*=\s*)(.+?)(;?\s*$)/s);
-        if (exportMatch) {
-          content = content.replace(
-            exportMatch[0],
-            `${exportMatch[1]}[\n  ...paceCoreConfig,\n  ${exportMatch[2]}\n];`
-          );
-        }
-      }
-    }
-    
-    // Write updated config
-    fs.writeFileSync(existingConfig.path, content, 'utf8');
-    console.log(`${colors.green}✓${colors.reset} Updated ${existingConfig.name}`);
-    return { action: 'updated', file: existingConfig.name, backup: backupPath, format };
-  } else {
-    // Create new ESLint config (default to ES modules)
-    const configPath = path.join(cwd, 'eslint.config.js');
-    const configContent = `import paceCoreConfig from '@jmruthers/pace-core/eslint-config';
-
-export default [
-  ...paceCoreConfig,
-  // Your app-specific ESLint configuration
-  {
-    // Add your rules here
-  },
-];
-`;
-    
-    console.log(`${colors.cyan}No ESLint config found. Creating eslint.config.js...${colors.reset}`);
-    fs.writeFileSync(configPath, configContent, 'utf8');
-    console.log(`${colors.green}✓${colors.reset} Created eslint.config.js`);
-    console.log(`${colors.cyan}  Edit eslint.config.js to add your app-specific ESLint rules.${colors.reset}`);
-    return { action: 'created', file: 'eslint.config.js', format: 'ES Module' };
-  }
-}
 
 // Main execution
 function main() {
   const force = process.argv.includes('--force');
-  const skipCursor = process.argv.includes('--skip-cursor');
-  const skipESLint = process.argv.includes('--skip-eslint');
   
   if (force) {
     console.log(`${colors.yellow}Warning: --force flag is set. This will overwrite existing configurations.${colors.reset}\n`);
   }
   
   try {
-    let cursorResult = null;
-    let eslintResult = null;
-    
-    // Install cursor rules
-    if (!skipCursor) {
-      installCursorRules(force);
-      cursorResult = { completed: true };
-    } else {
-      console.log(`${colors.blue}Skipping cursor rules installation (--skip-cursor)${colors.reset}\n`);
-    }
-    
-    // Setup ESLint config
-    if (!skipESLint) {
-      console.log(`\n${colors.cyan}Setting up ESLint configuration...${colors.reset}\n`);
-      eslintResult = setupESLintConfig(force, true);
-      
-      // Output is handled in setupESLintConfig, but ensure we have consistent formatting
-    } else {
-      console.log(`\n${colors.blue}Skipping ESLint setup (--skip-eslint)${colors.reset}`);
-    }
-    
-    // Summary
-    console.log(`\n${colors.bold}Setup Summary:${colors.reset}`);
-    if (cursorResult) {
-      console.log(`  ${colors.green}Cursor Rules:${colors.reset} Installed`);
-    }
-    if (eslintResult) {
-      if (eslintResult.action === 'created') {
-        console.log(`  ${colors.green}ESLint Config:${colors.reset} Created ${eslintResult.file} (${eslintResult.format})`);
-      } else if (eslintResult.action === 'updated') {
-        console.log(`  ${colors.green}ESLint Config:${colors.reset} Updated ${eslintResult.file} (${eslintResult.format})`);
-        console.log(`  ${colors.yellow}Backup:${colors.reset} ${path.basename(eslintResult.backup)}`);
-      } else {
-        console.log(`  ${colors.blue}ESLint Config:${colors.reset} Already configured (${eslintResult.file}, ${eslintResult.format})`);
-      }
-    }
+    installCursorRules(force);
     
     console.log(`\n${colors.cyan}Next steps:${colors.reset}`);
-    if (!skipCursor) {
-      console.log(`  • Restart Cursor to load the new rules`);
-    }
-    if (!skipESLint) {
-      console.log(`  • Run ${colors.bold}npm run lint${colors.reset} to verify ESLint is working`);
-      console.log(`  • Edit your ESLint config to add app-specific rules`);
-    }
+    console.log(`  • Restart Cursor to load the new rules`);
+    console.log(`  • For ESLint setup, run: ${colors.bold}node node_modules/@jmruthers/pace-core/scripts/install-eslint-config.cjs${colors.reset}`);
     
   } catch (error) {
     console.error(`${colors.red}Error during setup:${colors.reset}`);
@@ -481,7 +251,5 @@ if (require.main === module) {
 
 module.exports = { 
   installCursorRules, 
-  getCursorRulesTarget,
-  setupESLintConfig,
-  findESLintConfig
+  getCursorRulesTarget
 };