react-code-smell-detector 1.2.0 → 1.3.1

package/dist/detectors/memoryLeak.js

@@ -0,0 +1,111 @@
+import * as t from '@babel/types';
+/**
+ * Detect potential memory leaks in React components
+ */
+export function detectMemoryLeaks(component, filePath, sourceCode, config) {
+    if (!config.checkMemoryLeaks)
+        return [];
+    const smells = [];
+    // Check for setInterval without cleanup
+    component.path.traverse({
+        CallExpression(path) {
+            const node = path.node;
+            const { callee } = node;
+            if (t.isIdentifier(callee) && callee.name === 'setInterval') {
+                // Check if result is stored
+                const parent = path.parentPath;
+                if (!parent || !t.isVariableDeclarator(parent.node)) {
+                    const loc = node.loc;
+                    smells.push({
+                        type: 'memory-leak-timer',
+                        severity: 'error',
+                        message: `setInterval without storing ID for cleanup in "${component.name}"`,
+                        file: filePath,
+                        line: loc?.start.line || 0,
+                        column: loc?.start.column || 0,
+                        suggestion: 'Store the interval ID and call clearInterval in cleanup.',
+                    });
+                }
+            }
+        },
+    });
+    // Check useEffect hooks for cleanup issues
+    for (const effect of component.hooks.useEffect) {
+        const hasCleanup = checkEffectHasCleanup(effect);
+        const hasSubscription = checkEffectHasSubscription(effect, component);
+        const hasEventListener = checkEffectHasEventListener(effect, component);
+        if (hasEventListener && !hasCleanup) {
+            const loc = effect.loc;
+            smells.push({
+                type: 'memory-leak-event-listener',
+                severity: 'error',
+                message: `useEffect adds event listener without cleanup in "${component.name}"`,
+                file: filePath,
+                line: loc?.start.line || 0,
+                column: 0,
+                suggestion: 'Return a cleanup function that calls removeEventListener.',
+            });
+        }
+        if (hasSubscription && !hasCleanup) {
+            const loc = effect.loc;
+            smells.push({
+                type: 'memory-leak-subscription',
+                severity: 'error',
+                message: `useEffect creates subscription without cleanup in "${component.name}"`,
+                file: filePath,
+                line: loc?.start.line || 0,
+                column: 0,
+                suggestion: 'Return a cleanup function that unsubscribes.',
+            });
+        }
+    }
+    return smells;
+}
+function checkEffectHasCleanup(effect) {
+    const args = effect.arguments;
+    if (args.length === 0)
+        return false;
+    const callback = args[0];
+    if (!t.isArrowFunctionExpression(callback) && !t.isFunctionExpression(callback)) {
+        return false;
+    }
+    // Check if callback body has a return statement
+    if (t.isBlockStatement(callback.body)) {
+        for (const stmt of callback.body.body) {
+            if (t.isReturnStatement(stmt) && stmt.argument) {
+                return true;
+            }
+        }
+    }
+    return false;
+}
+function checkEffectHasEventListener(effect, component) {
+    let hasListener = false;
+    const args = effect.arguments;
+    if (args.length === 0)
+        return false;
+    const callback = args[0];
+    if (!t.isArrowFunctionExpression(callback) && !t.isFunctionExpression(callback)) {
+        return false;
+    }
+    if (t.isBlockStatement(callback.body)) {
+        const sourceCode = JSON.stringify(callback.body);
+        hasListener = sourceCode.includes('addEventListener');
+    }
+    return hasListener;
+}
+function checkEffectHasSubscription(effect, component) {
+    let hasSubscription = false;
+    const args = effect.arguments;
+    if (args.length === 0)
+        return false;
+    const callback = args[0];
+    if (!t.isArrowFunctionExpression(callback) && !t.isFunctionExpression(callback)) {
+        return false;
+    }
+    if (t.isBlockStatement(callback.body)) {
+        const sourceCode = JSON.stringify(callback.body);
+        hasSubscription = sourceCode.includes('subscribe') || sourceCode.includes('addListener');
+    }
+    return hasSubscription;
+}

package/dist/fixer.d.ts

@@ -0,0 +1,23 @@
+import { CodeSmell, SmellType } from './types/index.js';
+export interface FixResult {
+    file: string;
+    fixedSmells: CodeSmell[];
+    skippedSmells: CodeSmell[];
+}
+/**
+ * Check if a smell type can be auto-fixed
+ */
+export declare function isFixable(smell: CodeSmell): boolean;
+/**
+ * Apply fixes to a file based on detected smells
+ */
+export declare function fixFile(filePath: string, smells: CodeSmell[]): Promise<FixResult>;
+/**
+ * Get list of fixable smell types
+ */
+export declare function getFixableTypes(): SmellType[];
+/**
+ * Describe what fix will be applied for a smell type
+ */
+export declare function describeFixAction(type: SmellType): string;
+//# sourceMappingURL=fixer.d.ts.map

