react-code-smell-detector 1.4.2 → 1.5.1

package/dist/detectors/formValidation.js

@@ -0,0 +1,193 @@
+import * as t from '@babel/types';
+/**
+ * Detect form-related anti-patterns and validation issues
+ */
+export function detectFormIssues(component, filePath, sourceCode, config) {
+    if (!config.checkForms)
+        return [];
+    const smells = [];
+    // Track form elements and their attributes
+    const formElements = [];
+    const inputElements = [];
+    const labeledInputIds = new Set();
+    const inputIds = new Set();
+    component.path.traverse({
+        JSXElement(path) {
+            const openingElement = path.node.openingElement;
+            if (!t.isJSXIdentifier(openingElement.name))
+                return;
+            const elementName = openingElement.name.name.toLowerCase();
+            const loc = openingElement.loc;
+            const attributes = openingElement.attributes;
+            // Check form elements
+            if (elementName === 'form') {
+                const hasOnSubmit = attributes.some(attr => t.isJSXAttribute(attr) &&
+                    t.isJSXIdentifier(attr.name) &&
+                    attr.name.name === 'onSubmit');
+                const hasAction = attributes.some(attr => t.isJSXAttribute(attr) &&
+                    t.isJSXIdentifier(attr.name) &&
+                    attr.name.name === 'action');
+                formElements.push({
+                    line: loc?.start.line || 0,
+                    hasOnSubmit,
+                    hasAction,
+                });
+                if (!hasOnSubmit && !hasAction) {
+                    smells.push({
+                        type: 'form-without-onsubmit',
+                        severity: 'warning',
+                        message: `Form without onSubmit handler in "${component.name}"`,
+                        file: filePath,
+                        line: loc?.start.line || 0,
+                        column: loc?.start.column || 0,
+                        suggestion: 'Add an onSubmit handler to handle form submission and prevent default behavior.',
+                    });
+                }
+            }
+            // Check input elements
+            if (elementName === 'input' || elementName === 'textarea' || elementName === 'select') {
+                const hasValue = attributes.some(attr => t.isJSXAttribute(attr) &&
+                    t.isJSXIdentifier(attr.name) &&
+                    (attr.name.name === 'value' || attr.name.name === 'checked'));
+                const hasDefaultValue = attributes.some(attr => t.isJSXAttribute(attr) &&
+                    t.isJSXIdentifier(attr.name) &&
+                    (attr.name.name === 'defaultValue' || attr.name.name === 'defaultChecked'));
+                const hasOnChange = attributes.some(attr => t.isJSXAttribute(attr) &&
+                    t.isJSXIdentifier(attr.name) &&
+                    attr.name.name === 'onChange');
+                const hasName = attributes.some(attr => t.isJSXAttribute(attr) &&
+                    t.isJSXIdentifier(attr.name) &&
+                    attr.name.name === 'name');
+                // Get input type
+                let inputType = 'text';
+                const typeAttr = attributes.find(attr => t.isJSXAttribute(attr) &&
+                    t.isJSXIdentifier(attr.name) &&
+                    attr.name.name === 'type');
+                if (typeAttr && t.isJSXAttribute(typeAttr) && t.isStringLiteral(typeAttr.value)) {
+                    inputType = typeAttr.value.value;
+                }
+                // Get input id for label checking
+                const idAttr = attributes.find(attr => t.isJSXAttribute(attr) &&
+                    t.isJSXIdentifier(attr.name) &&
+                    attr.name.name === 'id');
+                if (idAttr && t.isJSXAttribute(idAttr) && t.isStringLiteral(idAttr.value)) {
+                    inputIds.add(idAttr.value.value);
+                }
+                inputElements.push({
+                    line: loc?.start.line || 0,
+                    hasValue,
+                    hasOnChange,
+                    hasName,
+                    type: inputType,
+                });
+                // Check for uncontrolled inputs (no value or defaultValue)
+                if (!hasValue && !hasDefaultValue && !hasOnChange) {
+                    // Skip hidden, submit, button, reset types
+                    const skipTypes = ['hidden', 'submit', 'button', 'reset', 'image'];
+                    if (!skipTypes.includes(inputType)) {
+                        smells.push({
+                            type: 'uncontrolled-form',
+                            severity: 'info',
+                            message: `Uncontrolled ${elementName} element in "${component.name}" - consider using controlled components`,
+                            file: filePath,
+                            line: loc?.start.line || 0,
+                            column: loc?.start.column || 0,
+                            suggestion: 'Use value and onChange for controlled components, or ref for uncontrolled access.',
+                        });
+                    }
+                }
+                // Check for controlled input without onChange (will be read-only)
+                if (hasValue && !hasOnChange && !hasDefaultValue) {
+                    smells.push({
+                        type: 'uncontrolled-form',
+                        severity: 'error',
+                        message: `Input has value prop but no onChange handler in "${component.name}" - input will be read-only`,
+                        file: filePath,
+                        line: loc?.start.line || 0,
+                        column: loc?.start.column || 0,
+                        suggestion: 'Add an onChange handler or use defaultValue instead of value.',
+                    });
+                }
+            }
+            // Check for label elements and collect their "for/htmlFor" attributes
+            if (elementName === 'label') {
+                const forAttr = attributes.find(attr => t.isJSXAttribute(attr) &&
+                    t.isJSXIdentifier(attr.name) &&
+                    (attr.name.name === 'htmlFor' || attr.name.name === 'for'));
+                if (forAttr && t.isJSXAttribute(forAttr) && t.isStringLiteral(forAttr.value)) {
+                    labeledInputIds.add(forAttr.value.value);
+                }
+            }
+        },
+    });
+    // Check for inputs without associated labels
+    inputIds.forEach(id => {
+        if (!labeledInputIds.has(id)) {
+            // This is already handled by accessibility detector, but we note it for forms
+        }
+    });
+    // Check for missing form validation patterns
+    const hasValidationLibrary = hasFormValidation(sourceCode);
+    if (formElements.length > 0 && !hasValidationLibrary) {
+        smells.push({
+            type: 'missing-form-validation',
+            severity: 'info',
+            message: `Form in "${component.name}" without apparent validation library`,
+            file: filePath,
+            line: formElements[0].line,
+            column: 0,
+            suggestion: 'Consider using a form validation library like react-hook-form, formik, or yup for robust validation.',
+        });
+    }
+    return smells;
+}
+/**
+ * Check if code uses common form validation patterns/libraries
+ */
+function hasFormValidation(sourceCode) {
+    const validationPatterns = [
+        'useForm', // react-hook-form
+        'useFormik', // formik
+        'Formik', // formik component
+        'yup.', // yup validation
+        'zod.', // zod validation
+        '.validate(', // custom validation
+        'validator.', // validator.js
+        'Joi.', // joi validation
+        'required:', // validation rules
+        'pattern:', // regex validation
+        'minLength:', // length validation
+        'schema.validate', // schema validation
+    ];
+    return validationPatterns.some(pattern => sourceCode.includes(pattern));
+}
+/**
+ * Detect form-related patterns at file level
+ */
+export function detectFormPatterns(sourceCode, filePath, config) {
+    if (!config.checkForms)
+        return [];
+    const smells = [];
+    const lines = sourceCode.split('\n');
+    // Check for form submission without preventDefault
+    lines.forEach((line, index) => {
+        // Pattern: onSubmit={(e) => { ... }} without e.preventDefault()
+        if (line.includes('onSubmit') && line.includes('=>')) {
+            // Look ahead for preventDefault in nearby lines
+            const nextLines = lines.slice(index, index + 5).join('\n');
+            if (!nextLines.includes('preventDefault')) {
+                smells.push({
+                    type: 'form-without-onsubmit',
+                    severity: 'warning',
+                    message: 'Form onSubmit handler may be missing e.preventDefault()',
+                    file: filePath,
+                    line: index + 1,
+                    column: 0,
+                    suggestion: 'Call e.preventDefault() in form submit handlers to prevent default page reload.',
+                    codeSnippet: line.trim(),
+                });
+            }
+        }
+    });
+    return smells;
+}

