Hi, team,

I noticed you are using eslint-plugin-header on ESLint 8.

I forked _@tony.ganchev/eslint-plugin-header_ mid-2024 to add support
for ESLint 9 and hoped it would be a temporary measure but since the
original has not been updated for five years I decided to continue
improving the new plugin and have been doing so for the last two years.

Specific improvements include:
- full JSON schema for validating the configuration.
- fixed multiple bugs with the behavior of the plugin on Windows.
- many other bug-fixes.
- improved autofixing and error-reporting behavior.
- added support for leading pragma comments before the header such as
  `@jest-environment` or `eslint-env`. In the case of the current
  review, this allowed me to remove the `eslint-disable` directive from
  one source and reintroduce validation for its header.

I switched the configuration to use the new config format for v3.2.x
as the leading comments feature is not supported by the array-based
format inherited from _eslint-plugin-header_.

I've sent PR proposals to other projects in the _cloudscape-design_
space:
- cloudscape-design/code-view#122
- cloudscape-design/collection-hooks#136
- cloudscape-design/component-toolkit#200
- cloudscape-design/components#4308
- cloudscape-design/demos#243
- cloudscape-design/jest-preset#58
- cloudscape-design/theming-core#148
- cloudscape-design/test-utils#114

Looking forward to your feedback.

Author: tonyganchev

.eslintrc

@@ -15,7 +15,7 @@
       "jsx": true
     }
   },
-  "plugins": ["unicorn", "react-hooks", "no-unsanitized", "header", "import", "simple-import-sort"],
+  "plugins": ["unicorn", "react-hooks", "no-unsanitized", "@tony.ganchev/header", "import", "simple-import-sort"],
   "rules": {
     "@typescript-eslint/no-unused-vars": "error",
     "@typescript-eslint/consistent-type-definitions": ["error", "interface"],
@@ -52,10 +52,22 @@
     "eqeqeq": "error",
     "no-return-await": "error",
     "require-await": "error",
-    "header/header": [
+    "@tony.ganchev/header/header": [
       "error",
-      "line",
-      [" Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.", " SPDX-License-Identifier: Apache-2.0"]
+      {
+        "header": {
+          "commentType": "line",
+          "lines": [" Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.", " SPDX-License-Identifier: Apache-2.0"]
+        },
+        "leadingComments": {
+          "comments": [
+            {
+              "commentType": "block",
+              "lines": [" eslint-env node "]
+            }
+          ]
+        }
+      }
     ],
     "no-restricted-imports": [
       "error",

package-lock.json

@@ -70,6 +70,7 @@
     "@juggle/resize-observer": "^3.4.0",
     "@testing-library/jest-dom": "^5.16.5",
     "@testing-library/react": "^13.4.0",
+    "@tony.ganchev/eslint-plugin-header": "^3.3.1",
     "@types/jest-image-snapshot": "^6.1.0",
     "@types/lodash": "^4.14.191",
     "@types/node": "^18.19.4",
@@ -83,7 +84,6 @@
     "deep-freeze-es6": "^1.4.1",
     "eslint": "^8.57.0",
     "eslint-config-prettier": "^9.1.0",
-    "eslint-plugin-header": "^3.1.1",
     "eslint-plugin-import": "^2.26.0",
     "eslint-plugin-no-unsanitized": "^4.0.2",
     "eslint-plugin-prettier": "^5.1.3",

package.json

@@ -1,5 +1,4 @@
 /* eslint-env node */
-/* eslint-disable header/header */
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 import * as fs from "node:fs";