Skip to content

fix: cancel consolidation across session reloads - #58

Open
gpmarques wants to merge 1 commit into
elpapi42:masterfrom
gpmarques:fix/reload-cancel-consolidation
Open

fix: cancel consolidation across session reloads#58
gpmarques wants to merge 1 commit into
elpapi42:masterfrom
gpmarques:fix/reload-cancel-consolidation

Conversation

@gpmarques

Copy link
Copy Markdown

Summary

Reloading or replacing a session during active consolidation reproduced a stale-runtime append: late observer/reflector output could write through the old extension instance, while reflection work was lost instead of being retried by the fresh session.

This change:

  • cancels observer, reflector, dropper, model resolution, and deferred compaction work on shutdown
  • guards every post-await result and ledger append with a session/runtime generation
  • lets a fresh runtime retry consolidation without accepting stale output

Verification

  • 265 native tests passed
  • typecheck passed

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