claude-dev-env 1.0.0

package/hooks/validators/test_files/04_pure_component_import.tsx

@@ -0,0 +1,10 @@
+// TEST: PureComponent without React prefix (SHOULD BE CAUGHT but might not be)
+import { PureComponent } from 'react';
+
+class MyComponent extends PureComponent {
+  render() {
+    return <div>Hello</div>;
+  }
+}
+
+export default MyComponent;

package/hooks/validators/test_files/05_typescript_generics.tsx

@@ -0,0 +1,14 @@
+// TEST: Class component with TypeScript generics (SHOULD BE CAUGHT but might not be)
+import React from 'react';
+
+interface Props {
+  name: string;
+}
+
+class MyComponent extends React.Component<Props> {
+  render() {
+    return <div>{this.props.name}</div>;
+  }
+}
+
+export default MyComponent;

package/hooks/validators/test_files/06_typescript_two_generics.tsx

@@ -0,0 +1,18 @@
+// TEST: Class component with Props and State generics (SHOULD BE CAUGHT but might not be)
+import React from 'react';
+
+interface Props {
+  name: string;
+}
+
+interface State {
+  count: number;
+}
+
+class MyComponent extends React.Component<Props, State> {
+  render() {
+    return <div>{this.props.name}</div>;
+  }
+}
+
+export default MyComponent;

package/hooks/validators/test_files/07_multiline_declaration.tsx

@@ -0,0 +1,11 @@
+// TEST: Multi-line class declaration (SHOULD BE CAUGHT but might not be)
+import React from 'react';
+
+class MyComponent
+  extends React.Component {
+  render() {
+    return <div>Hello</div>;
+  }
+}
+
+export default MyComponent;

package/hooks/validators/test_files/08_error_boundary_valid.tsx

@@ -0,0 +1,14 @@
+// TEST: Valid error boundary (SHOULD BE ALLOWED)
+import React from 'react';
+
+class ErrorBoundary extends React.Component {
+  componentDidCatch(error: Error, errorInfo: React.ErrorInfo) {
+    console.error(error, errorInfo);
+  }
+
+  render() {
+    return this.props.children;
+  }
+}
+
+export default ErrorBoundary;

package/hooks/validators/test_files/09_error_boundary_with_other_class.tsx

@@ -0,0 +1,20 @@
+// TEST: Error boundary + another class component in same file (OTHER CLASS SHOULD BE CAUGHT)
+import React from 'react';
+
+class ErrorBoundary extends React.Component {
+  componentDidCatch(error: Error, errorInfo: React.ErrorInfo) {
+    console.error(error, errorInfo);
+  }
+
+  render() {
+    return this.props.children;
+  }
+}
+
+class RegularComponent extends React.Component {
+  render() {
+    return <div>This should be flagged!</div>;
+  }
+}
+
+export default ErrorBoundary;

package/hooks/validators/test_files/10_inheritance_chain.tsx

@@ -0,0 +1,16 @@
+// TEST: Inheritance chain (SHOULD BE CAUGHT but won't be)
+import React from 'react';
+
+class BaseComponent extends React.Component {
+  commonMethod() {
+    return 'base';
+  }
+}
+
+class MyComponent extends BaseComponent {
+  render() {
+    return <div>{this.commonMethod()}</div>;
+  }
+}
+
+export default MyComponent;

package/hooks/validators/test_files/11_ts_file.ts

@@ -0,0 +1,10 @@
+// TEST: Class component in .ts file (SHOULD BE IGNORED - correct behavior)
+import React from 'react';
+
+class MyComponent extends React.Component {
+  render() {
+    return React.createElement('div', null, 'Hello');
+  }
+}
+
+export default MyComponent;

package/hooks/validators/test_files/12_non_react_class.tsx

@@ -0,0 +1,14 @@
+// TEST: Class that extends non-React component (SHOULD BE ALLOWED)
+class CustomBase {
+  doSomething() {
+    return 'custom';
+  }
+}
+
+class MyClass extends CustomBase {
+  render() {
+    return this.doSomething();
+  }
+}
+
+export default MyClass;