package/dist/detectors/index.d.ts

@@ -20,4 +20,10 @@ export { detectComplexity } from './complexity.js';
 export { detectMemoryLeaks } from './memoryLeak.js';
 export { detectImportIssues, analyzeImports } from './imports.js';
 export { detectUnusedCode } from './unusedCode.js';
+export { detectServerComponentIssues, detectAsyncComponentIssues } from './serverComponents.js';
+export { detectContextIssues, detectContextProviderIssues } from './contextApi.js';
+export { detectErrorBoundaryIssues, detectMissingErrorBoundaries } from './errorBoundary.js';
+export { detectFormIssues, detectFormPatterns } from './formValidation.js';
+export { detectStateManagementIssues, detectStateManagementPatterns } from './stateManagement.js';
+export { detectTestingGaps, generateTestSuggestions } from './testingGaps.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/detectors/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/detectors/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,sBAAsB,EAAE,MAAM,gBAAgB,CAAC;AACxD,OAAO,EAAE,kBAAkB,EAAE,wBAAwB,EAAE,MAAM,mBAAmB,CAAC;AACjF,OAAO,EAAE,oBAAoB,EAAE,MAAM,qBAAqB,CAAC;AAC3D,OAAO,EAAE,4BAA4B,EAAE,MAAM,kBAAkB,CAAC;AAChE,OAAO,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAC;AACpD,OAAO,EAAE,0BAA0B,EAAE,MAAM,iBAAiB,CAAC;AAC7D,OAAO,EAAE,2BAA2B,EAAE,MAAM,sBAAsB,CAAC;AACnE,OAAO,EAAE,qBAAqB,EAAE,MAAM,oBAAoB,CAAC;AAC3D,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAC/C,OAAO,EAAE,iBAAiB,EAAE,MAAM,kBAAkB,CAAC;AAErD,OAAO,EAAE,kBAAkB,EAAE,MAAM,aAAa,CAAC;AACjD,OAAO,EAAE,uBAAuB,EAAE,MAAM,kBAAkB,CAAC;AAC3D,OAAO,EAAE,kBAAkB,EAAE,MAAM,aAAa,CAAC;AACjD,OAAO,EAAE,sBAAsB,EAAE,MAAM,iBAAiB,CAAC;AACzD,OAAO,EAAE,sBAAsB,EAAE,MAAM,iBAAiB,CAAC;AAEzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,YAAY,CAAC;AACnD,OAAO,EAAE,oBAAoB,EAAE,MAAM,eAAe,CAAC;AACrD,OAAO,EAAE,yBAAyB,EAAE,MAAM,oBAAoB,CAAC;AAE/D,OAAO,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AACnD,OAAO,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAC;AACpD,OAAO,EAAE,kBAAkB,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAClE,OAAO,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/detectors/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,sBAAsB,EAAE,MAAM,gBAAgB,CAAC;AACxD,OAAO,EAAE,kBAAkB,EAAE,wBAAwB,EAAE,MAAM,mBAAmB,CAAC;AACjF,OAAO,EAAE,oBAAoB,EAAE,MAAM,qBAAqB,CAAC;AAC3D,OAAO,EAAE,4BAA4B,EAAE,MAAM,kBAAkB,CAAC;AAChE,OAAO,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAC;AACpD,OAAO,EAAE,0BAA0B,EAAE,MAAM,iBAAiB,CAAC;AAC7D,OAAO,EAAE,2BAA2B,EAAE,MAAM,sBAAsB,CAAC;AACnE,OAAO,EAAE,qBAAqB,EAAE,MAAM,oBAAoB,CAAC;AAC3D,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAC/C,OAAO,EAAE,iBAAiB,EAAE,MAAM,kBAAkB,CAAC;AAErD,OAAO,EAAE,kBAAkB,EAAE,MAAM,aAAa,CAAC;AACjD,OAAO,EAAE,uBAAuB,EAAE,MAAM,kBAAkB,CAAC;AAC3D,OAAO,EAAE,kBAAkB,EAAE,MAAM,aAAa,CAAC;AACjD,OAAO,EAAE,sBAAsB,EAAE,MAAM,iBAAiB,CAAC;AACzD,OAAO,EAAE,sBAAsB,EAAE,MAAM,iBAAiB,CAAC;AAEzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,YAAY,CAAC;AACnD,OAAO,EAAE,oBAAoB,EAAE,MAAM,eAAe,CAAC;AACrD,OAAO,EAAE,yBAAyB,EAAE,MAAM,oBAAoB,CAAC;AAE/D,OAAO,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AACnD,OAAO,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAC;AACpD,OAAO,EAAE,kBAAkB,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAClE,OAAO,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AAEnD,OAAO,EAAE,2BAA2B,EAAE,0BAA0B,EAAE,MAAM,uBAAuB,CAAC;AAEhG,OAAO,EAAE,mBAAmB,EAAE,2BAA2B,EAAE,MAAM,iBAAiB,CAAC;AAEnF,OAAO,EAAE,yBAAyB,EAAE,4BAA4B,EAAE,MAAM,oBAAoB,CAAC;AAE7F,OAAO,EAAE,gBAAgB,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AAE3E,OAAO,EAAE,2BAA2B,EAAE,6BAA6B,EAAE,MAAM,sBAAsB,CAAC;AAElG,OAAO,EAAE,iBAAiB,EAAE,uBAAuB,EAAE,MAAM,kBAAkB,CAAC"}

