@sarj/eslint-plugin 1.0.1 → 2.0.0

package/package.json

@@ -1,52 +1,66 @@
 {
   "name": "@sarj/eslint-plugin",
-  "version": "1.0.1",
-  "description": "Custom ESLint rules collection for Sarj projects",
+  "version": "2.0.0",
+  "description": "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   "type": "module",
-  "main": "dist/index.js",
-  "types": "dist/index.d.ts",
+  "main": "./dist/index.cjs",
+  "module": "./dist/index.js",
+  "types": "./dist/index.d.ts",
   "exports": {
     ".": {
+      "types": "./dist/index.d.ts",
       "import": "./dist/index.js",
-      "types": "./dist/index.d.ts"
+      "require": "./dist/index.cjs"
     }
   },
   "files": [
-    "dist"
+    "dist",
+    "README.md"
   ],
   "publishConfig": {
-    "access": "public"
+    "access": "public",
+    "provenance": true
+  },
+  "sideEffects": false,
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/sarj-ai/linting.git",
+    "directory": "packages/typescript"
+  },
+  "homepage": "https://github.com/sarj-ai/linting/tree/main/packages/typescript",
+  "bugs": {
+    "url": "https://github.com/sarj-ai/linting/issues"
   },
   "scripts": {
-    "build": "tsc -p tsconfig.build.json",
+    "build": "tsup",
     "test": "vitest run",
-    "test:watch": "vitest",
-    "lint": "eslint src",
-    "prepublishOnly": "yarn build"
+    "typecheck": "tsc --noEmit"
   },
   "keywords": [
     "eslint",
     "eslintplugin",
     "eslint-plugin",
-    "zod",
-    "typescript",
-    "flat-config"
+    "sarj",
+    "react",
+    "nextjs",
+    "zod"
   ],
-  "author": "Sarj",
+  "author": "sarj-ai",
   "license": "MIT",
   "devDependencies": {
-    "@types/eslint": "^9.6.0",
-    "@types/node": "^22.0.0",
+    "@types/node": "^20.0.0",
     "@typescript-eslint/parser": "^8.0.0",
+    "@typescript-eslint/rule-tester": "^8.0.0",
+    "@typescript-eslint/utils": "^8.0.0",
     "eslint": "^9.0.0",
-    "typescript": "^5.7.0",
+    "tsup": "^8.3.0",
+    "typescript": "^5.6.0",
     "vitest": "^2.0.0"
   },
   "peerDependencies": {
     "eslint": ">=9.0.0"
   },
   "engines": {
-    "node": ">=18.0.0"
-  },
-  "packageManager": "yarn@4.12.0+sha512.f45ab632439a67f8bc759bf32ead036a1f413287b9042726b7cc4818b7b49e14e9423ba49b18f9e06ea4941c1ad062385b1d8760a8d5091a1a31e5f6219afca8"
+    "node": ">=20.0.0"
+  }
 }

package/dist/index.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAUnD;;;;;GAKG;AAEH,QAAA,MAAM,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,IAAI,CAAC,UAAU,CAQ1C,CAAC;AAEF,QAAA,MAAM,MAAM,EAAE,MAAM,CAAC,MAMpB,CAAC;AAGF,QAAA,MAAM,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,CAsE5C,CAAC;AAEF,OAAO,EAAE,MAAM,IAAI,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC"}

package/dist/rules/enforce-file-structure.d.ts

@@ -1,4 +0,0 @@
-import type { Rule } from "eslint";
-export declare const enforceFileStructure: Rule.RuleModule;
-export default enforceFileStructure;
-//# sourceMappingURL=enforce-file-structure.d.ts.map

package/dist/rules/enforce-file-structure.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"enforce-file-structure.d.ts","sourceRoot":"","sources":["../../src/rules/enforce-file-structure.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAgDnC,eAAO,MAAM,oBAAoB,EAAE,IAAI,CAAC,UA8DvC,CAAC;AAEF,eAAe,oBAAoB,CAAC"}

package/dist/rules/enforce-file-structure.js

