@newsteam/eslint-config 1.2.15 → 1.2.17

package/dist/rules/import-x.js

@@ -0,0 +1,420 @@
+import eslintImportX from "eslint-plugin-import-x";
+export const importXPluginConfig = {
+    plugins: {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-type-assertion -- don't have a choice
+        "import-x": eslintImportX,
+    },
+    rules: {
+        /*
+         * In both Flow and TypeScript you can mark an import as a type-only
+         * import by adding a "kind" marker to the import. Both languages
+         * support two positions for marker.
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/consistent-type-specifier-style.md
+         */
+        "import-x/consistent-type-specifier-style": [
+            "warn",
+            "prefer-top-level",
+        ],
+        /*
+         * Ensure a default export is present, given a default import.
+         *
+         * BROKEN
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/default.md
+         */
+        "import-x/default": "off",
+        /*
+         * Enforce a leading comment with the webpackChunkName for dynamic imports
+         *
+         * This is turned off for now because it doesn't work correctly. Says
+         * that valid comments are invalid. Likely a bug in the plugin.
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/dynamic-import-chunkname.md
+         */
+        "import-x/dynamic-import-chunkname": "off",
+        /*
+         * Report any invalid exports, i.e. re-export of the same name
+         *
+         * BROKEN
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/export.md
+         */
+        "import-x/export": "off",
+        /*
+         * Ensure all exports appear after other statements
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/exports-last.md
+         */
+        "import-x/exports-last": "warn",
+        /*
+         * Ensure consistent use of file extension within the import path
+         *
+         * Turning this off since the future of node >= 16 requires extensions
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/extensions.md
+         */
+        "import-x/extensions": "off",
+        /*
+         * Ensure all imports appear before other statements
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/first.md
+         */
+        "import-x/first": "warn",
+        /*
+         * Prefer named exports to be grouped together in a single export declaration
+         *
+         * This is off for now because when you create anything that uses an interface
+         * in Typescript, you have to export that interface otherwise you'll get an error
+         * to the tune of:
+         *
+         * 'Default export of the module has or is using private name'
+         *
+         * As of the writing of this comment, you can't export your interface in the
+         * group, because the typescript parser sees that as an undefined variable.
+         * This means that the only option available is to export the interface and
+         * members separately instead of in a group.
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/group-exports.md
+         */
+        "import-x/group-exports": "off",
+        /*
+         * Limit the maximum number of dependencies a module can have
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/max-dependencies.md
+         */
+        "import-x/max-dependencies": [
+            "warn",
+            {
+                max: 25,
+            },
+        ],
+        /*
+         * Ensure named imports correspond to a named export in the remote file.
+         *
+         * Typescript takes care of this for us already, and @react-email/components bugs this out
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/named.md
+         */
+        "import-x/named": "off",
+        /*
+         * Ensure imported namespaces contain dereferenced properties as they are dereferenced.
+         *
+         * BROKEN
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/namespace.md
+         */
+        "import-x/namespace": "off",
+        /*
+         * Enforce a newline after import statements
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/newline-after-import.md
+         */
+        "import-x/newline-after-import": [
+            "warn",
+            {
+                count: 2,
+            },
+        ],
+        /*
+         * Forbid import of modules using absolute paths
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-absolute-path.md
+         */
+        "import-x/no-absolute-path": "warn",
+        /*
+         * Report AMD require and define calls.
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-amd.md
+         */
+        "import-x/no-amd": "warn",
+        /*
+         * Forbid anonymous values as default exports
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-anonymous-default-export.md
+         */
+        "import-x/no-anonymous-default-export": [
+            "warn",
+            {
+                allowObject: true,
+            },
+        ],
+        /*
+         * Report CommonJS require calls and module.exports or exports.*.
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-commonjs.md
+         */
+        "import-x/no-commonjs": "warn",
+        /*
+         * Forbid a module from importing a module with a dependency path back to itself
+         *
+         * This rule doesn't work correctly, off for now
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-cycle.md
+         */
+        "import-x/no-cycle": "off",
+        /*
+         * Forbid default exports
+         *
+         * Broken in eslint 10
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-default-export.md
+         */
+        "import-x/no-default-export": "off",
+        /*
+         * Report imported names marked with @deprecated documentation tag
+         *
+         * BROKEN
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-deprecated.md
+         */
+        "import-x/no-deprecated": "off",
+        /*
+         * Report repeated import of the same module in multiple places
+         *
+         * BROKEN
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-duplicates.md
+         */
+        "import-x/no-duplicates": "warn",
+        /*
+         * Forbid require() calls with expressions
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-dynamic-require.md
+         */
+        "import-x/no-dynamic-require": "warn",
+        /*
+         * Reports the use of empty named import blocks.
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-empty-named-blocks.md
+         */
+        "import-x/no-empty-named-blocks": "warn",
+        /*
+         * Forbid the use of extraneous packages
+         *
+         * Taken care of by node/no-extraneous-import because it works better
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-extraneous-dependencies.md
+         */
+        "import-x/no-extraneous-dependencies": "off",
+        /*
+         * Forbid imports with CommonJS exports
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-import-module-exports.md
+         */
+        "import-x/no-import-module-exports": "warn",
+        /*
+         * Prevent importing the submodules of other modules
+         *
+         * Off for now because organizing internal modules in parent modules makes sense to me
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-internal-modules.md
+         */
+        "import-x/no-internal-modules": "off",
+        /*
+         * Forbid the use of mutable exports with var or let.
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-mutable-exports.md
+         */
+        "import-x/no-mutable-exports": "warn",
+        /*
+         * Report use of exported name as identifier of default export
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-named-as-default.md
+         */
+        "import-x/no-named-as-default": "off",
+        /*
+         * Report use of exported name as property of default export
+         *
+         * Think this is bombing eslint with "heap limit Allocation failed"
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-named-as-default-member.md
+         */
+        "import-x/no-named-as-default-member": "off",
+        /*
+         * Forbid named default exports
+         *
+         * This is off because named defaults are sometimes required in the new node >= 16 module system
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-named-default.md
+         */
+        "import-x/no-named-default": "off",
+        /*
+         * Forbid named exports
+         *
+         * Off for now because named exports are fine
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-named-export.md
+         */
+        "import-x/no-named-export": "off",
+        /*
+         * Report namespace imports
+         *
+         * Off for now because namespace imports are chilled
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-namespace.md
+         */
+        "import-x/no-namespace": "off",
+        /*
+         * No Node.js builtin modules.
+         *
+         * Off for now since we're assuming that most code is nodejs code
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-nodejs-modules.md
+         */
+        "import-x/no-nodejs-modules": "off",
+        /*
+         * Prevent importing packages through relative paths
+         *
+         * Off for now because we use relative imports
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-relative-packages.md
+         */
+        "import-x/no-relative-packages": "off",
+        /*
+         * Forbid importing modules from parent directories
+         *
+         * Off for now because doing something like a utils collection would become impossible
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-relative-parent-imports.md
+         */
+        "import-x/no-relative-parent-imports": "off",
+        /*
+         * Forbid importing a default export by a different name.
+         *
+         * BROKEN
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-rename-default.md
+         */
+        "import-x/no-rename-default": "off",
+        /*
+         * Restrict which files can be imported in a given folder
+         *
+         * BROKEN
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-restricted-paths.md
+         */
+        "import-x/no-restricted-paths": "off",
+        /*
+         * Forbid a module from importing itself
+         *
+         * BROKEN
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-self-import.md
+         */
+        "import-x/no-self-import": "off",
+        /*
+         * Forbid unassigned imports
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-unassigned-import.md
+         */
+        "import-x/no-unassigned-import": [
+            "warn",
+            {
+                allow: [
+                    "**/*.css",
+                    "**/*.scss",
+                    "@babel/polyfill",
+                    "@babel/register",
+                    "@total-typescript/ts-reset",
+                    "core-js/stable",
+                    "dotenv/config",
+                    "jasmine",
+                    "reflect-metadata",
+                    "regenerator-runtime/runtime",
+                    "server-only",
+                ],
+            },
+        ],
+        /*
+         * Ensure imports point to a file/module that can be resolved.
+         *
+         * This rule has no way to resolve aliases of any kind - off for now
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-unresolved.md
+         */
+        "import-x/no-unresolved": "off",
+        /*
+         * Forbid modules without any export, and exports not imported by any modules
+         *
+         * Off because it doesn't currently support dynamic imports
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-unused-modules.md
+         */
+        "import-x/no-unused-modules": "off",
+        /*
+         * Prevent unnecessary path segments in import and require statements
+         *
+         * BROKEN
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-useless-path-segments.md
+         */
+        "import-x/no-useless-path-segments": "off",
+        /*
+         * Forbid webpack loader syntax in imports
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-webpack-loader-syntax.md
+         */
+        "import-x/no-webpack-loader-syntax": "warn",
+        /*
+         * Enforce a convention in module import order
+         *
+         * Leaving this to eslint-plugin-perfectionist
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/order.md
+         */
+        "import-x/order": [
+            "off",
+            {
+                groups: [
+                    "builtin",
+                    "external",
+                    "internal",
+                    "parent",
+                    "sibling",
+                    "index",
+                    "object",
+                    "type",
+                ],
+                "newlines-between": "always",
+                pathGroups: [
+                    {
+                        group: "internal",
+                        pattern: "@src/**",
+                    },
+                ],
+                pathGroupsExcludedImportTypes: ["builtin"],
+            },
+        ],
+        /*
+         * Prefer a default export if module exports a single name
+         *
+         * Off for now because we don't prefer this
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/prefer-default-export.md
+         */
+        "import-x/prefer-default-export": "off",
+        /*
+         * Enforce using namespace imports for specific modules, like react/react-dom, etc.
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/prefer-namespace-import.md
+         */
+        "import-x/prefer-namespace-import": [
+            "warn",
+            {
+                patterns: [
+                    "react",
+                    "react-dom",
+                ],
+            },
+        ],
+        /*
+         * Report potentially ambiguous parse goal (script vs. module)
+         *
+         * Broken in eslint 10
+         *
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/unambiguous.md
+         */
+        "import-x/unambiguous": "off",
+    },
+};

