fix: parse browser.RemoveUserContextParameters (#1905)

---------

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: sadym-chromium <[email protected]>

src/bidiMapper/BidiNoOpParser.ts

@@ -16,6 +16,7 @@
  */
 
 import type {
+  Browser,
   BrowsingContext,
   Cdp,
   Input,
@@ -29,6 +30,15 @@ import type {
 import type {BidiCommandParameterParser} from './BidiParser.js';
 
 export class BidiNoOpParser implements BidiCommandParameterParser {
+  // Browser domain
+  // keep-sorted start block=yes
+  parseRemoveUserContextParams(
+    params: unknown
+  ): Browser.RemoveUserContextParameters {
+    return params as Browser.RemoveUserContextParameters;
+  }
+  // keep-sorted end
+
   // Browsing Context domain
   // keep-sorted start block=yes
   parseActivateParams(params: unknown): BrowsingContext.ActivateParameters {

src/bidiMapper/BidiParser.ts

@@ -16,6 +16,7 @@
  */
 
 import type {
+  Browser,
   BrowsingContext,
   Cdp,
   Input,
@@ -27,6 +28,13 @@ import type {
 } from '../protocol/protocol.js';
 
 export interface BidiCommandParameterParser {
+  // Browser domain
+  // keep-sorted start block=yes
+  parseRemoveUserContextParams(
+    params: unknown
+  ): Browser.RemoveUserContextParameters;
+  // keep-sorted end
+
   // Browsing Context domain
   // keep-sorted start block=yes
   parseActivateParams(params: unknown): BrowsingContext.ActivateParameters;

src/bidiMapper/CommandProcessor.ts

@@ -156,7 +156,7 @@ export class CommandProcessor extends EventEmitter<CommandProcessorEventsMap> {
         return await this.#browserProcessor.getUserContexts();
       case 'browser.removeUserContext':
         return await this.#browserProcessor.removeUserContext(
-          command.params.userContext
+          this.#parser.parseRemoveUserContextParams(command.params)
         );
       // keep-sorted end
 

src/bidiMapper/domains/browser/BrowserProcessor.ts

@@ -61,8 +61,9 @@ export class BrowserProcessor {
   }
 
   async removeUserContext(
-    userContext: Browser.UserContext
+    params: Browser.RemoveUserContextParameters
   ): Promise<EmptyResult> {
+    const userContext = params.userContext;
     if (userContext === 'default') {
       throw new InvalidArgumentException(
         '`default` user context cannot be removed'

src/bidiTab/BidiParser.ts

@@ -16,6 +16,7 @@
  */
 import type {BidiCommandParameterParser} from '../bidiMapper/BidiMapper.js';
 import type {
+  Browser,
   BrowsingContext,
   Cdp,
   Input,
@@ -28,6 +29,15 @@ import type {
 import * as Parser from '../protocol-parser/protocol-parser.js';
 
 export class BidiParser implements BidiCommandParameterParser {
+  // Browser domain
+  // keep-sorted start block=yes
+  parseRemoveUserContextParams(
+    params: unknown
+  ): Browser.RemoveUserContextParameters {
+    return Parser.Browser.parseRemoveUserContextParams(params);
+  }
+  // keep-sorted end
+
   // Browsing Context domain
   // keep-sorted start block=yes
   parseActivateParams(params: unknown): BrowsingContext.ActivateParameters {

src/protocol-parser/protocol-parser.ts

@@ -46,6 +46,18 @@ export function parseObject<T extends ZodType>(
   throw new InvalidArgumentException(errorMessage);
 }
 
+/** @see https://w3c.github.io/webdriver-bidi/#module-browser */
+export namespace Browser {
+  export function parseRemoveUserContextParams(
+    params: unknown
+  ): Protocol.Browser.RemoveUserContextParameters {
+    return parseObject(
+      params,
+      WebDriverBidi.Browser.RemoveUserContextParametersSchema
+    );
+  }
+}
+
 /** @see https://w3c.github.io/webdriver-bidi/#module-network */
 export namespace Network {
   export function parseAddInterceptParameters(params: unknown) {