linter-bundle 2.20.0 → 2.21.0

package/CHANGELOG.md

@@ -6,7 +6,24 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ## [Unreleased]
 
-[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v2.20.0...HEAD)
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v2.21.0...HEAD)
+
+## [2.21.0] - 2022-09-14
+
+### Added
+
+- [eslint] New rule [`no-unnecessary-typeof`](./eslint/rules/no-unnecessary-typeof.md), which prevents `typeof` checks at runtime, if a `typeof` operant has only one type in TypeScript.
+
+### Changed
+
+- [eslint] Updated `@typescript-eslint` from `5.36.1` to `5.37.0`
+- [eslint] Updated `eslint` from `8.23.0` to `8.23.1`
+- [eslint] Updated `eslint-import-resolver-typescript` from `3.5.0` to `3.5.1`
+- [eslint] Updated `eslint-plugin-functional` from `4.2.2` to `4.3.1`
+- [eslint] Updated `eslint-plugin-jest` from `27.0.1` to `27.0.4`
+- [eslint] Updated `eslint-plugin-react` from `7.31.1` to `7.31.8`
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v2.20.0...v2.21.0)
 
 ## [2.20.0] - 2022-09-01
 
@@ -969,7 +986,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ### Added
 
-- [gatsby/overrides-gatsby] New rule `no-global-undefined-check`, which prevents `typeof window === 'undefined'` checks, which are often [the source of rehydration problems](https://www.joshwcomeau.com/react/the-perils-of-rehydration/) in Gatsby
+- [eslint/overrides-gatsby] New rule [`no-global-undefined-check`](./eslint/rules/no-global-undefined-check.md), which prevents `typeof window === 'undefined'` checks, which are often [the source of rehydration problems](https://www.joshwcomeau.com/react/the-perils-of-rehydration/) in Gatsby
 
 [Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v1.1.0...v1.2.0)
 

package/README.md

@@ -38,6 +38,7 @@ This setup is using the following additional plugins:
 Beside that, the following additional rules are part of this bundle:
 
 - [no-global-undefined-check](./eslint/rules/no-global-undefined-check.md)
+- [no-unnecessary-typeof](./eslint/rules/no-unnecessary-typeof.md)
 
 ### stylelint
 

package/eslint/index.js

@@ -91,6 +91,11 @@ module.exports = {
 	},
 	reportUnusedDisableDirectives: true,
 	rules: {
+		/**
+		 * ./rules
+		 */
+		'no-unnecessary-typeof': 'error',
+
 		/**
 		 * eslint
 		 *

package/eslint/rules/no-unnecessary-typeof.js

@@ -0,0 +1,53 @@
+/**
+ * @file ESLint rule which ensures that `typeof window === 'undefined'` is not used, since it's often the source of rehydration issues in Gatsby.
+ *
+ * @see https://www.joshwcomeau.com/react/the-perils-of-rehydration/
+ */
+
+const { ESLintUtils } = require('@typescript-eslint/utils');
+
+/**
+ * @type {import('eslint').Rule.RuleModule}
+ */
+module.exports = {
+	meta: {
+		docs: {
+			description: 'If a `typeof` operant has only one type in TypeScript, it\'s unnecessary to check it\'s type at runtime.',
+			recommended: true
+		},
+		messages: {
+			text: 'Unnecessary `typeof`, because the only possible type of `{{ name }}` is `{{ type }}`.'
+		}
+	},
+	create (context) {
+		return {
+			UnaryExpression (node) {
+				if (node.operator !== 'typeof' || node.argument.type !== 'Identifier') {
+					return;
+				}
+
+				// @ts-expect-error -- Different type definitions for `Rule.RuleContext` in ESLint and @typescript-eslint
+				const parserServices = ESLintUtils.getParserServices(context);
+				const checker = parserServices.program.getTypeChecker();
+
+				// @ts-expect-error -- ESLint `Identifier` is not recognized as `Node` by @typescript-eslint
+				const originalNode = parserServices.esTreeNodeToTSNodeMap.get(node.argument);
+
+				/** @type {import('typescript').Type & { intrinsicName?: string; }} */
+				const nodeType = checker.getTypeAtLocation(originalNode);
+
+				// `intrinsicName` only exists if there is exactly one type
+				if (nodeType.intrinsicName && !['any', 'error', 'unknown'].includes(nodeType.intrinsicName)) {
+					context.report({
+						node,
+						messageId: 'text',
+						data: {
+							name: node.argument.name,
+							type: nodeType.intrinsicName
+						}
+					});
+				}
+			}
+		};
+	}
+};

package/eslint/rules/no-unnecessary-typeof.md

@@ -0,0 +1,18 @@
+# Disallow unnecessary `typeof` checks (`no-unnecessary-typeof`)
+
+## Rule Details
+
+If a `typeof` operant has only one type in TypeScript, it's unnecessary to check it's type at runtime.
+
+Examples of **incorrect** code for this rule:
+
+```ts
+declare var myString: string;
+
+if (typeof myString === 'string') {}
+if (typeof myString === 'boolean') {}
+
+declare var myBoolean: (boolean | string) & (boolean | number);
+
+if (typeof myBoolean === 'boolean') {}
+```

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "linter-bundle",
-	"version": "2.20.0",
+	"version": "2.21.0",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
 		"eslint",
@@ -40,20 +40,21 @@
 		"check-outdated": "npx --yes -- check-outdated --ignore-pre-releases"
 	},
 	"dependencies": {
-		"@typescript-eslint/eslint-plugin": "5.36.1",
-		"@typescript-eslint/parser": "5.36.1",
-		"eslint": "8.23.0",
-		"eslint-import-resolver-typescript": "3.5.0",
+		"@typescript-eslint/eslint-plugin": "5.37.0",
+		"@typescript-eslint/parser": "5.37.0",
+		"@typescript-eslint/utils": "5.37.0",
+		"eslint": "8.23.1",
+		"eslint-import-resolver-typescript": "3.5.1",
 		"eslint-import-resolver-webpack": "0.13.2",
 		"eslint-plugin-eslint-comments": "3.2.0",
-		"eslint-plugin-functional": "4.2.2",
+		"eslint-plugin-functional": "4.3.1",
 		"eslint-plugin-import": "2.26.0",
-		"eslint-plugin-jest": "27.0.1",
+		"eslint-plugin-jest": "27.0.4",
 		"eslint-plugin-jsdoc": "39.3.6",
 		"eslint-plugin-jsx-a11y": "6.6.1",
 		"eslint-plugin-node": "11.1.0",
 		"eslint-plugin-promise": "6.0.1",
-		"eslint-plugin-react": "7.31.1",
+		"eslint-plugin-react": "7.31.8",
 		"eslint-plugin-react-hooks": "4.6.0",
 		"eslint-plugin-unicorn": "43.0.2",
 		"markdownlint-cli": "0.32.2",
@@ -73,8 +74,8 @@
 	"devDependencies": {
 		"@types/eslint": "8.4.6",
 		"@types/micromatch": "4.0.2",
-		"@types/node": "18.7.14",
+		"@types/node": "18.7.18",
 		"stylelint-find-new-rules": "4.0.0",
-		"typescript": "4.8.2"
+		"typescript": "4.8.3"
 	}
 }