Skip to content

Clarify status telemetry and thread freshness in the UI and CLI - #81

Merged
tarekziade merged 1 commit into
mainfrom
fix/status-freshness-46
Sep 16, 2026
Merged

tarekziade merged 1 commit into
mainfrom
fix/status-freshness-46

Conversation

@tarekziade

Copy link
Copy Markdown
Collaborator

Collector timestamps currently look like response freshness, making the index appear both newer and older than the thread data it serves. Label status as collector telemetry in the web UI, both CLIs, and JSON; explain that bulk comment checkpoints do not track per-thread refreshes. Replace thread and outline “current to” stamps with “indexed at,” naming body/comment scope and the unknown later GitHub changes.

Closes #46.

Validation: make check with SQLite and Postgres — 1,194 passed, 4 skipped; final JSON stamp assertion also passed on both backends. Includes renderer and status JSON regression coverage and UI JavaScript syntax checks.

Live investigation: transformers polling is healthy (successful poll at 2026-09-16 08:37:54 UTC); the nightly files job also succeeded. The September 9 bulk-comment rows are historical checkpoints, not evidence of a stalled poller.

Bumps 0.3.16 → 0.3.17 for the visible/API changes. Deploy client and daemon together when merging; this PR has not been deployed.

@tarekziade tarekziade self-assigned this Sep 16, 2026
@tarekziade
tarekziade merged commit e0cbec9 into main Sep 16, 2026
2 checks passed
@tarekziade
tarekziade deleted the fix/status-freshness-46 branch September 16, 2026 08:46
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.

status overstates thread-comment freshness by ~5h, and no status row matches the current to the responses carry

1 participant