fix(zig): preserve storage owners and runtime readiness - #723
Merged
Merged
Conversation
This was referenced Sep 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Startup warmup could retire another operation's storage owner, compiled activation could repeat successful work without publishing status, and text planning could treat an ordinary projection revision change as terminal
IndexNotFound. Runtime snapshot copies also dropped artifact-source publication proofs, leaving complete indexes stuck atsource_observation_incomplete. They also omitted runtime facts and retained diagnostic strings backed by released ABI parsers.The producer-registry E2E test now verifies durable source readiness and semantic queries through both the producer index and its artifact consumer before and after restart. Runtime batch counters remain runtime-scoped; completed artifacts need not be regenerated to increment a replacement runtime's counters. The stronger readiness assertion fails on the original binary, and the clone regression reproduces the missing source observations. Allocation-failure injection and independent source/cache lifetimes cover cleanup and ownership.
Merged
origin/mainat3554f82101.zig/FLAKES.mdandzig/e2e/FLAKES.mdrecord CI evidence, deterministic negative controls, and validation provenance. Earlier ownership and planning defects trace to #536 and #502; the exact runtime-replacement timing of the newest CI counter failure was not captured.Validation on the merged runtime:
The final Debug executable was used for both the full modules and the 500-case soak. Negative controls cover owner adoption, activation publication, text planning/replacement, missing source observations, dropped runtime facts, and aliased diagnostic strings. Repository-locked Ruff formatting and lint checks pass.
Local validation is native macOS arm64 Debug. Linux CI must validate the pushed merge revision; no failed-case retries or timeout increases were added.