package/dist/fixer.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"fixer.d.ts","sourceRoot":"","sources":["../src/fixer.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,SAAS,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAExD,MAAM,WAAW,SAAS;IACxB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,SAAS,EAAE,CAAC;IACzB,aAAa,EAAE,SAAS,EAAE,CAAC;CAC5B;AAUD;;GAEG;AACH,wBAAgB,SAAS,CAAC,KAAK,EAAE,SAAS,GAAG,OAAO,CAEnD;AAED;;GAEG;AACH,wBAAsB,OAAO,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,GAAG,OAAO,CAAC,SAAS,CAAC,CAuCvF;AAyED;;GAEG;AACH,wBAAgB,eAAe,IAAI,SAAS,EAAE,CAE7C;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,IAAI,EAAE,SAAS,GAAG,MAAM,CAazD"}

package/dist/fixer.js

@@ -0,0 +1,133 @@
+import fs from 'fs/promises';
+// Smell types that can be auto-fixed
+const FIXABLE_TYPES = [
+    'debug-statement',
+    'js-var-usage',
+    'js-loose-equality',
+    'a11y-missing-alt',
+];
+/**
+ * Check if a smell type can be auto-fixed
+ */
+export function isFixable(smell) {
+    return FIXABLE_TYPES.includes(smell.type);
+}
+/**
+ * Apply fixes to a file based on detected smells
+ */
+export async function fixFile(filePath, smells) {
+    const fixableSmells = smells.filter(isFixable);
+    const skippedSmells = smells.filter(s => !isFixable(s));
+    if (fixableSmells.length === 0) {
+        return { file: filePath, fixedSmells: [], skippedSmells };
+    }
+    let content = await fs.readFile(filePath, 'utf-8');
+    const lines = content.split('\n');
+    const fixedSmells = [];
+    // Sort by line descending to fix from bottom up (preserves line numbers)
+    const sortedSmells = [...fixableSmells].sort((a, b) => b.line - a.line);
+    for (const smell of sortedSmells) {
+        const lineIndex = smell.line - 1;
+        if (lineIndex < 0 || lineIndex >= lines.length)
+            continue;
+        const originalLine = lines[lineIndex];
+        const fixedLine = applyFix(smell, originalLine);
+        if (fixedLine !== originalLine) {
+            if (fixedLine === null) {
+                // Remove the line entirely
+                lines.splice(lineIndex, 1);
+            }
+            else {
+                lines[lineIndex] = fixedLine;
+            }
+            fixedSmells.push(smell);
+        }
+    }
+    // Write back if any fixes were applied
+    if (fixedSmells.length > 0) {
+        await fs.writeFile(filePath, lines.join('\n'), 'utf-8');
+    }
+    return { file: filePath, fixedSmells, skippedSmells };
+}
+/**
+ * Apply a specific fix to a line of code
+ * Returns the fixed line, or null to remove the line
+ */
+function applyFix(smell, line) {
+    switch (smell.type) {
+        case 'debug-statement':
+            return fixDebugStatement(line);
+        case 'js-var-usage':
+            return fixVarUsage(line);
+        case 'js-loose-equality':
+            return fixLooseEquality(line);
+        case 'a11y-missing-alt':
+            return fixMissingAlt(line);
+        default:
+            return line;
+    }
+}
+/**
+ * Remove console.log, console.debug, debugger statements
+ */
+function fixDebugStatement(line) {
+    const trimmed = line.trim();
+    // If line is just a console statement or debugger, remove it
+    if (/^console\.(log|debug|info|warn|error|trace|dir)\s*\(.*\);?\s*$/.test(trimmed)) {
+        return null;
+    }
+    if (/^debugger;?\s*$/.test(trimmed)) {
+        return null;
+    }
+    // If console is part of a larger expression, comment it out
+    if (/console\.(log|debug|info|warn|error|trace|dir)\s*\(/.test(line)) {
+        return line.replace(/console\.(log|debug|info|warn|error|trace|dir)\s*\([^)]*\);?/g, '/* $& */');
+    }
+    return line;
+}
+/**
+ * Replace var with let or const
+ */
+function fixVarUsage(line) {
+    // Simple heuristic: if reassigned later, use let; otherwise const
+    // For auto-fix, default to let (safer)
+    return line.replace(/\bvar\s+/, 'let ');
+}
+/**
+ * Replace == with === and != with !==
+ */
+function fixLooseEquality(line) {
+    return line
+        .replace(/([^=!])={2}([^=])/g, '$1===$2')
+        .replace(/!={1}([^=])/g, '!==$1');
+}
+/**
+ * Add alt="" to img tags missing alt attribute
+ */
+function fixMissingAlt(line) {
+    // Add alt="" to <img> tags without alt
+    return line.replace(/<img\s+(?![^>]*\balt\b)([^>]*?)(\/?>)/gi, '<img $1alt="" $2');
+}
+/**
+ * Get list of fixable smell types
+ */
+export function getFixableTypes() {
+    return [...FIXABLE_TYPES];
+}
+/**
+ * Describe what fix will be applied for a smell type
+ */
+export function describeFixAction(type) {
+    switch (type) {
+        case 'debug-statement':
+            return 'Remove console.log/debugger statements';
+        case 'js-var-usage':
+            return 'Replace var with let';
+        case 'js-loose-equality':
+            return 'Replace == with === and != with !==';
+        case 'a11y-missing-alt':
+            return 'Add alt="" to images';
+        default:
+            return 'Not auto-fixable';
+    }
+}

