Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: dbroeglin/github-copilot-lab
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: dbroeglin/github-copilot-lab
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: dbroeglin/otel-cache-token-keys
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 1 commit
  • 3 files changed
  • 2 contributors

Commits on Jun 27, 2026

  1. test: broaden coverage of analysis, OTel merge, and rendering 

    Add offline unit tests targeting the session-analysis internals that were
thinly covered:

- analyze_trajectory (ATIF) driven by synthetic, hand-controlled inputs
  (metadata, tool histogram + failure detection, final_metrics economics,
  summed-output fallbacks, malformed-input guards). Real trajectory.json is
  produced by this package's own converter, so it can't serve as authoritative
  golden data; these tests exercise the parsing logic directly.
- _apply_otel_records merge: shutdown stays authoritative while turns are still
  enriched; OTel-only reconstruction without a shutdown; unmatched turn_id and
  no-chat-span paths.
- OTel decoders: _otel_attrs (dict + OTLP list), _otel_value typed scalars and
  array/kvlist, _otel_attr_value flat + nested dotted lookup, _parse_otel_time
  across ISO / [sec,nanos] / epoch s|ms|ns, and llm_calls_from_otel over
  OTLP list-format attributes.
- sessionlog: load_events blank/invalid skipping, copy_events roundtrip and
  missing-source, compaction_start peak tracking without a shutdown.
- render: render_session_analysis overview incl. the multi-model "Per model"
  table and the no-economics path; remaining LiveEventFormatter branches.

Coverage: analysis.py 77%->90%, render.py 87%->93%, sessionlog.py 90%->93%.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    @dbroeglin
    dbroeglin and Copilot committed Jun 27, 2026
    Configuration menu
    Copy the full SHA
    d3c47b2 View commit details
    Browse the repository at this point in the history
Loading