remediation 0.1.2 → 0.2.0

package/dist/core/rules/components/index.d.ts

@@ -2,4 +2,5 @@ import { Rule } from '../../types';
 export declare const componentRules: Rule[];
 export { deadComponentsRule } from './dead/rule';
 export { duplicatesRule } from './duplicates/rule';
+export { variantSplitRule } from './variant-split/rule';
 //# sourceMappingURL=index.d.ts.map

package/dist/core/rules/components/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/core/rules/components/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AAInC,eAAO,MAAM,cAAc,EAAE,IAAI,EAGhC,CAAC;AAEF,OAAO,EAAE,kBAAkB,EAAE,MAAM,aAAa,CAAC;AACjD,OAAO,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/core/rules/components/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AAKnC,eAAO,MAAM,cAAc,EAAE,IAAI,EAIhC,CAAC;AAEF,OAAO,EAAE,kBAAkB,EAAE,MAAM,aAAa,CAAC;AACjD,OAAO,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC;AACnD,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC"}

package/dist/core/rules/components/index.js

@@ -1,14 +1,18 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.duplicatesRule = exports.deadComponentsRule = exports.componentRules = void 0;
+exports.variantSplitRule = exports.duplicatesRule = exports.deadComponentsRule = exports.componentRules = void 0;
 const rule_1 = require("./dead/rule");
 const rule_2 = require("./duplicates/rule");
+const rule_3 = require("./variant-split/rule");
 exports.componentRules = [
     rule_1.deadComponentsRule,
     rule_2.duplicatesRule,
+    rule_3.variantSplitRule,
 ];
-var rule_3 = require("./dead/rule");
-Object.defineProperty(exports, "deadComponentsRule", { enumerable: true, get: function () { return rule_3.deadComponentsRule; } });
-var rule_4 = require("./duplicates/rule");
-Object.defineProperty(exports, "duplicatesRule", { enumerable: true, get: function () { return rule_4.duplicatesRule; } });
+var rule_4 = require("./dead/rule");
+Object.defineProperty(exports, "deadComponentsRule", { enumerable: true, get: function () { return rule_4.deadComponentsRule; } });
+var rule_5 = require("./duplicates/rule");
+Object.defineProperty(exports, "duplicatesRule", { enumerable: true, get: function () { return rule_5.duplicatesRule; } });
+var rule_6 = require("./variant-split/rule");
+Object.defineProperty(exports, "variantSplitRule", { enumerable: true, get: function () { return rule_6.variantSplitRule; } });
 //# sourceMappingURL=index.js.map

package/dist/core/rules/components/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../src/core/rules/components/index.ts"],"names":[],"mappings":";;;AACA,sCAAiD;AACjD,4CAAmD;AAEtC,QAAA,cAAc,GAAW;IACpC,yBAAkB;IAClB,qBAAc;CACf,CAAC;AAEF,oCAAiD;AAAxC,0GAAA,kBAAkB,OAAA;AAC3B,0CAAmD;AAA1C,sGAAA,cAAc,OAAA"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../src/core/rules/components/index.ts"],"names":[],"mappings":";;;AACA,sCAAiD;AACjD,4CAAmD;AACnD,+CAAwD;AAE3C,QAAA,cAAc,GAAW;IACpC,yBAAkB;IAClB,qBAAc;IACd,uBAAgB;CACjB,CAAC;AAEF,oCAAiD;AAAxC,0GAAA,kBAAkB,OAAA;AAC3B,0CAAmD;AAA1C,sGAAA,cAAc,OAAA;AACvB,6CAAwD;AAA/C,wGAAA,gBAAgB,OAAA"}

package/dist/core/rules/components/variant-split/rule.d.ts

@@ -0,0 +1,3 @@
+import { Rule } from '../../../types';
+export declare const variantSplitRule: Rule;
+//# sourceMappingURL=rule.d.ts.map

package/dist/core/rules/components/variant-split/rule.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"rule.d.ts","sourceRoot":"","sources":["../../../../../src/core/rules/components/variant-split/rule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAA0B,MAAM,gBAAgB,CAAC;AA+G9D,eAAO,MAAM,gBAAgB,EAAE,IA+B9B,CAAC"}

package/dist/core/rules/components/variant-split/rule.js