package/dist/{rules.d.ts → rules/index.d.ts}

@@ -1,3 +1,3 @@
 import type { Linter } from "eslint";
 export declare const rules: Linter.Config[];
-//# sourceMappingURL=rules.d.ts.map
+//# sourceMappingURL=index.d.ts.map

package/dist/rules/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/rules/index.ts"],"names":[],"mappings":"AAmDA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAgCrC,eAAO,MAAM,KAAK,EAAE,MAAM,CAAC,MAAM,EAgmDhC,CAAC"}

package/dist/{rules.js → rules/index.js}

@@ -15,7 +15,56 @@
  * https://eslint.org/docs/latest/rules/#best-practices
  */
 import confusingBrowserGlobals from "confusing-browser-globals";
-import { MAX_FUNCTION_PARAMETERS, MAXIMUM_CYCLOMATIC_COMPLEXITY, MAXIMUM_FILE_LINE_COUNT, NO_MAGIC_NUMBERS_CONFIG, } from "./settings.js";
+import { MAX_FUNCTION_PARAMETERS, MAXIMUM_CYCLOMATIC_COMPLEXITY, MAXIMUM_FILE_LINE_COUNT, NO_MAGIC_NUMBERS_CONFIG, } from "../settings.js";
+import { arrayFuncPluginConfig } from "./array-function.js";
+import { cssModulesPluginConfig } from "./css-modules.js";
+import { destructuringPluginConfig } from "./destructuring.js";
+import { eslintCommentsPluginConfig } from "./eslint-comments.js";
+import { importNewlinesPluginConfig } from "./import-newlines.js";
+import { importXPluginConfig } from "./import-x.js";
+import { noUnsanitizedPluginConfig } from "./no-unsanitized.js";
+import { noUselessAssignPluginConfig } from "./no-useless-assign.js";
+import { nodePluginConfig } from "./node.js";
+import { perfectionistPluginConfig } from "./perfectionist.js";
+import { promisePluginConfig } from "./promise.js";
+import { react19UpgradePluginConfig } from "./react-19-upgrade.js";
+import { reactCompilerPluginConfig } from "./react-compiler.js";
+import { reactHooksPluginConfig } from "./react-hooks.js";
+import { reactPerfPluginConfig } from "./react-perf.js";
+import { reactRefreshPluginConfig } from "./react-refresh.js";
+import { reactPluginConfig } from "./react.js";
+import { securityPluginConfig } from "./security.js";
+import { sortReactDependenyArraysPluginConfig } from "./sort-react-dependency-arrays.js";
+import { stylisticPluginConfig } from "./stylistic.js";
+import { typescriptPluginConfigs } from "./typescript.js";
+import { unicornPluginConfig } from "./unicorn.js";
+// Always place pluginConfig after rules in case there are overrides (e.g. in the typescriptPlugin)
+const pluginConfig = [
+    arrayFuncPluginConfig,
+    cssModulesPluginConfig,
+    destructuringPluginConfig,
+    eslintCommentsPluginConfig,
+    importNewlinesPluginConfig,
+    // contains rules turned off because they're not compatible with eslint 10
+    importXPluginConfig,
+    noUnsanitizedPluginConfig,
+    noUselessAssignPluginConfig,
+    nodePluginConfig,
+    perfectionistPluginConfig,
+    promisePluginConfig,
+    react19UpgradePluginConfig,
+    reactCompilerPluginConfig,
+    reactHooksPluginConfig,
+    reactPerfPluginConfig,
+    // contains rules turned off because they're not compatible with eslint 10
+    reactPluginConfig,
+    reactRefreshPluginConfig,
+    securityPluginConfig,
+    sortReactDependenyArraysPluginConfig,
+    stylisticPluginConfig,
+    ...typescriptPluginConfigs,
+    unicornPluginConfig,
+];
 export const rules = [
     {
         rules: {
@@ -1447,4 +1496,5 @@ export const rules = [
             yoda: "warn",
         },
     },
+    ...pluginConfig,
 ];

package/dist/rules/next.d.ts

@@ -0,0 +1,3 @@
+import type { Linter } from "eslint";
+export declare const nextPluginConfig: Linter.Config;
+//# sourceMappingURL=next.d.ts.map

package/dist/rules/next.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"next.d.ts","sourceRoot":"","sources":["../../src/rules/next.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAOrC,eAAO,MAAM,gBAAgB,EAAE,MAAM,CAAC,MAyJrC,CAAC"}

package/dist/rules/next.js

@@ -0,0 +1,140 @@
+import { fixupPluginRules } from "@eslint/compat";
+import next from "@next/eslint-plugin-next";
+/*
+ * Extra rules for Eslint
+ *
+ * https://www.npmjs.com/package/eslint-plugin-next
+ */
+export const nextPluginConfig = {
+    plugins: {
+        next: fixupPluginRules(next),
+    },
+    rules: {
+        /*
+         * Enforce font-display behavior with Google Fonts.
+         *
+         * https://nextjs.org/docs/messages/google-font-display
+         */
+        "next/google-font-display": "warn",
+        /*
+         * Ensure preconnect is used with Google Fonts.
+         *
+         * https://nextjs.org/docs/messages/google-font-preconnect
+         */
+        "next/google-font-preconnect": "warn",
+        /*
+         * Enforce id attribute on next/script components with inline content.
+         *
+         * https://nextjs.org/docs/messages/inline-script-id
+         */
+        "next/inline-script-id": "warn",
+        /*
+         * Prefer next/script component when using the inline script for Google Analytics.
+         *
+         * https://nextjs.org/docs/messages/next-script-for-ga
+         */
+        "next/next-script-for-ga": "warn",
+        /*
+         * Prevent assignment to the module variable.
+         *
+         * https://nextjs.org/docs/messages/no-assign-module-variable
+         */
+        "next/no-assign-module-variable": "warn",
+        /*
+         * Prevent client components from being async functions.
+         *
+         * https://nextjs.org/docs/messages/no-async-client-component
+         */
+        "next/no-async-client-component": "warn",
+        /*
+         * Prevent usage of next/script's beforeInteractive strategy outside of pages/_document.js.
+         *
+         * https://nextjs.org/docs/messages/no-before-interactive-script-outside-document
+         */
+        "next/no-before-interactive-script-outside-document": "warn",
+        /*
+         * Prevent manual stylesheet tags.
+         *
+         * https://nextjs.org/docs/messages/no-css-tags
+         */
+        "next/no-css-tags": "warn",
+        /*
+         * Prevent importing next/document outside of pages/_document.js.
+         *
+         * https://nextjs.org/docs/messages/no-document-import-in-page
+         */
+        "next/no-document-import-in-page": "warn",
+        /*
+         * Prevent duplicate usage of <Head> in pages/_document.js.
+         *
+         * https://nextjs.org/docs/messages/no-duplicate-head
+         */
+        "next/no-duplicate-head": "warn",
+        /*
+         * Prevent usage of <head> element.
+         *
+         * https://nextjs.org/docs/messages/no-head-element
+         */
+        "next/no-head-element": "warn",
+        /*
+         * Prevent usage of next/head in pages/_document.js.
+         *
+         * https://nextjs.org/docs/messages/no-head-import-in-document
+         */
+        "next/no-head-import-in-document": "warn",
+        /*
+         * Prevent usage of <a> elements to navigate to internal Next.js pages.
+         *
+         * https://nextjs.org/docs/messages/no-html-link-for-pages
+         */
+        "next/no-html-link-for-pages": "warn",
+        /*
+         * Prevent usage of <img> element due to slower LCP and higher bandwidth.
+         *
+         * https://nextjs.org/docs/messages/no-img-element
+         */
+        "next/no-img-element": "warn",
+        /*
+         * Prevent page-only custom fonts.
+         *
+         * https://nextjs.org/docs/messages/no-page-custom-font
+         */
+        "next/no-page-custom-font": "warn",
+        /*
+         * Prevent usage of next/script in next/head component.
+         *
+         * https://nextjs.org/docs/messages/no-script-component-in-head
+         */
+        "next/no-script-component-in-head": "warn",
+        /*
+         * Prevent usage of styled-jsx in pages/_document.js.
+         *
+         * https://nextjs.org/docs/messages/no-styled-jsx-in-document
+         */
+        "next/no-styled-jsx-in-document": "warn",
+        /*
+         * Prevent synchronous scripts.
+         *
+         * https://nextjs.org/docs/messages/no-sync-scripts
+         */
+        "next/no-sync-scripts": "warn",
+        /*
+         * Prevent usage of <title> with Head component from next/document.
+         *
+         * https://nextjs.org/docs/messages/no-title-in-document-head
+         */
+        "next/no-title-in-document-head": "warn",
+        /*
+         * Prevent common typos in Next.js's data fetching functions
+         *
+         * https://nextjs.org/docs/messages/no-typos
+         */
+        "next/no-typos": "warn",
+        /*
+         * Prevent duplicate polyfills from Polyfill.io.
+         *
+         * https://nextjs.org/docs/messages/no-unwanted-polyfillio
+         */
+        "next/no-unwanted-polyfillio": "warn",
+    },
+};

package/dist/rules/no-unsanitized.d.ts

@@ -0,0 +1,3 @@
+import type { Linter } from "eslint";
+export declare const noUnsanitizedPluginConfig: Linter.Config;
+//# sourceMappingURL=no-unsanitized.d.ts.map

package/dist/rules/no-unsanitized.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-unsanitized.d.ts","sourceRoot":"","sources":["../../src/rules/no-unsanitized.ts"],"names":[],"mappings":"AAgBA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAGrC,eAAO,MAAM,yBAAyB,EAAE,MAAM,CAAC,MA6B9C,CAAC"}