From d10854129711a78bdd52b255777097da4090e63a Mon Sep 17 00:00:00 2001
From: Kenzo-Wada <used.acormhib@gmail.com>
Date: Tue, 23 Dec 2025 00:05:14 +0900
Subject: [PATCH 01/11] feat(linter): added rule for `react/jsx-max-depth`

---
 apps/oxlint/fixtures/tsgolint_fix/fix.ts      |   2 +-
 ...tsgolint_--type-aware --silent@oxlint.snap |   7 +-
 ...ixtures__tsgolint_--type-aware@oxlint.snap | 389 +-----------------
 ...lint_rule_options_--type-aware@oxlint.snap |  26 +-
 .../snapshots/stacktrace_is_correct.snap      |   2 +-
 .../src/generated/rule_runner_impls.rs        |   6 +
 crates/oxc_linter/src/rules.rs                |   2 +
 .../src/rules/react/jsx_max_depth.rs          | 206 ++++++++++
 .../src/snapshots/react_jsx_max_depth.snap    |  18 +
 9 files changed, 255 insertions(+), 403 deletions(-)
 create mode 100644 crates/oxc_linter/src/rules/react/jsx_max_depth.rs
 create mode 100644 crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap

diff --git a/apps/oxlint/fixtures/tsgolint_fix/fix.ts b/apps/oxlint/fixtures/tsgolint_fix/fix.ts
index cd1059e4b9761..eb625ccf188e6 100644
--- a/apps/oxlint/fixtures/tsgolint_fix/fix.ts
+++ b/apps/oxlint/fixtures/tsgolint_fix/fix.ts
@@ -1,6 +1,6 @@
 // This file has a fixable tsgolint error: no-unnecessary-type-assertion
 // The type assertion `as string` is unnecessary because str is already a string
 const str: string = 'hello';
-const redundant = str as string;
+const redundant = str  ;
 
 export { redundant };
diff --git a/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware --silent@oxlint.snap b/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware --silent@oxlint.snap
index f741c06ce2202..bd486930c3fc7 100644
--- a/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware --silent@oxlint.snap	
+++ b/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware --silent@oxlint.snap	
@@ -5,9 +5,6 @@ source: apps/oxlint/src/tester.rs
 arguments: --type-aware --silent
 working directory: fixtures/tsgolint
 ----------
-
-Found 0 warnings and 50 errors.
-Finished in <variable>ms on 45 files using 1 threads.
-----------
-CLI result: LintFoundErrors
+Error running tsgolint: "exit status: exit status: 2"----------
+CLI result: TsGoLintError
 ----------
diff --git a/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware@oxlint.snap b/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware@oxlint.snap
index a0692950c286e..a9b2b41f42b3f 100644
--- a/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware@oxlint.snap
+++ b/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware@oxlint.snap
@@ -5,391 +5,6 @@ source: apps/oxlint/src/tester.rs
 arguments: --type-aware
 working directory: fixtures/tsgolint
 ----------