@@ -1,90 +0,0 @@
-/**
- * @fileoverview Enforce consistent file structure
- * @description Enforces: imports -> types -> constants -> functions -> exports
- */
-const SECTIONS = ["imports", "types", "constants", "functions", "exports"];
-function getStatementSection(statement) {
-    const type = statement.type;
-    switch (type) {
-        case "ImportDeclaration":
-            return 0; // imports
-        case "TSTypeAliasDeclaration":
-        case "TSInterfaceDeclaration":
-        case "TSEnumDeclaration":
-            return 1; // types
-        case "VariableDeclaration": {
-            // Check if it's a constant (const with UPPER_CASE or simple values)
-            const varDecl = statement;
-            if (varDecl.kind === "const") {
-                const declarator = varDecl.declarations[0];
-                if (declarator?.id.type === "Identifier" &&
-                    declarator.id.name === declarator.id.name.toUpperCase()) {
-                    return 2; // constants
-                }
-            }
-            return 3; // functions (const fn = ...)
-        }
-        case "FunctionDeclaration":
-            return 3; // functions
-        case "ExportNamedDeclaration":
-        case "ExportDefaultDeclaration":
-            return 4; // exports
-        default:
-            return 3; // default to functions
-    }
-}
-export const enforceFileStructure = {
-    meta: {
-        type: "suggestion",
-        docs: {
-            description: "Enforce consistent file structure: imports -> types -> constants -> functions -> exports",
-            recommended: true,
-        },
-        schema: [],
-        messages: {
-            incorrectOrder: "File structure violation: {{current}} should come after {{expected}}",
-            useServerDirective: "Server action files must start with 'use server' directive",
-        },
-    },
-    create(context) {
-        const filename = context.filename;
-        const isServerAction = filename.includes("/actions/") || filename.includes("action");
-        return {
-            Program(node) {
-                const program = node;
-                const body = program.body;
-                let currentSection = 0;
-                // Check for "use server" directive in server action files
-                if (isServerAction) {
-                    const firstNode = body[0];
-                    if (!firstNode ||
-                        firstNode.type !== "ExpressionStatement" ||
-                        firstNode.expression.type !== "Literal" ||
-                        firstNode.expression.value !== "use server") {
-                        context.report({
-                            node,
-                            messageId: "useServerDirective",
-                        });
-                    }
-                }
-                for (const statement of body) {
-                    const statementSection = getStatementSection(statement);
-                    if (statementSection < currentSection) {
-                        context.report({
-                            node: statement,
-                            messageId: "incorrectOrder",
-                            data: {
-                                current: SECTIONS[statementSection],
-                                expected: SECTIONS[currentSection],
-                            },
-                        });
-                    }
-                    else {
-                        currentSection = Math.max(currentSection, statementSection);
-                    }
-                }
-            },
-        };
-    },
-};
-export default enforceFileStructure;

package/dist/rules/no-enum.d.ts

@@ -1,8 +0,0 @@
-import type { Rule } from "eslint";
-/**
- * @fileoverview Disallow TypeScript enums, prefer union types
- * @description Enums add runtime code and have unintuitive behavior. Use union types instead.
- */
-export declare const noEnum: Rule.RuleModule;
-export default noEnum;
-//# sourceMappingURL=no-enum.d.ts.map

package/dist/rules/no-enum.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"no-enum.d.ts","sourceRoot":"","sources":["../../src/rules/no-enum.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAEnC;;;GAGG;AAEH,eAAO,MAAM,MAAM,EAAE,IAAI,CAAC,UAyBzB,CAAC;AAEF,eAAe,MAAM,CAAC"}

package/dist/rules/no-enum.js

@@ -1,29 +0,0 @@
-/**
- * @fileoverview Disallow TypeScript enums, prefer union types
- * @description Enums add runtime code and have unintuitive behavior. Use union types instead.
- */
-export const noEnum = {
-    meta: {
-        type: "suggestion",
-        docs: {
-            description: "Disallow TypeScript enums, prefer union types or const objects",
-            recommended: true,
-        },
-        schema: [],
-        messages: {
-            noEnum: 'Enums are discouraged. Use union types (\'type Status = "active" | "inactive"\') or const objects instead.',
-        },
-    },
-    create(context) {
-        return {
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
-            TSEnumDeclaration(node) {
-                context.report({
-                    node,
-                    messageId: "noEnum",
-                });
-            },
-        };
-    },
-};
-export default noEnum;

