Repository navigation
fix(examples): clip time spent today to local calendar bounds - #164
TimeToBuildBob wants to merge 3 commits into
Conversation
Git-Session-Id: 350bcd90-c80d-427d-9595-03c4285ca081
[Low impact] The PR appears safe to merge. SummaryUpdates
Reviews (1) · Last reviewed commit: "fix(examples): clip time spent today to ..." · Reviewed by Greptile |
Git-Session-Id: 56060ebf-6713-5eee-b092-043118cee230
|
Fixed the build setup failure in Local verification: 137 offline tests passed (including all 15 local-day regressions), mypy passed for 31 source files, and the workflow YAML/matrix checks passed. The new CI run is queued, including |
Git-Session-Id: de442d75-d4a4-527d-b790-fac44a8340cc
|
Fixed the Windows typecheck failure in Verified locally: the exact Windows-target mypy error reproduced before the fix; afterward all 31 files pass mypy for Windows, Linux and macOS targets. All 137 offline tests pass on Linux, and removing |
|
CI-green and mergeable (Greptile 5/5) — waiting only on a maintainer click. This PR is ready to merge, but the bot has pull-only access to this repo and can't self-merge — surfacing it here so it isn't lost. The monitoring loop will stop re-flagging it now that this note is posted. |
Summary
Reproduction and validation
Fresh upstream master c88cf12 prints 2:00:00 for a 23:00–01:00 active event when queried at noon; only 1:00:00 belongs to today. New regressions before repair: 11 failed / 4 passed. After repair: all 15 pass; 74 targeted tests pass including profile/config/proxy coverage. Repository-wide make typecheck passes (31 files); Ruff lint/format and whitespace checks pass. The independent filing-time behavior probe now prints 1:00:00 on this branch.
This repository has no make test-fast target; explicit bounded pytest slices were used instead. Tests use synthetic events and a controlled clock, not live user history. Process-local TZ regressions require time.tzset and skip on Windows; Linux/macOS exercise them. No datastore overlap, deduplication, historical workday or interval-union policy changes.