eslint-config-agent 3.0.0 → 3.0.2

package/CHANGELOG.md

@@ -4,6 +4,23 @@ All notable changes to this project will be documented in this file. See [Conven
 
 
 
+## [3.0.2](https://github.com/tupe12334/eslint-config-agent/compare/v3.0.0...v3.0.2) (2026-04-26)
+
+### Features
+
+* **jsdoc:** upgrade to recommended-typescript-error config ([2ecc02e](https://github.com/tupe12334/eslint-config-agent/commit/2ecc02e9031d42263c01f926a1adfed2424355b6))
+* **plugins:** add eslint-plugin-jsdoc recommended config ([ff9fb97](https://github.com/tupe12334/eslint-config-agent/commit/ff9fb97dbaa88db3486b08904a31d9002bdec59d))
+* **settings:** add additional WebFetch domains to permissions ([3a94e18](https://github.com/tupe12334/eslint-config-agent/commit/3a94e18b5128c6901b72e88218e12928d965dbc3))
+* **settings:** add GitHub API permission to settings ([529c09d](https://github.com/tupe12334/eslint-config-agent/commit/529c09da004e2e369e3f639edbe6401d11f944b0))
+
+### Bug Fixes
+
+* **ci:** publish to npm after workflow_dispatch releases ([4267e8a](https://github.com/tupe12334/eslint-config-agent/commit/4267e8a3be526c8700ca843c9c6f421e5a6608ea))
+* **deps:** downgrade cspell to v9.8.0 for Node 20 compatibility ([ececdb2](https://github.com/tupe12334/eslint-config-agent/commit/ececdb297843f3317a431aa3075a148659e7659e))
+* **jsdoc:** add missing parameters and fix tag formatting ([07bafcc](https://github.com/tupe12334/eslint-config-agent/commit/07bafcc6cb307e3646a29305ad70c848836ba766))
+* **jsdoc:** escape inline [@typescript](https://github.com/typescript) tag in spec file comments ([f4918e5](https://github.com/tupe12334/eslint-config-agent/commit/f4918e5b3f9f3192884ddf99aa05328f98a4a5a6))
+* **knip:** update configuration for knip v6 compatibility ([75967d6](https://github.com/tupe12334/eslint-config-agent/commit/75967d6a224910ca7c3f78b70e86e5d77074f3a8))
+
 ## 3.0.0 (2026-04-02)
 
 * feat: add centy configuration and update claude agent/command definitions ([6f656c5](https://github.com/tupe12334/eslint-config-agent/commit/6f656c5))

package/configs/config-files.js

@@ -20,6 +20,9 @@ export const configFilesConfig = [
       'max-lines-per-function': 'off', // Allow long functions in config files
       'max-lines': 'off', // Allow long files in config files
       'ddd/require-spec-file': 'off', // Config files don't need spec files
+      'jsdoc/require-jsdoc': 'off', // Config files don't need JSDoc
+      'jsdoc/require-param': 'off',
+      'jsdoc/require-returns': 'off',
     },
   },
 ]

package/configs/examples.js

@@ -83,6 +83,10 @@ export const examplesConfig = [
       // Relax file/function length limits for concise examples
       'max-lines-per-function': 'off',
       'max-lines': 'off',
+      // Examples don't need JSDoc
+      'jsdoc/require-jsdoc': 'off',
+      'jsdoc/require-param': 'off',
+      'jsdoc/require-returns': 'off',
     },
   },
 ]

package/configs/test-files.js

@@ -24,8 +24,6 @@ const sharedRules = {
   'no-continue': 'off',
   // Additional built-in error handling rules
   'prefer-promise-reject-errors': 'error',
-  // Optional chaining restriction
-  'no-optional-chaining/no-optional-chaining': 'error',
 }
 
 // Shared no-restricted-syntax rules for both JS and TS
@@ -202,6 +200,26 @@ export const testFilesConfig = [
       'error/no-generic-error': 'off',
       'error/require-custom-error': 'off',
       'error/no-throw-literal': 'off',
+      // Disable JSDoc rules for non-test script files
+      'jsdoc/require-jsdoc': 'off',
+      'jsdoc/require-param': 'off',
+      'jsdoc/require-returns': 'off',
+    },
+  },
+
+  // Disable JSDoc rules for test 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}',
+    ],
+    rules: {
+      'jsdoc/require-jsdoc': 'off',
+      'jsdoc/require-param': 'off',
+      'jsdoc/require-returns': 'off',
     },
   },
 

package/index.js

@@ -2,6 +2,7 @@ import js from '@eslint/js'
 import earlyReturn from 'eslint-plugin-early-return'
 import switchCase from 'eslint-plugin-switch-case'
 import jsxClassname from 'eslint-plugin-jsx-classname'
+import jsdoc from 'eslint-plugin-jsdoc'
 import reactHooks from 'eslint-plugin-react-hooks'
 import { defineConfig } from 'eslint/config'
 import tseslint from 'typescript-eslint'
@@ -41,8 +42,6 @@ const sharedRules = {
   'no-continue': 'off',
   // Additional built-in error handling rules
   'prefer-promise-reject-errors': 'error',
-  // Optional chaining restriction
-  'no-optional-chaining/no-optional-chaining': 'error',
 }
 
 // Shared no-restricted-syntax rules for both JS and TS
@@ -98,15 +97,27 @@ const tsOnlyRestrictedSyntax = [
 ]
 
 const config = defineConfig([
-  // Global ignores for non-JS/TS files
+  // Global ignores for non-JS/TS files and build outputs
   {
-    ignores: ['**/*.json', '**/*.md', '**/*.yaml', '**/*.yml'],
+    ignores: [
+      '**/*.json',
+      '**/*.md',
+      '**/*.yaml',
+      '**/*.yml',
+      'dist/**',
+      'coverage/**',
+    ],
   },
   // Global plugin definitions
   {
     plugins,
   },
-  reactHooks.configs['recommended-latest'],
+  {
+    rules: {
+      'react-hooks/rules-of-hooks': 'error',
+      'react-hooks/exhaustive-deps': 'warn',
+    },
+  },
   js.configs.recommended,
   ...tseslint.configs.strictTypeChecked,
   ...tseslint.configs.stylisticTypeChecked,
@@ -123,6 +134,7 @@ const config = defineConfig([
     ...tseslint.configs.disableTypeChecked,
   },
   earlyReturn.configs.recommended,
+  jsdoc.configs['flat/recommended-typescript-error'],
   { plugins: { 'switch-case': switchCase }, ...switchCase.configs.recommended },
 
   // Base plugin strict configs (error, default, guard-clauses)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-agent",
-  "version": "3.0.0",
+  "version": "3.0.2",
   "description": "ESLint configuration package with TypeScript support",
   "main": "index.js",
   "type": "module",
@@ -73,16 +73,17 @@
   },
   "packageManager": "pnpm@10.18.2",
   "devDependencies": {
-    "@commitlint/cli": "^20.1.0",
-    "@commitlint/config-conventional": "^20.0.0",
+    "@commitlint/cli": "^20.5.2",
+    "@commitlint/config-conventional": "^20.5.0",
     "@eslint/eslintrc": "^3.3.1",
     "@eslint/js": "^9.37.0",
-    "@release-it/conventional-changelog": "^10.0.1",
+    "@release-it/conventional-changelog": "^11.0.0",
     "@typescript-eslint/eslint-plugin": "^8.46.1",
     "@typescript-eslint/parser": "^8.46.1",
-    "@typescript-eslint/rule-tester": "^8.46.1",
-    "cspell": "^9.3.1",
+    "@typescript-eslint/rule-tester": "^8.59.0",
+    "cspell": "^9.8.0",
     "eslint": "^9.37.0",
+    "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-class-export": "^1.0.1",
     "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-n": "^17.23.1",
@@ -92,36 +93,36 @@
     "eslint-plugin-security": "^3.0.1",
     "eslint-plugin-storybook": "^9.1.10",
     "husky": "^9.1.7",
-    "knip": "^5.70.2",
-    "lint-staged": "^16.2.6",
-    "prettier": "^3.6.2",
-    "release-it": "^19.0.5",
+    "knip": "^6.7.0",
+    "lint-staged": "^16.4.0",
+    "prettier": "^3.8.3",
+    "release-it": "^20.0.1",
     "typescript-eslint": "^8.46.1"
   },
   "dependencies": {
-    "@eslint/eslintrc": "^3.3.1",
-    "@eslint/js": "^9.34.0",
-    "@typescript-eslint/eslint-plugin": "^8.40.0",
-    "@typescript-eslint/parser": "^8.40.0",
-    "eslint": "^9.34.0",
+    "@eslint/eslintrc": "^3.3.5",
+    "@eslint/js": "^9.39.4",
+    "@typescript-eslint/eslint-plugin": "^8.59.0",
+    "@typescript-eslint/parser": "^8.59.0",
+    "eslint": "^9.39.4",
     "eslint-plugin-class-export": "^1.0.1",
     "eslint-plugin-ddd": "^0.5.2",
     "eslint-plugin-default": "^1.1.0",
-    "eslint-plugin-early-return": "^1.0.0",
+    "eslint-plugin-early-return": "^1.1.0",
     "eslint-plugin-error": "^1.3.0",
     "eslint-plugin-import": "^2.32.0",
+    "eslint-plugin-jsdoc": "^62.9.0",
     "eslint-plugin-jsx-classname": "^1.0.1",
-    "eslint-plugin-n": "^17.21.3",
-    "eslint-plugin-no-optional-chaining": "^1.0.0",
+    "eslint-plugin-n": "^17.24.0",
     "eslint-plugin-preact": "^0.1.0",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^5.2.0",
     "eslint-plugin-required-exports": "^0.2.0",
-    "eslint-plugin-security": "^3.0.1",
+    "eslint-plugin-security": "^4.0.0",
     "eslint-plugin-single-export": "^1.1.2",
-    "eslint-plugin-storybook": "^9.1.5",
-    "eslint-plugin-switch-case": "^3.0.1",
-    "globals": "^16.4.0",
-    "typescript-eslint": "^8.40.0"
+    "eslint-plugin-storybook": "^10.3.5",
+    "eslint-plugin-switch-case": "^4.0.0",
+    "globals": "^17.5.0",
+    "typescript-eslint": "^8.59.0"
   }
 }

package/plugins/index.js

@@ -6,6 +6,7 @@
  */
 
 import reactPlugin from 'eslint-plugin-react'
+import reactHooksPlugin from 'eslint-plugin-react-hooks'
 import importPlugin from 'eslint-plugin-import'
 import securityPlugin from 'eslint-plugin-security'
 import nPlugin from 'eslint-plugin-n'
@@ -15,7 +16,6 @@ import requiredExportsPlugin from 'eslint-plugin-required-exports'
 import storybookPlugin from 'eslint-plugin-storybook'
 import errorPlugin from 'eslint-plugin-error'
 import defaultPlugin from 'eslint-plugin-default'
-import noOptionalChainingPlugin from 'eslint-plugin-no-optional-chaining'
 import dddPlugin from 'eslint-plugin-ddd'
 import preactPlugin from 'eslint-plugin-preact'
 import { noDefaultClassExportRule } from '../rules/no-default-class-export/index.js'
@@ -23,6 +23,7 @@ import { noDefaultClassExportRule } from '../rules/no-default-class-export/index
 // Centralized plugin configuration
 export const plugins = {
   react: reactPlugin,
+  'react-hooks': reactHooksPlugin,
   import: importPlugin,
   security: securityPlugin,
   n: nPlugin,
@@ -32,7 +33,6 @@ export const plugins = {
   storybook: storybookPlugin,
   error: errorPlugin,
   default: defaultPlugin,
-  'no-optional-chaining': noOptionalChainingPlugin,
   ddd: dddPlugin,
   preact: preactPlugin,
   custom: {

package/rules/error-only-exports/index.js

@@ -7,6 +7,8 @@
 
 /**
  * Check if a class declaration extends Error or another Error class
+ * @param node - The AST node to check
+ * @returns True if the node extends Error
  */
 function extendsError(node) {
   if (!node.superClass) {
@@ -29,9 +31,11 @@ function extendsError(node) {
 
 /**
  * Check if the file only contains error class exports
+ * @param context - The ESLint rule context
+ * @returns True if file only exports error classes
  */
 function isErrorOnlyFile(context) {
-  const sourceCode = context.sourceCode || context.getSourceCode()
+  const sourceCode = context.sourceCode
   const ast = sourceCode.ast
 
   let hasExports = false
@@ -124,7 +128,7 @@ export const errorOnlyExportsRule = {
         }
 
         // Check if the file already has the eslint-disable comment
-        const sourceCode = context.sourceCode || context.getSourceCode()
+        const sourceCode = context.sourceCode
         const comments = sourceCode.getAllComments()
 
         const hasDisableComment = comments.some(

package/rules/max-file-lines/index.js

@@ -5,7 +5,6 @@
  * smaller, more maintainable files and better separation of concerns.
  *
  * Warnings at >70 lines, errors at >100 lines
- *
  * @see https://eslint.org/docs/latest/rules/max-lines
  */
 

package/rules/max-function-lines/index.js

@@ -5,7 +5,6 @@
  * smaller, more maintainable functions.
  *
  * Warnings at >50 lines, errors at >70 lines
- *
  * @see https://eslint.org/docs/latest/rules/max-lines-per-function
  */
 

package/rules/no-default-class-export/index.js

@@ -31,7 +31,7 @@ export const noDefaultClassExportRule = {
     schema: [],
   },
   create(context) {
-    const sourceCode = context.getSourceCode()
+    const sourceCode = context.sourceCode
 
     return {
       ExportDefaultDeclaration(node) {

package/rules/no-inline-union-types/index.js

@@ -11,7 +11,6 @@
  * - ✅ type Status = 'active' | 'inactive'; function foo(status: Status) {}
  * - ✅ type Role = 'admin' | 'user'; interface User { role: Role; }
  * - ✅ type Mode = 'dev' | 'prod'; class Config { mode: Mode; }
- *
  * @see https://eslint.org/docs/latest/rules/no-restricted-syntax
  */
 

package/rules/no-process-env-properties/index.js

@@ -9,7 +9,6 @@
  * - ❌ process.env.PORT
  * - ✅ validate(process.env)
  * - ✅ const { NODE_ENV } = process.env
- *
  * @see https://eslint.org/docs/latest/rules/no-restricted-syntax
  */
 

package/rules/no-record-literal-types/index.js

@@ -16,7 +16,6 @@
  * - ✅ interface UserInfo { name: string; age: string; }
  * - ✅ type Status = { active: boolean; }
  * - ✅ Record<string, unknown> (generic keys are allowed)
- *
  * @see https://eslint.org/docs/latest/rules/no-restricted-syntax
  * @see https://www.typescriptlang.org/docs/handbook/utility-types.html#recordkeys-type
  */

package/rules/no-trailing-spaces/index.js

@@ -2,7 +2,6 @@
  * Rule configuration for no-trailing-spaces
  *
  * This rule disallows trailing whitespace (spaces and tabs) at the end of lines.
- *
  * @see https://eslint.org/docs/latest/rules/no-trailing-spaces
  */
 

package/rules/no-trivial-type-aliases/index.js

@@ -54,7 +54,6 @@
  * // Type references with generics
  * type ListOfUsers = Array<User>;
  * ```
- *
  * @see https://eslint.org/docs/latest/rules/no-restricted-syntax
  * @see ./examples/invalid.js for more invalid examples
  * @see ./examples/valid.js for more valid examples

package/rules/no-type-assertions/index.js

@@ -10,7 +10,6 @@
  * - ❌ obj as { name: string }
  * - ✅ { mode: 'prod' } as const
  * - ✅ ['a', 'b'] as const
- *
  * @see https://eslint.org/docs/latest/rules/no-restricted-syntax
  */
 

package/rules/nullish-coalescing/index.js

@@ -13,7 +13,6 @@
  * - ❌ const name = user?.name ?? 'Anonymous'
  * - ✅ const result = value !== null && value !== undefined ? value : 'default'
  * - ✅ const name = user?.name !== undefined ? user.name : 'Anonymous'
- *
  * @see https://eslint.org/docs/latest/rules/no-restricted-syntax
  * @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Nullish_coalescing
  */

package/rules/plugin/class-export/index.js

@@ -3,7 +3,6 @@
  *
  * This plugin enforces consistent class export patterns to improve code organization
  * and import consistency across the codebase.
- *
  * @see https://www.npmjs.com/package/eslint-plugin-class-export
  */
 

package/rules/plugin/n/no-process-env/index.js

@@ -3,7 +3,6 @@
  *
  * This rule disallows the use of process.env to encourage centralized configuration
  * management instead of scattered environment variable access throughout the codebase.
- *
  * @see https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-process-env.md
  */