petOnNeopetsDotCom GQL improvements

We now support returning `null` from `petAppearance` when a pet genuinely has no customization data.

We also deprecate some old fields, and update our own call site to match.

src/app/HomePage.js

@@ -217,14 +217,16 @@ function SubmitPetForm() {
     gql`
       query SubmitPetForm($petName: String!) {
         petOnNeopetsDotCom(petName: $petName) {
-          color {
+          petAppearance {
-            id
+            color {
+              id
+            }
+            species {
+              id
+            }
+            pose
           }
-          species {
+          wornItems {
-            id
-          }
-          pose
-          items {
             id
           }
         }
@@ -235,14 +237,25 @@ function SubmitPetForm() {
       onCompleted: (data) => {
         if (!data) return;
 
-        const { species, color, pose, items } = data.petOnNeopetsDotCom;
+        const { petAppearance, wornItems } = data.petOnNeopetsDotCom;
+        if (petAppearance == null) {
+          toast({
+            title: "This pet exists, but is in a glitchy state on Neopets.com.",
+            description:
+              "Hopefully it gets fixed soon! If this doesn't sound right to you, contact us and let us know!",
+            status: "error",
+          });
+          return;
+        }
+
+        const { species, color, pose } = petAppearance;
         const params = new URLSearchParams({
           name: petName,
           species: species.id,
           color: color.id,
           pose,
         });
-        for (const item of items) {
+        for (const item of wornItems) {
           params.append("objects[]", item.id);
         }
         pushHistory(`/outfits/new?${params}`);

src/server/types/Pet.js

@@ -8,17 +8,13 @@ const typeDefs = gql`
   type Pet {
     id: ID!
     name: String!
-    petAppearance: PetAppearance!
     wornItems: [Item!]!
 
-    "to be deprecated? can use petAppearance? 🤔"
+    """
-    species: Species!
+    The pet's appearance. Can be null if the pet is in an invalid buggy state
-    "to be deprecated? can use petAppearance? 🤔"
+    that Neopets.com's own customization system doesn't support yet.
-    color: Color!
+    """
-    "to be deprecated? can use petAppearance? 🤔"
+    petAppearance: PetAppearance
-    pose: Pose!
-    "deprecated alias for wornItems"
-    items: [Item!]!
   }
 
   extend type Query {
@@ -29,19 +25,15 @@ const typeDefs = gql`
 const resolvers = {
   Pet: {
     id: ({ name }) => name,
-    species: ({ customPetData }) => ({
-      id: String(customPetData.custom_pet.species_id),
-    }),
-    color: ({ customPetData }) => ({
-      id: String(customPetData.custom_pet.color_id),
-    }),
-    pose: ({ customPetData, petMetaData }) =>
-      getPoseFromPetData(petMetaData, customPetData),
     petAppearance: async (
       { name, customPetData, petMetaData },
       _,
       { petTypeBySpeciesAndColorLoader, petStatesForPetTypeLoader }
     ) => {
+      if (customPetData == null) {
+        return null;
+      }
+
       const petType = await petTypeBySpeciesAndColorLoader.load({
         speciesId: customPetData.custom_pet.species_id,
         colorId: customPetData.custom_pet.color_id,
@@ -99,15 +91,19 @@ const resolvers = {
           `sorry! Try using the Modeling Hub on Classic DTI to upload it first?`
       );
     },
-    wornItems: ({ customPetData }) =>
+    wornItems: ({ customPetData }) => {
-      Object.values(customPetData.object_info_registry).map((o) => ({
+      if (customPetData == null) {
+        return [];
+      }
+
+      return Object.values(customPetData.object_info_registry).map((o) => ({
         id: o.obj_info_id,
         name: o.name,
         description: o.description,
         thumbnailUrl: o.thumbnail_url,
         rarityIndex: o.rarity_index,
-      })),
+      }));
-    items: (...args) => resolvers.Pet.wornItems(...args),
+    },
   },
   Query: {
     petOnNeopetsDotCom: async (
@@ -127,14 +123,16 @@ const resolvers = {
         loadPetMetaData(petName),
       ]);
 
-      await saveModelingData(customPetData, petMetaData, {
+      if (customPetData != null) {
-        db,
+        await saveModelingData(customPetData, petMetaData, {
-        petTypeBySpeciesAndColorLoader,
+          db,
-        petStateByPetTypeAndAssetsLoader,
+          petTypeBySpeciesAndColorLoader,
-        itemLoader,
+          petStateByPetTypeAndAssetsLoader,
-        itemTranslationLoader,
+          itemLoader,
-        swfAssetByRemoteIdLoader,
+          itemTranslationLoader,
-      });
+          swfAssetByRemoteIdLoader,
+        });
+      }
 
       return { name: petName, customPetData, petMetaData };
     },
@@ -155,10 +153,22 @@ async function loadPetMetaData(petName) {
 }
 
 async function loadCustomPetData(petName) {
-  const response = await neopetsXmlrpcCall("CustomPetService.getViewerData", [
+  try {
-    petName,
+    const response = await neopetsXmlrpcCall("CustomPetService.getViewerData", [
-  ]);
+      petName,
-  return response;
+    ]);
+    return response;
+  } catch (error) {
+    // If Neopets.com fails to find valid customization data, we return null.
+    if (
+      error.code === "AMFPHP_RUNTIME_ERROR" &&
+      error.faultString === "Unable to find body artwork for this combination."
+    ) {
+      return null;
+    } else {
+      throw error;
+    }
+  }
 }
 
 function getPoseFromPetData(petMetaData, petCustomData) {