Make Team Library repositories team-unique - #164
Merged
Conversation
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.
Outcome
Completes the provider-aware context integration without allowing two teams under one GitHub owner to adopt or delete the same managed Team Library repository, and prepares API rc62.
Work authority
stagingat465afa5016bbbfcfa95ff606ada72eeb89104f3ce726c6f0921ed4c6ee6bca9ae3796eb18e34f761Contributor mode (select one):
Plan
Audit the cross-repository context architecture, repair the only ownership collision, verify current staging, publish rc62, and compose it with SDK rc66, Agent rc40, and TreeDX rc13.
Changes and commits
e726c6f0921ed4c6ee6bca9ae3796eb18e34f761— deterministic team-unique repository identities, exact recorded-name reconciliation/deletion, legacy compatibility, tests, and rc62 metadata.Verification
Fresh
npm ciagainst the updated SDK rc66 lock.npm run verify:direct: build passed; 76 files / 222 tests passed.Tests prove deterministic uniqueness, legacy compatibility, and rejection of another team's repository identity.
Platform remains configuration/documentation-only; managed libraries remain API/TreeDX runtime resources and are not source workset entries.
I ran the narrowest relevant package verification and documented any checks that could not be run.
Risk and rollback
Existing system projects retain the recorded legacy
team-libraryidentity. New teams receiveteam-library-<12 hex>derived from the stable team ID. A mismatched deletion identity fails closed. Revert this commit and retain API rc61 if release gates fail; no database migration is introduced.Completion summary
The context architecture is compatible with multi-team production and ready for protected staging verification and rc62 publication. Closes #163.
AGPL committer authorization
The provider-authenticated author is the approved account
adrianwebb; no identity override or token is included in the workspace.Submission checklist