fix: three consumer reports, measured against what is actually here - #24
Merged
Conversation
ai_stp#418 -- and it was never about empty setups. provider_plan_digest was null after every operation of every kind: it was read out of the plan object, which never carries it, since the digest is taken over the plan and travels beside it. It comes from the checked --plan-digest value now. ai_stp#419 -- a capture always refused a link, but refused while copying: the slot was created, files were written into it, and the walk then stopped, leaving a partial operation for a shape that was knowable for free. The owned paths are read before planning and again before any capture, and every unsupported entry is named at once. Nothing is followed. ai_stp#422 -- already dissolved by identity becoming the owned projection, and now asserted: a restore reaches the exact identity the slot recorded even when the product has written its own runtime files between the two. Rendered from NDDev-it-com/setup-systems@ca229cb. Claude-Session: https://claude.ai/code/session_012UZkeU7yQhyZqKAaDaQEjA
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.
Rendered from
NDDev-it-com/setup-systems@ca229cb. Every open issue describing this provider, checked against the current implementation rather than the version it was reported on.ai_stp#418— never about empty setupsprovider_plan_digestwas null after every operation of every kind: it was read out of the plan object, which never carries it — the digest is taken over the plan and travels beside it in the envelope. It comes from the checked--plan-digestvalue now.It went unnoticed for four releases because the field was absent from
status, and an absent field is skipped by the consumer. A published null is one it compares and refuses — so publishing what we persist is what made this visible.ai_stp#419— refused before anything movesA capture always refused a link, but while copying: slot created, files written, then it stopped — a partial operation and control artifacts for a shape knowable for free. Reported from a Windows target whose owned
config/skillsheld four Junctions.Owned paths are read before planning and again before any capture, and every unsupported entry is named at once. Nothing is followed. A link outside every owned namespace still goes through untouched.
ai_stp#422— already dissolvedSince identity became the owned projection, a real product run writing sessions, caches and logs does not move
target_identity_digest. Now asserted: a restore reaches the exact recorded identity with runtime files written between the two, and leaves them alone.Gate
351 tests,
clippy -D warnings, conformance 23/23 on all seven, three-OS matrix.https://claude.ai/code/session_012UZkeU7yQhyZqKAaDaQEjA