@@ -0,0 +1,97 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.variantSplitRule = void 0;
+const VARIANT_PROPS = ['variant', 'size', 'type', 'kind'];
+const propPattern = (props) => new RegExp(`(?:function\\s+\\w+\\s*\\(|const\\s+\\w+\\s*=\\s*(?:\\([^)]*\\)|\\w+)\\s*=>|const\\s+\\w+\\s*=\\s*\\()\\s*\\{\\s*(?:${props.join('|')})\\b`, 'g');
+const ternaryPattern = (props) => new RegExp(`\\b(${props.join('|')})\\s*(?:===?|!==?)\\s*['"][^'"]+['"]\\s*\\?`, 'g');
+const switchPattern = (props) => new RegExp(`switch\\s*\\(\\s*(${props.join('|')})\\s*\\)`, 'g');
+function findPropsInFile(content) {
+    const found = [];
+    for (const prop of VARIANT_PROPS) {
+        const regex = propPattern([prop]);
+        if (regex.test(content)) {
+            found.push(prop);
+        }
+    }
+    return found;
+}
+function findVariantsOnProp(content, prop) {
+    const issues = [];
+    const lines = content.split('\n');
+    const ternaryOnProp = new RegExp(`\\b${prop}\\s*(?:===?|!==?)\\s*['"][^'"]+['"]\\s*\\?`, 'g');
+    const ternaryShorthand = new RegExp(`\\b${prop}\\s*\\?`, 'g');
+    const sw = new RegExp(`switch\\s*\\(\\s*${prop}\\s*\\)`, 'g');
+    const ternaryConditionOnly = new RegExp(`\\b${prop}\\s*(?:===?|!==?)\\s*['"][^'"]+['"]`, 'g');
+    lines.forEach((line, index) => {
+        let match;
+        ternaryOnProp.lastIndex = 0;
+        while ((match = ternaryOnProp.exec(line)) !== null) {
+            issues.push({
+                line: index + 1,
+                column: match.index + 1,
+                message: `Conditional rendering based on ${prop}`,
+            });
+        }
+        ternaryShorthand.lastIndex = 0;
+        while ((match = ternaryShorthand.exec(line)) !== null) {
+            const alreadyFound = issues.some(i => i.line === index + 1 && i.column <= match.index + 10);
+            if (!alreadyFound) {
+                issues.push({
+                    line: index + 1,
+                    column: match.index + 1,
+                    message: `Conditional rendering based on ${prop}`,
+                });
+            }
+        }
+        sw.lastIndex = 0;
+        while ((match = sw.exec(line)) !== null) {
+            issues.push({
+                line: index + 1,
+                column: match.index + 1,
+                message: `Switch statement on ${prop}`,
+            });
+        }
+        ternaryConditionOnly.lastIndex = 0;
+        while ((match = ternaryConditionOnly.exec(line)) !== null) {
+            const nextLine = lines[index + 1];
+            if (nextLine && nextLine.trim().startsWith('?')) {
+                const alreadyFound = issues.some(i => i.line === index + 1);
+                if (!alreadyFound) {
+                    issues.push({
+                        line: index + 1,
+                        column: match.index + 1,
+                        message: `Conditional rendering based on ${prop}`,
+                    });
+                }
+            }
+        }
+    });
+    return issues;
+}
+exports.variantSplitRule = {
+    name: 'components/variant-split',
+    description: 'Detects components with conditional rendering based on variant props that should be split',
+    detect(file) {
+        const violations = [];
+        if (!file.path.match(/\.(tsx?|jsx?)$/)) {
+            return violations;
+        }
+        const propsInFile = findPropsInFile(file.content);
+        for (const prop of propsInFile) {
+            const issues = findVariantsOnProp(file.content, prop);
+            for (const issue of issues) {
+                violations.push({
+                    rule: 'components/variant-split',
+                    file: file.path,
+                    line: issue.line,
+                    column: issue.column,
+                    message: `${issue.message} — consider splitting into separate components`,
+                    severity: 'warning',
+                    suggestion: `Split into separate components based on ${prop} values`,
+                });
+            }
+        }
+        return violations;
+    },
+};
+//# sourceMappingURL=rule.js.map