package/dist/git.d.ts

@@ -0,0 +1,28 @@
+export interface GitInfo {
+    isGitRepo: boolean;
+    currentBranch?: string;
+    changedFiles: string[];
+    stagedFiles: string[];
+    untrackedFiles: string[];
+}
+/**
+ * Get git information for a directory
+ */
+export declare function getGitInfo(rootDir: string): GitInfo;
+/**
+ * Get files changed since a specific commit or branch
+ */
+export declare function getFilesSince(rootDir: string, ref: string): string[];
+/**
+ * Get files changed in the last N commits
+ */
+export declare function getFilesFromLastCommits(rootDir: string, count?: number): string[];
+/**
+ * Get all modified files (changed + staged + untracked)
+ */
+export declare function getAllModifiedFiles(rootDir: string): string[];
+/**
+ * Filter file list to only include React/JS/TS files
+ */
+export declare function filterReactFiles(files: string[]): string[];
+//# sourceMappingURL=git.d.ts.map

package/dist/git.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"git.d.ts","sourceRoot":"","sources":["../src/git.ts"],"names":[],"mappings":"AAGA,MAAM,WAAW,OAAO;IACtB,SAAS,EAAE,OAAO,CAAC;IACnB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,WAAW,EAAE,MAAM,EAAE,CAAC;IACtB,cAAc,EAAE,MAAM,EAAE,CAAC;CAC1B;AAED;;GAEG;AACH,wBAAgB,UAAU,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CA0DnD;AAED;;GAEG;AACH,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,EAAE,CAcpE;AAED;;GAEG;AACH,wBAAgB,uBAAuB,CAAC,OAAO,EAAE,MAAM,EAAE,KAAK,GAAE,MAAU,GAAG,MAAM,EAAE,CAcpF;AAED;;GAEG;AACH,wBAAgB,mBAAmB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAW7D;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,MAAM,EAAE,CAG1D"}

package/dist/git.js

