From bbdf2a80c701ad428b4f7a55580a9b9ef15a0228 Mon Sep 17 00:00:00 2001
From: Uriah Finkel <Ufinkel@gmail.com>
Date: Mon, 24 Aug 2026 18:39:11 +0300
Subject: [PATCH 1/9] test: exercise summary report in headless browser

---
 tests/test_summary_report_browser.py | 49 ++++++++++++++++++++++++++++
 1 file changed, 49 insertions(+)

diff --git a/tests/test_summary_report_browser.py b/tests/test_summary_report_browser.py
index 7c32b8ed..406ef989 100644
--- a/tests/test_summary_report_browser.py
+++ b/tests/test_summary_report_browser.py
@@ -1,8 +1,11 @@
 import json
+import shutil
+import subprocess
 from pathlib import Path
 from typing import Any, cast
 
 import numpy as np
+import pytest
 
 from rtichoke.summary_report import summary_report as summary_report_module
 from rtichoke.summary_report.summary_report import create_summary_report
@@ -38,6 +41,33 @@ def _embedded_report(html: str) -> dict[str, Any]:
     return cast(dict[str, Any], json.loads(html[start:end]))
 
 
+def _chrome_executable() -> str:
+    for candidate in ("google-chrome", "google-chrome-stable", "chromium", "chromium-browser"):
+        executable = shutil.which(candidate)
+        if executable is not None:
+            return executable
+    pytest.skip("headless Chrome/Chromium is not available")
+
+
+def _dump_dom(path: Path) -> subprocess.CompletedProcess[str]:
+    return subprocess.run(
+        [
+            _chrome_executable(),
+            "--headless=new",
+            "--no-sandbox",
+            "--disable-gpu",
+            "--enable-logging=stderr",
+            "--log-level=0",
+            "--dump-dom",
+            path.resolve().as_uri(),
+        ],
+        check=False,
+        capture_output=True,
+        text=True,
+        timeout=30,
+    )
+
+
 def test_default_summary_report_keeps_historical_r_backend(monkeypatch, capsys):
     probs, reals = _inputs()
     calls = []
