feat: prefix federated resource names with gateway names - #6780
Draft
gandhipratik203 wants to merge 1 commit into
Draft
feat: prefix federated resource names with gateway names#6780gandhipratik203 wants to merge 1 commit into
gandhipratik203 wants to merge 1 commit into
Conversation
Refs #6621. Preserve resource URIs and local names; deterministic same-URI routing remains outside this change. Signed-off-by: Pratik Gandhi <gandhipratik203@gmail.com>
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
Scope and Limitations
References #6621; does not close it. This implements the naming behavior in AC#1 and AC#2. AC#3 remains unresolved: prefixing names cannot disambiguate URI-keyed reads. Existing server-scoped routing and ambiguity errors are unchanged. Partial issue scope still needs agreement from the issue author.
Names are not guaranteed unique: gateway slugs can collide and 255-character truncation can consume the resource portion. Export fidelity is limited by validation and column bounds; invalid or oversized bases may fall through to another candidate. Provenance fields are not restored on import. Separator changes do not rederive stored bases. Downgrade discards federated manual overrides.
Draft Status
Not ready for merge. Live gateway and PostgreSQL validation are deferred by agreement; the production-stack and live RBAC gates have not been completed.
The full unit run passed with 23,230 tests; diff coverage was 91%. Subsequent focused checks passed after type-annotation fixes. Pre-commit, Ruff, Interrogate, Pylint, package verification, and the secret scan passed. Scoped mypy and Pyrefly checks passed for the new files. Broad type checks and full Bandit still report findings outside this change.