Revert "Bugfix - include extra query parameters in ExtraParameters in PlatformAuthRequest" (#8565)

Reverts #8560

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lalimasharda <26092202+lalimasharda@users.noreply.github.com>

Author: lalimasharda

change/@azure-msal-browser-5eb1d50e-211d-4f44-9c4a-4b5139543531.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Revert extra query parameters forwarding to native broker [#8565](https://github.com/AzureAD/microsoft-authentication-library-for-js/pull/8565)",
+  "packageName": "@azure/msal-browser",
+  "email": "copilot@microsoft.com",
+  "dependentChangeType": "patch"
+}

change/@azure-msal-browser-60df5ac5-af09-46f9-b318-efaf42240e4a.json

@@ -983,8 +983,7 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
                 : this.config.auth.clientCapabilities;
 
         // scopes are expected to be received by the native broker as "scope" and will be added to the request below. Other properties that should be dropped from the request to the native broker can be included in the object destructuring here.
-        const { scopes, claims, extraQueryParameters, ...remainingProperties } =
-            request;
+        const { scopes, claims, ...remainingProperties } = request;
         const scopeSet = new ScopeSet(scopes || []);
         scopeSet.appendScopes(Constants.OIDC_DEFAULT_SCOPES);
 
@@ -1014,7 +1013,6 @@ export class PlatformAuthInteractionClient extends BaseInteractionClient {
             tokenType: request.authenticationScheme,
             windowTitleSubstring: document.title,
             extraParameters: {
-                ...extraQueryParameters,
                 ...request.extraParameters,
             },
             extendedExpiryToken: false, // Make this configurable?

lib/msal-browser/src/interaction_client/PlatformAuthInteractionClient.ts

@@ -311,93 +311,6 @@ describe("PlatformAuthInteractionClient Tests", () => {
             );
         });
 
-        it("Extension: token request merges extraQueryParameters into extraParameters", async () => {
-            const sendMessageSpy = jest
-                .spyOn(PlatformAuthExtensionHandler.prototype, "sendMessage")
-                .mockImplementation((): Promise<PlatformAuthResponse> => {
-                    return Promise.resolve(MOCK_WAM_RESPONSE);
-                });
-
-            await platformAuthInteractionClient.acquireToken({
-                scopes: ["User.Read"],
-                extraQueryParameters: {
-                    testEQP1: "testEQP1",
-                    testEQP2: "testEQP2",
-                },
-                extraParameters: {
-                    testEP1: "testEP1",
-                },
-            });
-
-            expect(sendMessageSpy).toHaveProperty(
-                "mock.calls[0][0].extraParameters",
-                {
-                    telemetry: "MATS",
-                    testEQP1: "testEQP1",
-                    testEQP2: "testEQP2",
-                    testEP1: "testEP1",
-                    "x-client-xtra-sku": `${BrowserConstants.MSAL_SKU}|${version},|,|,|`,
-                }
-            );
-        });
-
-        it("DOM API: token request merges extraQueryParameters into extraParameters", async () => {
-            platformAuthDOMHandler = new PlatformAuthDOMHandler(
-                pca.getLogger(),
-                getDefaultPerformanceClient(),
-                "test-correlation-id"
-            );
-
-            const testInteractionClient = new PlatformAuthInteractionClient(
-                // @ts-ignore
-                pca.config,
-                // @ts-ignore
-                pca.browserStorage,
-                // @ts-ignore
-                pca.browserCrypto,
-                pca.getLogger(),
-                // @ts-ignore
-                pca.eventHandler,
-                // @ts-ignore
-                pca.navigationClient,
-                ApiId.acquireTokenRedirect,
-                perfClient,
-                platformAuthDOMHandler,
-                "nativeAccountId",
-                // @ts-ignore
-                pca.nativeInternalStorage,
-                RANDOM_TEST_GUID
-            );
-
-            const sendMessageSpy = jest
-                .spyOn(PlatformAuthDOMHandler.prototype, "sendMessage")
-                .mockImplementation((): Promise<PlatformAuthResponse> => {
-                    return Promise.resolve(MOCK_WAM_RESPONSE);
-                });
-
-            await testInteractionClient.acquireToken({
-                scopes: ["User.Read"],
-                extraQueryParameters: {
-                    testEQP1: "testEQP1",
-                    testEQP2: "testEQP2",
-                },
-                extraParameters: {
-                    testEP1: "testEP1",
-                },
-            });
-
-            expect(sendMessageSpy).toHaveProperty(
-                "mock.calls[0][0].extraParameters",
-                {
-                    telemetry: "MATS",
-                    testEQP1: "testEQP1",
-                    testEQP2: "testEQP2",
-                    testEP1: "testEP1",
-                    "x-client-xtra-sku": `${BrowserConstants.MSAL_SKU}|${version},|,|,|`,
-                }
-            );
-        });
-
         it("DOM API: acquires token successfully", async () => {
             platformAuthDOMHandler = new PlatformAuthDOMHandler(
                 pca.getLogger(),