@jmruthers/pace-core 0.6.6 → 0.6.7

package/cursor-rules/11-performance-optimization.mdc

@@ -1,169 +0,0 @@
----
-description: Enforce performance optimization patterns including React memoization, database query optimization, and caching strategies
-globs: ["src/**/*.{ts,tsx,js,jsx}", "supabase/migrations/**/*.sql"]
-alwaysApply: false
-paceCoreVersion: "0.6.x"
-rulesVersion: "2025-01-28"
----
-# Performance Optimization Guide
-
-**📚 Human-Readable Standard**: See [11-performance-optimization.md](../../packages/core/docs/standards/11-performance-optimization.md) for complete documentation.
-
-This guide enforces performance optimization patterns to ensure fast, responsive applications.
-
-**AI Agent Instructions**: When writing performance-sensitive code, ALWAYS apply these optimizations. Use the decision trees below to determine when to optimize.
-
-## MUST: Optimize React Re-renders
-
-**MUST use memoization for expensive computations and stable references:**
-
-```tsx
-// ✅ CORRECT - Memoize expensive computation
-const expensiveValue = useMemo(() => computeExpensiveValue(data), [data]);
-
-// ✅ CORRECT - Stable callback reference
-const handleClick = useCallback(() => doSomething(id), [id]);
-
-// ✅ CORRECT - Memoize component
-const ExpensiveComponent = React.memo(({ data }) => <div>{/* ... */}</div>);
-
-// ❌ WRONG - Recomputes on every render
-const expensiveValue = computeExpensiveValue(data);
-const handleClick = () => doSomething(id);
-```
-
-## MUST: Avoid Creating New Objects in Render
-
-**MUST NOT create new objects/arrays in render:**
-
-```tsx
-// ❌ WRONG - New object on every render
-function Component({ items }) {
-  const config = { items, enabled: true };
-  return <Child config={config} />;
-}
-
-// ✅ CORRECT - Memoize object
-function Component({ items }) {
-  const config = useMemo(() => ({ items, enabled: true }), [items]);
-  return <Child config={config} />;
-}
-```
-
-## MUST: Use Helper Functions in RLS Policies
-
-**MUST use STABLE SECURITY DEFINER helper functions (never subqueries):**
-
-```sql
--- ✅ CORRECT - Helper function (evaluated once)
-CREATE POLICY "rbac_select_users" ON users FOR SELECT USING (
-  check_user_organisation_access(organisation_id)
-);
-
--- ❌ WRONG - Subquery (N+1 performance issue)
-CREATE POLICY "rbac_select_users" ON users FOR SELECT USING (
-  organisation_id IN (SELECT organisation_id FROM organisation_memberships WHERE user_id = auth.uid())
-);
-```
-
-## SHOULD: Lazy Load Heavy Components
-
-**SHOULD lazy load heavy components:**
-
-```tsx
-// ✅ CORRECT - Lazy load
-import { lazy, Suspense } from 'react';
-const HeavyComponent = lazy(() => import('./HeavyComponent'));
-<Suspense fallback={<Loading />}><HeavyComponent /></Suspense>
-```
-
-## SHOULD: Configure TanStack Query Cache
-
-**SHOULD configure appropriate cache times:**
-
-```tsx
-// ✅ CORRECT - Configure cache
-const queryClient = new QueryClient({
-  defaultOptions: {
-    queries: {
-      staleTime: 5 * 60 * 1000,
-      cacheTime: 10 * 60 * 1000,
-      retry: 1,
-      refetchOnWindowFocus: false,
-    },
-  },
-});
-```
-
-## SHOULD: Use RPC Functions for Complex Queries
-
-**SHOULD use RPC functions instead of complex client-side queries:**
-
-```tsx
-// ✅ CORRECT - RPC for complex query
-const { data } = await supabase.rpc('data_events_list', { organisation_id: orgId });
-
-// ❌ AVOID - Complex client-side query
-const { data } = await supabase.from('events').select('*, users(*), organisations(*)').eq('organisation_id', orgId);
-```
-
-## Decision Tree: Performance Optimization
-
-**ALWAYS follow this decision tree:**
-
-```
-1. Is this expensive computation?
-   ├─ YES → Use useMemo
-   └─ NO → Don't memoize (overhead not worth it)
-
-2. Is this function passed as prop?
-   ├─ YES → Use useCallback
-   └─ NO → Regular function is fine
-
-3. Is this component expensive to render?
-   ├─ YES → Use React.memo
-   └─ NO → Regular component is fine
-
-4. Is this creating new object/array in render?
-   ├─ YES → Memoize with useMemo
-   └─ NO → Continue
-
-5. Is this RLS policy?
-   ├─ YES → Use helper function (STABLE SECURITY DEFINER)
-   └─ NO → Continue
-
-6. Is this complex query?
-   ├─ YES → Use RPC function
-   └─ NO → Client-side query is fine
-```
-
-## Performance Checklist
-
-Before committing performance-sensitive code, verify:
-
-- [ ] Expensive computations memoized
-- [ ] Callbacks stable with useCallback
-- [ ] Components memoized when appropriate
-- [ ] No new objects/arrays in render
-- [ ] Heavy components lazy loaded
-- [ ] RLS policies use helper functions
-- [ ] Queries use indexes appropriately
-- [ ] No N+1 query patterns
-- [ ] TanStack Query configured properly
-- [ ] Bundle size optimized
-
-## Common Performance Pitfalls
-
-1. **Creating new objects in render** - Memoize with useMemo
-2. **Using inline functions** - Use useCallback
-3. **Over-memoizing** - Only memoize expensive operations
-4. **Subqueries in RLS** - Use helper functions
-5. **N+1 queries** - Use joins or RPC functions
-6. **Not lazy loading** - Lazy load heavy components
-7. **Poor cache configuration** - Configure TanStack Query properly
-
-## Reference
-
-- **Standard**: `packages/core/docs/standards/11-performance-optimization.md`
-- **RLS**: See `09-rbac-compliance.mdc` for RLS performance requirements
-- **Code Quality**: See `06-code-quality.mdc` for React patterns

