feat(artifacts): add lifecycle planning and provider seam - #464
Open
jmagar wants to merge 3 commits into
Open
Conversation
jmagar
force-pushed
the
codex/w19-artifact-lifecycle-20260819
branch
from
August 20, 2026 06:07
1c1d7af to
1fb7302
Compare
Collaborator
Author
|
Phase 2 current-main revalidation is complete on head
Local verification after rebasing onto current main also passed: full No reviews or review threads are outstanding. No further branch mutation is needed for this Phase 2 slice. |
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.
Summary\n\n- add deterministic Artifact revision diffs and editable-workspace snapshots\n- add explicit, read-only update plans and a transport-neutral ArtifactProvider seam\n- add verified local provider acquisition with per-file/package byte budgets\n- add expected-base compare-and-swap head transitions and safe cross-platform atomic JSON replacement\n- preserve exact immutable revision reuse for repeated/reverted content\n- update Artifact spec, contract, implementation plan, and progress evidence\n\n## Safety / behavior\n\n- provider acquisition verifies path, size, SHA-256, component count, and local materialization budgets\n- update planning never mutates head, workspace, lineage, or stored bytes\n- stale head transitions fail with head_changed\n- snapshots reuse existing immutable revisions instead of rewriting historical metadata\n- no retired Stash/Marketplace/Fleet/Deploy product/protocol aliases introduced\n\n## Verification\n\n- hardened Artifact tests: 27/27 passed\n- full labby-runtime all-features tests passed\n- cargo clippy -p labby-runtime --all-features --all-targets --locked -- -D warnings\n- RUSTFLAGS=-D warnings cargo check -p labby-runtime --all-features --all-targets --locked\n- cargo deny check: advisories, bans, licenses, sources all passed\n- cargo fmt --all -- --check\n- git diff --check\n- labby docs check: 17 generated docs fresh\n- documentation link check: 347 local links verified\n- final detached post-doc verifier exited 0 with FINAL_OK\n\nPhase 1 landed in #462 at 8723910.