package/dist/detectors/index.js

@@ -23,3 +23,15 @@ export { detectComplexity } from './complexity.js';
 export { detectMemoryLeaks } from './memoryLeak.js';
 export { detectImportIssues, analyzeImports } from './imports.js';
 export { detectUnusedCode } from './unusedCode.js';
+// Server Components (React 19)
+export { detectServerComponentIssues, detectAsyncComponentIssues } from './serverComponents.js';
+// Context API Analysis
+export { detectContextIssues, detectContextProviderIssues } from './contextApi.js';
+// Error Boundary Detection
+export { detectErrorBoundaryIssues, detectMissingErrorBoundaries } from './errorBoundary.js';
+// Form Validation
+export { detectFormIssues, detectFormPatterns } from './formValidation.js';
+// State Management
+export { detectStateManagementIssues, detectStateManagementPatterns } from './stateManagement.js';
+// Testing Gaps
+export { detectTestingGaps, generateTestSuggestions } from './testingGaps.js';

package/dist/detectors/serverComponents.d.ts

@@ -0,0 +1,11 @@
+import { ParsedComponent } from '../parser/index.js';
+import { CodeSmell, DetectorConfig } from '../types/index.js';
+/**
+ * Detect React 19 Server/Client component boundary issues
+ */
+export declare function detectServerComponentIssues(component: ParsedComponent, filePath: string, sourceCode: string, config: DetectorConfig, imports?: string[]): CodeSmell[];
+/**
+ * Detect proper async component patterns in React 19
+ */
+export declare function detectAsyncComponentIssues(component: ParsedComponent, filePath: string, sourceCode: string, config: DetectorConfig): CodeSmell[];
+//# sourceMappingURL=serverComponents.d.ts.map

