fix(cloud-agent-next): preserve cleanup retirement target - #5916
Conversation
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryIncremental rebase review: unique session-delivery commit is unchanged versus the prior review; the only tree delta is a parent-inherited Files Reviewed (1 file)
Previous Review Summaries (6 snapshots, latest commit 9438fc4)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 9438fc4)Status: No Issues Found | Recommendation: Merge Executive SummaryRebased unique session-delivery commit is unchanged versus the prior review; cleanup stays pinned to the retiring runtime, and no high-confidence issues were found. Files Reviewed (20 files)
Previous review (commit ff4fd8d)Status: No Issues Found | Recommendation: Merge Executive SummaryFull review of the rewritten session-delivery commit: cleanup stays pinned to the retiring runtime, and no high-confidence issues were found. Files Reviewed (20 files)
Previous review (commit 81e9788)Status: No Issues Found | Recommendation: Merge Executive SummaryFull review of the rewritten session-delivery commit: cleanup stays pinned to the retiring runtime, and no high-confidence issues were found. Files Reviewed (20 files)
Previous review (commit 1db50ec)Status: No Issues Found | Recommendation: Merge Executive SummaryIncremental review of the sandbox-control integration test that now asserts cleanup remains pinned to the retiring runtime until stop completes; no issues found. Files Reviewed (1 file)
Previous review (commit 757bb70)Status: No Issues Found | Recommendation: Merge Files Reviewed (20 files)
Previous review (commit f8de89b)Status: No Issues Found | Recommendation: Merge Files Reviewed (20 files)
Reviewed by grok-4.6 · Input: 52.9K · Output: 6.6K · Cached: 332.3K Review guidance: REVIEW.md from base branch |
757bb70 to
1db50ec
Compare
1db50ec to
81e9788
Compare
81e9788 to
ff4fd8d
Compare
ff4fd8d to
9438fc4
Compare
9438fc4 to
3a942b4
Compare
Summary
Stack
Control-plane recovery stack. Merge from the bottom. Each PR targets its parent, not
main(except #5912).eshurakov/recovery-observation→maineshurakov/recovery-results→eshurakov/recovery-observationeshurakov/recovery-stop-scope→eshurakov/recovery-resultseshurakov/recovery-native-cleanup→eshurakov/recovery-stop-scopeeshurakov/recovery-session-delivery→eshurakov/recovery-native-cleanupeshurakov/recovery-native-feed→eshurakov/recovery-session-deliveryeshurakov/recovery-control-connection→eshurakov/recovery-native-feedeshurakov/recovery-stack→eshurakov/recovery-control-connectioneshurakov/recovery-not-ready-retry→eshurakov/recovery-stackDo not merge out of order.
Reviewer Notes
One commit. Unique vs parent.