@@ -114,6 +144,25 @@ def test_browser_summary_report_is_opt_in_and_uses_real_canonical_components(
     assert "renderCalibrationV2" not in html
 
 
+def test_browser_summary_report_executes_when_opened_directly(tmp_path):
+    probs, reals = _inputs()
+    output = create_summary_report(
+        probs,
+        reals,
+        renderer="browser",
+        output_file=tmp_path / "browser_report.html",
+    )
+    assert isinstance(output, Path)
+
+    browser = _dump_dom(output)
+
+    assert browser.returncode == 0, browser.stderr
+    assert 'id="rtichoke-report"></div>' not in browser.stdout
+    assert "Performance" in browser.stdout
+    assert "ROC" in browser.stdout
+    assert "Calibration" in browser.stdout
+
+
 def test_browser_summary_report_preserves_component_local_identity(tmp_path):
     probs = {
         "Population A": np.array([0.05, 0.2, 0.7, 0.95]),

From 39c0baa475387d85a2cd6d086b2f390dabe52eda Mon Sep 17 00:00:00 2001
From: Uriah Finkel <Ufinkel@gmail.com>
Date: Mon, 24 Aug 2026 18:40:35 +0300
Subject: [PATCH 2/9] style: format browser smoke test

---
 tests/test_summary_report_browser.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/tests/test_summary_report_browser.py b/tests/test_summary_report_browser.py
index 406ef989..1306a202 100644
--- a/tests/test_summary_report_browser.py
+++ b/tests/test_summary_report_browser.py
@@ -42,7 +42,12 @@ def _embedded_report(html: str) -> dict[str, Any]:
 
 
 def _chrome_executable() -> str:
-    for candidate in ("google-chrome", "google-chrome-stable", "chromium", "chromium-browser"):
+    for candidate in (
+        "google-chrome",
+        "google-chrome-stable",
+        "chromium",
+        "chromium-browser",
+    ):
         executable = shutil.which(candidate)
         if executable is not None:
             return executable

From 103d978f3504ea51ed2cbddf374094228b57c727 Mon Sep 17 00:00:00 2001
From: Uriah Finkel <Ufinkel@gmail.com>
Date: Mon, 24 Aug 2026 18:43:22 +0300
Subject: [PATCH 3/9] test: compare file and localhost browser execution

---
 tests/test_summary_report_browser.py | 66 ++++++++++++++++++++++++----
 1 file changed, 57 insertions(+), 9 deletions(-)

diff --git a/tests/test_summary_report_browser.py b/tests/test_summary_report_browser.py
index 1306a202..ae0c525f 100644
--- a/tests/test_summary_report_browser.py
+++ b/tests/test_summary_report_browser.py
@@ -1,8 +1,12 @@
 import json
 import shutil
 import subprocess
+from contextlib import contextmanager
+from functools import partial
+from http.server import SimpleHTTPRequestHandler, ThreadingHTTPServer
 from pathlib import Path
-from typing import Any, cast
+from threading import Thread
+from typing import Any, Iterator, cast
 
 import numpy as np
 import pytest
@@ -54,7 +58,7 @@ def _chrome_executable() -> str:
     pytest.skip("headless Chrome/Chromium is not available")
 
 
-def _dump_dom(path: Path) -> subprocess.CompletedProcess[str]:
+def _dump_dom(url: str) -> subprocess.CompletedProcess[str]:
     return subprocess.run(
         [
             _chrome_executable(),
@@ -64,7 +68,7 @@ def _dump_dom(path: Path) -> subprocess.CompletedProcess[str]:
             "--enable-logging=stderr",
             "--log-level=0",
             "--dump-dom",
-            path.resolve().as_uri(),
+            url,
         ],
         check=False,
         capture_output=True,
@@ -73,6 +77,38 @@ def _dump_dom(path: Path) -> subprocess.CompletedProcess[str]:
     )
 
 
+def _rendered_report_html(dom: str) -> str:
+    marker = '<div id="rtichoke-report">'
+    start = dom.index(marker) + len(marker)
+    end = dom.index('<script id="rtichoke-report-spec"', start)
+    return dom[start:end]
+
+
+@contextmanager
+def _serve(directory: Path) -> Iterator[str]:
+    handler = partial(SimpleHTTPRequestHandler, directory=str(directory))
+    server = ThreadingHTTPServer(("127.0.0.1", 0), handler)
+    thread = Thread(target=server.serve_forever, daemon=True)
+    thread.start()
+    try:
+        yield f"http://127.0.0.1:{server.server_port}"
+    finally:
+        server.shutdown()
+        thread.join()
+        server.server_close()
+
+
+def _assert_report_rendered(browser: subprocess.CompletedProcess[str]) -> None:
+    assert browser.returncode == 0, browser.stderr
+    rendered = _rendered_report_html(browser.stdout)
+    assert rendered.strip(), browser.stderr
+    assert "Performance" in rendered
+    assert "ROC" in rendered
+    assert "Calibration" in rendered
+    assert "<table" in rendered
+    assert rendered.count("<svg") >= 2
+
+
 def test_default_summary_report_keeps_historical_r_backend(monkeypatch, capsys):
     probs, reals = _inputs()
     calls = []
@@ -159,13 +195,25 @@ def test_browser_summary_report_executes_when_opened_directly(tmp_path):
     )
     assert isinstance(output, Path)
 
-    browser = _dump_dom(output)
+    browser = _dump_dom(output.resolve().as_uri())
 
-    assert browser.returncode == 0, browser.stderr
-    assert 'id="rtichoke-report"></div>' not in browser.stdout
-    assert "Performance" in browser.stdout
-    assert "ROC" in browser.stdout
-    assert "Calibration" in browser.stdout
+    _assert_report_rendered(browser)
+
+
+def test_browser_summary_report_executes_over_localhost(tmp_path):
+    probs, reals = _inputs()
+    output = create_summary_report(
+        probs,
+        reals,
+        renderer="browser",
+        output_file=tmp_path / "browser_report.html",
+    )
+    assert isinstance(output, Path)
+
+    with _serve(output.parent) as base_url:
+        browser = _dump_dom(f"{base_url}/{output.name}")
+
+    _assert_report_rendered(browser)
 
 
 def test_browser_summary_report_preserves_component_local_identity(tmp_path):

From 45d1d463735adc81fcdf485e188a6b76deebab25 Mon Sep 17 00:00:00 2001
From: Uriah Finkel <Ufinkel@gmail.com>
Date: Mon, 24 Aug 2026 18:45:16 +0300
Subject: [PATCH 4/9] test: surface direct-open browser errors

---
 tests/test_summary_report_browser.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tests/test_summary_report_browser.py b/tests/test_summary_report_browser.py
index ae0c525f..280a33a4 100644
--- a/tests/test_summary_report_browser.py
+++ b/tests/test_summary_report_browser.py
@@ -101,8 +101,7 @@ def _serve(directory: Path) -> Iterator[str]:
 def _assert_report_rendered(browser: subprocess.CompletedProcess[str]) -> None:
     assert browser.returncode == 0, browser.stderr
     rendered = _rendered_report_html(browser.stdout)
-    assert rendered.strip(), browser.stderr
-    assert "Performance" in rendered
+    assert "Performance" in rendered, browser.stderr
     assert "ROC" in rendered
     assert "Calibration" in rendered
     assert "<table" in rendered

From f2d108deebf7e57768bd4575952699b950b3d76d Mon Sep 17 00:00:00 2001
From: Uriah Finkel <Ufinkel@gmail.com>
Date: Mon, 24 Aug 2026 18:46:08 +0300
Subject: [PATCH 5/9] fix: inline shared report renderer for offline HTML

---
 src/rtichoke/_report_browser.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/rtichoke/_report_browser.py b/src/rtichoke/_report_browser.py
index 4a1e1836..c3693502 100644
--- a/src/rtichoke/_report_browser.py
+++ b/src/rtichoke/_report_browser.py
@@ -24,6 +24,7 @@ def write_html(self, path: str | Path) -> Path:
             (output.parent / asset).write_bytes(vendor.joinpath(asset).read_bytes())
 
         spec_json = json.dumps(self.spec, separators=(",", ":")).replace("</", "<\\/")
+        viz_js = vendor.joinpath("rtichoke-viz.js").read_text(encoding="utf-8")
         html = f"""<!doctype html>
 <html lang="en">
 <head>
@@ -36,7 +37,7 @@ def write_html(self, path: str | Path) -> Path:
   <div id="rtichoke-report"></div>
   <script id="rtichoke-report-spec" type="application/json">{spec_json}</script>
   <script type="module">
-    import {{ renderReport }} from "./rtichoke-viz.js";
+{viz_js}
     const spec = JSON.parse(
       document.querySelector("#rtichoke-report-spec").textContent
     );

From 8ffa18da955e60520e18580e8f5ef52b537adfe2 Mon Sep 17 00:00:00 2001
From: Uriah Finkel <Ufinkel@gmail.com>
Date: Mon, 24 Aug 2026 18:46:36 +0300
Subject: [PATCH 6/9] test: assert shared renderer is embedded unchanged

---
 tests/test_summary_report_browser.py | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/tests/test_summary_report_browser.py b/tests/test_summary_report_browser.py
index 280a33a4..ede6be11 100644
--- a/tests/test_summary_report_browser.py
+++ b/tests/test_summary_report_browser.py
@@ -160,7 +160,8 @@ def test_browser_summary_report_is_opt_in_and_uses_real_canonical_components(
 
     assert result == output
     assert output.exists()
-    assert (tmp_path / "rtichoke-viz.js").exists()
+    viz_js_path = tmp_path / "rtichoke-viz.js"
+    assert viz_js_path.exists()
     assert (tmp_path / "rtichoke-viz.css").exists()
 
     html = output.read_text(encoding="utf-8")
@@ -177,11 +178,9 @@ def test_browser_summary_report_is_opt_in_and_uses_real_canonical_components(
     ]
     assert report["components"][1]["spec"]["schemaVersion"] == "2.0"
     assert report["components"][2]["spec"]["schemaVersion"] == "2.0"
-    assert 'import { renderReport } from "./rtichoke-viz.js";' in html
+    assert 'import { renderReport } from "./rtichoke-viz.js";' not in html
+    assert viz_js_path.read_text(encoding="utf-8") in html
     assert "append(renderReport(spec))" in html
-    assert "renderPerformanceTable" not in html
-    assert "renderRocV2" not in html
-    assert "renderCalibrationV2" not in html
 
 
 def test_browser_summary_report_executes_when_opened_directly(tmp_path):

From 1d9ad30256e14c9cdb1a90c2f19c8b8212d5000d Mon Sep 17 00:00:00 2001
From: Uriah Finkel <Ufinkel@gmail.com>
Date: Mon, 24 Aug 2026 18:47:28 +0300
Subject: [PATCH 7/9] test: fail on browser console errors

---
 tests/test_summary_report_browser.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/test_summary_report_browser.py b/tests/test_summary_report_browser.py
index ede6be11..27096ce8 100644
--- a/tests/test_summary_report_browser.py
+++ b/tests/test_summary_report_browser.py
@@ -100,6 +100,7 @@ def _serve(directory: Path) -> Iterator[str]:
 
 def _assert_report_rendered(browser: subprocess.CompletedProcess[str]) -> None:
     assert browser.returncode == 0, browser.stderr
+    assert "INFO:CONSOLE" not in browser.stderr, browser.stderr
     rendered = _rendered_report_html(browser.stdout)
     assert "Performance" in rendered, browser.stderr
     assert "ROC" in rendered

From 5267cb9d80700a226d4c0ef8a340e338dc727f62 Mon Sep 17 00:00:00 2001
From: Uriah Finkel <Ufinkel@gmail.com>
Date: Mon, 24 Aug 2026 18:48:09 +0300
Subject: [PATCH 8/9] test: expect embedded shared report renderer

---
 tests/test_report_browser.py | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/tests/test_report_browser.py b/tests/test_report_browser.py
index 328ced1d..5874a3bc 100644
--- a/tests/test_report_browser.py
+++ b/tests/test_report_browser.py
@@ -49,12 +49,11 @@ def test_browser_report_uses_real_producers_and_only_shared_render_report(tmp_pa
         tmp_path / "report.html"
     )
     html = output.read_text(encoding="utf-8")
+    viz_js = (tmp_path / "rtichoke-viz.js").read_text(encoding="utf-8")
 
-    assert 'import { renderReport } from "./rtichoke-viz.js";' in html
+    assert 'import { renderReport } from "./rtichoke-viz.js";' not in html
+    assert viz_js in html
     assert "append(renderReport(spec))" in html
-    assert "renderRocV2" not in html
-    assert "renderPerformanceTable" not in html
-    assert "renderGainsV2" not in html
     assert _embedded_report(html) == report
     assert [component["id"] for component in report["components"]] == [
         "performance-table",

From f89c7b06fb1463e411c63a2d667d65fdb90b3f09 Mon Sep 17 00:00:00 2001
From: Uriah Finkel <Ufinkel@gmail.com>
Date: Mon, 24 Aug 2026 18:48:50 +0300
Subject: [PATCH 9/9] test: expect embedded renderer in calibration report

---
 tests/test_calibration_v2.py | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/tests/test_calibration_v2.py b/tests/test_calibration_v2.py
index a77e1a7e..3efa3988 100644
--- a/tests/test_calibration_v2.py
+++ b/tests/test_calibration_v2.py
@@ -273,6 +273,7 @@ def test_real_performance_table_roc_calibration_report_uses_shared_renderer(tmp_
         tmp_path / "report.html"
     )
     html = output.read_text(encoding="utf-8")
+    viz_js = (tmp_path / "rtichoke-viz.js").read_text(encoding="utf-8")
 
     assert [component["id"] for component in report["components"]] == [
         "performance-table",
@@ -281,11 +282,9 @@ def test_real_performance_table_roc_calibration_report_uses_shared_renderer(tmp_
     ]
     assert report["components"][2]["spec"] is calibration
     assert _embedded_report(html) == report
-    assert 'import { renderReport } from "./rtichoke-viz.js";' in html
+    assert 'import { renderReport } from "./rtichoke-viz.js";' not in html
+    assert viz_js in html
     assert "append(renderReport(spec))" in html
-    assert "renderCalibrationV2" not in html
-    assert "renderRocV2" not in html
-    assert "renderPerformanceTable" not in html
 
 
 def test_duplicate_evaluation_ids_remain_component_local_in_real_report():