fix(capture): suppress mypy arg-type for page_load_strategy (#37)

* fix(capture): suppress mypy arg-type for Playwright page_load_strategy

Playwright's page.goto() expects a Literal type for wait_until but
the parameter is str for API flexibility. Pre-existing issue exposed
by extraction into _run_browser_session(). Dual ignore covers both
environments (with and without Playwright stubs).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix(build): add playwright stubs to pre-commit mypy hook

The mypy pre-commit hook ran without Playwright type stubs, so it
treated all Playwright types as Any — hiding real type errors and
flagging legitimate type: ignore comments as unused. Adding
playwright>=1.40 to additional_dependencies makes the hook see the
same types as the release script and local development.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Ken Schulz <kwschulz@users.noreply.github.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: kwschulz

.pre-commit-config.yaml

@@ -53,6 +53,7 @@ repos:
       - id: mypy
         additional_dependencies:
           - typer>=0.12
+          - playwright>=1.40
         args: [--strict, --ignore-missing-imports]
         pass_filenames: false
         entry: mypy src/

src/har_capture/capture/browser.py

@@ -554,7 +554,7 @@ def _on_frame_navigated(frame: Any) -> None:
                 page.wait_for_load_state("domcontentloaded")
                 _quiescence_disabled[0] = True
         else:
-            page.goto(target_url, wait_until=page_load_strategy)
+            page.goto(target_url, wait_until=page_load_strategy)  # type: ignore[arg-type]
 
         if wait_for_data and not _quiescence_disabled[0]:
             _wait_for_network_quiescence(page)