From cd16ca756f8ee62fcb19ee933a33b846b4c6ef84 Mon Sep 17 00:00:00 2001
From: starship-s <45587122+starship-s@users.noreply.github.com>
Date: Tue, 12 May 2026 18:39:19 -0600
Subject: [PATCH] docs(sessions): document CLI metadata lookup markers

---
 api/routes.py | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/api/routes.py b/api/routes.py
index 1bc4b2fe..6b1a8421 100644
--- a/api/routes.py
+++ b/api/routes.py
@@ -1560,6 +1560,14 @@ def _is_messaging_session_record(session) -> bool:
 
 
 def _session_requires_cli_metadata_lookup(session) -> bool:
+    """Return True when a sidecar/session row still needs CLI metadata.
+
+    Legacy imported sidecars may predate the ``read_only`` field and therefore
+    load with ``read_only=False``. They still persist ``is_cli_session`` and/or
+    source metadata from import time, so those markers intentionally keep them
+    on the CLI lookup path while ordinary WebUI-native sessions take the fast
+    path.
+    """
     if not session:
         return False