package/cursor-rules/12-ci-cd-integration.mdc

@@ -1,150 +0,0 @@
----
-description: Enforce CI/CD integration patterns including automated quality checks, deployment processes, and security scanning
-globs: [".github/workflows/**/*.yml", ".github/workflows/**/*.yaml", "package.json"]
-alwaysApply: false
-paceCoreVersion: "0.6.x"
-rulesVersion: "2025-01-28"
----
-# CI/CD Integration Guide
-
-**📚 Human-Readable Standard**: See [12-ci-cd-integration.md](../../packages/core/docs/standards/12-ci-cd-integration.md) for complete documentation.
-
-This guide enforces CI/CD integration patterns to ensure automated quality checks and consistent deployments.
-
-**AI Agent Instructions**: When setting up CI/CD pipelines, ALWAYS include the required checks below. Use the decision tree to determine what checks are needed.
-
-## MUST: Include Required CI Checks
-
-**MUST run these checks in every CI pipeline:**
-
-```yaml
-# ✅ CORRECT - Required checks
-jobs:
-  quality-checks:
-    steps:
-      - name: Lint
-        run: npm run lint
-      - name: Type check
-        run: npx tsc --noEmit
-      - name: Run tests
-        run: npm run test
-      - name: Build
-        run: npm run build
-```
-
-## MUST: Have Required Scripts
-
-**MUST have these scripts in package.json:**
-
-```json
-// ✅ CORRECT - Required scripts
-{
-  "scripts": {
-    "lint": "eslint .",
-    "type-check": "tsc --noEmit",
-    "test": "vitest run",
-    "build": "vite build"
-  }
-}
-```
-
-## SHOULD: Include Additional Checks
-
-**SHOULD include these checks:**
-
-```yaml
-# ✅ RECOMMENDED - Additional checks
-- name: Format check
-  run: npm run format:check
-- name: Security scan
-  run: npm audit --audit-level=moderate
-- name: Coverage check
-  run: npm run test:coverage
-```
-
-## MUST: Use Secrets for Sensitive Data
-
-**MUST NOT commit secrets to repository:**
-
-```yaml
-# ✅ CORRECT - Use secrets
-env:
-  SUPABASE_URL: ${{ secrets.SUPABASE_URL }}
-  VITE_SUPABASE_ANON_KEY: ${{ secrets.VITE_SUPABASE_ANON_KEY }}
-
-# ❌ WRONG - Hardcoded secrets
-env:
-  SUPABASE_URL: "https://hardcoded-url.supabase.co"
-```
-
-## SHOULD: Deploy to Staging First
-
-**SHOULD deploy to staging before production:**
-
-```yaml
-# ✅ CORRECT - Staging first
-deploy-staging:
-  if: github.ref == 'refs/heads/develop'
-  environment: staging
-
-deploy-production:
-  needs: deploy-staging
-  if: github.ref == 'refs/heads/main'
-  environment: production
-```
-
-## Decision Tree: CI/CD Setup
-
-**ALWAYS follow this decision tree:**
-
-```
-1. What type of check is this?
-   ├─ Lint → MUST include
-   ├─ Type Check → MUST include
-   ├─ Tests → MUST include
-   ├─ Build → MUST include
-   ├─ Compliance → MUST include
-   ├─ Format → SHOULD include
-   ├─ Security → SHOULD include
-   └─ Coverage → SHOULD include
-
-2. Is this sensitive data?
-   ├─ YES → Use secrets (never commit)
-   └─ NO → Can be in config
-
-3. What environment?
-   ├─ Staging → Deploy on develop branch
-   └─ Production → Deploy on main branch only
-
-4. Are migrations needed?
-   ├─ YES → Run in staging first, test in CI
-   └─ NO → Continue
-```
-
-## CI/CD Checklist
-
-Before committing CI/CD configuration, verify:
-
-- [ ] Lint check configured
-- [ ] Type check configured
-- [ ] Tests run in CI
-- [ ] Build succeeds in CI
-- [ ] Security scan configured (recommended)
-- [ ] Environment variables set as secrets
-- [ ] Staging deployment configured
-- [ ] Production deployment configured
-- [ ] Migration strategy defined
-- [ ] Required scripts in package.json
-
-## Common CI/CD Mistakes
-
-1. **Missing required checks** - Include all MUST checks
-2. **Hardcoded secrets** - Always use secrets
-3. **No staging environment** - Deploy to staging first
-4. **No rollback plan** - Document rollback process
-
-## Reference
-
-- **Standard**: `packages/core/docs/standards/12-ci-cd-integration.md`
-- **Testing**: See `04-testing-standards.mdc` for test configuration
-- **Code Quality**: See `06-code-quality.mdc` for linting

