Put in hacks to make .d.ts generation succeed for @emotion/eslint-plugin

packages/eslint-plugin/src/index.ts

@@ -5,7 +5,7 @@ import styledImport from './rules/styled-import'
 import jsxImport from './rules/jsx-import'
 import pkgRenaming from './rules/pkg-renaming'
 
-export let rules = {
+export const rules = {
   'import-from-emotion': importFromEmotion,
   'no-vanilla': noVanilla,
   'syntax-preference': syntaxPreference,

packages/eslint-plugin/src/rules/import-from-emotion.ts

@@ -1,5 +1,5 @@
 import { AST_NODE_TYPES, TSESTree } from '@typescript-eslint/experimental-utils'
-import { createRule } from '../utils'
+import { createRule, EmotionESLintRule } from '../utils'
 
 export default createRule({
   name: __filename,
@@ -65,4 +65,4 @@ export default createRule({
       }
     }
   }
-})
+}) as EmotionESLintRule

packages/eslint-plugin/src/rules/jsx-import.ts

@@ -1,5 +1,5 @@
 import { AST_NODE_TYPES, TSESTree } from '@typescript-eslint/experimental-utils'
-import { createRule, REPO_URL } from '../utils'
+import { createRule, EmotionESLintRule, REPO_URL } from '../utils'
 
 const JSX_ANNOTATION_REGEX = /\*?\s*@jsx\s+([^\s]+)/
 const JSX_IMPORT_SOURCE_REGEX = /\*?\s*@jsxImportSource\s+([^\s]+)/
@@ -273,4 +273,4 @@ export default createRule<RuleOptions, keyof typeof messages>({
       }
     }
   }
-})
+}) as EmotionESLintRule

packages/eslint-plugin/src/rules/no-vanilla.ts

@@ -1,4 +1,4 @@
-import { createRule } from '../utils'
+import { createRule, EmotionESLintRule } from '../utils'
 
 export default createRule({
   name: __filename,
@@ -27,4 +27,4 @@ export default createRule({
       }
     }
   }
-})
+}) as EmotionESLintRule

packages/eslint-plugin/src/rules/pkg-renaming.ts

@@ -1,5 +1,5 @@
 import { AST_NODE_TYPES } from '@typescript-eslint/experimental-utils'
-import { createRule } from '../utils'
+import { createRule, EmotionESLintRule } from '../utils'
 
 const simpleMappings = new Map<unknown, string>([
   ['@emotion/core', '@emotion/react'],
@@ -85,4 +85,4 @@ export default createRule({
       }
     }
   }
-})
+}) as EmotionESLintRule

packages/eslint-plugin/src/rules/styled-import.ts

@@ -1,5 +1,5 @@
 import { AST_NODE_TYPES } from '@typescript-eslint/experimental-utils'
-import { createRule } from '../utils'
+import { createRule, EmotionESLintRule } from '../utils'
 
 export default createRule({
   name: __filename,
@@ -37,4 +37,4 @@ export default createRule({
       }
     }
   }
-})
+}) as EmotionESLintRule

packages/eslint-plugin/src/rules/syntax-preference.ts

@@ -3,7 +3,7 @@ import {
   TSESLint,
   TSESTree
 } from '@typescript-eslint/experimental-utils'
