eslint-config-agent 1.3.6 → 1.3.8

package/CHANGELOG.md

@@ -4,6 +4,28 @@ All notable changes to this project will be documented in this file. See [Conven
 
 
 
+## [1.3.8](https://github.com/tupe12334/eslint-config/compare/v1.3.7...v1.3.8) (2025-09-27)
+
+### Features
+
+* add eslint-plugin-default for enhanced linting rules; configure default plugin strict settings ([70d3ac6](https://github.com/tupe12334/eslint-config/commit/70d3ac6f7ab6666eb7652840cf8eb6b68de74df7))
+* add test files configuration and refactor index.js to import shared rules ([28d3799](https://github.com/tupe12334/eslint-config/commit/28d3799edbc7d4726027adbbdc597746579e516e))
+* add TODO.md with function export and translation function guidelines ([94bfd21](https://github.com/tupe12334/eslint-config/commit/94bfd2167b29480cf7f3d2bb11120756239b0ee7))
+* implement no-nullish-coalescing rule with examples and tests ([838196f](https://github.com/tupe12334/eslint-config/commit/838196f500125d09e92f9fa0d7f6bb65280a2f3b))
+* implement no-record-literal-types rule to enforce type safety; add examples and tests ([3308ccb](https://github.com/tupe12334/eslint-config/commit/3308ccb59ba4e9d05725e2f810453df454c63ba5))
+* include configs directory in package.json files list ([3bce2b4](https://github.com/tupe12334/eslint-config/commit/3bce2b41440a7330816603918bf74dfea3900b2b))
+* refine .npmignore and package.json to better manage test and example files in rules directory ([9ed652b](https://github.com/tupe12334/eslint-config/commit/9ed652bae2410faa4428dab583e3494abf9cc5bc))
+* update .npmignore to include examples and test files in rules directory ([d1bb23d](https://github.com/tupe12334/eslint-config/commit/d1bb23df50c623ada2bc2805f7638f9398f2ba64))
+* update no-empty-exports tests for consistency and clarity; enhance test-runner to scan for spec files ([25a7d26](https://github.com/tupe12334/eslint-config/commit/25a7d2660cc992e9031e0577197bfb57faf2c570))
+
+## [1.3.7](https://github.com/tupe12334/eslint-config/compare/v1.3.6...v1.3.7) (2025-09-24)
+
+### Features
+
+* add error handling rules from eslint-plugin-error ([082701f](https://github.com/tupe12334/eslint-config/commit/082701f8abe8730ed12afe206caf8f10825b4d1c))
+* add jsx-classname-required rule to enforce className attribute on HTML elements in JSX ([3603865](https://github.com/tupe12334/eslint-config/commit/360386582f05e63d6ba1625548725b9216c4f4d6))
+* integrate error handling rules from eslint-plugin-error and update configuration ([7c8b846](https://github.com/tupe12334/eslint-config/commit/7c8b84694550b92ac2a265591fac736d5ddb1180))
+
 ## [1.3.6](https://github.com/tupe12334/eslint-config/compare/v1.3.5...v1.3.6) (2025-09-21)
 
 ### Bug Fixes

package/configs/test-files.js

@@ -0,0 +1,248 @@
+import allRules from "../rules/index.js";
+import { noRecordLiteralTypesConfigs } from "../rules/no-record-literal-types/index.js";
+
+// Shared rules for both JS and TS files
+const sharedRules = {
+  ...allRules.pluginRules,
+  "object-curly-newline": "off",
+  "no-shadow": "off",
+  "comma-dangle": "off",
+  "function-paren-newline": "off",
+  quotes: "off",
+  "no-unused-vars": "off",
+  "@typescript-eslint/no-unused-vars": "off",
+  "max-lines-per-function": allRules.maxFunctionLinesWarning,
+  "max-lines": allRules.maxFileLinesWarning,
+  semi: "off",
+  complexity: "off",
+  "no-trailing-spaces": allRules.noTrailingSpacesConfig,
+  "operator-linebreak": "off",
+  "implicit-arrow-linebreak": "off",
+  "arrow-body-style": "off",
+  "no-continue": "off",
+  // Additional built-in error handling rules
+  "prefer-promise-reject-errors": "error",
+};
+
+// Shared no-restricted-syntax rules for both JS and TS
+const sharedRestrictedSyntax = [
+  {
+    selector: "MemberExpression[optional=true]",
+    message: "Optional chaining is not allowed.",
+  },
+  {
+    selector: "CallExpression[optional=true]",
+    message: "Optional chaining is not allowed.",
+  },
+  allRules.noNullishCoalescingConfig,
+  {
+    selector:
+      "ExportNamedDeclaration[exportKind=type]:not([source]):has(ExportSpecifier)",
+    message:
+      "Type-only exports are not allowed. Use regular export or re-export with 'from' clause.",
+  },
+  {
+    selector: "ExportSpecifier[local.name=default][exported.name!=default]",
+    message:
+      "Re-exporting default as named export is not allowed. Use explicit export declaration instead.",
+  },
+  {
+    selector:
+      "Program:has(ImportDeclaration) ExportNamedDeclaration:has(VariableDeclaration > VariableDeclarator[init.type=Identifier]):not(:has(ClassDeclaration))",
+    message:
+      "Exporting imported variables is not allowed. Use direct re-export with 'from' clause or define new values.",
+  },
+  {
+    selector: "SwitchStatement > SwitchCase > ReturnStatement[argument=null]",
+    message:
+      "Switch case functions must provide an explicit return value. Default return values are not allowed.",
+  },
+  {
+    selector:
+      "SwitchStatement > SwitchCase > BlockStatement > ReturnStatement[argument=null]",
+    message:
+      "Switch case functions must provide an explicit return value. Default return values are not allowed.",
+  },
+  {
+    selector: "SwitchStatement > SwitchCase[test=null]",
+    message:
+      "Default cases are not allowed in switch statements. Handle all possible cases explicitly.",
+  },
+  {
+    selector:
+      "ExportNamedDeclaration[source.value=/^[a-z]/]:not([source.value=/^@/])",
+    message:
+      "Exporting from external libraries is not allowed. Only re-export from relative paths or scoped packages.",
+  },
+  allRules.noProcessEnvPropertiesConfig,
+  allRules.noExportSpecifiersConfig,
+  ...allRules.noDefaultClassExportRules,
+];
+
+// TypeScript-specific no-restricted-syntax rules
+const tsOnlyRestrictedSyntax = [
+  ...noRecordLiteralTypesConfigs,
+  {
+    selector:
+      "TSTypeAnnotation > TSUnionType:not(PropertyDefinition > .typeAnnotation > .typeAnnotation):not(TSPropertySignature > .typeAnnotation > .typeAnnotation)",
+    message: "Use a named type declaration instead of inline union types.",
+  },
+  {
+    selector:
+      "TSPropertySignature > TSTypeAnnotation > TSUnionType:has(TSLiteralType)",
+    message:
+      "Interface properties with literal unions should use a named type declaration.",
+  },
+  {
+    selector:
+      "PropertyDefinition > TSTypeAnnotation > TSUnionType:has(TSLiteralType)",
+    message:
+      "Class properties with literal unions should use a named type declaration.",
+  },
+  allRules.noTypeAssertionsConfig,
+  allRules.noClassPropertyDefaultsConfig,
+  {
+    selector: "TSAsExpression:has(> TSIndexedAccessType > TSTypeQuery)",
+    message:
+      'Type assertions with indexed access types like "as (typeof X)[number]" are not allowed. Use a named type instead.',
+  },
+  {
+    selector:
+      "SwitchStatement > SwitchCase ArrowFunctionExpression:not([returnType])",
+    message:
+      "Switch case arrow functions must have explicit return type annotations.",
+  },
+  {
+    selector:
+      "SwitchStatement > SwitchCase FunctionExpression:not([returnType])",
+    message:
+      "Switch case function expressions must have explicit return type annotations.",
+  },
+  {
+    selector:
+      "SwitchStatement > SwitchCase > BlockStatement ArrowFunctionExpression:not([returnType])",
+    message:
+      "Switch case arrow functions must have explicit return type annotations.",
+  },
+  {
+    selector:
+      "SwitchStatement > SwitchCase > BlockStatement FunctionExpression:not([returnType])",
+    message:
+      "Switch case function expressions must have explicit return type annotations.",
+  },
+  {
+    selector: "FunctionDeclaration:has(SwitchStatement):not([returnType])",
+    message:
+      "Functions containing switch statements must have explicit return type annotations.",
+  },
+  {
+    selector: "ArrowFunctionExpression:has(SwitchStatement):not([returnType])",
+    message:
+      "Arrow functions containing switch statements must have explicit return type annotations.",
+  },
+  {
+    selector: "FunctionExpression:has(SwitchStatement):not([returnType])",
+    message:
+      "Function expressions containing switch statements must have explicit return type annotations.",
+  },
+];
+
+// Test and spec files configuration
+export const testFilesConfig = [
+  // Disable function and file size limits for test and spec files
+  {
+    files: [
+      "**/*.test.{js,jsx,ts,tsx}",
+      "**/*.spec.{js,jsx,ts,tsx}",
+      "**/test/**/*.{js,jsx,ts,tsx}",
+      "**/tests/**/*.{js,jsx,ts,tsx}",
+      "**/__tests__/**/*.{js,jsx,ts,tsx}",
+    ],
+    ignores: [
+      "**/long-function-test.tsx", // Exception: this file tests the max-lines rule itself
+      "**/test/export/**", // Export tests should follow strict export rules
+      "**/test/required-exports/**", // Required export tests should follow strict export rules
+    ],
+    rules: {
+      "max-lines-per-function": "off",
+      "max-lines": "off", // Ignore file length limits in test and spec files
+      // Allow multiple exports in test files for testing import/export patterns
+      "no-restricted-syntax": [
+        "warn",
+        ...sharedRestrictedSyntax.filter(
+          (rule) =>
+            rule.selector !==
+              "ExportNamedDeclaration[specifiers.length>1]:not([source])" &&
+            rule.selector !==
+              "Program:has(ExportNamedDeclaration:not([source]) ~ ExportNamedDeclaration:not([source]))" &&
+            rule.selector !==
+              "ExportNamedDeclaration:not([source]):not(:has(VariableDeclaration)):not(:has(FunctionDeclaration)):not(:has(ClassDeclaration)):not(:has(TSInterfaceDeclaration)):not(:has(TSTypeAliasDeclaration)):not(:has(TSEnumDeclaration))"
+        ),
+        ...tsOnlyRestrictedSyntax,
+      ],
+    },
+  },
+
+  // Test files that should have ERROR level rules but exclude export specifier rule
+  {
+    files: [
+      "**/test/type-assertions/**",
+      "**/test/test-optional.ts",
+      "**/test/test-js-optional.js",
+      "**/test/test-record-literals.ts",
+      "**/test/no-env-access-test.ts",
+      "**/test/import-export-rules.ts",
+    ],
+    rules: {
+      "max-lines-per-function": "off",
+      "no-restricted-syntax": [
+        "error", // Error level for these test files
+        ...sharedRestrictedSyntax.filter(
+          (rule) =>
+            rule.selector !==
+            "ExportNamedDeclaration:not([source]):not(:has(VariableDeclaration)):not(:has(FunctionDeclaration)):not(:has(ClassDeclaration)):not(:has(TSInterfaceDeclaration)):not(:has(TSTypeAliasDeclaration)):not(:has(TSEnumDeclaration))"
+        ),
+        ...tsOnlyRestrictedSyntax,
+      ],
+    },
+  },
+
+  // Test files configuration with mixed severity levels
+  {
+    files: [
+      "**/test/invalid.tsx", // Special handling for the main test file
+      "**/test/single-export-valid.ts", // Allow export specifiers for import/group-exports testing
+      "**/test/typescript-rules.ts", // Allow export specifiers for typescript rules testing
+      "**/test/type-assertions/indexed-access-valid.ts", // Allow export specifiers for type assertions testing
+    ],
+    rules: {
+      "max-lines-per-function": "off",
+      "no-restricted-syntax": [
+        "warn", // Base level for most rules
+        ...sharedRestrictedSyntax.filter(
+          (rule) =>
+            rule.selector !==
+              "ExportNamedDeclaration[specifiers.length>1]:not([source])" &&
+            rule.selector !==
+              "Program:has(ExportNamedDeclaration:not([source]) ~ ExportNamedDeclaration:not([source]))" &&
+            rule.selector !==
+              "ExportNamedDeclaration:not([source]):not(:has(VariableDeclaration)):not(:has(FunctionDeclaration)):not(:has(ClassDeclaration)):not(:has(TSInterfaceDeclaration)):not(:has(TSTypeAliasDeclaration)):not(:has(TSEnumDeclaration))"
+        ),
+        ...tsOnlyRestrictedSyntax,
+      ],
+    },
+  },
+
+  // Disable file length rules for configuration and spec files
+  {
+    files: [
+      "index.js", // Main configuration file
+      "**/rules/**/*.spec.js", // Spec files in rules directory
+      "**/scripts/**/*.js", // Script files
+    ],
+    rules: {
+      "max-lines": "off",
+      "max-lines-per-function": "off",
+    },
+  },
+];