From d0d00ba9f8ff4ba0f99a76f6839c417b88bf564d Mon Sep 17 00:00:00 2001 From: DS Boyce <260543580+ds-boyce@users.noreply.github.com> Date: Wed, 8 Apr 2026 15:54:43 -0700 Subject: [PATCH] Address lint warnings (#10188) --- src/App.native.tsx | 4 +- src/App.web.tsx | 4 +- src/ageAssurance/data.tsx | 24 +-- src/ageAssurance/index.tsx | 2 +- src/ageAssurance/state.ts | 1 + src/screens/Settings/Settings.tsx | 20 ++- .../Signup/StepCaptcha/CaptchaWebView.web.tsx | 4 +- src/screens/Signup/StepCaptcha/index.tsx | 5 +- src/screens/Signup/StepInfo/index.tsx | 42 ++--- src/screens/Signup/index.tsx | 25 ++- src/screens/Signup/state.ts | 28 +-- src/state/birthdate.ts | 2 +- .../queries/messages/actor-declaration.ts | 2 +- src/state/session/agent.ts | 163 +++++++++--------- src/state/session/index.tsx | 17 +- 15 files changed, 170 insertions(+), 173 deletions(-) diff --git a/src/App.native.tsx b/src/App.native.tsx index a6ec93d313..af93a75c84 100644 --- a/src/App.native.tsx +++ b/src/App.native.tsx @@ -215,11 +215,11 @@ function InnerApp() { } function App() { - const [isReady, setReady] = useState(false) + const [isReady, setIsReady] = useState(false) useEffect(() => { void Promise.all([initPersistedState(), Geo.resolve(), setupDeviceId]).then( - () => setReady(true), + () => setIsReady(true), ) }, []) diff --git a/src/App.web.tsx b/src/App.web.tsx index c391d69925..a840437a1b 100644 --- a/src/App.web.tsx +++ b/src/App.web.tsx @@ -194,11 +194,11 @@ function InnerApp() { } function App() { - const [isReady, setReady] = useState(false) + const [isReady, setIsReady] = useState(false) useEffect(() => { void Promise.all([initPersistedState(), Geo.resolve(), setupDeviceId]).then( - () => setReady(true), + () => setIsReady(true), ) }, []) diff --git a/src/ageAssurance/data.tsx b/src/ageAssurance/data.tsx index d2abfb5f88..cfea297f67 100644 --- a/src/ageAssurance/data.tsx +++ b/src/ageAssurance/data.tsx @@ -105,19 +105,18 @@ export function getConfigFromCache(): ) } let configPrefetchPromise: Promise | undefined -export async function prefetchConfig() { +export function prefetchConfig() { if (configPrefetchPromise) { logger.debug(`prefetchAgeAssuranceConfig: already in progress`) return } - configPrefetchPromise = new Promise(async resolve => { + configPrefetchPromise = (async () => { await cacheHydrationPromise const cached = getConfigFromCache() if (cached) { logger.debug(`prefetchAgeAssuranceConfig: using cache`) - resolve() } else { try { logger.debug(`prefetchAgeAssuranceConfig: resolving...`) @@ -126,15 +125,14 @@ export async function prefetchConfig() { configQueryKey, res, ) - } catch (e: any) { + } catch (err) { + const e = err as Error logger.warn(`prefetchAgeAssuranceConfig: failed`, { safeMessage: e.message, }) - } finally { - resolve() } } - }) + })() } export async function refetchConfig() { logger.debug(`refetchConfig: fetching...`) @@ -228,7 +226,8 @@ export async function prefetchServerState({agent}: {agent: AtpAgent}) { logger.debug(`prefetchServerState: resolving...`) const res = await networkRetry(3, () => getServerState({agent})) qc.setQueryData(qk, res) - } catch (e: any) { + } catch (err) { + const e = err as Error logger.warn(`prefetchServerState: failed`, { safeMessage: e.message, }) @@ -248,7 +247,7 @@ export async function refetchServerState({agent}: {agent: AtpAgent}) { export function usePatchServerState() { const {currentAccount} = useSession() return useCallback( - async (next: AppBskyAgeassuranceDefs.State) => { + (next: AppBskyAgeassuranceDefs.State) => { if (!currentAccount) return const did = currentAccount.did const prev = getServerStateFromCache({did}) @@ -313,7 +312,7 @@ export function useServerStateQuery() { // only refetch when needed if (isAssured || !isAArequired) return - refetch() + void refetch() }) }, [did, refetch, isAssured]) @@ -409,7 +408,8 @@ export async function prefetchOtherRequiredData({agent}: {agent: AtpAgent}) { logger.debug(`prefetchOtherRequiredData: resolving...`) const res = await networkRetry(3, () => getOtherRequiredData({agent})) qc.setQueryData(qk, res) - } catch (e: any) { + } catch (err) { + const e = err as Error logger.warn(`prefetchOtherRequiredData: failed`, { safeMessage: e.message, }) @@ -418,7 +418,7 @@ export async function prefetchOtherRequiredData({agent}: {agent: AtpAgent}) { export function usePatchOtherRequiredData() { const {currentAccount} = useSession() return useCallback( - async (next: OtherRequiredData) => { + (next: OtherRequiredData) => { if (!currentAccount) return const did = currentAccount.did const prev = getOtherRequiredDataFromCache({did}) diff --git a/src/ageAssurance/index.tsx b/src/ageAssurance/index.tsx index 9bffce7feb..80b44f1dcf 100644 --- a/src/ageAssurance/index.tsx +++ b/src/ageAssurance/index.tsx @@ -85,7 +85,7 @@ function InnerProvider({children}: {children: React.ReactNode}) { const handleAccessUpdate = useCallback( (s: AgeAssuranceState) => { - getAndRegisterPushToken({ + void getAndRegisterPushToken({ isAgeRestricted: s.access !== AgeAssuranceAccess.Full, }) }, diff --git a/src/ageAssurance/state.ts b/src/ageAssurance/state.ts index f4d1e41e3a..6499a705e6 100644 --- a/src/ageAssurance/state.ts +++ b/src/ageAssurance/state.ts @@ -99,6 +99,7 @@ export function useOnAgeAssuranceAccessUpdate( useEffect(() => { if (prevAccess !== state.access) { + // eslint-disable-next-line react-hooks/set-state-in-effect setPrevAccess(state.access) cb(state) logger.debug(`useOnAgeAssuranceAccessUpdate`, {state}) diff --git a/src/screens/Settings/Settings.tsx b/src/screens/Settings/Settings.tsx index 6e02297fc2..223a003638 100644 --- a/src/screens/Settings/Settings.tsx +++ b/src/screens/Settings/Settings.tsx @@ -161,7 +161,9 @@ export function SettingsScreen({}: Props) { p => p.did === account.did, )} pendingDid={pendingDid} - onPressSwitchAccount={onPressSwitchAccount} + onPressSwitchAccount={(account, logContext) => + void onPressSwitchAccount(account, logContext) + } /> ))} @@ -245,7 +247,7 @@ export function SettingsScreen({}: Props) { Linking.openURL(HELP_DESK_URL)} + onPress={() => void Linking.openURL(HELP_DESK_URL)} label={_(msg`Help`)} accessibilityHint={_(msg`Opens helpdesk in browser`)}> @@ -392,7 +394,7 @@ function DevOptions() { } = useApplyPullRequestOTAUpdate() const [actyNotifNudged, setActyNotifNudged] = useActivitySubscriptionsNudged() - const resetOnboarding = async () => { + const resetOnboarding = () => { navigation.navigate('Home') onboardingDispatch({type: 'start'}) Toast.show(_(msg`Onboarding reset`)) @@ -407,7 +409,7 @@ function DevOptions() { const lastEmailConfirm = new Date() // wind back 3 days lastEmailConfirm.setDate(lastEmailConfirm.getDate() - 3) - persisted.write('reminders', { + void persisted.write('reminders', { ...persisted.get('reminders'), lastEmailConfirm: lastEmailConfirm.toISOString(), }) @@ -431,7 +433,7 @@ function DevOptions() { style: 'default', text: 'Apply', onPress: (channel?: string) => { - tryApplyUpdate(channel ?? '') + void tryApplyUpdate(channel ?? '') }, }, ], @@ -473,7 +475,7 @@ function DevOptions() { resetOnboarding()} + onPress={() => void resetOnboarding()} label={_(msg`Reset onboarding state`)}> Reset onboarding state @@ -496,7 +498,7 @@ function DevOptions() { )} clearAllStorage()} + onPress={() => void clearAllStorage()} label={_(msg`Clear all storage data`)}> Clear all storage data (restart after this) @@ -513,7 +515,7 @@ function DevOptions() { ) : null} {IS_NATIVE && isCurrentlyRunningPullRequestDeployment ? ( void revertToEmbedded()} label={_(msg`Unapply Pull Request`)}> Unapply Pull Request {currentChannel} @@ -543,7 +545,7 @@ function DevOptions() {