# Run all verification checks
npm run lint && npm run typecheck && npm run test

# Or use a combined script
npm run verify

// ❌ BAD: Implicit any
function processData(data) {
  return data.map((item) => item.name);
}

// ✅ GOOD: Explicit types
function processData(data: User[]): string[] {
  return data.map((item) => item.name);
}

// ❌ BAD: Missing null check
function getUsername(user: User | null): string {
  return user.name; // Error: user might be null
}

// ✅ GOOD: Null handling
function getUsername(user: User | null): string {
  return user?.name ?? 'Anonymous';
}

// ✅ ALSO GOOD: Early return
function getUsername(user: User | null): string {
  if (!user) return 'Anonymous';
  return user.name;
}

// ❌ BAD: Overly generic
function fetch<T>(url: string): Promise<T> {
  return fetch(url).then((r) => r.json());
}

// ✅ GOOD: Constrained generics
function fetchData<T extends Record<string, unknown>>(
  url: string,
  schema: z.ZodSchema<T>,
): Promise<T> {
  return fetch(url)
    .then((r) => r.json())
    .then((data) => schema.parse(data));
}

// ❌ BAD: No cleanup
useEffect(() => {
  window.addEventListener('resize', handleResize);
}, []);

// ✅ GOOD: With cleanup
useEffect(() => {
  window.addEventListener('resize', handleResize);
  return () => window.removeEventListener('resize', handleResize);
}, []);

// ❌ BAD: Uncleaned subscription
useEffect(() => {
  const subscription = observable.subscribe(handler);
}, []);

// ✅ GOOD: With cleanup
useEffect(() => {
  const subscription = observable.subscribe(handler);
  return () => subscription.unsubscribe();
}, []);

// ❌ BAD: No cancellation
useEffect(() => {
  fetch('/api/data')
    .then((r) => r.json())
    .then(setData);
}, []);

// ✅ GOOD: With abort
useEffect(() => {
  const controller = new AbortController();

  fetch('/api/data', { signal: controller.signal })
    .then((r) => r.json())
    .then(setData)
    .catch((e) => {
      if (e.name !== 'AbortError') throw e;
    });

  return () => controller.abort();
}, []);

// ❌ BAD: No cleanup
useEffect(() => {
  setInterval(() => tick(), 1000);
}, []);

// ✅ GOOD: With cleanup
useEffect(() => {
  const id = setInterval(() => tick(), 1000);
  return () => clearInterval(id);
}, []);

// ❌ BAD: Missing dependency
useEffect(() => {
  fetchUser(userId);
}, []); // userId missing!

// ✅ GOOD: Complete dependencies
useEffect(() => {
  fetchUser(userId);
}, [userId]);

// ✅ GOOD: Stable callback reference
const fetchUserCallback = useCallback(() => {
  fetchUser(userId);
}, [userId]);

// ❌ BAD: Index as key (unstable)
{items.map((item, index) => (
  <Item key={index} data={item} />
))}

// ✅ GOOD: Unique, stable key
{items.map(item => (
  <Item key={item.id} data={item} />
))}

// ❌ BAD: Unnecessary re-renders
const ExpensiveList = ({ items, onSelect }) => {
  const sorted = items.sort((a, b) => a.name.localeCompare(b.name));
  return sorted.map(item => <Item key={item.id} {...item} />);
};

// ✅ GOOD: Memoized computation
const ExpensiveList = ({ items, onSelect }) => {
  const sorted = useMemo(
    () => [...items].sort((a, b) => a.name.localeCompare(b.name)),
    [items]
  );
  return sorted.map(item => <Item key={item.id} {...item} />);
};

// ❌ BAD: Deeply nested
function processUser(user: User | null) {
  if (user) {
    if (user.isActive) {
      if (user.hasPermission) {
        return performAction(user);
      }
    }
  }
  return null;
}

// ✅ GOOD: Guard clauses
function processUser(user: User | null) {
  if (!user) return null;
  if (!user.isActive) return null;
  if (!user.hasPermission) return null;

  return performAction(user);
}

// ❌ BAD: Does too much
function handleSubmit(formData: FormData) {
  // Validates
  // Transforms
  // Makes API call
  // Updates state
  // Shows notification
  // Logs analytics
}

