Skip to content

fix(engine): settle resumed change zone members - #7246

Closed
matthewevans wants to merge 2 commits into
mainfrom
active/issue-6209-change-zone-terminal-outcome
Closed

fix(engine): settle resumed change zone members#7246
matthewevans wants to merge 2 commits into
mainfrom
active/issue-6209-change-zone-terminal-outcome

Conversation

@matthewevans

Copy link
Copy Markdown
Member

Fixes #6209.\n\nRecords the terminal outcome of a resumed paused zone-change delivery from its own captured event slice when an older path did not retain the sidecar classification. This prevents a logical zone-change group from completing with a member still pending.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a3368d94-99a0-4480-a1da-fc45cf3bb2d4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Base automatically changed from agent/on-deck-expend-auto-payment to main August 12, 2026 04:15
@matthewevans

Copy link
Copy Markdown
Member Author

Superseded by non-draft #7268, which carries the same zone-change fix plus its prompt-census update and has now been rebased onto current main. Closing this stale draft to avoid duplicate CI.

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.

Non-fatal engine panic: completed resumed ChangeZone owns every terminal member outcome: "logical zon…

1 participant