Skip to content

fix(engine): reconcile serialized zone change occurrences - #7077

Merged
matthewevans merged 1 commit into
ship/fix-turn-occurrence-provenance-7065from
fix/serialized-zone-change-provenance-7066
Aug 7, 2026
Merged

fix(engine): reconcile serialized zone change occurrences#7077
matthewevans merged 1 commit into
ship/fix-turn-occurrence-provenance-7065from
fix/serialized-zone-change-provenance-7066

Conversation

@matthewevans

Copy link
Copy Markdown
Member

Fixes #7066 after #7065.

The shared persisted-state decoder now reconciles serialized zone-change occurrence keys before Raw or Trusted state materialization. It canonicalizes ledger positions, repairs only uniquely identifiable carriers, rejects ambiguous or inconsistent data, and preserves correct prior-turn provenance.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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: 3f28153c-4360-4247-b454-889e0651a702

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.

@matthewevans
matthewevans merged commit 507a03e into ship/fix-turn-occurrence-provenance-7065 Aug 7, 2026
3 checks passed
@matthewevans
matthewevans deleted the fix/serialized-zone-change-provenance-7066 branch August 7, 2026 13:33
matthewevans added a commit that referenced this pull request Aug 7, 2026
Co-authored-by: matthewevans <matthewevans@users.noreply.github.com>
matthewevans added a commit to jsdevninja/phase that referenced this pull request Aug 7, 2026
* fix(engine): qualify zone-change occurrences by turn

* test(engine): cover cross-turn zone occurrences

* fix(engine): migrate legacy batched trigger keys

* test(engine): cover cross-turn batched replay keys

* fix(tests): update batched trigger entry access

* fix(tests): refresh zone-change provenance fixtures

* fix(engine): reconcile serialized zone change occurrences (phase-rs#7077)

Co-authored-by: matthewevans <matthewevans@users.noreply.github.com>

* fix(engine): scope persisted zone event reconciliation

* fix(engine): reconcile paused sacrifice events

* fix(engine): respect resolution frame boundary

* fix(engine): reconcile canonical frame event carriers

* fix(engine): prune stale zone-change ledger rows

* fix(engine): reindex batched zone-change replay keys

* fix(engine): reject stale zone-change replay aliases

* fix(engine): reconcile serialized resolution frame events

* fix(engine): cover v1 resolution event carriers

* test(engine): pass resolution fixture as JSON value

* fix(engine): retain unkeyed legacy resolution events

* test(engine): assert persisted provenance canonicalization

* test(engine): initialize boxed resolution stacks

* test(engine): box serialized frame fixture stack

---------

Co-authored-by: matthewevans <matthewevans@users.noreply.github.com>
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