package/dist/detectors/serverComponents.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"serverComponents.d.ts","sourceRoot":"","sources":["../../src/detectors/serverComponents.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EAAE,SAAS,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC;AAqC9D;;GAEG;AACH,wBAAgB,2BAA2B,CACzC,SAAS,EAAE,eAAe,EAC1B,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,cAAc,EACtB,OAAO,GAAE,MAAM,EAAO,GACrB,SAAS,EAAE,CA+Jb;AAED;;GAEG;AACH,wBAAgB,0BAA0B,CACxC,SAAS,EAAE,eAAe,EAC1B,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,cAAc,GACrB,SAAS,EAAE,CAmCb"}

package/dist/detectors/serverComponents.js

@@ -0,0 +1,222 @@
+import * as t from '@babel/types';
+// Hooks that are not allowed in Server Components
+const CLIENT_ONLY_HOOKS = [
+    'useState',
+    'useReducer',
+    'useEffect',
+    'useLayoutEffect',
+    'useInsertionEffect',
+    'useRef',
+    'useImperativeHandle',
+    'useSyncExternalStore',
+    'useTransition',
+    'useDeferredValue',
+    'useOptimistic',
+    'useFormStatus',
+    'useActionState',
+];
+// Browser-only APIs
+const BROWSER_APIS = [
+    'window',
+    'document',
+    'localStorage',
+    'sessionStorage',
+    'navigator',
+    'location',
+    'history',
+    'alert',
+    'confirm',
+    'prompt',
+    'fetch', // Can be used in Server Components but behaves differently
+];
+// Event handler props (indicate client interactivity)
+const EVENT_HANDLER_PATTERN = /^on[A-Z]/;
+/**
+ * Detect React 19 Server/Client component boundary issues
+ */
+export function detectServerComponentIssues(component, filePath, sourceCode, config, imports = []) {
+    if (!config.checkServerComponents)
+        return [];
+    const smells = [];
+    const lines = sourceCode.split('\n');
+    // Check if file has 'use client' or 'use server' directive
+    const hasUseClient = lines.some(line => /^['"]use client['"];?\s*$/.test(line.trim()));
+    const hasUseServer = lines.some(line => /^['"]use server['"];?\s*$/.test(line.trim()));
+    // Detect if this is a Server Component (no 'use client' directive in app/ directory)
+    const isInAppDir = filePath.includes('/app/') || filePath.includes('\\app\\');
+    const isServerComponent = isInAppDir && !hasUseClient;
+    if (isServerComponent) {
+        // Check for client-only hooks in Server Components
+        const clientHooksUsed = [];
+        for (const hookName of CLIENT_ONLY_HOOKS) {
+            const hooks = component.hooks[hookName];
+            if (hooks && hooks.length > 0) {
+                clientHooksUsed.push(hookName);
+            }
+        }
+        // Also traverse for any hook calls
+        component.path.traverse({
+            CallExpression(path) {
+                const callee = path.node.callee;
+                if (t.isIdentifier(callee) && CLIENT_ONLY_HOOKS.includes(callee.name)) {
+                    if (!clientHooksUsed.includes(callee.name)) {
+                        clientHooksUsed.push(callee.name);
+                    }
+                }
+            },
+        });
+        if (clientHooksUsed.length > 0) {
+            smells.push({
+                type: 'server-component-hooks',
+                severity: 'error',
+                message: `Server Component "${component.name}" uses client-only hooks: ${clientHooksUsed.join(', ')}`,
+                file: filePath,
+                line: component.startLine,
+                column: 0,
+                suggestion: `Add 'use client' directive at the top of the file, or move stateful logic to a Client Component.`,
+            });
+        }
+        // Check for event handlers in JSX (onClick, onChange, etc.)
+        const eventHandlers = [];
+        component.path.traverse({
+            JSXAttribute(path) {
+                if (t.isJSXIdentifier(path.node.name)) {
+                    const name = path.node.name.name;
+                    if (EVENT_HANDLER_PATTERN.test(name)) {
+                        if (!eventHandlers.includes(name)) {
+                            eventHandlers.push(name);
+                        }
+                    }
+                }
+            },
+        });
+        if (eventHandlers.length > 0) {
+            smells.push({
+                type: 'server-component-events',
+                severity: 'error',
+                message: `Server Component "${component.name}" uses event handlers: ${eventHandlers.join(', ')}`,
+                file: filePath,
+                line: component.startLine,
+                column: 0,
+                suggestion: `Add 'use client' directive or extract interactive elements to a Client Component.`,
+            });
+        }
+        // Check for browser APIs usage
+        const browserApisUsed = [];
+        component.path.traverse({
+            Identifier(path) {
+                const name = path.node.name;
+                if (BROWSER_APIS.includes(name)) {
+                    // Make sure it's not a property access like obj.window
+                    if (!t.isMemberExpression(path.parent) || !t.isIdentifier(path.parent.property) || path.parent.object === path.node) {
+                        if (!browserApisUsed.includes(name)) {
+                            browserApisUsed.push(name);
+                        }
+                    }
+                }
+            },
+        });
+        if (browserApisUsed.length > 0) {
+            smells.push({
+                type: 'server-component-browser-api',
+                severity: 'warning',
+                message: `Server Component "${component.name}" may use browser APIs: ${browserApisUsed.join(', ')}`,
+                file: filePath,
+                line: component.startLine,
+                column: 0,
+                suggestion: `Browser APIs are not available in Server Components. Move to a Client Component or use conditional checks.`,
+            });
+        }
+    }
+    // Check for 'use server' in client components (async actions)
+    if (hasUseClient && hasUseServer) {
+        smells.push({
+            type: 'mixed-directives',
+            severity: 'error',
+            message: `File has both 'use client' and 'use server' directives`,
+            file: filePath,
+            line: 1,
+            column: 0,
+            suggestion: `A file can only be either a Client Component or contain Server Actions, not both. Separate them into different files.`,
+        });
+    }
+    // Check for async component without 'use server' actions (they should be in Server Components)
+    if (!isServerComponent && !hasUseServer) {
+        let hasAsyncServerAction = false;
+        component.path.traverse({
+            FunctionDeclaration(path) {
+                if (path.node.async) {
+                    // Check if body starts with 'use server'
+                    const body = path.node.body;
+                    if (body.directives?.some(d => d.value.value === 'use server')) {
+                        hasAsyncServerAction = true;
+                    }
+                }
+            },
+            ArrowFunctionExpression(path) {
+                if (path.node.async && t.isBlockStatement(path.node.body)) {
+                    const directives = path.node.body.directives;
+                    if (directives?.some(d => d.value.value === 'use server')) {
+                        hasAsyncServerAction = true;
+                    }
+                }
+            },
+        });
+        // This is fine - Server Actions can be defined inline with 'use server' inside
+    }
+    // Check for passing server actions incorrectly to client components
+    // (This is complex to detect statically, so we provide guidance)
+    // Check for importing Server Component into Client Component (potential issue)
+    if (hasUseClient) {
+        // Look for imports that might be Server Components
+        const serverComponentImports = imports.filter(imp => {
+            // Heuristic: imports from app/ directory without 'use client'
+            return imp.includes('/app/') && !imp.includes('.client');
+        });
+        // Can't definitively detect, but can warn about patterns
+    }
+    return smells;
+}
+/**
+ * Detect proper async component patterns in React 19
+ */
+export function detectAsyncComponentIssues(component, filePath, sourceCode, config) {
+    if (!config.checkServerComponents)
+        return [];
+    const smells = [];
+    const lines = sourceCode.split('\n');
+    const hasUseClient = lines.some(line => /^['"]use client['"];?\s*$/.test(line.trim()));
+    // Check if component is async
+    let isAsync = false;
+    const node = component.path.node;
+    if (t.isFunctionDeclaration(node) || t.isFunctionExpression(node) || t.isArrowFunctionExpression(node)) {
+        isAsync = node.async || false;
+    }
+    // Async components in Client Components are not allowed
+    if (isAsync && hasUseClient) {
+        smells.push({
+            type: 'async-client-component',
+            severity: 'error',
+            message: `Client Component "${component.name}" is async, which is not supported`,
+            file: filePath,
+            line: component.startLine,
+            column: 0,
+            suggestion: `Remove async from the component. Use useEffect or React Query for data fetching in Client Components.`,
+        });
+    }
+    // Check for proper Suspense usage with async Server Components
+    if (isAsync && !hasUseClient) {
+        // Good pattern - async Server Component
+        // Could check if parent has Suspense boundary, but that requires cross-file analysis
+    }
+    return smells;
+}
+/**
+ * Get code snippet for context
+ */
+function getCodeSnippet(sourceCode, line) {
+    const lines = sourceCode.split('\n');
+    const startLine = Math.max(0, line - 2);
+    const endLine = Math.min(lines.length, line + 1);
+    return lines.slice(startLine, endLine).join('\n');
+}

package/dist/detectors/stateManagement.d.ts

@@ -0,0 +1,11 @@
+import { ParsedComponent } from '../parser/index.js';
+import { CodeSmell, DetectorConfig } from '../types/index.js';
+/**
+ * Detect state management anti-patterns (Redux, Zustand, MobX, etc.)
+ */
+export declare function detectStateManagementIssues(component: ParsedComponent, filePath: string, sourceCode: string, config: DetectorConfig): CodeSmell[];
+/**
+ * Detect file-level state management patterns
+ */
+export declare function detectStateManagementPatterns(sourceCode: string, filePath: string, config: DetectorConfig): CodeSmell[];
+//# sourceMappingURL=stateManagement.d.ts.map

package/dist/detectors/stateManagement.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"stateManagement.d.ts","sourceRoot":"","sources":["../../src/detectors/stateManagement.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EAAE,SAAS,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC;AAE9D;;GAEG;AACH,wBAAgB,2BAA2B,CACzC,SAAS,EAAE,eAAe,EAC1B,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,cAAc,GACrB,SAAS,EAAE,CA6Hb;AAiDD;;GAEG;AACH,wBAAgB,6BAA6B,CAC3C,UAAU,EAAE,MAAM,EAClB,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,cAAc,GACrB,SAAS,EAAE,CAyCb"}