feat(cloud): negotiate idempotent steer delivery#3538
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
React Doctor found no issues in the changed files. 🎉 Reviewed by React Doctor for commit |
bf461dd to
8bc8d62
Compare
7c71a62 to
5f639c8
Compare
|
👋 Visual changes detected for this PR. Review and approve in PostHog Visual Review If these changes are unexpected, they may be caused by a flaky test or a broken snapshot on master. Don't approve — rerun the job or wait for a fix. |
8bc8d62 to
49fc050
Compare
5f639c8 to
9192b57
Compare
There was a problem hiding this comment.
This is a T1d-complex PR (741 lines, 7 files) with zero reviews, touching core agent protocol semantics: new concurrent delivery deduplication via inFlightMessageDeliveries, a counter-based owned-turn tracker for steer eligibility, and a behavioral policy change to sessionSupportsNativeSteer that enables cloud steering when explicitly negotiated. The code appears internally consistent and well-tested, but the concurrency state and protocol-level changes warrant at least one domain expert sign-off before auto-approval.
49fc050 to
eba3248
Compare
9192b57 to
a04d2b1
Compare

Problem
Cloud steer delivery needs negotiated capability and idempotent retry behavior.
Changes
Adds protocol negotiation, in-flight delivery deduplication, and safe normal-turn fallback.