-
-  x typescript-eslint(await-thenable): Unexpected `await` of a non-Promise (non-"Thenable") value.
-   ,-[await-thenable.ts:1:1]
- 1 | await 12;
-   : ^^^^^^^^
- 2 | 
-   `----
-
-  x typescript-eslint(no-array-delete): Using the `delete` operator with an array expression is unsafe.
-   ,-[no-array-delete.ts:2:1]
- 1 | declare const arr: number[];
- 2 | delete arr[0];
-   : ^^^^^^^^^^^^^
-   `----
-
-  x typescript-eslint(no-base-to-string): '({})' will use Object's default stringification format ('[object Object]') when stringified.
-   ,-[no-base-to-string.ts:1:1]
- 1 | ({}).toString();
-   : ^^^^
-   `----
-
-  x typescript-eslint(no-meaningless-void-operator): void operator shouldn't be used on void; it should convey that a return value is being ignored
-   ,-[no-confusing-void-expression.ts:2:19]
- 1 | declare function bar(): void;
- 2 | const foo = () => void bar();
-   :                   ^^^^^^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(no-confusing-void-expression): Placing a void expression inside another expression is forbidden. Move it to its own statement instead.
-   ,-[no-confusing-void-expression.ts:2:24]
- 1 | declare function bar(): void;
- 2 | const foo = () => void bar();
-   :                        ^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(no-deprecated): `getVersion` is deprecated. Use apiV2 instead.
-   ,-[no-deprecated.ts:8:1]
- 7 | 
- 8 | getVersion();
-   : ^^^^^^^^^^
-   `----
-
-  x typescript-eslint(no-duplicate-type-constituents): Union type constituent is duplicated with  'A'.
-   ,-[no-duplicate-type-constituents.ts:1:17]
- 1 | type T1 = 'A' | 'A';
-   :                 ^^^
- 2 | 
-   `----
-
-  x typescript-eslint(no-confusing-void-expression): Returning a void expression from an arrow function shorthand is forbidden. Please add braces to the arrow function.
-   ,-[no-floating-promises.ts:1:51]
- 1 | const promise = new Promise((resolve, _reject) => resolve("value"));
-   :                                                   ^^^^^^^^^^^^^^^^
- 2 | promise;
-   `----
-
-  x typescript-eslint(no-floating-promises): Promises must be awaited.
-   ,-[no-floating-promises.ts:2:1]
- 1 | const promise = new Promise((resolve, _reject) => resolve("value"));
- 2 | promise;
-   : ^^^^^^^^
- 3 | 
-   `----
-  help: The promise must end with a call to .catch, or end with a call to .then with a rejection handler, or be explicitly marked as ignored with the `void` operator.
-
-  x typescript-eslint(no-for-in-array): For-in loops over arrays skips holes, returns indices as strings, and may visit the prototype chain or other enumerable properties. Use a more robust
-  | iteration method such as for-of or array.forEach instead.
-   ,-[no-for-in-array.ts:2:1]
- 1 | const arr = [1, 2, 3];
- 2 | for (const i in arr) {
-   : ^^^^^^^^^^^^^^^^^^^^
- 3 |   console.log(arr[i]);
-   `----
-
-  x typescript-eslint(no-implied-eval): Implied eval. Consider passing a function.
-   ,-[no-implied-eval.ts:1:12]
- 1 | setTimeout('alert("Hi!");', 100);
-   :            ^^^^^^^^^^^^^^^
-   `----
-
-  x typescript-eslint(no-meaningless-void-operator): void operator shouldn't be used on void; it should convey that a return value is being ignored
-   ,-[no-meaningless-void-operator.ts:4:1]
- 3 | }
- 4 | void foo();
-   : ^^^^^^^^^^
- 5 | 
-   `----
-
-  x typescript-eslint(no-confusing-void-expression): Placing a void expression inside another expression is forbidden. Move it to its own statement instead.
-   ,-[no-meaningless-void-operator.ts:4:6]
- 3 | }
- 4 | void foo();
-   :      ^^^^^
- 5 | 
-   `----
-
-  x typescript-eslint(no-misused-spread): Using the spread operator on Promise in an object can cause unexpected behavior. Did you forget to await the promise?
-   ,-[no-misused-spread.ts:2:25]
- 1 | declare const promise: Promise<number>;
- 2 | const spreadPromise = { ...promise };
-   :                         ^^^^^^^^^^
-   `----
-
-  x typescript-eslint(no-mixed-enums): Mixing number and string enums can be confusing.
-   ,-[no-mixed-enums.ts:3:12]
- 2 |   Open = 1,
- 3 |   Closed = 'closed',
-   :            ^^^^^^^^
- 4 | }
-   `----
-
-  x typescript-eslint(no-redundant-type-constituents): 'unknown' overrides all other types in this union type.
-   ,-[no-redundant-type-constituents.ts:1:20]
- 1 | type T1 = string | unknown;
-   :                    ^^^^^^^
-   `----
-
-  x typescript-eslint(no-unnecessary-boolean-literal-compare): This expression unnecessarily compares a boolean value to a boolean instead of using it directly.
-   ,-[no-unnecessary-boolean-literal-compare.ts:2:5]
- 1 | declare const someCondition: boolean;
- 2 | if (someCondition === true) {
-   :     ^^^^^^^^^^^^^^^^^^^^^^
- 3 | }
-   `----
-
-  x typescript-eslint(no-unnecessary-type-arguments): This is the default value for this type parameter, so it can be omitted.
-   ,-[no-unnecessary-type-arguments.ts:4:25]
- 3 | }
- 4 | const result = identity<string>('hello');
-   :                         ^^^^^^
-   `----
-
-  x typescript-eslint(no-unnecessary-type-assertion): This assertion is unnecessary since it does not change the type of the expression.
-   ,-[no-unnecessary-type-assertion.ts:2:19]
- 1 | const str: string = 'hello';
- 2 | const redundant = str as string;
-   :                   ^^^^^^^^^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(no-unsafe-argument): Unsafe argument of type any assigned to a parameter of type string.
-   ,-[no-unsafe-argument.ts:3:13]
- 2 | function takesString(str: string): void {}
- 3 | takesString(anyValue);
-   :             ^^^^^^^^
- 4 | 
-   `----
-
-  x typescript-eslint(no-unsafe-assignment): Unsafe assignment of an any value.
-   ,-[no-unsafe-assignment.ts:2:7]
- 1 | declare const anyValue: any;
- 2 | const str: string = anyValue;
-   :       ^^^^^^^^^^^^^^^^^^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(no-unsafe-call): Unsafe call of a(n) `any` typed value.
-   ,-[no-unsafe-call.ts:2:1]
- 1 | declare const anyValue: any;
- 2 | anyValue();
-   : ^^^^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(no-unsafe-enum-comparison): The two values in this comparison do not have a shared enum type.
-    ,-[no-unsafe-enum-comparison.ts:9:20]
-  8 | }
-  9 | const comparison = Status.Open === Color.Red;
-    :                    ^^^^^^^^^^^^^^^^^^^^^^^^^
- 10 | 
-    `----
-
-  x typescript-eslint(no-unsafe-member-access): Unsafe member access .foo on an `any` value.
-   ,-[no-unsafe-member-access.ts:2:10]
- 1 | declare const anyValue: any;
- 2 | anyValue.foo;
-   :          ^^^
- 3 | 
-   `----
-
-  x typescript-eslint(no-unsafe-return): Unsafe return of a value of type `any`.
-   ,-[no-unsafe-return.ts:3:3]
- 2 | function getString(): string {
- 3 |   return anyValue;
-   :   ^^^^^^^^^^^^^^^^
- 4 | }
-   `----
-
-  x typescript-eslint(no-unsafe-assignment): Unsafe assignment of an any value.
-   ,-[no-unsafe-type-assertion.ts:2:7]
- 1 | declare const value: unknown;
- 2 | const str = value as any;
-   :       ^^^^^^^^^^^^^^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(no-unsafe-type-assertion): Unsafe assertion to `any` detected: consider using a more specific type to ensure safety.
-   ,-[no-unsafe-type-assertion.ts:2:13]
- 1 | declare const value: unknown;
- 2 | const str = value as any;
-   :             ^^^^^^^^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(no-unsafe-type-assertion): Unsafe type assertion: type 'string' is more narrow than the original type.
-   ,-[non-nullable-type-assertion-style.ts:2:17]
- 1 | declare const value: string | null;
- 2 | const result1 = value as string;
-   :                 ^^^^^^^^^^^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(non-nullable-type-assertion-style): Use a ! assertion to more succinctly remove null and undefined from the type.
-   ,-[non-nullable-type-assertion-style.ts:2:17]
- 1 | declare const value: string | null;
- 2 | const result1 = value as string;
-   :                 ^^^^^^^^^^^^^^^
- 3 | 
-   `----
-
-  x ]8;;https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-debugger.html\eslint(no-debugger)]8;;\: `debugger` statement is not allowed
-   ,-[non-tsgolint.ts:1:1]
- 1 | debugger;
-   : ^^^^^^^^^
- 2 | 
-   `----
-  help: Remove the debugger statement
-
-  x typescript-eslint(only-throw-error): Expected an error object to be thrown.
-   ,-[only-throw-error.ts:1:7]
- 1 | throw 'error';
-   :       ^^^^^^^
- 2 | 
-   `----
-
-  x typescript-eslint(prefer-includes): Use 'includes()' method instead.
-   ,-[prefer-includes.ts:2:5]
- 1 | const text = 'hello';
- 2 | if (text.indexOf('h') !== -1) {
-   :     ^^^^^^^^^^^^^^^^^^^^^^^^
- 3 | }
-   `----
-
-  x typescript-eslint(prefer-includes): Use 'includes()' method instead.
-   ,-[prefer-includes.ts:6:5]
- 5 | const items = [1, 2];
- 6 | if (items.indexOf(1) !== -1) {
-   :     ^^^^^^^^^^^^^^^^^^^^^^^
- 7 | }
-   `----
-
-  x typescript-eslint(prefer-includes): Use `String#includes()` method with a string instead.
-    ,-[prefer-includes.ts:9:5]
-  8 | 
-  9 | if (/test/.test(text)) {
-    :     ^^^^^^^^^^^^^^^^^
- 10 | }
-    `----
-
-  x typescript-eslint(prefer-nullish-coalescing): Prefer using nullish coalescing operator (`??`) instead of a ternary expression, as it is simpler to read.
-   ,-[prefer-nullish-coalescing.ts:2:23]
- 1 | declare const nullableString: string | null;
- 2 | const nullishResult = nullableString !== null && nullableString !== undefined ? nullableString : 'default';
-   :                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-   `----
-
-  x typescript-eslint(prefer-promise-reject-errors): Expected the Promise rejection reason to be an Error.
-   ,-[prefer-promise-reject-errors.ts:1:1]
- 1 | Promise.reject('error');
-   : ^^^^^^^^^^^^^^^^^^^^^^^
-   `----
-
-  x typescript-eslint(no-floating-promises): Promises must be awaited.
-   ,-[prefer-promise-reject-errors.ts:1:1]
- 1 | Promise.reject('error');
-   : ^^^^^^^^^^^^^^^^^^^^^^^^
-   `----
-  help: The promise must end with a call to .catch, or end with a call to .then with a rejection handler, or be explicitly marked as ignored with the `void` operator.
-
-  x typescript-eslint(no-unnecessary-type-assertion): This assertion is unnecessary since it does not change the type of the expression.
-   ,-[prefer-reduce-type-parameter.ts:2:13]
- 1 | const numbers = [1, 2, 3];
- 2 | const sum = numbers.reduce((acc, val) => acc + val, 0) as number;
-   :             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(prefer-return-this-type): Use `this` type instead.
-   ,-[prefer-return-this-type.ts:3:28]
- 2 |   private value: string = '';
- 3 |   setValue(value: string): Builder {
-   :                            ^^^^^^^
- 4 |     this.value = value;
-   `----
-
-  x typescript-eslint(promise-function-async): Functions that return promises must be async.
-   ,-[promise-function-async.ts:2:1]
- 1 |     declare function fetch(url: string): Promise<Response>;
- 2 | ,-> function fetchData(): Promise<string> {
- 3 | |     return fetch('/api/data').then(res => res.text());
- 4 | `-> }
- 5 |     
-   `----
-
-  x typescript-eslint(promise-function-async): Functions that return promises must be async.
-   ,-[promise-function-async.ts:3:34]
- 2 | function fetchData(): Promise<string> {
- 3 |   return fetch('/api/data').then(res => res.text());
-   :                                  ^^^^^^^^^^^^^^^^^
- 4 | }
-   `----
-
-  x typescript-eslint(related-getter-setter-pairs): `get()` type should be assignable to its equivalent `set()` type.
-   ,-[related-getter-setter-pairs.ts:3:16]
- 2 |   private _value: number = 0;
- 3 |   get value(): string {
-   :                ^^^^^^
- 4 |     return this._value.toString();
-   `----
-
-  x typescript-eslint(require-array-sort-compare): Require 'compare' argument.
-   ,-[require-array-sort-compare.ts:2:1]
- 1 | const numbers = [3, 1, 4, 1, 5];
- 2 | numbers.sort();
-   : ^^^^^^^^^^^^^^
- 3 | 
-   `----
-
-  x typescript-eslint(no-unsafe-assignment): Unsafe assignment of an any value.
-   ,-[restrict-plus-operands.ts:1:5]
- 1 | let foo = 1n + 1;
-   :     ^^^^^^^^^^^^
-   `----
-
-  x typescript-eslint(restrict-plus-operands): Numeric '+' operations must either be both bigints or both numbers. Got `bigint` + `number`.
-   ,-[restrict-plus-operands.ts:1:11]
- 1 | let foo = 1n + 1;
-   :           ^^^^^^
-   `----
-
-  x typescript-eslint(no-base-to-string): 'obj' will use Object's default stringification format ('[object Object]') when stringified.
-   ,-[restrict-template-expressions.ts:2:24]
- 1 | declare const obj: object;
- 2 | const str1 = `Value: ${obj}`;
-   :                        ^^^
- 3 | 
-   `----
-
-  x typescript-eslint(restrict-template-expressions): Invalid type "object" of template literal expression.
-   ,-[restrict-template-expressions.ts:2:24]
- 1 | declare const obj: object;
- 2 | const str1 = `Value: ${obj}`;
-   :                        ^^^
- 3 | 
-   `----
-
-  x typescript-eslint(switch-exhaustiveness-check): Switch is not exhaustive
-   ,-[switch-exhaustiveness-check.ts:3:11]
- 2 | function handleStatus(status: Status) {
- 3 |   switch (status) {
-   :           ^^^^^^
- 4 |     case 'pending':
-   `----
-
-  x ]8;;https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-debugger.html\eslint(no-debugger)]8;;\: `debugger` statement is not allowed
-   ,-[test.svelte:2:2]
- 1 | <script>
- 2 |     debugger;
-   :     ^^^^^^^^^
- 3 | 
-   `----
-  help: Remove the debugger statement
-
-  x typescript-eslint(unbound-method): Avoid referencing unbound methods which may cause unintentional scoping of `this`.
-  | If your function does not access `this`, you can annotate it with `this: void`, or consider using an arrow function instead.
-   ,-[unbound-method.ts:9:19]
- 8 | const calc = new Calculator();
- 9 | const addMethod = calc.add;
-   :                   ^^^^^^^^
-   `----
-
-Found 0 warnings and 50 errors.
-Finished in <variable>ms on 45 files using 1 threads.
-----------
-CLI result: LintFoundErrors
+Error running tsgolint: "exit status: exit status: 2"----------
+CLI result: TsGoLintError
 ----------
diff --git a/apps/oxlint/src/snapshots/fixtures__tsgolint_rule_options_--type-aware@oxlint.snap b/apps/oxlint/src/snapshots/fixtures__tsgolint_rule_options_--type-aware@oxlint.snap
index 2a5717793b952..575e9298a64be 100644
--- a/apps/oxlint/src/snapshots/fixtures__tsgolint_rule_options_--type-aware@oxlint.snap
+++ b/apps/oxlint/src/snapshots/fixtures__tsgolint_rule_options_--type-aware@oxlint.snap
@@ -22,6 +22,22 @@ working directory: fixtures/tsgolint_rule_options
  48 | 
     `----
 
+  x typescript-eslint(no-unnecessary-type-assertion): This assertion is unnecessary since it does not change the type of the expression.
+    ,-[test.ts:51:22]
+ 50 | // When checkLiteralConstAssertions is true, this SHOULD error
+ 51 | const literalConst = 'hello' as const;
+    :                      ^^^^^^^^^^^^^^^^
+ 52 | 
+    `----
+
+  x typescript-eslint(no-unsafe-member-access): Unsafe member access .foo on an `any` value.
+    ,-[test.ts:56:34]
+ 55 | // This should NOT error because allowOptionalChaining is true and we use ?.
+ 56 | const optionalAccess = anyValue?.foo;
+    :                                  ^^^
+ 57 | // This SHOULD error because it's not using optional chaining
+    `----
+
   x typescript-eslint(no-unsafe-member-access): Unsafe member access .bar on an `any` value.
     ,-[test.ts:58:31]
  57 | // This SHOULD error because it's not using optional chaining
@@ -30,15 +46,7 @@ working directory: fixtures/tsgolint_rule_options
  59 | 
     `----
 
-  x typescript-eslint(no-base-to-string): 'unknownValue' may use Object's default stringification format ('[object Object]') when stringified.
-    ,-[test.ts:63:20]
- 62 | // This SHOULD error because checkUnknown is true
- 63 | const unknownStr = unknownValue.toString();
-    :                    ^^^^^^^^^^^^
- 64 | 
-    `----
-
-Found 0 warnings and 4 errors.
+Found 0 warnings and 5 errors.
 Finished in <variable>ms on 1 file using 1 threads.
 ----------
 CLI result: LintFoundErrors
diff --git a/crates/oxc_codegen/tests/integration/snapshots/stacktrace_is_correct.snap b/crates/oxc_codegen/tests/integration/snapshots/stacktrace_is_correct.snap
index e5e11e74b71e9..a762ca0b340cb 100644
--- a/crates/oxc_codegen/tests/integration/snapshots/stacktrace_is_correct.snap
+++ b/crates/oxc_codegen/tests/integration/snapshots/stacktrace_is_correct.snap
@@ -1,7 +1,7 @@
 ---
 source: crates/oxc_codegen/tests/integration/sourcemap.rs
 ---
-Node.js version: v24.10.0
+Node.js version: v24.12.0
 
 ## Input
 const fn = () => {
diff --git a/crates/oxc_linter/src/generated/rule_runner_impls.rs b/crates/oxc_linter/src/generated/rule_runner_impls.rs
index 73bfffbc465f0..239fd2c5ea051 100644
--- a/crates/oxc_linter/src/generated/rule_runner_impls.rs
+++ b/crates/oxc_linter/src/generated/rule_runner_impls.rs
@@ -2312,6 +2312,12 @@ impl RuleRunner for crate::rules::react::jsx_key::JsxKey {
     const RUN_FUNCTIONS: RuleRunFunctionsImplemented = RuleRunFunctionsImplemented::Run;
 }
 
+impl RuleRunner for crate::rules::react::jsx_max_depth::JsxMaxDepth {
+    const NODE_TYPES: Option<&AstTypesBitset> =
+        Some(&AstTypesBitset::from_types(&[AstType::JSXElement, AstType::JSXFragment]));
+    const RUN_FUNCTIONS: RuleRunFunctionsImplemented = RuleRunFunctionsImplemented::Run;
+}
+
 impl RuleRunner for crate::rules::react::jsx_no_comment_textnodes::JsxNoCommentTextnodes {
     const NODE_TYPES: Option<&AstTypesBitset> =
         Some(&AstTypesBitset::from_types(&[AstType::JSXText]));
diff --git a/crates/oxc_linter/src/rules.rs b/crates/oxc_linter/src/rules.rs
index a1df7ec7f35bc..917ce900b8f3c 100644
--- a/crates/oxc_linter/src/rules.rs
+++ b/crates/oxc_linter/src/rules.rs
@@ -367,6 +367,7 @@ pub(crate) mod react {
     pub mod jsx_fragments;
     pub mod jsx_handler_names;
     pub mod jsx_key;
+    pub mod jsx_max_depth;
     pub mod jsx_no_comment_textnodes;
     pub mod jsx_no_duplicate_props;
     pub mod jsx_no_script_url;
@@ -1067,6 +1068,7 @@ oxc_macros::declare_all_lint_rules! {
     react::jsx_curly_brace_presence,
     react::jsx_handler_names,
     react::jsx_key,
+    react::jsx_max_depth,
     react::jsx_no_comment_textnodes,
     react::jsx_no_duplicate_props,
     react::jsx_no_script_url,
diff --git a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
new file mode 100644
index 0000000000000..c67047e042937
--- /dev/null
+++ b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
@@ -0,0 +1,206 @@
+use schemars::JsonSchema;
+use serde::{Deserialize, Serialize};
+
+use oxc_ast::AstKind;
+use oxc_diagnostics::OxcDiagnostic;
+use oxc_macros::declare_oxc_lint;
+use oxc_span::{GetSpan, Span};
+
+use crate::{
+    AstNode,
+    context::{ContextHost, LintContext},
+    rule::{DefaultRuleConfig, Rule},
+};
+
+fn jsx_max_depth_diagnostic(depth: usize, max: usize, span: Span) -> OxcDiagnostic {
+    OxcDiagnostic::warn(format!(
+        "JSX nesting depth of {depth} exceeds the configured maximum of {max}"
+    ))
+    .with_label(span)
+}
+
+#[derive(Debug, Clone, Default)]
+pub struct JsxMaxDepth(Box<JsxMaxDepthConfig>);
+
+impl std::ops::Deref for JsxMaxDepth {
+    type Target = JsxMaxDepthConfig;
+
+    fn deref(&self) -> &Self::Target {
+        &self.0
+    }
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize, JsonSchema)]
+#[serde(default)]
+pub struct JsxMaxDepthConfig {
+    #[serde(default = "JsxMaxDepthConfig::default_max")]
+    pub max: usize,
+}
+
+impl JsxMaxDepthConfig {
+    const fn default_max() -> usize {
+        2
+    }
+}
+
+impl Default for JsxMaxDepthConfig {
+    fn default() -> Self {
+        Self { max: Self::default_max() }
+    }
+}
+
+declare_oxc_lint!(
+    /// ### What it does
+    ///
+    /// Enforces a maximum depth for nested JSX elements and fragments.
+    ///
+    /// ### Why is this bad?
+    ///
+    /// Excessively nested JSX makes components harder to read and maintain.
+    ///
+    /// ### Examples
+    ///
+    /// Examples of **incorrect** code for this rule:
+    /// ```jsx
+    /// const Component = () => (
+    ///   <div>
+    ///     <div>
+    ///       <div>
+    ///         <span />
+    ///       </div>
+    ///     </div>
+    ///   </div>
+    /// );
+    /// ```
+    ///
+    /// Examples of **correct** code for this rule:
+    /// ```jsx
+    /// const Component = () => (
+    ///   <div>
+    ///     <div>
+    ///       <span />
+    ///     </div>
+    ///   </div>
+    /// );
+    /// ```
+    ///
+    /// ### Options
+    ///
+    /// `react/jsx-max-depth: [<enabled>, { "max": <number> }]`
+    ///
+    /// The `max` option defaults to `2`.
+    JsxMaxDepth,
+    react,
+    style,
+    config = JsxMaxDepthConfig,
+);
+
+impl Rule for JsxMaxDepth {
+    fn from_configuration(value: serde_json::Value) -> Self {
+        let config = serde_json::from_value::<DefaultRuleConfig<JsxMaxDepthConfig>>(value)
+            .map(DefaultRuleConfig::into_inner)
+            .unwrap_or_default();
+
+        Self(Box::new(config))
+    }
+
+    fn run<'a>(&self, node: &AstNode<'a>, ctx: &LintContext<'a>) {
+        match node.kind() {
+            AstKind::JSXElement(_) | AstKind::JSXFragment(_) => {}
+            _ => return,
+        }
+
+        let depth = jsx_ancestor_depth(node, ctx);
+        if depth > self.max {
+            ctx.diagnostic(jsx_max_depth_diagnostic(depth, self.max, node.span()));
+        }
+    }
+
+    fn should_run(&self, ctx: &ContextHost) -> bool {
+        ctx.source_type().is_jsx()
+    }
+}
+
+fn jsx_ancestor_depth(node: &AstNode<'_>, ctx: &LintContext<'_>) -> usize {
+    ctx.nodes()
+        .ancestors(node.id())
+        .filter(|ancestor| {
+            matches!(ancestor.kind(), AstKind::JSXElement(_) | AstKind::JSXFragment(_))
+        })
+        .count()
+}
+
+#[test]
+fn test() {
+    use crate::tester::Tester;
+
+    let pass = vec![
+        (
+            r#"
+            const Component = () => (
+              <div>
+                <section />
+              </div>
+            );
+            "#,
+            None,
+        ),
+        (
+            r#"
+            const Component = () => (
+              <div>
+                <div>
+                  <div />
+                </div>
+              </div>
+            );
+            "#,
+            Some(serde_json::json!([{ "max": 3 }])),
+        ),
+        (
+            r#"
+            const Component = () => (
+              <>
+                <div>
+                  <span />
+                </div>
+              </>
+            );
+            "#,
+            None,
+        ),
+    ];
+
+    let fail = vec![
+        (
+            r#"
+            const Component = () => (
+              <div>
+                <div>
+                  <div>
+                    <span />
+                  </div>
+                </div>
+              </div>
+            );
+            "#,
+            None,
+        ),
+        (
+            r#"
+            const Component = () => (
+              <>
+                <div>
+                  <>
+                    <div />
+                  </>
+                </div>
+              </>
+            );
+            "#,
+            Some(serde_json::json!([{ "max": 2 }])),
+        ),
+    ];
+
+    Tester::new(JsxMaxDepth::NAME, JsxMaxDepth::PLUGIN, pass, fail).test_and_snapshot();
+}
diff --git a/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap b/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap
new file mode 100644
index 0000000000000..58f19fc4094f4
--- /dev/null
+++ b/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap
@@ -0,0 +1,18 @@
+---
+source: crates/oxc_linter/src/tester.rs
+---
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 3 exceeds the configured maximum of 2
+   ╭─[jsx_max_depth.tsx:6:21]
+ 5 │                   <div>
+ 6 │                     <span />
+   ·                     ────────
+ 7 │                   </div>
+   ╰────
+
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 3 exceeds the configured maximum of 2
+   ╭─[jsx_max_depth.tsx:6:21]
+ 5 │                   <>
+ 6 │                     <div />
+   ·                     ───────
+ 7 │                   </>
+   ╰────

From 51e3ec9d1be3d9190acca398a912827a4aab2597 Mon Sep 17 00:00:00 2001
From: Kenzo-Wada <used.acormhib@gmail.com>
Date: Tue, 23 Dec 2025 00:32:09 +0900
Subject: [PATCH 02/11] update

---
 apps/oxlint/fixtures/tsgolint_fix/fix.ts      |  2 +-
 apps/oxlint/src/lint.rs                       |  2 +-
 .../snapshots/stacktrace_is_correct.snap      |  2 +-
 .../src/rules/react/jsx_max_depth.rs          | 20 +++++++++----------
 4 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/apps/oxlint/fixtures/tsgolint_fix/fix.ts b/apps/oxlint/fixtures/tsgolint_fix/fix.ts
index eb625ccf188e6..cd1059e4b9761 100644
--- a/apps/oxlint/fixtures/tsgolint_fix/fix.ts
+++ b/apps/oxlint/fixtures/tsgolint_fix/fix.ts
@@ -1,6 +1,6 @@
 // This file has a fixable tsgolint error: no-unnecessary-type-assertion
 // The type assertion `as string` is unnecessary because str is already a string
 const str: string = 'hello';
-const redundant = str  ;
+const redundant = str as string;
 
 export { redundant };
diff --git a/apps/oxlint/src/lint.rs b/apps/oxlint/src/lint.rs
index 595b91d1a8092..9a414c4065178 100644
--- a/apps/oxlint/src/lint.rs
+++ b/apps/oxlint/src/lint.rs
@@ -1461,7 +1461,7 @@ export { redundant };
             "// This file has a fixable tsgolint error: no-unnecessary-type-assertion
 // The type assertion `as string` is unnecessary because str is already a string
 const str: string = 'hello';
-const redundant = str;
+const redundant = str  ;
 
 export { redundant };
 ",
diff --git a/crates/oxc_codegen/tests/integration/snapshots/stacktrace_is_correct.snap b/crates/oxc_codegen/tests/integration/snapshots/stacktrace_is_correct.snap
index a762ca0b340cb..e5e11e74b71e9 100644
--- a/crates/oxc_codegen/tests/integration/snapshots/stacktrace_is_correct.snap
+++ b/crates/oxc_codegen/tests/integration/snapshots/stacktrace_is_correct.snap
@@ -1,7 +1,7 @@
 ---
 source: crates/oxc_codegen/tests/integration/sourcemap.rs
 ---
-Node.js version: v24.12.0
+Node.js version: v24.10.0
 
 ## Input
 const fn = () => {
diff --git a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
index c67047e042937..417ea65c0a2ae 100644
--- a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
+++ b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
@@ -136,17 +136,17 @@ fn test() {
 
     let pass = vec![
         (
-            r#"
+            r"
             const Component = () => (
               <div>
                 <section />
               </div>
             );
-            "#,
+            ",
             None,
         ),
         (
-            r#"
+            r"
             const Component = () => (
               <div>
                 <div>
@@ -154,11 +154,11 @@ fn test() {
                 </div>
               </div>
             );
-            "#,
+            ",
             Some(serde_json::json!([{ "max": 3 }])),
         ),
         (
-            r#"
+            r"
             const Component = () => (
               <>
                 <div>
@@ -166,14 +166,14 @@ fn test() {
                 </div>
               </>
             );
-            "#,
+            ",
             None,
         ),
     ];
 
     let fail = vec![
         (
-            r#"
+            r"
             const Component = () => (
               <div>
                 <div>
@@ -183,11 +183,11 @@ fn test() {
                 </div>
               </div>
             );
-            "#,
+            ",
             None,
         ),
         (
-            r#"
+            r"
             const Component = () => (
               <>
                 <div>
@@ -197,7 +197,7 @@ fn test() {
                 </div>
               </>
             );
-            "#,
+            ",
             Some(serde_json::json!([{ "max": 2 }])),
         ),
     ];

From 210f064ed18878f0958f9177c91df7bf94806807 Mon Sep 17 00:00:00 2001
From: Kenzo-Wada <used.acormhib@gmail.com>
Date: Tue, 23 Dec 2025 00:44:44 +0900
Subject: [PATCH 03/11] update

---
 ...tsgolint_--type-aware --silent@oxlint.snap |   7 +-
 ...ixtures__tsgolint_--type-aware@oxlint.snap | 389 +++++++++++++++++-
 ...lint_rule_options_--type-aware@oxlint.snap |  26 +-
 3 files changed, 401 insertions(+), 21 deletions(-)

diff --git a/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware --silent@oxlint.snap b/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware --silent@oxlint.snap
index bd486930c3fc7..f741c06ce2202 100644
--- a/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware --silent@oxlint.snap	
+++ b/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware --silent@oxlint.snap	
@@ -5,6 +5,9 @@ source: apps/oxlint/src/tester.rs
 arguments: --type-aware --silent
 working directory: fixtures/tsgolint
 ----------
-Error running tsgolint: "exit status: exit status: 2"----------
-CLI result: TsGoLintError
+
+Found 0 warnings and 50 errors.
+Finished in <variable>ms on 45 files using 1 threads.
+----------
+CLI result: LintFoundErrors
 ----------
diff --git a/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware@oxlint.snap b/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware@oxlint.snap
index a9b2b41f42b3f..a0692950c286e 100644
--- a/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware@oxlint.snap
+++ b/apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware@oxlint.snap
@@ -5,6 +5,391 @@ source: apps/oxlint/src/tester.rs
 arguments: --type-aware
 working directory: fixtures/tsgolint
 ----------
-Error running tsgolint: "exit status: exit status: 2"----------
-CLI result: TsGoLintError
+
+  x typescript-eslint(await-thenable): Unexpected `await` of a non-Promise (non-"Thenable") value.
+   ,-[await-thenable.ts:1:1]
+ 1 | await 12;
+   : ^^^^^^^^
+ 2 | 
+   `----
+
+  x typescript-eslint(no-array-delete): Using the `delete` operator with an array expression is unsafe.
+   ,-[no-array-delete.ts:2:1]
+ 1 | declare const arr: number[];
+ 2 | delete arr[0];
+   : ^^^^^^^^^^^^^
+   `----
+
+  x typescript-eslint(no-base-to-string): '({})' will use Object's default stringification format ('[object Object]') when stringified.
+   ,-[no-base-to-string.ts:1:1]
+ 1 | ({}).toString();
+   : ^^^^
+   `----
+
+  x typescript-eslint(no-meaningless-void-operator): void operator shouldn't be used on void; it should convey that a return value is being ignored
+   ,-[no-confusing-void-expression.ts:2:19]
+ 1 | declare function bar(): void;
+ 2 | const foo = () => void bar();
+   :                   ^^^^^^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(no-confusing-void-expression): Placing a void expression inside another expression is forbidden. Move it to its own statement instead.
+   ,-[no-confusing-void-expression.ts:2:24]
+ 1 | declare function bar(): void;
+ 2 | const foo = () => void bar();
+   :                        ^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(no-deprecated): `getVersion` is deprecated. Use apiV2 instead.
+   ,-[no-deprecated.ts:8:1]
+ 7 | 
+ 8 | getVersion();
+   : ^^^^^^^^^^
+   `----
+
+  x typescript-eslint(no-duplicate-type-constituents): Union type constituent is duplicated with  'A'.
+   ,-[no-duplicate-type-constituents.ts:1:17]
+ 1 | type T1 = 'A' | 'A';
+   :                 ^^^
+ 2 | 
+   `----
+
+  x typescript-eslint(no-confusing-void-expression): Returning a void expression from an arrow function shorthand is forbidden. Please add braces to the arrow function.
+   ,-[no-floating-promises.ts:1:51]
+ 1 | const promise = new Promise((resolve, _reject) => resolve("value"));
+   :                                                   ^^^^^^^^^^^^^^^^
+ 2 | promise;
+   `----
+
+  x typescript-eslint(no-floating-promises): Promises must be awaited.
+   ,-[no-floating-promises.ts:2:1]
+ 1 | const promise = new Promise((resolve, _reject) => resolve("value"));
+ 2 | promise;
+   : ^^^^^^^^
+ 3 | 
+   `----
+  help: The promise must end with a call to .catch, or end with a call to .then with a rejection handler, or be explicitly marked as ignored with the `void` operator.
+
+  x typescript-eslint(no-for-in-array): For-in loops over arrays skips holes, returns indices as strings, and may visit the prototype chain or other enumerable properties. Use a more robust
+  | iteration method such as for-of or array.forEach instead.
+   ,-[no-for-in-array.ts:2:1]
+ 1 | const arr = [1, 2, 3];
+ 2 | for (const i in arr) {
+   : ^^^^^^^^^^^^^^^^^^^^
+ 3 |   console.log(arr[i]);
+   `----
+
+  x typescript-eslint(no-implied-eval): Implied eval. Consider passing a function.
+   ,-[no-implied-eval.ts:1:12]
+ 1 | setTimeout('alert("Hi!");', 100);
+   :            ^^^^^^^^^^^^^^^
+   `----
+
+  x typescript-eslint(no-meaningless-void-operator): void operator shouldn't be used on void; it should convey that a return value is being ignored
+   ,-[no-meaningless-void-operator.ts:4:1]
+ 3 | }
+ 4 | void foo();
+   : ^^^^^^^^^^
+ 5 | 
+   `----
+
+  x typescript-eslint(no-confusing-void-expression): Placing a void expression inside another expression is forbidden. Move it to its own statement instead.
+   ,-[no-meaningless-void-operator.ts:4:6]
+ 3 | }
+ 4 | void foo();
+   :      ^^^^^
+ 5 | 
+   `----
+
+  x typescript-eslint(no-misused-spread): Using the spread operator on Promise in an object can cause unexpected behavior. Did you forget to await the promise?
+   ,-[no-misused-spread.ts:2:25]
+ 1 | declare const promise: Promise<number>;
+ 2 | const spreadPromise = { ...promise };
+   :                         ^^^^^^^^^^
+   `----
+
+  x typescript-eslint(no-mixed-enums): Mixing number and string enums can be confusing.
+   ,-[no-mixed-enums.ts:3:12]
+ 2 |   Open = 1,
+ 3 |   Closed = 'closed',
+   :            ^^^^^^^^
+ 4 | }
+   `----
+
+  x typescript-eslint(no-redundant-type-constituents): 'unknown' overrides all other types in this union type.
+   ,-[no-redundant-type-constituents.ts:1:20]
+ 1 | type T1 = string | unknown;
+   :                    ^^^^^^^
+   `----
+
+  x typescript-eslint(no-unnecessary-boolean-literal-compare): This expression unnecessarily compares a boolean value to a boolean instead of using it directly.
+   ,-[no-unnecessary-boolean-literal-compare.ts:2:5]
+ 1 | declare const someCondition: boolean;
+ 2 | if (someCondition === true) {
+   :     ^^^^^^^^^^^^^^^^^^^^^^
+ 3 | }
+   `----
+
+  x typescript-eslint(no-unnecessary-type-arguments): This is the default value for this type parameter, so it can be omitted.
+   ,-[no-unnecessary-type-arguments.ts:4:25]
+ 3 | }
+ 4 | const result = identity<string>('hello');
+   :                         ^^^^^^
+   `----
+
+  x typescript-eslint(no-unnecessary-type-assertion): This assertion is unnecessary since it does not change the type of the expression.
+   ,-[no-unnecessary-type-assertion.ts:2:19]
+ 1 | const str: string = 'hello';
+ 2 | const redundant = str as string;
+   :                   ^^^^^^^^^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(no-unsafe-argument): Unsafe argument of type any assigned to a parameter of type string.
+   ,-[no-unsafe-argument.ts:3:13]
+ 2 | function takesString(str: string): void {}
+ 3 | takesString(anyValue);
+   :             ^^^^^^^^
+ 4 | 
+   `----
+
+  x typescript-eslint(no-unsafe-assignment): Unsafe assignment of an any value.
+   ,-[no-unsafe-assignment.ts:2:7]
+ 1 | declare const anyValue: any;
+ 2 | const str: string = anyValue;
+   :       ^^^^^^^^^^^^^^^^^^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(no-unsafe-call): Unsafe call of a(n) `any` typed value.
+   ,-[no-unsafe-call.ts:2:1]
+ 1 | declare const anyValue: any;
+ 2 | anyValue();
+   : ^^^^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(no-unsafe-enum-comparison): The two values in this comparison do not have a shared enum type.
+    ,-[no-unsafe-enum-comparison.ts:9:20]
+  8 | }
+  9 | const comparison = Status.Open === Color.Red;
+    :                    ^^^^^^^^^^^^^^^^^^^^^^^^^
+ 10 | 
+    `----
+
+  x typescript-eslint(no-unsafe-member-access): Unsafe member access .foo on an `any` value.
+   ,-[no-unsafe-member-access.ts:2:10]
+ 1 | declare const anyValue: any;
+ 2 | anyValue.foo;
+   :          ^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(no-unsafe-return): Unsafe return of a value of type `any`.
+   ,-[no-unsafe-return.ts:3:3]
+ 2 | function getString(): string {
+ 3 |   return anyValue;
+   :   ^^^^^^^^^^^^^^^^
+ 4 | }
+   `----
+
+  x typescript-eslint(no-unsafe-assignment): Unsafe assignment of an any value.
+   ,-[no-unsafe-type-assertion.ts:2:7]
+ 1 | declare const value: unknown;
+ 2 | const str = value as any;
+   :       ^^^^^^^^^^^^^^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(no-unsafe-type-assertion): Unsafe assertion to `any` detected: consider using a more specific type to ensure safety.
+   ,-[no-unsafe-type-assertion.ts:2:13]
+ 1 | declare const value: unknown;
+ 2 | const str = value as any;
+   :             ^^^^^^^^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(no-unsafe-type-assertion): Unsafe type assertion: type 'string' is more narrow than the original type.
+   ,-[non-nullable-type-assertion-style.ts:2:17]
+ 1 | declare const value: string | null;
+ 2 | const result1 = value as string;
+   :                 ^^^^^^^^^^^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(non-nullable-type-assertion-style): Use a ! assertion to more succinctly remove null and undefined from the type.
+   ,-[non-nullable-type-assertion-style.ts:2:17]
+ 1 | declare const value: string | null;
+ 2 | const result1 = value as string;
+   :                 ^^^^^^^^^^^^^^^
+ 3 | 
+   `----
+
+  x ]8;;https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-debugger.html\eslint(no-debugger)]8;;\: `debugger` statement is not allowed
+   ,-[non-tsgolint.ts:1:1]
+ 1 | debugger;
+   : ^^^^^^^^^
+ 2 | 
+   `----
+  help: Remove the debugger statement
+
+  x typescript-eslint(only-throw-error): Expected an error object to be thrown.
+   ,-[only-throw-error.ts:1:7]
+ 1 | throw 'error';
+   :       ^^^^^^^
+ 2 | 
+   `----
+
+  x typescript-eslint(prefer-includes): Use 'includes()' method instead.
+   ,-[prefer-includes.ts:2:5]
+ 1 | const text = 'hello';
+ 2 | if (text.indexOf('h') !== -1) {
+   :     ^^^^^^^^^^^^^^^^^^^^^^^^
+ 3 | }
+   `----
+
+  x typescript-eslint(prefer-includes): Use 'includes()' method instead.
+   ,-[prefer-includes.ts:6:5]
+ 5 | const items = [1, 2];
+ 6 | if (items.indexOf(1) !== -1) {
+   :     ^^^^^^^^^^^^^^^^^^^^^^^
+ 7 | }
+   `----
+
+  x typescript-eslint(prefer-includes): Use `String#includes()` method with a string instead.
+    ,-[prefer-includes.ts:9:5]
+  8 | 
+  9 | if (/test/.test(text)) {
+    :     ^^^^^^^^^^^^^^^^^
+ 10 | }
+    `----
+
+  x typescript-eslint(prefer-nullish-coalescing): Prefer using nullish coalescing operator (`??`) instead of a ternary expression, as it is simpler to read.
+   ,-[prefer-nullish-coalescing.ts:2:23]
+ 1 | declare const nullableString: string | null;
+ 2 | const nullishResult = nullableString !== null && nullableString !== undefined ? nullableString : 'default';
+   :                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+   `----
+
+  x typescript-eslint(prefer-promise-reject-errors): Expected the Promise rejection reason to be an Error.
+   ,-[prefer-promise-reject-errors.ts:1:1]
+ 1 | Promise.reject('error');
+   : ^^^^^^^^^^^^^^^^^^^^^^^
+   `----
+
+  x typescript-eslint(no-floating-promises): Promises must be awaited.
+   ,-[prefer-promise-reject-errors.ts:1:1]
+ 1 | Promise.reject('error');
+   : ^^^^^^^^^^^^^^^^^^^^^^^^
+   `----
+  help: The promise must end with a call to .catch, or end with a call to .then with a rejection handler, or be explicitly marked as ignored with the `void` operator.
+
+  x typescript-eslint(no-unnecessary-type-assertion): This assertion is unnecessary since it does not change the type of the expression.
+   ,-[prefer-reduce-type-parameter.ts:2:13]
+ 1 | const numbers = [1, 2, 3];
+ 2 | const sum = numbers.reduce((acc, val) => acc + val, 0) as number;
+   :             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(prefer-return-this-type): Use `this` type instead.
+   ,-[prefer-return-this-type.ts:3:28]
+ 2 |   private value: string = '';
+ 3 |   setValue(value: string): Builder {
+   :                            ^^^^^^^
+ 4 |     this.value = value;
+   `----
+
+  x typescript-eslint(promise-function-async): Functions that return promises must be async.
+   ,-[promise-function-async.ts:2:1]
+ 1 |     declare function fetch(url: string): Promise<Response>;
+ 2 | ,-> function fetchData(): Promise<string> {
+ 3 | |     return fetch('/api/data').then(res => res.text());
+ 4 | `-> }
+ 5 |     
+   `----
+
+  x typescript-eslint(promise-function-async): Functions that return promises must be async.
+   ,-[promise-function-async.ts:3:34]
+ 2 | function fetchData(): Promise<string> {
+ 3 |   return fetch('/api/data').then(res => res.text());
+   :                                  ^^^^^^^^^^^^^^^^^
+ 4 | }
+   `----
+
+  x typescript-eslint(related-getter-setter-pairs): `get()` type should be assignable to its equivalent `set()` type.
+   ,-[related-getter-setter-pairs.ts:3:16]
+ 2 |   private _value: number = 0;
+ 3 |   get value(): string {
+   :                ^^^^^^
+ 4 |     return this._value.toString();
+   `----
+
+  x typescript-eslint(require-array-sort-compare): Require 'compare' argument.
+   ,-[require-array-sort-compare.ts:2:1]
+ 1 | const numbers = [3, 1, 4, 1, 5];
+ 2 | numbers.sort();
+   : ^^^^^^^^^^^^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(no-unsafe-assignment): Unsafe assignment of an any value.
+   ,-[restrict-plus-operands.ts:1:5]
+ 1 | let foo = 1n + 1;
+   :     ^^^^^^^^^^^^
+   `----
+
+  x typescript-eslint(restrict-plus-operands): Numeric '+' operations must either be both bigints or both numbers. Got `bigint` + `number`.
+   ,-[restrict-plus-operands.ts:1:11]
+ 1 | let foo = 1n + 1;
+   :           ^^^^^^
+   `----
+
+  x typescript-eslint(no-base-to-string): 'obj' will use Object's default stringification format ('[object Object]') when stringified.
+   ,-[restrict-template-expressions.ts:2:24]
+ 1 | declare const obj: object;
+ 2 | const str1 = `Value: ${obj}`;
+   :                        ^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(restrict-template-expressions): Invalid type "object" of template literal expression.
+   ,-[restrict-template-expressions.ts:2:24]
+ 1 | declare const obj: object;
+ 2 | const str1 = `Value: ${obj}`;
+   :                        ^^^
+ 3 | 
+   `----
+
+  x typescript-eslint(switch-exhaustiveness-check): Switch is not exhaustive
+   ,-[switch-exhaustiveness-check.ts:3:11]
+ 2 | function handleStatus(status: Status) {
+ 3 |   switch (status) {
+   :           ^^^^^^
+ 4 |     case 'pending':
+   `----
+
+  x ]8;;https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-debugger.html\eslint(no-debugger)]8;;\: `debugger` statement is not allowed
+   ,-[test.svelte:2:2]
+ 1 | <script>
+ 2 |     debugger;
+   :     ^^^^^^^^^
+ 3 | 
+   `----
+  help: Remove the debugger statement
+
+  x typescript-eslint(unbound-method): Avoid referencing unbound methods which may cause unintentional scoping of `this`.
+  | If your function does not access `this`, you can annotate it with `this: void`, or consider using an arrow function instead.
+   ,-[unbound-method.ts:9:19]
+ 8 | const calc = new Calculator();
+ 9 | const addMethod = calc.add;
+   :                   ^^^^^^^^
+   `----
+
+Found 0 warnings and 50 errors.
+Finished in <variable>ms on 45 files using 1 threads.
+----------
+CLI result: LintFoundErrors
 ----------
diff --git a/apps/oxlint/src/snapshots/fixtures__tsgolint_rule_options_--type-aware@oxlint.snap b/apps/oxlint/src/snapshots/fixtures__tsgolint_rule_options_--type-aware@oxlint.snap
index 575e9298a64be..2a5717793b952 100644
--- a/apps/oxlint/src/snapshots/fixtures__tsgolint_rule_options_--type-aware@oxlint.snap
+++ b/apps/oxlint/src/snapshots/fixtures__tsgolint_rule_options_--type-aware@oxlint.snap
@@ -22,22 +22,6 @@ working directory: fixtures/tsgolint_rule_options
  48 | 
     `----
 
-  x typescript-eslint(no-unnecessary-type-assertion): This assertion is unnecessary since it does not change the type of the expression.
-    ,-[test.ts:51:22]
- 50 | // When checkLiteralConstAssertions is true, this SHOULD error
- 51 | const literalConst = 'hello' as const;
-    :                      ^^^^^^^^^^^^^^^^
- 52 | 
-    `----
-
-  x typescript-eslint(no-unsafe-member-access): Unsafe member access .foo on an `any` value.
-    ,-[test.ts:56:34]
- 55 | // This should NOT error because allowOptionalChaining is true and we use ?.
- 56 | const optionalAccess = anyValue?.foo;
-    :                                  ^^^
- 57 | // This SHOULD error because it's not using optional chaining
-    `----
-
   x typescript-eslint(no-unsafe-member-access): Unsafe member access .bar on an `any` value.
     ,-[test.ts:58:31]
  57 | // This SHOULD error because it's not using optional chaining
@@ -46,7 +30,15 @@ working directory: fixtures/tsgolint_rule_options
  59 | 
     `----
 
-Found 0 warnings and 5 errors.
+  x typescript-eslint(no-base-to-string): 'unknownValue' may use Object's default stringification format ('[object Object]') when stringified.
+    ,-[test.ts:63:20]
+ 62 | // This SHOULD error because checkUnknown is true
+ 63 | const unknownStr = unknownValue.toString();
+    :                    ^^^^^^^^^^^^
+ 64 | 
+    `----
+
+Found 0 warnings and 4 errors.
 Finished in <variable>ms on 1 file using 1 threads.
 ----------
 CLI result: LintFoundErrors

From 88b4a7d4cd796a810872357560297ab20e338bbb Mon Sep 17 00:00:00 2001
From: Kenzo-Wada <used.acormhib@gmail.com>
Date: Tue, 23 Dec 2025 00:47:38 +0900
Subject: [PATCH 04/11] update

---
 apps/oxlint/src/lint.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apps/oxlint/src/lint.rs b/apps/oxlint/src/lint.rs
index 9a414c4065178..595b91d1a8092 100644
--- a/apps/oxlint/src/lint.rs
+++ b/apps/oxlint/src/lint.rs
@@ -1461,7 +1461,7 @@ export { redundant };
             "// This file has a fixable tsgolint error: no-unnecessary-type-assertion
 // The type assertion `as string` is unnecessary because str is already a string
 const str: string = 'hello';
-const redundant = str  ;
+const redundant = str;
 
 export { redundant };
 ",

From 73fe0702e83ecc806b64f8970dafd3ec34c46085 Mon Sep 17 00:00:00 2001
From: Cameron Clark <cameron.clark@hey.com>
Date: Mon, 22 Dec 2025 15:55:24 +0000
Subject: [PATCH 05/11] re-port test cases

---
 .../src/rules/react/jsx_max_depth.rs          | 289 ++++++++++++++----
 1 file changed, 233 insertions(+), 56 deletions(-)

diff --git a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
index 417ea65c0a2ae..ea3b84cd577c7 100644
--- a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
+++ b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
@@ -19,7 +19,7 @@ fn jsx_max_depth_diagnostic(depth: usize, max: usize, span: Span) -> OxcDiagnost
     .with_label(span)
 }
 
-#[derive(Debug, Clone, Default)]
+#[derive(Debug, Clone, Default, Serialize, Deserialize)]
 pub struct JsxMaxDepth(Box<JsxMaxDepthConfig>);
 
 impl std::ops::Deref for JsxMaxDepth {
@@ -97,11 +97,9 @@ declare_oxc_lint!(
 
 impl Rule for JsxMaxDepth {
     fn from_configuration(value: serde_json::Value) -> Self {
-        let config = serde_json::from_value::<DefaultRuleConfig<JsxMaxDepthConfig>>(value)
+        serde_json::from_value::<DefaultRuleConfig<JsxMaxDepth>>(value)
             .map(DefaultRuleConfig::into_inner)
-            .unwrap_or_default();
-
-        Self(Box::new(config))
+            .unwrap_or_default()
     }
 
     fn run<'a>(&self, node: &AstNode<'a>, ctx: &LintContext<'a>) {
@@ -136,69 +134,248 @@ fn test() {
 
     let pass = vec![
         (
-            r"
-            const Component = () => (
-              <div>
-                <section />
-              </div>
-            );
-            ",
+            "
+			        <App />
+			      ",
+            None,
+        ),
+        (
+            "
+			        <App>
+			          <foo />
+			        </App>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        <App>
+			          <foo>
+			            <bar />
+			          </foo>
+			        </App>
+			      ",
+            None,
+        ),
+        (
+            "
+			        <App>
+			          <foo>
+			            <bar />
+			          </foo>
+			        </App>
+			      ",
+            Some(serde_json::json!([{ "max": 2 }])),
+        ),
+        (
+            "
+			        const x = <div><em>x</em></div>;
+			        <div>{x}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 2 }])),
+        ),
+        ("const foo = (x) => <div><em>{x}</em></div>;", Some(serde_json::json!([{ "max": 2 }]))),
+        (
+            "
+			        <></>
+			      ",
+            None,
+        ),
+        (
+            "
+			        <>
+			          <foo />
+			        </>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <><em>x</em></>;
+			        <>{x}</>
+			      ",
+            Some(serde_json::json!([{ "max": 2 }])),
+        ),
+        (
+            "
+			        const x = (
+			          <tr>
+			            <td>1</td>
+			            <td>2</td>
+			          </tr>
+			        );
+			        <tbody>
+			          {x}
+			        </tbody>
+			      ",
+            Some(serde_json::json!([{ "max": 2 }])),
+        ),
+        (
+            "
+			        const Example = props => {
+			          for (let i = 0; i < length; i++) {
+			            return <Text key={i} />;
+			          }
+			        };
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        export function MyComponent() {
+			          const A = <React.Fragment>{<div />}</React.Fragment>;
+			          return <div>{A}</div>;
+			        }
+			      ",
             None,
         ),
         (
-            r"
-            const Component = () => (
-              <div>
-                <div>
-                  <div />
-                </div>
-              </div>
-            );
-            ",
-            Some(serde_json::json!([{ "max": 3 }])),
-        ),
-        (
-            r"
-            const Component = () => (
-              <>
-                <div>
-                  <span />
-                </div>
-              </>
-            );
-            ",
+            r#"
+			        function Component() {
+			          let first = "";
+			          const second = first;
+			          first = second;
+			          return <div id={first} />;
+			        };
+			      "#,
+            None,
+        ),
+        (
+            r#"
+			        function Component() {
+			          let first = "";
+			          let second = "";
+			          let third = "";
+			          let fourth = "";
+			          const fifth = first;
+			          first = second;
+			          second = third;
+			          third = fourth;
+			          fourth = fifth;
+			          return <div id={first} />;
+			        };
+			      "#,
             None,
         ),
     ];
 
     let fail = vec![
         (
-            r"
-            const Component = () => (
-              <div>
-                <div>
-                  <div>
-                    <span />
-                  </div>
-                </div>
-              </div>
-            );
-            ",
+            "
+			        <App>
+			          <foo />
+			        </App>
+			      ",
+            Some(serde_json::json!([{ "max": 0 }])),
+        ),
+        (
+            "
+			        <App>
+			          <foo>{bar}</foo>
+			        </App>
+			      ",
+            Some(serde_json::json!([{ "max": 0 }])),
+        ),
+        (
+            "
+			        <App>
+			          <foo>
+			            <bar />
+			          </foo>
+			        </App>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <div><span /></div>;
+			        <div>{x}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <div><span /></div>;
+			        let y = x;
+			        <div>{y}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <div><span /></div>;
+			        let y = x;
+			        <div>{x}-{y}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        <div>
+			        {<div><div><span /></div></div>}
+			        </div>
+			      ",
             None,
         ),
         (
-            r"
-            const Component = () => (
-              <>
-                <div>
-                  <>
-                    <div />
-                  </>
-                </div>
-              </>
-            );
-            ",
-            Some(serde_json::json!([{ "max": 2 }])),
+            "
+			        <>
+			          <foo />
+			        </>
+			      ",
+            Some(serde_json::json!([{ "max": 0 }])),
+        ),
+        (
+            "
+			        <>
+			          <>
+			            <bar />
+			          </>
+			        </>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <><span /></>;
+			        let y = x;
+			        <>{x}-{y}</>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = (
+			          <tr>
+			            <td>1</td>
+			            <td>2</td>
+			          </tr>
+			        );
+			        <tbody>
+			          {x}
+			        </tbody>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            r#"
+			        <div className="custom_modal">
+			          <Modal className={classes.modal} open={isOpen} closeAfterTransition>
+			            <Fade in={isOpen}>
+			              <DialogContent>
+			                <Icon icon="cancel" onClick={onClose} popoverText="Close Modal" />
+			                <div className="modal_content">{children}</div>
+			                <div className={clxs('modal_buttons', classes.buttons)}>
+			                  <Button className="modal_buttons--cancel" onClick={onCancel}>
+			                    {cancelMsg ? cancelMsg : 'Cancel'}
+			                  </Button>
+			                </div>
+			              </DialogContent>
+			            </Fade>
+			          </Modal>
+			        </div>
+			      "#,
+            Some(serde_json::json!([{ "max": 4 }])),
         ),
     ];
 

From fcc3598bf1e83b3354865886b69cdf03678320ac Mon Sep 17 00:00:00 2001
From: Kenzo-Wada <used.acormhib@gmail.com>
Date: Tue, 23 Dec 2025 01:10:40 +0900
Subject: [PATCH 06/11] chore: update to pass re-ported tests

---
 .../src/rules/react/jsx_max_depth.rs          | 90 +++++++++----------
 .../src/snapshots/react_jsx_max_depth.snap    | 63 ++++++++++---
 2 files changed, 97 insertions(+), 56 deletions(-)

diff --git a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
index ea3b84cd577c7..7dd67fe197deb 100644
--- a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
+++ b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
@@ -257,6 +257,51 @@ fn test() {
 			      "#,
             None,
         ),
+        (
+            "
+			        const x = <div><span /></div>;
+			        <div>{x}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <div><span /></div>;
+			        let y = x;
+			        <div>{y}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <div><span /></div>;
+			        let y = x;
+			        <div>{x}-{y}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <><span /></>;
+			        let y = x;
+			        <>{x}-{y}</>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = (
+			          <tr>
+			            <td>1</td>
+			            <td>2</td>
+			          </tr>
+			        );
+			        <tbody>
+			          {x}
+			        </tbody>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
     ];
 
     let fail = vec![
@@ -286,29 +331,6 @@ fn test() {
 			      ",
             Some(serde_json::json!([{ "max": 1 }])),
         ),
-        (
-            "
-			        const x = <div><span /></div>;
-			        <div>{x}</div>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
-        (
-            "
-			        const x = <div><span /></div>;
-			        let y = x;
-			        <div>{y}</div>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
-        (
-            "
-			        const x = <div><span /></div>;
-			        let y = x;
-			        <div>{x}-{y}</div>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
         (
             "
 			        <div>
@@ -335,28 +357,6 @@ fn test() {
 			      ",
             Some(serde_json::json!([{ "max": 1 }])),
         ),
-        (
-            "
-			        const x = <><span /></>;
-			        let y = x;
-			        <>{x}-{y}</>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
-        (
-            "
-			        const x = (
-			          <tr>
-			            <td>1</td>
-			            <td>2</td>
-			          </tr>
-			        );
-			        <tbody>
-			          {x}
-			        </tbody>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
         (
             r#"
 			        <div className="custom_modal">
diff --git a/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap b/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap
index 58f19fc4094f4..8ca8225d85c93 100644
--- a/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap
+++ b/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap
@@ -1,18 +1,59 @@
 ---
 source: crates/oxc_linter/src/tester.rs
 ---
-  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 3 exceeds the configured maximum of 2
-   ╭─[jsx_max_depth.tsx:6:21]
- 5 │                   <div>
- 6 │                     <span />
-   ·                     ────────
- 7 │                   </div>
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 1 exceeds the configured maximum of 0
+   ╭─[jsx_max_depth.tsx:3:14]
+ 2 │                     <App>
+ 3 │                       <foo />
+   ·                       ───────
+ 4 │                     </App>
+   ╰────
+
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 1 exceeds the configured maximum of 0
+   ╭─[jsx_max_depth.tsx:3:14]
+ 2 │                     <App>
+ 3 │                       <foo>{bar}</foo>
+   ·                       ────────────────
+ 4 │                     </App>
+   ╰────
+
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 2 exceeds the configured maximum of 1
+   ╭─[jsx_max_depth.tsx:4:16]
+ 3 │                       <foo>
+ 4 │                         <bar />
+   ·                         ───────
+ 5 │                       </foo>
    ╰────
 
   ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 3 exceeds the configured maximum of 2
-   ╭─[jsx_max_depth.tsx:6:21]
- 5 │                   <>
- 6 │                     <div />
-   ·                     ───────
- 7 │                   </>
+   ╭─[jsx_max_depth.tsx:3:23]
+ 2 │                     <div>
+ 3 │                     {<div><div><span /></div></div>}
+   ·                                ────────
+ 4 │                     </div>
    ╰────
+
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 1 exceeds the configured maximum of 0
+   ╭─[jsx_max_depth.tsx:3:14]
+ 2 │                     <>
+ 3 │                       <foo />
+   ·                       ───────
+ 4 │                     </>
+   ╰────
+
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 2 exceeds the configured maximum of 1
+   ╭─[jsx_max_depth.tsx:4:16]
+ 3 │                       <>
+ 4 │                         <bar />
+   ·                         ───────
+ 5 │                       </>
+   ╰────
+
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 5 exceeds the configured maximum of 4
+    ╭─[jsx_max_depth.tsx:9:22]
+  8 │                                 <div className={clxs('modal_buttons', classes.buttons)}>
+  9 │ ╭─▶                               <Button className="modal_buttons--cancel" onClick={onCancel}>
+ 10 │ │                                   {cancelMsg ? cancelMsg : 'Cancel'}
+ 11 │ ╰─▶                               </Button>
+ 12 │                                 </div>
+    ╰────

From 3859142ea73824fe21e5c66aa408ca6f2d43ab76 Mon Sep 17 00:00:00 2001
From: Kenzo-Wada <used.acormhib@gmail.com>
Date: Tue, 23 Dec 2025 01:30:14 +0900
Subject: [PATCH 07/11] update snapshots

---
 .../src/rules/react/jsx_max_depth.rs          | 234 ++++++++++++++----
 .../src/snapshots/react_jsx_max_depth.snap    |  41 +++
 2 files changed, 226 insertions(+), 49 deletions(-)

diff --git a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
index 7dd67fe197deb..3ea7e15996e17 100644
--- a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
+++ b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
@@ -1,10 +1,12 @@
 use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 
-use oxc_ast::AstKind;
+use oxc_ast::{AstKind, ast::Expression};
 use oxc_diagnostics::OxcDiagnostic;
 use oxc_macros::declare_oxc_lint;
+use oxc_semantic::SymbolId;
 use oxc_span::{GetSpan, Span};
+use rustc_hash::FxHashSet;
 
 use crate::{
     AstNode,
@@ -108,9 +110,19 @@ impl Rule for JsxMaxDepth {
             _ => return,
         }
 
-        let depth = jsx_ancestor_depth(node, ctx);
-        if depth > self.max {
-            ctx.diagnostic(jsx_max_depth_diagnostic(depth, self.max, node.span()));
+        if !is_leaf_jsx_node(node, ctx) {
+            return;
+        }
+
+        let ancestor_depth = jsx_ancestor_depth(node, ctx);
+
+        let mut visited_symbols = FxHashSet::default();
+        let child_depth = calculate_node_jsx_depth(node, ctx, &mut visited_symbols);
+
+        let total_depth = ancestor_depth + child_depth;
+
+        if total_depth > self.max {
+            ctx.diagnostic(jsx_max_depth_diagnostic(total_depth, self.max, node.span()));
         }
     }
 
@@ -119,6 +131,130 @@ impl Rule for JsxMaxDepth {
     }
 }
 
+/// Calculate the depth of JSX referenced by a variable
+fn calculate_variable_jsx_depth(
+    symbol_id: SymbolId,
+    ctx: &LintContext<'_>,
+    visited: &mut FxHashSet<SymbolId>,
+) -> usize {
+    if !visited.insert(symbol_id) {
+        return 0;
+    }
+
+    let scoping = ctx.semantic().scoping();
+
+    let decl_id = scoping.symbol_declaration(symbol_id);
+    let decl_node = ctx.nodes().get_node(decl_id);
+
+    if let AstKind::VariableDeclarator(declarator) = decl_node.kind() {
+        if let Some(init) = &declarator.init {
+            return calculate_expression_jsx_depth(init, ctx, visited);
+        }
+    }
+
+    for reference_id in scoping.get_resolved_reference_ids(symbol_id) {
+        let reference = scoping.get_reference(*reference_id);
+        if !reference.is_write() {
+            continue;
+        }
+
+        let ref_node = ctx.nodes().get_node(reference.node_id());
+        let current = ctx.nodes().parent_node(ref_node.id());
+        let parent = ctx.nodes().parent_node(current.id());
+
+        if let AstKind::AssignmentExpression(assign) = parent.kind() {
+            let depth = calculate_expression_jsx_depth(&assign.right, ctx, visited);
+            if depth > 0 {
+                return depth;
+            }
+        }
+    }
+
+    0
+}
+
+/// Calculate the depth of JSX in an expression
+fn calculate_expression_jsx_depth(
+    expr: &Expression<'_>,
+    ctx: &LintContext<'_>,
+    visited: &mut FxHashSet<SymbolId>,
+) -> usize {
+    match expr {
+        Expression::JSXElement(_) | Expression::JSXFragment(_) => {
+            if let Some(node) = ctx.nodes().iter().find(|node| node.kind().span() == expr.span()) {
+                return calculate_node_jsx_depth(node, ctx, visited);
+            }
+            0
+        }
+        Expression::Identifier(ident) => {
+            if let Some(reference_id) = ident.reference_id.get() {
+                let scoping = ctx.semantic().scoping();
+                let reference = scoping.get_reference(reference_id);
+                if let Some(symbol_id) = reference.symbol_id() {
+                    return calculate_variable_jsx_depth(symbol_id, ctx, visited);
+                }
+            }
+            0
+        }
+        Expression::ParenthesizedExpression(paren) => {
+            calculate_expression_jsx_depth(&paren.expression, ctx, visited)
+        }
+        _ => 0,
+    }
+}
+
+/// Calculate JSX depth for a node, accounting for variable references
+fn calculate_node_jsx_depth(
+    node: &AstNode<'_>,
+    ctx: &LintContext<'_>,
+    visited_symbols: &mut FxHashSet<SymbolId>,
+) -> usize {
+    let mut max_child_depth = 0;
+
+    for child_node in ctx.nodes().iter() {
+        let parent = ctx.nodes().parent_node(child_node.id());
+        if parent.id() != node.id() {
+            continue;
+        }
+
+        match child_node.kind() {
+            AstKind::JSXElement(_) | AstKind::JSXFragment(_) => {
+                let child_depth = calculate_node_jsx_depth(child_node, ctx, visited_symbols);
+                max_child_depth = max_child_depth.max(child_depth + 1);
+            }
+            AstKind::JSXExpressionContainer(container) => {
+                if let Some(Expression::Identifier(ident)) = container.expression.as_expression() {
+                    if let Some(reference_id) = ident.reference_id.get() {
+                        let scoping = ctx.semantic().scoping();
+                        let reference = scoping.get_reference(reference_id);
+                        if let Some(symbol_id) = reference.symbol_id() {
+                            let referenced_depth =
+                                calculate_variable_jsx_depth(symbol_id, ctx, visited_symbols);
+                            max_child_depth = max_child_depth.max(referenced_depth + 1);
+                        }
+                    }
+                }
+            }
+            _ => {}
+        }
+    }
+
+    max_child_depth
+}
+
+/// Check if a JSX node is a leaf (has no JSX children)
+fn is_leaf_jsx_node(node: &AstNode<'_>, ctx: &LintContext<'_>) -> bool {
+    for child_node in ctx.nodes().iter() {
+        let parent = ctx.nodes().parent_node(child_node.id());
+        if parent.id() == node.id() {
+            if matches!(child_node.kind(), AstKind::JSXElement(_) | AstKind::JSXFragment(_)) {
+                return false;
+            }
+        }
+    }
+    true
+}
+
 fn jsx_ancestor_depth(node: &AstNode<'_>, ctx: &LintContext<'_>) -> usize {
     ctx.nodes()
         .ancestors(node.id())
@@ -257,51 +393,6 @@ fn test() {
 			      "#,
             None,
         ),
-        (
-            "
-			        const x = <div><span /></div>;
-			        <div>{x}</div>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
-        (
-            "
-			        const x = <div><span /></div>;
-			        let y = x;
-			        <div>{y}</div>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
-        (
-            "
-			        const x = <div><span /></div>;
-			        let y = x;
-			        <div>{x}-{y}</div>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
-        (
-            "
-			        const x = <><span /></>;
-			        let y = x;
-			        <>{x}-{y}</>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
-        (
-            "
-			        const x = (
-			          <tr>
-			            <td>1</td>
-			            <td>2</td>
-			          </tr>
-			        );
-			        <tbody>
-			          {x}
-			        </tbody>
-			      ",
-            Some(serde_json::json!([{ "max": 1 }])),
-        ),
     ];
 
     let fail = vec![
@@ -331,6 +422,29 @@ fn test() {
 			      ",
             Some(serde_json::json!([{ "max": 1 }])),
         ),
+        (
+            "
+			        const x = <div><span /></div>;
+			        <div>{x}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <div><span /></div>;
+			        let y = x;
+			        <div>{y}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = <div><span /></div>;
+			        let y = x;
+			        <div>{x}-{y}</div>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
         (
             "
 			        <div>
@@ -357,6 +471,28 @@ fn test() {
 			      ",
             Some(serde_json::json!([{ "max": 1 }])),
         ),
+        (
+            "
+			        const x = <><span /></>;
+			        let y = x;
+			        <>{x}-{y}</>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
+        (
+            "
+			        const x = (
+			          <tr>
+			            <td>1</td>
+			            <td>2</td>
+			          </tr>
+			        );
+			        <tbody>
+			          {x}
+			        </tbody>
+			      ",
+            Some(serde_json::json!([{ "max": 1 }])),
+        ),
         (
             r#"
 			        <div className="custom_modal">
diff --git a/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap b/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap
index 8ca8225d85c93..9092e53b2b3ba 100644
--- a/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap
+++ b/crates/oxc_linter/src/snapshots/react_jsx_max_depth.snap
@@ -25,6 +25,30 @@ source: crates/oxc_linter/src/tester.rs
  5 │                       </foo>
    ╰────
 
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 2 exceeds the configured maximum of 1
+   ╭─[jsx_max_depth.tsx:3:12]
+ 2 │                     const x = <div><span /></div>;
+ 3 │                     <div>{x}</div>
+   ·                     ──────────────
+ 4 │                   
+   ╰────
+
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 2 exceeds the configured maximum of 1
+   ╭─[jsx_max_depth.tsx:4:12]
+ 3 │                     let y = x;
+ 4 │                     <div>{y}</div>
+   ·                     ──────────────
+ 5 │                   
+   ╰────
+
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 2 exceeds the configured maximum of 1
+   ╭─[jsx_max_depth.tsx:4:12]
+ 3 │                     let y = x;
+ 4 │                     <div>{x}-{y}</div>
+   ·                     ──────────────────
+ 5 │                   
+   ╰────
+
   ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 3 exceeds the configured maximum of 2
    ╭─[jsx_max_depth.tsx:3:23]
  2 │                     <div>
@@ -49,6 +73,23 @@ source: crates/oxc_linter/src/tester.rs
  5 │                       </>
    ╰────
 
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 2 exceeds the configured maximum of 1
+   ╭─[jsx_max_depth.tsx:4:12]
+ 3 │                     let y = x;
+ 4 │                     <>{x}-{y}</>
+   ·                     ────────────
+ 5 │                   
+   ╰────
+
+  ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 2 exceeds the configured maximum of 1
+    ╭─[jsx_max_depth.tsx:8:12]
+  7 │                         );
+  8 │ ╭─▶                     <tbody>
+  9 │ │                         {x}
+ 10 │ ╰─▶                     </tbody>
+ 11 │                       
+    ╰────
+
   ⚠ eslint-plugin-react(jsx-max-depth): JSX nesting depth of 5 exceeds the configured maximum of 4
     ╭─[jsx_max_depth.tsx:9:22]
   8 │                                 <div className={clxs('modal_buttons', classes.buttons)}>

From 56ea23d3b990c4d019a6c88d343ae7dfab27271c Mon Sep 17 00:00:00 2001
From: Kenzo-Wada <used.acormhib@gmail.com>
Date: Tue, 23 Dec 2025 01:38:28 +0900
Subject: [PATCH 08/11] ran clippy

---
 .../src/rules/react/jsx_max_depth.rs          | 34 +++++++++----------
 1 file changed, 17 insertions(+), 17 deletions(-)

diff --git a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
index 3ea7e15996e17..2cff64cec4154 100644
--- a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
+++ b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
@@ -146,10 +146,10 @@ fn calculate_variable_jsx_depth(
     let decl_id = scoping.symbol_declaration(symbol_id);
     let decl_node = ctx.nodes().get_node(decl_id);
 
-    if let AstKind::VariableDeclarator(declarator) = decl_node.kind() {
-        if let Some(init) = &declarator.init {
-            return calculate_expression_jsx_depth(init, ctx, visited);
-        }
+    if let AstKind::VariableDeclarator(declarator) = decl_node.kind()
+        && let Some(init) = &declarator.init
+    {
+        return calculate_expression_jsx_depth(init, ctx, visited);
     }
 
     for reference_id in scoping.get_resolved_reference_ids(symbol_id) {
@@ -223,15 +223,15 @@ fn calculate_node_jsx_depth(
                 max_child_depth = max_child_depth.max(child_depth + 1);
             }
             AstKind::JSXExpressionContainer(container) => {
-                if let Some(Expression::Identifier(ident)) = container.expression.as_expression() {
-                    if let Some(reference_id) = ident.reference_id.get() {
-                        let scoping = ctx.semantic().scoping();
-                        let reference = scoping.get_reference(reference_id);
-                        if let Some(symbol_id) = reference.symbol_id() {
-                            let referenced_depth =
-                                calculate_variable_jsx_depth(symbol_id, ctx, visited_symbols);
-                            max_child_depth = max_child_depth.max(referenced_depth + 1);
-                        }
+                if let Some(Expression::Identifier(ident)) = container.expression.as_expression()
+                    && let Some(reference_id) = ident.reference_id.get()
+                {
+                    let scoping = ctx.semantic().scoping();
+                    let reference = scoping.get_reference(reference_id);
+                    if let Some(symbol_id) = reference.symbol_id() {
+                        let referenced_depth =
+                            calculate_variable_jsx_depth(symbol_id, ctx, visited_symbols);
+                        max_child_depth = max_child_depth.max(referenced_depth + 1);
                     }
                 }
             }
@@ -246,10 +246,10 @@ fn calculate_node_jsx_depth(
 fn is_leaf_jsx_node(node: &AstNode<'_>, ctx: &LintContext<'_>) -> bool {
     for child_node in ctx.nodes().iter() {
         let parent = ctx.nodes().parent_node(child_node.id());
-        if parent.id() == node.id() {
-            if matches!(child_node.kind(), AstKind::JSXElement(_) | AstKind::JSXFragment(_)) {
-                return false;
-            }
+        if parent.id() == node.id()
+            && matches!(child_node.kind(), AstKind::JSXElement(_) | AstKind::JSXFragment(_))
+        {
+            return false;
         }
     }
     true

From 24d1667e5ceebab50fa68cbf68a4c7d2c2ee0e47 Mon Sep 17 00:00:00 2001
From: Kenzo-Wada <used.acormhib@gmail.com>
Date: Tue, 23 Dec 2025 11:40:28 +0900
Subject: [PATCH 09/11] improved performance for n x n loops

---
 .../src/rules/react/jsx_max_depth.rs          | 86 ++++++++++++-------
 1 file changed, 55 insertions(+), 31 deletions(-)

diff --git a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
index 2cff64cec4154..edde25cce74ed 100644
--- a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
+++ b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
@@ -1,7 +1,10 @@
 use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 
-use oxc_ast::{AstKind, ast::Expression};
+use oxc_ast::{
+    AstKind,
+    ast::{Expression, JSXChild, JSXElement, JSXFragment},
+};
 use oxc_diagnostics::OxcDiagnostic;
 use oxc_macros::declare_oxc_lint;
 use oxc_semantic::SymbolId;
@@ -131,7 +134,6 @@ impl Rule for JsxMaxDepth {
     }
 }
 
-/// Calculate the depth of JSX referenced by a variable
 fn calculate_variable_jsx_depth(
     symbol_id: SymbolId,
     ctx: &LintContext<'_>,
@@ -173,19 +175,14 @@ fn calculate_variable_jsx_depth(
     0
 }
 
-/// Calculate the depth of JSX in an expression
 fn calculate_expression_jsx_depth(
     expr: &Expression<'_>,
     ctx: &LintContext<'_>,
     visited: &mut FxHashSet<SymbolId>,
 ) -> usize {
     match expr {
-        Expression::JSXElement(_) | Expression::JSXFragment(_) => {
-            if let Some(node) = ctx.nodes().iter().find(|node| node.kind().span() == expr.span()) {
-                return calculate_node_jsx_depth(node, ctx, visited);
-            }
-            0
-        }
+        Expression::JSXElement(elem) => calculate_jsx_element_depth(elem, ctx, visited),
+        Expression::JSXFragment(frag) => calculate_jsx_fragment_depth(frag, ctx, visited),
         Expression::Identifier(ident) => {
             if let Some(reference_id) = ident.reference_id.get() {
                 let scoping = ctx.semantic().scoping();
@@ -203,26 +200,40 @@ fn calculate_expression_jsx_depth(
     }
 }
 
-/// Calculate JSX depth for a node, accounting for variable references
 fn calculate_node_jsx_depth(
     node: &AstNode<'_>,
     ctx: &LintContext<'_>,
     visited_symbols: &mut FxHashSet<SymbolId>,
 ) -> usize {
-    let mut max_child_depth = 0;
-
-    for child_node in ctx.nodes().iter() {
-        let parent = ctx.nodes().parent_node(child_node.id());
-        if parent.id() != node.id() {
-            continue;
+    match node.kind() {
+        AstKind::JSXElement(elem) => {
+            calculate_jsx_children_depth(&elem.children, ctx, visited_symbols)
+        }
+        AstKind::JSXFragment(frag) => {
+            calculate_jsx_children_depth(&frag.children, ctx, visited_symbols)
         }
+        _ => 0,
+    }
+}
+
+fn calculate_jsx_children_depth(
+    children: &[JSXChild<'_>],
+    ctx: &LintContext<'_>,
+    visited_symbols: &mut FxHashSet<SymbolId>,
+) -> usize {
+    let mut max_child_depth = 0;
 
-        match child_node.kind() {
-            AstKind::JSXElement(_) | AstKind::JSXFragment(_) => {
-                let child_depth = calculate_node_jsx_depth(child_node, ctx, visited_symbols);
+    for child in children {
+        match child {
+            JSXChild::Element(elem) => {
+                let child_depth = calculate_jsx_element_depth(elem, ctx, visited_symbols);
                 max_child_depth = max_child_depth.max(child_depth + 1);
             }
-            AstKind::JSXExpressionContainer(container) => {
+            JSXChild::Fragment(frag) => {
+                let child_depth = calculate_jsx_fragment_depth(frag, ctx, visited_symbols);
+                max_child_depth = max_child_depth.max(child_depth + 1);
+            }
+            JSXChild::ExpressionContainer(container) => {
                 if let Some(Expression::Identifier(ident)) = container.expression.as_expression()
                     && let Some(reference_id) = ident.reference_id.get()
                 {
@@ -242,17 +253,30 @@ fn calculate_node_jsx_depth(
     max_child_depth
 }
 
-/// Check if a JSX node is a leaf (has no JSX children)
-fn is_leaf_jsx_node(node: &AstNode<'_>, ctx: &LintContext<'_>) -> bool {
-    for child_node in ctx.nodes().iter() {
-        let parent = ctx.nodes().parent_node(child_node.id());
-        if parent.id() == node.id()
-            && matches!(child_node.kind(), AstKind::JSXElement(_) | AstKind::JSXFragment(_))
-        {
-            return false;
-        }
-    }
-    true
+fn calculate_jsx_element_depth(
+    elem: &JSXElement<'_>,
+    ctx: &LintContext<'_>,
+    visited_symbols: &mut FxHashSet<SymbolId>,
+) -> usize {
+    calculate_jsx_children_depth(&elem.children, ctx, visited_symbols)
+}
+
+fn calculate_jsx_fragment_depth(
+    frag: &JSXFragment<'_>,
+    ctx: &LintContext<'_>,
+    visited_symbols: &mut FxHashSet<SymbolId>,
+) -> usize {
+    calculate_jsx_children_depth(&frag.children, ctx, visited_symbols)
+}
+
+fn is_leaf_jsx_node(node: &AstNode<'_>, _ctx: &LintContext<'_>) -> bool {
+    let children = match node.kind() {
+        AstKind::JSXElement(elem) => &elem.children,
+        AstKind::JSXFragment(frag) => &frag.children,
+        _ => return true,
+    };
+
+    !children.iter().any(|child| matches!(child, JSXChild::Element(_) | JSXChild::Fragment(_)))
 }
 
 fn jsx_ancestor_depth(node: &AstNode<'_>, ctx: &LintContext<'_>) -> usize {

From 654e836e08554fb22835317188b14cae5d938bec Mon Sep 17 00:00:00 2001
From: Kenzo-Wada <used.acormhib@gmail.com>
Date: Tue, 23 Dec 2025 21:01:33 +0900
Subject: [PATCH 10/11] improve perfomance

---
 .../src/rules/react/jsx_max_depth.rs          | 118 ++++++------------
 1 file changed, 40 insertions(+), 78 deletions(-)

diff --git a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
index edde25cce74ed..cac93dc2e56d4 100644
--- a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
+++ b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
@@ -3,7 +3,7 @@ use serde::{Deserialize, Serialize};
 
 use oxc_ast::{
     AstKind,
-    ast::{Expression, JSXChild, JSXElement, JSXFragment},
+    ast::{Expression, JSXChild},
 };
 use oxc_diagnostics::OxcDiagnostic;
 use oxc_macros::declare_oxc_lint;
@@ -143,9 +143,7 @@ fn calculate_variable_jsx_depth(
         return 0;
     }
 
-    let scoping = ctx.semantic().scoping();
-
-    let decl_id = scoping.symbol_declaration(symbol_id);
+    let decl_id = ctx.semantic().scoping().symbol_declaration(symbol_id);
     let decl_node = ctx.nodes().get_node(decl_id);
 
     if let AstKind::VariableDeclarator(declarator) = decl_node.kind()
@@ -154,24 +152,6 @@ fn calculate_variable_jsx_depth(
         return calculate_expression_jsx_depth(init, ctx, visited);
     }
 
-    for reference_id in scoping.get_resolved_reference_ids(symbol_id) {
-        let reference = scoping.get_reference(*reference_id);
-        if !reference.is_write() {
-            continue;
-        }
-
-        let ref_node = ctx.nodes().get_node(reference.node_id());
-        let current = ctx.nodes().parent_node(ref_node.id());
-        let parent = ctx.nodes().parent_node(current.id());
-
-        if let AstKind::AssignmentExpression(assign) = parent.kind() {
-            let depth = calculate_expression_jsx_depth(&assign.right, ctx, visited);
-            if depth > 0 {
-                return depth;
-            }
-        }
-    }
-
     0
 }
 
@@ -181,18 +161,13 @@ fn calculate_expression_jsx_depth(
     visited: &mut FxHashSet<SymbolId>,
 ) -> usize {
     match expr {
-        Expression::JSXElement(elem) => calculate_jsx_element_depth(elem, ctx, visited),
-        Expression::JSXFragment(frag) => calculate_jsx_fragment_depth(frag, ctx, visited),
-        Expression::Identifier(ident) => {
-            if let Some(reference_id) = ident.reference_id.get() {
-                let scoping = ctx.semantic().scoping();
-                let reference = scoping.get_reference(reference_id);
-                if let Some(symbol_id) = reference.symbol_id() {
-                    return calculate_variable_jsx_depth(symbol_id, ctx, visited);
-                }
-            }
-            0
-        }
+        Expression::JSXElement(elem) => calculate_jsx_children_depth(&elem.children, ctx, visited),
+        Expression::JSXFragment(frag) => calculate_jsx_children_depth(&frag.children, ctx, visited),
+        Expression::Identifier(ident) => ident
+            .reference_id
+            .get()
+            .and_then(|ref_id| ctx.semantic().scoping().get_reference(ref_id).symbol_id())
+            .map_or(0, |symbol_id| calculate_variable_jsx_depth(symbol_id, ctx, visited)),
         Expression::ParenthesizedExpression(paren) => {
             calculate_expression_jsx_depth(&paren.expression, ctx, visited)
         }
@@ -205,15 +180,12 @@ fn calculate_node_jsx_depth(
     ctx: &LintContext<'_>,
     visited_symbols: &mut FxHashSet<SymbolId>,
 ) -> usize {
-    match node.kind() {
-        AstKind::JSXElement(elem) => {
-            calculate_jsx_children_depth(&elem.children, ctx, visited_symbols)
-        }
-        AstKind::JSXFragment(frag) => {
-            calculate_jsx_children_depth(&frag.children, ctx, visited_symbols)
-        }
-        _ => 0,
-    }
+    let children = match node.kind() {
+        AstKind::JSXElement(elem) => &elem.children,
+        AstKind::JSXFragment(frag) => &frag.children,
+        _ => return 0,
+    };
+    calculate_jsx_children_depth(children, ctx, visited_symbols)
 }
 
 fn calculate_jsx_children_depth(
@@ -221,52 +193,42 @@ fn calculate_jsx_children_depth(
     ctx: &LintContext<'_>,
     visited_symbols: &mut FxHashSet<SymbolId>,
 ) -> usize {
-    let mut max_child_depth = 0;
+    if children.is_empty() {
+        return 0;
+    }
+
+    let mut max_depth = 0;
 
     for child in children {
-        match child {
+        let depth = match child {
             JSXChild::Element(elem) => {
-                let child_depth = calculate_jsx_element_depth(elem, ctx, visited_symbols);
-                max_child_depth = max_child_depth.max(child_depth + 1);
+                calculate_jsx_children_depth(&elem.children, ctx, visited_symbols) + 1
             }
             JSXChild::Fragment(frag) => {
-                let child_depth = calculate_jsx_fragment_depth(frag, ctx, visited_symbols);
-                max_child_depth = max_child_depth.max(child_depth + 1);
+                calculate_jsx_children_depth(&frag.children, ctx, visited_symbols) + 1
             }
             JSXChild::ExpressionContainer(container) => {
-                if let Some(Expression::Identifier(ident)) = container.expression.as_expression()
-                    && let Some(reference_id) = ident.reference_id.get()
-                {
-                    let scoping = ctx.semantic().scoping();
-                    let reference = scoping.get_reference(reference_id);
-                    if let Some(symbol_id) = reference.symbol_id() {
-                        let referenced_depth =
-                            calculate_variable_jsx_depth(symbol_id, ctx, visited_symbols);
-                        max_child_depth = max_child_depth.max(referenced_depth + 1);
-                    }
+                if let Some(Expression::Identifier(ident)) = container.expression.as_expression() {
+                    let depth = ident
+                        .reference_id
+                        .get()
+                        .and_then(|ref_id| {
+                            ctx.semantic().scoping().get_reference(ref_id).symbol_id()
+                        })
+                        .map_or(0, |symbol_id| {
+                            calculate_variable_jsx_depth(symbol_id, ctx, visited_symbols)
+                        });
+                    if depth > 0 { depth + 1 } else { 0 }
+                } else {
+                    0
                 }
             }
-            _ => {}
-        }
+            _ => 0,
+        };
+        max_depth = max_depth.max(depth);
     }
 
-    max_child_depth
-}
-
-fn calculate_jsx_element_depth(
-    elem: &JSXElement<'_>,
-    ctx: &LintContext<'_>,
-    visited_symbols: &mut FxHashSet<SymbolId>,
-) -> usize {
-    calculate_jsx_children_depth(&elem.children, ctx, visited_symbols)
-}
-
-fn calculate_jsx_fragment_depth(
-    frag: &JSXFragment<'_>,
-    ctx: &LintContext<'_>,
-    visited_symbols: &mut FxHashSet<SymbolId>,
-) -> usize {
-    calculate_jsx_children_depth(&frag.children, ctx, visited_symbols)
+    max_depth
 }
 
 fn is_leaf_jsx_node(node: &AstNode<'_>, _ctx: &LintContext<'_>) -> bool {

From 5aef67a9395d44d320be7d0b1a840a983315cb8a Mon Sep 17 00:00:00 2001
From: Cameron Clark <cameron.clark@hey.com>
Date: Tue, 23 Dec 2025 13:26:41 +0000
Subject: [PATCH 11/11] u

---
 crates/oxc_linter/src/rules/react/jsx_max_depth.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
index cac93dc2e56d4..f841ecd1df0be 100644
--- a/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
+++ b/crates/oxc_linter/src/rules/react/jsx_max_depth.rs
@@ -113,7 +113,7 @@ impl Rule for JsxMaxDepth {
             _ => return,
         }
 
-        if !is_leaf_jsx_node(node, ctx) {
+        if !is_leaf_jsx_node(node) {
             return;
         }
 
@@ -231,7 +231,7 @@ fn calculate_jsx_children_depth(
     max_depth
 }
 
-fn is_leaf_jsx_node(node: &AstNode<'_>, _ctx: &LintContext<'_>) -> bool {
+fn is_leaf_jsx_node(node: &AstNode<'_>) -> bool {
     let children = match node.kind() {
         AstKind::JSXElement(elem) => &elem.children,
         AstKind::JSXFragment(frag) => &frag.children,