eslint-plugin-n 16.1.0 → 16.3.0

package/README.md

@@ -2,9 +2,11 @@
 
 > forked from [eslint-plugin-node](https://www.npmjs.com/package/eslint-plugin-node) v11.1.0. as the original repository seems [no longer maintained](https://github.com/mysticatea/eslint-plugin-node/issues/300).
 
-[![npm version](https://img.shields.io/npm/v/eslint-plugin-n.svg)](https://www.npmjs.com/package/eslint-plugin-n) [![Downloads/month](https://img.shields.io/npm/dm/eslint-plugin-n.svg)](http://www.npmtrends.com/eslint-plugin-n) [![Build Status](https://github.com/weiran-zsd/eslint-plugin-node/workflows/CI/badge.svg)](https://github.com/weiran-zsd/eslint-plugin-node/actions) [![Coverage Status](https://codecov.io/gh/weiran-zsd/eslint-plugin-node/branch/master/graph/badge.svg)](https://codecov.io/gh/weiran-zsd/eslint-plugin-node) [![Dependency Status](https://david-dm.org/weiran-zsd/eslint-plugin-node.svg)](https://david-dm.org/weiran-zsd/eslint-plugin-node)
+[![npm version](https://img.shields.io/npm/v/eslint-plugin-n.svg)](https://www.npmjs.com/package/eslint-plugin-n)
+[![Downloads](https://img.shields.io/npm/dm/eslint-plugin-n.svg)](https://www.npmjs.com/package/eslint-plugin-n)
+[![Build Status](https://github.com/eslint-community/eslint-plugin-n/workflows/CI/badge.svg)](https://github.com/eslint-community/eslint-plugin-n/actions)
 
-Additional ESLint's rules for Node.js
+Additional ESLint rules for Node.js
 
 ## 💿 Install & Usage
 
@@ -207,7 +209,7 @@ Deprecated rules follow ESLint's [deprecation policy](https://eslint.org/docs/us
 
 ## 📰 Changelog
 
-- [GitHub Releases](https://github.com/weiran-zsd/eslint-plugin-node/releases)
+- [GitHub Releases](https://github.com/eslint-community/eslint-plugin-n/releases)
 
 ## ❤️ Contributing
 

package/lib/rules/callback-return.js

@@ -9,9 +9,8 @@ module.exports = {
         type: "suggestion",
         docs: {
             description: "require `return` statements after callbacks",
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/callback-return.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/callback-return.md",
         },
         schema: [
             {

package/lib/rules/exports-style.js

@@ -227,9 +227,8 @@ module.exports = {
     meta: {
         docs: {
             description: "enforce either `module.exports` or `exports`",
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/exports-style.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/exports-style.md",
         },
         type: "suggestion",
         fixable: "code",

package/lib/rules/file-extension-in-import.js

@@ -8,9 +8,6 @@ const path = require("path")
 const fs = require("fs")
 const mapTypescriptExtension = require("../util/map-typescript-extension")
 const visitImport = require("../util/visit-import")
-const packageNamePattern = /^(?:@[^/\\]+[/\\])?[^/\\]+$/u
-const corePackageOverridePattern =
-    /^(?:assert|async_hooks|buffer|child_process|cluster|console|constants|crypto|dgram|dns|domain|events|fs|http|http2|https|inspector|module|net|os|path|perf_hooks|process|punycode|querystring|readline|repl|stream|string_decoder|sys|timers|tls|trace_events|tty|url|util|v8|vm|worker_threads|zlib)[/\\]$/u
 
 /**
  * Get all file extensions of the files which have the same basename.
@@ -37,9 +34,8 @@ module.exports = {
         docs: {
             description:
                 "enforce the style of file extensions in `import` declarations",
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/file-extension-in-import.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/file-extension-in-import.md",
         },
         fixable: "code",
         messages: {
@@ -67,13 +63,13 @@ module.exports = {
         const defaultStyle = context.options[0] || "always"
         const overrideStyle = context.options[1] || {}
 
-        function verify({ filePath, name, node }) {
+        /**
+         * @param {import("../util/import-target.js")} target
+         * @returns {void}
+         */
+        function verify({ filePath, name, node, moduleType }) {
             // Ignore if it's not resolved to a file or it's a bare module.
-            if (
-                !filePath ||
-                packageNamePattern.test(name) ||
-                corePackageOverridePattern.test(name)
-            ) {
+            if (moduleType !== "relative" && moduleType !== "absolute") {
                 return
             }
 

package/lib/rules/global-require.js

@@ -53,9 +53,8 @@ module.exports = {
         docs: {
             description:
                 "require `require()` calls to be placed at top-level module scope",
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/global-require.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/global-require.md",
         },
         fixable: null,
         schema: [],

package/lib/rules/handle-callback-err.js

@@ -9,9 +9,8 @@ module.exports = {
         type: "suggestion",
         docs: {
             description: "require error handling in callbacks",
-            category: "Possible Errors",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/handle-callback-err.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/handle-callback-err.md",
         },
         fixable: null,
         schema: [

package/lib/rules/no-callback-literal.js

@@ -9,9 +9,8 @@ module.exports = {
         docs: {
             description:
                 "enforce Node.js-style error-first callback pattern is followed",
-            category: "Possible Errors",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-callback-literal.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-callback-literal.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-deprecated-api.js

@@ -691,9 +691,8 @@ module.exports = {
     meta: {
         docs: {
             description: "disallow deprecated APIs",
-            category: "Best Practices",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-deprecated-api.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-deprecated-api.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-exports-assign.js

@@ -38,9 +38,8 @@ module.exports = {
     meta: {
         docs: {
             description: "disallow the assignment to `exports`",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-exports-assign.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-exports-assign.md",
         },
         fixable: null,
         messages: {

package/lib/rules/no-extraneous-import.js

@@ -15,9 +15,8 @@ module.exports = {
         docs: {
             description:
                 "disallow `import` declarations which import extraneous modules",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-extraneous-import.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-extraneous-import.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-extraneous-require.js

@@ -16,9 +16,8 @@ module.exports = {
         docs: {
             description:
                 "disallow `require()` expressions which import extraneous modules",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-extraneous-require.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-extraneous-require.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-hide-core-modules.js

@@ -57,9 +57,8 @@ module.exports = {
         docs: {
             description:
                 "disallow third-party modules which are hiding core modules",
-            category: "Possible Errors",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-hide-core-modules.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-hide-core-modules.md",
         },
         type: "problem",
         deprecated: true,
@@ -81,8 +80,9 @@ module.exports = {
             },
         ],
         messages: {
-            "unexpectedImport": "Unexpected import of third-party module '{{name}}'.",
-        }
+            unexpectedImport:
+                "Unexpected import of third-party module '{{name}}'.",
+        },
     },
     create(context) {
         if (context.getFilename() === "<input>") {

package/lib/rules/no-missing-import.js

@@ -16,9 +16,8 @@ module.exports = {
         docs: {
             description:
                 "disallow `import` declarations which import non-existence modules",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-missing-import.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-missing-import.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-missing-require.js

@@ -17,9 +17,8 @@ module.exports = {
         docs: {
             description:
                 "disallow `require()` expressions which import non-existence modules",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-missing-require.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-missing-require.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-mixed-requires.js

@@ -71,9 +71,8 @@ module.exports = {
         docs: {
             description:
                 "disallow `require` calls to be mixed with regular variable declarations",
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-mixed-requires.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-mixed-requires.md",
         },
         fixable: null,
         schema: [

package/lib/rules/no-new-require.js

@@ -9,9 +9,8 @@ module.exports = {
         type: "suggestion",
         docs: {
             description: "disallow `new` operators with calls to `require`",
-            category: "Possible Errors",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-new-require.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-new-require.md",
         },
         fixable: null,
         schema: [],

package/lib/rules/no-path-concat.js

@@ -166,9 +166,8 @@ module.exports = {
         docs: {
             description:
                 "disallow string concatenation with `__dirname` and `__filename`",
-            category: "Possible Errors",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-path-concat.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-path-concat.md",
         },
         fixable: null,
         schema: [],

package/lib/rules/no-process-env.js

@@ -13,9 +13,8 @@ module.exports = {
         type: "suggestion",
         docs: {
             description: "disallow the use of `process.env`",
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-process-env.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-process-env.md",
         },
         fixable: null,
         schema: [],

package/lib/rules/no-process-exit.js

@@ -9,9 +9,8 @@ module.exports = {
         type: "suggestion",
         docs: {
             description: "disallow the use of `process.exit()`",
-            category: "Possible Errors",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-process-exit.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-process-exit.md",
         },
         fixable: null,
         schema: [],

package/lib/rules/no-restricted-import.js

@@ -13,9 +13,8 @@ module.exports = {
         docs: {
             description:
                 "disallow specified modules when loaded by `import` declarations",
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-restricted-import.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-restricted-import.md",
         },
         fixable: null,
         schema: [

package/lib/rules/no-restricted-require.js

@@ -13,9 +13,8 @@ module.exports = {
         type: "suggestion",
         docs: {
             description: "disallow specified modules when loaded by `require`",
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-restricted-require.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-restricted-require.md",
         },
         fixable: null,
         schema: [

package/lib/rules/no-sync.js

@@ -4,14 +4,31 @@
  */
 "use strict"
 
+const allowedAtRootLevelSelector = [
+    // fs.readFileSync()
+    ":function MemberExpression > Identifier[name=/Sync$/]",
+    // readFileSync.call(null, 'path')
+    ":function MemberExpression > Identifier[name=/Sync$/]",
+    // readFileSync()
+    ":function :not(MemberExpression) > Identifier[name=/Sync$/]",
+]
+
+const disallowedAtRootLevelSelector = [
+    // fs.readFileSync()
+    "MemberExpression > Identifier[name=/Sync$/]",
+    // readFileSync.call(null, 'path')
+    "MemberExpression > Identifier[name=/Sync$/]",
+    // readFileSync()
+    ":not(MemberExpression) > Identifier[name=/Sync$/]",
+]
+
 module.exports = {
     meta: {
         type: "suggestion",
         docs: {
             description: "disallow synchronous methods",
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-sync.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-sync.md",
         },
         fixable: null,
         schema: [
@@ -32,18 +49,17 @@ module.exports = {
     },
 
     create(context) {
-        const selector =
-            context.options[0] && context.options[0].allowAtRootLevel
-                ? ":function MemberExpression[property.name=/.*Sync$/]"
-                : "MemberExpression[property.name=/.*Sync$/]"
+        const selector = context.options[0]?.allowAtRootLevel
+            ? allowedAtRootLevelSelector
+            : disallowedAtRootLevelSelector
 
         return {
             [selector](node) {
                 context.report({
-                    node,
+                    node: node.parent,
                     messageId: "noSync",
                     data: {
-                        propertyName: node.property.name,
+                        propertyName: node.name,
                     },
                 })
             },

package/lib/rules/no-unpublished-bin.js

@@ -33,9 +33,8 @@ module.exports = {
     meta: {
         docs: {
             description: "disallow `bin` files that npm ignores",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-unpublished-bin.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-unpublished-bin.md",
         },
         type: "problem",
         fixable: null,
@@ -49,8 +48,9 @@ module.exports = {
             },
         ],
         messages: {
-            invalidIgnored: "npm ignores '{{name}}'. Check 'files' field of 'package.json' or '.npmignore'."
-        }
+            invalidIgnored:
+                "npm ignores '{{name}}'. Check 'files' field of 'package.json' or '.npmignore'.",
+        },
     },
     create(context) {
         return {

package/lib/rules/no-unpublished-import.js

@@ -15,9 +15,8 @@ module.exports = {
         docs: {
             description:
                 "disallow `import` declarations which import private modules",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-unpublished-import.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-unpublished-import.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-unpublished-require.js

@@ -16,9 +16,8 @@ module.exports = {
         docs: {
             description:
                 "disallow `require()` expressions which import private modules",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-unpublished-require.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-unpublished-require.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-unsupported-features/es-builtins.js

@@ -143,9 +143,8 @@ module.exports = {
         docs: {
             description:
                 "disallow unsupported ECMAScript built-ins on the specified version",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-unsupported-features/es-builtins.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-unsupported-features/es-builtins.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-unsupported-features/es-syntax.js

@@ -523,9 +523,8 @@ module.exports = {
         docs: {
             description:
                 "disallow unsupported ECMAScript syntax on the specified version",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-unsupported-features/es-syntax.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-unsupported-features/es-syntax.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-unsupported-features/node-builtins.js

@@ -379,9 +379,8 @@ module.exports = {
         docs: {
             description:
                 "disallow unsupported Node.js built-in APIs on the specified version",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-unsupported-features/node-builtins.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-unsupported-features/node-builtins.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/no-unsupported-features.js

@@ -1048,10 +1048,9 @@ module.exports = {
         docs: {
             description:
                 "disallow unsupported ECMAScript features on the specified version",
-            category: "Possible Errors",
             recommended: false,
 
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-unsupported-features.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-unsupported-features.md",
         },
         type: "problem",
         deprecated: true,

package/lib/rules/prefer-global/buffer.js

@@ -23,9 +23,8 @@ module.exports = {
         docs: {
             description:
                 'enforce either `Buffer` or `require("buffer").Buffer`',
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/prefer-global/buffer.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-global/buffer.md",
         },
         type: "suggestion",
         fixable: null,

package/lib/rules/prefer-global/console.js

@@ -20,9 +20,8 @@ module.exports = {
     meta: {
         docs: {
             description: 'enforce either `console` or `require("console")`',
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/prefer-global/console.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-global/console.md",
         },
         type: "suggestion",
         fixable: null,

package/lib/rules/prefer-global/process.js

@@ -20,9 +20,8 @@ module.exports = {
     meta: {
         docs: {
             description: 'enforce either `process` or `require("process")`',
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/prefer-global/process.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-global/process.md",
         },
         type: "suggestion",
         fixable: null,

package/lib/rules/prefer-global/text-decoder.js

@@ -23,9 +23,8 @@ module.exports = {
         docs: {
             description:
                 'enforce either `TextDecoder` or `require("util").TextDecoder`',
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/prefer-global/text-decoder.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-global/text-decoder.md",
         },
         type: "suggestion",
         fixable: null,

package/lib/rules/prefer-global/text-encoder.js

@@ -23,9 +23,8 @@ module.exports = {
         docs: {
             description:
                 'enforce either `TextEncoder` or `require("util").TextEncoder`',
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/prefer-global/text-encoder.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-global/text-encoder.md",
         },
         type: "suggestion",
         fixable: null,

package/lib/rules/prefer-global/url-search-params.js

@@ -23,9 +23,8 @@ module.exports = {
         docs: {
             description:
                 'enforce either `URLSearchParams` or `require("url").URLSearchParams`',
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/prefer-global/url-search-params.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-global/url-search-params.md",
         },
         type: "suggestion",
         fixable: null,

package/lib/rules/prefer-global/url.js

@@ -22,9 +22,8 @@ module.exports = {
     meta: {
         docs: {
             description: 'enforce either `URL` or `require("url").URL`',
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/prefer-global/url.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-global/url.md",
         },
         type: "suggestion",
         fixable: null,

package/lib/rules/prefer-promises/dns.js

@@ -38,9 +38,8 @@ module.exports = {
     meta: {
         docs: {
             description: 'enforce `require("dns").promises`',
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/prefer-promises/dns.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-promises/dns.md",
         },
         fixable: null,
         messages: {

package/lib/rules/prefer-promises/fs.js

@@ -40,9 +40,8 @@ module.exports = {
     meta: {
         docs: {
             description: 'enforce `require("fs").promises`',
-            category: "Stylistic Issues",
             recommended: false,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/prefer-promises/fs.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-promises/fs.md",
         },
         fixable: null,
         messages: {

package/lib/rules/process-exit-as-throw.js

@@ -149,9 +149,8 @@ module.exports = {
         docs: {
             description:
                 "require that `process.exit()` expressions use the same code path as `throw`",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/process-exit-as-throw.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/process-exit-as-throw.md",
         },
         type: "problem",
         fixable: null,

package/lib/rules/shebang.js

@@ -70,9 +70,8 @@ module.exports = {
     meta: {
         docs: {
             description: "require correct usage of shebang",
-            category: "Possible Errors",
             recommended: true,
-            url: "https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/shebang.md",
+            url: "https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/shebang.md",
         },
         type: "problem",
         fixable: "code",

package/lib/util/get-npmignore.js

@@ -12,7 +12,6 @@ const exists = require("./exists")
 const getPackageJson = require("./get-package-json")
 
 const cache = new Cache()
-const SLASH_AT_BEGIN_AND_END = /^!?\/+|^!|\/+$/gu
 const PARENT_RELATIVE_PATH = /^\.\./u
 const NEVER_IGNORED =
     /^(?:readme\.[^.]*|(?:licen[cs]e|changes|changelog|history)(?:\.[^.]*)?)$/iu
@@ -90,7 +89,10 @@ function parseWhiteList(files) {
 
     for (const file of files) {
         if (typeof file === "string" && file) {
-            const body = file.replace(SLASH_AT_BEGIN_AND_END, "")
+            const body = path.posix
+                .normalize(file.replace(/^!/u, ""))
+                .replace(/\/+$/u, "")
+
             if (file.startsWith("!")) {
                 igN.add(`${body}`)
                 igN.add(`${body}/**`)

package/lib/util/import-target.js

@@ -6,6 +6,7 @@
 
 const path = require("path")
 const { pathToFileURL, fileURLToPath } = require("url")
+const { isBuiltin } = require("module")
 const resolve = require("resolve")
 const {
     defaultResolve: importResolve,
@@ -72,6 +73,14 @@ function getFilePath(isModule, id, options, moduleType) {
     }
 }
 
+function isNodeModule(name, options) {
+    try {
+        return require.resolve(name, options).startsWith(path.sep)
+    } catch {
+        return false
+    }
+}
+
 /**
  * Gets the module name of a given path.
  *
@@ -115,6 +124,24 @@ module.exports = class ImportTarget {
          */
         this.name = name
 
+        /**
+         * What type of module is this
+         * @type {'unknown'|'relative'|'absolute'|'node'|'npm'|'http'|void}
+         */
+        this.moduleType = "unknown"
+
+        if (name.startsWith("./") || name.startsWith(".\\")) {
+            this.moduleType = "relative"
+        } else if (name.startsWith("/") || name.startsWith("\\")) {
+            this.moduleType = "absolute"
+        } else if (isBuiltin(name)) {
+            this.moduleType = "node"
+        } else if (isNodeModule(name, options)) {
+            this.moduleType = "npm"
+        } else if (name.startsWith("http://") || name.startsWith("https://")) {
+            this.moduleType = "http"
+        }
+
         /**
          * The full path of this import target.
          * If the target is a module and it does not exist then this is `null`.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "eslint-plugin-n",
-    "version": "16.1.0",
+    "version": "16.3.0",
     "description": "Additional ESLint's rules for Node.js",
     "engines": {
         "node": ">=16.0.0"
@@ -25,11 +25,9 @@
         "semver": "^7.5.3"
     },
     "devDependencies": {
-        "@eslint/eslintrc": "^2.0.3",
         "@eslint/js": "^8.43.0",
         "@types/eslint": "^8.44.2",
         "@typescript-eslint/parser": "^5.60.0",
-        "codecov": "^3.8.2",
         "esbuild": "^0.18.7",
         "eslint": "^8.43.0",
         "eslint-config-prettier": "^8.8.0",
@@ -43,7 +41,7 @@
         "lint-staged": "^13.2.2",
         "markdownlint-cli": "^0.35.0",
         "mocha": "^10.2.0",
-        "npm-run-all": "^4.1.5",
+        "npm-run-all2": "^6.1.1",
         "nyc": "^15.1.0",
         "opener": "^1.5.2",
         "prettier": "^2.8.8",
@@ -55,7 +53,6 @@
     "scripts": {
         "build": "node scripts/update",
         "clean": "rimraf .nyc_output coverage",
-        "codecov": "nyc report --reporter text-lcov | codecov --pipe --disable=gcov -t $CODECOV_TOKEN",
         "coverage": "opener ./coverage/lcov-report/index.html",
         "format": "prettier --loglevel warn --write \"**/*.{js,css,md}\"",
         "lint": "npm-run-all \"lint:*\"",