Skip to content

fix(daily-checkin): verify a week of health email reports, not two - #197

Merged
harlan-zw merged 4 commits into
mainfrom
fix/issue-194
Sep 14, 2026
Merged

harlan-zw merged 4 commits into
mainfrom
fix/issue-194

Conversation

@harlan-github-agent

Copy link
Copy Markdown
Contributor

🔗 Linked issue

Closes #194

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

The daily check-in gate expects one sent operator report per Melbourne date, but the probe only ever read the two newest rows:

-- before
SELECT ... FROM daily_health_checks ORDER BY report_date DESC LIMIT 2
-- after
SELECT ... FROM daily_health_checks WHERE report_date >= date('now', '-8 days') ORDER BY report_date DESC LIMIT 8

A gap three days back could never show up, so continuity was unverifiable. Eight UTC days is the smallest bound that always covers seven Melbourne dates, and report_date is the primary key of daily_health_checks, so the row cap can never hide an in-window date. The query now lives beside the other built queries where it can be asserted on through the export.

The skill's gate wording never named a visible window, so it needed no change. One uncertainty: this sandbox has no outbound network, so the Nuxt test environment could not boot here; the vitest suite runs in CI where it does.

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

…verifies one sent operator report per Melbourne date, and a two-row page could only ever confirm the two newest days, so a gap three days back stayed invisible. Eight UTC
@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Sep 9, 2026
@harlan-github-agent

harlan-github-agent Bot commented Sep 9, 2026 •

Copy link
Copy Markdown
Contributor Author

🤖 MERGED

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. Last updated: 2026-09-14 10:53 UTC.

GitHub merged this pull request.

No material findings were recorded.

Review completed. No material findings.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-pending The automated Review is waiting on a gate for this head commit. harlan-agent-blocked The automated Review found a material defect in this head commit. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. harlan-agent-pending The automated Review is waiting on a gate for this head commit. labels Sep 9, 2026
@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-pending The automated Review is waiting on a gate for this head commit. harlan-agent-blocked The automated Review found a material defect in this head commit. and removed harlan-agent-blocked The automated Review found a material defect in this head commit. harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-pending The automated Review is waiting on a gate for this head commit. labels Sep 14, 2026
@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-blocked The automated Review found a material defect in this head commit. harlan-agent-running An Agent holds a Task on this issue or pull request right now. labels Sep 14, 2026
@harlan-zw
harlan-zw merged commit 8dd3882 into main Sep 14, 2026
5 checks passed
@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. labels Sep 14, 2026
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.

Health email probe verifies only two Melbourne dates

1 participant