package/hooks/validators/test_files/13_functional_component.tsx

@@ -0,0 +1,8 @@
+// TEST: Functional component (SHOULD BE ALLOWED)
+import React from 'react';
+
+const MyComponent: React.FC = () => {
+  return <div>Hello</div>;
+};
+
+export default MyComponent;

package/hooks/validators/test_files/14_indented_class.tsx

@@ -0,0 +1,13 @@
+// TEST: Heavily indented class component (SHOULD BE CAUGHT)
+import React from 'react';
+
+function createComponent() {
+      class MyComponent extends React.Component {
+        render() {
+          return <div>Hello</div>;
+        }
+      }
+  return MyComponent;
+}
+
+export default createComponent();

package/hooks/validators/test_files/15_getDerivedStateFromError.tsx

@@ -0,0 +1,14 @@
+// TEST: Error boundary with getDerivedStateFromError (SHOULD BE ALLOWED)
+import React from 'react';
+
+class ErrorBoundary extends React.Component {
+  static getDerivedStateFromError(error: Error) {
+    return { hasError: true };
+  }
+
+  render() {
+    return this.props.children;
+  }
+}
+
+export default ErrorBoundary;

package/hooks/validators/test_files/16_mixed_components.tsx

@@ -0,0 +1,20 @@
+// TEST: Mixed Component, PureComponent, and functional (should catch first two)
+import React, { Component, PureComponent } from 'react';
+
+class RegularComponent extends Component {
+  render() {
+    return <div>Regular</div>;
+  }
+}
+
+class OptimizedComponent extends PureComponent {
+  render() {
+    return <div>Optimized</div>;
+  }
+}
+
+const FunctionalComponent: React.FC = () => {
+  return <div>Functional</div>;
+};
+
+export { RegularComponent, OptimizedComponent, FunctionalComponent };

package/hooks/validators/test_files/EXECUTIVE_SUMMARY.md

