Add is_cdp flag and CDP robustness improvements

[wwakabobik]

Summary

• Add DriverWrapper.is_cdp flag to identify CDP-connected driver instances
• Extend quit() across Playwright and Selenium drivers for CDP robustness
• Null-safe viewport handling for CDP connections
• Version bump: 3.4.2 → 3.4.3

Problem

When users connect to a remote browser via CDP (e.g., Electron app with --remote-debugging-port=9222) and wrap it in a DriverWrapper, several runtime issues occur:

• PlayDriver.quit() calls tracing.stop() which fails because tracing was never started on pre-existing CDP contexts
• PlayDriver.quit() calls page.close() / context.close() which can throw PlaywrightError for externally-managed browsers
• CoreDriver.quit() calls driver.quit() which can throw WebDriverException for externally-managed browsers
• PlayDriver.get_inner_window_size() crashes when viewport_size is None (default for CDP pages without explicit viewport)

These are runtime bugs that every project connecting via CDP would hit independently.

Solution

A new is_cdp boolean flag on DriverWrapper (default False) that users set after creating their CDP connection:

from playwright.sync_api import sync_playwright
from mops.base.driver_wrapper import DriverWrapper
from mops.mixins.objects.driver import Driver

pw = sync_playwright().start()
browser = pw.chromium.connect_over_cdp("http://localhost:9222")
context = browser.contexts[0]
page = context.pages[0]

wrapper = DriverWrapper(Driver(driver=page, context=context, instance=browser))
wrapper.is_cdp = True  # enables CDP robustness

# ... use wrapper normally ...
wrapper.quit()
pw.stop()

Driver creation stays on the user's project side (per wrapper-pattern rules). MOPS only provides the robustness layer via is_cdp.

Robustness changes

Driver	Change
PlayDriver.quit()	tracing.stop() skipped when is_cdp=True; page.close() and context.close() wrapped in try/except PlaywrightError for CDP only
CoreDriver.quit()	driver.quit() wrapped in try/except SeleniumWebDriverException for CDP only
PlayDriver.get_inner_window_size()	Returns Size(0, 0) when viewport_size is None

Non-CDP behavior is completely unchanged — error suppression only applies when is_cdp=True.

Files Changed

File	Change
mops/__init__.py	Version bump 3.4.2 → 3.4.3
mops/base/driver_wrapper.py	is_cdp flag
mops/abstraction/driver_wrapper_abc.py	is_cdp attribute in ABC
mops/playwright/play_driver.py	quit(): tracing skip + try/except for CDP; get_inner_window_size() null-safe
mops/selenium/core/core_driver.py	quit(): try/except guarded by is_cdp
docs/source/driver_wrapper/index.md	is_cdp in status attributes
CHANGELOG.md	v3.4.3 entry
tests/static_tests/unit/test_cdp_robustness.py	11 unit tests

Backward Compatibility

Purely additive — no existing attributes, methods, or signatures change. is_cdp defaults to False. PlayDriver.quit() and CoreDriver.quit() changes only add safety guards that activate when is_cdp=True.

Test Plan

• 4 is_cdp flag tests: default False (Playwright/Selenium), set True (Playwright/Selenium)
• 3 PlayDriver CDP quit tests: suppress page close error, suppress context close error, skip tracing
• 1 PlayDriver non-CDP quit test: tracing still called
• 1 PlayDriver non-CDP quit test: close error propagates (not suppressed)
• 1 CoreDriver CDP quit test: suppress WebDriverException
• 1 CoreDriver non-CDP quit test: WebDriverException propagates
• 2 viewport null-safe tests: None → Size(0,0), set → correct values
• All 390 existing + new static tests pass
• All 15 ABC compatibility tests pass

[VladimirPodolian]

@wwakabobik thanks for the contribution, but this PR cannot be accepted in its current state as it violates the wrapper pattern rules. DriverWrapper should not be responsible for creating new driver connections. The driver setup should be handled on the user's project side instead, since there can be a wide variety of settings and capabilities involved — custom browser options, capability configs, environment-specific flags, and so on. The wrapper's sole responsibility is to wrap an already-initialized driver, not to manage its creation or lifecycle bootstrapping even with cdp connection. That said, the is_cdp flag and the related robustness changes in quit() and get_inner_window_size() look genuinely useful and are worth keeping

[wwakabobik]

Updated ro conform changes proposed/approved by maintainer.