From 36c81e7ce9baa12e3b134977cc18ddc9b82e0fa8 Mon Sep 17 00:00:00 2001
From: Daniel Gonzalez <danielhumgon@gmail.com>
Date: Tue, 3 Jun 2025 19:34:57 -0400
Subject: [PATCH] feat(cache): caching mutable data

---
 .../app-body/nfts-for-sale/index.js           | 27 ++++++++++++++++---
 src/hooks/state.js                            | 19 ++++++++++++-
 2 files changed, 41 insertions(+), 5 deletions(-)

diff --git a/src/components/app-body/nfts-for-sale/index.js b/src/components/app-body/nfts-for-sale/index.js
index 8578313..a9b5c78 100644
--- a/src/components/app-body/nfts-for-sale/index.js
+++ b/src/components/app-body/nfts-for-sale/index.js
@@ -98,10 +98,12 @@ function NftsForSale (props) {
 
         // Try to get token data.
         const tokenData = await appData.wallet.getTokenData(thisToken.tokenId)
-
+        console.log('tokenData', tokenData)
         if (tokenData) {
           // Set data to the token object , this can be used to display the token name in the token card component.
           thisToken.tokenData = tokenData
+          // Update the cache data  providing the tokenId to update and the data
+          appData.updateNFTCacheData(thisToken.tokenId, { tokenData })
         }
 
         // Mark token to prevent fetch token data again.
@@ -159,6 +161,8 @@ function NftsForSale (props) {
         if (iconUrl) {
           // Set the icon url to the token , this can be used to display the icon in the token card component.
           thisToken.icon = iconUrl
+          // Update the cache data  providing the tokenId to update and the data
+          appData.updateNFTCacheData(thisToken.tokenId, { tokenIcon: iconUrl })
         }
 
         // Mark token to prevent fetch token icon again.
@@ -169,7 +173,22 @@ function NftsForSale (props) {
     } catch (error) {
       setIconsAreLoaded(true)
     }
-  }, [fetchTokenInfo])
+  }, [fetchTokenInfo, appData])
+
+  // Load tokens data from the cache
+  const loadCacheData = useCallback(async (offers) => {
+    const cacheData = appData.nftForSaleData
+    console.log(`Token data from cache: ${JSON.stringify(cacheData, null, 2)}`)
+    // Map all offers and add cache data to the offers if it exists
+    for (let i = 0; i < offers.length; i++) {
+      const thisToken = offers[i]
+      const cacheToken = cacheData[thisToken.tokenId]
+      if (cacheToken) {
+        thisToken.tokenData = cacheToken.tokenData
+        thisToken.icon = cacheToken.tokenIcon
+      }
+    }
+  }, [appData])
 
   // Main function to load NFT offers
   const loadNftOffers = useCallback(async () => {
@@ -177,7 +196,7 @@ function NftsForSale (props) {
       setIsLoading(true)
       // Get tokens in offers
       const offers = await getNftOffers()
-      console.log('offers: ', offers)
+      await loadCacheData(offers)
       setOffers(offers)
       // Load tokens data
       await lazyLoadTokenData(offers)
@@ -188,7 +207,7 @@ function NftsForSale (props) {
       setIsLoading(false)
       console.error('Error loading NFT offers:', err)
     }
-  }, [lazyLoadTokenData, lazyLoadTokenIcons, getNftOffers])
+  }, [lazyLoadTokenData, lazyLoadTokenIcons, getNftOffers, loadCacheData])
 
   // Effect to load NFTs on component mount
   useEffect(() => {
diff --git a/src/hooks/state.js b/src/hooks/state.js
index 61be583..85c73b7 100644
--- a/src/hooks/state.js
+++ b/src/hooks/state.js
@@ -37,6 +37,9 @@ function useAppState () {
   const [hideSpinner, setHideSpinner] = useState(false)
   const [denyClose, setDenyClose] = useState(false)
 
+  // NFTs for sale state data
+  const [nftForSaleData, setNftForSaleData] = useState({})
+
   // The wallet state makes this a true progressive web app (PWA). As
   // balances, UTXOs, and tokens are retrieved, this state is updated.
   // properties are enumerated here for the purpose of documentation.
@@ -96,6 +99,17 @@ function useAppState () {
     }
   }
 
+  // Update NFT for sale tokens data
+  function updateNFTCacheData (tokenId, data) {
+    const allCacheData = nftForSaleData // Get all cache data
+    const cacheData = allCacheData[tokenId] || {} // Get cache data for the tokenId
+
+    const newCacheData = Object.assign({}, cacheData, data) // Merge the new data with the cache data
+    console.log('newCacheData: ', newCacheData)
+    allCacheData[tokenId] = newCacheData // Update the cache data
+    setNftForSaleData(allCacheData) // Update the state
+  }
+
   return {
     serverUrl,
     setServerUrl,
@@ -131,7 +145,10 @@ function useAppState () {
     dexLib,
     setDexLib,
     nostr,
-    setNostr
+    setNostr,
+    nftForSaleData,
+    setNftForSaleData,
+    updateNFTCacheData
   }
 }