@@ -0,0 +1,175 @@
+# React Validator Audit - Executive Summary
+
+## Validation Status: FUNCTIONAL WITH CRITICAL GAPS
+
+---
+
+## Critical Findings
+
+### GAP 1: PureComponent Not Detected (CRITICAL - MUST FIX)
+
+**Pattern:** `React.PureComponent` and `PureComponent` are NOT caught
+
+**Test Evidence:**
+```bash
+$ python react_checks.py test_files/16_mixed_components.tsx
+test_files/16_mixed_components.tsx:4: Use functional components...
+# Only caught Component, missed PureComponent on line 9
+```
+
+**File Content:**
+```tsx
+class RegularComponent extends Component { }      // Line 4 - CAUGHT ✓
+class OptimizedComponent extends PureComponent { } // Line 9 - MISSED ✗
+```
+
+**Impact:** HIGH - PureComponent is commonly used for performance optimization. Developers will bypass the check by using PureComponent instead of Component.
+
+**Fix:** Add to regex pattern:
+```python
+CLASS_COMPONENT_PATTERN = re.compile(
+    r'^\s*class\s+\w+\s+extends\s+(Component|React\.Component|PureComponent|React\.PureComponent)\b',
+    re.MULTILINE
+)
+```
+
+---
+
+### GAP 2: File-Level Error Boundary Exception (MEDIUM - SHOULD FIX)
+
+**Pattern:** If file contains error boundary methods, ENTIRE file is skipped
+
+**Test Evidence:**
+```tsx
+// File: 09_error_boundary_with_other_class.tsx
+class ErrorBoundary extends React.Component {
+  componentDidCatch() { /* ... */ }  // This makes entire file skip
+}
+
+class RegularComponent extends React.Component {
+  render() { return <div>Should be flagged!</div>; }  // NOT CAUGHT ✗
+}
+```
+
+**Impact:** MEDIUM - Rare to have multiple classes in one file, but it's a logic flaw. The exception should be class-level, not file-level.
+
+**Fix:** More complex - requires checking each class individually:
+1. Find all class components
+2. For each class, check if IT implements error boundary methods
+3. Skip only specific error boundary classes
+4. Flag other classes
+
+---
+
+### GAP 3: Inheritance Chain Detection (LOW - DOCUMENT)
+
+**Pattern:** Only direct inheritance from React.Component is caught
+
+**Test Evidence:**
+```tsx
+class BaseComponent extends React.Component { }  // CAUGHT ✓
+class MyComponent extends BaseComponent { }       // NOT CAUGHT ✗
+```
+
+**Impact:** LOW - Base class is caught, which forces refactoring anyway. Detecting full inheritance chains requires complex static analysis.
+
+**Fix:** Document limitation in docstring. Not worth the complexity to fix.
+
+---
+
+## Test Results Summary
+
+| Category | Count | Tests |
+|----------|-------|-------|
+| **Correctly Caught** | 7 | 01, 02, 05, 06, 07, 10 (base only), 14 |
+| **Missed Violations (GAPS)** | 3 | 03, 04, 09 |
+| **Correctly Allowed** | 5 | 08, 11, 12, 13, 15 |
+| **Total Tests** | 15 | |
+
+---
+
+## What Works Well
+
+1. **TypeScript Generics** - Handles `Component<Props>` and `Component<Props, State>` ✓
+2. **Multi-line Declarations** - Catches `class Foo\n  extends Component` ✓
+3. **Indentation** - Works regardless of indentation level ✓
+4. **Import Variations** - Catches both `React.Component` and `Component` ✓
+5. **File Filtering** - Correctly ignores `.ts` files, only checks `.tsx`/`.jsx` ✓
+6. **Error Boundaries** - Allows `componentDidCatch` and `getDerivedStateFromError` ✓
+
+---
+
+## Recommended Actions
+
+### Priority 1: Fix PureComponent Gap (CRITICAL)
+
+**Time:** 5 minutes
+**Risk:** Very low - simple regex update
+**Impact:** Closes critical bypass route
+
+```python
+CLASS_COMPONENT_PATTERN = re.compile(
+    r'^\s*class\s+\w+\s+extends\s+(Component|React\.Component|PureComponent|React\.PureComponent)\b',
+    re.MULTILINE
+)
+```
+
+### Priority 2: Document Limitations (LOW EFFORT)
+
+**Time:** 2 minutes
+**Risk:** None
+
+Add to docstring:
+```python
+"""Check that no class components exist (except error boundaries).
+
+Detects:
+- class X extends Component
+- class X extends React.Component
+- class X extends PureComponent
+- class X extends React.PureComponent
+
+Limitations:
+- Only detects direct inheritance (not inheritance chains)
+- File-level error boundary exception (skips entire file if error boundary found)
+"""
+```
+
+### Priority 3: Class-Level Error Boundary Check (FUTURE)
+
+**Time:** 1-2 hours
+**Risk:** Medium - complex logic change
+**Impact:** Fixes logic flaw for edge case
+
+Consider for future iteration if multi-class files become common.
+
+---
+
+## Test Files Available
+
+All test files in `test_files/` directory:
+- `01-16_*.tsx` - Individual test cases
+- `VALIDATION_REPORT.md` - Detailed test results
+- `EXECUTIVE_SUMMARY.md` - This file
+
+Run all tests:
+```bash
+cd hooks/validators
+for file in test_files/*.tsx; do
+  echo "=== $file ==="
+  python react_checks.py "$file"
+  echo
+done
+```
+
+---
+
+## Conclusion
+
+The validator is **production-ready with one critical fix needed**:
+
+1. **MUST FIX NOW:** Add PureComponent to pattern (5 min)
+2. **SHOULD DOCUMENT:** Add limitations to docstring (2 min)
+3. **CONSIDER LATER:** Class-level error boundary check (future iteration)
+
+After fixing PureComponent gap, the validator will catch 90%+ of real-world class component violations.

package/hooks/validators/test_files/TEST_RESULTS_TABLE.txt

