chore(deps): update eslint monorepo to v10 (major) (#1061)

* chore(deps): update eslint monorepo to v10
# Conflicts:
#	package.json
#	pnpm-lock.yaml

* Update lint and fix scripts in package.json

* Fix lint

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Daniel Tschinder <231804+danez@users.noreply.github.com>
Co-authored-by: Daniel Tschinder <code@tschinder.de>

Author: renovate[bot]

.changeset/mighty-taxis-appear.md

@@ -0,0 +1,5 @@
+---
+'react-docgen': patch
+---
+
+Added more defensive checks throughout the codebase. No functional changes.

eslint.config.mjs

@@ -89,7 +89,6 @@ export default tseslint.config([
       ],
 
       '@typescript-eslint/consistent-type-imports': 'error',
-      '@typescript-eslint/no-non-null-assertion': 'off',
       '@typescript-eslint/sort-type-constituents': 'error',
     },
   },

package.json

@@ -6,8 +6,8 @@
   },
   "scripts": {
     "build": "nx run-many --target=build --exclude='@react-docgen-internal/*'",
-    "lint": "eslint . --flag v10_config_lookup_from_file --report-unused-disable-directives --max-warnings=0",
-    "fix": "eslint . --flag v10_config_lookup_from_file --fix --report-unused-disable-directives --max-warnings=0",
+    "lint": "eslint . --report-unused-disable-directives --max-warnings=0",
+    "fix": "eslint . --fix --report-unused-disable-directives --max-warnings=0",
     "test": "nx run-many --target=test --exclude='@react-docgen-internal/*' --output-style=stream",
     "copy:changelog:react-docgen": "cp ./packages/react-docgen/CHANGELOG.md ./packages/website/src/app/docs/release-notes/react-docgen/page.mdx",
     "copy:changelog:cli": "cp ./packages/react-docgen-cli/CHANGELOG.md ./packages/website/src/app/docs/release-notes/cli/page.mdx",
@@ -19,11 +19,11 @@
     "@changesets/changelog-github": "0.6.0",
     "@changesets/cli": "2.30.0",
     "@eslint/eslintrc": "3.3.5",
-    "@eslint/js": "9.39.4",
+    "@eslint/js": "10.0.1",
     "@types/node": "20.19.37",
     "@vitest/coverage-v8": "4.1.0",
     "cpy": "13.2.1",
-    "eslint": "9.39.4",
+    "eslint": "10.0.3",
     "eslint-config-next": "16.1.6",
     "eslint-config-prettier": "10.1.8",
     "eslint-plugin-prettier": "5.5.5",

packages/react-docgen-cli/src/commands/parse/options/loadReactDocgenPlugin.ts

@@ -6,8 +6,10 @@ export default async function loadReactDocgenPlugin<T>(
   name: string,
   builtins?: Record<string, T>,
 ): Promise<T> {
-  if (builtins?.[input]) {
-    return builtins[input]!;
+  const builtin = builtins?.[input];
+
+  if (builtin !== undefined) {
+    return builtin;
   }
 
   const path = resolve(process.cwd(), input);

packages/react-docgen-cli/src/commands/parse/options/loadResolvers.ts

@@ -57,5 +57,11 @@ export default async function loadResolvers(
     });
   }
 
-  return loadResolver(input[0]!);
+  const [resolver] = input;
+
+  if (resolver === undefined) {
+    return;
+  }
+
+  return loadResolver(resolver);
 }

packages/react-docgen/src/handlers/componentDocblockHandler.ts

