Address feedback

Author: clean6378-max-it

api/projects.py

@@ -34,7 +34,16 @@ def _session_row_error(s: SessionListItemDict) -> ProjectSessionRowDict:
 
 
 def _peek_or_cache_summary(path: str, mtime: float, rules_fp: str) -> SummaryCacheRowDict:
-    """Return a cached summary row or peek the file and store a partial row."""
+    """Return a cached summary row (any completeness) or peek the file and store a partial row.
+
+    Used by get_projects for fast landing-page counts. Partial rows (is_complete=False)
+    are acceptable here — is_untitled is derived from the same first-user-text peek that
+    quick_session_info uses; peek and full-parse agree for the vast majority of sessions
+    (first user message within the first 80 lines). The session list path always upgrades
+    to a complete row via get_cached_session, so session_count and list count align after
+    the first session-list visit. With no exclusion rules the counts are identical on first
+    visit too, because is_excluded is always False for both paths.
+    """
     cached = get_summary(path, mtime, rules_fp)
     if cached is not None:
         return cached

tests/test_api_integration.py

@@ -142,9 +142,16 @@ def summary_cache_db(tmp_path, monkeypatch):
 
 
 def test_project_session_count_matches_list(client, summary_cache_db):
+    """Card count and session list agree when no exclusion rules are active.
+
+    get_projects uses peek (partial row); get_project_sessions uses full parse.
+    Both filter on is_untitled, and with no rules is_excluded is always False,
+    so counts align — this is the alignment guarantee from issue #109.
+    """
+    # Hit session list first so disk cache is warm with complete rows.
+    sessions = client.get("/api/projects/test-project/sessions").get_json()
     projects = client.get("/api/projects").get_json()
     project = next(p for p in projects if p["name"] == "test-project")
-    sessions = client.get("/api/projects/test-project/sessions").get_json()
     assert project["session_count"] == len(sessions)
 
 

tests/test_session_summary_cache.py

@@ -124,6 +124,14 @@ def test_lru_eviction(
     sample_session: Path, cache_db: Path, monkeypatch: pytest.MonkeyPatch
 ) -> None:
     monkeypatch.setattr("utils.session_summary_cache.DEFAULT_MAX_ROWS", 2)
+    clock = {"t": 100.0}
+
+    def fake_time() -> float:
+        clock["t"] += 100.0
+        return clock["t"]
+
+    monkeypatch.setattr("utils.session_summary_cache.time.time", fake_time)
+
     content = sample_session.read_text(encoding="utf-8")
     paths = []
     for name in ("a.jsonl", "b.jsonl", "c.jsonl"):
@@ -140,6 +148,10 @@ def test_lru_eviction(
             summary_from_peek(quick_session_info(str(p))),
         )
 
+    first = paths[0]
+    first_mtime = first.stat().st_mtime
+    assert get_summary(str(first), first_mtime, "none") is not None
+
     third = paths[2]
     third_mtime = third.stat().st_mtime
     put_summary(
@@ -149,8 +161,9 @@ def test_lru_eviction(
         summary_from_peek(quick_session_info(str(third))),
     )
 
-    first = paths[0]
-    assert get_summary(str(first), first.stat().st_mtime, "none") is None
+    second = paths[1]
+    assert get_summary(str(second), second.stat().st_mtime, "none") is None
+    assert get_summary(str(first), first_mtime, "none") is not None
     assert get_summary(str(third), third_mtime, "none") is not None
 
 

utils/session_summary_cache.py

@@ -116,6 +116,15 @@ def get_summary(path: str, mtime: float, rules_fingerprint: str) -> SummaryCache
         ).fetchone()
         if row is None:
             return None
+        conn.execute(
+            """
+            UPDATE summary_cache
+            SET accessed_at = ?
+            WHERE path = ? AND mtime = ? AND rules_fp = ?
+            """,
+            (time.time(), abspath, mtime, rules_fingerprint),
+        )
+        conn.commit()
         return _payload_to_row(str(row[0]))
 
 
@@ -174,7 +183,16 @@ def clear_cache() -> None:
 
 
 def summary_from_peek(info: QuickSessionInfoDict) -> SummaryCacheRowDict:
-    """Build a partial summary row from a lightweight JSONL peek."""
+    """Build a partial summary row from a lightweight JSONL peek.
+
+    ``is_excluded`` is hard-coded ``False`` because peek cannot evaluate exclusion
+    rules without reading full message bodies. Callers that need accurate exclusion
+    (``get_project_sessions``) must check ``is_complete`` and fall back to a full
+    parse. ``get_projects`` uses partial rows for fast card counts: with no
+    exclusion rules the value is correct; with rules active, excluded sessions may
+    be counted until the session list upgrades the row to a complete entry — this
+    is an accepted trade-off documented in issue #109.
+    """
     title = info["title"]
     return SummaryCacheRowDict(
         title=title,