define summoning message to prepare for equip->circles and add error …

…handling on api calls with explicit throws for better ui/ux flows

src/inventory/maliks-majik.ts

@@ -36,9 +36,11 @@ const equip: HoF = async () => {
         // if (!isMobile()) just send apply request to server without signWithId NFC interaction
         const address = await getStorage<string>(ID_PLAYER_SLOT);
         console.log('address to get verified: ', address);
-        const result = address
-            ? await signWithId(`summon:${address}`)
-            : await signWithId(`summon:${(await getSpellBook()).address}`);
+        const summonSign = address
+            ? `summon:${address}`
+            : `summon:${(await getSpellBook()).address}`;
+        const result = await signWithId(summonSign);
+        console.log('summon sign', summonSign, result);
 
         console.log('verified address signature: ', result);
         if (!result) throw Error('Majik not found');

src/utils/api.ts

@@ -12,6 +12,7 @@ import {
     saveStorage,
 } from 'utils/config';
 import { getSpellBook } from 'utils/zkpid';
+import { debug } from './logging';
 
 // TODO persist cache to local storage for better offline use once internet connection lost?
 // https://www.apollographql.com/docs/react/caching/advanced-topics#persisting-the-cache
@@ -66,7 +67,7 @@ export const qu =
         console.log('api:qu:vars', variables);
         console.log('api:qu:verification ', `'${cleaned}'`, '---', majikMsg);
 
-        return query
+        const response = query
             ? getGqlClient().query({
                   ...baseRequest,
                   //   fetchPolicy: 'cache-first', // TODO add useCache: boolean to switch between query vs readQuery?
@@ -81,6 +82,24 @@ export const qu =
                       ${cleaned}
                   `,
               });
+
+        if (response.error) {
+            debug(response.error, {
+                user: { id: spellbook.address },
+                tags: {
+                    api: true,
+                },
+                extra: {
+                    query,
+                    mutation,
+                    variables,
+                },
+            });
+
+            throw Error(response.err);
+        } else {
+            return response;
+        }
     };
 
 export const QU_GET_PLAYER_CONFIG = `
@@ -231,7 +250,7 @@ export const getHomeConfig = async (pid?: string): Promise<HomeConfigMap> => {
             return {}; // no player. return nil
         })
         .catch((error) => {
-            console.error('Home:config:get: ERR ', error);
+            console.log('Home:config:get: ERR ', error);
             return {}; // no player. return nil
         });
 };

src/utils/oauth.ts

@@ -129,6 +129,7 @@ export const getProviderId = async ({ playerId, provider }: obj): Promise<string
         id &&
             playerId === (await getStorage<string>(ID_PLAYER_SLOT)) &&
             (await saveStorage(ID_PROVIDER_IDS_SLOT, { [provider]: id }, true));
+        // asssume local save always succeeds
         return id;
     } catch (e) {
         console.log('util:oauth:getProviderId:ERROR', e);

src/utils/serviceWorkerRegistration.js

@@ -96,7 +96,7 @@ function registerValidSW(swUrl, config) {
             };
         })
         .catch((error) => {
-            console.error('Error during service worker registration:', error);
+            console.log('Error during service worker registration:', error);
         });
 }
 
@@ -137,7 +137,7 @@ export function unregister() {
                 registration.unregister();
             })
             .catch((error) => {
-                console.error(error.message);
+                console.log(error.message);
             });
     }
 }

src/utils/zkpid.ts

@@ -77,7 +77,7 @@ export const saveId = async (idType: string, id: Identity): Promise<void> => {
             // console.log("anon id saved to storage!", idType, id)
         }
     } catch (error) {
-        console.error('Store Err: ', error);
+        console.log('Store Err: ', error);
     }
 };