@@ -0,0 +1,60 @@
+================================================================================
+REACT CLASS COMPONENT VALIDATOR - COMPREHENSIVE TEST RESULTS
+================================================================================
+
+TEST | FILE                              | EXPECTED    | ACTUAL      | STATUS
+-----|-----------------------------------|-------------|-------------|--------
+01   | 01_basic_component.tsx            | CAUGHT      | CAUGHT      | PASS ✓
+02   | 02_component_without_react.tsx    | CAUGHT      | CAUGHT      | PASS ✓
+03   | 03_pure_component.tsx             | CAUGHT      | NOT CAUGHT  | FAIL ✗ GAP1
+04   | 04_pure_component_import.tsx      | CAUGHT      | NOT CAUGHT  | FAIL ✗ GAP1
+05   | 05_typescript_generics.tsx        | CAUGHT      | CAUGHT      | PASS ✓
+06   | 06_typescript_two_generics.tsx    | CAUGHT      | CAUGHT      | PASS ✓
+07   | 07_multiline_declaration.tsx      | CAUGHT      | CAUGHT      | PASS ✓
+08   | 08_error_boundary_valid.tsx       | ALLOWED     | ALLOWED     | PASS ✓
+09   | 09_error_boundary_with_other.tsx  | CAUGHT(2nd) | NOT CAUGHT  | FAIL ✗ GAP2
+10   | 10_inheritance_chain.tsx          | CAUGHT(1st) | CAUGHT(1st) | PASS ✓*
+11   | 11_ts_file.ts                     | IGNORED     | IGNORED     | PASS ✓
+12   | 12_non_react_class.tsx            | ALLOWED     | ALLOWED     | PASS ✓
+13   | 13_functional_component.tsx       | ALLOWED     | ALLOWED     | PASS ✓
+14   | 14_indented_class.tsx             | CAUGHT      | CAUGHT      | PASS ✓
+15   | 15_getDerivedStateFromError.tsx   | ALLOWED     | ALLOWED     | PASS ✓
+16   | 16_mixed_components.tsx           | CAUGHT(2)   | CAUGHT(1)   | FAIL ✗ GAP1
+
+SUMMARY:
+--------
+Total Tests:           16
+Passes:                11 (69%)
+Failures:              5  (31%)
+Critical Gaps:         2  (GAP1, GAP2)
+
+* Test 10: Only catches base class, not child (documented limitation - GAP3)
+
+================================================================================
+GAPS IDENTIFIED:
+================================================================================
+
+GAP 1: PureComponent Not Detected (CRITICAL)
+---------------------------------------------
+Affected Tests: 03, 04, 16
+Pattern Missing: React.PureComponent, PureComponent
+Fix: Add to regex pattern
+Impact: HIGH - Common bypass route
+
+GAP 2: File-Level Error Boundary Exception (MEDIUM)
+----------------------------------------------------
+Affected Tests: 09
+Issue: Entire file skipped if error boundary found
+Fix: Class-level detection (complex)
+Impact: MEDIUM - Rare edge case
+
+GAP 3: Inheritance Chain Detection (LOW)
+-----------------------------------------
+Affected Tests: 10 (partial)
+Issue: Only direct inheritance caught
+Fix: Document limitation
+Impact: LOW - Base class caught anyway
+
+================================================================================
+RECOMMENDATION: Fix GAP 1 immediately (5-minute regex update)
+================================================================================

package/hooks/validators/test_files/VALIDATION_REPORT.md