package/dist/rules/no-raw-env.d.ts

@@ -1,8 +0,0 @@
-import type { Rule } from "eslint";
-/**
- * @fileoverview Disallow direct process.env access
- * @description Use Zod-validated env schema instead of process.env
- */
-export declare const noRawEnv: Rule.RuleModule;
-export default noRawEnv;
-//# sourceMappingURL=no-raw-env.d.ts.map

package/dist/rules/no-raw-env.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"no-raw-env.d.ts","sourceRoot":"","sources":["../../src/rules/no-raw-env.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAGnC;;;GAGG;AAEH,eAAO,MAAM,QAAQ,EAAE,IAAI,CAAC,UA8B3B,CAAC;AAEF,eAAe,QAAQ,CAAC"}

package/dist/rules/no-raw-env.js

@@ -1,33 +0,0 @@
-/**
- * @fileoverview Disallow direct process.env access
- * @description Use Zod-validated env schema instead of process.env
- */
-export const noRawEnv = {
-    meta: {
-        type: "problem",
-        docs: {
-            description: "Disallow direct process.env access",
-            recommended: true,
-        },
-        schema: [],
-        messages: {
-            noRawEnv: "Use Zod-validated env schema instead of process.env directly",
-        },
-    },
-    create(context) {
-        return {
-            MemberExpression(node) {
-                if (node.object.type === "Identifier" &&
-                    node.object.name === "process" &&
-                    node.property.type === "Identifier" &&
-                    node.property.name === "env") {
-                    context.report({
-                        node,
-                        messageId: "noRawEnv",
-                    });
-                }
-            },
-        };
-    },
-};
-export default noRawEnv;

package/dist/rules/prefer-shadcn.d.ts

@@ -1,4 +0,0 @@
-import type { Rule } from "eslint";
-export declare const preferShadcn: Rule.RuleModule;
-export default preferShadcn;
-//# sourceMappingURL=prefer-shadcn.d.ts.map

package/dist/rules/prefer-shadcn.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"prefer-shadcn.d.ts","sourceRoot":"","sources":["../../src/rules/prefer-shadcn.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAgBnC,eAAO,MAAM,YAAY,EAAE,IAAI,CAAC,UAwC/B,CAAC;AAEF,eAAe,YAAY,CAAC"}

package/dist/rules/prefer-shadcn.js

@@ -1,50 +0,0 @@
-/**
- * @fileoverview Prefer shadcn/ui components over native HTML elements
- * @description Use shadcn components for consistent design system
- */
-const REPLACEMENTS = {
-    button: "Button",
-    input: "Input",
-    select: "Select",
-    textarea: "Textarea",
-    table: "Table",
-    dialog: "Dialog",
-};
-export const preferShadcn = {
-    meta: {
-        type: "suggestion",
-        docs: {
-            description: "Prefer shadcn/ui components over native HTML elements",
-            recommended: true,
-        },
-        schema: [],
-        messages: {
-            preferShadcn: "Use shadcn <{{replacement}}> component from @/components/ui/{{lowercase}} instead of native <{{element}}>",
-        },
-    },
-    create(context) {
-        return {
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
-            JSXOpeningElement(node) {
-                // Only check lowercase element names (native HTML)
-                if (!node.name || node.name.type !== "JSXIdentifier") {
-                    return;
-                }
-                const elementName = node.name.name;
-                const replacement = REPLACEMENTS[elementName];
-                if (replacement) {
-                    context.report({
-                        node,
-                        messageId: "preferShadcn",
-                        data: {
-                            element: elementName,
-                            replacement,
-                            lowercase: elementName,
-                        },
-                    });
-                }
-            },
-        };
-    },
-};
-export default preferShadcn;

