Skip to content

Pop-only inbox API, self-send gate, assignee 'self' -> 'user' - #94

Open
rr0hit wants to merge 1 commit into
mainfrom
feat/inbox-pop-only
Open

Pop-only inbox API, self-send gate, assignee 'self' -> 'user'#94
rr0hit wants to merge 1 commit into
mainfrom
feat/inbox-pop-only

Conversation

@rr0hit

@rr0hit rr0hit commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Follow-up simplification to #93 per design review:

  • flow inbox pop is the only consumption API. ack and due subcommands removed, escalation machinery (notify schedule, backoff, attempts) deleted — reminder cadence is the user's notifier script's business (loop/cron flow inbox --as user --json). Ack-on-reply and wait metrics are unchanged (hook behavior, not API).
  • Self-send gate: no sender can message its own address (session → own inbox, human → own queue) — that's always a confused agent.
  • Reserved assignee renamed selfuser: agents misread message self as talking to themselves. Existing rows migrate idempotently on open. The one-shot post→broadcast kind migration is deleted (no released DB needed it).
  • Docs/skill rewritten; suite green; live DB migrated and verified.

🤖 Generated with Claude Code

https://claude.ai/code/session_01PFGNJQo5xNBBrT7rZMuUTu

- flow inbox pop is the ONLY consumption API: ack and due subcommands
  removed, along with the whole escalation apparatus (next_notify_at /
  attempts columns for fresh installs, DueBusMessages, BumpNotifyAttempt,
  AckMessageByID). Popping answers, delivers, and clears; agents and
  notifier scripts just loop it. Ack-on-reply (the UserPromptSubmit
  hook) and wait metrics remain — that is behavior, not API surface.
- No sender can message its own address: a bound session is rejected
  addressing its own inbox, and the human their own queue.
- Reserved assignee renamed self -> user (agents misread 'message self'
  as talking to themselves). Existing rows migrated idempotently on
  open; the one-shot post->broadcast kind migration is deleted (no
  released DB ever needed it).
- Usage, skill §4.18, and references/messaging.md rewritten to match.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PFGNJQo5xNBBrT7rZMuUTu
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