wup 0.2.67__tar.gz → 0.2.69__tar.gz

{wup-0.2.67/wup.egg-info → wup-0.2.69}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: wup
-Version: 0.2.67
+Version: 0.2.69
 Summary: WUP (What's Up) - Intelligent file watcher for regression testing in large projects
 Author-email: Tom Sapletta <tom@sapletta.com>
 License-Expression: Apache-2.0
@@ -42,15 +42,17 @@ Dynamic: license-file
 
 ## AI Cost Tracking
 
-![AI Cost](https://img.shields.io/badge/AI%20Cost-$7.50-yellow) ![AI Model](https://img.shields.io/badge/AI%20Model-openrouter%2Fqwen%2Fqwen3-coder-next-lightgrey)
+![PyPI](https://img.shields.io/badge/pypi-costs-blue) ![Version](https://img.shields.io/badge/version-0.2.69-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
+![AI Cost](https://img.shields.io/badge/AI%20Cost-$5.86-orange) ![Human Time](https://img.shields.io/badge/Human%20Time-51.1h-blue) ![Model](https://img.shields.io/badge/Model-openrouter%2Fqwen%2Fqwen3--coder--next-lightgrey)
 
-This project uses AI-generated code. Total cost: **$7.5000** with **80** AI commits.
+- 🤖 **LLM usage:** $5.8610 (97 commits)
+- 👤 **Human dev:** ~$5115 (51.1h @ $100/h, 30min dedup)
 
-Generated on 2026-06-29 using [openrouter/qwen/qwen3-coder-next](https://openrouter.ai/models/openrouter/qwen/qwen3-coder-next)
+Generated on 2026-07-16 using [openrouter/qwen/qwen3-coder-next](https://openrouter.ai/qwen/qwen3-coder-next)
 
 ---
 
-![PyPI](https://img.shields.io/badge/pypi-wup-blue) ![Version](https://img.shields.io/badge/version-0.2.67-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
+![PyPI](https://img.shields.io/badge/pypi-wup-blue) ![Version](https://img.shields.io/badge/version-0.2.69-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
 
 **WUP (What's Up)** - Intelligent file watcher for regression testing in large projects.
 

{wup-0.2.67 → wup-0.2.69}/README.md

@@ -3,15 +3,17 @@
 
 ## AI Cost Tracking
 
-![AI Cost](https://img.shields.io/badge/AI%20Cost-$7.50-yellow) ![AI Model](https://img.shields.io/badge/AI%20Model-openrouter%2Fqwen%2Fqwen3-coder-next-lightgrey)
+![PyPI](https://img.shields.io/badge/pypi-costs-blue) ![Version](https://img.shields.io/badge/version-0.2.69-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
+![AI Cost](https://img.shields.io/badge/AI%20Cost-$5.86-orange) ![Human Time](https://img.shields.io/badge/Human%20Time-51.1h-blue) ![Model](https://img.shields.io/badge/Model-openrouter%2Fqwen%2Fqwen3--coder--next-lightgrey)
 
-This project uses AI-generated code. Total cost: **$7.5000** with **80** AI commits.
+- 🤖 **LLM usage:** $5.8610 (97 commits)
+- 👤 **Human dev:** ~$5115 (51.1h @ $100/h, 30min dedup)
 
-Generated on 2026-06-29 using [openrouter/qwen/qwen3-coder-next](https://openrouter.ai/models/openrouter/qwen/qwen3-coder-next)
+Generated on 2026-07-16 using [openrouter/qwen/qwen3-coder-next](https://openrouter.ai/qwen/qwen3-coder-next)
 
 ---
 
-![PyPI](https://img.shields.io/badge/pypi-wup-blue) ![Version](https://img.shields.io/badge/version-0.2.67-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
+![PyPI](https://img.shields.io/badge/pypi-wup-blue) ![Version](https://img.shields.io/badge/version-0.2.69-blue) ![Python](https://img.shields.io/badge/python-3.9+-blue) ![License](https://img.shields.io/badge/license-Apache--2.0-green)
 
 **WUP (What's Up)** - Intelligent file watcher for regression testing in large projects.
 

{wup-0.2.67 → wup-0.2.69}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "wup"
-version = "0.2.67"
+version = "0.2.69"
 description = "WUP (What's Up) - Intelligent file watcher for regression testing in large projects"
 readme = "README.md"
 requires-python = ">=3.10"

wup-0.2.69/tests/test_planfile_reporter_dedupe.py

@@ -0,0 +1,78 @@
+"""Regression: dedupe registry must not mute recurrences of CLOSED tickets.
+
+A stale `.wup/planfile-tickets.json` entry whose ticket was already
+done/canceled used to suppress every future ticket for the same
+failure fingerprint — a regression of an old outage became silently
+invisible (found live: `docker stop connect-scenario` produced no ticket
+because PLF-1950 from a month earlier still held the fingerprint).
+"""
+
+from __future__ import annotations
+
+import json
+from pathlib import Path
+from types import SimpleNamespace
+
+from wup import planfile_reporter as reporter_mod
+from wup.models.config import PlanfileConfig
+from wup.planfile_reporter import PlanfileReporter
+
+
+def _reporter(tmp_path: Path) -> PlanfileReporter:
+    cfg = PlanfileConfig(enabled=True, command="planfile")
+    return PlanfileReporter(tmp_path, cfg)
+
+
+def _seed_dedupe(rep: PlanfileReporter, fingerprint_kwargs: dict, ticket_id: str) -> None:
+    fingerprint = rep._fingerprint(**fingerprint_kwargs)
+    rep.dedupe_path.parent.mkdir(parents=True, exist_ok=True)
+    rep.dedupe_path.write_text(
+        json.dumps({fingerprint: {"ticket_id": ticket_id, "service": fingerprint_kwargs["service"], "stage": fingerprint_kwargs["stage"]}}),
+        encoding="utf-8",
+    )
+
+
+FAIL = {"service": "svc", "status": "down", "stage": "probe", "message": "connection refused"}
+
+
+def test_open_ticket_still_mutes_recurrence(tmp_path, monkeypatch):
+    rep = _reporter(tmp_path)
+    _seed_dedupe(rep, FAIL, "PLF-1")
+
+    def fake_run(cmd, **kwargs):
+        assert "show" in cmd
+        return SimpleNamespace(returncode=0, stdout=json.dumps({"id": "PLF-1", "status": "open"}), stderr="")
+
+    monkeypatch.setattr(reporter_mod.subprocess, "run", fake_run)
+    assert rep.report_failure(**FAIL) == "PLF-1"
+
+
+def test_closed_ticket_refiles_fresh_ticket(tmp_path, monkeypatch):
+    rep = _reporter(tmp_path)
+    _seed_dedupe(rep, FAIL, "PLF-1")
+    calls = []
+
+    def fake_run(cmd, **kwargs):
+        calls.append(list(cmd))
+        if "show" in cmd:
+            return SimpleNamespace(returncode=0, stdout=json.dumps({"id": "PLF-1", "status": "done"}), stderr="")
+        return SimpleNamespace(returncode=0, stdout="✓ Created PLF-2: [AUTO-DIAG] wup-svc probe down", stderr="")
+
+    monkeypatch.setattr(reporter_mod.subprocess, "run", fake_run)
+    monkeypatch.setattr(rep, "_wait_for_planfile_store_ready", lambda timeout_s=30.0: True)
+
+    assert rep.report_failure(**FAIL) == "PLF-2"
+    # registry now points at the fresh ticket
+    dedupe = json.loads(rep.dedupe_path.read_text(encoding="utf-8"))
+    assert [v["ticket_id"] for v in dedupe.values()] == ["PLF-2"]
+
+
+def test_show_error_keeps_muting_conservatively(tmp_path, monkeypatch):
+    rep = _reporter(tmp_path)
+    _seed_dedupe(rep, FAIL, "PLF-1")
+
+    def fake_run(cmd, **kwargs):
+        return SimpleNamespace(returncode=1, stdout="", stderr="boom")
+
+    monkeypatch.setattr(reporter_mod.subprocess, "run", fake_run)
+    assert rep.report_failure(**FAIL) == "PLF-1"

{wup-0.2.67 → wup-0.2.69}/tests/test_testql_watcher.py

@@ -246,7 +246,10 @@ def test_planfile_reporter_creates_deduped_ticket(monkeypatch):
 
         assert first == "PLF-999"
         assert second == "PLF-999"
-        assert len(calls) == 1
+        # Recurrence check may `ticket show` the deduped id, but only ONE
+        # ticket may ever be created for the same open fingerprint.
+        create_calls = [c for c in calls if c[:3] == ["planfile", "ticket", "create"]]
+        assert len(create_calls) == 1
         assert calls[0][:3] == ["planfile", "ticket", "create"]
         assert "--label" in calls[0]
         assert "llm-ready" in calls[0]
@@ -283,6 +286,34 @@ def test_planfile_reporter_clears_dedupe_after_recovery(monkeypatch):
 
         assert first == "PLF-1000"
         assert second == "PLF-1001"
+
+
+def test_planfile_reporter_retries_without_files_for_old_planfile_cli(monkeypatch):
+    with tempfile.TemporaryDirectory() as tmpdir:
+        root = Path(tmpdir)
+        calls = []
+
+        def fake_run(cmd, **kwargs):
+            calls.append(cmd)
+            if "--files" in cmd:
+                return CompletedProcess(cmd, returncode=2, stdout="", stderr="Error: No such option: --files")
+            return CompletedProcess(cmd, returncode=0, stdout="Created PLF-998\n", stderr="")
+
+        monkeypatch.setattr("wup.planfile_reporter.subprocess.run", fake_run)
+        reporter = PlanfileReporter(root, PlanfileConfig(enabled=True))
+
+        ticket_id = reporter.report_failure(
+            service="firmware",
+            status="failed",
+            stage="detail",
+            message="detail failed",
+            track_file=".wup/tracks/firmware_detail.json",
+        )
+
+        assert ticket_id == "PLF-998"
+        assert len(calls) == 2
+        assert "--files" in calls[0]
+        assert "--files" not in calls[1]
         assert len(calls) == 2
 
 
@@ -414,7 +445,13 @@ def test_visual_differ_disabled_by_default():
         # Differ is created but flagged disabled — run_for_service() must be a no-op
         assert watcher.visual_differ is not None
         assert watcher.visual_differ.cfg.enabled is False
-        results = asyncio.run(watcher.visual_differ.run_for_service("svc", ["/x"]))
+        from wup.models.target import ServiceTestTarget
+
+        results = asyncio.run(
+            watcher.visual_differ.run_for_service(
+                ServiceTestTarget(service="svc", endpoints=["/x"])
+            )
+        )
         assert results == []
 
 
@@ -504,8 +541,8 @@ def test_quick_pass_actions_prefer_config_endpoints_for_visual_diff():
                 self.cfg = VisualDiffConfig(enabled=True)
                 self.calls = []
 
-            async def run_for_service(self, service, endpoints):
-                self.calls.append((service, list(endpoints)))
+            async def run_for_service(self, target):
+                self.calls.append((target.service, list(target.endpoints)))
                 return []
 
         differ = RecordingDiffer()

wup-0.2.69/tests/test_watch_no_paths_exit.py

@@ -0,0 +1,30 @@
+"""Regression: a watcher that never watched must not exit as a clean run.
+
+`start_watching` used to bare-`return` when every configured watch path was
+missing — under systemd (Restart=on-failure) the clean exit 0 meant the unit
+silently died and never came back. It now returns False and the CLI turns
+that into exit code 1.
+"""
+
+from __future__ import annotations
+
+from wup.core import WupWatcher
+from wup.models.config import ProjectConfig, WatchConfig, WupConfig
+
+
+def _watcher_with_paths(paths: list[str]) -> WupWatcher:
+    cfg = WupConfig(
+        project=ProjectConfig(name="t"),
+        watch=WatchConfig(paths=paths),
+    )
+    return WupWatcher(project_root="/nonexistent-wup-test-root", config=cfg)
+
+
+def test_start_watching_returns_false_when_no_valid_paths() -> None:
+    watcher = _watcher_with_paths(["does-not-exist/**"])
+    assert watcher.start_watching() is False
+
+
+def test_start_watching_returns_false_for_explicit_missing_paths() -> None:
+    watcher = _watcher_with_paths([])
+    assert watcher.start_watching(["/nonexistent-wup-test-root/a"]) is False

{wup-0.2.67 → wup-0.2.69}/tests/test_wup.py

@@ -11,6 +11,7 @@ import pytest
 from wup.config import load_config, save_config, get_default_config
 from wup.core import WupWatcher
 from wup.dependency_mapper import DependencyMapper
+from wup.models.target import ServiceTestTarget
 from wup.models.config import (
     WupConfig,
     WatchConfig,
@@ -26,6 +27,7 @@ from wup.models.config import (
 from wup.testql_watcher import TestQLWatcher
 from wup.visual_diff import (
     VisualDiffer,
+    _chromium_launch_options,
     _diff_snapshots,
     _looks_like_visual_page,
     _page_slug,
@@ -994,6 +996,9 @@ class TestConfigModels:
         assert cfg.pages == []
         assert cfg.pages_from_endpoints is True
         assert cfg.headless is True
+        assert cfg.page_settle_ms == 750
+        assert cfg.issue_retry_count == 0
+        assert cfg.issue_retry_delay_seconds == 2.0
 
     def test_visual_diff_config_custom(self):
         """Test VisualDiffConfig with custom values."""
@@ -1002,11 +1007,17 @@ class TestConfigModels:
             base_url="http://localhost:9000",
             pages=["/dashboard", "/users"],
             threshold_added=10,
+            page_settle_ms=1500,
+            issue_retry_count=2,
+            issue_retry_delay_seconds=3.0,
         )
         assert cfg.enabled is True
         assert cfg.base_url == "http://localhost:9000"
         assert cfg.pages == ["/dashboard", "/users"]
         assert cfg.threshold_added == 10
+        assert cfg.page_settle_ms == 1500
+        assert cfg.issue_retry_count == 2
+        assert cfg.issue_retry_delay_seconds == 3.0
 
 
 class TestVisualDiffer:
@@ -1026,6 +1037,14 @@ class TestVisualDiffer:
         monkeypatch.delenv("WUP_VD_NONE", raising=False)
         assert _resolve_base_url(cfg) == ""
 
+    def test_chromium_launch_options_uses_env_executable(self, tmp_path, monkeypatch):
+        chrome = tmp_path / "chrome"
+        chrome.write_text("#!/bin/sh\n", encoding="utf-8")
+        monkeypatch.setenv("WUP_CHROMIUM_EXECUTABLE_PATH", str(chrome))
+        options = _chromium_launch_options(True)
+        assert options["headless"] is True
+        assert options["executable_path"] == str(chrome)
+
     def test_page_slug(self):
         assert _page_slug("http://x/api/v1/users") == "api_v1_users"
         assert _page_slug("http://x/") == "root"
@@ -1038,7 +1057,7 @@ class TestVisualDiffer:
                 pages_from_endpoints=False,
             )
             differ = VisualDiffer(tmpdir, cfg)
-            pages = differ._pages_for_service("users", ["/api/users"])
+            pages = differ._pages_for_service(ServiceTestTarget(service="users", endpoints=["/api/users"]))
             assert pages == ["http://localhost:8080/dashboard"]
 
     def test_pages_for_service_from_endpoints(self):
@@ -1048,7 +1067,9 @@ class TestVisualDiffer:
                 pages_from_endpoints=True,
             )
             differ = VisualDiffer(tmpdir, cfg)
-            pages = differ._pages_for_service("users", ["/connect-users", "/connect-users/1"])
+            pages = differ._pages_for_service(
+                ServiceTestTarget(service="users", endpoints=["/connect-users", "/connect-users/1"])
+            )
             assert "http://localhost:8080/connect-users" in pages
             assert "http://localhost:8080/connect-users/1" in pages
 
@@ -1065,8 +1086,10 @@ class TestVisualDiffer:
             )
             differ = VisualDiffer(tmpdir, cfg)
             pages = differ._pages_for_service(
-                "backend",
-                ["/api/v3/health", "/connect-config", "http://localhost:8080/connect-data"],
+                ServiceTestTarget(
+                    service="backend",
+                    endpoints=["/api/v3/health", "/connect-config", "http://localhost:8080/connect-data"],
+                )
             )
             assert "http://localhost:8080/api/v3/health" not in pages
             assert "http://localhost:8080/connect-config" in pages
@@ -1079,7 +1102,7 @@ class TestVisualDiffer:
                 pages_from_endpoints=False,
             )
             differ = VisualDiffer(tmpdir, cfg)
-            pages = differ._pages_for_service("users", [])
+            pages = differ._pages_for_service(ServiceTestTarget(service="users"))
             assert pages == ["http://localhost:8080/users"]
 
     def test_pages_for_service_absolute_url_passthrough(self):
@@ -1090,7 +1113,7 @@ class TestVisualDiffer:
                 pages_from_endpoints=False,
             )
             differ = VisualDiffer(tmpdir, cfg)
-            pages = differ._pages_for_service("svc", [])
+            pages = differ._pages_for_service(ServiceTestTarget(service="svc"))
             assert pages == ["https://example.com/connect-dashboard"]
 
     def test_diff_snapshots_baseline(self):
@@ -1126,7 +1149,9 @@ class TestVisualDiffer:
         with tempfile.TemporaryDirectory() as tmpdir:
             cfg = VisualDiffConfig(enabled=False)
             differ = VisualDiffer(tmpdir, cfg)
-            results = asyncio.run(differ.run_for_service("svc", ["/x"]))
+            results = asyncio.run(
+                differ.run_for_service(ServiceTestTarget(service="svc", endpoints=["/x"]))
+            )
             assert results == []
 
     def test_run_for_service_summarizes_fetch_errors(self, monkeypatch):
@@ -1159,7 +1184,9 @@ class TestVisualDiffer:
             monkeypatch.setattr(visual_diff_module.console, "print", lambda message: printed.append(str(message)))
 
             results = asyncio.run(
-                differ.run_for_service("frontend", ["/a", "/b", "/c", "/d"])
+                differ.run_for_service(
+                    ServiceTestTarget(service="frontend", endpoints=["/a", "/b", "/c", "/d"])
+                )
             )
 
             assert len(results) == 4
@@ -1168,6 +1195,57 @@ class TestVisualDiffer:
             assert "/a, /b, /c (+1 more)" in printed[0]
             assert "4x BrowserType.launch: Executable doesn't exist at /tmp/chrome" in printed[0]
 
+    def test_check_page_retries_transient_visual_issue(self, monkeypatch):
+        """A transient Vite overlay/tiny shell should be retried before reporting an issue."""
+        import asyncio
+        from wup import visual_diff as visual_diff_module
+
+        bad_snapshot = {
+            "tag": "HTML",
+            "meta": {
+                "text_length": 33,
+                "dom_nodes": 8,
+                "matched_error_selectors": ["vite-error-overlay"],
+            },
+        }
+        good_snapshot = {
+            "tag": "HTML",
+            "children": [{"tag": "BODY", "children": [{"tag": "MAIN"}]}],
+            "meta": {
+                "text_length": 500,
+                "dom_nodes": 50,
+                "matched_error_selectors": [],
+            },
+        }
+        snapshots = [bad_snapshot, good_snapshot]
+        calls = []
+
+        async def fake_fetch(*args):
+            calls.append(args)
+            return snapshots.pop(0), None
+
+        async def fake_sleep(_seconds):
+            return None
+
+        with tempfile.TemporaryDirectory() as tmpdir:
+            cfg = VisualDiffConfig(
+                enabled=True,
+                issue_retry_count=1,
+                issue_retry_delay_seconds=0,
+                page_settle_ms=1234,
+            )
+            differ = VisualDiffer(tmpdir, cfg)
+
+            monkeypatch.setattr(visual_diff_module, "_fetch_dom_snapshot", fake_fetch)
+            monkeypatch.setattr(visual_diff_module.asyncio, "sleep", fake_sleep)
+
+            result = asyncio.run(differ._check_page("frontend", "http://localhost/page"))
+
+        assert len(calls) == 2
+        assert calls[0][-1] == 1234
+        assert result["diff"]["status"] in {"new", "ok", "changed"}
+        assert "issues" not in result["diff"]
+
     def test_get_recent_diffs_empty(self):
         with tempfile.TemporaryDirectory() as tmpdir:
             cfg = VisualDiffConfig()
@@ -1376,6 +1454,9 @@ testql:
                     threshold_added=7,
                     threshold_removed=4,
                     threshold_changed=8,
+                    page_settle_ms=1600,
+                    issue_retry_count=3,
+                    issue_retry_delay_seconds=4.5,
                     headless=False,
                 ),
             )
@@ -1392,6 +1473,9 @@ testql:
             assert vd.threshold_added == 7
             assert vd.threshold_removed == 4
             assert vd.threshold_changed == 8
+            assert vd.page_settle_ms == 1600
+            assert vd.issue_retry_count == 3
+            assert vd.issue_retry_delay_seconds == 4.5
             assert vd.headless is False
 
     def test_load_config_visual_diff_from_yaml(self):
@@ -1415,6 +1499,9 @@ visual_diff:
   threshold_added: 5
   threshold_removed: 2
   threshold_changed: 9
+  page_settle_ms: 1500
+  issue_retry_count: 2
+  issue_retry_delay_seconds: 3
   headless: false
 """
             config_path = Path(tmpdir) / "wup.yaml"
@@ -1433,6 +1520,9 @@ visual_diff:
             assert vd.threshold_added == 5
             assert vd.threshold_removed == 2
             assert vd.threshold_changed == 9
+            assert vd.page_settle_ms == 1500
+            assert vd.issue_retry_count == 2
+            assert vd.issue_retry_delay_seconds == 3.0
             assert vd.headless is False
 
     def test_load_config_visual_diff_defaults_when_section_absent(self):

{wup-0.2.67 → wup-0.2.69}/wup/__init__.py

@@ -7,7 +7,7 @@ WUP monitors file changes and runs intelligent regression tests using a 3-layer
 3. Detail Layer: Full tests with blame reports (only on failure)
 """
 
-__version__ = "0.2.67"
+__version__ = "0.2.69"
 __author__ = "Tom Sapletta"
 
 from .config import load_config, save_config, get_default_config

{wup-0.2.67 → wup-0.2.69}/wup/cli.py

@@ -198,7 +198,8 @@ def watch(
         asyncio.run(watcher.run_with_dashboard())
     else:
         console.print("[green]Starting watcher...[/green]")
-        watcher.start_watching()
+        if watcher.start_watching() is False:
+            raise typer.Exit(1)
 
 
 def _auto_generate_config(project_path: Path, mode: str):

{wup-0.2.67 → wup-0.2.69}/wup/config.py

@@ -292,6 +292,9 @@ def _parse_visual_diff_config(raw: dict) -> VisualDiffConfig:
         threshold_changed=int(vd_raw.get("threshold_changed", 5)),
         min_text_length=int(vd_raw.get("min_text_length", 200)),
         min_dom_nodes=int(vd_raw.get("min_dom_nodes", 20)),
+        page_settle_ms=int(vd_raw.get("page_settle_ms", 750)),
+        issue_retry_count=int(vd_raw.get("issue_retry_count", 0)),
+        issue_retry_delay_seconds=float(vd_raw.get("issue_retry_delay_seconds", 2.0)),
         error_selectors=vd_raw.get("error_selectors", [
             "#error-container",
             ".error-container",
@@ -494,6 +497,9 @@ def save_config(config: WupConfig, output_path: Path):
             "threshold_changed": config.visual_diff.threshold_changed,
             "min_text_length": config.visual_diff.min_text_length,
             "min_dom_nodes": config.visual_diff.min_dom_nodes,
+            "page_settle_ms": config.visual_diff.page_settle_ms,
+            "issue_retry_count": config.visual_diff.issue_retry_count,
+            "issue_retry_delay_seconds": config.visual_diff.issue_retry_delay_seconds,
             "error_selectors": config.visual_diff.error_selectors,
             "headless": config.visual_diff.headless,
         },

{wup-0.2.67 → wup-0.2.69}/wup/core.py

@@ -25,6 +25,7 @@ from watchdog.observers.polling import PollingObserver
 from .config import load_config
 from .dependency_mapper import DependencyMapper
 from .models.config import WupConfig, ServiceConfig
+from .models.target import ServiceTestTarget
 from .planfile_reporter import PlanfileReporter
 
 
@@ -278,14 +279,15 @@ class WupWatcher:
             return
 
         test_type, service, endpoints = self.test_queue.popleft()
+        target = ServiceTestTarget(service, endpoints)
 
         try:
             if test_type == "quick":
-                passed = await self.run_quick_test(service, endpoints)
+                passed = await self.run_quick_test(target)
                 if not passed:
                     self.schedule_detail_test(service)
             elif test_type == "detail":
-                await self.run_detail_test(service, endpoints)
+                await self.run_detail_test(target)
         except Exception as e:
             self.console.print(f"[red]Error testing {service}: {e}[/red]")
     
@@ -302,17 +304,17 @@ class WupWatcher:
         except Exception:
             return True
     
-    async def run_quick_test(self, service: str, endpoints: List[str]) -> bool:
+    async def run_quick_test(self, target: ServiceTestTarget) -> bool:
         """
         Run a quick test for a service (smoke test).
 
         Args:
-            service: Service name
-            endpoints: List of endpoints to test
+            target: Service and endpoints to test
 
         Returns:
             True if all tests passed, False otherwise
         """
+        service, endpoints = target.service, target.endpoints
         self.console.print(f"[cyan]🧪 Quick testing {service} ({len(endpoints)} endpoints)[/cyan]")
 
         if not endpoints:
@@ -346,17 +348,17 @@ class WupWatcher:
 
         return passed
     
-    async def run_detail_test(self, service: str, endpoints: List[str]) -> Dict:
+    async def run_detail_test(self, target: ServiceTestTarget) -> Dict:
         """
         Run a detailed test for a service with blame report.
 
         Args:
-            service: Service name
-            endpoints: List of endpoints to test
+            target: Service and endpoints to test
 
         Returns:
             Dictionary with test results and blame information
         """
+        service, endpoints = target.service, target.endpoints
         self.console.print(f"[cyan]🔍 Detail testing {service} ({len(endpoints)} endpoints)[/cyan]")
 
         results = {
@@ -564,35 +566,42 @@ class WupWatcher:
                 return observer
             raise
 
-    def start_watching(self, watch_paths: Optional[List[str]] = None):
+    def start_watching(self, watch_paths: Optional[List[str]] = None) -> bool:
         """
         Start watching for file changes.
-        
+
         Args:
             watch_paths: List of paths to watch (default: from config or common source directories)
+
+        Returns:
+            False when no valid paths could be watched (configuration error —
+            CLI turns this into a non-zero exit so supervisors notice instead
+            of treating a watcher that never watched as a clean run),
+            True after a graceful interrupt.
         """
         if watch_paths is None:
             watch_paths = self.build_watched_paths()
-        
+
         # Filter to existing paths
         watch_paths = [p for p in watch_paths if Path(p).exists()]
-        
+
         if not watch_paths:
             self.console.print("[red]No valid paths to watch[/red]")
-            return
-        
+            return False
+
         event_handler = WupEventHandler(self)
         observer = self._create_and_start_observer(event_handler, watch_paths)
         self.console.print(f"[green]🕵️  Watching: {', '.join(watch_paths)}[/green]")
-        
+
         try:
             while True:
                 asyncio.run(self.process_test_queue_once())
                 time.sleep(1)
         except KeyboardInterrupt:
             observer.stop()
-        
+
         observer.join()
+        return True
     
     def create_status_table(self) -> Table:
         """

{wup-0.2.67 → wup-0.2.69}/wup/models/__init__.py

@@ -15,6 +15,7 @@ from .config import (
     WebConfig,
     WupConfig,
 )
+from .target import ServiceTestTarget
 
 ServiceType = str  # Type alias: 'web', 'shell', 'auto'
 
@@ -25,6 +26,7 @@ __all__ = [
     "ServiceConfig",
     "ServiceTestConfig",
     "ServiceType",
+    "ServiceTestTarget",
     "TestQLConfig",
     "TestStrategyConfig",
     "VisualDiffConfig",

{wup-0.2.67 → wup-0.2.69}/wup/models/config.py

@@ -97,6 +97,9 @@ class VisualDiffConfig:
     threshold_changed: int = 5       # min changed attrs to report
     min_text_length: int = 200       # anomaly if rendered text is too short
     min_dom_nodes: int = 20          # anomaly if DOM is suspiciously tiny
+    page_settle_ms: int = 750        # wait after networkidle before snapshotting SPA DOM
+    issue_retry_count: int = 0       # retry transient Vite/HMR/page-shell issues
+    issue_retry_delay_seconds: float = 2.0
     error_selectors: List[str] = field(default_factory=lambda: [
         "#error-container",
         ".error-container",