Repository navigation
Record Store acceptance and propose legacy Setup owner resolution - #91
Merged
Merged
Conversation
Open
42 tasks
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.
Session deletion can remove the history needed to determine whether legacy Setup already ran. Document a complete owner-resolution candidate for S1 review before marker/schema/init implementation: retain the Store-lifetime epoch and exact stored directory key; distinguish positive historical evidence from unproven history; and let the owner authorize a new attempt or explicitly continue without dispatching Setup. Specify the legacy root-creation compatibility exception, CLI/API proposal, revision fencing, request binding, retry-after-deletion behavior and adversarial acceptance cases.
This publishes a candidate, not an accepted policy or implemented capability. S1 remains unchecked, and recording or mechanically merging this document does not authorize the proposed behavior. The full original Setup contract remains required until the complete migration/support decision is reviewed. Native Linux support and DirectoryAdded review gates remain unchanged.
Also retain merged #90's source-bound Store acceptance receipt and reconcile its implementation/delivery notes. The exact named Store/runtime regressions execute and pass on Linux/macOS/Windows; the tested tree equals accepted main. Linux/macOS pass 5,508 cases (21/23 skips), Windows passes 3,785 selected cases (647 skips) plus metadata/migration/AppContainer checks, and all 16 source-revision checks pass. The Windows messaging leaked-process warning is recorded with its cause unverified. These are accepted #90 results, not claims that this new documentation tree ran the executable suites.
Verification:
The full plugins/hooks, daily-coding and binary-editor goal remains incomplete. This change preserves the required policy decisions and provides concrete review material for the next implementation step.
Final documentation verification: OpenSpec and Changes checks passed; six code/build jobs were intentionally skipped by the documentation-only filter. The actual OpenSpec checkout is
d6d3e50d8810e5df0d88da7368048527b46bd69b, treed75eff79b944c0825cf601d912431560dddf7ee9, equal to the reviewed head and final prospective merge. The hosted log confirms 114/0 strict validation and lint 0 errors/25 warnings. Final documentation review found no blocking issue. Publishing this candidate leaves S1 unchecked and does not select the proposed compatibility policy or authorize dependent implementation.