test(#133): update detail/notebook stubs for the lazy-session decoupling

Two existing tests encoded pre-#133 behavior — that api_entry_detail carries a buckaroo_session, and that get_session returns a bare {ws_url}. Detail is now metadata-only and get_session calls load_session, so update the BuckarooManager stubs to implement load_session and assert the new decoupled contract (detail session is null; the grid loads via /api/session with a typed status).

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

Author: paddymul

tests/test_buckaroo.py

@@ -733,21 +733,30 @@ def ws_base_url(self) -> str:
     def ensure_session(self, content_hash: str, project: str, column_config_overrides=None) -> str | None:
         return self.session
 
+    def load_session(self, content_hash: str, project: str, column_config_overrides=None) -> dict:
+        return {"status": "ok", "session_id": self.session, "detail": ""}
 
-def test_entry_detail_embeds_react_widget_when_buckaroo_present(project: str, orders_parquet: Path):
-    """API returns buckaroo_session and buckaroo_ws_base when Buckaroo is running."""
+
+def test_entry_detail_decoupled_grid_session_via_session_endpoint(project: str, orders_parquet: Path):
+    """The detail payload is metadata-only; the grid session loads lazily via
+    /api/session so the detail request never blocks on Buckaroo (#133)."""
     from tallyman_companion import create_app
 
     res = build_and_persist(project, _code(project))
     bk: Any = _StubBuckaroo(session="abc123", port=8700)
     app = create_app(project, buckaroo=bk)
     c = TestClient(app)
-    r = c.get(f"/{project}/api/entry/{res.content_hash}")
-    assert r.status_code == 200
-    body = r.json()
-    assert body["buckaroo_session"] == "abc123"
+
+    # Detail no longer carries a session — it's decoupled.
+    body = c.get(f"/{project}/api/entry/{res.content_hash}").json()
+    assert body["buckaroo_session"] is None
     assert body["buckaroo_ws_base"] == "ws://127.0.0.1:8700"
 
+    # The session endpoint provides the widget on demand, with a typed status.
+    sr = c.get(f"/{project}/api/session/{res.content_hash}").json()
+    assert sr["status"] == "ok"
+    assert sr["ws_url"] == "ws://127.0.0.1:8700/ws/abc123"
+
 
 def test_entry_detail_falls_back_when_session_unavailable(project: str, orders_parquet: Path):
     """When ensure_session returns None, API returns null for buckaroo fields."""

tests/test_notebook.py

@@ -385,6 +385,9 @@ def ws_base_url(self):
         def ensure_session(self, content_hash, project):
             return f"sess-{content_hash[:6]}"
 
+        def load_session(self, content_hash, project, column_config_overrides=None):
+            return {"status": "ok", "session_id": f"sess-{content_hash[:6]}", "detail": ""}
+
     app = create_app(project, buckaroo=_StubBuckaroo())  # type: ignore[arg-type]
     c = TestClient(app)
     r = c.get(f"/{project}/api/notebook_full")