react-code-smell-detector 1.5.0 → 1.5.2

package/dist/detectors/contextApi.js

@@ -0,0 +1,151 @@
+import * as t from '@babel/types';
+/**
+ * Detect Context API anti-patterns and performance issues
+ */
+export function detectContextIssues(component, filePath, sourceCode, config) {
+    if (!config.checkContextApi)
+        return [];
+    const smells = [];
+    let useContextCount = 0;
+    const contextLocations = [];
+    // Check for useContext calls
+    component.path.traverse({
+        CallExpression(path) {
+            const node = path.node;
+            const { callee } = node;
+            // Detect useContext calls
+            if (t.isIdentifier(callee) && callee.name === 'useContext') {
+                useContextCount++;
+                const loc = node.loc;
+                if (loc) {
+                    contextLocations.push({ line: loc.start.line, column: loc.start.column });
+                }
+                // Check if context value is destructured with many properties
+                const parent = path.parentPath;
+                if (parent && t.isVariableDeclarator(parent.node)) {
+                    const id = parent.node.id;
+                    if (t.isObjectPattern(id) && id.properties.length > 5) {
+                        smells.push({
+                            type: 'large-context-value',
+                            severity: 'warning',
+                            message: `Context destructures ${id.properties.length} properties in "${component.name}" - consider splitting context`,
+                            file: filePath,
+                            line: loc?.start.line || 0,
+                            column: loc?.start.column || 0,
+                            suggestion: 'Split large contexts into smaller, focused contexts to prevent unnecessary re-renders.',
+                        });
+                    }
+                }
+            }
+            // Detect useContext inside loops or conditionals (anti-pattern)
+            if (t.isIdentifier(callee) && callee.name === 'useContext') {
+                let currentPath = path.parentPath;
+                while (currentPath) {
+                    if (t.isForStatement(currentPath.node) ||
+                        t.isWhileStatement(currentPath.node) ||
+                        t.isForInStatement(currentPath.node) ||
+                        t.isForOfStatement(currentPath.node)) {
+                        const loc = node.loc;
+                        smells.push({
+                            type: 'context-in-loop',
+                            severity: 'error',
+                            message: `useContext called inside a loop in "${component.name}"`,
+                            file: filePath,
+                            line: loc?.start.line || 0,
+                            column: loc?.start.column || 0,
+                            suggestion: 'Move useContext outside of loops. Hooks must be called at the top level.',
+                        });
+                        break;
+                    }
+                    currentPath = currentPath.parentPath;
+                }
+            }
+        },
+    });
+    // Check for too many useContext calls (context overuse)
+    if (useContextCount > config.maxContextConsumers) {
+        const firstLoc = contextLocations[0] || { line: component.path.node.loc?.start.line || 0, column: 0 };
+        smells.push({
+            type: 'context-overuse',
+            severity: 'warning',
+            message: `Component "${component.name}" uses ${useContextCount} contexts (max: ${config.maxContextConsumers})`,
+            file: filePath,
+            line: firstLoc.line,
+            column: firstLoc.column,
+            suggestion: 'Consider using composition or combining related contexts. Too many contexts can make components hard to test and maintain.',
+        });
+    }
+    // Check if component using context is not memoized
+    if (useContextCount > 0) {
+        const isMemoized = checkIfMemoized(component, sourceCode);
+        if (!isMemoized && useContextCount > 1) {
+            smells.push({
+                type: 'missing-context-memo',
+                severity: 'info',
+                message: `Component "${component.name}" uses multiple contexts but is not memoized`,
+                file: filePath,
+                line: component.path.node.loc?.start.line || 0,
+                column: 0,
+                suggestion: 'Consider wrapping with React.memo() to prevent unnecessary re-renders when unrelated context values change.',
+            });
+        }
+    }
+    return smells;
+}
+/**
+ * Check if a component is wrapped with React.memo
+ */
+function checkIfMemoized(component, sourceCode) {
+    const lines = sourceCode.split('\n');
+    const componentName = component.name;
+    // Check for React.memo or memo wrapper patterns
+    for (const line of lines) {
+        if (line.includes(`memo(${componentName})`) ||
+            line.includes(`React.memo(${componentName})`) ||
+            line.includes(`memo(function ${componentName}`)) {
+            return true;
+        }
+    }
+    return false;
+}
+/**
+ * Detect Context Provider issues
+ */
+export function detectContextProviderIssues(sourceCode, filePath, config) {
+    if (!config.checkContextApi)
+        return [];
+    const smells = [];
+    const lines = sourceCode.split('\n');
+    // Check for context provider with object literal value (causes re-renders)
+    lines.forEach((line, index) => {
+        // Pattern: <SomeContext.Provider value={{ ... }}>
+        const providerMatch = line.match(/\.Provider\s+value=\{\{/);
+        if (providerMatch) {
+            smells.push({
+                type: 'large-context-value',
+                severity: 'warning',
+                message: 'Context Provider has inline object value - causes re-renders on every render',
+                file: filePath,
+                line: index + 1,
+                column: providerMatch.index || 0,
+                suggestion: 'Memoize the context value with useMemo() or extract to a stable reference.',
+                codeSnippet: line.trim(),
+            });
+        }
+        // Pattern: <SomeContext.Provider value={[ ... ]}>
+        const arrayMatch = line.match(/\.Provider\s+value=\{\[/);
+        if (arrayMatch) {
+            smells.push({
+                type: 'large-context-value',
+                severity: 'warning',
+                message: 'Context Provider has inline array value - causes re-renders on every render',
+                file: filePath,
+                line: index + 1,
+                column: arrayMatch.index || 0,
+                suggestion: 'Memoize the context value with useMemo() or extract to a stable reference.',
+                codeSnippet: line.trim(),
+            });
+        }
+    });
+    return smells;
+}

package/dist/detectors/errorBoundary.d.ts

@@ -0,0 +1,11 @@
+import { ParsedComponent } from '../parser/index.js';
+import { CodeSmell, DetectorConfig } from '../types/index.js';
+/**
+ * Detect missing Error Boundaries and Suspense issues
+ */
+export declare function detectErrorBoundaryIssues(component: ParsedComponent, filePath: string, sourceCode: string, config: DetectorConfig): CodeSmell[];
+/**
+ * Detect file-level error boundary patterns
+ */
+export declare function detectMissingErrorBoundaries(sourceCode: string, filePath: string, config: DetectorConfig): CodeSmell[];
+//# sourceMappingURL=errorBoundary.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"errorBoundary.d.ts","sourceRoot":"","sources":["../../src/detectors/errorBoundary.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,yBAAyB,CACvC,SAAS,EAAE,eAAe,EAC1B,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,cAAc,GACrB,SAAS,EAAE,CAoIb;AAED;;GAEG;AACH,wBAAgB,4BAA4B,CAC1C,UAAU,EAAE,MAAM,EAClB,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,cAAc,GACrB,SAAS,EAAE,CAyDb"}

package/dist/detectors/errorBoundary.js

@@ -0,0 +1,167 @@
+import * as t from '@babel/types';
+/**
+ * Detect missing Error Boundaries and Suspense issues
+ */
+export function detectErrorBoundaryIssues(component, filePath, sourceCode, config) {
+    if (!config.checkErrorBoundaries)
+        return [];
+    const smells = [];
+    // Track JSX elements and their wrappers
+    let hasSuspense = false;
+    let hasErrorBoundary = false;
+    let hasAsyncComponent = false;
+    let hasLazyLoad = false;
+    const suspenseLocations = [];
+    component.path.traverse({
+        JSXElement(path) {
+            const openingElement = path.node.openingElement;
+            if (t.isJSXIdentifier(openingElement.name)) {
+                const name = openingElement.name.name;
+                // Check for Suspense
+                if (name === 'Suspense') {
+                    hasSuspense = true;
+                    const loc = openingElement.loc;
+                    const hasFallback = openingElement.attributes.some(attr => t.isJSXAttribute(attr) && t.isJSXIdentifier(attr.name) && attr.name.name === 'fallback');
+                    suspenseLocations.push({
+                        line: loc?.start.line || 0,
+                        hasFallback,
+                    });
+                    if (!hasFallback) {
+                        smells.push({
+                            type: 'suspense-missing-fallback',
+                            severity: 'warning',
+                            message: `Suspense without fallback prop in "${component.name}"`,
+                            file: filePath,
+                            line: loc?.start.line || 0,
+                            column: loc?.start.column || 0,
+                            suggestion: 'Add a fallback prop to Suspense: <Suspense fallback={<Loading />}>',
+                        });
+                    }
+                }
+                // Check for ErrorBoundary (custom or library)
+                if (name === 'ErrorBoundary' ||
+                    name.includes('ErrorBoundary') ||
+                    name === 'ErrorBoundaryWrapper') {
+                    hasErrorBoundary = true;
+                    const hasFallback = openingElement.attributes.some(attr => t.isJSXAttribute(attr) &&
+                        t.isJSXIdentifier(attr.name) &&
+                        (attr.name.name === 'fallback' || attr.name.name === 'FallbackComponent' || attr.name.name === 'fallbackRender'));
+                    if (!hasFallback) {
+                        const loc = openingElement.loc;
+                        smells.push({
+                            type: 'error-boundary-missing-fallback',
+                            severity: 'warning',
+                            message: `ErrorBoundary without fallback in "${component.name}"`,
+                            file: filePath,
+                            line: loc?.start.line || 0,
+                            column: loc?.start.column || 0,
+                            suggestion: 'Add a fallback prop or FallbackComponent to ErrorBoundary.',
+                        });
+                    }
+                }
+            }
+        },
+        CallExpression(path) {
+            const node = path.node;
+            const { callee } = node;
+            // Check for React.lazy or lazy imports
+            if ((t.isIdentifier(callee) && callee.name === 'lazy') ||
+                (t.isMemberExpression(callee) &&
+                    t.isIdentifier(callee.object) &&
+                    callee.object.name === 'React' &&
+                    t.isIdentifier(callee.property) &&
+                    callee.property.name === 'lazy')) {
+                hasLazyLoad = true;
+            }
+            // Check for async data fetching patterns
+            if (t.isIdentifier(callee)) {
+                const asyncPatterns = ['useQuery', 'useSWR', 'useAsync', 'useFetch', 'useData'];
+                if (asyncPatterns.includes(callee.name)) {
+                    hasAsyncComponent = true;
+                }
+            }
+        },
+        // Detect async components (often need Suspense)
+        AwaitExpression() {
+            hasAsyncComponent = true;
+        },
+    });
+    // Check for lazy-loaded components without Suspense
+    if (hasLazyLoad && !hasSuspense) {
+        smells.push({
+            type: 'missing-error-boundary',
+            severity: 'error',
+            message: `Component "${component.name}" uses lazy loading but has no Suspense wrapper`,
+            file: filePath,
+            line: component.path.node.loc?.start.line || 0,
+            column: 0,
+            suggestion: 'Wrap lazy-loaded components with <Suspense fallback={<Loading />}>',
+        });
+    }
+    // Suggest ErrorBoundary for complex components
+    if (!hasErrorBoundary && hasAsyncComponent) {
+        smells.push({
+            type: 'missing-error-boundary',
+            severity: 'info',
+            message: `Component "${component.name}" has async operations but no ErrorBoundary`,
+            file: filePath,
+            line: component.path.node.loc?.start.line || 0,
+            column: 0,
+            suggestion: 'Consider wrapping async components with an ErrorBoundary to handle failures gracefully.',
+        });
+    }
+    return smells;
+}
+/**
+ * Detect file-level error boundary patterns
+ */
+export function detectMissingErrorBoundaries(sourceCode, filePath, config) {
+    if (!config.checkErrorBoundaries)
+        return [];
+    const smells = [];
+    const lines = sourceCode.split('\n');
+    // Check for lazy imports without corresponding Suspense
+    let hasLazyImport = false;
+    let hasSuspenseImport = false;
+    let hasErrorBoundaryImport = false;
+    for (const line of lines) {
+        if (line.includes('lazy(') || line.includes('React.lazy(')) {
+            hasLazyImport = true;
+        }
+        if (line.includes('Suspense') && (line.includes('import') || line.includes('React.Suspense'))) {
+            hasSuspenseImport = true;
+        }
+        if (line.includes('ErrorBoundary') && line.includes('import')) {
+            hasErrorBoundaryImport = true;
+        }
+    }
+    // If file has lazy imports but no Suspense import, flag it
+    if (hasLazyImport && !hasSuspenseImport) {
+        smells.push({
+            type: 'missing-error-boundary',
+            severity: 'warning',
+            message: 'File uses React.lazy but does not import Suspense',
+            file: filePath,
+            line: 1,
+            column: 0,
+            suggestion: 'Import Suspense from React and wrap lazy components.',
+        });
+    }
+    // Check for fetch/axios without error handling patterns
+    const hasDataFetching = lines.some(line => line.includes('fetch(') ||
+        line.includes('axios.') ||
+        line.includes('useQuery') ||
+        line.includes('useSWR'));
+    if (hasDataFetching && !hasErrorBoundaryImport) {
+        smells.push({
+            type: 'missing-error-boundary',
+            severity: 'info',
+            message: 'File has data fetching but no ErrorBoundary import',
+            file: filePath,
+            line: 1,
+            column: 0,
+            suggestion: 'Consider using an ErrorBoundary to handle data fetching failures gracefully.',
+        });
+    }
+    return smells;
+}

package/dist/detectors/formValidation.d.ts

@@ -0,0 +1,11 @@
+import { ParsedComponent } from '../parser/index.js';
+import { CodeSmell, DetectorConfig } from '../types/index.js';
+/**
+ * Detect form-related anti-patterns and validation issues
+ */
+export declare function detectFormIssues(component: ParsedComponent, filePath: string, sourceCode: string, config: DetectorConfig): CodeSmell[];
+/**
+ * Detect form-related patterns at file level
+ */
+export declare function detectFormPatterns(sourceCode: string, filePath: string, config: DetectorConfig): CodeSmell[];
+//# sourceMappingURL=formValidation.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"formValidation.d.ts","sourceRoot":"","sources":["../../src/detectors/formValidation.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,gBAAgB,CAC9B,SAAS,EAAE,eAAe,EAC1B,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,cAAc,GACrB,SAAS,EAAE,CA4Kb;AAwBD;;GAEG;AACH,wBAAgB,kBAAkB,CAChC,UAAU,EAAE,MAAM,EAClB,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,cAAc,GACrB,SAAS,EAAE,CA4Bb"}

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

@@ -21,4 +21,9 @@ 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;AAEnD,OAAO,EAAE,2BAA2B,EAAE,0BAA0B,EAAE,MAAM,uBAAuB,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

@@ -25,3 +25,13 @@ 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/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"}