fix: Do not generate React Native build targets by default (#3943)

* fix: Do not generate React Native build targets by default

Previously, the SDK generator always included React Native build targets
(warp target, tsconfig reference, package.json exports/imports) for all
generated packages. This caused CI failures in the azure-sdk-for-js
monorepo because most packages don't need RN support and the RN build
infrastructure isn't fully set up.

This change:
- Adds a new `generateReactNativeTarget` option (default: false)
- Updates buildWarpConfig to conditionally include the react-native target
- Updates buildTsConfig to conditionally include tsconfig.src.react-native.json
- Updates package.json generation to conditionally include:
  - `react-native` top-level field
  - `react-native` condition in exports
  - `react-native` in imports subpath patterns
  - `react-native` in tshy esmDialects (for non-monorepo packages)
- Updates tests to verify the new default behavior

Packages that need React Native support can opt-in by setting
`generateReactNativeTarget: true` in the emitter options.

Fixes: Azure/azure-sdk-for-js#37XXX

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* chore: Regenerate test fixtures without react-native targets

Updates test fixtures to reflect the new default behavior where
react-native build targets are not generated unless explicitly enabled.

- Updated RLC integration tests (removed react-native tsconfig references)
- Updated smoke tests (removed react-native from package.json)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* chore: Regenerate all test fixtures without react-native targets

Regenerated HLC integration tests, RLC integration tests, and
version-tolerance tests with the new codegen that excludes
react-native targets by default.

Note: Smoke tests could not be regenerated locally due to environment
limitations but were manually updated in the previous commit.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* feat: remove react-native from HLC generator by default

Add generateReactNativeTarget option to AutorestOptions interface and
update packageFileGenerator.ts to conditionally include react-native
targets based on this option (defaults to false).

Regenerated all HLC test fixtures to remove react-native from package.json.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* chore: remove react-native from smoke and typespec test fixtures

Manual update since regeneration scripts cannot run locally.
These match the expected output from the generators with
generateReactNativeTarget: false (default).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* chore: regenerate TypeSpec test fixtures with proper emitter

Previous manual update missed react-native entries in sub-exports.
Proper regeneration removes all react-native from all export paths.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: wire generateReactNativeTarget option through TypeSpec emitter

The TypeSpec emitter checked option.generateReactNativeTarget but never
wired it from emitter options. Add the option to EmitterOptions interface,
RLCOptionsSchema, and extractRLCOptions so TypeSpec users can opt in via
tspconfig.yaml.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Deyaa Eldeen <deyaaeldeen@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Jialin Huang <jialinhuang@microsoft.com>
Co-authored-by: JialinHuang803 <139532647+JialinHuang803@users.noreply.github.com>

Author: 3 people

packages/autorest.typescript/src/autorestSession.ts

@@ -44,6 +44,10 @@ export interface AutorestOptions {
   lenientModelDeduplication?: boolean;
   useLegacyLro?: boolean;
   flavor?: PackageFlavor;
+  /**
+   * When true, generates React Native build targets. Defaults to false.
+   */
+  generateReactNativeTarget?: boolean;
 }
 
 let host: AutorestExtensionHost;

packages/autorest.typescript/src/generators/static/packageFileGenerator.ts

@@ -8,7 +8,26 @@ import { getAutorestOptions, getSession } from "../../autorestSession";
 import { hasPollingOperations } from "../../restLevelClient/helpers/hasPollingOperations";
 import { NameType, normalizeName } from "../../utils/nameUtils";
 import { getSecurityInfoFromModel } from "../../utils/schemaHelpers";
-import { WarpConfigTemplate } from "@azure-tools/rlc-common";
+
+/** Warp config template for HLC packages (does not include react-native by default). */
+const WarpConfigTemplate = `# warp.config.yml — build configuration
+
+exports:
+{{exports}}
+
+targets:
+  - name: browser
+    tsconfig: "./config/tsconfig.src.browser.json"
+
+  - name: esm
+    condition: import
+    tsconfig: "./config/tsconfig.src.esm.json"
+
+  - name: commonjs
+    condition: require
+    tsconfig: "./config/tsconfig.src.cjs.json"
+    moduleType: commonjs
+`;
 
 export function generatePackageJson(
   project: Project,
@@ -62,7 +81,8 @@ function regularAutorestPackage(
     generateSample,
     coreHttpCompatMode,
     azureSdkForJs,
-    isTestPackage
+    isTestPackage,
+    generateReactNativeTarget
   } = getAutorestOptions();
   const { model } = getSession();
   const { addCredentials } = getSecurityInfoFromModel(model.security);
@@ -163,33 +183,38 @@ function regularAutorestPackage(
     },
     autoPublish: true,
     browser: "./dist/browser/index.js",
-    "react-native": "./dist/react-native/index.js",
   };
+  if (generateReactNativeTarget) {
+    packageInfo["react-native"] = "./dist/react-native/index.js";
+  }
   if (azureOutputDirectory) {
     packageInfo.homepage = `https://github.com/Azure/azure-sdk-for-js/tree/main/${azureOutputDirectory}/README.md`;
   }
   if (azureSdkForJs) {
     // Azure monorepo packages use warp (via dev-tool run build-package) instead of tshy
+    const exportsEntry: Record<string, any> = {
+      browser: {
+        types: "./dist/browser/index.d.ts",
+        default: "./dist/browser/index.js"
+      },
+      import: {
+        types: "./dist/esm/index.d.ts",
+        default: "./dist/esm/index.js"
+      },
+      require: {
+        types: "./dist/commonjs/index.d.ts",
+        default: "./dist/commonjs/index.js"
+      }
+    };
+    if (generateReactNativeTarget) {
+      exportsEntry["react-native"] = {
+        types: "./dist/react-native/index.d.ts",
+        default: "./dist/react-native/index.js"
+      };
+    }
     packageInfo.exports = {
       "./package.json": "./package.json",
-      ".": {
-        browser: {
-          types: "./dist/browser/index.d.ts",
-          default: "./dist/browser/index.js"
-        },
-        "react-native": {
-          types: "./dist/react-native/index.d.ts",
-          default: "./dist/react-native/index.js"
-        },
-        import: {
-          types: "./dist/esm/index.d.ts",
-          default: "./dist/esm/index.js"
-        },
-        require: {
-          types: "./dist/commonjs/index.d.ts",
-          default: "./dist/commonjs/index.js"
-        }
-      }
+      ".": exportsEntry
     };
     packageInfo.devDependencies["@azure/dev-tool"] = "workspace:^";
     packageInfo.devDependencies["cross-env"] = "catalog:";
@@ -210,13 +235,16 @@ function regularAutorestPackage(
     packageInfo.scripts["format"] = "prettier --write --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.{ts,cts,mts}\" \"test/**/*.{ts,cts,mts}\" \"*.{js,cjs,mjs,json}\" ";
   } else {
     // Non-Azure packages use tshy for building
+    const esmDialects = generateReactNativeTarget
+      ? ["browser", "react-native"]
+      : ["browser"];
     packageInfo.tshy = {
       exports: {
         "./package.json": "./package.json",
         ".": "./src/index.ts"
       },
       dialects: ["esm", "commonjs"],
-      esmDialects: ["browser", "react-native"],
+      esmDialects,
       selfLink: false
     };
     packageInfo.devDependencies["@rollup/plugin-commonjs"] = "^24.0.0";

packages/autorest.typescript/src/restLevelClient/generateRestLevel.ts

@@ -138,7 +138,9 @@ export async function generateRestLevelClient() {
     if (azureSdkForJs) {
       generateFileByBuilder(project, buildTsSrcEsmConfig, rlcModels);
       generateFileByBuilder(project, buildTsSrcBrowserConfig, rlcModels);
-      generateFileByBuilder(project, buildTsSrcReactNativeConfig, rlcModels);
+      if (rlcModels.options?.generateReactNativeTarget) {
+        generateFileByBuilder(project, buildTsSrcReactNativeConfig, rlcModels);
+      }
       generateFileByBuilder(project, buildTsSrcCjsConfig, rlcModels);
       if (generateSample) {
         // buildTsSampleConfig

packages/autorest.typescript/src/utils/autorestOptions.ts

@@ -80,9 +80,21 @@ export async function extractAutorestOptions(): Promise<AutorestOptions> {
     dependencyInfo,
     useLegacyLro,
     flavor,
+    generateReactNativeTarget: await getGenerateReactNativeTarget(host)
   };
 }
 
+async function getGenerateReactNativeTarget(
+  host: AutorestExtensionHost
+): Promise<boolean> {
+  const generateReactNativeTarget = await host.getValue(
+    "generate-react-native-target"
+  );
+  return generateReactNativeTarget === null
+    ? false
+    : Boolean(generateReactNativeTarget);
+}
+
 async function getUseLegacyLro(host: AutorestExtensionHost): Promise<boolean> {
   const useLegacyLroOption = await host.getValue("use-legacy-lro");
   return useLegacyLroOption === null ? false : Boolean(useLegacyLroOption);

packages/autorest.typescript/test/integration/generated/additionalProperties/package.json

@@ -72,11 +72,10 @@
   },
   "autoPublish": true,
   "browser": "./dist/browser/index.js",
-  "react-native": "./dist/react-native/index.js",
   "tshy": {
     "exports": { "./package.json": "./package.json", ".": "./src/index.ts" },
     "dialects": ["esm", "commonjs"],
-    "esmDialects": ["browser", "react-native"],
+    "esmDialects": ["browser"],
     "selfLink": false
   }
 }

packages/autorest.typescript/test/integration/generated/appconfiguration/package.json

@@ -56,18 +56,13 @@
   },
   "autoPublish": true,
   "browser": "./dist/browser/index.js",
-  "react-native": "./dist/react-native/index.js",
   "exports": {
     "./package.json": "./package.json",
     ".": {
       "browser": {
         "types": "./dist/browser/index.d.ts",
         "default": "./dist/browser/index.js"
       },
-      "react-native": {
-        "types": "./dist/react-native/index.d.ts",
-        "default": "./dist/react-native/index.js"
-      },
       "import": {
         "types": "./dist/esm/index.d.ts",
         "default": "./dist/esm/index.js"

packages/autorest.typescript/test/integration/generated/appconfiguration/warp.config.yml

@@ -8,9 +8,6 @@ targets:
   - name: browser
     tsconfig: "./config/tsconfig.src.browser.json"
 
-  - name: react-native
-    tsconfig: "./config/tsconfig.src.react-native.json"
-
   - name: esm
     condition: import
     tsconfig: "./config/tsconfig.src.esm.json"

packages/autorest.typescript/test/integration/generated/appconfigurationexport/package.json

@@ -56,18 +56,13 @@
   },
   "autoPublish": true,
   "browser": "./dist/browser/index.js",
-  "react-native": "./dist/react-native/index.js",
   "exports": {
     "./package.json": "./package.json",
     ".": {
       "browser": {
         "types": "./dist/browser/index.d.ts",
         "default": "./dist/browser/index.js"
       },
-      "react-native": {
-        "types": "./dist/react-native/index.d.ts",
-        "default": "./dist/react-native/index.js"
-      },
       "import": {
         "types": "./dist/esm/index.d.ts",
         "default": "./dist/esm/index.js"

packages/autorest.typescript/test/integration/generated/appconfigurationexport/warp.config.yml

@@ -8,9 +8,6 @@ targets:
   - name: browser
     tsconfig: "./config/tsconfig.src.browser.json"
 
-  - name: react-native
-    tsconfig: "./config/tsconfig.src.react-native.json"
-
   - name: esm
     condition: import
     tsconfig: "./config/tsconfig.src.esm.json"

packages/autorest.typescript/test/integration/generated/arrayConstraints/package.json

@@ -55,18 +55,13 @@
   },
   "autoPublish": true,
   "browser": "./dist/browser/index.js",
-  "react-native": "./dist/react-native/index.js",
   "exports": {
     "./package.json": "./package.json",
     ".": {
       "browser": {
         "types": "./dist/browser/index.d.ts",
         "default": "./dist/browser/index.js"
       },
-      "react-native": {
-        "types": "./dist/react-native/index.d.ts",
-        "default": "./dist/react-native/index.js"
-      },
       "import": {
         "types": "./dist/esm/index.d.ts",
         "default": "./dist/esm/index.js"