@@ -0,0 +1,201 @@
+# React Class Component Validator - Test Results
+
+## Summary
+
+**Total Tests:** 15
+**Correctly Caught:** 7
+**Missed Violations (GAPS):** 3
+**Correctly Allowed:** 5
+
+---
+
+## CRITICAL GAPS FOUND
+
+### GAP 1: PureComponent Not Detected
+
+**Files:**
+- `03_pure_component.tsx` - `class X extends React.PureComponent`
+- `04_pure_component_import.tsx` - `class X extends PureComponent`
+
+**Issue:** The regex pattern only matches `Component` and `React.Component`, but misses `PureComponent` and `React.PureComponent`.
+
+**Current Pattern:**
+```python
+r'^\s*class\s+\w+\s+extends\s+(Component|React\.Component)\b'
+```
+
+**Should Be:**
+```python
+r'^\s*class\s+\w+\s+extends\s+(Component|React\.Component|PureComponent|React\.PureComponent)\b'
+```
+
+**Impact:** HIGH - PureComponent is a common class component pattern that will bypass the check.
+
+---
+
+### GAP 2: Error Boundary File-Level Exception Too Broad
+
+**File:**
+- `09_error_boundary_with_other_class.tsx` - Contains valid error boundary + regular class component
+
+**Issue:** The check skips the ENTIRE file if it finds `componentDidCatch` or `getDerivedStateFromError` anywhere. This means:
+- If you have an error boundary AND a regular class component in the same file, the regular component won't be flagged
+- The exception should be class-level, not file-level
+
+**Current Logic:**
+```python
+if ERROR_BOUNDARY_PATTERN.search(content):
+    continue  # Skips entire file
+```
+
+**Should Be:**
+- Check each class individually
+- Only skip classes that implement error boundary methods
+- Flag other classes in the same file
+
+**Impact:** MEDIUM - Uncommon to have multiple classes in same file, but it's a logic flaw.
+
+---
+
+### GAP 3: Inheritance Chain Detection
+
+**File:**
+- `10_inheritance_chain.tsx` - `class MyComponent extends BaseComponent` (where BaseComponent extends React.Component)
+
+**Issue:** Only the base class is caught, not the child class. The child class `MyComponent extends BaseComponent` is NOT flagged because it doesn't directly extend React.Component.
+
+**Current Behavior:** Catches base class only
+**Expected Behavior:** Should catch both (or at least warn about inheritance chain)
+
+**Impact:** LOW - This is a complex edge case. The base class is caught, which would force refactoring anyway.
+
+---
+
+## Test Results by Category
+
+### ✓ Correctly Caught (7 tests)
+
+| Test | File | Line | Status |
+|------|------|------|--------|
+| 01 | `01_basic_component.tsx` | 4 | CAUGHT ✓ |
+| 02 | `02_component_without_react.tsx` | 4 | CAUGHT ✓ |
+| 05 | `05_typescript_generics.tsx` | 8 | CAUGHT ✓ |
+| 06 | `06_typescript_two_generics.tsx` | 12 | CAUGHT ✓ |
+| 07 | `07_multiline_declaration.tsx` | 4 | CAUGHT ✓ |
+| 10 | `10_inheritance_chain.tsx` | 4 | CAUGHT ✓ (base class only) |
+| 14 | `14_indented_class.tsx` | 5 | CAUGHT ✓ |
+
+**Notes:**
+- TypeScript generics work correctly (both single and multiple type params)
+- Multi-line declarations work (even with newline between `class` and `extends`)
+- Indentation level doesn't matter
+- Inheritance base classes are caught
+
+---
+
+### ✗ Missed Violations - GAPS (3 tests)
+
+| Test | File | Expected | Actual | GAP |
+|------|------|----------|--------|-----|
+| 03 | `03_pure_component.tsx` | CAUGHT | NOT CAUGHT | GAP 1 |
+| 04 | `04_pure_component_import.tsx` | CAUGHT | NOT CAUGHT | GAP 1 |
+| 09 | `09_error_boundary_with_other_class.tsx` | CAUGHT (RegularComponent) | NOT CAUGHT | GAP 2 |
+
+---
+
+### ✓ Correctly Allowed (5 tests)
+
+| Test | File | Reason | Status |
+|------|------|--------|--------|
+| 08 | `08_error_boundary_valid.tsx` | Error boundary with componentDidCatch | ALLOWED ✓ |
+| 11 | `11_ts_file.ts` | .ts file (not .tsx/.jsx) | IGNORED ✓ |
+| 12 | `12_non_react_class.tsx` | Non-React class | ALLOWED ✓ |
+| 13 | `13_functional_component.tsx` | Functional component | ALLOWED ✓ |
+| 15 | `15_getDerivedStateFromError.tsx` | Error boundary with getDerivedStateFromError | ALLOWED ✓ |
+
+---
+
+## Recommended Fixes
+
+### Fix 1: Add PureComponent to Pattern (CRITICAL)
+
+```python
+CLASS_COMPONENT_PATTERN = re.compile(
+    r'^\s*class\s+\w+\s+extends\s+(Component|React\.Component|PureComponent|React\.PureComponent)\b',
+    re.MULTILINE
+)
+```
+
+### Fix 2: Class-Level Error Boundary Detection (MEDIUM)
+
+Instead of file-level skip, check each class:
+1. Find all class components in file
+2. For each class, check if IT implements error boundary methods
+3. Skip only those specific classes
+4. Flag other classes
+
+This requires more complex logic:
+- Extract class definition including its body
+- Search for error boundary methods within that specific class
+- Not trivial with regex alone
+
+### Fix 3: Document Inheritance Chain Limitation (LOW)
+
+Add to docstring:
+```python
+"""Check that no class components exist (except error boundaries).
+
+Note: Only detects direct inheritance from React.Component/PureComponent.
+Indirect inheritance chains (class A extends B where B extends Component)
+will only flag the base class.
+"""
+```
+
+---
+
+## Additional Edge Cases to Consider
+
+### Not Tested (Future Tests)
+
+1. **Default exports:**
+   ```tsx
+   export default class MyComponent extends React.Component {}
+   ```
+
+2. **Named class expressions:**
+   ```tsx
+   const MyComponent = class extends React.Component {};
+   ```
+
+3. **Anonymous class expressions:**
+   ```tsx
+   const MyComponent = class extends React.Component {};
+   ```
+
+4. **Class with decorators:**
+   ```tsx
+   @observer
+   class MyComponent extends React.Component {}
+   ```
+
+5. **Class inside namespace:**
+   ```tsx
+   namespace Components {
+     class MyComponent extends React.Component {}
+   }
+   ```
+
+---
+
+## Conclusion
+
+The validator is **functional but has critical gaps**:
+
+1. **MUST FIX:** PureComponent detection (GAP 1) - This is a common pattern
+2. **SHOULD FIX:** File-level error boundary exception (GAP 2) - Logic flaw
+3. **DOCUMENT:** Inheritance chain limitation (GAP 3) - Edge case
+
+**Priority:**
+1. Add PureComponent to regex pattern (5-minute fix)
+2. Document inheritance limitation in docstring
+3. Consider class-level error boundary detection for future iteration

package/hooks/validators/test_files/async_views.py

@@ -0,0 +1,23 @@
+"""Test file with async Django views to verify AsyncFunctionDef support."""
+
+from django.http import HttpRequest, HttpResponse
+
+
+async def async_home_view(request: HttpRequest) -> HttpResponse:
+    """Async view with correct naming - should pass."""
+    return HttpResponse("Home")
+
+
+async def async_about(request: HttpRequest) -> HttpResponse:
+    """Async view without _view suffix - should fail."""
+    return HttpResponse("About")
+
+
+async def async_helper(data: str) -> str:
+    """Async function not a view - should be ignored."""
+    return data.upper()
+
+
+async def async_profile_view(request: HttpRequest) -> HttpResponse:
+    """Another async view with correct naming - should pass."""
+    return HttpResponse("Profile")

package/hooks/validators/test_files/async_with_imports.py

@@ -0,0 +1,14 @@
+"""Test file with async functions and inline imports."""
+
+import asyncio
+
+
+async def async_func_with_inline_import():
+    """Async function with inline import - should fail."""
+    import json
+    return json.dumps({"status": "ok"})
+
+
+async def regular_async_func():
+    """Regular async function - should pass."""
+    await asyncio.sleep(0)