Skip to content

Remove hourly health-check push notifications - #13

Merged
oz6un merged 1 commit into
mainfrom
chore/remove-health-notifications
Jul 20, 2026
Merged

Remove hourly health-check push notifications#13
oz6un merged 1 commit into
mainfrom
chore/remove-health-notifications

Conversation

@oz6un

@oz6un oz6un commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Why

Following the notification cleanup: the user wants zero background health pushes. The hourly devbox-health timer only ever pushed on disk ≥85% or a failed unit, and in practice was provably silent (0 failed units, disk 52%, no pushes in days). For a personal devbox with a minimal-notifications goal, it's not worth keeping a channel that either says nothing or pings you about ops.

What

Removed the self-alerting feature entirely:

  • Deleted files/devbox-health.tmpl.
  • Removed the systemd service+timer install block from remote-setup.sh and the render step from setup-user.sh.
  • Dropped it from the Makefile gate list and the README (feature bullet, architecture-table row, daily-use mention).
  • Kept the Pushover key handling — claude-notify (the "Claude is waiting on you" hook) still uses it.

Live box brought to parity: timer disabled, units + script removed, daemon-reload. Push notifications are now exclusively the Claude-waiting/errored hook.

Verified

make check green; repo grep for devbox-health/self-alert references returns nothing; box shows 0 health units and hooks still ["Notification","StopFailure"].

🤖 Generated with Claude Code

The devbox-health systemd timer pushed to Pushover on disk >=85% or a failed
unit, but was provably silent in practice (0 failed units, disk 52%) and the
user wants no background ops pushes. Removed the template, the systemd
service+timer install, the render step, and all README/Makefile references.
claude-notify (Claude-is-waiting hook) keeps the Pushover keys. Live box
brought to parity (timer disabled, units + script removed).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@oz6un
oz6un merged commit f636d40 into main Jul 20, 2026
1 check passed
@oz6un
oz6un deleted the chore/remove-health-notifications branch July 20, 2026 07:06
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.

1 participant