package/dist/core/rules/components/variant-split/rule.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"rule.js","sourceRoot":"","sources":["../../../../../src/core/rules/components/variant-split/rule.ts"],"names":[],"mappings":";;;AAEA,MAAM,aAAa,GAAG,CAAC,SAAS,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC;AAE1D,MAAM,WAAW,GAAG,CAAC,KAAe,EAAE,EAAE,CACtC,IAAI,MAAM,CACR,uHAAuH,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,MAAM,EAC5I,GAAG,CACJ,CAAC;AAEJ,MAAM,cAAc,GAAG,CAAC,KAAe,EAAE,EAAE,CACzC,IAAI,MAAM,CACR,OAAO,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,6CAA6C,EACnE,GAAG,CACJ,CAAC;AAEJ,MAAM,aAAa,GAAG,CAAC,KAAe,EAAE,EAAE,CACxC,IAAI,MAAM,CACR,qBAAqB,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,UAAU,EAC9C,GAAG,CACJ,CAAC;AAEJ,SAAS,eAAe,CAAC,OAAe;IACtC,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,KAAK,MAAM,IAAI,IAAI,aAAa,EAAE,CAAC;QACjC,MAAM,KAAK,GAAG,WAAW,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;QAClC,IAAI,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;YACxB,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACnB,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAS,kBAAkB,CAAC,OAAe,EAAE,IAAY;IACvD,MAAM,MAAM,GAA6D,EAAE,CAAC;IAC5E,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAElC,MAAM,aAAa,GAAG,IAAI,MAAM,CAC9B,MAAM,IAAI,4CAA4C,EACtD,GAAG,CACJ,CAAC;IAEF,MAAM,gBAAgB,GAAG,IAAI,MAAM,CACjC,MAAM,IAAI,SAAS,EACnB,GAAG,CACJ,CAAC;IAEF,MAAM,EAAE,GAAG,IAAI,MAAM,CACnB,oBAAoB,IAAI,SAAS,EACjC,GAAG,CACJ,CAAC;IAEF,MAAM,oBAAoB,GAAG,IAAI,MAAM,CACrC,MAAM,IAAI,qCAAqC,EAC/C,GAAG,CACJ,CAAC;IAEF,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;QAC5B,IAAI,KAA6B,CAAC;QAElC,aAAa,CAAC,SAAS,GAAG,CAAC,CAAC;QAC5B,OAAO,CAAC,KAAK,GAAG,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;YACnD,MAAM,CAAC,IAAI,CAAC;gBACV,IAAI,EAAE,KAAK,GAAG,CAAC;gBACf,MAAM,EAAE,KAAK,CAAC,KAAK,GAAG,CAAC;gBACvB,OAAO,EAAE,kCAAkC,IAAI,EAAE;aAClD,CAAC,CAAC;QACL,CAAC;QAED,gBAAgB,CAAC,SAAS,GAAG,CAAC,CAAC;QAC/B,OAAO,CAAC,KAAK,GAAG,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;YACtD,MAAM,YAAY,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,KAAK,GAAG,CAAC,IAAI,CAAC,CAAC,MAAM,IAAI,KAAM,CAAC,KAAK,GAAG,EAAE,CAAC,CAAC;YAC7F,IAAI,CAAC,YAAY,EAAE,CAAC;gBAClB,MAAM,CAAC,IAAI,CAAC;oBACV,IAAI,EAAE,KAAK,GAAG,CAAC;oBACf,MAAM,EAAE,KAAK,CAAC,KAAK,GAAG,CAAC;oBACvB,OAAO,EAAE,kCAAkC,IAAI,EAAE;iBAClD,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,EAAE,CAAC,SAAS,GAAG,CAAC,CAAC;QACjB,OAAO,CAAC,KAAK,GAAG,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;YACxC,MAAM,CAAC,IAAI,CAAC;gBACV,IAAI,EAAE,KAAK,GAAG,CAAC;gBACf,MAAM,EAAE,KAAK,CAAC,KAAK,GAAG,CAAC;gBACvB,OAAO,EAAE,uBAAuB,IAAI,EAAE;aACvC,CAAC,CAAC;QACL,CAAC;QAED,oBAAoB,CAAC,SAAS,GAAG,CAAC,CAAC;QACnC,OAAO,CAAC,KAAK,GAAG,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;YAC1D,MAAM,QAAQ,GAAG,KAAK,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC;YAClC,IAAI,QAAQ,IAAI,QAAQ,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;gBAChD,MAAM,YAAY,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,KAAK,GAAG,CAAC,CAAC,CAAC;gBAC5D,IAAI,CAAC,YAAY,EAAE,CAAC;oBAClB,MAAM,CAAC,IAAI,CAAC;wBACV,IAAI,EAAE,KAAK,GAAG,CAAC;wBACf,MAAM,EAAE,KAAK,CAAC,KAAK,GAAG,CAAC;wBACvB,OAAO,EAAE,kCAAkC,IAAI,EAAE;qBAClD,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,OAAO,MAAM,CAAC;AAChB,CAAC;AAEY,QAAA,gBAAgB,GAAS;IACpC,IAAI,EAAE,0BAA0B;IAChC,WAAW,EAAE,2FAA2F;IAExG,MAAM,CAAC,IAAiB;QACtB,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,gBAAgB,CAAC,EAAE,CAAC;YACvC,OAAO,UAAU,CAAC;QACpB,CAAC;QAED,MAAM,WAAW,GAAG,eAAe,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAElD,KAAK,MAAM,IAAI,IAAI,WAAW,EAAE,CAAC;YAC/B,MAAM,MAAM,GAAG,kBAAkB,CAAC,IAAI,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC;YAEtD,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;gBAC3B,UAAU,CAAC,IAAI,CAAC;oBACd,IAAI,EAAE,0BAA0B;oBAChC,IAAI,EAAE,IAAI,CAAC,IAAI;oBACf,IAAI,EAAE,KAAK,CAAC,IAAI;oBAChB,MAAM,EAAE,KAAK,CAAC,MAAM;oBACpB,OAAO,EAAE,GAAG,KAAK,CAAC,OAAO,gDAAgD;oBACzE,QAAQ,EAAE,SAAS;oBACnB,UAAU,EAAE,2CAA2C,IAAI,SAAS;iBACrE,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,OAAO,UAAU,CAAC;IACpB,CAAC;CACF,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "remediation",
-  "version": "0.1.2",
+  "version": "0.2.0",
   "description": "CLI tool that scans React source code and detects design system inconsistencies",
   "license": "MIT",
   "bin": {