refactor: add error handling in routes (#429)

Author: AmirSa12

packages/app/server/routes/check.post.ts

@@ -1,44 +1,70 @@
 export default eventHandler(async (event) => {
-  const data = await readRawBody(event);
-  const workflowsBucket = useWorkflowsBucket(event);
+  try {
+    const data = await readRawBody(event);
+    const workflowsBucket = useWorkflowsBucket(event);
 
-  const { owner, repo, key } = JSON.parse(data!);
+    const { owner, repo, key } = JSON.parse(data!);
 
-  const app = useOctokitApp(event);
+    const app = useOctokitApp(event);
 
-  let authenticated = false;
+    let authenticated = false;
 
-  try {
-    await app.octokit.request("GET /repos/{owner}/{repo}/installation", {
-      owner,
-      repo,
-    });
-    authenticated = true;
-  } catch {}
+    try {
+      await app.octokit.request("GET /repos/{owner}/{repo}/installation", {
+        owner,
+        repo,
+      });
+      authenticated = true;
+    } catch {}
 
-  try {
-    await app.octokit.request("GET /orgs/{org}/installation", {
-      org: owner,
-    });
-    authenticated = true;
-  } catch {}
+    try {
+      await app.octokit.request("GET /orgs/{org}/installation", {
+        org: owner,
+      });
+      authenticated = true;
+    } catch {}
 
-  if (!authenticated) {
-    throw createError({
-      statusCode: 404,
-      fatal: true,
-      message: `The app https://github.com/apps/pkg-pr-new is not installed on ${owner}/${repo}.`,
-    });
-  }
+    if (!authenticated) {
+      throw createError({
+        statusCode: 404,
+        fatal: true,
+        message: `The app https://github.com/apps/pkg-pr-new is not installed on ${owner}/${repo}.`,
+      });
+    }
+
+    const workflowData = await workflowsBucket.getItem(key);
+
+    if (!workflowData) {
+      throw createError({
+        statusCode: 404,
+        fatal: true,
+        message: `There is no workflow defined for ${key}`,
+      });
+    }
+    return { sha: workflowData.sha };
+  } catch (error: unknown) {
+    console.error("Check route error:", error);
+
+    if (error && typeof error === "object" && "statusCode" in error) {
+      throw error;
+    }
 
-  const workflowData = await workflowsBucket.getItem(key);
+    const message =
+      error instanceof Error
+        ? error.message
+        : "An unexpected error occurred during check";
+    const stack = error instanceof Error ? error.stack : undefined;
 
-  if (!workflowData) {
     throw createError({
-      statusCode: 404,
-      fatal: true,
-      message: `There is no workflow defined for ${key}`,
+      statusCode: 500,
+      statusMessage: "Internal Server Error",
+      data: {
+        error: true,
+        message,
+        stack,
+        originalError: error,
+        type: "check_error",
+      },
     });
   }
-  return { sha: workflowData.sha };
 });