fix(claude-code): address Greptile review (num_llm_calls, dead code, session_id)

- TurnUsage.num_llm_calls is now Optional and defaults to None: it is
  provider-reported (from claude-code's num_turns) and may be absent, so None
  ("not reported") is distinct from a real 0. claude_code_usage_to_turn_usage
  returns None when num_turns is missing (real zeros still preserved). Counted
  fields (num_tool_calls / num_reasoning_blocks) stay int=0.
- Removed a dead `full_text` assignment in the thinking content_block_stop path.
- Added a public ClaudeCodeTurn.session_id accessor (mirrors CodexTurn) so the
  temporal activity no longer reaches into the private _result_envelope.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: declan-scale

examples/tutorials/10_async/10_temporal/140_claude_code/project/activities.py

@@ -136,8 +136,4 @@ async def run_claude_code_turn(params: RunClaudeCodeTurnParams) -> dict[str, Any
     turn = ClaudeCodeTurn(_spawn_claude(params.prompt, session_id=params.session_id))
     result = await emitter.auto_send_turn(turn, created_at=params.created_at)
 
-    session_id: str | None = None
-    if getattr(turn, "_result_envelope", None):
-        session_id = turn._result_envelope.get("session_id")
-
-    return RunClaudeCodeTurnResult(final_text=result.final_text, session_id=session_id).model_dump()
+    return RunClaudeCodeTurnResult(final_text=result.final_text, session_id=turn.session_id).model_dump()

src/agentex/lib/adk/_modules/_claude_code_sync.py

@@ -341,7 +341,6 @@ async def convert_claude_code_to_agentex_events(
 
             elif se_type == "content_block_stop":
                 if _thinking_open:
-                    full_text = _thinking_buf
                     _thinking_open = False
                     _thinking_buf = ""
                     _pending_thinking_block_index = None

src/agentex/lib/adk/_modules/_claude_code_turn.py

@@ -81,8 +81,11 @@ def _float(d: dict[str, Any], *keys: str) -> float | None:
     cost_usd = _float(result_envelope, "cost_usd", "total_cost_usd")
     duration_ms = _int(result_envelope, "duration_ms")
 
+    # num_llm_calls is provider-reported (from num_turns): default None ("not
+    # reported") rather than 0 so callers can distinguish it from a real zero,
+    # matching the None convention used for the token fields above.
     num_turns = result_envelope.get("num_turns")
-    num_llm_calls = 0
+    num_llm_calls: int | None = None
     if num_turns is not None:
         try:
             num_llm_calls = int(num_turns)
@@ -129,6 +132,18 @@ def events(self) -> AsyncIterator[StreamTaskMessage]:
             )
         return self._events_stream
 
+    @property
+    def session_id(self) -> str | None:
+        """The Claude Code session id, for resuming a multi-turn session.
+
+        Valid only after ``events`` has been fully consumed (populated by the
+        ``result`` envelope). Returns ``None`` if the stream was truncated or
+        Claude Code reported no session id.
+        """
+        if not self._result_envelope:
+            return None
+        return self._result_envelope.get("session_id")
+
     def usage(self) -> TurnUsage:
         """Return normalised usage for this turn.
 

src/agentex/lib/core/harness/types.py

@@ -64,7 +64,10 @@ class TurnUsage(BaseModel):
     total_tokens: int | None = None
     cost_usd: float | None = None
     duration_ms: int | None = None
-    num_llm_calls: int = 0
+    # num_llm_calls is provider-reported and may be absent (None = "not
+    # reported"). num_tool_calls / num_reasoning_blocks are counted locally from
+    # the observed stream, so 0 is always a real count.
+    num_llm_calls: int | None = None
     num_tool_calls: int = 0
     num_reasoning_blocks: int = 0
 

tests/lib/adk/test_claude_code_turn.py

@@ -84,7 +84,7 @@ def test_missing_usage_key_returns_nones(self):
         assert usage.total_tokens is None
         assert usage.cost_usd is None
         assert usage.duration_ms is None
-        assert usage.num_llm_calls == 0
+        assert usage.num_llm_calls is None
 
     def test_real_zeros_preserved(self):
         result = {