Skip to content

Notify source threads after verified deployment - #29

Merged
mberman84 merged 1 commit into
mainfrom
codex/thread-deployed-notifications-20260620
Jun 21, 2026
Merged

mberman84 merged 1 commit into
mainfrom
codex/thread-deployed-notifications-20260620

Conversation

@mberman84

Copy link
Copy Markdown
Contributor

What changed

  • records a durable per-thread notification obligation when an authorized pull request merges
  • promotes the obligation only after the containing exact main deployment and health checks verify
  • returns retryable native-thread notification payloads with stable idempotency IDs
  • adds provider delivery and acknowledgement guidance for Codex, Claude Code, and Cursor adapters
  • preserves notifications across thread reuse, expiry, coordinator replacement, and newer-main supersession

Safety

  • generic clients cannot claim the controller-owned deployed phase
  • acknowledgements are bound to provider, thread, merge generation, and notification ID
  • failed native delivery remains pending; registry comments never count as user-visible delivery
  • stale acknowledgements cannot complete newer work in the same thread

Validation

  • 144 unit tests passed
  • ruff format --check .
  • ruff check .
  • git diff --check
  • independent autoreview: clean, no accepted/actionable findings

@mberman84
mberman84 merged commit 5926136 into main Jun 21, 2026
3 checks passed
@mberman84
mberman84 deleted the codex/thread-deployed-notifications-20260620 branch June 21, 2026 15:46
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