feat(wire): S5/#110 — §4.2 run manifest builder (commit marker, byte-stable)#117
Merged
Merged
Conversation
build_run_manifest emits the run's single commit marker byte-stable against the §10 fixture: pinned key order + indent=2; closed shard records (name/target/time_id/sha256 — a 'file_id' key raises, the exact field §3.2 corrected away) and closed sidecar record (E1's home). expected_cell_count is per-shard N per the §3.2 ruling. Everything declared by the caller; nothing inspected or minted. Note: gained a real dependency on S2 (CONTRACT_VERSION from wire/header.py) — tracking checklist ordering amended in the PR. Part of epic #105. Closes #110. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
1 participant
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.
Story S5 of epic #105 (taken out of order — parallel-eligible; note below).
What
unfao/wire/run_manifest.py:build_run_manifest(...)→ the §4.2 JSON bytes. Pinned key order +indent=2(fixture serialization); closed shard records — an extrafile_idkey raises by name (the exact field the §3 audit corrected away); closed sidecar record (Erratum E1's home of the sidecar hash);expected_cell_count= per-shard N (§3.2 ruling).AC (all met)
fixture_run_0__manifest.json.Dependency amendment: planned as S1-only, but
CONTRACT_VERSIONcorrectly lives in S2'swire/header.py, so S5 depends on S2 (merged). #114's ordering note stands corrected by this PR.Closes #110. Epic #105 · Tracking #114.
🤖 Generated with Claude Code