fix(robot): fix viewport type and update ADR with Playwright findings

Cast viewport width/height to int in SalesforcePlaywright.open_test_browser.
Update ADR 0004 to note pre-existing wait_until_salesforce_is_ready bug
that prevents Playwright E2E execution.

Author: jstvz

cumulusci/robotframework/SalesforcePlaywright.py

@@ -107,7 +107,8 @@ def open_test_browser(
 
         browser_id = self.browser.new_browser(browser=browser_enum, headless=headless)
         context_id = self.browser.new_context(
-            viewport={"width": width, "height": height}, recordVideo=record_video
+            viewport={"width": int(width), "height": int(height)},
+            recordVideo=record_video,
         )
         self.browser.set_browser_timeout("15 seconds")
         page_details = self.browser.new_page(login_url)

docs/adrs/0004-robot-framework-selenium-vs-playwright.md

@@ -74,9 +74,10 @@ The PoC produced concrete evidence for this recommendation:
 
 **Track B — Playwright keyword port:**
 
--   **~8 keywords** were ported using accessibility-first selectors (`role=`, `text=`).
+-   **10 keywords** were ported using accessibility-first selectors (CSS, `text=`, SLDS classes).
 -   **Zero version-specific locators** were required.
 -   Playwright's **auto-wait** eliminated the need for `@selenium_retry` decorators entirely.
+-   Pre-existing bug in `SalesforcePlaywright.wait_until_salesforce_is_ready` prevents end-to-end execution (URL regex mismatch). Keywords are structurally correct; test infrastructure needs a separate fix.
 
 **Shadow DOM evidence:** The actions ribbon and List View Controls button are confirmed to render inside shadow DOM. This class of failure grows with each API version as Salesforce migrates more components from Aura to LWC.