diff options
Diffstat (limited to 'src/state')
-rw-r--r-- | src/state/persisted/schema.ts | 21 | ||||
-rw-r--r-- | src/state/session/index.tsx | 20 |
2 files changed, 31 insertions, 10 deletions
diff --git a/src/state/persisted/schema.ts b/src/state/persisted/schema.ts index f090365a3..43f08e12e 100644 --- a/src/state/persisted/schema.ts +++ b/src/state/persisted/schema.ts @@ -4,7 +4,10 @@ import {deviceLocales, prefersReducedMotion} from '#/platform/detection' const externalEmbedOptions = ['show', 'hide'] as const -// only data needed for rendering account page +/** + * A account persisted to storage. Stored in the `accounts[]` array. Contains + * base account info and access tokens. + */ const accountSchema = z.object({ service: z.string(), did: z.string(), @@ -19,12 +22,26 @@ const accountSchema = z.object({ }) export type PersistedAccount = z.infer<typeof accountSchema> +/** + * The current account. Stored in the `currentAccount` field. + * + * In previous versions, this included tokens and other info. Now, it's used + * only to reference the `did` field, and all other fields are marked as + * optional. They should be considered deprecated and not used, but are kept + * here for backwards compat. + */ +const currentAccountSchema = accountSchema.extend({ + service: z.string().optional(), + handle: z.string().optional(), +}) +export type PersistedCurrentAccount = z.infer<typeof currentAccountSchema> + export const schema = z.object({ colorMode: z.enum(['system', 'light', 'dark']), darkTheme: z.enum(['dim', 'dark']).optional(), session: z.object({ accounts: z.array(accountSchema), - currentAccount: accountSchema.optional(), + currentAccount: currentAccountSchema.optional(), }), reminders: z.object({ lastEmailConfirm: z.string().optional(), diff --git a/src/state/session/index.tsx b/src/state/session/index.tsx index e45aa031f..37b108766 100644 --- a/src/state/session/index.tsx +++ b/src/state/session/index.tsx @@ -618,20 +618,24 @@ export function Provider({children}: React.PropsWithChildren<{}>) { logger.debug(`session: persisted onUpdate`, {}) - if (session.currentAccount && session.currentAccount.refreshJwt) { - if (session.currentAccount?.did !== state.currentAccount?.did) { + const selectedAccount = session.accounts.find( + a => a.did === session.currentAccount?.did, + ) + + if (selectedAccount && selectedAccount.refreshJwt) { + if (selectedAccount.did !== state.currentAccount?.did) { logger.debug(`session: persisted onUpdate, switching accounts`, { from: { did: state.currentAccount?.did, handle: state.currentAccount?.handle, }, to: { - did: session.currentAccount.did, - handle: session.currentAccount.handle, + did: selectedAccount.did, + handle: selectedAccount.handle, }, }) - initSession(session.currentAccount) + initSession(selectedAccount) } else { logger.debug(`session: persisted onUpdate, updating session`, {}) @@ -641,9 +645,9 @@ export function Provider({children}: React.PropsWithChildren<{}>) { * already persisted, and we'll get a loop between tabs. */ // @ts-ignore we checked for `refreshJwt` above - __globalAgent.session = session.currentAccount + __globalAgent.session = selectedAccount } - } else if (!session.currentAccount && state.currentAccount) { + } else if (!selectedAccount && state.currentAccount) { logger.debug( `session: persisted onUpdate, logging out`, {}, @@ -662,7 +666,7 @@ export function Provider({children}: React.PropsWithChildren<{}>) { setState(s => ({ ...s, accounts: session.accounts, - currentAccount: session.currentAccount, + currentAccount: selectedAccount, })) }) }, [state, setState, clearCurrentAccount, initSession]) |