refactor(playwright-browser-tunnel): eliminate TS-private status setter

Replace the mixed-visibility 'public get status() / private set
status()' accessor pair with a public 'get status()' plus a native
private method '#setStatus(newStatus)'. Native ECMAScript private
accessors require the getter and setter to share one name with
uniform visibility, so the private setter could not be converted to
a same-named '#status' accessor without colliding with the existing
'#status' backing field. Renaming the mutator to a private method
sidesteps that restriction while keeping the public read-only
getter contract unchanged.

Updated all 5 internal 'this.status = <value>' assignments to
'this.#setStatus(<value>)'.

This removes the last remaining TS 'private' class member across
the requested conversion scopes; only the two previously-documented
unsafe members (rush-buildxl-graph-plugin's declare-only
'_configHash' and AmazonS3Client's reflection-accessed
'_writeWarningLine') remain TS-private by design.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 8ebd5bf2-c44b-42d5-be25-e7936d4b0a14

Author: bmiddha

apps/playwright-browser-tunnel/src/PlaywrightBrowserTunnel.ts

@@ -136,8 +136,7 @@ export class PlaywrightTunnel {
     return this.#status;
   }
 
-  // eslint-disable-next-line @typescript-eslint/naming-convention
-  private set status(newStatus: TunnelStatus) {
+  #setStatus(newStatus: TunnelStatus): void {
     this.#status = newStatus;
     this.#onStatusChange(newStatus);
   }
@@ -524,7 +523,7 @@ export class PlaywrightTunnel {
     let client: WebSocket | undefined = undefined;
     let browserServer: BrowserServer | undefined = undefined;
 
-    this.status = 'waiting-for-connection';
+    this.#setStatus('waiting-for-connection');
     const ws: WebSocket =
       this.#mode === 'poll-connection'
         ? await this.#pollConnectionAsync()
@@ -543,7 +542,7 @@ export class PlaywrightTunnel {
       const reasonStr: string = reason.toString() || 'no reason provided';
       const codeDescription: string = getWebSocketCloseReason(code);
       this.#initWsPromise = undefined;
-      this.status = 'stopped';
+      this.#setStatus('stopped');
       this.#terminal.writeLine(
         `WebSocket connection closed - code: ${code} (${codeDescription}), reason: ${reasonStr}`
       );
@@ -580,7 +579,7 @@ export class PlaywrightTunnel {
               terminal.writeLine('User approved browser server launch.');
             }
 
-            this.status = 'setting-up-browser-server';
+            this.#setStatus('setting-up-browser-server');
             const browserServerProxy: IBrowserServerProxy =
               await this.#getPlaywrightBrowserServerProxyAsync(handshake);
             client = browserServerProxy.client;
@@ -600,7 +599,7 @@ export class PlaywrightTunnel {
               terminal.writeDebugLine('Warning: Browser server process handle not available for monitoring');
             }
 
-            this.status = 'browser-server-running';
+            this.#setStatus('browser-server-running');
 
             // Send ack so that the counterpart also knows to start forwarding messages.
             // NOTE: The 1-second delay is an intentional workaround. In the current
@@ -623,7 +622,7 @@ export class PlaywrightTunnel {
             resolve(ws);
           } catch (error) {
             terminal.writeLine(`Error processing handshake: ${error}`);
-            this.status = 'error';
+            this.#setStatus('error');
 
             // Cleanup and close connection on error
             ws.off('message', onMessageHandler);