@sarj/eslint-plugin 1.0.0 → 2.0.0

package/package.json

@@ -1,52 +1,66 @@
 {
   "name": "@sarj/eslint-plugin",
-  "version": "1.0.0",
-  "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",
+    "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-enum.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=no-enum.test.d.ts.map

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

@@ -1 +0,0 @@
-{"version":3,"file":"no-enum.test.d.ts","sourceRoot":"","sources":["../../src/rules/no-enum.test.ts"],"names":[],"mappings":""}

package/dist/rules/no-enum.test.js

@@ -1,44 +0,0 @@
-import { RuleTester } from "eslint";
-import { describe, it } from "vitest";
-import { noEnum } from "./no-enum.js";
-// Note: This test requires @typescript-eslint/parser to work properly
-// For now, we'll skip the actual test execution and just verify the rule exports correctly
-describe("no-enum", () => {
-    it("should export a valid rule module", () => {
-        if (!noEnum.meta || !noEnum.create) {
-            throw new Error("Rule is not properly exported");
-        }
-        if (noEnum.meta.type !== "suggestion") {
-            throw new Error("Expected rule type to be 'suggestion'");
-        }
-        if (!noEnum.meta.messages?.noEnum) {
-            throw new Error("Expected noEnum message to be defined");
-        }
-    });
-    // Full test requires TypeScript parser
-    it("should work with TypeScript parser", async () => {
-        const parser = await import("@typescript-eslint/parser");
-        const ruleTester = new RuleTester({
-            languageOptions: {
-                ecmaVersion: 2022,
-                sourceType: "module",
-                parser,
-            },
-        });
-        ruleTester.run("no-enum", noEnum, {
-            valid: [
-                // Union types are OK
-                { code: 'type Status = "active" | "inactive";' },
-                // Const objects are OK
-                { code: 'const Status = { ACTIVE: "active", INACTIVE: "inactive" } as const;' },
-            ],
-            invalid: [
-                // Regular enum
-                {
-                    code: "enum Status { Active, Inactive }",
-                    errors: [{ messageId: "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/no-raw-env.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=no-raw-env.test.d.ts.map

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

@@ -1 +0,0 @@
-{"version":3,"file":"no-raw-env.test.d.ts","sourceRoot":"","sources":["../../src/rules/no-raw-env.test.ts"],"names":[],"mappings":""}

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

@@ -1,36 +0,0 @@
-import { RuleTester } from "eslint";
-import { describe, it } from "vitest";
-import { noRawEnv } from "./no-raw-env.js";
-const ruleTester = new RuleTester({
-    languageOptions: {
-        ecmaVersion: 2022,
-        sourceType: "module",
-    },
-});
-describe("no-raw-env", () => {
-    it("should pass valid and fail invalid cases", () => {
-        ruleTester.run("no-raw-env", noRawEnv, {
-            valid: [
-                // Using validated env schema
-                { code: 'const apiKey = env.API_KEY;' },
-                { code: 'const dbUrl = config.databaseUrl;' },
-                { code: 'const { NODE_ENV } = validatedEnv;' },
-            ],
-            invalid: [
-                // Direct process.env access
-                {
-                    code: 'const apiKey = process.env.API_KEY;',
-                    errors: [{ messageId: "noRawEnv" }],
-                },
-                {
-                    code: 'if (process.env.NODE_ENV === "production") {}',
-                    errors: [{ messageId: "noRawEnv" }],
-                },
-                {
-                    code: 'console.log(process.env.DEBUG);',
-                    errors: [{ messageId: "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/prefer-shadcn.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=prefer-shadcn.test.d.ts.map

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

@@ -1 +0,0 @@
-{"version":3,"file":"prefer-shadcn.test.d.ts","sourceRoot":"","sources":["../../src/rules/prefer-shadcn.test.ts"],"names":[],"mappings":""}

package/dist/rules/prefer-shadcn.test.js

@@ -1,58 +0,0 @@
-import { RuleTester } from "eslint";
-import { describe, it } from "vitest";
-import { preferShadcn } from "./prefer-shadcn.js";
-const ruleTester = new RuleTester({
-    languageOptions: {
-        ecmaVersion: 2022,
-        sourceType: "module",
-        parserOptions: {
-            ecmaFeatures: {
-                jsx: true,
-            },
-        },
-    },
-});
-describe("prefer-shadcn", () => {
-    it("should pass valid and fail invalid cases", () => {
-        ruleTester.run("prefer-shadcn", preferShadcn, {
-            valid: [
-                // shadcn components are OK
-                { code: '<Button onClick={handleClick}>Click me</Button>' },
-                { code: '<Input type="text" placeholder="Enter name" />' },
-                { code: '<Select value={value} onValueChange={setValue} />' },
-                { code: '<Table><TableBody /></Table>' },
-                // Other elements are OK
-                { code: '<div className="container">Content</div>' },
-                { code: '<span>Text</span>' },
-                { code: '<form onSubmit={handleSubmit}></form>' },
-            ],
-            invalid: [
-                // Native button
-                {
-                    code: '<button onClick={handleClick}>Click me</button>',
-                    errors: [{ messageId: "preferShadcn" }],
-                },
-                // Native input
-                {
-                    code: '<input type="text" placeholder="Enter name" />',
-                    errors: [{ messageId: "preferShadcn" }],
-                },
-                // Native select
-                {
-                    code: '<select value={value} onChange={handleChange}><option>A</option></select>',
-                    errors: [{ messageId: "preferShadcn" }],
-                },
-                // Native textarea
-                {
-                    code: '<textarea rows={5} />',
-                    errors: [{ messageId: "preferShadcn" }],
-                },
-                // Native table
-                {
-                    code: '<table><tbody /></table>',
-                    errors: [{ messageId: "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-assert-never.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=require-assert-never.test.d.ts.map

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

@@ -1 +0,0 @@
-{"version":3,"file":"require-assert-never.test.d.ts","sourceRoot":"","sources":["../../src/rules/require-assert-never.test.ts"],"names":[],"mappings":""}

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

@@ -1,76 +0,0 @@
-import { RuleTester } from "eslint";
-import { describe, it } from "vitest";
-import { requireAssertNever } from "./require-assert-never.js";
-const ruleTester = new RuleTester({
-    languageOptions: {
-        ecmaVersion: 2022,
-        sourceType: "module",
-    },
-});
-describe("require-assert-never", () => {
-    it("should pass valid and fail invalid cases", () => {
-        ruleTester.run("require-assert-never", requireAssertNever, {
-            valid: [
-                // Switch with assertNever in default case
-                {
-                    code: `
-            switch (status) {
-              case "active":
-                return 1;
-              default:
-                assertNever(status);
-            }
-          `,
-                },
-                // Switch with throw assertNever
-                {
-                    code: `
-            switch (type) {
-              case "a":
-                break;
-              default:
-                throw assertNever(type);
-            }
-          `,
-                },
-                // Switch without default case is OK
-                {
-                    code: `
-            switch (value) {
-              case 1:
-                return "one";
-              case 2:
-                return "two";
-            }
-          `,
-                },
-            ],
-            invalid: [
-                // Default case without assertNever
-                {
-                    code: `
-            switch (status) {
-              case "active":
-                return 1;
-              default:
-                return 0;
-            }
-          `,
-                    errors: [{ messageId: "missingAssertNever" }],
-                },
-                // Empty default case
-                {
-                    code: `
-            switch (type) {
-              case "a":
-                break;
-              default:
-                break;
-            }
-          `,
-                    errors: [{ messageId: "missingAssertNever" }],
-                },
-            ],
-        });
-    });
-});