[Tech] update staleTime for get quest and get user playstreak (#35)

* update staleTime

* set stale time for get playstreak to 60s, rm dateTimeCurrentSessionStartedInMsSinceEpoch

* rm unused syncPlaySession

* rm refetchOnMount: false

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hyperplay/quests-ui",
-  "version": "0.0.36",
+  "version": "0.0.37",
   "description": "",
   "main": "index.js",
   "scripts": {

src/components/QuestDetailsWrapper/QuestDetailsWrapper.stories.tsx

@@ -92,7 +92,6 @@ Rise among Craft World's top ranks. 🚀 Join now and make your mark before the
   getExternalTaskCredits: async () => {
     return '100'
   },
-  syncPlaySession: async () => {},
   logInfo: () => {},
   openDiscordLink: () => {},
   getDepositContracts: async () => {

src/components/QuestDetailsWrapper/index.tsx

@@ -17,7 +17,6 @@ import {
   Reward,
   RewardClaimSignature,
   ConfirmClaimParams,
-  Runner,
   DepositContract
 } from '@hyperplay/utils'
 import { mintReward } from '../../helpers/mintReward'
@@ -76,7 +75,6 @@ export interface QuestDetailsWrapperProps {
   syncPlayStreakWithExternalSource: (questId: number) => Promise<unknown>
   resyncExternalTask: (rewardId: string) => Promise<void>
   getExternalTaskCredits: (rewardId: string) => Promise<string>
-  syncPlaySession: (appName: string, runner: Runner) => Promise<void>
   logInfo: (message: string) => void
   openDiscordLink: () => void
   getDepositContracts: (questId: number) => Promise<DepositContract[]>
@@ -622,6 +620,9 @@ export function QuestDetailsWrapper({
       ['ERC1155', 'ERC721', 'ERC20'].includes(reward.reward_type)
     )
 
+    const dateTimeCurrentSessionStartedInMsSinceEpoch =
+      questPlayStreakResult?.data.dataUpdatedAt ?? Date.now()
+
     const questDetailsProps: QuestDetailsProps = {
       className,
       alertProps,
@@ -642,8 +643,8 @@ export function QuestDetailsWrapper({
         playStreak: getPlaystreakArgsFromQuestData({
           questMeta,
           questPlayStreakData,
-          useModuleInitTimeForSessionStartTime: isSignedIn,
-          rightSection: streakRightSection
+          rightSection: streakRightSection,
+          dateTimeCurrentSessionStartedInMsSinceEpoch
         })
       },
       rewards: questRewards ?? [],

src/helpers/getPlaystreakArgsFromQuestData.ts

@@ -2,24 +2,17 @@ import { PlayStreakEligibility } from '@hyperplay/ui'
 import { Quest, UserPlayStreak } from '@hyperplay/utils'
 import { ReactNode } from 'react'
 
-// this is initialized when the overlay is started for this game
-let dateTimeCurrentSessionStartedInMsSinceEpoch = Date.now()
-
 export function getPlaystreakArgsFromQuestData({
   questMeta,
   questPlayStreakData,
-  useModuleInitTimeForSessionStartTime,
-  rightSection
+  rightSection,
+  dateTimeCurrentSessionStartedInMsSinceEpoch
 }: {
   questMeta: Quest
   questPlayStreakData: UserPlayStreak | undefined | null
-  useModuleInitTimeForSessionStartTime?: boolean
   rightSection?: ReactNode
+  dateTimeCurrentSessionStartedInMsSinceEpoch: number
 }): PlayStreakEligibility {
-  let sessionStartedTime = undefined
-  if (useModuleInitTimeForSessionStartTime) {
-    sessionStartedTime = dateTimeCurrentSessionStartedInMsSinceEpoch
-  }
   return {
     rightSection,
     requiredStreakInDays:
@@ -32,10 +25,6 @@ export function getPlaystreakArgsFromQuestData({
       new Date().toISOString(),
     accumulatedPlaytimeTodayInSeconds:
       questPlayStreakData?.accumulated_playtime_today_in_seconds ?? 0,
-    dateTimeCurrentSessionStartedInMsSinceEpoch: sessionStartedTime
+    dateTimeCurrentSessionStartedInMsSinceEpoch
   }
 }
-
-export function resetSessionStartedTime() {
-  dateTimeCurrentSessionStartedInMsSinceEpoch = Date.now()
-}

src/hooks/useGetQuest.ts

@@ -18,7 +18,9 @@ export function getQuestQueryOptions(
       return response
     },
     refetchOnWindowFocus: false,
-    enabled: questId !== null
+    enabled: questId !== null,
+    // choosing 60s over Infinity here to keep the num of claims left value fresh
+    staleTime: 60 * 1000
   })
 }
 

src/hooks/useGetUserPlayStreak.ts

@@ -9,6 +9,7 @@ export function getUserPlaystreakQueryOptions(
   const queryKey = getGetUserPlayStreakQueryKey(questId)
   return queryOptions({
     queryKey,
+    staleTime: 60 * 1000,
     queryFn: async () => {
       if (questId === null) {
         return null

src/state/QuestPlayStreakSyncState.ts

@@ -1,11 +1,7 @@
 import { Quest, Runner, UserPlayStreak } from '@hyperplay/utils'
 import { makeAutoObservable } from 'mobx'
 import { QueryClient } from '@tanstack/query-core'
-import { resetSessionStartedTime } from '@/helpers/getPlaystreakArgsFromQuestData'
-import {
-  getGetUserPlayStreakQueryKey,
-  getSyncPlaysessionQueryKey
-} from '@/helpers/getQueryKeys'
+import { getSyncPlaysessionQueryKey } from '@/helpers/getQueryKeys'
 import { getQuestQueryOptions } from '@/hooks/useGetQuest'
 import { getUserPlaystreakQueryOptions } from '@/hooks/useGetUserPlayStreak'
 
@@ -101,14 +97,9 @@ class QuestPlayStreakSyncState {
                 projectId,
                 questMeta?.quest_external_game?.runner ?? 'hyperplay'
               )
-              resetSessionStartedTime()
               // all quest user playstreak data needs to be refetched after playsession sync
-              for (const questToInvalidate of quests) {
-                const queryKey = getGetUserPlayStreakQueryKey(
-                  questToInvalidate.id
-                )
-                this.appQueryClient?.invalidateQueries({ queryKey })
-              }
+              const queryKey = ['getUserPlayStreak']
+              this.appQueryClient?.invalidateQueries({ queryKey })
               return null
             }
           })