cctally 1.56.0 → 1.56.2

package/CHANGELOG.md

@@ -5,6 +5,20 @@ based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [1.56.2] - 2026-06-27
+
+### Fixed
+- Conversation viewer: the browse-rail model chip is no longer truncated mid-glyph by the adjacent cost/message-count — it previously clipped "opus" to "opu"/"op" when the fixed-width rail ran short of room; the rail now shows a single rigid primary-model chip plus a "+N" counter for any additional models, and the project/branch text ellipsizes first so the chip and the cost/message numbers always render in full (#243).
+- Conversation viewer: a conversation whose main session ran one model but spawned subagents on another (e.g. opus driving haiku code-explorers) now shows the main-session model as the primary one — both the rail chip and the reader header now list the main model first ("opus, haiku") instead of alphabetically ("haiku, opus"), which previously made a haiku subagent look like the conversation's main model (#243).
+
+## [1.56.1] - 2026-06-27
+
+### Fixed
+- Conversation viewer: large subagent threads now render an internally windowed slice (centered on a deep-linked / found / pinned turn) instead of mounting the entire thread, with "Show N earlier/later" and "Show all" controls to grow it on demand — a deep-link into a giant subagent no longer renders a ~106k-node DOM (#239).
+- Conversation comparison view: the A→B metrics strip no longer clips the widest "A → B" value pairs (e.g. a high cost or long duration) at desktop widths — on a smaller desktop window the discovery rail stays mounted and narrows the strip, which the previous viewport-based reflow couldn't detect, so the six fixed columns truncated a trailing digit (`$337.05 → $169…`); the strip now reflows on its own width to a clean 3×2 (and 2-up when narrower) so every value stays fully visible (#240).
+- Conversation viewer: a deep-linked turn now re-lands after a browser reload — reloading a deep-linked conversation URL (one shared or bookmarked deep inside a large session) used to leave you stranded at the previous scroll position with the target turn's owning subagent collapsed and never scrolled to, because the browser restored the prior scroll offset on top of the deep-link's own positioning and that restore landed after the reader had finished settling; the viewer now opts out of the browser's native scroll restoration so the deep-link is always the sole thing positioning the viewport and reliably re-lands on its target (#241).
+- Conversation comparison view: the A→B metrics strip now stays fully un-clipped even at the very narrowest widths — in the 650–720px desktop band the mounted discovery rail can squeeze the strip below its two-column floor, and the same squeeze happens on sub-343px phones, where the widest "A → B" pairs (a high cost, a long duration) could still ellipsize a trailing character or two; the strip now drops to a single full-width column in that band so every value is shown in full, finishing the container-width reflow started in #240 (#242).
+
 ## [1.56.0] - 2026-06-26
 
 ### Fixed

package/bin/_lib_conversation_query.py

@@ -799,19 +799,51 @@ def _session_cost_map(conn, session_ids):
     return costs
 
 
+def _models_main_first(rows):
+    """Order distinct non-null models so MAIN-SESSION models sort first, then
+    models that appear ONLY in subagent / sidechain threads; alphabetical within
+    each group. ``rows`` is an iterable of ``(model, source_path, is_sidechain)``.
+
+    A model is "main" iff it appears in at least one row that is neither a
+    subagent file (``_subagent_key(source_path) is None``) nor a sidechain
+    (``not is_sidechain``) — the exact negation of the subagent/sidechain
+    predicate used elsewhere in this module. A model used in BOTH the main
+    session AND a subagent counts as MAIN (it's a model the user actually drove
+    the session with). This is what makes the reader header + browse-rail chip
+    surface the real primary model (e.g. opus) first when the main session
+    spawned haiku subagents, rather than the alphabetical-first model — pre-fix
+    a plain ``sorted()`` ranked ``haiku`` ahead of ``opus`` (#243)."""
+    main, sub = set(), set()
+    for model, source_path, is_sidechain in rows:
+        if not model:
+            continue
+        if _subagent_key(source_path) is None and not is_sidechain:
+            main.add(model)
+        else:
+            sub.add(model)
+    return sorted(main) + sorted(sub - main)
+
+
 def _session_models_map(conn, session_ids):
-    """{session_id: sorted distinct non-null models}."""
+    """{session_id: models ordered MAIN-session-first (subagent/sidechain-only
+    models last), alphabetical within each group — see _models_main_first}."""
     out = {sid: [] for sid in session_ids}
     if not session_ids:
         return out
     placeholders = ",".join("?" for _ in session_ids)
     sql = (
-        "SELECT DISTINCT session_id, model FROM conversation_messages "
-        "WHERE session_id IN (%s) AND model IS NOT NULL AND model != '' "
-        "ORDER BY model" % placeholders
+        "SELECT DISTINCT session_id, model, source_path, is_sidechain "
+        "FROM conversation_messages "
+        "WHERE session_id IN (%s) AND model IS NOT NULL AND model != ''"
+        % placeholders
     )
-    for sid, model in conn.execute(sql, list(session_ids)):
-        out.setdefault(sid, []).append(model)
+    per_session = {}
+    for sid, model, source_path, is_sidechain in conn.execute(
+            sql, list(session_ids)):
+        per_session.setdefault(sid, []).append(
+            (model, source_path, is_sidechain))
+    for sid, rows in per_session.items():
+        out[sid] = _models_main_first(rows)
     return out
 
 
@@ -1764,7 +1796,9 @@ def get_conversation(conn, session_id, *, after=None, before=None, tail=False,
             "started_utc": logical[0][2],
             "last_activity_utc": logical[-1][2],
             "cost_usd": header_cost,
-            "models": sorted({r[6] for r in logical if r[6]}),
+            # #243: main-session models first (r[6]=model, r[12]=source_path,
+            # r[9]=is_sidechain) so opus outranks a haiku subagent, not sorted().
+            "models": _models_main_first((r[6], r[12], r[9]) for r in logical),
             "last_anchor": last_anchor,
             "items": [],
             "subagent_meta": subagent_meta,
@@ -1814,7 +1848,9 @@ def get_conversation(conn, session_id, *, after=None, before=None, tail=False,
 
     first = logical[0]
     last = logical[-1]
-    models = sorted({r[6] for r in logical if r[6]})
+    # #243: main-session models first (r[6]=model, r[12]=source_path,
+    # r[9]=is_sidechain) so opus outranks a haiku subagent, not sorted().
+    models = _models_main_first((r[6], r[12], r[9]) for r in logical)
     return {
         "session_id": session_id,
         "title": _title,