@@ -19,7 +19,11 @@ function getDocblockFromComponent(path: NodePath): string | null {
     // If we have a class declaration or expression, then the comment might be
     // attached to the last decorator instead as trailing comment.
     if (decorators && decorators.length > 0) {
-      description = getDocblock(decorators[decorators.length - 1]!, true);
+      const lastDecorator = decorators[decorators.length - 1];
+
+      if (lastDecorator) {
+        description = getDocblock(lastDecorator, true);
+      }
     }
   }
   if (description == null) {
@@ -41,9 +45,10 @@ function getDocblockFromComponent(path: NodePath): string | null {
     }
   }
   if (!description) {
-    const searchPath = isReactForwardRefCall(path)
-      ? path.get('arguments')[0]!
-      : path;
+    const [forwardRefArgument] = isReactForwardRefCall(path)
+      ? path.get('arguments')
+      : [];
+    const searchPath = forwardRefArgument ?? path;
     const inner = resolveToValue(searchPath);
 
     if (inner.node !== path.node) {

packages/react-docgen/src/handlers/componentMethodsHandler.ts

@@ -141,9 +141,13 @@ function findStatelessComponentBody(
       return body;
     }
   } else if (isReactForwardRefCall(componentDefinition)) {
-    const inner = resolveToValue(componentDefinition.get('arguments')[0]!);
+    const [forwardRefArgument] = componentDefinition.get('arguments');
 
-    return findStatelessComponentBody(inner);
+    if (forwardRefArgument) {
+      const inner = resolveToValue(forwardRefArgument);
+
+      return findStatelessComponentBody(inner);
+    }
   }
 
   return undefined;

packages/react-docgen/src/handlers/defaultPropsHandler.ts

@@ -23,7 +23,6 @@ import type { Handler } from './index.js';
 
 function getDefaultValue(path: NodePath): DefaultValueDescriptor | null {
   let defaultValue: string | undefined;
-  let resolvedPath = path;
   let valuePath = path;
 
   if (path.isBooleanLiteral()) {
@@ -33,6 +32,8 @@ function getDefaultValue(path: NodePath): DefaultValueDescriptor | null {
   } else if (path.isLiteral()) {
     defaultValue = path.node.extra?.raw as string;
   } else {
+    let resolvedPath: NodePath;
+
     if (path.isAssignmentPattern()) {
       resolvedPath = resolveToValue(path.get('right'));
     } else {
@@ -64,7 +65,13 @@ function getStatelessPropsPath(
   let value: NodePath = componentDefinition;
 
   if (isReactForwardRefCall(componentDefinition)) {
-    value = resolveToValue(componentDefinition.get('arguments')[0]!);
+    const [forwardRefArgument] = componentDefinition.get('arguments');
+
+    if (!forwardRefArgument) {
+      return undefined;
+    }
+
+    value = resolveToValue(forwardRefArgument);
   }
 
   if (!value.isFunction()) {

packages/react-docgen/src/utils/docblock.ts

@@ -35,7 +35,11 @@ export function getDocblock(path: NodePath, trailing = false): string | null {
   }
 
   if (comments.length > 0) {
-    return parseDocblock(comments[comments.length - 1]!.value);
+    const lastComment = comments[comments.length - 1];
+
+    if (lastComment) {
+      return parseDocblock(lastComment.value);
+    }
   }
 
   return null;
@@ -50,7 +54,11 @@ export function getDoclets(str: string): Record<string, string> {
   let match: RegExpExecArray | null;
 
   while ((match = DOCLET_PATTERN.exec(str))) {
-    doclets[match[1]!] = match[2] || true;
+    const [, name, value] = match;
+
+    if (name) {
+      doclets[name] = value || true;
+    }
   }
 
   return doclets;

packages/react-docgen/src/utils/findComponentDefinition.ts

@@ -24,7 +24,13 @@ function resolveComponentDefinition(
 ): NodePath<ComponentNode> | null {
   if (isReactCreateClassCall(definition)) {
     // return argument
-    const resolvedPath = resolveToValue(definition.get('arguments')[0]!);
+    const [argument] = definition.get('arguments');
+
+    if (!argument) {
+      return null;
+    }
+
+    const resolvedPath = resolveToValue(argument);
 
     if (resolvedPath.isObjectExpression()) {
       return resolvedPath;