fix(transformer): Always update jsx options from comments (#10966)

> [!NOTE]  
> The explanation in the comment is incorrect because the `h` function
might be locally defined rather than imported. Removing the conditional
check will fix the issue.

- Closes: #10956

Co-authored-by: Dunqing <dengqing0821@gmail.com>

Author: magic-akari

crates/oxc_transformer/src/lib.rs

@@ -120,12 +120,7 @@ impl<'a> Transformer<'a> {
         self.ctx.source_type = program.source_type;
         self.ctx.source_text = program.source_text;
 
-        // Update options from comments when source type is JSX or TypeScript which has enabled `only_remove_type_imports`.
-        // Because if `only_remove_type_imports` is enabled, no imports will be removed, so that we don't care about
-        // TypeScript's `jsx_pragma` and `jsx_pragma_frag` options.
-        if program.source_type.is_jsx()
-            && (!program.source_type.is_typescript() || !self.typescript.only_remove_type_imports)
-        {
+        if program.source_type.is_jsx() {
             jsx::update_options_with_comments(
                 &program.comments,
                 &mut self.typescript,

tasks/transform_conformance/snapshots/oxc.snap.md

@@ -1,6 +1,6 @@
 commit: 578ac4df
 
-Passed: 152/254
+Passed: 153/255
 
 # All Passed:
 * babel-plugin-transform-class-static-block
@@ -44,7 +44,7 @@ after transform: SymbolId(0): [ReferenceId(0), ReferenceId(2), ReferenceId(6), R
 rebuilt        : SymbolId(0): [ReferenceId(0), ReferenceId(2), ReferenceId(6), ReferenceId(10)]
 
 
-# babel-plugin-transform-typescript (4/25)
+# babel-plugin-transform-typescript (5/26)
 * allow-declare-fields-false/input.ts
 Unresolved references mismatch:
 after transform: ["dce"]

tasks/transform_conformance/tests/babel-plugin-transform-typescript/test/fixtures/jsx/issue-10956/input.tsx

@@ -0,0 +1,3 @@
+/** @jsx h */
+/** @jsxRuntime classic */
+export const foo = <div></div>

tasks/transform_conformance/tests/babel-plugin-transform-typescript/test/fixtures/jsx/issue-10956/options.json

@@ -0,0 +1,6 @@
+{
+  "plugins": [
+    ["transform-typescript", { "onlyRemoveTypeImports": true }],
+    "transform-react-jsx"
+  ]
+}

tasks/transform_conformance/tests/babel-plugin-transform-typescript/test/fixtures/jsx/issue-10956/output.js

@@ -0,0 +1 @@
+export const foo = h("div", null);