fix: type errors (#427)

Author: ota-meshi

lib/rules/callback-return.js

@@ -5,7 +5,10 @@
 "use strict"
 const { getSourceCode } = require("../util/eslint-compat")
 
-/** @type {import('eslint').Rule.RuleModule} */
+/**
+ * @typedef {[string[]?]} RuleOptions
+ */
+/** @type {import('./rule-module').RuleModule<{RuleOptions: RuleOptions}>} */
 module.exports = {
     meta: {
         type: "suggestion",

lib/rules/exports-style.js

@@ -267,7 +267,13 @@ function fixModuleExports(node, sourceCode, fixer) {
     return fixer.replaceText(node.parent, statements.join("\n\n"))
 }
 
-/** @type {import('eslint').Rule.RuleModule} */
+/**
+ * @typedef {[
+ *   ("module.exports" | "exports")?,
+ *   {allowBatchAssign?: boolean}?
+ * ]} RuleOptions
+ */
+/** @type {import('./rule-module').RuleModule<{RuleOptions: RuleOptions}>} */
 module.exports = {
     meta: {
         docs: {

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

@@ -29,7 +29,13 @@ function getExistingExtensions(filePath) {
     }
 }
 
-/** @type {import('eslint').Rule.RuleModule} */
+/**
+ * @typedef {[
+ *   ("always" | "never")?,
+ *   {[ext in string]?: "always" | "never"}?
+ * ]} RuleOptions
+ */
+/** @type {import('./rule-module').RuleModule<{RuleOptions: RuleOptions}>} */
 module.exports = {
     meta: {
         docs: {

lib/rules/global-require.js

@@ -43,7 +43,7 @@ function isShadowed(scope, node) {
     return Boolean(reference?.resolved?.defs?.length)
 }
 
-/** @type {import('eslint').Rule.RuleModule} */
+/** @type {import('./rule-module').RuleModule} */
 module.exports = {
     meta: {
         type: "suggestion",

lib/rules/handle-callback-err.js

@@ -6,7 +6,10 @@
 
 const { getScope } = require("../util/eslint-compat")
 
-/** @type {import('eslint').Rule.RuleModule} */
+/**
+ * @typedef {[string?]} RuleOptions
+ */
+/** @type {import('./rule-module').RuleModule<{RuleOptions: RuleOptions}>} */
 module.exports = {
     meta: {
         type: "suggestion",

lib/rules/hashbang.js

@@ -78,7 +78,17 @@ function getShebangInfo(sourceCode) {
     }
 }
 
-/** @type {import('eslint').Rule.RuleModule} */
+/**
+ * @typedef {[
+ *   {
+ *     convertPath?: import('../util/get-convert-path').ConvertPath;
+ *     ignoreUnpublished?: boolean;
+ *     additionalExecutables?: string[];
+ *     executableMap?: Record<string, string>;
+ *   }?
+ * ]} RuleOptions
+ */
+/** @type {import('./rule-module').RuleModule<{RuleOptions: RuleOptions}>} */
 module.exports = {
     meta: {
         docs: {
@@ -146,7 +156,7 @@ module.exports = {
             convertedRelativePath
         )
 
-        /** @type {{ additionalExecutables: string[] }} */
+        /** @type {{ additionalExecutables?: string[] }} */
         const { additionalExecutables = [] } = context.options?.[0] ?? {}
 
         const executable = matcher()

lib/rules/no-callback-literal.js

@@ -6,7 +6,7 @@
 
 const callbackNames = ["callback", "cb"]
 
-/** @type {import('eslint').Rule.RuleModule} */
+/** @type {import('./rule-module').RuleModule} */
 module.exports = {
     meta: {
         docs: {

lib/rules/no-deprecated-api.js

@@ -749,7 +749,7 @@ function parseOptions(context) {
     return Object.freeze({ version, ignoredGlobalItems, ignoredModuleItems })
 }
 
-/** @type {import('eslint').Rule.RuleModule} */
+/** @type {import('./rule-module').RuleModule} */
 module.exports = {
     meta: {
         docs: {

lib/rules/no-exports-assign.js

@@ -45,7 +45,7 @@ function isModuleExports(node, scope) {
     )
 }
 
-/** @type {import('eslint').Rule.RuleModule} */
+/** @type {import('./rule-module').RuleModule} */
 module.exports = {
     meta: {
         docs: {

lib/rules/no-extraneous-import.js

@@ -11,7 +11,7 @@ const getResolvePaths = require("../util/get-resolve-paths")
 const getResolverConfig = require("../util/get-resolver-config")
 const visitImport = require("../util/visit-import")
 
-/** @type {import('eslint').Rule.RuleModule} */
+/** @type {import('./rule-module').RuleModule} */
 module.exports = {
     meta: {
         docs: {