@typescript-eslint/eslint-plugin 8.21.1-alpha.1 → 8.21.1-alpha.10

package/docs/rules/no-unused-vars.mdx

@@ -12,7 +12,11 @@ import TabItem from '@theme/TabItem';
 This rule extends the base [`eslint/no-unused-vars`](https://eslint.org/docs/rules/no-unused-vars) rule.
 It adds support for TypeScript features, such as types.
 
-## Benefits Over TypeScript
+## Options
+
+## FAQs
+
+### What benefits does this rule have over TypeScript?
 
 TypeScript provides [`noUnusedLocals`](https://www.typescriptlang.org/tsconfig#noUnusedLocals) and [`noUnusedParameters`](https://www.typescriptlang.org/tsconfig#noUnusedParameters) compiler options that can report errors on unused local variables or parameters, respectively.
 Those compiler options can be convenient to use if you don't want to set up ESLint and typescript-eslint.
@@ -52,3 +56,66 @@ Also see similar rules provided by ESLint:
 
 - [`no-unused-private-class-members`](https://eslint.org/docs/latest/rules/no-unused-private-class-members)
 - [`no-unused-labels`](https://eslint.org/docs/latest/rules/no-unused-labels)
+
+### Why does this rule report variables used only for types?
+
+This rule does not count type-only uses when determining whether a variable is used.
+Declaring variables only to use them for types adds code and runtime complexity.
+The variables are never actually used at runtime.
+They can be misleading to readers of the code.
+
+<Tabs>
+
+<TabItem value="typeof Variables">
+
+For example, if a variable is only used for `typeof`, this rule will report:
+
+```ts
+const box = {
+  //  ~~~
+  //  'box' is assigned a value but only used as a type.
+  value: 123,
+};
+
+export type Box = typeof box;
+```
+
+Instead, it's often cleaner and less code to write out the types directly:
+
+```ts
+export interface Box {
+  value: number;
+}
+```
+
+</TabItem>
+
+<TabItem value="Zod Schemas">
+
+For example, if a Zod schema variable is only used for `typeof`, this rule will report:
+
+```ts
+import { z } from 'zod';
+
+const schema = z.object({
+  //  ~~~~~~
+  //  'schema' is assigned a value but only used as a type.
+  value: z.number(),
+});
+
+export type Box = z.infer<typeof schema>;
+```
+
+Instead, it's often cleaner and less code to write out the types directly:
+
+```ts
+export interface Box {
+  value: number;
+}
+```
+
+</TabItem>
+
+</Tabs>
+
+If you find yourself writing runtime values only for types, consider refactoring your code to declare types directly.

package/docs/rules/prefer-nullish-coalescing.mdx

@@ -16,6 +16,7 @@ This rule reports when you may consider replacing:
 
 - An `||` operator with `??`
 - An `||=` operator with `??=`
+- Ternary expressions (`?:`) that are equivalent to `||` or `??` with `??`
 
 :::caution
 This rule will not work as expected if [`strictNullChecks`](https://www.typescriptlang.org/tsconfig#strictNullChecks) is not enabled.
@@ -42,7 +43,9 @@ foo === undefined ? 'a string' : foo;
 
 const foo: string | null = 'bar';
 foo !== null ? foo : 'a string';
+foo ? foo : 'a string';
 foo === null ? 'a string' : foo;
+!foo ? 'a string' : foo;
 ```
 
 Correct code for `ignoreTernaryTests: false`:
@@ -61,6 +64,8 @@ foo ?? 'a string';
 const foo: string | null = 'bar';
 foo ?? 'a string';
 foo ?? 'a string';
+foo ?? 'a string';
+foo ?? 'a string';
 ```
 
 ### `ignoreConditionalTests`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@typescript-eslint/eslint-plugin",
-  "version": "8.21.1-alpha.1",
+  "version": "8.21.1-alpha.10",
   "description": "TypeScript plugin for ESLint",
   "files": [
     "dist",
@@ -61,10 +61,10 @@
   },
   "dependencies": {
     "@eslint-community/regexpp": "^4.10.0",
-    "@typescript-eslint/scope-manager": "8.21.1-alpha.1",
-    "@typescript-eslint/type-utils": "8.21.1-alpha.1",
-    "@typescript-eslint/utils": "8.21.1-alpha.1",
-    "@typescript-eslint/visitor-keys": "8.21.1-alpha.1",
+    "@typescript-eslint/scope-manager": "8.21.1-alpha.10",
+    "@typescript-eslint/type-utils": "8.21.1-alpha.10",
+    "@typescript-eslint/utils": "8.21.1-alpha.10",
+    "@typescript-eslint/visitor-keys": "8.21.1-alpha.10",
     "graphemer": "^1.4.0",
     "ignore": "^5.3.1",
     "natural-compare": "^1.4.0",
@@ -75,8 +75,8 @@
     "@types/marked": "^5.0.2",
     "@types/mdast": "^4.0.3",
     "@types/natural-compare": "*",
-    "@typescript-eslint/rule-schema-to-typescript-types": "8.21.1-alpha.1",
-    "@typescript-eslint/rule-tester": "8.21.1-alpha.1",
+    "@typescript-eslint/rule-schema-to-typescript-types": "8.21.1-alpha.10",
+    "@typescript-eslint/rule-tester": "8.21.1-alpha.10",
     "ajv": "^6.12.6",
     "cross-env": "^7.0.3",
     "cross-fetch": "*",