Improve recast error handling

Author: bfollington

typescript/packages/toolshed/routes/ai/spell/behavior/effects.ts

@@ -21,10 +21,13 @@ export class MemoryError extends Error {
 }
 
 function handleErrorResponse(data: any) {
+  if (typeof data.error === "string") {
+    throw new MemoryError(data.error, data);
+  }
   if ("cause" in data.error) {
     throw new MemoryError(data?.error?.cause?.message, data);
   } else {
-    throw new MemoryError(data?.error?.message, data);
+    throw new MemoryError(data?.error?.message || "Unknown error", data);
   }
 }
 

typescript/packages/toolshed/routes/ai/spell/spell.handlers.ts

@@ -446,12 +446,14 @@ async function processRecast(
   const argument = source.value.argument;
   const type = source.value.$TYPE;
   logger.debug({ type, argument }, "Extracted argument and type from source");
-
   const spellId = "spell-" + type;
   logger.info({ spellId }, "Looking up spell");
 
   const spell = await getBlob<Spell>(spellId);
   logger.info({ spellId }, "Retrieved spell");
+  if (!spell) {
+    throw new Error("No spell found for id: " + spellId);
+  }
 
   const schema = spell.recipe.argumentSchema;
   logger.debug({ schema }, "Extracted argument schema from spell");