package/dist/rules/require-assert-never.d.ts

@@ -1,8 +0,0 @@
-import type { Rule } from "eslint";
-/**
- * @fileoverview Require assertNever in switch statement default cases
- * @description Ensures exhaustive type checking in switch statements
- */
-export declare const requireAssertNever: Rule.RuleModule;
-export default requireAssertNever;
-//# sourceMappingURL=require-assert-never.d.ts.map

package/dist/rules/require-assert-never.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"require-assert-never.d.ts","sourceRoot":"","sources":["../../src/rules/require-assert-never.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAGnC;;;GAGG;AAEH,eAAO,MAAM,kBAAkB,EAAE,IAAI,CAAC,UAwDrC,CAAC;AAEF,eAAe,kBAAkB,CAAC"}

package/dist/rules/require-assert-never.js

@@ -1,53 +0,0 @@
-/**
- * @fileoverview Require assertNever in switch statement default cases
- * @description Ensures exhaustive type checking in switch statements
- */
-export const requireAssertNever = {
-    meta: {
-        type: "problem",
-        docs: {
-            description: "Require assertNever in switch statement default cases for exhaustive type checking",
-            recommended: true,
-        },
-        schema: [],
-        messages: {
-            missingAssertNever: "Switch statement default case must call assertNever() for exhaustive type checking",
-        },
-    },
-    create(context) {
-        return {
-            SwitchStatement(node) {
-                const defaultCase = node.cases.find((caseNode) => caseNode.test === null);
-                if (!defaultCase) {
-                    return;
-                }
-                const hasAssertNever = defaultCase.consequent.some((statement) => {
-                    // Check for ExpressionStatement containing CallExpression
-                    if (statement.type === "ExpressionStatement") {
-                        const exprStmt = statement;
-                        if (exprStmt.expression.type === "CallExpression") {
-                            const callee = exprStmt.expression.callee;
-                            return callee.type === "Identifier" && callee.name === "assertNever";
-                        }
-                    }
-                    // Check for ThrowStatement with assertNever call
-                    if (statement.type === "ThrowStatement") {
-                        const throwStmt = statement;
-                        if (throwStmt.argument?.type === "CallExpression") {
-                            const callee = throwStmt.argument.callee;
-                            return callee.type === "Identifier" && callee.name === "assertNever";
-                        }
-                    }
-                    return false;
-                });
-                if (!hasAssertNever) {
-                    context.report({
-                        node: defaultCase,
-                        messageId: "missingAssertNever",
-                    });
-                }
-            },
-        };
-    },
-};
-export default requireAssertNever;

package/dist/rules/require-zod-form-validation.d.ts

@@ -1,8 +0,0 @@
-import type { Rule } from "eslint";
-/**
- * @fileoverview Require Zod validation when parsing FormData
- * @description Ensures form data is validated with Zod schemas
- */
-export declare const requireZodFormValidation: Rule.RuleModule;
-export default requireZodFormValidation;
-//# sourceMappingURL=require-zod-form-validation.d.ts.map

package/dist/rules/require-zod-form-validation.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"require-zod-form-validation.d.ts","sourceRoot":"","sources":["../../src/rules/require-zod-form-validation.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAGnC;;;GAGG;AAEH,eAAO,MAAM,wBAAwB,EAAE,IAAI,CAAC,UAkE3C,CAAC;AAEF,eAAe,wBAAwB,CAAC"}

package/dist/rules/require-zod-form-validation.js

