@zthun/janitor-eslint-config 19.0.0

package/LICENSE.md

@@ -0,0 +1,20 @@
+# MIT License
+
+Copyright (c) 2024 Anthony Bonta
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of
+this software and associated documentation files (the "Software"), to deal in
+the Software without restriction, including without limitation the rights to
+use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
+the Software, and to permit persons to whom the Software is furnished to do so,
+subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
+FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
+COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
+IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

package/README.md

@@ -0,0 +1,28 @@
+# ESLint Config
+
+This is the shared eslint configuration for zthun scoped projects.
+
+## Getting Started
+
+```sh
+# NPM
+npm install @zthun/janitor-eslint-config --save-dev
+# Yarn
+yarn add @zthun/janitor-eslint-config --dev
+```
+
+Make sure to install typescript and @typescript-eslint/eslint-plugin with all
+peer dependencies.
+
+## Usage
+
+In the .eslintrc file, add the following
+
+```json
+{
+  "extends": ["@zthun/janitor-eslint-config"]
+}
+```
+
+That's it. You now have the recommended rules for zthun scoped projects with
+typescript.

package/dist/configs/environments.d.mts

@@ -0,0 +1,5 @@
+import { Linter } from 'eslint';
+export declare const environments: {
+    browser: Linter.Config<Linter.RulesRecord>[];
+    node: Linter.Config<Linter.RulesRecord>[];
+};

package/dist/configs/imports.d.mts

@@ -0,0 +1,2 @@
+import { Linter } from 'eslint';
+export declare const imports: Linter.Config[];

package/dist/configs/javascript.d.mts

@@ -0,0 +1,2 @@
+import { Linter } from 'eslint';
+export declare const javascript: Linter.Config[];

package/dist/configs/prettier.d.mts

@@ -0,0 +1 @@
+export declare const prettier: import("eslint").Linter.Config<import("eslint").Linter.RulesRecord>[];

package/dist/configs/react.d.mts

@@ -0,0 +1,2 @@
+import { Linter } from 'eslint';
+export declare const react: Linter.Config[];

package/dist/configs/recommended.d.mts

@@ -0,0 +1 @@
+export declare const recommended: (import("eslint").Linter.Config<import("eslint").Linter.RulesRecord> | import("@typescript-eslint/utils/ts-eslint").FlatConfig.Config)[];

package/dist/configs/typescript.d.mts

@@ -0,0 +1 @@
+export declare const typescript: import("@typescript-eslint/utils/ts-eslint").FlatConfig.Config[];

package/dist/index.cjs

