Repository navigation
fix(daily-checkin): flag skipped days in the archived window - #196
Merged
Merged
Conversation
harlan-github-agent
Bot
added
harlan-agent-running
Contributor
Author
🤖 MERGED
GitHub merged this pull request. No material findings were recorded. Review completed. No material findings. |
harlan-github-agent
Bot
added
harlan-agent-blocked
harlan-github-agent
Bot
added
harlan-agent-running
harlan-github-agent
Bot
added
harlan-agent-running
Resolve test conflict with #199. Claude-Session: https://claude.ai/code/session_01AkTC7eqkf2tpVZJhYDU2YL
harlan-github-agent
Bot
added
harlan-agent-running
harlan-github-agent
Bot
removed
the
harlan-agent-running
harlan-zw
added a commit
that referenced
this pull request
Sep 14, 2026
Resolve test conflict with #196. Claude-Session: https://claude.ai/code/session_01AkTC7eqkf2tpVZJhYDU2YL
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 Linked issue
Closes #195
❓ Type of change
📚 Description
The check-in routine does not run every day, and when it skips days the archive still reads as one ordinary overnight window. The only trace of the skip was a raw
sincetimestamp nobody derived from, so a six-day gap got its rates compared against single-day windows in past verdicts.The archived doc now carries the derived window next to
since:A window past 36 hours is
stale, so one fully missed day flags while ordinary cadence jitter staysfresh. A corrupted or future baseline reads asinvalidrather than a confident negative gap, sincestate.jsonis hand-editable. The check-in skill gates on the tag, so the morning verdict names the wider window and compares rates per day.The 36-hour cut is a judgement call, and it is the one thing worth a reviewer's eye: a run twelve hours late flags, a run a few hours late does not.