@jmruthers/pace-core 0.6.6 → 0.6.7

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
 };

package/scripts/install-eslint-config.cjs

@@ -0,0 +1,284 @@
+#!/usr/bin/env node
+
+/**
+ * Installation Script for pace-core ESLint Config
+ * @package @jmruthers/pace-core
+ * @module Scripts/install-eslint-config
+ * 
+ * Sets up pace-core ESLint configuration in consuming apps.
+ * Adds pace-core ESLint config to consuming app's ESLint configuration.
+ * 
+ * This is an opt-in script - it does NOT run automatically via postinstall.
+ * 
+ * Usage:
+ *   node install-eslint-config.cjs        # Setup ESLint config
+ *   node install-eslint-config.cjs --force # Force update even if already configured
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+// Check for safety guards
+function checkSafetyGuards() {
+  // Check environment variable
+  if (process.env.PACE_ESLINT_CONFIG_DISABLED === '1') {
+    console.log(`${colors.yellow}Skipping ESLint config installation: PACE_ESLINT_CONFIG_DISABLED=1${colors.reset}`);
+    process.exit(0);
+  }
+  
+  // Check for .git folder (safety guard for CI)
+  const cwd = process.cwd();
+  if (!fs.existsSync(path.join(cwd, '.git'))) {
+    console.log(`${colors.yellow}Skipping ESLint config installation: No .git folder found${colors.reset}`);
+    console.log(`${colors.yellow}This is a safety guard. If you want to install anyway, run with --force${colors.reset}`);
+    if (!process.argv.includes('--force')) {
+      process.exit(0);
+    }
+  }
+}
+
+// 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'
+};
+
+// 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) {
+  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 (!force) {
+        console.log(`${colors.blue}○${colors.reset} ${existingConfig.name} already includes pace-core rules`);
+        console.log(`${colors.blue}  Use --force to update.${colors.reset}`);
+        return { action: 'skipped', file: existingConfig.name, format };
+      }
+      
+      console.log(`${colors.yellow}Updating existing ESLint config...${colors.reset}`);
+    } 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() {
+  checkSafetyGuards();
+  
+  const force = process.argv.includes('--force');
+  
+  if (force) {
+    console.log(`${colors.yellow}Warning: --force flag is set. This will overwrite existing configurations.${colors.reset}\n`);
+  }
+  
+  try {
+    console.log(`${colors.cyan}Setting up ESLint configuration from pace-core...${colors.reset}\n`);
+    const result = setupESLintConfig(force);
+    
+    // Summary
+    console.log(`\n${colors.bold}Setup Summary:${colors.reset}`);
+    if (result.action === 'created') {
+      console.log(`  ${colors.green}ESLint Config:${colors.reset} Created ${result.file} (${result.format})`);
+    } else if (result.action === 'updated') {
+      console.log(`  ${colors.green}ESLint Config:${colors.reset} Updated ${result.file} (${result.format})`);
+      console.log(`  ${colors.yellow}Backup:${colors.reset} ${path.basename(result.backup)}`);
+    } else {
+      console.log(`  ${colors.blue}ESLint Config:${colors.reset} Already configured (${result.file}, ${result.format})`);
+    }
+    
+    console.log(`\n${colors.cyan}ESLint configuration is now available${colors.reset}`);
+    console.log(`\n${colors.cyan}Next steps:${colors.reset}`);
+    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(`  • See ${colors.bold}packages/core/docs/standards/${colors.reset} for complete standards documentation`);
+    
+  } catch (error) {
+    console.error(`${colors.red}Error during setup:${colors.reset}`);
+    console.error(error.message);
+    if (error.stack) {
+      console.error(error.stack);
+    }
+    process.exit(1);
+  }
+}
+
+// Run if called directly
+if (require.main === module) {
+  main();
+}
+
+module.exports = { 
+  setupESLintConfig,
+  findESLintConfig
+};

package/src/__tests__/helpers/__tests__/component-test-utils.test.tsx

@@ -23,7 +23,7 @@ vi.mock('../test-utils', () => ({
 describe('[helpers] componentTestPatterns', () => {
     describe('testRenders', () => {
     it('creates test that renders component without crashing', () => {
-      const TestComponent = () => <div data-testid="test-component">Test</div>;
+      const TestComponent = () => <p data-testid="test-component">Test</p>;
       const testFn = componentTestPatterns.testRenders(TestComponent);
       
       expect(typeof testFn).toBe('function');
@@ -33,7 +33,7 @@ describe('[helpers] componentTestPatterns', () => {
     });
 
     it('creates test that renders component with props', () => {
-      const TestComponent = ({ name }: { name: string }) => <div data-testid="test-component">{name}</div>;
+      const TestComponent = ({ name }: { name: string }) => <p data-testid="test-component">{name}</p>;
       const testFn = componentTestPatterns.testRenders(TestComponent, { name: 'Test Name' });
       
       expect(typeof testFn).toBe('function');

package/src/__tests__/helpers/__tests__/test-providers.test.tsx

@@ -14,11 +14,11 @@ import { TestProviderWrapper, renderWithProviders, createMockUseUnifiedAuth, use
 function TestComponent() {
   const auth = useUnifiedAuth();
   return (
-    <div>
+    <p>
       <span data-testid="user-email">{auth.user?.email}</span>
       <span data-testid="is-authenticated">{auth.isAuthenticated ? 'true' : 'false'}</span>
       <span data-testid="app-name">{auth.appName}</span>
-    </div>
+    </p>
   );
 }
 

package/src/__tests__/helpers/__tests__/test-utils.test.tsx

@@ -41,32 +41,32 @@ vi.mock('../fixtures/test-data', () => ({
 
 describe('[helpers] renderWithProviders', () => {
     it('renders component with QueryClient provider by default', () => {
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     renderWithProviders(<TestComponent />);
     
-    expect(screen.getByTestId('test-component')).toBeInTheDocument();
+    expect(screen.getByTestId('test-component')).toBeDefined();
   });
 
   it('renders component without QueryClient when withQueryClient is false', () => {
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     renderWithProviders(<TestComponent />, { withQueryClient: false });
     
-    expect(screen.getByTestId('test-component')).toBeInTheDocument();
+    expect(screen.getByTestId('test-component')).toBeDefined();
   });
 
   it('uses custom QueryClient when provided', () => {
     const customQueryClient = new QueryClient();
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     renderWithProviders(<TestComponent />, { queryClient: customQueryClient });
     
-    expect(screen.getByTestId('test-component')).toBeInTheDocument();
+    expect(screen.getByTestId('test-component')).toBeDefined();
   });
 
   it('forwards additional render options', () => {
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     const { container } = renderWithProviders(<TestComponent />, {
       container: document.body
@@ -273,7 +273,7 @@ describe('[helpers] createMockSupabaseClient', () => {
 
   it('creates mock query builder with chaining methods', () => {
     const mockClient = createMockSupabaseClient();
-    const queryBuilder = mockClient.from('test_table');
+    const queryBuilder = (mockClient.from as any)('test_table');
     
     expect(queryBuilder).toHaveProperty('select');
     expect(queryBuilder).toHaveProperty('insert');
@@ -302,7 +302,7 @@ describe('[helpers] createMockSupabaseClient', () => {
 
 describe('[helpers] TestWrapper', () => {
     it('renders children with QueryClient provider', () => {
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     render(
       <TestWrapper>
@@ -310,7 +310,7 @@ describe('[helpers] TestWrapper', () => {
       </TestWrapper>
     );
     
-    expect(screen.getByTestId('test-component')).toBeInTheDocument();
+    expect(screen.getByTestId('test-component')).toBeDefined();
   });
 });