Standardize the runtime graph - #885
Closed
anth-volk wants to merge 6 commits into
Closed
Conversation
This was referenced Sep 8, 2026
Contributor
Author
|
Superseded by #893 |
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.
Implements the OpenSpec change
standardize-runtime-graphas an independently owned child of #873.Status: blocked
This pull request is blocked on both PR #873 and discussion with @MaxGhenis to confirm the intended runtime-graph direction. It should not be merged until #873 is ready and that design direction has been agreed.
Recorded #873 base:
ee617ea672a7a81b4a86b3c03091a29f084682d4This branch pushes only to
origin/codex/standardize-runtime-graph. It does not modify Max Ghenis'smodel-artifact-graph-20260904branch.Implemented commit series
e713860ddocuments the typed-artifact baseline inherited from Add reusable typed model artifacts and transfer graphs #873, the Root identity: hash-pin every raw microdata input and reference its Chronicle registration (#848) #853 source-identity boundary, and the child-branch update procedure.5f09e9a9adds restricted, versioned Graph YAML; safe deterministic module composition; declared parameter binding; exact lowering; and optional generated Graph JSON.08abe4a0adds recursively immutable parameters, unchanged-row value revision, multi-population union, named products, and explicit earlier-weight-state references.6707637aadds source identity verification, declaration-controlled decoding, source/codec identity propagation, stored validation outcomes, required-success dependencies, and graph-bound schema-v4 run manifests.54a0bdb3adds saved-run population reconstruction, versioned local materializers, candidate indexes, the one-plan shared runner, and storage deduplication evidence.d08ebcebadds UK/US YAML and generated-JSON equivalence tests, package and wheel checks, interface documentation/locking, focused acceptance scenarios, and the changelog entry.Result
microcosm.graph.Graphis executed once per shared-runner invocation.Verification
openspec validate standardize-runtime-graph --strictuv run --package microcosm-graph ruff check .uv run --package microcosm-graph pytest packages/microcosm-graph/tests -quv run tools/ci_test_groups.py --verifyuv lock --checkOpenSpec progress is 36/38. The two unchecked tasks are conditional lifecycle work: rebase and revalidate if #873 changes, then rebase onto
mainand retarget this pull request after #873 merges. #873 remains open at the recorded commit, so neither condition currently applies.