chore: error handling

Author: Keyrxng

src/handlers/comment-created-callback.ts

@@ -23,7 +23,7 @@ export async function issueCommentCreatedCallback(
     return { status: 204, reason: logger.info("Comment is empty. Skipping.").logMessage.raw };
   }
   logger.info(`Asking question: ${question}`);
-  let commentToPost;
+
   try {
     const response = await askQuestion(context, question);
     const { answer, tokenUsage } = response;
@@ -32,10 +32,10 @@ export async function issueCommentCreatedCallback(
     }
     logger.info(`Answer: ${answer}`, { tokenUsage });
     const tokens = `\n\n<!--\n${JSON.stringify(tokenUsage, null, 2)}\n--!>`;
-    commentToPost = answer + tokens;
+    const commentToPost = answer + tokens;
     await addCommentToIssue(context, commentToPost);
     return { status: 200, reason: logger.info("Comment posted successfully").logMessage.raw };
-  } catch (err) {
-    throw err;
+  } catch (error) {
+    throw await bubbleUpErrorComment(context, error, false);
   }
 }

src/handlers/comments.ts

@@ -1,3 +1,4 @@
+import { logger } from "../helpers/errors";
 import { splitKey } from "../helpers/issue";
 import { LinkedIssues, SimplifiedComment } from "../types/github-types";
 import { StreamlinedComment } from "../types/llm";
@@ -53,7 +54,10 @@ export function createKey(issueUrl: string, issue?: number) {
   }
 
   if (!key) {
-    throw new Error("Invalid issue url");
+    throw logger.error("Invalid issue URL", {
+      issueUrl,
+      issueNumber: issue,
+    });
   }
 
   if (key.includes("#")) {

src/helpers/errors.ts

@@ -16,7 +16,7 @@ export async function bubbleUpErrorComment(context: Context, err: unknown, post
   if (err instanceof LogReturn) {
     errorMessage = err;
   } else if (err instanceof Error) {
-    errorMessage = context.logger.error(err.message, { error: err });
+    errorMessage = context.logger.error(err.message, { stack: err.stack });
   } else {
     errorMessage = context.logger.error("An error occurred", { err });
   }

src/helpers/issue-fetching.ts

@@ -42,7 +42,7 @@ export async function fetchLinkedIssues(params: FetchParams) {
     return { streamlinedComments: {}, linkedIssues: [], specAndBodies: {}, seen: new Set<string>() };
   }
   if (!issue.body || !issue.html_url) {
-    throw logger.error("Issue body or URL not found");
+    throw logger.error("Issue body or URL not found", { issueUrl: issue.html_url });
   }
 
   if (!params.owner || !params.repo) {

src/helpers/issue.ts

@@ -2,6 +2,7 @@ import { createKey } from "../handlers/comments";
 import { FetchedCodes, FetchParams, LinkedIssues } from "../types/github-types";
 import { StreamlinedComment } from "../types/llm";
 import { Context } from "../types/context"; // Import Context type
+import { logger } from "./errors";
 
 /**
  * Removes duplicate streamlined comments based on their body content.
@@ -239,7 +240,7 @@ export async function pullReadmeFromRepoForIssue(params: FetchParams): Promise<s
       readme = Buffer.from(response.data.content, "base64").toString();
     }
   } catch (error) {
-    throw new Error(`Error fetching README from repository: ${error}`);
+    throw logger.error(`Error fetching README from repository: ${error}`);
   }
   return readme;
 }

src/main.ts

@@ -1,4 +1,6 @@
+// @ts-expect-error - no types found
 import * as core from "@actions/core";
+// @ts-expect-error - no types found
 import * as github from "@actions/github";
 import { Value } from "@sinclair/typebox/value";
 import { envSchema } from "./types/env";