Fix(playwright): fix ESM/CommonJS mismatch causing "exports/require i… (Azure#35927)

kashish2508 · Copilot · web-flow · commit 55b45ae99e6d · 2025-09-15T12:19:18.000+05:30
…s not defined" by converting samples to ESM and updating configs ### Packages impacted by this PR ### Issues associated with this PR ### Describe the problem that is addressed by this PR ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary) --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/customising-service-parameters/package.json b/sdk/loadtesting/playwright/samples/v1/javascript/customising-service-parameters/package.json
@@ -1,6 +1,7 @@
 {
   "name": "customising-service-parameters",
   "version": "1.0.0",
+  "type": "module",
   "main": "index.js",
   "scripts": {},
   "keywords": [],
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/customising-service-parameters/playwright.config.js b/sdk/loadtesting/playwright/samples/v1/javascript/customising-service-parameters/playwright.config.js
@@ -1,5 +1,5 @@
 // @ts-check
-const { defineConfig, devices } = require('@playwright/test');
+import { defineConfig, devices } from '@playwright/test';
 
 /**
  * Read environment variables from file.
@@ -10,7 +10,7 @@ const { defineConfig, devices } = require('@playwright/test');
 /**
  * @see https://playwright.dev/docs/test-configuration
  */
-module.exports = defineConfig({
+export default defineConfig({
   testDir: './tests',
   /* Run tests in files in parallel */
   fullyParallel: true,
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/customising-service-parameters/playwright.service.config.js b/sdk/loadtesting/playwright/samples/v1/javascript/customising-service-parameters/playwright.service.config.js
@@ -1,10 +1,9 @@
-const { AzureCliCredential } = require("@azure/identity");
-const { createAzurePlaywrightConfig, ServiceOS, ServiceAuth } = require("@azure/playwright");
-const { defineConfig } = require('@playwright/test');
-const config = require("./playwright.config");
+import { AzureCliCredential } from "@azure/identity";
+import { createAzurePlaywrightConfig, ServiceOS, ServiceAuth } from "@azure/playwright";
+import { defineConfig } from '@playwright/test';
+import config from "./playwright.config.js";
 
 const azureCredential = new AzureCliCredential();
-
 const serviceAuthType = ServiceAuth.ENTRA_ID;
 const os = ServiceOS.LINUX;
 
@@ -18,5 +17,7 @@ const playwrightServiceAdditionalOptions = {
   runName: "JavaScript V1 - Sample Run", // Run name for the test run
 };
 
-
-export default defineConfig(config, createAzurePlaywrightConfig(config, playwrightServiceAdditionalOptions));
+export default defineConfig(
+  config,
+  createAzurePlaywrightConfig(config, playwrightServiceAdditionalOptions)
+);
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/customising-service-parameters/tests/sample.spec.js b/sdk/loadtesting/playwright/samples/v1/javascript/customising-service-parameters/tests/sample.spec.js
@@ -1,5 +1,5 @@
 // @ts-check
-const { test, expect } = require('@playwright/test');
+import { test, expect } from '@playwright/test';
 
 test('has title', async ({ page }) => {
   await page.goto('https://playwright.dev/');
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/manually-connecting-to-browsers/package.json b/sdk/loadtesting/playwright/samples/v1/javascript/manually-connecting-to-browsers/package.json
@@ -1,6 +1,7 @@
 {
   "name": "using-manual-launch",
   "version": "1.0.0",
+  "type": "module",
   "main": "index.js",
   "scripts": {},
   "keywords": [],
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/manually-connecting-to-browsers/playwright.config.js b/sdk/loadtesting/playwright/samples/v1/javascript/manually-connecting-to-browsers/playwright.config.js
@@ -1,5 +1,5 @@
 // @ts-check
-const { defineConfig, devices } = require('@playwright/test');
+import { defineConfig, devices } from '@playwright/test';
 
 /**
  * Read environment variables from file.
@@ -10,7 +10,7 @@ const { defineConfig, devices } = require('@playwright/test');
 /**
  * @see https://playwright.dev/docs/test-configuration
  */
-module.exports = defineConfig({
+export default defineConfig({
   testDir: './tests',
   /* Run tests in files in parallel */
   fullyParallel: true,
@@ -30,8 +30,8 @@ module.exports = defineConfig({
     /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
     trace: 'on-first-retry',
   },
-  globalSetup: require.resolve('./global-setup'),
-  globalTeardown: require.resolve('./global-teardown'),
+  globalSetup: './global-setup.js',
+  globalTeardown: './global-teardown.js',
 
   /* Configure projects for major browsers */
   projects: [
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/manually-connecting-to-browsers/playwright.service.config.js b/sdk/loadtesting/playwright/samples/v1/javascript/manually-connecting-to-browsers/playwright.service.config.js
@@ -1,8 +1,11 @@
-const { createAzurePlaywrightConfig } = require("@azure/playwright");
-const { defineConfig } = require('@playwright/test');
-const { DefaultAzureCredential } = require("@azure/identity");
-const config = require("./playwright.config");
+import { createAzurePlaywrightConfig } from "@azure/playwright";
+import { defineConfig } from '@playwright/test';
+import { DefaultAzureCredential } from "@azure/identity";
+import config from "./playwright.config.js";
 
-export default defineConfig(config, createAzurePlaywrightConfig(config, {
-  credential: new DefaultAzureCredential()
-}));
+export default defineConfig(
+  config,
+  createAzurePlaywrightConfig(config, {
+    credential: new DefaultAzureCredential()
+  })
+);
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/manually-connecting-to-browsers/tests/sample.spec.js b/sdk/loadtesting/playwright/samples/v1/javascript/manually-connecting-to-browsers/tests/sample.spec.js
@@ -1,8 +1,7 @@
-const playwright = require("@playwright/test");
-const { getConnectOptions, ServiceOS } = require("@azure/playwright");
-const { AzureCliCredential } = require("@azure/identity");
+import { test, expect } from "@playwright/test";
+import { getConnectOptions, ServiceOS } from "@azure/playwright";
+import { AzureCliCredential } from "@azure/identity";
 
-const { test, expect } = playwright;
 
 test("has title", async ({ browserName }) => {
   const { wsEndpoint, options } = await getConnectOptions();
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/set-default-authentication-mechanism/package.json b/sdk/loadtesting/playwright/samples/v1/javascript/set-default-authentication-mechanism/package.json
@@ -1,6 +1,7 @@
 {
   "name": "set-default-authentication-mechanism",
   "version": "1.0.0",
+  "type": "module",
   "main": "index.js",
   "scripts": {},
   "keywords": [],
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/set-default-authentication-mechanism/playwright.config.js b/sdk/loadtesting/playwright/samples/v1/javascript/set-default-authentication-mechanism/playwright.config.js
@@ -1,5 +1,5 @@
 // @ts-check
-const { defineConfig, devices } = require('@playwright/test');
+import { defineConfig, devices } from '@playwright/test';
 
 /**
  * Read environment variables from file.
@@ -10,7 +10,7 @@ const { defineConfig, devices } = require('@playwright/test');
 /**
  * @see https://playwright.dev/docs/test-configuration
  */
-module.exports = defineConfig({
+export default defineConfig({
   testDir: './tests',
   /* Run tests in files in parallel */
   fullyParallel: true,
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/set-default-authentication-mechanism/playwright.service.config.js b/sdk/loadtesting/playwright/samples/v1/javascript/set-default-authentication-mechanism/playwright.service.config.js
@@ -1,11 +1,15 @@
-const { createAzurePlaywrightConfig, ServiceAuth, ServiceEnvironmentVariable } = require("@azure/playwright");
-const { defineConfig } = require('@playwright/test');
-const config = require("./playwright.config");
+import { createAzurePlaywrightConfig, ServiceAuth, ServiceEnvironmentVariable } from "@azure/playwright";
+import { defineConfig } from '@playwright/test';
+import config from "./playwright.config.js";
 
 // You can get the below values from the MPT portal. Alternatively you can directly set the environment variables PLAYWRIGHT_SERVICE_URL & PLAYWRIGHT_SERVICE_ACCESS_TOKEN
 process.env[ServiceEnvironmentVariable.PLAYWRIGHT_SERVICE_URL] = "Remote Browser URL";
 process.env[ServiceEnvironmentVariable.PLAYWRIGHT_SERVICE_ACCESS_TOKEN] = "***Access Token***";
 
-export default defineConfig(config, createAzurePlaywrightConfig(config, {
-  serviceAuthType: ServiceAuth.ACCESS_TOKEN
-}));
+
+export default defineConfig(
+  config,
+  createAzurePlaywrightConfig(config, {
+    serviceAuthType: ServiceAuth.ACCESS_TOKEN
+  })
+);
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/set-default-authentication-mechanism/tests/sample.spec.js b/sdk/loadtesting/playwright/samples/v1/javascript/set-default-authentication-mechanism/tests/sample.spec.js
@@ -1,5 +1,5 @@
 // @ts-check
-const { test, expect } = require('@playwright/test');
+import { test, expect } from "@playwright/test";
 
 test('has title', async ({ page }) => {
   await page.goto('https://playwright.dev/');
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/using-connect-options/package.json b/sdk/loadtesting/playwright/samples/v1/javascript/using-connect-options/package.json
@@ -1,6 +1,7 @@
 {
   "name": "using-connect-options",
   "version": "1.0.0",
+  "type": "module",
   "main": "index.js",
   "scripts": {},
   "keywords": [],
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/using-connect-options/playwright.config.js b/sdk/loadtesting/playwright/samples/v1/javascript/using-connect-options/playwright.config.js
@@ -1,5 +1,5 @@
 // @ts-check
-const { defineConfig, devices } = require('@playwright/test');
+import { defineConfig, devices } from '@playwright/test';
 
 /**
  * Read environment variables from file.
@@ -10,7 +10,7 @@ const { defineConfig, devices } = require('@playwright/test');
 /**
  * @see https://playwright.dev/docs/test-configuration
  */
-module.exports = defineConfig({
+export default defineConfig({
   testDir: './tests',
   /* Run tests in files in parallel */
   fullyParallel: true,
@@ -30,8 +30,8 @@ module.exports = defineConfig({
     /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
     trace: 'on-first-retry',
   },
-  globalSetup: require.resolve('./global-setup'),
-  globalTeardown: require.resolve('./global-teardown'),
+  globalSetup: './global-setup.js',
+  globalTeardown: './global-teardown.js',
 
   /* Configure projects for major browsers */
   projects: [
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/using-connect-options/playwright.service.config.js b/sdk/loadtesting/playwright/samples/v1/javascript/using-connect-options/playwright.service.config.js
@@ -1,8 +1,11 @@
-const { createAzurePlaywrightConfig } = require("@azure/playwright");
-const { defineConfig } = require('@playwright/test');
-const { DefaultAzureCredential } = require("@azure/identity");
-const config = require("./playwright.config");
+import { createAzurePlaywrightConfig } from "@azure/playwright";
+import { defineConfig } from '@playwright/test';
+import { DefaultAzureCredential } from "@azure/identity";
+import config from "./playwright.config.js";
 
-export default defineConfig(config, createAzurePlaywrightConfig(config, {
-  credential: new DefaultAzureCredential()
-}));
+export default defineConfig(
+  config,
+  createAzurePlaywrightConfig(config, {
+    credential: new DefaultAzureCredential()
+  })
+);
diff --git a/sdk/loadtesting/playwright/samples/v1/javascript/using-connect-options/tests/sample.spec.js b/sdk/loadtesting/playwright/samples/v1/javascript/using-connect-options/tests/sample.spec.js
@@ -1,5 +1,5 @@
 // @ts-check
-const { test, expect } = require('@playwright/test');
+import { test, expect } from "@playwright/test";
 
 test('has title', async ({ page }) => {
   await page.goto('https://playwright.dev/');
diff --git a/sdk/loadtesting/playwright/samples/v1/typescript/customising-service-parameters/package.json b/sdk/loadtesting/playwright/samples/v1/typescript/customising-service-parameters/package.json
@@ -1,6 +1,7 @@
 {
   "name": "customising-service-parameters",
   "version": "1.0.0",
+  "type": "module",
   "main": "index.js",
   "scripts": {},
   "keywords": [],
diff --git a/sdk/loadtesting/playwright/samples/v1/typescript/manually-connecting-to-browsers/package.json b/sdk/loadtesting/playwright/samples/v1/typescript/manually-connecting-to-browsers/package.json
@@ -1,6 +1,7 @@
 {
   "name": "using-manual-launch",
   "version": "1.0.0",
+  "type": "module",
   "main": "index.js",
   "scripts": {},
   "keywords": [],
diff --git a/sdk/loadtesting/playwright/samples/v1/typescript/manually-connecting-to-browsers/playwright.config.ts b/sdk/loadtesting/playwright/samples/v1/typescript/manually-connecting-to-browsers/playwright.config.ts
@@ -29,8 +29,8 @@ export default defineConfig({
     /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
     trace: "on-first-retry",
   },
-  globalSetup: require.resolve("./global-setup"),
-  globalTeardown: require.resolve("./global-teardown"),
+  globalSetup: "./global-setup.ts",
+  globalTeardown: "./global-teardown.ts",
 
   /* Configure projects for major browsers */
   projects: [
diff --git a/sdk/loadtesting/playwright/samples/v1/typescript/set-default-authentication-mechanism/package.json b/sdk/loadtesting/playwright/samples/v1/typescript/set-default-authentication-mechanism/package.json
@@ -1,6 +1,7 @@
 {
   "name": "set-default-authentication-mechanism",
   "version": "1.0.0",
+  "type": "module",
   "main": "index.js",
   "scripts": {},
   "keywords": [],
diff --git a/sdk/loadtesting/playwright/samples/v1/typescript/using-connect-options/global-setup.ts b/sdk/loadtesting/playwright/samples/v1/typescript/using-connect-options/global-setup.ts
@@ -3,4 +3,4 @@ const func = async function () {
   console.log("Customer Global Setup");
 };
 
-module.exports = func;
+export default func;
diff --git a/sdk/loadtesting/playwright/samples/v1/typescript/using-connect-options/package.json b/sdk/loadtesting/playwright/samples/v1/typescript/using-connect-options/package.json
@@ -1,6 +1,7 @@
 {
   "name": "using-connect-options",
   "version": "1.0.0",
+  "type": "module",
   "main": "index.js",
   "scripts": {},
   "keywords": [],
diff --git a/sdk/loadtesting/playwright/samples/v1/typescript/using-connect-options/playwright.config.ts b/sdk/loadtesting/playwright/samples/v1/typescript/using-connect-options/playwright.config.ts
@@ -29,8 +29,8 @@ export default defineConfig({
     /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
     trace: "on-first-retry",
   },
-  globalSetup: require.resolve("./global-setup"),
-  globalTeardown: require.resolve("./global-teardown"),
+  globalSetup: "./global-setup.ts",
+  globalTeardown: "./global-teardown.ts",
 
   /* Configure projects for major browsers */
   projects: [

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "customising-service-parameters",`
`3`	`3`	`"version": "1.0.0",`
	`4`	`+ "type": "module",`
`4`	`5`	`"main": "index.js",`
`5`	`6`	`"scripts": {},`
`6`	`7`	`"keywords": [],`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "using-manual-launch",`
`3`	`3`	`"version": "1.0.0",`
	`4`	`+ "type": "module",`
`4`	`5`	`"main": "index.js",`
`5`	`6`	`"scripts": {},`
`6`	`7`	`"keywords": [],`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "set-default-authentication-mechanism",`
`3`	`3`	`"version": "1.0.0",`
	`4`	`+ "type": "module",`
`4`	`5`	`"main": "index.js",`
`5`	`6`	`"scripts": {},`
`6`	`7`	`"keywords": [],`