chore(self): wire ntfy push notifications into the notify seam - #202
Merged
Conversation
robercano
approved these changes
Jul 30, 2026
…99 follow-up) Set the self-adapter's notify command to a curl one-liner posting to ntfy.sh. The topic name is machine-local (untracked .claude/state/ntfy-topic, so no secret lands in the repo per CLAUDE.md); a missing topic file degrades to a silent no-op, preserving notify.sh's offline/CI-safe contract. Co-Authored-By: Claude Fable 5 <[email protected]>
robercano
force-pushed
the
chore/ntfy-notify
branch
from
July 30, 2026 10:20
819c8cc to
5aff87d
Compare
Collaborator
Author
|
Rebased |
robercano-ghbot
added
the
needs-human
Collaborator
Author
|
chore(self): wire ntfy push notifications into the notify seam (approval is stale -- please re-review the current head) |
robercano
approved these changes
Jul 30, 2026
robercano-ghbot
removed
the
needs-human
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
2 participants
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.
Fills the self-adapter's
notifykey (issue #99 seam, shipped empty) with an ntfy.sh curl one-liner, so everyneeds_human_flagescalation — attempt-budget exhaustion, stall, PR-awaiting-review, protected-paths — reaches the owner's phone as a push notification..claude/state/ntfy-topic(.claude/state/is gitignored), so no secret lands in the repo.ntfy.sh/and fails → swallowed by notify.sh's never-propagate contract: offline/CI/consumer-safe no-op, same as before.high/low) are valid ntfyPriorityvalues..claude/gates.jsonstays pristine/empty per convention.Context: the v0.3.0 release escalation (#178, 5× phantom drivers on 2026-07-29) fired this seam into the void — the owner only found out by asking the next day.
🤖 Generated with Claude Code