package/dist/DataTable-LRJL4IRV.js

@@ -1,15 +0,0 @@
-export { ActionButtons, BulkOperationsDropdown, ColumnFactory, ColumnVisibilityDropdown, DataTable, DataTableCore, DataTableErrorBoundary, DataTableModals, DataTableToolbar, EditableRow, EmptyState, EnhancedPaginationControls, GroupHeader, GroupingDropdown, ImportModal, LoadingState, PaginationControls, UnifiedTableBody, announce, announceBulkOperation, announceFilterChange, announceLoadingState, announcePaginationChange, announceSearchResults, announceSelectionChange, announceSortChange, average, calculateAllDepths, calculateAllIndentation, calculateIndentation, calculateOptimalPageSize, cleanupLiveRegion, count, createHierarchicalStructure, defaultDataTableFeatures, exportToCSV, exportToCSVWithTableRows, generateCSVContent, getAriaSortValue, getCellRenderer, getColumnHeaderText, getHierarchicalSortConfig, getPageSizeOptions, getPaginationBinding, getRowDepth, getRowDescription, getRowIdSafe, getSortButtonLabel, groupHierarchicalData, hasValidRowId, initializeLiveRegion, isHierarchicalSortableColumn, max, min, normalizeDataTableFeatures, shouldShowColumnForRow, sortHierarchicalDataByStructure, sortHierarchicalDataWithSorting, sum, validateHierarchicalData, validatePaginationConfig } from './chunk-ZS5VO5JB.js';
-import './chunk-7TYHROIV.js';
-export { CircuitBreaker, DEFAULT_FALLBACK_CONFIG, DataChunkManager, DataTableError, DataTableErrorType, ErrorRecoveryManager, MemoryMonitor, SearchIndex, VisibilityTracker, chunkData, debounce, determinePaginationMode, getOptimalPageSizeOptions, safeExecute, throttle, useDataTablePerformance } from './chunk-S7DKJPLT.js';
-import './chunk-3O3WHILE.js';
-import './chunk-C7NSAPTL.js';
-import './chunk-FTCRZOG2.js';
-import './chunk-ZFYPMX46.js';
-import './chunk-AHU7G2R5.js';
-import './chunk-4SXLQIZO.js';
-import './chunk-3QC3KRHK.js';
-import './chunk-A55DK444.js';
-import './chunk-SD6WQY43.js';
-import './chunk-HF6O3O37.js';
-import './chunk-TTRFSOKR.js';
-import './chunk-3RG5ZIWI.js';

