@jmruthers/pace-core 0.6.6 → 0.6.8

package/scripts/audit/audit-components.cjs

@@ -1,260 +0,0 @@
-#!/usr/bin/env node
-
-/**
- * Component Usage Audit Script
- * 
- * MIGRATED TO ESLINT: All component usage checks have been migrated to ESLint rules:
- * - Plain <form> tags → ESLint: prefer-pace-core-form
- * - Direct react-hook-form usage → ESLint: prefer-pace-core-form
- * 
- * This file is kept for reference but is no longer actively used.
- * Use ESLint rules for real-time feedback during development.
- * 
- * Usage:
- *   const { runComponentAudit } = require('./audit-components.cjs');
- *   const issues = runComponentAudit(consumingAppPath);
- */
-
-const fs = require('fs');
-const path = require('path');
-
-/**
- * Recursively find all TypeScript/JavaScript files in a directory
- */
-function findSourceFiles(dir, fileList = []) {
-  const files = fs.readdirSync(dir);
-  
-  files.forEach(file => {
-    const filePath = path.join(dir, file);
-    const stat = fs.statSync(filePath);
-    
-    if (stat.isDirectory()) {
-      // Skip node_modules, dist, build, .git, etc.
-      if (!['node_modules', 'dist', 'build', '.git', '.next', '.vite', 'coverage'].includes(file)) {
-        findSourceFiles(filePath, fileList);
-      }
-    } else if (/\.(ts|tsx|js|jsx)$/.test(file)) {
-      fileList.push(filePath);
-    }
-  });
-  
-  return fileList;
-}
-
-/**
- * Get line number from index in content
- */
-function getLineNumber(content, index) {
-  return content.substring(0, index).split('\n').length;
-}
-
-/**
- * Check if a file imports Form from pace-core
- */
-function importsPaceCoreForm(content) {
-  // Check for imports like:
-  // import { Form } from '@jmruthers/pace-core
-  // import { Form } from '@jmruthers/pace-core/components'
-  // import Form from '@jmruthers/pace-core'
-  const paceCoreFormPattern = /from\s+['"]@jmruthers\/pace-core(?:\/components)?['"].*Form|import\s+.*Form.*from\s+['"]@jmruthers\/pace-core/;
-  return paceCoreFormPattern.test(content);
-}
-
-/**
- * Check for plain <form> tags in JSX
- * 
- * MIGRATED TO ESLINT: This check is now handled by 'prefer-pace-core-form' ESLint rule.
- * Kept for reference only.
- */
-function checkPlainFormTags_MIGRATED_TO_ESLINT(content, filePath, consumingAppPath) {
-  const issues = [];
-  
-  // Pattern to match <form> tags (but not <Form> component)
-  // This regex looks for <form followed by whitespace or attributes, but not <Form
-  const formTagPattern = /<form(\s|>)/g;
-  let match;
-  
-  while ((match = formTagPattern.exec(content)) !== null) {
-    // Check if this is actually a Form component (capital F)
-    const beforeMatch = content.substring(Math.max(0, match.index - 10), match.index);
-    const afterMatch = content.substring(match.index, match.index + 20);
-    
-    // Skip if it's a comment or string literal
-    const beforeContext = content.substring(0, match.index);
-    const lastComment = beforeContext.lastIndexOf('//');
-    const lastNewline = beforeContext.lastIndexOf('\n');
-    const isInComment = lastComment > lastNewline && !beforeContext.substring(lastComment, match.index).includes('\n');
-    
-    if (isInComment) continue;
-    
-    // Check if it's inside a string literal
-    const beforeQuote = beforeContext.lastIndexOf('"');
-    const afterQuote = content.indexOf('"', match.index);
-    const isInString = beforeQuote > lastNewline && afterQuote > match.index && beforeQuote < match.index;
-    
-    if (isInString) continue;
-    
-    // This is a plain <form> tag
-    // Get relative path from consuming app root
-    const relativePath = path.relative(consumingAppPath || process.cwd(), filePath);
-    
-    issues.push({
-      type: 'plainFormTag',
-      file: relativePath,
-      line: getLineNumber(content, match.index),
-      message: 'Plain <form> tag detected. Use pace-core Form component instead.',
-      code: content.substring(Math.max(0, match.index - 30), Math.min(content.length, match.index + 50)).trim(),
-    });
-  }
-  
-  return issues;
-}
-
-/**
- * Check for direct react-hook-form imports and usage
- * 
- * MIGRATED TO ESLINT: This check is now handled by 'prefer-pace-core-form' ESLint rule.
- * Kept for reference only.
- */
-function checkReactHookFormUsage_MIGRATED_TO_ESLINT(content, filePath, consumingAppPath) {
-  const issues = [];
-  
-  // Pattern to match imports from react-hook-form
-  const importPattern = /from\s+['"]react-hook-form['"]/g;
-  let match;
-  
-  while ((match = importPattern.exec(content)) !== null) {
-    // Get the import statement
-    const lineStart = content.lastIndexOf('\n', match.index) + 1;
-    const lineEnd = content.indexOf('\n', match.index);
-    const importLine = content.substring(lineStart, lineEnd === -1 ? content.length : lineEnd);
-    
-    // Check if this file also imports Form from pace-core
-    // If it does, this might be acceptable (e.g., using useFormContext within Form)
-    const hasPaceCoreForm = importsPaceCoreForm(content);
-    
-    // Extract what's being imported
-    const importMatch = importLine.match(/import\s+(?:{([^}]+)}|(\w+))\s+from/);
-    if (importMatch) {
-      const importedItems = importMatch[1] || importMatch[2] || '';
-      const items = importedItems.split(',').map(item => item.trim().split(' as ')[0].trim());
-      
-      // Check for problematic imports
-      const problematicImports = [
-        'useForm',
-        'FormProvider',
-        'Controller',
-        'useFormContext',
-        'useWatch',
-        'useFieldArray',
-        'useController',
-      ];
-      
-      const foundProblematic = items.filter(item => problematicImports.includes(item));
-      
-      if (foundProblematic.length > 0) {
-        // If they're using useFormContext within a pace-core Form, that's acceptable
-        // But if they're using useForm or FormProvider, that's not acceptable
-        const isAcceptable = hasPaceCoreForm && 
-                            foundProblematic.length === 1 && 
-                            foundProblematic[0] === 'useFormContext';
-        
-        if (!isAcceptable) {
-          // Get relative path from consuming app root
-          const relativePath = path.relative(consumingAppPath || process.cwd(), filePath);
-          
-          issues.push({
-            type: 'reactHookFormImport',
-            file: relativePath,
-            line: getLineNumber(content, match.index),
-            message: `Direct import from 'react-hook-form' detected: ${foundProblematic.join(', ')}. Use pace-core Form component instead.`,
-            code: importLine.trim(),
-            imports: foundProblematic,
-          });
-        }
-      }
-    }
-  }
-  
-  return issues;
-}
-
-/**
- * Main audit function - scans source files for component usage issues
- */
-function runComponentAudit(consumingAppPath = process.cwd()) {
-  const srcPath = path.join(consumingAppPath, 'src');
-  
-  // If src directory doesn't exist, try root directory
-  const searchPath = fs.existsSync(srcPath) ? srcPath : consumingAppPath;
-  
-  if (!fs.existsSync(searchPath)) {
-    return {
-      error: `Source directory not found at ${searchPath}`,
-      issues: { formIssues: [] },
-    };
-  }
-  
-  // Find all source files
-  const sourceFiles = findSourceFiles(searchPath);
-  
-  if (sourceFiles.length === 0) {
-    return {
-      error: `No source files found in ${searchPath}`,
-      issues: { formIssues: [] },
-    };
-  }
-  
-  const issues = {
-    formIssues: [],
-  };
-  
-  // NOTE: All component checks have been migrated to ESLint rules.
-  // This function now returns empty results as checks are handled by ESLint.
-  // Kept for API compatibility.
-  sourceFiles.forEach(filePath => {
-    // No checks performed - all migrated to ESLint
-    // (checkPlainFormTags and checkReactHookFormUsage are now ESLint rules)
-  });
-  
-  return {
-    issues,
-  };
-}
-
-// Export for use by other scripts
-if (typeof module !== 'undefined' && module.exports) {
-  module.exports = { runComponentAudit };
-}
-
-// If run directly, output results
-if (require.main === module) {
-  const consumingAppPath = process.argv[2] || process.cwd();
-  const result = runComponentAudit(consumingAppPath);
-  
-  if (result.error) {
-    console.error(`Error: ${result.error}`);
-    process.exit(1);
-  }
-  
-  const { issues } = result;
-  
-  if (issues.formIssues.length === 0) {
-    console.log('✅ No form usage issues found!');
-    process.exit(0);
-  }
-  
-  console.log(`\n❌ Found ${issues.formIssues.length} form usage issue(s):\n`);
-  
-  issues.formIssues.forEach(issue => {
-    console.log(`${issue.file}:${issue.line}`);
-    console.log(`  ${issue.message}`);
-    if (issue.code) {
-      console.log(`  Code: ${issue.code.substring(0, 80)}...`);
-    }
-    console.log();
-  });
-  
-  process.exit(1);
-}
-