@@ -0,0 +1,117 @@
+import { execSync } from 'child_process';
+import path from 'path';
+/**
+ * Get git information for a directory
+ */
+export function getGitInfo(rootDir) {
+    try {
+        // Check if it's a git repo
+        execSync('git rev-parse --is-inside-work-tree', {
+            cwd: rootDir,
+            stdio: 'pipe',
+        });
+        const currentBranch = execSync('git branch --show-current', {
+            cwd: rootDir,
+            encoding: 'utf-8',
+        }).trim();
+        // Get changed files (modified but not staged)
+        const changedOutput = execSync('git diff --name-only', {
+            cwd: rootDir,
+            encoding: 'utf-8',
+        });
+        const changedFiles = changedOutput
+            .split('\n')
+            .filter(f => f.trim())
+            .map(f => path.resolve(rootDir, f));
+        // Get staged files
+        const stagedOutput = execSync('git diff --cached --name-only', {
+            cwd: rootDir,
+            encoding: 'utf-8',
+        });
+        const stagedFiles = stagedOutput
+            .split('\n')
+            .filter(f => f.trim())
+            .map(f => path.resolve(rootDir, f));
+        // Get untracked files
+        const untrackedOutput = execSync('git ls-files --others --exclude-standard', {
+            cwd: rootDir,
+            encoding: 'utf-8',
+        });
+        const untrackedFiles = untrackedOutput
+            .split('\n')
+            .filter(f => f.trim())
+            .map(f => path.resolve(rootDir, f));
+        return {
+            isGitRepo: true,
+            currentBranch,
+            changedFiles,
+            stagedFiles,
+            untrackedFiles,
+        };
+    }
+    catch {
+        return {
+            isGitRepo: false,
+            changedFiles: [],
+            stagedFiles: [],
+            untrackedFiles: [],
+        };
+    }
+}
+/**
+ * Get files changed since a specific commit or branch
+ */
+export function getFilesSince(rootDir, ref) {
+    try {
+        const output = execSync(`git diff --name-only ${ref}`, {
+            cwd: rootDir,
+            encoding: 'utf-8',
+        });
+        return output
+            .split('\n')
+            .filter(f => f.trim())
+            .map(f => path.resolve(rootDir, f));
+    }
+    catch {
+        return [];
+    }
+}
+/**
+ * Get files changed in the last N commits
+ */
+export function getFilesFromLastCommits(rootDir, count = 1) {
+    try {
+        const output = execSync(`git diff --name-only HEAD~${count}`, {
+            cwd: rootDir,
+            encoding: 'utf-8',
+        });
+        return output
+            .split('\n')
+            .filter(f => f.trim())
+            .map(f => path.resolve(rootDir, f));
+    }
+    catch {
+        return [];
+    }
+}
+/**
+ * Get all modified files (changed + staged + untracked)
+ */
+export function getAllModifiedFiles(rootDir) {
+    const info = getGitInfo(rootDir);
+    if (!info.isGitRepo)
+        return [];
+    const allFiles = new Set([
+        ...info.changedFiles,
+        ...info.stagedFiles,
+        ...info.untrackedFiles,
+    ]);
+    return Array.from(allFiles);
+}
+/**
+ * Filter file list to only include React/JS/TS files
+ */
+export function filterReactFiles(files) {
+    const extensions = ['.tsx', '.jsx', '.ts', '.js'];
+    return files.filter(f => extensions.some(ext => f.endsWith(ext)));
+}

package/dist/reporter.js

@@ -255,6 +255,19 @@ function formatSmellType(type) {
         'a11y-interactive-role': '♿ Interactive Role',
         'a11y-keyboard': '♿ Keyboard Handler',
         'a11y-semantic': '♿ Semantic HTML',
+        // Complexity
+        'high-cyclomatic-complexity': '🧮 High Cyclomatic Complexity',
+        'high-cognitive-complexity': '🧠 High Cognitive Complexity',
+        // Memory Leaks
+        'memory-leak-event-listener': '💧 Event Listener Leak',
+        'memory-leak-subscription': '💧 Subscription Leak',
+        'memory-leak-timer': '💧 Timer Leak',
+        'memory-leak-async': '💧 Async Leak Risk',
+        // Import Issues
+        'circular-dependency': '🔄 Circular Dependency',
+        'barrel-file-import': '📦 Barrel File Import',
+        'namespace-import': '📦 Namespace Import',
+        'excessive-imports': '📦 Excessive Imports',
     };
     return labels[type] || type;
 }

package/dist/types/index.d.ts

@@ -1,5 +1,5 @@
 export type SmellSeverity = 'error' | 'warning' | 'info';