// ✅ GOOD: Single responsibility
function handleSubmit(formData: FormData) {
  const validated = validateFormData(formData);
  const transformed = transformForAPI(validated);
  return submitToAPI(transformed);
}

// ❌ BAD: Unclear names
const d = new Date();
const x = users.filter((u) => u.a);
const fn = () => {
  /* ... */
};

// ✅ GOOD: Descriptive names
const createdAt = new Date();
const activeUsers = users.filter((user) => user.isActive);
const handleFormSubmit = () => {
  /* ... */
};

// ✅ GOOD: Error boundary for fallible code
<ErrorBoundary fallback={<ErrorFallback />}>
  <FeatureComponent />
</ErrorBoundary>

// ❌ BAD: Swallowed error
try {
  await riskyOperation();
} catch {
  // Silent failure
}

// ✅ GOOD: Explicit handling
try {
  await riskyOperation();
} catch (error) {
  console.error('Operation failed:', error);
  showErrorNotification('Something went wrong');
  reportToErrorTracker(error);
}

// ❌ BAD: Forces multiple reflows
elements.forEach((el) => {
  const width = el.offsetWidth; // Read
  el.style.width = width + 10 + 'px'; // Write
});

// ✅ GOOD: Batch reads and writes
const widths = elements.map((el) => el.offsetWidth); // All reads
elements.forEach((el, i) => {
  el.style.width = widths[i] + 10 + 'px'; // All writes
});

// ✅ GOOD: Lazy load heavy components
const HeavyChart = lazy(() => import('./HeavyChart'));

function Dashboard() {
  return (
    <Suspense fallback={<ChartSkeleton />}>
      <HeavyChart />
    </Suspense>
  );
}

### Type Safety

- [ ] No implicit `any` types
- [ ] No `@ts-ignore` without justification
- [ ] Proper null/undefined handling
- [ ] Generic types used where appropriate
- [ ] No type assertions without validation

### Memory & Resources

- [ ] Event listeners cleaned up
- [ ] Subscriptions unsubscribed
- [ ] Timers cleared
- [ ] Fetch requests cancellable
- [ ] No circular references

### React Patterns

- [ ] useEffect dependencies complete
- [ ] useMemo/useCallback used appropriately
- [ ] Keys are stable and unique
- [ ] Error boundaries for fallible code
- [ ] No state updates on unmounted components

### Code Quality

- [ ] Early return pattern used
- [ ] Single responsibility principle
- [ ] Descriptive naming
- [ ] No magic numbers/strings
- [ ] Comments explain "why", not "what"

### Error Handling

- [ ] Errors caught and handled
- [ ] User-friendly error messages
- [ ] Errors logged appropriately
- [ ] Recovery paths defined

### Performance

- [ ] No unnecessary re-renders
- [ ] Heavy computations memoized
- [ ] Lazy loading for large components
- [ ] No layout thrashing
- [ ] Images optimized

{
  "rules": {
    "@typescript-eslint/no-explicit-any": "error",
    "@typescript-eslint/no-unused-vars": "error",
    "@typescript-eslint/strict-boolean-expressions": "warn",
    "react-hooks/rules-of-hooks": "error",
    "react-hooks/exhaustive-deps": "warn",
    "no-console": ["warn", { "allow": ["warn", "error"] }]
  }
}

{
  "compilerOptions": {
    "strict": true,
    "noImplicitAny": true,
    "strictNullChecks": true,
    "noUncheckedIndexedAccess": true,
    "noImplicitReturns": true,
    "noFallthroughCasesInSwitch": true
  }
}

# Type checking
npm run typecheck

# Linting
npm run lint
npm run lint:fix

# All tests
npm run test

# Combined verification
npm run verify

# Find unused exports
npx ts-prune

# Analyze bundle size
npx vite-bundle-visualizer

## Code Review: [PR Title]

### Verification Status

| Check          | Status | Notes |
| -------------- | ------ | ----- |
| Type Safety    | ✅/❌  |       |
| Memory Leaks   | ✅/❌  |       |
| React Patterns | ✅/❌  |       |
| Error Handling | ✅/❌  |       |
| Tests          | ✅/❌  |       |
| Performance    | ✅/❌  |       |

### Issues Found

- [ ] Issue 1: Description
- [ ] Issue 2: Description

### Suggestions

1. Consider...
2. Could improve...

### Approved: Yes/No