package/dist/eslint-rules/rules/compliance.cjs

@@ -1,348 +0,0 @@
-/**
- * General compliance rules
- * @package @jmruthers/pace-core
- * @module ESLintRules/rules/compliance
- */
-
-const path = require('path');
-const { getPaceCoreComponents, getPaceCoreHooks, getPaceCoreUtils } = require('../utils/manifest-loader.cjs');
-const { isPaceCoreSourceFile } = require('../utils/helpers.cjs');
-
-module.exports = {
-  rules: {
-    /**
-     * Prefer pace-core components over native HTML elements or custom implementations
-     */
-    'prefer-pace-core-components': {
-      meta: {
-        type: 'suggestion',
-        docs: {
-          description: 'Suggest using pace-core components instead of native HTML elements',
-          category: 'Best Practices',
-          recommended: true
-        },
-        hasSuggestions: true,
-        messages: {
-          preferButton: "Use 'Button' component from '@jmruthers/pace-core' instead of <button>",
-          preferInput: "Use 'Input' component from '@jmruthers/pace-core' instead of <input>",
-          preferTextarea: "Use 'Textarea' component from '@jmruthers/pace-core' instead of <textarea>",
-          preferLabel: "Use 'Label' component from '@jmruthers/pace-core' instead of <label>",
-          preferForm: "Use 'Form' component from '@jmruthers/pace-core' instead of custom form implementation"
-        }
-      },
-      create(context) {
-        const filename = context.getFilename();
-        
-        // Exclude pace-core source files - these rules are for consuming apps
-        if (isPaceCoreSourceFile(filename)) {
-          return {};
-        }
-        
-        const paceCoreComponents = getPaceCoreComponents();
-        
-        return {
-          JSXOpeningElement(node) {
-            const elementName = node.name.name;
-            
-            if (!elementName) return;
-            
-            // Only flag lowercase native HTML elements, not capitalized components
-            // Native HTML: <button>, <input>, <label>, <textarea>
-            // Components: <Button>, <Input>, <Label>, <Textarea>
-            const isNativeHTMLElement = elementName === elementName.toLowerCase();
-            
-            if (!isNativeHTMLElement) {
-              // This is a capitalized component, not a native HTML element
-              return;
-            }
-            
-            // Check for native HTML elements that have pace-core alternatives
-            const nativeToPaceCore = {
-              'button': 'Button',
-              'input': 'Input',
-              'textarea': 'Textarea',
-              'label': 'Label'
-            };
-            
-            if (nativeToPaceCore[elementName]) {
-              const paceCoreComponent = nativeToPaceCore[elementName];
-              if (paceCoreComponents.includes(paceCoreComponent)) {
-                context.report({
-                  node,
-                  messageId: `prefer${paceCoreComponent}`,
-                  suggest: [{
-                    desc: `Import and use ${paceCoreComponent} from pace-core`,
-                    fix(fixer) {
-                      // This is a complex fix, so we'll just report
-                      return null;
-                    }
-                  }]
-                });
-              }
-            }
-          }
-        };
-      }
-    },
-
-    /**
-     * Detect custom hooks that duplicate pace-core functionality
-     */
-    'prefer-pace-core-hooks': {
-      meta: {
-        type: 'suggestion',
-        docs: {
-          description: 'Suggest using pace-core hooks instead of custom implementations',
-          category: 'Best Practices',
-          recommended: true
-        },
-        messages: {
-          preferPaceCoreHook: "Consider using '{{hook}}' from '@jmruthers/pace-core' instead of custom hook '{{customHook}}'"
-        }
-      },
-      create(context) {
-        const filename = context.getFilename();
-        
-        // Exclude pace-core source files - these rules are for consuming apps
-        if (isPaceCoreSourceFile(filename)) {
-          return {};
-        }
-        
-        const paceCoreHooks = getPaceCoreHooks();
-        const hookPatterns = {
-          'useToast': ['useToast', 'useNotification', 'useSnackbar'],
-          'useDebounce': ['useDebounce', 'useDebounced'],
-          'useAuth': ['useAuth', 'useAuthentication', 'useUser'],
-          'useFile': ['useFile', 'useFileUpload', 'useFileReference'],
-          'useForm': ['useForm', 'useZodForm'],
-          'useTable': ['useTable', 'useDataTable']
-        };
-        
-        return {
-          FunctionDeclaration(node) {
-            const functionName = node.id?.name;
-            if (!functionName || !functionName.startsWith('use')) return;
-            
-            // Check if this looks like a hook that pace-core provides
-            for (const [paceCoreHook, patterns] of Object.entries(hookPatterns)) {
-              if (paceCoreHooks.includes(paceCoreHook)) {
-                for (const pattern of patterns) {
-                  if (functionName.toLowerCase().includes(pattern.toLowerCase().replace('use', ''))) {
-                    context.report({
-                      node: node.id,
-                      messageId: 'preferPaceCoreHook',
-                      data: {
-                        hook: paceCoreHook,
-                        customHook: functionName
-                      }
-                    });
-                    return;
-                  }
-                }
-              }
-            }
-          }
-        };
-      }
-    },
-
-    /**
-     * Detect utility functions that duplicate pace-core functionality
-     */
-    'prefer-pace-core-utils': {
-      meta: {
-        type: 'suggestion',
-        docs: {
-          description: 'Suggest using pace-core utilities instead of custom implementations',
-          category: 'Best Practices',
-          recommended: true
-        },
-        messages: {
-          preferPaceCoreUtil: "Consider using '{{util}}' from '@jmruthers/pace-core' instead of custom function '{{customUtil}}'"
-        }
-      },
-      create(context) {
-        const filename = context.getFilename();
-        
-        // Exclude pace-core source files - these rules are for consuming apps
-        if (isPaceCoreSourceFile(filename)) {
-          return {};
-        }
-        
-        const paceCoreUtils = getPaceCoreUtils();
-        const utilPatterns = {
-          'formatDate': ['formatDate', 'formatDateTime', 'dateFormat'],
-          'formatCurrency': ['formatCurrency', 'formatMoney', 'currencyFormat'],
-          'formatNumber': ['formatNumber', 'numberFormat'],
-          'cn': ['cn', 'classNames', 'clsx', 'mergeClasses'],
-          'validateUserInput': ['validateInput', 'validateUser', 'validateUserInput'],
-          'sanitizeUserInput': ['sanitize', 'sanitizeInput', 'sanitizeUser']
-        };
-        
-        // Patterns that should NOT trigger validateUserInput warnings
-        // These are for API/config validation, not user input validation
-        const excludePatterns = [
-          'validateApi',
-          'validateRequest',
-          'validateConfig',
-          'validateKey',
-          'validateToken',
-          'validateAuth',
-          'validateRateLimit',
-          'validatePermission',
-          'validateAccess'
-        ];
-        
-        return {
-          FunctionDeclaration(node) {
-            const functionName = node.id?.name;
-            if (!functionName) return;
-            
-            // Skip if function name matches exclude patterns (API/config validation, not user input)
-            const lowerName = functionName.toLowerCase();
-            if (excludePatterns.some(pattern => lowerName.includes(pattern.toLowerCase()))) {
-              return;
-            }
-            
-            // Check if this looks like a util that pace-core provides
-            for (const [paceCoreUtil, patterns] of Object.entries(utilPatterns)) {
-              if (paceCoreUtils.includes(paceCoreUtil)) {
-                for (const pattern of patterns) {
-                  // Only match if the pattern is a significant part of the function name
-                  // Don't match generic "validate" - it's too broad
-                  if (paceCoreUtil === 'validateUserInput' && pattern === 'validate') {
-                    // Skip generic "validate" - too many false positives
-                    continue;
-                  }
-                  
-                  if (functionName.toLowerCase().includes(pattern.toLowerCase())) {
-                    context.report({
-                      node: node.id,
-                      messageId: 'preferPaceCoreUtil',
-                      data: {
-                        util: paceCoreUtil,
-                        customUtil: functionName
-                      }
-                    });
-                    return;
-                  }
-                }
-              }
-            }
-          }
-        };
-      }
-    },
-
-    /**
-     * Detect component files with names matching pace-core components
-     */
-    'no-local-component-duplication': {
-      meta: {
-        type: 'problem',
-        docs: {
-          description: 'Disallow local components with names matching pace-core components',
-          category: 'Best Practices',
-          recommended: true
-        },
-        messages: {
-          duplicateComponent: "Component '{{componentName}}' conflicts with pace-core component. Use '@jmruthers/pace-core' instead of creating a local version."
-        }
-      },
-      create(context) {
-        const filename = context.getFilename();
-        
-        // Exclude pace-core source files - these rules are for consuming apps
-        if (isPaceCoreSourceFile(filename)) {
-          return {};
-        }
-        
-        const paceCoreComponents = getPaceCoreComponents();
-        
-        // Only check component files (components/, src/components/, etc.)
-        if (!filename.match(/(components|Components)\//)) {
-          return {};
-        }
-        
-        // Extract component name from filename
-        const basename = path.basename(filename, path.extname(filename));
-        const componentName = basename.replace(/\.(test|spec)$/, '');
-        
-        return {
-          Program(node) {
-            // Check if this file exports a component with a name matching pace-core
-            if (paceCoreComponents.includes(componentName)) {
-              // Check if file exports this component
-              const hasExport = node.body.some(stmt => {
-                if (stmt.type === 'ExportNamedDeclaration') {
-                  return stmt.declaration?.id?.name === componentName ||
-                         stmt.specifiers?.some(spec => spec.exported.name === componentName);
-                }
-                if (stmt.type === 'ExportDefaultDeclaration') {
-                  return stmt.declaration?.id?.name === componentName ||
-                         stmt.declaration?.name === componentName;
-                }
-                return false;
-              });
-              
-              if (hasExport) {
-                context.report({
-                  node,
-                  messageId: 'duplicateComponent',
-                  data: {
-                    componentName
-                  }
-                });
-              }
-            }
-          }
-        };
-      }
-    },
-
-    /**
-     * Disallow inline styles
-     */
-    'no-inline-styles': {
-      meta: {
-        type: 'problem',
-        docs: {
-          description: 'Disallow inline styles. Use pace-core components or Tailwind classes instead.',
-          category: 'Best Practices',
-          recommended: true
-        },
-        messages: {
-          inlineStyle: 'Inline style detected. Use pace-core components or Tailwind classes instead.'
-        },
-        hasSuggestions: true
-      },
-      create(context) {
-        const filename = context.getFilename();
-        
-        // Exclude pace-core source files - these rules are for consuming apps
-        if (isPaceCoreSourceFile(filename)) {
-          return {};
-        }
-        
-        return {
-          JSXAttribute(node) {
-            if (node.name && node.name.name === 'style') {
-              context.report({
-                node,
-                messageId: 'inlineStyle',
-                suggest: [{
-                  desc: 'Remove inline style and use pace-core component styling or Tailwind utility classes',
-                  fix(fixer) {
-                    // Remove the style attribute
-                    return fixer.remove(node);
-                  }
-                }]
-              });
-            }
-          }
-        };
-      }
-    }
-  }
-};
-