pasika 0.4.2 → 0.4.3

package/dist/eslint/pasika/index.js

@@ -66,11 +66,12 @@ function componentFromVariable(node) {
     smart: containsSmartCall(initializer)
   };
 }
-function parseComponentInfo(text, filename) {
+function parseComponentInfo(text, filename, options) {
   const sourceFile = ts.createSourceFile(path.resolve(filename), text, ts.ScriptTarget.Latest, true, ts.ScriptKind.TSX);
   const components = [];
   for (const statement of sourceFile.statements) {
-    if (!isExported(statement)) continue;
+    const exported = isExported(statement);
+    if (!exported && !options?.includeNonExported) continue;
     if (ts.isFunctionDeclaration(statement)) {
       const component = componentFromFunction(statement);
       if (component) components.push(component);
@@ -350,59 +351,26 @@ var importBoundariesRule = {
 };
 
 // eslint/rules/no-mixed-concerns.ts
-function getExportName(declaration) {
-  if (declaration?.type === "FunctionDeclaration" && declaration.id) {
-    return declaration.id.name ?? null;
-  }
-  if (declaration?.type === "VariableDeclaration") {
-    const declarator = declaration.declarations?.[0];
-    if (declarator?.id?.type === "Identifier" && (declarator.init?.type === "ArrowFunctionExpression" || declarator.init?.type === "FunctionExpression")) {
-      return declarator.id.name ?? null;
-    }
-  }
-  if (declaration?.type === "ArrowFunctionExpression") {
-    return "default";
-  }
-  if (declaration?.type === "FunctionExpression" && declaration.id) {
-    return declaration.id.name ?? null;
-  }
-  return null;
-}
 var noMixedConcernsRule = {
   meta: {
     schema: [],
     type: "problem",
     docs: {
-      description: "Enforce one exported React component per .tsx file."
+      description: "Enforce one React component per .tsx file, counting private components."
     }
   },
   create(context) {
     if (!context.filename.endsWith(".tsx")) return {};
-    let exportedComponentCount = 0;
-    const extraExports = [];
-    function registerExport(name) {
-      exportedComponentCount++;
-      if (exportedComponentCount > 1) {
-        extraExports.push({ name });
-      }
-    }
+    const sourceCode = context.sourceCode.text;
     return {
-      ExportNamedDeclaration(node) {
-        const name = getExportName(node.declaration);
-        if (name) registerExport(name);
-      },
-      ExportDefaultDeclaration(node) {
-        const name = getExportName(node.declaration);
-        if (name) registerExport(name);
-      },
       "Program:exit"() {
-        if (exportedComponentCount > 1) {
-          for (const extra of extraExports) {
-            context.report({
-              loc: { line: 1, column: 0 },
-              message: `File exports multiple components. "${extra.name}" is an extra component export. Move it to its own file. Each .tsx file MUST contain exactly one component. See docs/code-organization-guide/rules/no-mixed-concerns-rule.md`
-            });
-          }
+        const components = parseComponentInfo(sourceCode, context.filename, { includeNonExported: true });
+        if (components.length <= 1) return;
+        for (const extra of components.slice(1)) {
+          context.report({
+            loc: { line: 1, column: 0 },
+            message: `File defines multiple components. "${extra.name}" is an extra component. Move it to its own file. Each component file MUST define exactly one component. See docs/code-organization-guide/rules/no-mixed-concerns-rule.md`
+          });
         }
       }
     };
@@ -885,7 +853,9 @@ function isOuterLayoutClass(className) {
 function stringArguments(node) {
   if (node?.type === "Literal" && typeof node.value === "string") return [node.value];
   if (node?.type === "TemplateLiteral") return node.quasis.map((quasi) => quasi.value.raw);
-  if (node?.type === "ConditionalExpression") return [...stringArguments(node.consequent), ...stringArguments(node.alternate)];
+  if (node?.type === "ConditionalExpression") {
+    return [...stringArguments(node.consequent), ...stringArguments(node.alternate)];
+  }
   if (node?.type === "LogicalExpression") return [...stringArguments(node.left), ...stringArguments(node.right)];
   if (node?.type === "CallExpression" && node.callee.type === "Identifier" && node.callee.name === "cn") {
     return node.arguments.flatMap((argument) => stringArguments(argument));
@@ -901,7 +871,10 @@ var enforceCnMergeRule = {
         const element = node.parent.parent;
         const isComponentProp = isComponentName(element.openingElement?.name);
         if (isComponentProp && attributeName !== "className" && attributeName.endsWith("ClassName")) {
-          context.report({ node, message: "Expose appearance through typed variant props instead of separate internal class-name props. See docs/styling-guide/rules/class-composition-rule.md" });
+          context.report({
+            node,
+            message: "Expose appearance through typed variant props instead of separate internal class-name props. See docs/styling-guide/rules/class-composition-rule.md"
+          });
           return;
         }
         if (attributeName !== "className" && attributeName !== "class") return;
@@ -910,31 +883,60 @@ var enforceCnMergeRule = {
         if (isComponentProp && attributeName === "className") {
           const expression = valueNode.type === "JSXExpressionContainer" ? valueNode.expression : valueNode;
           const invalidClass = stringArguments(expression).flatMap((value) => value.split(/\s+/).filter(Boolean)).find((className) => !isOuterLayoutClass(className));
-          if (invalidClass) context.report({ node, message: `Passed className contains non-layout utility "${invalidClass}". Expose appearance through typed variant props. See docs/styling-guide/rules/class-composition-rule.md` });
+          if (invalidClass) {
+            context.report({
+              node,
+              message: `Passed className contains non-layout utility "${invalidClass}". Expose appearance through typed variant props. See docs/styling-guide/rules/class-composition-rule.md`
+            });
+          }
           return;
         }
         if (valueNode.type === "Literal" && typeof valueNode.value === "string") {
-          if (classCount(valueNode.value) > 5) context.report({ node, message: "Static className with more than 5 classes must use cn() with grouped string literals. See docs/styling-guide/rules/class-composition-rule.md" });
+          if (classCount(valueNode.value) > 5) {
+            context.report({
+              node,
+              message: "Static className with more than 5 classes must use cn() with grouped string literals. See docs/styling-guide/rules/class-composition-rule.md"
+            });
+          }
           return;
         }
         if (valueNode.type !== "JSXExpressionContainer") return;
         const expr = valueNode.expression;
         if (expr.type === "BinaryExpression" && expr.operator === "+") {
-          context.report({ node, message: "Use cn() instead of + operator for className. See docs/styling-guide/rules/class-composition-rule.md" });
+          context.report({
+            node,
+            message: "Use cn() instead of + operator for className. See docs/styling-guide/rules/class-composition-rule.md"
+          });
           return;
         }
         if (expr.type === "TemplateLiteral" && expr.expressions.length > 0) {
-          context.report({ node, message: "Use cn() instead of template literals with conditionals for className. See docs/styling-guide/rules/class-composition-rule.md" });
+          context.report({
+            node,
+            message: "Use cn() instead of template literals with conditionals for className. See docs/styling-guide/rules/class-composition-rule.md"
+          });
+          return;
+        }
+        if (expr.type === "LogicalExpression" || expr.type === "ConditionalExpression") {
+          context.report({
+            node,
+            message: "Use cn() for conditional classes in className. See docs/styling-guide/rules/class-composition-rule.md"
+          });
           return;
         }
         if (expr.type === "Literal" && typeof expr.value === "string" && classCount(expr.value) > 5) {
-          context.report({ node, message: "Static className with more than 5 classes must use cn() with grouped string literals. See docs/styling-guide/rules/class-composition-rule.md" });
+          context.report({
+            node,
+            message: "Static className with more than 5 classes must use cn() with grouped string literals. See docs/styling-guide/rules/class-composition-rule.md"
+          });
           return;
         }
         if (expr.type === "CallExpression" && expr.callee.type === "Identifier" && expr.callee.name === "cn") {
           for (const arg of expr.arguments) {
             if (arg.type === "Literal" && typeof arg.value === "string" && classCount(arg.value) > 5) {
-              context.report({ node: arg, message: "Each cn() string argument must contain at most 5 class names. Group by styling concern. See docs/styling-guide/rules/class-composition-rule.md" });
+              context.report({
+                node: arg,
+                message: "Each cn() string argument must contain at most 5 class names. Group by styling concern. See docs/styling-guide/rules/class-composition-rule.md"
+              });
             }
           }
         }
@@ -3284,6 +3286,7 @@ var ROLE_POSTFIXES = /* @__PURE__ */ new Set([
   "ScrollBar"
 ]);
 var CAMEL_CASE = /^[a-z][a-zA-Z0-9]*$/;
+var ENGLISH = /^[A-Za-z0-9_]*$/;
 function isLocalesFile2(filename) {
   const segments = path31.resolve(filename).split(path31.sep);
   const srcIdx = segments.lastIndexOf("src");
@@ -3298,6 +3301,14 @@ function reportFor(context, node, message) {
   context.report({ node, message });
 }
 function checkKey(context, node, name) {
+  if (!ENGLISH.test(name)) {
+    reportFor(
+      context,
+      node,
+      `Locale key "${name}" must be English; write it in the Latin alphabet. See docs/code-organization-guide/rules/locales-rule.md`
+    );
+    return;
+  }
   if (!CAMEL_CASE.test(name)) {
     reportFor(
       context,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pasika",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "description": "Reusable agent setup package",
   "repository": {
     "type": "git",