feat: open Following alerts in changed review (Closes #1317)

tests/e2e/test_mobile_following_release.py

@@ -80,3 +80,34 @@ def test_following_queue_is_phone_usable_at_narrow_viewport(viewport):
         overflow = page.evaluate("document.documentElement.scrollWidth > document.documentElement.clientWidth")
         assert overflow is False
         browser.close()
+
+
+def test_following_alert_route_opens_truthful_empty_state_with_one_refresh():
+    with sync_playwright() as playwright:
+        browser = playwright.chromium.launch(headless=True)
+        page = browser.new_page(viewport={"width": 390, "height": 844})
+        page.set_content((FRONTEND / "index.html").read_text())
+        page.add_style_tag(path=FRONTEND / "dashboard.css")
+        page.add_script_tag(path=FRONTEND / "following.js")
+
+        result = page.evaluate("""async () => {
+          let requests=0;
+          globalThis.fetch=async () => {
+            requests += 1;
+            return {ok:true,json:async () => ({revision:2,items:[]})};
+          };
+          const queue=attachFollowing(() => {});
+          await queue.route();
+          return {requests,open:document.querySelector('#following-sheet').open,
+            status:document.querySelector('#following-status').textContent};
+        }""")
+
+        assert result == {
+            "requests": 1,
+
+            "open": True,
+            "status": "No watched items yet. Watch an issue or pull request from Search to keep it here.",
+        }
+        expect(page.locator("#following-sheet")).to_be_visible()
+        assert page.locator("#close-following").bounding_box()["height"] >= 44
+        browser.close()

tests/test_following_frontend.py

@@ -76,6 +76,26 @@ const feature=createFollowing({{
     assert result["session"]["items"][0]["kind"] == "pull"
 
 
+def test_following_route_loads_authoritative_state_before_starting_review():
+    following = MODULE.read_text()
+    start = following.index("async route()")
+    route = following[start:following.index("open()", start)]
+
+    assert "await feature.load().then(feature.startReview)" in route
+    assert ".catch(() => false)" in route
+    assert "show();" in route
+
+
+def test_dashboard_dispatches_following_route_to_changed_first_review():
+    dashboard = (ROOT / "frontend" / "dashboard.js").read_text()
+
+    start = dashboard.index("async function openWorkQueueRoute")
+    route_handler = dashboard[start:dashboard.index("qs('#work-milestone-filter')", start)]
+    assert "if (filter === 'following')" in route_handler
+    assert "return followingQueue.route();" in route_handler
+    assert route_handler.index("followingQueue.route") < route_handler.index("selectWorkQueue")
+
+
 def test_following_typed_identity_prevents_issue_pull_collisions():
     script = f"""
 const createFollowing = require({json.dumps(str(MODULE))});

tests/test_my_work.py

@@ -676,6 +676,34 @@ process.stdout.write(JSON.stringify(routes.parse('#/my-work/agenda/protect-today
     assert json.loads(result.stdout) == {
         "kind": "queue", "filter": "agenda", "action": "protect-today"
     }
+
+
+def test_following_route_waits_for_hydration_then_opens_once():
+    script = f"""
+const routes = require({json.dumps(str(WORK_ROUTE))});
+const listeners = {{}};
+const location = {{hash:'#/my-work/following'}};
+const calls = [];
+const controller = routes.createController({{
+  location,
+  history: {{pushState() {{}}, replaceState() {{}}, back() {{}}}},
+  eventTarget: {{addEventListener(name, fn) {{ listeners[name] = fn; }}}},
+  onQueue: (queue, action) => calls.push([queue, action]),
+  onInvalid: () => calls.push(['invalid']),
+}});
+controller.start();
+controller.setItems([]);
+controller.sync();
+process.stdout.write(JSON.stringify({{route:routes.parse(location.hash), calls}}));
+"""
+    result = subprocess.run(
+        ["node", "-e", script], check=True, capture_output=True, text=True
+    )
+
+    assert json.loads(result.stdout) == {
+        "route": {"kind": "queue", "filter": "following"},
+        "calls": [["following", None]],
+    }
 CARD_PLANNING = Path(__file__).parents[1] / "frontend" / "card-planning.js"
 TODAY_WORK = Path(__file__).parents[1] / "frontend" / "today-work.js"
 WORK_SELECTION = Path(__file__).parents[1] / "frontend" / "work-selection.js"