fix(training-agent): resolve sync_governance accounts across open-mode session partitions - #7100
Open
aleksUIX wants to merge 1 commit into
Open
Conversation
…e session partitions Controller seeding writes into open:default while sync_governance keys off the nested account brand domain. Scan every principal-owned account map so sales_dooh can register governance without a prior sync_accounts call.
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.
Fixes #7080.
sales_doohnever callssync_accounts. Controller seeding writes the sandbox account into theopen:defaultpartition (seed_accountkeys off empty args).sync_governancethen keys offaccounts[0].account.brand.domain(open:acmeoutdoor.example) and looks only at that map, so local macOS open-mode runs getACCOUNT_NOT_FOUND. Training-mode CI hides the split because both sides collapse totraining:${userId}:${moduleId}.This matches the existing
accountMapsForPrincipalscan used bygetAccountNotificationSubscribersandresolveAccountIdForRef. No protocol or schema change.Test plan
npx vitest run --config server/vitest.config.ts tests/unit/account-handlers.test.ts(41 passed)sync_governanceafterseed_accountinto a different session partition returnsstatus: "synced"