@@ -1,59 +0,0 @@
-/**
- * @fileoverview Require Zod validation when parsing FormData
- * @description Ensures form data is validated with Zod schemas
- */
-export const requireZodFormValidation = {
-    meta: {
-        type: "problem",
-        docs: {
-            description: "Require Zod validation when parsing FormData",
-            recommended: true,
-        },
-        schema: [],
-        messages: {
-            missingZodValidation: "FormData parsing must use Zod schema validation (e.g., Schema.parse())",
-        },
-    },
-    create(context) {
-        return {
-            CallExpression(node) {
-                const callExpr = node;
-                // Check for formData.get() calls
-                if (callExpr.callee.type !== "MemberExpression") {
-                    return;
-                }
-                const callee = callExpr.callee;
-                if (callee.property.type !== "Identifier" ||
-                    callee.property.name !== "get" ||
-                    callee.object.type !== "Identifier" ||
-                    callee.object.name !== "formData") {
-                    return;
-                }
-                // Check if this is inside a Zod .parse() call
-                // eslint-disable-next-line @typescript-eslint/no-explicit-any
-                let parent = node.parent;
-                let hasZodValidation = false;
-                // Traverse up to find if we're inside a .parse() call
-                while (parent) {
-                    if (parent.type === "CallExpression" &&
-                        parent.callee.type === "MemberExpression") {
-                        const parentCallee = parent.callee;
-                        if (parentCallee.property.type === "Identifier" &&
-                            parentCallee.property.name === "parse") {
-                            hasZodValidation = true;
-                            break;
-                        }
-                    }
-                    parent = parent.parent;
-                }
-                if (!hasZodValidation) {
-                    context.report({
-                        node: node,
-                        messageId: "missingZodValidation",
-                    });
-                }
-            },
-        };
-    },
-};
-export default requireZodFormValidation;

package/dist/rules/zod-naming-convention.d.ts

@@ -1,8 +0,0 @@
-import type { Rule } from "eslint";
-/**
- * @fileoverview Enforce Zod schemas to be named with a Z prefix
- * @description Ensures consistent naming for Zod schemas (e.g., ZUserSchema instead of userSchema)
- */
-export declare const zodNamingConvention: Rule.RuleModule;
-export default zodNamingConvention;
-//# sourceMappingURL=zod-naming-convention.d.ts.map

package/dist/rules/zod-naming-convention.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"zod-naming-convention.d.ts","sourceRoot":"","sources":["../../src/rules/zod-naming-convention.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAGnC;;;GAGG;AAEH,eAAO,MAAM,mBAAmB,EAAE,IAAI,CAAC,UAqDtC,CAAC;AAEF,eAAe,mBAAmB,CAAC"}

package/dist/rules/zod-naming-convention.js

@@ -1,53 +0,0 @@
-/**
- * @fileoverview Enforce Zod schemas to be named with a Z prefix
- * @description Ensures consistent naming for Zod schemas (e.g., ZUserSchema instead of userSchema)
- */
-export const zodNamingConvention = {
-    meta: {
-        type: "suggestion",
-        docs: {
-            description: "Enforce Zod schemas to be named with a Z prefix",
-            recommended: false,
-        },
-        schema: [],
-        messages: {
-            zodPrefix: "Zod schema names should start with 'Z' (e.g., Z{{suggestedName}})",
-        },
-    },
-    create(context) {
-        return {
-            VariableDeclarator(node) {
-                const declarator = node;
-                if (!declarator.init || declarator.init.type !== "CallExpression") {
-                    return;
-                }
-                const callExpr = declarator.init;
-                if (!callExpr.callee || callExpr.callee.type !== "MemberExpression") {
-                    return;
-                }
-                const callee = callExpr.callee;
-                const isDirectZodCall = callee.object.type === "Identifier" &&
-                    callee.object.name === "z";
-                const isChainedZodCall = callee.object.type === "CallExpression" &&
-                    callee.object.callee?.type === "MemberExpression" &&
-                    callee.object.callee.object?.type === "Identifier" &&
-                    callee.object.callee.object.name === "z";
-                if (isDirectZodCall || isChainedZodCall) {
-                    if (declarator.id.type !== "Identifier") {
-                        return;
-                    }
-                    const variableName = declarator.id.name;
-                    if (!variableName.startsWith("Z")) {
-                        const suggestedName = variableName.charAt(0).toUpperCase() + variableName.slice(1);
-                        context.report({
-                            node: declarator.id,
-                            messageId: "zodPrefix",
-                            data: { suggestedName },
-                        });
-                    }
-                }
-            },
-        };
-    },
-};
-export default zodNamingConvention;