Repository navigation
fix(quest): prepare scoped member visibility repair - #4667
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…-proposal-20261104
This was referenced Nov 4, 2026
This branch was successfully deployed
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.
My Quest is available to every authenticated member in the current access contract, but the root
/crewpage setting still requires the retired communitycrewrole. That stale setting redirects ordinary members to the feed.This PR prepares a narrowly scoped operator repair and an OPEN manual-verification ledger row. It does not execute SQL or change production access by merging. The script requires exactly one root Space and exactly one published root
/crewrow whose visibility andupdated_atmatch the reviewed snapshot. It locks that row, changes only its visibility to native NULL (any authenticated member) and its timestamp, and verifies all other target columns remain unchanged. Missing, stale, ambiguous, already-cleared, wrong-status or wrong-visibility states refuse and roll back.No generic route gate, authentication, RLS, grant, role, migration or unrelated Space setting changes. The existing layout still protects other host-only/draft routes and signed-out access. This is an operator script, not a migration, and it has no automatic invocation.
Validation:
93c68083049621c341498a67f68135113bba3697, with only the two reviewed files in the PR diff; no current open PR file or row overlap.258956ee13d656f2b67959fc9a4c3c15690dc3e6a1740d0ced9d2127e1e43cecis unchanged from the reviewed candidate.Release sequence: the serialized root driver must verify exact-head CI and the real deployment, merge the exact script onto main, then perform a fresh read-only schema/root/row compare-and-set preflight and separately cleared single execution. Verify row/postconditions and actual authenticated member/guest behavior afterward.
NAV-QUEST-VISIBILITYremains OPEN until that proof exists. Never rerun against a changed or already-cleared row.Coordination: #4306 (comment). No auto-merge; root owns release and production verification.