fix(web): Course Vendor dropdown populates from study root directly

_courses_roots() required a Study/Courses/ subdir that the real vault
doesn't have (vendors live directly under Study/), leaving the Course
Vendor picker empty and cascading to an empty Course picker. Now scans
each study root directly when it has no Courses/ child, honouring the
nested layout when present. Verified live in browser: 8 vendors + 29
courses populate with correct parent linkage.

Author: NetDevAutomate

packages/studyloop/src/studyloop/web/routes/session.py

@@ -1256,9 +1256,10 @@ async def ws_to_pty() -> None:
                 if not request_id:
                     # Missing requestId — silently drop, can't correlate.
                     pass
-                elif hasattr(transport, "send_permission_response"):
-                    if isinstance(outcome, dict):
-                        await transport.send_permission_response(request_id, outcome)
+                elif hasattr(transport, "send_permission_response") and isinstance(
+                    outcome, dict
+                ):
+                    await transport.send_permission_response(request_id, outcome)
             # Silently drop unknown frame types — no error channel needed.
 
     # --- Pump with TaskGroup (plan Blocker B5) ---------------------------
@@ -1418,7 +1419,22 @@ def _lesson_options() -> list[SessionOption]:
 
 
 def _courses_roots() -> list[Path]:
-    candidates = [root / "Courses" for root in _study_roots()]
+    # Course vendors (ArjanCodes, CodeWithMosh, …) live directly under each
+    # study root — the same level "Topic" targets scan. Requiring an
+    # intermediate ``Courses/`` directory left the vendor picker empty
+    # because the real vault has no such level.
+    #
+    # A ``Courses/`` subdirectory is still honoured when one exists, so a
+    # vault that nests courses under it keeps working. To avoid surfacing
+    # ``Courses`` itself as a bogus vendor in that case, a study root is
+    # only used directly when it has no ``Courses/`` child.
+    candidates: list[Path] = []
+    for root in _study_roots():
+        nested = root / "Courses"
+        if nested.is_dir():
+            candidates.append(nested)
+        else:
+            candidates.append(root)
     candidates.extend(
         [
             Path("~/Obsidian/Personal/Study/Courses").expanduser(),

packages/studyloop/tests/test_web_live_session.py

@@ -51,3 +51,39 @@ def __init__(self) -> None:
     assert "agents" in body
     assert all(agent["recommended_transport"] == "ttyd" for agent in body["agents"])
     assert all(agent["acp_ready"] is False for agent in body["agents"])
+
+
+def test_session_options_lists_vendors_directly_under_study_root(
+    tmp_path: Path, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    """Course vendors live directly under the study root (no ``Courses/`` level).
+
+    Regression: ``_courses_roots()`` required an intermediate ``Courses/``
+    directory that does not exist in the real vault — vendor dirs
+    (ArjanCodes, CodeWithMosh, …) sit directly under ``Study/``. The missing
+    level left the Course Vendor picker empty, which cascaded to empty
+    Course and Lesson pickers.
+    """
+    study_root = tmp_path / "Study"
+    lesson = study_root / "ArjanCodes" / "The_Software_Designer_Mindset" / "Module_01"
+    lesson.mkdir(parents=True)
+    (study_root / "CodeWithMosh").mkdir()
+
+    class Content:
+        def __init__(self) -> None:
+            self.study_paths = [study_root]
+
+    class Settings:
+        def __init__(self) -> None:
+            self.content = Content()
+
+    monkeypatch.setattr("studyloop.settings.load_settings", Settings)
+
+    client = TestClient(create_app())
+    body = client.get("/api/session/options").json()
+
+    vendor_labels = {v["label"] for v in body["vendors"]}
+    assert "ArjanCodes" in vendor_labels
+    assert "CodeWithMosh" in vendor_labels
+    assert any(c["label"] == "The Software Designer Mindset" for c in body["courses"])
+    assert any(lesson_["label"] == "Module 01" for lesson_ in body["lessons"])