fix(jsonl): address parser-split review follow-ups

Author: clean6378-max-it

tests/test_jsonl_parser.py

@@ -234,6 +234,16 @@ def test_plan_result(self):
         r = _parse_tool_result({"plan": [], "filePath": "/plan.md"})
         assert r["result_type"] == "plan"
 
+    def test_plan_with_content_not_classified_as_file_write(self):
+        """plan is registered before file_write in _TOOL_RESULT_DISPATCH."""
+        r = _parse_tool_result({
+            "plan": [],
+            "filePath": "/plan.md",
+            "content": "plan body",
+        })
+        assert r["result_type"] == "plan"
+        assert r["file_path"] == "/plan.md"
+
     def test_unknown_fallback(self):
         r = _parse_tool_result({"unexpected": True})
         assert r["result_type"] == "unknown"

utils/jsonl_parser.py

@@ -15,7 +15,6 @@
     normalize_content as _normalize_content,
     strip_system_tags as _strip_system_tags,
 )
-from utils.session_peek import quick_session_info
 from utils.tool_dispatch import _TOOL_RESULT_DISPATCH, _parse_tool_result
 from utils.validation import validate_session_dict
 
@@ -38,6 +37,14 @@
 ]
 
 
+def __getattr__(name: str) -> Any:
+    if name == "quick_session_info":
+        from utils.session_peek import quick_session_info
+
+        return quick_session_info
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")
+
+
 def parse_session(filepath: str) -> SessionDict:
     """Main entry point. Reads every line from a .jsonl file and builds up
     a session dict with messages, metadata (tokens, models, tool counts),

utils/session_peek.py

@@ -15,8 +15,8 @@ def quick_session_info(filepath: str) -> QuickSessionInfoDict:
     without fully parsing all messages.  Much faster than parse_session() for
     large files.
 
-    Strategy: read the first ~50 lines for the title, then seek to the end of
-    the file and read the last chunk to find the last timestamp."""
+    Strategy: files over 10 KiB cap the head scan at 80 lines for title, then
+    tail-read for last_timestamp; smaller files are scanned fully in pass 1."""
     title = None
     first_ts = None
     last_ts = None

utils/tool_dispatch.py

@@ -229,20 +229,11 @@ def _tool_result_build_user_input(tr: dict[str, Any], base: dict[str, Any]) -> d
     return result
 
 
-# Dispatch registry: **first matching predicate wins** (same as legacy if/elif).
-# Order is load-bearing — do not sort alphabetically or “more specific first”
-# without replaying tests and real session fixtures.
-#
-# Notably ``task_message`` is intentionally broad (``task_id`` or ``message``)
-# and sits before ``task_retrieval`` / ``task_completed`` / ``task_async`` so
-# payloads that include overlapping keys still match the legacy branch order.
-#
-# To add a shape: append ``(pred, build)`` here, or insert only after verifying
-# predicates above would not steal intended matches.
+# Registry order is load-bearing (see module docstring).
+# ``plan`` before ``file_write``: plan blobs may carry ``filePath`` + ``content``.
 _TOOL_RESULT_DISPATCH = (
     (_tool_result_pred_bash, _tool_result_build_bash),
     (_tool_result_pred_file_edit, _tool_result_build_file_edit),
-    # plan before file_write: plan blobs may also carry filePath + content
     (_tool_result_pred_plan, _tool_result_build_plan),
     (_tool_result_pred_file_write, _tool_result_build_file_write),
     (_tool_result_pred_glob, _tool_result_build_glob),