Add error logging

franekmagiera · Feb 11, 2024 · be511cc · be511cc
1 parent ea595e1
commit be511cc
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 8 deletions.
diff --git a/app/src/get-youtube-video-summary.ts b/app/src/get-youtube-video-summary.ts
@@ -31,10 +31,20 @@ async function getYoutubeVideoSummary(
 ): Promise<Ok<string> | InternalFailure> {
   const getYoutubeCaptionsResult = await getYoutubeCaptions(videoId);
   if (getYoutubeCaptionsResult.result === Result.Failure) {
+    console.error(
+      getYoutubeCaptionsResult.failure,
+      getYoutubeCaptionsResult.message || "",
+    );
     // Couldn't get youtube captions, propagate the failure.
     return getYoutubeCaptionsResult;
   }
   const captions = getYoutubeCaptionsResult.data;
-  const summary = getCaptionsSummary(captions);
+  const summary = await getCaptionsSummary(captions);
+  if (summary.result === Result.Failure) {
+    console.error(
+      summary.failure,
+      summary.message || "",
+    );
+  }
   return summary;
 }
diff --git a/run.ts b/run.ts
@@ -22,13 +22,7 @@ async function run() {
   } else {
     const appConfig = appConfigResult.data;
     const summaryResult = await appConfig.getYoutubeVideoSummary(videoId);
-    if (summaryResult.result === Result.Failure) {
-      console.log(
-        summaryResult.failure,
-        "\n",
-        summaryResult.message,
-      );
-    } else {
+    if (summaryResult.result === Result.Ok) {
       console.log(summaryResult.data);
     }
   }