Skip to content

test(statistics): cover the timeline and ignore a start-only timing - #39

Open
efirvida wants to merge 2 commits into
upstream/statistics-10-timelinefrom
upstream/statistics-11-timeline-tests
Open

efirvida wants to merge 2 commits into
upstream/statistics-10-timelinefrom
upstream/statistics-11-timeline-tests

Conversation

@efirvida

@efirvida efirvida commented Oct 1, 2026

Copy link
Copy Markdown
Owner

Chain Context

Field Value
Chain per-session statistics module (upstream Gentleman-Programming#1583)
Tracker upstream/statistics-tracker → upstream integration PR Gentleman-Programming#1627 (draft)
Position 11 of 27
Base upstream/statistics-10-timeline
Depends on #38
Follow-up upstream/statistics-11-timeline-tests (slice 12)
Review budget 217 / 400
Starts at upstream/statistics-10-timeline
Ends with test(statistics): cover the timeline and ignore a start-only timing

Chain Overview

upstream/main
 └── upstream/statistics-tracker (root, integration PR #1627)
      └── … slices 1..10 … (##38 is the immediate parent)
           └── 📍 upstream/statistics-11-timeline-tests
                └── … slices 12..27 …

Scope

  • Includes: test(statistics): cover the timeline and ignore a start-only timing
  • Excludes: every other slice. A polluted diff against upstream/statistics-10-timeline is a branching bug, not a scope surprise.

Verification

Focused and affected suites are green on this slice; tests and docs for this unit ship with it.

…half override

A `gentle-ai-elapsed-timing/v1` entry can exist with only `startedAt` when a
tool started and never recorded its end. Using that start with the derived end
was a silent half override, and it was not flagged `whitelisted`. Only a
complete timing (with `endedAt`) is now authoritative and flagged; a start-only
entry is ignored and the derived duration stands.

Found by the independent oracle comparison against a frozen real transcript:
the one 13 ms divergence on `gentle_review` was the intended whitelisted
override, which made the half-override case visible.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant