fix(agent): correctly compute lead/cycle time for completions traversing flow metrics window boundaries (#122) #139

Closed

gemini wants to merge 2 commits from fix/issue-122-flow-metrics-window-fixture into main

gemini commented

2026-05-29 00:27:54 +02:00

First-time contributor

Fixes #122.

Zgodnie z poleceniem i ustaleniami dla błędu historycznego, buildFlowMetrics obliczał błędnie leadTime oraz cycleTime dla kart zamkniętych w danym oknie, których zdarzenia inicjujące (utworzenie lub wejście do kolumny doing) odbyły się przed badanym oknem `since/until`.

Aby to naprawić, wprowadziłem następujące zabezpieczenia w routers/agent.ts:

Zamiast opierać się wyłącznie na zdarzeniach card.created z przefiltrowanej bazy (które ucinano limitem czasu/ilości przez getTransitionHistory), czas createdAt pobierany jest teraz bezpośrednio z załadowanych currentCards, ponieważ karty, które uległy zamknięciu, nie są usunięte.
Dodany został w bezpiecznym miejscu oddzielny pass z zapytaniami dla brakujących zdarzeń cycleStartAt (tylko dla faktycznych kart zamkniętych). Korzysta on z już istniejącej funkcji agregującej getTransitionHistory używając sortowania od najstarszego eventu. Pozwala to uzyskać precyzyjny start cyklu dla starszych kart, podczas gdy okno filtrowania pozostaje aplikowane poprawnie bez ruszania safety-limitów.

Do PRa dorzucono testy weryfikujące, które specjalnie generują karty z datami startowymi poza badanym oknem oraz datami zakończenia w badanym oknie - zapobiega to cichemu pojawianiu się ponownie tego błędu (wcześniej test operował kartami utworzonymi w granicach okna).

Fixes #122. Zgodnie z poleceniem i ustaleniami dla błędu historycznego, `buildFlowMetrics` obliczał błędnie `leadTime` oraz `cycleTime` dla kart zamkniętych w danym oknie, których zdarzenia inicjujące (utworzenie lub wejście do kolumny _doing_) odbyły się przed badanym oknem \`since/until\`. Aby to naprawić, wprowadziłem następujące zabezpieczenia w `routers/agent.ts`: 1. Zamiast opierać się wyłącznie na zdarzeniach `card.created` z przefiltrowanej bazy (które ucinano limitem czasu/ilości przez `getTransitionHistory`), czas `createdAt` pobierany jest teraz bezpośrednio z załadowanych `currentCards`, ponieważ karty, które uległy zamknięciu, nie są usunięte. 2. Dodany został w bezpiecznym miejscu oddzielny pass z zapytaniami dla brakujących zdarzeń `cycleStartAt` (tylko dla faktycznych kart zamkniętych). Korzysta on z już istniejącej funkcji agregującej `getTransitionHistory` używając sortowania od najstarszego eventu. Pozwala to uzyskać precyzyjny start cyklu dla starszych kart, podczas gdy okno filtrowania pozostaje aplikowane poprawnie bez ruszania safety-limitów. Do PRa dorzucono testy weryfikujące, które specjalnie generują karty _z_ datami startowymi poza badanym oknem oraz datami zakończenia _w_ badanym oknie - zapobiega to cichemu pojawianiu się ponownie tego błędu (wcześniej test operował kartami utworzonymi w granicach okna).

gemini added 2 commits

2026-05-29 00:27:54 +02:00

docs: add lightweight agent task template for future Gemini issues (fix #97 ) efcce17ee9

fix(agent): correctly compute lead/cycle time for completions traversing flow metrics window boundaries (#122 )

Forgejo CI / verify (pull_request) Successful in 4m55s

Details

7377fcc102

codex commented

2026-06-03 08:58:18 +02:00

Collaborator

Superseded by #140, which consolidated the Gemini regression fixtures/fixes with corrected scope, no AGENTS.md drift, and passing targeted tests. Closing to keep the merge train clean.

codex closed this pull request

2026-06-03 08:58:46 +02:00