@@ -0,0 +1,196 @@
+'use strict';
+
+Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
+
+const globals = require('globals');
+const _import = require('eslint-plugin-import');
+const js = require('@eslint/js');
+const pretty = require('eslint-plugin-prettier/recommended');
+const _react = require('eslint-plugin-react');
+const ts = require('typescript-eslint');
+
+function _define_property(obj, key, value) {
+    if (key in obj) {
+        Object.defineProperty(obj, key, {
+            value: value,
+            enumerable: true,
+            configurable: true,
+            writable: true
+        });
+    } else {
+        obj[key] = value;
+    }
+    return obj;
+}
+function _object_spread(target) {
+    for(var i = 1; i < arguments.length; i++){
+        var source = arguments[i] != null ? arguments[i] : {};
+        var ownKeys = Object.keys(source);
+        if (typeof Object.getOwnPropertySymbols === "function") {
+            ownKeys = ownKeys.concat(Object.getOwnPropertySymbols(source).filter(function(sym) {
+                return Object.getOwnPropertyDescriptor(source, sym).enumerable;
+            }));
+        }
+        ownKeys.forEach(function(key) {
+            _define_property(target, key, source[key]);
+        });
+    }
+    return target;
+}
+function environment(environment) {
+    return {
+        languageOptions: {
+            globals: _object_spread({}, environment)
+        }
+    };
+}
+const environments = {
+    browser: [
+        environment(globals.browser)
+    ],
+    node: [
+        environment(globals.node),
+        environment(globals.nodeBuiltin)
+    ]
+};
+
+const imports = [
+    _import.flatConfigs.recommended,
+    {
+        rules: {
+            // This lint error is the main reason to use import as we want to make
+            // sure we've installed our dependencies correctly.
+            "import/no-extraneous-dependencies": "error",
+            // These are straight up broken with Typescript when you need to work with
+            // mts files that must have the file extension present.  These can be fixed
+            // using resolves, but it's such a pain and it's just not worth the hassle
+            // for basic linting support.  The no-extraneous-dependencies is really
+            // the recommended config we want, so these being forced off are fine.
+            "import/named": "off",
+            "import/no-unresolved": "off"
+        }
+    }
+];
+
+const javascript = [
+    js.configs.recommended,
+    {
+        rules: {
+            // We want to support == null so we get a good check for undefined
+            // or null
+            eqeqeq: [
+                "error",
+                "smart"
+            ],
+            // Would be fine, but there's a bug in this where you have a function with
+            // access arguments.  Those constructors are often empty - so we want to let
+            // a part of this one through.
+            "no-empty-function": "off"
+        }
+    }
+];
+
+const prettier = [
+    pretty
+];
+
+const react = [
+    {
+        files: [
+            "**/*.{js,cjs,mjs,ts,mts,jsx,tsx}"
+        ],
+        plugins: {
+            react: _react
+        },
+        languageOptions: {
+            parserOptions: {
+                ecmaFeatures: {
+                    jsx: true
+                }
+            }
+        },
+        settings: {
+            react: {
+                version: "detect"
+            }
+        },
+        rules: {
+            // This one is not needed any longer as TypeScript jsx option
+            // should be set to react-jsx.  You will need to turn this on
+            // manually if it is set to the legacy react setting. See
+            // https://legacy.reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html#whats-different-in-the-new-transform
+            "react/react-in-jsx-scope": "off"
+        }
+    }
+];
+
+const typescript = [
+    ...ts.configs.recommended,
+    {
+        rules: {
+            // There are times when any and legacy namespaces makes sense.
+            // If you use any kind of decorator library, you will almost
+            // be required to use any at some point.  Forcing a non use of them
+            // means you're spending a bunch of time play type gymnastics and to
+            // hell with that.
+            "@typescript-eslint/no-explicit-any": "off",
+            "@typescript-eslint/no-namespace": "off",
+            // Would be fine, but there's a bug in this where you have a function with
+            // access arguments.  Those constructors are often empty - so we want to let
+            // a part of this one through.
+            "no-empty-function": "off",
+            "@typescript-eslint/no-empty-function": [
+                "error",
+                {
+                    allow: [
+                        "constructors"
+                    ]
+                }
+            ],
+            // A lot of 3rd party libraries still don't support esm
+            // and trying to force this right now just isn't feasible.
+            "@typescript-eslint/no-var-requires": "off",
+            "@typescript-eslint/no-require-imports": "off",
+            // I want aliasing support.
+            "@typescript-eslint/no-empty-object-type": "off",
+            "@typescript-eslint/no-empty-interface": "off",
+            // You will need unsafe declaration merging if you are doing anything
+            // with decorators as what often happens is that TypeScript cannot infer
+            // the output type of a decorator. So this has to be on to deal with
+            // TypeScripts shortcoming in this department.  See
+            // https://github.com/microsoft/TypeScript/issues/4881 for more information.
+            "@typescript-eslint/no-unsafe-declaration-merging": "off",
+            // I can technically agree with this, but where this comes in
+            // handy is unit testing and I value that, so I want support
+            // to make the assumption that I know what I'm doing when
+            // I make a non-null assertion.
+            "@typescript-eslint/no-non-null-assertion": "off",
+            // This is actually fine, but this is broken in typescript eslint 8.14.x.
+            // See https://github.com/typescript-eslint/typescript-eslint/issues/10353
+            // for the bug.
+            "no-unused-expressions": "off",
+            "@typescript-eslint/no-unused-expressions": [
+                "error",
+                {
+                    allowShortCircuit: false
+                }
+            ]
+        }
+    }
+];
+
+const recommended = [
+    ...javascript,
+    ...typescript,
+    ...imports,
+    ...prettier
+];
+
+exports.environments = environments;
+exports.imports = imports;
+exports.javascript = javascript;
+exports.prettier = prettier;
+exports.react = react;
+exports.recommended = recommended;
+exports.typescript = typescript;
+//# sourceMappingURL=index.cjs.map

package/dist/index.cjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.cjs","sources":["../src/configs/environments.mts","../src/configs/imports.mts","../src/configs/javascript.mts","../src/configs/prettier.mts","../src/configs/react.mts","../src/configs/typescript.mts","../src/configs/recommended.mts"],"sourcesContent":["import { Linter } from \"eslint\";\nimport globals from \"globals\";\n\nfunction environment(environment: object): Linter.Config {\n  return {\n    languageOptions: {\n      globals: {\n        ...environment,\n      },\n    },\n  };\n}\n\nexport const environments = {\n  browser: [environment(globals.browser)],\n  node: [environment(globals.node), environment(globals.nodeBuiltin)],\n};\n","import { Linter } from \"eslint\";\nimport _import from \"eslint-plugin-import\";\n\nexport const imports: Linter.Config[] = [\n  _import.flatConfigs.recommended,\n  {\n    rules: {\n      // This lint error is the main reason to use import as we want to make\n      // sure we've installed our dependencies correctly.\n      \"import/no-extraneous-dependencies\": \"error\",\n\n      // These are straight up broken with Typescript when you need to work with\n      // mts files that must have the file extension present.  These can be fixed\n      // using resolves, but it's such a pain and it's just not worth the hassle\n      // for basic linting support.  The no-extraneous-dependencies is really\n      // the recommended config we want, so these being forced off are fine.\n      \"import/named\": \"off\",\n      \"import/no-unresolved\": \"off\",\n    },\n  } satisfies Linter.Config,\n];\n","import js from \"@eslint/js\";\nimport { Linter } from \"eslint\";\n\nexport const javascript: Linter.Config[] = [\n  js.configs.recommended,\n  {\n    rules: {\n      // We want to support == null so we get a good check for undefined\n      // or null\n      eqeqeq: [\"error\", \"smart\"],\n      // Would be fine, but there's a bug in this where you have a function with\n      // access arguments.  Those constructors are often empty - so we want to let\n      // a part of this one through.\n      \"no-empty-function\": \"off\",\n    },\n  },\n];\n","import pretty from \"eslint-plugin-prettier/recommended\";\n\nexport const prettier = [pretty];\n","import { ESLint, Linter } from \"eslint\";\nimport _react from \"eslint-plugin-react\";\n\nexport const react: Linter.Config[] = [\n  {\n    files: [\"**/*.{js,cjs,mjs,ts,mts,jsx,tsx}\"],\n    plugins: {\n      react: _react as ESLint.Plugin,\n    },\n    languageOptions: {\n      parserOptions: {\n        ecmaFeatures: {\n          jsx: true,\n        },\n      },\n    },\n    settings: {\n      react: {\n        version: \"detect\",\n      },\n    },\n    rules: {\n      // This one is not needed any longer as TypeScript jsx option\n      // should be set to react-jsx.  You will need to turn this on\n      // manually if it is set to the legacy react setting. See\n      // https://legacy.reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html#whats-different-in-the-new-transform\n      \"react/react-in-jsx-scope\": \"off\",\n    },\n  },\n];\n","import { Linter } from \"eslint\";\nimport ts from \"typescript-eslint\";\n\nexport const typescript = [\n  ...ts.configs.recommended,\n  {\n    rules: {\n      // There are times when any and legacy namespaces makes sense.\n      // If you use any kind of decorator library, you will almost\n      // be required to use any at some point.  Forcing a non use of them\n      // means you're spending a bunch of time play type gymnastics and to\n      // hell with that.\n      \"@typescript-eslint/no-explicit-any\": \"off\",\n      \"@typescript-eslint/no-namespace\": \"off\",\n\n      // Would be fine, but there's a bug in this where you have a function with\n      // access arguments.  Those constructors are often empty - so we want to let\n      // a part of this one through.\n      \"no-empty-function\": \"off\",\n      \"@typescript-eslint/no-empty-function\": [\n        \"error\",\n        { allow: [\"constructors\"] },\n      ],\n\n      // A lot of 3rd party libraries still don't support esm\n      // and trying to force this right now just isn't feasible.\n      \"@typescript-eslint/no-var-requires\": \"off\",\n      \"@typescript-eslint/no-require-imports\": \"off\",\n\n      // I want aliasing support.\n      \"@typescript-eslint/no-empty-object-type\": \"off\",\n      \"@typescript-eslint/no-empty-interface\": \"off\",\n\n      // You will need unsafe declaration merging if you are doing anything\n      // with decorators as what often happens is that TypeScript cannot infer\n      // the output type of a decorator. So this has to be on to deal with\n      // TypeScripts shortcoming in this department.  See\n      // https://github.com/microsoft/TypeScript/issues/4881 for more information.\n      \"@typescript-eslint/no-unsafe-declaration-merging\": \"off\",\n\n      // I can technically agree with this, but where this comes in\n      // handy is unit testing and I value that, so I want support\n      // to make the assumption that I know what I'm doing when\n      // I make a non-null assertion.\n      \"@typescript-eslint/no-non-null-assertion\": \"off\",\n\n      // This is actually fine, but this is broken in typescript eslint 8.14.x.\n      // See https://github.com/typescript-eslint/typescript-eslint/issues/10353\n      // for the bug.\n      \"no-unused-expressions\": \"off\",\n      \"@typescript-eslint/no-unused-expressions\": [\n        \"error\",\n        {\n          allowShortCircuit: false,\n        },\n      ],\n    },\n  } satisfies Linter.Config,\n];\n","import { imports } from \"./imports.mjs\";\nimport { javascript } from \"./javascript.mjs\";\nimport { prettier } from \"./prettier.mjs\";\nimport { typescript } from \"./typescript.mjs\";\n\nexport const recommended = [\n  ...javascript,\n  ...typescript,\n  ...imports,\n  ...prettier,\n];\n"],"names":["environment","languageOptions","globals","environments","browser","node","nodeBuiltin","imports","_import","flatConfigs","recommended","rules","javascript","js","configs","eqeqeq","prettier","pretty","react","files","plugins","_react","parserOptions","ecmaFeatures","jsx","settings","version","typescript","ts","allow","allowShortCircuit"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAGA,SAASA,YAAYA,WAAmB,EAAA;IACtC,OAAO;QACLC,eAAiB,EAAA;AACfC,YAAAA,OAAAA,EAAS,cACJF,CAAAA,EAAAA,EAAAA,WAAAA;AAEP;AACF,KAAA;AACF;MAEaG,YAAe,GAAA;IAC1BC,OAAS,EAAA;AAACJ,QAAAA,WAAAA,CAAYE,QAAQE,OAAO;AAAE,KAAA;IACvCC,IAAM,EAAA;AAACL,QAAAA,WAAAA,CAAYE,QAAQG,IAAI,CAAA;AAAGL,QAAAA,WAAAA,CAAYE,QAAQI,WAAW;AAAE;AACrE;;MCbaC,OAA2B,GAAA;IACtCC,OAAQC,CAAAA,WAAW,CAACC,WAAW;AAC/B,IAAA;QACEC,KAAO,EAAA;;;YAGL,mCAAqC,EAAA,OAAA;;;;;;YAOrC,cAAgB,EAAA,KAAA;YAChB,sBAAwB,EAAA;AAC1B;AACF;;;MChBWC,UAA8B,GAAA;IACzCC,EAAGC,CAAAA,OAAO,CAACJ,WAAW;AACtB,IAAA;QACEC,KAAO,EAAA;;;YAGLI,MAAQ,EAAA;AAAC,gBAAA,OAAA;AAAS,gBAAA;AAAQ,aAAA;;;;YAI1B,mBAAqB,EAAA;AACvB;AACF;;;MCbWC,QAAW,GAAA;AAACC,IAAAA;;;MCCZC,KAAyB,GAAA;AACpC,IAAA;QACEC,KAAO,EAAA;AAAC,YAAA;AAAmC,SAAA;QAC3CC,OAAS,EAAA;YACPF,KAAOG,EAAAA;AACT,SAAA;QACApB,eAAiB,EAAA;YACfqB,aAAe,EAAA;gBACbC,YAAc,EAAA;oBACZC,GAAK,EAAA;AACP;AACF;AACF,SAAA;QACAC,QAAU,EAAA;YACRP,KAAO,EAAA;gBACLQ,OAAS,EAAA;AACX;AACF,SAAA;QACAf,KAAO,EAAA;;;;;YAKL,0BAA4B,EAAA;AAC9B;AACF;;;MCzBWgB,UAAa,GAAA;OACrBC,EAAGd,CAAAA,OAAO,CAACJ,WAAW;AACzB,IAAA;QACEC,KAAO,EAAA;;;;;;YAML,oCAAsC,EAAA,KAAA;YACtC,iCAAmC,EAAA,KAAA;;;;YAKnC,mBAAqB,EAAA,KAAA;YACrB,sCAAwC,EAAA;AACtC,gBAAA,OAAA;AACA,gBAAA;oBAAEkB,KAAO,EAAA;AAAC,wBAAA;AAAe;AAAC;AAC3B,aAAA;;;YAID,oCAAsC,EAAA,KAAA;YACtC,uCAAyC,EAAA,KAAA;;YAGzC,yCAA2C,EAAA,KAAA;YAC3C,uCAAyC,EAAA,KAAA;;;;;;YAOzC,kDAAoD,EAAA,KAAA;;;;;YAMpD,0CAA4C,EAAA,KAAA;;;;YAK5C,uBAAyB,EAAA,KAAA;YACzB,0CAA4C,EAAA;AAC1C,gBAAA,OAAA;AACA,gBAAA;oBACEC,iBAAmB,EAAA;AACrB;AACD;AACH;AACF;;;MCpDWpB,WAAc,GAAA;AACtBE,IAAAA,GAAAA,UAAAA;AACAe,IAAAA,GAAAA,UAAAA;AACApB,IAAAA,GAAAA,OAAAA;AACAS,IAAAA,GAAAA;;;;;;;;;;;"}

package/dist/index.d.ts

@@ -0,0 +1,7 @@
+export * from './configs/environments.mjs';
+export * from './configs/imports.mjs';
+export * from './configs/javascript.mjs';
+export * from './configs/prettier.mjs';
+export * from './configs/react.mjs';
+export * from './configs/recommended.mjs';
+export * from './configs/typescript.mjs';

package/dist/index.js

@@ -0,0 +1,186 @@
+import globals from 'globals';
+import _import from 'eslint-plugin-import';
+import js from '@eslint/js';
+import pretty from 'eslint-plugin-prettier/recommended';
+import _react from 'eslint-plugin-react';
+import ts from 'typescript-eslint';
+
+function _define_property(obj, key, value) {
+    if (key in obj) {
+        Object.defineProperty(obj, key, {
+            value: value,
+            enumerable: true,
+            configurable: true,
+            writable: true
+        });
+    } else {
+        obj[key] = value;
+    }
+    return obj;
+}
+function _object_spread(target) {
+    for(var i = 1; i < arguments.length; i++){
+        var source = arguments[i] != null ? arguments[i] : {};
+        var ownKeys = Object.keys(source);
+        if (typeof Object.getOwnPropertySymbols === "function") {
+            ownKeys = ownKeys.concat(Object.getOwnPropertySymbols(source).filter(function(sym) {
+                return Object.getOwnPropertyDescriptor(source, sym).enumerable;
+            }));
+        }
+        ownKeys.forEach(function(key) {
+            _define_property(target, key, source[key]);
+        });
+    }
+    return target;
+}
+function environment(environment) {
+    return {
+        languageOptions: {
+            globals: _object_spread({}, environment)
+        }
+    };
+}
+const environments = {
+    browser: [
+        environment(globals.browser)
+    ],
+    node: [
+        environment(globals.node),
+        environment(globals.nodeBuiltin)
+    ]
+};
+
+const imports = [
+    _import.flatConfigs.recommended,
+    {
+        rules: {
+            // This lint error is the main reason to use import as we want to make
+            // sure we've installed our dependencies correctly.
+            "import/no-extraneous-dependencies": "error",
+            // These are straight up broken with Typescript when you need to work with
+            // mts files that must have the file extension present.  These can be fixed
+            // using resolves, but it's such a pain and it's just not worth the hassle
+            // for basic linting support.  The no-extraneous-dependencies is really
+            // the recommended config we want, so these being forced off are fine.
+            "import/named": "off",
+            "import/no-unresolved": "off"
+        }
+    }
+];
+
+const javascript = [
+    js.configs.recommended,
+    {
+        rules: {
+            // We want to support == null so we get a good check for undefined
+            // or null
+            eqeqeq: [
+                "error",
+                "smart"
+            ],
+            // Would be fine, but there's a bug in this where you have a function with
+            // access arguments.  Those constructors are often empty - so we want to let
+            // a part of this one through.
+            "no-empty-function": "off"
+        }
+    }
+];
+
+const prettier = [
+    pretty
+];
+
+const react = [
+    {
+        files: [
+            "**/*.{js,cjs,mjs,ts,mts,jsx,tsx}"
+        ],
+        plugins: {
+            react: _react
+        },
+        languageOptions: {
+            parserOptions: {
+                ecmaFeatures: {
+                    jsx: true
+                }
+            }
+        },
+        settings: {
+            react: {
+                version: "detect"
+            }
+        },
+        rules: {
+            // This one is not needed any longer as TypeScript jsx option
+            // should be set to react-jsx.  You will need to turn this on
+            // manually if it is set to the legacy react setting. See
+            // https://legacy.reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html#whats-different-in-the-new-transform
+            "react/react-in-jsx-scope": "off"
+        }
+    }
+];
+
+const typescript = [
+    ...ts.configs.recommended,
+    {
+        rules: {
+            // There are times when any and legacy namespaces makes sense.
+            // If you use any kind of decorator library, you will almost
+            // be required to use any at some point.  Forcing a non use of them
+            // means you're spending a bunch of time play type gymnastics and to
+            // hell with that.
+            "@typescript-eslint/no-explicit-any": "off",
+            "@typescript-eslint/no-namespace": "off",
+            // Would be fine, but there's a bug in this where you have a function with
+            // access arguments.  Those constructors are often empty - so we want to let
+            // a part of this one through.
+            "no-empty-function": "off",
+            "@typescript-eslint/no-empty-function": [
+                "error",
+                {
+                    allow: [
+                        "constructors"
+                    ]
+                }
+            ],
+            // A lot of 3rd party libraries still don't support esm
+            // and trying to force this right now just isn't feasible.
+            "@typescript-eslint/no-var-requires": "off",
+            "@typescript-eslint/no-require-imports": "off",
+            // I want aliasing support.
+            "@typescript-eslint/no-empty-object-type": "off",
+            "@typescript-eslint/no-empty-interface": "off",
+            // You will need unsafe declaration merging if you are doing anything
+            // with decorators as what often happens is that TypeScript cannot infer
+            // the output type of a decorator. So this has to be on to deal with
+            // TypeScripts shortcoming in this department.  See
+            // https://github.com/microsoft/TypeScript/issues/4881 for more information.
+            "@typescript-eslint/no-unsafe-declaration-merging": "off",
+            // I can technically agree with this, but where this comes in
+            // handy is unit testing and I value that, so I want support
+            // to make the assumption that I know what I'm doing when
+            // I make a non-null assertion.
+            "@typescript-eslint/no-non-null-assertion": "off",
+            // This is actually fine, but this is broken in typescript eslint 8.14.x.
+            // See https://github.com/typescript-eslint/typescript-eslint/issues/10353
+            // for the bug.
+            "no-unused-expressions": "off",
+            "@typescript-eslint/no-unused-expressions": [
+                "error",
+                {
+                    allowShortCircuit: false
+                }
+            ]
+        }
+    }
+];
+
+const recommended = [
+    ...javascript,
+    ...typescript,
+    ...imports,
+    ...prettier
+];
+
+export { environments, imports, javascript, prettier, react, recommended, typescript };
+//# sourceMappingURL=index.js.map

package/dist/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sources":["../src/configs/environments.mts","../src/configs/imports.mts","../src/configs/javascript.mts","../src/configs/prettier.mts","../src/configs/react.mts","../src/configs/typescript.mts","../src/configs/recommended.mts"],"sourcesContent":["import { Linter } from \"eslint\";\nimport globals from \"globals\";\n\nfunction environment(environment: object): Linter.Config {\n  return {\n    languageOptions: {\n      globals: {\n        ...environment,\n      },\n    },\n  };\n}\n\nexport const environments = {\n  browser: [environment(globals.browser)],\n  node: [environment(globals.node), environment(globals.nodeBuiltin)],\n};\n","import { Linter } from \"eslint\";\nimport _import from \"eslint-plugin-import\";\n\nexport const imports: Linter.Config[] = [\n  _import.flatConfigs.recommended,\n  {\n    rules: {\n      // This lint error is the main reason to use import as we want to make\n      // sure we've installed our dependencies correctly.\n      \"import/no-extraneous-dependencies\": \"error\",\n\n      // These are straight up broken with Typescript when you need to work with\n      // mts files that must have the file extension present.  These can be fixed\n      // using resolves, but it's such a pain and it's just not worth the hassle\n      // for basic linting support.  The no-extraneous-dependencies is really\n      // the recommended config we want, so these being forced off are fine.\n      \"import/named\": \"off\",\n      \"import/no-unresolved\": \"off\",\n    },\n  } satisfies Linter.Config,\n];\n","import js from \"@eslint/js\";\nimport { Linter } from \"eslint\";\n\nexport const javascript: Linter.Config[] = [\n  js.configs.recommended,\n  {\n    rules: {\n      // We want to support == null so we get a good check for undefined\n      // or null\n      eqeqeq: [\"error\", \"smart\"],\n      // Would be fine, but there's a bug in this where you have a function with\n      // access arguments.  Those constructors are often empty - so we want to let\n      // a part of this one through.\n      \"no-empty-function\": \"off\",\n    },\n  },\n];\n","import pretty from \"eslint-plugin-prettier/recommended\";\n\nexport const prettier = [pretty];\n","import { ESLint, Linter } from \"eslint\";\nimport _react from \"eslint-plugin-react\";\n\nexport const react: Linter.Config[] = [\n  {\n    files: [\"**/*.{js,cjs,mjs,ts,mts,jsx,tsx}\"],\n    plugins: {\n      react: _react as ESLint.Plugin,\n    },\n    languageOptions: {\n      parserOptions: {\n        ecmaFeatures: {\n          jsx: true,\n        },\n      },\n    },\n    settings: {\n      react: {\n        version: \"detect\",\n      },\n    },\n    rules: {\n      // This one is not needed any longer as TypeScript jsx option\n      // should be set to react-jsx.  You will need to turn this on\n      // manually if it is set to the legacy react setting. See\n      // https://legacy.reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html#whats-different-in-the-new-transform\n      \"react/react-in-jsx-scope\": \"off\",\n    },\n  },\n];\n","import { Linter } from \"eslint\";\nimport ts from \"typescript-eslint\";\n\nexport const typescript = [\n  ...ts.configs.recommended,\n  {\n    rules: {\n      // There are times when any and legacy namespaces makes sense.\n      // If you use any kind of decorator library, you will almost\n      // be required to use any at some point.  Forcing a non use of them\n      // means you're spending a bunch of time play type gymnastics and to\n      // hell with that.\n      \"@typescript-eslint/no-explicit-any\": \"off\",\n      \"@typescript-eslint/no-namespace\": \"off\",\n\n      // Would be fine, but there's a bug in this where you have a function with\n      // access arguments.  Those constructors are often empty - so we want to let\n      // a part of this one through.\n      \"no-empty-function\": \"off\",\n      \"@typescript-eslint/no-empty-function\": [\n        \"error\",\n        { allow: [\"constructors\"] },\n      ],\n\n      // A lot of 3rd party libraries still don't support esm\n      // and trying to force this right now just isn't feasible.\n      \"@typescript-eslint/no-var-requires\": \"off\",\n      \"@typescript-eslint/no-require-imports\": \"off\",\n\n      // I want aliasing support.\n      \"@typescript-eslint/no-empty-object-type\": \"off\",\n      \"@typescript-eslint/no-empty-interface\": \"off\",\n\n      // You will need unsafe declaration merging if you are doing anything\n      // with decorators as what often happens is that TypeScript cannot infer\n      // the output type of a decorator. So this has to be on to deal with\n      // TypeScripts shortcoming in this department.  See\n      // https://github.com/microsoft/TypeScript/issues/4881 for more information.\n      \"@typescript-eslint/no-unsafe-declaration-merging\": \"off\",\n\n      // I can technically agree with this, but where this comes in\n      // handy is unit testing and I value that, so I want support\n      // to make the assumption that I know what I'm doing when\n      // I make a non-null assertion.\n      \"@typescript-eslint/no-non-null-assertion\": \"off\",\n\n      // This is actually fine, but this is broken in typescript eslint 8.14.x.\n      // See https://github.com/typescript-eslint/typescript-eslint/issues/10353\n      // for the bug.\n      \"no-unused-expressions\": \"off\",\n      \"@typescript-eslint/no-unused-expressions\": [\n        \"error\",\n        {\n          allowShortCircuit: false,\n        },\n      ],\n    },\n  } satisfies Linter.Config,\n];\n","import { imports } from \"./imports.mjs\";\nimport { javascript } from \"./javascript.mjs\";\nimport { prettier } from \"./prettier.mjs\";\nimport { typescript } from \"./typescript.mjs\";\n\nexport const recommended = [\n  ...javascript,\n  ...typescript,\n  ...imports,\n  ...prettier,\n];\n"],"names":["environment","languageOptions","globals","environments","browser","node","nodeBuiltin","imports","_import","flatConfigs","recommended","rules","javascript","js","configs","eqeqeq","prettier","pretty","react","files","plugins","_react","parserOptions","ecmaFeatures","jsx","settings","version","typescript","ts","allow","allowShortCircuit"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAGA,SAASA,YAAYA,WAAmB,EAAA;IACtC,OAAO;QACLC,eAAiB,EAAA;AACfC,YAAAA,OAAAA,EAAS,cACJF,CAAAA,EAAAA,EAAAA,WAAAA;AAEP;AACF,KAAA;AACF;MAEaG,YAAe,GAAA;IAC1BC,OAAS,EAAA;AAACJ,QAAAA,WAAAA,CAAYE,QAAQE,OAAO;AAAE,KAAA;IACvCC,IAAM,EAAA;AAACL,QAAAA,WAAAA,CAAYE,QAAQG,IAAI,CAAA;AAAGL,QAAAA,WAAAA,CAAYE,QAAQI,WAAW;AAAE;AACrE;;MCbaC,OAA2B,GAAA;IACtCC,OAAQC,CAAAA,WAAW,CAACC,WAAW;AAC/B,IAAA;QACEC,KAAO,EAAA;;;YAGL,mCAAqC,EAAA,OAAA;;;;;;YAOrC,cAAgB,EAAA,KAAA;YAChB,sBAAwB,EAAA;AAC1B;AACF;;;MChBWC,UAA8B,GAAA;IACzCC,EAAGC,CAAAA,OAAO,CAACJ,WAAW;AACtB,IAAA;QACEC,KAAO,EAAA;;;YAGLI,MAAQ,EAAA;AAAC,gBAAA,OAAA;AAAS,gBAAA;AAAQ,aAAA;;;;YAI1B,mBAAqB,EAAA;AACvB;AACF;;;MCbWC,QAAW,GAAA;AAACC,IAAAA;;;MCCZC,KAAyB,GAAA;AACpC,IAAA;QACEC,KAAO,EAAA;AAAC,YAAA;AAAmC,SAAA;QAC3CC,OAAS,EAAA;YACPF,KAAOG,EAAAA;AACT,SAAA;QACApB,eAAiB,EAAA;YACfqB,aAAe,EAAA;gBACbC,YAAc,EAAA;oBACZC,GAAK,EAAA;AACP;AACF;AACF,SAAA;QACAC,QAAU,EAAA;YACRP,KAAO,EAAA;gBACLQ,OAAS,EAAA;AACX;AACF,SAAA;QACAf,KAAO,EAAA;;;;;YAKL,0BAA4B,EAAA;AAC9B;AACF;;;MCzBWgB,UAAa,GAAA;OACrBC,EAAGd,CAAAA,OAAO,CAACJ,WAAW;AACzB,IAAA;QACEC,KAAO,EAAA;;;;;;YAML,oCAAsC,EAAA,KAAA;YACtC,iCAAmC,EAAA,KAAA;;;;YAKnC,mBAAqB,EAAA,KAAA;YACrB,sCAAwC,EAAA;AACtC,gBAAA,OAAA;AACA,gBAAA;oBAAEkB,KAAO,EAAA;AAAC,wBAAA;AAAe;AAAC;AAC3B,aAAA;;;YAID,oCAAsC,EAAA,KAAA;YACtC,uCAAyC,EAAA,KAAA;;YAGzC,yCAA2C,EAAA,KAAA;YAC3C,uCAAyC,EAAA,KAAA;;;;;;YAOzC,kDAAoD,EAAA,KAAA;;;;;YAMpD,0CAA4C,EAAA,KAAA;;;;YAK5C,uBAAyB,EAAA,KAAA;YACzB,0CAA4C,EAAA;AAC1C,gBAAA,OAAA;AACA,gBAAA;oBACEC,iBAAmB,EAAA;AACrB;AACD;AACH;AACF;;;MCpDWpB,WAAc,GAAA;AACtBE,IAAAA,GAAAA,UAAAA;AACAe,IAAAA,GAAAA,UAAAA;AACApB,IAAAA,GAAAA,OAAAA;AACAS,IAAAA,GAAAA;;;;;"}

package/package.json

@@ -0,0 +1,62 @@
+{
+  "name": "@zthun/janitor-eslint-config",
+  "version": "19.0.0",
+  "description": "A shared configuration for eslint for @zthun scoped projects.",
+  "keywords": [
+    "eslint",
+    "eslint-config"
+  ],
+  "type": "module",
+  "types": "./dist/index.d.ts",
+  "main": "./dist/index.cjs",
+  "module": "./dist/index.js",
+  "exports": {
+    "types": "./dist/index.d.ts",
+    "require": "./dist/index.cjs",
+    "import": "./dist/index.js"
+  },
+  "scripts": {
+    "build": "vite build"
+  },
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/zthun/janitor",
+    "directory": "packages/janitor-eslint-config"
+  },
+  "files": [
+    "dist"
+  ],
+  "dependencies": {
+    "@eslint/js": "^9.28.0",
+    "eslint-config-prettier": "^10.1.5",
+    "eslint-plugin-import": "^2.31.0",
+    "eslint-plugin-prettier": "^5.4.1",
+    "eslint-plugin-react": "^7.37.5",
+    "globals": "^16.2.0",
+    "typescript-eslint": "^8.33.0"
+  },
+  "devDependencies": {
+    "@zthun/janitor-build-config": "^19.0.0",
+    "eslint": "^9.28.0",
+    "prettier": "^3.5.3",
+    "typescript": "~5.8.3",
+    "vite": "^6.3.5"
+  },
+  "peerDependencies": {
+    "eslint": "^9.0.0",
+    "prettier": "^3.0.0",
+    "react": ">=18.0.0 <=20.0.0",
+    "typescript": "^5.0.0"
+  },
+  "peerDependenciesMeta": {
+    "react": {
+      "optional": true
+    }
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "author": "Anthony Bonta",
+  "gitHead": "8277288093dff6023427448ccf903d83912865b5"
+}