-export type SmellType = 'useEffect-overuse' | 'prop-drilling' | 'large-component' | 'unmemoized-calculation' | 'missing-dependency' | 'state-in-loop' | 'inline-function-prop' | 'deep-nesting' | 'missing-key' | 'hooks-rules-violation' | 'dependency-array-issue' | 'nested-ternary' | 'dead-code' | 'magic-value' | 'debug-statement' | 'todo-comment' | 'security-xss' | 'security-eval' | 'security-secrets' | 'a11y-missing-alt' | 'a11y-missing-label' | 'a11y-interactive-role' | 'a11y-keyboard' | 'a11y-semantic' | 'nextjs-client-server-boundary' | 'nextjs-missing-metadata' | 'nextjs-image-unoptimized' | 'nextjs-router-misuse' | 'rn-inline-style' | 'rn-missing-accessibility' | 'rn-performance-issue' | 'nodejs-callback-hell' | 'nodejs-unhandled-promise' | 'nodejs-sync-io' | 'nodejs-missing-error-handling' | 'js-var-usage' | 'js-loose-equality' | 'js-implicit-coercion' | 'js-global-pollution' | 'ts-any-usage' | 'ts-missing-return-type' | 'ts-non-null-assertion' | 'ts-type-assertion';
+export type SmellType = 'useEffect-overuse' | 'prop-drilling' | 'large-component' | 'unmemoized-calculation' | 'missing-dependency' | 'state-in-loop' | 'inline-function-prop' | 'deep-nesting' | 'missing-key' | 'hooks-rules-violation' | 'dependency-array-issue' | 'nested-ternary' | 'dead-code' | 'magic-value' | 'debug-statement' | 'todo-comment' | 'security-xss' | 'security-eval' | 'security-secrets' | 'a11y-missing-alt' | 'a11y-missing-label' | 'a11y-interactive-role' | 'a11y-keyboard' | 'a11y-semantic' | 'nextjs-client-server-boundary' | 'nextjs-missing-metadata' | 'nextjs-image-unoptimized' | 'nextjs-router-misuse' | 'rn-inline-style' | 'rn-missing-accessibility' | 'rn-performance-issue' | 'nodejs-callback-hell' | 'nodejs-unhandled-promise' | 'nodejs-sync-io' | 'nodejs-missing-error-handling' | 'js-var-usage' | 'js-loose-equality' | 'js-implicit-coercion' | 'js-global-pollution' | 'ts-any-usage' | 'ts-missing-return-type' | 'ts-non-null-assertion' | 'ts-type-assertion' | 'high-cyclomatic-complexity' | 'high-cognitive-complexity' | 'memory-leak-event-listener' | 'memory-leak-subscription' | 'memory-leak-timer' | 'memory-leak-async' | 'circular-dependency' | 'barrel-file-import' | 'namespace-import' | 'excessive-imports';
 export interface CodeSmell {
     type: SmellType;
     severity: SmellSeverity;
@@ -75,6 +75,12 @@ export interface DetectorConfig {
     checkDebugStatements: boolean;
     checkSecurity: boolean;
     checkAccessibility: boolean;
+    checkComplexity: boolean;
+    maxCyclomaticComplexity: number;
+    maxCognitiveComplexity: number;
+    maxNestingDepth: number;
+    checkMemoryLeaks: boolean;
+    checkImports: boolean;
 }
 export declare const DEFAULT_CONFIG: DetectorConfig;
 //# sourceMappingURL=index.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/types/index.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,aAAa,GAAG,OAAO,GAAG,SAAS,GAAG,MAAM,CAAC;AAEzD,MAAM,MAAM,SAAS,GACjB,mBAAmB,GACnB,eAAe,GACf,iBAAiB,GACjB,wBAAwB,GACxB,oBAAoB,GACpB,eAAe,GACf,sBAAsB,GACtB,cAAc,GACd,aAAa,GACb,uBAAuB,GACvB,wBAAwB,GACxB,gBAAgB,GAChB,WAAW,GACX,aAAa,GAEb,iBAAiB,GACjB,cAAc,GAEd,cAAc,GACd,eAAe,GACf,kBAAkB,GAElB,kBAAkB,GAClB,oBAAoB,GACpB,uBAAuB,GACvB,eAAe,GACf,eAAe,GAEf,+BAA+B,GAC/B,yBAAyB,GACzB,0BAA0B,GAC1B,sBAAsB,GAEtB,iBAAiB,GACjB,0BAA0B,GAC1B,sBAAsB,GAEtB,sBAAsB,GACtB,0BAA0B,GAC1B,gBAAgB,GAChB,+BAA+B,GAE/B,cAAc,GACd,mBAAmB,GACnB,sBAAsB,GACtB,qBAAqB,GAErB,cAAc,GACd,wBAAwB,GACxB,uBAAuB,GACvB,mBAAmB,CAAC;AAExB,MAAM,WAAW,SAAS;IACxB,IAAI,EAAE,SAAS,CAAC;IAChB,QAAQ,EAAE,aAAa,CAAC;IACxB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,MAAM,WAAW,aAAa;IAC5B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,cAAc,EAAE,MAAM,CAAC;IACvB,aAAa,EAAE,MAAM,CAAC;IACtB,YAAY,EAAE,MAAM,CAAC;IACrB,gBAAgB,EAAE,MAAM,CAAC;IACzB,UAAU,EAAE,MAAM,CAAC;IACnB,kBAAkB,EAAE,MAAM,CAAC;IAC3B,uBAAuB,EAAE,OAAO,CAAC;CAClC;AAED,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,UAAU,EAAE,aAAa,EAAE,CAAC;IAC5B,MAAM,EAAE,SAAS,EAAE,CAAC;IACpB,OAAO,EAAE,MAAM,EAAE,CAAC;CACnB;AAED,MAAM,WAAW,cAAc;IAC7B,KAAK,EAAE,YAAY,EAAE,CAAC;IACtB,OAAO,EAAE,eAAe,CAAC;IACzB,SAAS,EAAE,kBAAkB,CAAC;CAC/B;AAED,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAC;IACnB,eAAe,EAAE,MAAM,CAAC;IACxB,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,EAAE,MAAM,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC;IACxC,gBAAgB,EAAE,MAAM,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC;CACjD;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,GAAG,GAAG,GAAG,GAAG,GAAG,GAAG,GAAG,GAAG,GAAG,CAAC;IACnC,SAAS,EAAE;QACT,cAAc,EAAE,MAAM,CAAC;QACvB,iBAAiB,EAAE,MAAM,CAAC;QAC1B,kBAAkB,EAAE,MAAM,CAAC;QAC3B,gBAAgB,EAAE,MAAM,CAAC;KAC1B,CAAC;IACF,qBAAqB,EAAE,MAAM,CAAC;CAC/B;AAED,MAAM,WAAW,cAAc;IAC7B,yBAAyB,EAAE,MAAM,CAAC;IAClC,oBAAoB,EAAE,MAAM,CAAC;IAC7B,iBAAiB,EAAE,MAAM,CAAC;IAC1B,aAAa,EAAE,MAAM,CAAC;IACtB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,gBAAgB,EAAE,OAAO,CAAC;IAC1B,eAAe,EAAE,OAAO,CAAC;IACzB,qBAAqB,EAAE,OAAO,CAAC;IAC/B,eAAe,EAAE,MAAM,CAAC;IACxB,aAAa,EAAE,OAAO,CAAC;IACvB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,oBAAoB,EAAE,MAAM,CAAC;IAE7B,WAAW,EAAE,OAAO,CAAC;IACrB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,WAAW,EAAE,OAAO,CAAC;IACrB,eAAe,EAAE,OAAO,CAAC;IACzB,eAAe,EAAE,OAAO,CAAC;IACzB,gBAAgB,EAAE,MAAM,CAAC;IAEzB,oBAAoB,EAAE,OAAO,CAAC;IAC9B,aAAa,EAAE,OAAO,CAAC;IACvB,kBAAkB,EAAE,OAAO,CAAC;CAC7B;AAED,eAAO,MAAM,cAAc,EAAE,cAwB5B,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/types/index.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,aAAa,GAAG,OAAO,GAAG,SAAS,GAAG,MAAM,CAAC;AAEzD,MAAM,MAAM,SAAS,GACjB,mBAAmB,GACnB,eAAe,GACf,iBAAiB,GACjB,wBAAwB,GACxB,oBAAoB,GACpB,eAAe,GACf,sBAAsB,GACtB,cAAc,GACd,aAAa,GACb,uBAAuB,GACvB,wBAAwB,GACxB,gBAAgB,GAChB,WAAW,GACX,aAAa,GAEb,iBAAiB,GACjB,cAAc,GAEd,cAAc,GACd,eAAe,GACf,kBAAkB,GAElB,kBAAkB,GAClB,oBAAoB,GACpB,uBAAuB,GACvB,eAAe,GACf,eAAe,GAEf,+BAA+B,GAC/B,yBAAyB,GACzB,0BAA0B,GAC1B,sBAAsB,GAEtB,iBAAiB,GACjB,0BAA0B,GAC1B,sBAAsB,GAEtB,sBAAsB,GACtB,0BAA0B,GAC1B,gBAAgB,GAChB,+BAA+B,GAE/B,cAAc,GACd,mBAAmB,GACnB,sBAAsB,GACtB,qBAAqB,GAErB,cAAc,GACd,wBAAwB,GACxB,uBAAuB,GACvB,mBAAmB,GAEnB,4BAA4B,GAC5B,2BAA2B,GAE3B,4BAA4B,GAC5B,0BAA0B,GAC1B,mBAAmB,GACnB,mBAAmB,GAEnB,qBAAqB,GACrB,oBAAoB,GACpB,kBAAkB,GAClB,mBAAmB,CAAC;AAExB,MAAM,WAAW,SAAS;IACxB,IAAI,EAAE,SAAS,CAAC;IAChB,QAAQ,EAAE,aAAa,CAAC;IACxB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,MAAM,WAAW,aAAa;IAC5B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,cAAc,EAAE,MAAM,CAAC;IACvB,aAAa,EAAE,MAAM,CAAC;IACtB,YAAY,EAAE,MAAM,CAAC;IACrB,gBAAgB,EAAE,MAAM,CAAC;IACzB,UAAU,EAAE,MAAM,CAAC;IACnB,kBAAkB,EAAE,MAAM,CAAC;IAC3B,uBAAuB,EAAE,OAAO,CAAC;CAClC;AAED,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,UAAU,EAAE,aAAa,EAAE,CAAC;IAC5B,MAAM,EAAE,SAAS,EAAE,CAAC;IACpB,OAAO,EAAE,MAAM,EAAE,CAAC;CACnB;AAED,MAAM,WAAW,cAAc;IAC7B,KAAK,EAAE,YAAY,EAAE,CAAC;IACtB,OAAO,EAAE,eAAe,CAAC;IACzB,SAAS,EAAE,kBAAkB,CAAC;CAC/B;AAED,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAC;IACnB,eAAe,EAAE,MAAM,CAAC;IACxB,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,EAAE,MAAM,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC;IACxC,gBAAgB,EAAE,MAAM,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC;CACjD;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,GAAG,GAAG,GAAG,GAAG,GAAG,GAAG,GAAG,GAAG,GAAG,CAAC;IACnC,SAAS,EAAE;QACT,cAAc,EAAE,MAAM,CAAC;QACvB,iBAAiB,EAAE,MAAM,CAAC;QAC1B,kBAAkB,EAAE,MAAM,CAAC;QAC3B,gBAAgB,EAAE,MAAM,CAAC;KAC1B,CAAC;IACF,qBAAqB,EAAE,MAAM,CAAC;CAC/B;AAED,MAAM,WAAW,cAAc;IAC7B,yBAAyB,EAAE,MAAM,CAAC;IAClC,oBAAoB,EAAE,MAAM,CAAC;IAC7B,iBAAiB,EAAE,MAAM,CAAC;IAC1B,aAAa,EAAE,MAAM,CAAC;IACtB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,gBAAgB,EAAE,OAAO,CAAC;IAC1B,eAAe,EAAE,OAAO,CAAC;IACzB,qBAAqB,EAAE,OAAO,CAAC;IAC/B,eAAe,EAAE,MAAM,CAAC;IACxB,aAAa,EAAE,OAAO,CAAC;IACvB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,oBAAoB,EAAE,MAAM,CAAC;IAE7B,WAAW,EAAE,OAAO,CAAC;IACrB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,WAAW,EAAE,OAAO,CAAC;IACrB,eAAe,EAAE,OAAO,CAAC;IACzB,eAAe,EAAE,OAAO,CAAC;IACzB,gBAAgB,EAAE,MAAM,CAAC;IAEzB,oBAAoB,EAAE,OAAO,CAAC;IAC9B,aAAa,EAAE,OAAO,CAAC;IACvB,kBAAkB,EAAE,OAAO,CAAC;IAE5B,eAAe,EAAE,OAAO,CAAC;IACzB,uBAAuB,EAAE,MAAM,CAAC;IAChC,sBAAsB,EAAE,MAAM,CAAC;IAC/B,eAAe,EAAE,MAAM,CAAC;IAExB,gBAAgB,EAAE,OAAO,CAAC;IAE1B,YAAY,EAAE,OAAO,CAAC;CACvB;AAED,eAAO,MAAM,cAAc,EAAE,cAiC5B,CAAC"}

package/dist/types/index.js

@@ -22,4 +22,13 @@ export const DEFAULT_CONFIG = {
     checkDebugStatements: true,
     checkSecurity: true,
     checkAccessibility: true,
+    // Complexity
+    checkComplexity: true,
+    maxCyclomaticComplexity: 10,
+    maxCognitiveComplexity: 15,
+    maxNestingDepth: 4,
+    // Memory leaks
+    checkMemoryLeaks: true,
+    // Import analysis
+    checkImports: true,
 };

package/dist/watcher.d.ts

@@ -0,0 +1,16 @@
+import { DetectorConfig } from './types/index.js';
+export interface WatchOptions {
+    rootDir: string;
+    include: string[];
+    exclude: string[];
+    config: DetectorConfig;
+    showSnippets: boolean;
+    onAnalysis?: (fileCount: number, issueCount: number) => void;
+}
+/**
+ * Start watching a project for changes and re-analyze on each change
+ */
+export declare function startWatch(options: WatchOptions): {
+    close: () => void;
+};
+//# sourceMappingURL=watcher.d.ts.map

package/dist/watcher.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"watcher.d.ts","sourceRoot":"","sources":["../src/watcher.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,cAAc,EAAkB,MAAM,kBAAkB,CAAC;AAElE,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,MAAM,EAAE,cAAc,CAAC;IACvB,YAAY,EAAE,OAAO,CAAC;IACtB,UAAU,CAAC,EAAE,CAAC,SAAS,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,KAAK,IAAI,CAAC;CAC9D;AAED;;GAEG;AACH,wBAAgB,UAAU,CAAC,OAAO,EAAE,YAAY,GAAG;IAAE,KAAK,EAAE,MAAM,IAAI,CAAA;CAAE,CAgGvE"}

package/dist/watcher.js

@@ -0,0 +1,89 @@
+import chokidar from 'chokidar';
+import path from 'path';
+import chalk from 'chalk';
+import { analyzeProject } from './analyzer.js';
+import { reportResults } from './reporter.js';
+/**
+ * Start watching a project for changes and re-analyze on each change
+ */
+export function startWatch(options) {
+    const { rootDir, include = ['**/*.tsx', '**/*.jsx'], exclude = ['**/node_modules/**', '**/dist/**', '**/build/**'], config, showSnippets, onAnalysis, } = options;
+    console.log(chalk.cyan('\n👀 Watch mode started'));
+    console.log(chalk.dim(`   Watching: ${rootDir}`));
+    console.log(chalk.dim(`   Patterns: ${include.join(', ')}`));
+    console.log(chalk.dim('   Press Ctrl+C to stop\n'));
+    // Debounce timer
+    let debounceTimer = null;
+    let isAnalyzing = false;
+    const runAnalysis = async () => {
+        if (isAnalyzing)
+            return;
+        isAnalyzing = true;
+        console.log(chalk.dim('\n─────────────────────────────────────────'));
+        console.log(chalk.cyan('🔄 Re-analyzing...'));
+        try {
+            const result = await analyzeProject({
+                rootDir,
+                include,
+                exclude,
+                config,
+            });
+            // Clear console and show results
+            console.clear();
+            console.log(chalk.cyan('\n👀 Watch mode - Last update: ' + new Date().toLocaleTimeString()));
+            console.log(chalk.dim('   Press Ctrl+C to stop\n'));
+            const output = reportResults(result, {
+                format: 'console',
+                showCodeSnippets: showSnippets,
+                rootDir,
+            });
+            console.log(output);
+            if (onAnalysis) {
+                onAnalysis(result.summary.totalFiles, result.summary.totalSmells);
+            }
+        }
+        catch (error) {
+            console.error(chalk.red('Analysis error:'), error.message);
+        }
+        finally {
+            isAnalyzing = false;
+        }
+    };
+    // Debounced analysis trigger
+    const triggerAnalysis = () => {
+        if (debounceTimer) {
+            clearTimeout(debounceTimer);
+        }
+        debounceTimer = setTimeout(runAnalysis, 300);
+    };
+    // Set up file watcher
+    const watchPatterns = include.map(p => path.join(rootDir, p));
+    const watcher = chokidar.watch(watchPatterns, {
+        ignored: exclude.map(p => {
+            if (p.startsWith('**/'))
+                return p;
+            return path.join(rootDir, p);
+        }),
+        persistent: true,
+        ignoreInitial: false,
+        awaitWriteFinish: {
+            stabilityThreshold: 100,
+            pollInterval: 50,
+        },
+    });
+    watcher
+        .on('add', () => triggerAnalysis())
+        .on('change', () => triggerAnalysis())
+        .on('unlink', () => triggerAnalysis())
+        .on('error', error => console.error(chalk.red('Watcher error:'), error));
+    // Initial analysis
+    triggerAnalysis();
+    return {
+        close: () => {
+            if (debounceTimer)
+                clearTimeout(debounceTimer);
+            watcher.close();
+            console.log(chalk.yellow('\n👋 Watch mode stopped'));
+        },
+    };
+}

package/package.json

@@ -1,12 +1,17 @@
 {
   "name": "react-code-smell-detector",
-  "version": "1.2.0",
-  "description": "Detect code smells in React projects - useEffect overuse, prop drilling, large components, security issues, accessibility, and more",
+  "version": "1.3.1",
+  "description": "Detect code smells in React projects - useEffect overuse, prop drilling, large components, security issues, accessibility, memory leaks, and more",
   "main": "dist/index.js",
   "bin": {
     "react-smell": "dist/cli.js"
   },
   "type": "module",
+  "files": [
+    "dist",
+    "README.md",
+    "LICENSE"
+  ],
   "scripts": {
     "build": "tsc",
     "dev": "tsc --watch",
@@ -29,6 +34,7 @@
     "@babel/traverse": "^7.23.0",
     "@babel/types": "^7.23.0",
     "chalk": "^5.3.0",
+    "chokidar": "^5.0.0",
     "commander": "^11.1.0",
     "fast-glob": "^3.3.2",
     "ora": "^8.0.1"