docs(adr): push notifications — event-driven read-model over polling - #12
Draft
brettchien wants to merge 3 commits into
Draft
docs(adr): push notifications — event-driven read-model over polling#12brettchien wants to merge 3 commits into
brettchien wants to merge 3 commits into
Conversation
skin↔core via MCP resource subscriptions, core↔AWS via EventBridge, polling demoted to a reconciliation fallback. Two-phase: (1) skin stops polling, core pushes resources/updated (no AWS infra); (2) EventBridge → zero steady-state ECS polling. OS/mobile notifications are a downstream consumer of the stream. Supersedes ADR-3's "polling first" for live updates. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
brettchien
marked this pull request as draft
August 10, 2026 04:20
…k8s watch+events) Add §3.3: each provider's native stream maps to one resources/updated contract; k8s watch (informer list→watch→resync; 410 Gone → re-list) is the reference reconciliation shape the ECS adapter emulates. Tighten the multi-vendor open Q. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- 'Realizes' ADR-3's deferred streaming, not 'Supersedes' (ADR-3 not yet accepted) - subscriber semantics: read current state then resources/updated deltas (matches MCP; no auto-snapshot on subscribe) - reference the sidecar reader as PR #9 rather than a not-yet-on-main file link Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
Defines the "later" in ADR-3's "polling first, streaming later" for live updates.
Decision: push at two boundaries, two phases.
resources/subscribe, core sendsnotifications/resources/updatedon change. The 5 s poll leaves the skin. (Phase 1 — no AWS infra; core may still poll AWS internally.)resources/updated. Steady-state ECS polling → zero. (Phase 2.)Unhealthy) are a downstream consumer of the same stream, not a separate pipeline.Notes the desktop already half-receives notifications (the sidecar MCP reader gets id-less lines today; Phase 1 makes it act on them), and that remote/iOS skins need a streaming transport (ADR-3 deferred) before they can subscribe.
Draft — for review/wording before Accepted. Companion to
desktop-core-sidecar.md(#9).🤖 Generated with Claude Code