-import { createRule } from '../utils'
+import { createRule, EmotionESLintRule } from '../utils'
 
 /**
  * @fileoverview Choose between string or object syntax
@@ -265,4 +265,4 @@ export default createRule<RuleOptions, MessageId>({
         return {}
     }
   }
-})
+}) as EmotionESLintRule

packages/eslint-plugin/src/utils.ts

@@ -10,3 +10,18 @@ export const createRule = ESLintUtils.RuleCreator(name => {
 
   return `${REPO_URL}/blob/@emotion/eslint-plugin@${version}/packages/eslint-plugin/docs/rules/${ruleName}.md`
 })
+
+// This is based off of RuleModule from `@typescript-eslint/experimental-utils`.
+// All exported rules should use this type to prevent the error:
+//
+// TS2742: The inferred type of 'rules' cannot be named without a reference to
+// '@typescript-eslint/experimental-utils/node_modules/@typescript-eslint/types/dist/ast-spec'.
+// This is likely not portable. A type annotation is necessary.
+//
+// Really, there is no need to generate TypeScript declarations for this
+// package, but there does not seem to be a way to disable declaration
+// generation when using Preconstruct
+export interface EmotionESLintRule {
+  meta: unknown
+  create(context: unknown): unknown
+}

packages/eslint-plugin/test/rules/import-from-emotion.test.ts

@@ -4,7 +4,7 @@
 
 import { TSESLint } from '@typescript-eslint/experimental-utils'
 import rule from '../../src/rules/import-from-emotion'
-import { espreeParser } from '../test-utils'
+import { RuleModuleForTesting, espreeParser } from '../test-utils'
 
 const ruleTester = new TSESLint.RuleTester({
   parser: espreeParser,
@@ -17,7 +17,7 @@ const ruleTester = new TSESLint.RuleTester({
   }
 })
 
-ruleTester.run('emotion jsx', rule, {
+ruleTester.run('emotion jsx', rule as unknown as RuleModuleForTesting, {
   valid: [
     {
       code: `import { css } from 'emotion'`

packages/eslint-plugin/test/rules/jsx-import.test.ts

@@ -4,7 +4,7 @@
 
 import { TSESLint } from '@typescript-eslint/experimental-utils'
 import rule from '../../src/rules/jsx-import'
-import { espreeParser } from '../test-utils'
+import { RuleModuleForTesting, espreeParser } from '../test-utils'
 
 const ruleTester = new TSESLint.RuleTester({
   parser: espreeParser,
@@ -17,7 +17,7 @@ const ruleTester = new TSESLint.RuleTester({
   }
 })
 
-ruleTester.run('emotion jsx', rule, {
+ruleTester.run('emotion jsx', rule as unknown as RuleModuleForTesting, {
   valid: [
     {
       code: `

packages/eslint-plugin/test/rules/no-vanilla.test.ts

@@ -4,7 +4,7 @@
 
 import { TSESLint } from '@typescript-eslint/experimental-utils'
 import rule from '../../src/rules/no-vanilla'
-import { espreeParser } from '../test-utils'
+import { espreeParser, RuleModuleForTesting } from '../test-utils'
 
 const ruleTester = new TSESLint.RuleTester({
   parser: espreeParser,
@@ -17,7 +17,7 @@ const ruleTester = new TSESLint.RuleTester({
   }
 })
 
-ruleTester.run('no-vanilla', rule, {
+ruleTester.run('no-vanilla', rule as unknown as RuleModuleForTesting, {
   valid: [{ code: `import { css } from '@emotion/react'` }],
   invalid: [
     {

packages/eslint-plugin/test/rules/pkg-renaming.test.ts

@@ -4,7 +4,7 @@
 
 import { TSESLint } from '@typescript-eslint/experimental-utils'
 import rule from '../../src/rules/pkg-renaming'
-import { espreeParser } from '../test-utils'
+import { espreeParser, RuleModuleForTesting } from '../test-utils'
 
 const ruleTester = new TSESLint.RuleTester({
   parser: espreeParser,
@@ -17,7 +17,7 @@ const ruleTester = new TSESLint.RuleTester({
   }
 })
 
-ruleTester.run('pkg-renaming', rule, {
+ruleTester.run('pkg-renaming', rule as unknown as RuleModuleForTesting, {
   valid: [
     {
       code: `

packages/eslint-plugin/test/rules/styled-import.test.ts

@@ -4,7 +4,7 @@
 
 import { TSESLint } from '@typescript-eslint/experimental-utils'
 import rule from '../../src/rules/styled-import'
-import { espreeParser } from '../test-utils'
+import { espreeParser, RuleModuleForTesting } from '../test-utils'
 
 const ruleTester = new TSESLint.RuleTester({
   parser: espreeParser,
@@ -17,7 +17,7 @@ const ruleTester = new TSESLint.RuleTester({
   }
 })
 
-ruleTester.run('emotion styled', rule, {
+ruleTester.run('emotion styled', rule as unknown as RuleModuleForTesting, {
   valid: [
     {
       code: `