fix: add error logging to all bare catch blocks — silent failures now visible

Author: AlbinoGeek

src/server/changelog-draft-tool.ts

@@ -159,6 +159,10 @@ export function registerChangelogDraftTool(server: FastMCP): void {
 
         return lines.join("\n");
       } catch (err) {
+        console.error(
+          `[changelog_draft] Failed to generate changelog for ${owner}/${repo}:`,
+          err instanceof Error ? err.message : String(err),
+        );
         return errorRespond(classifyError(err));
       }
     },

src/server/ci-diagnosis-tool.ts

@@ -136,8 +136,12 @@ export function registerCiDiagnosisTool(server: FastMCP): void {
                 .join("\n");
             }
             logText = tailTruncate(raw, args.maxLogLines);
-          } catch {
+          } catch (err) {
             // logs expired or unavailable
+            console.error(
+              `[ci_diagnosis] Failed to download logs for job ${job.id}:`,
+              err instanceof Error ? err.message : String(err),
+            );
           }
           failedJobs.push({
             name: job.name,
@@ -186,6 +190,10 @@ export function registerCiDiagnosisTool(server: FastMCP): void {
 
         return lines.join("\n");
       } catch (err) {
+        console.error(
+          `[ci_diagnosis] Failed to diagnose CI for ${owner}/${repo}:`,
+          err instanceof Error ? err.message : String(err),
+        );
         return errorRespond(classifyError(err));
       }
     },

src/server/compare-refs.ts

@@ -79,7 +79,11 @@ export async function resolveRef(
     const obj = data.repository.object;
     if (!obj) return null;
     return { oid: obj.oid, committedDate: obj.committedDate };
-  } catch {
+  } catch (err) {
+    console.error(
+      `[resolveRef] Failed to resolve ref '${ref}' in ${owner}/${repo}:`,
+      err instanceof Error ? err.message : String(err),
+    );
     return null;
   }
 }

src/server/ecosystem-activity-tool.ts

@@ -108,6 +108,10 @@ async function fetchRepoCommits(
       }
     }
   } catch (err) {
+    console.error(
+      `[fetchRepoCommits] Failed to fetch commits for ${owner}/${repo}:`,
+      err instanceof Error ? err.message : String(err),
+    );
     return { owner, repo, commitCount: 0, error: classifyError(err), commits: [] };
   }
 

src/server/github-auth.ts

@@ -42,8 +42,12 @@ export function gateAuth(): AuthResult {
       cached = { ok: true, token };
       return cached;
     }
-  } catch {
+  } catch (err) {
     // gh not installed or not authenticated — fall through
+    console.error(
+      "[gateAuth] Failed to get token via 'gh auth token':",
+      err instanceof Error ? err.message : String(err),
+    );
   }
 
   cached = {

src/server/github-client.ts

@@ -164,8 +164,12 @@ export function resolveLocalRepoRemote(
       stdio: ["ignore", "pipe", "ignore"],
     }).trim();
     return parseGitHubRemoteUrl(url);
-  } catch {
+  } catch (err) {
     // Not a git repo or no origin
+    console.error(
+      `[resolveLocalRepoRemote] Failed to resolve remote for ${localPath}:`,
+      err instanceof Error ? err.message : String(err),
+    );
   }
   return undefined;
 }
@@ -219,6 +223,7 @@ export async function fetchPRMetadata(
     const classified = classifyError(err);
     console.error(
       `[fetchPRMetadata] Error fetching PR metadata for ${owner}/${repo} (PRs: ${batch.join(", ")}): ${classified.code} — ${classified.message}`,
+      err instanceof Error ? err.message : String(err),
     );
   }
   return map;

src/server/json.ts

@@ -13,7 +13,11 @@ export function readPackageVersion(): string {
   try {
     const j = JSON.parse(readFileSync(pkgPath, "utf8")) as { version?: string };
     _cachedVersion = j.version ?? "0.0.0";
-  } catch {
+  } catch (err) {
+    console.error(
+      `[readPackageVersion] Failed to read ${pkgPath}:`,
+      err instanceof Error ? err.message : String(err),
+    );
     _cachedVersion = "0.0.0";
   }
   return _cachedVersion;

src/server/module-pin-hint-tool.ts

@@ -175,6 +175,10 @@ export function registerModulePinHintTool(server: FastMCP): void {
 
         return formatModulePinHintMarkdown(result);
       } catch (err) {
+        console.error(
+          `[module_pin_hint] Failed to resolve pseudo-version for ${owner}/${repo}:`,
+          err instanceof Error ? err.message : String(err),
+        );
         return errorRespond(classifyError(err));
       }
     },

src/server/my-work-tool.ts

@@ -65,6 +65,10 @@ export function registerMyWorkTool(server: FastMCP): void {
           );
           username = viewer.viewer.login;
         } catch (err) {
+          console.error(
+            "[my_work_tool] Failed to fetch viewer login:",
+            err instanceof Error ? err.message : String(err),
+          );
           return errorRespond(classifyError(err));
         }
       }
@@ -196,6 +200,10 @@ export function registerMyWorkTool(server: FastMCP): void {
 
         return lines.join("\n");
       } catch (err) {
+        console.error(
+          `[my_work_tool] Failed to fetch work items for @${username}:`,
+          err instanceof Error ? err.message : String(err),
+        );
         return errorRespond(classifyError(err));
       }
     },

src/server/org-pulse-tool.ts

@@ -172,6 +172,10 @@ export function registerOrgPulseTool(server: FastMCP): void {
           first: args.maxRepos,
         });
       } catch (err) {
+        console.error(
+          `[org_pulse] Failed to fetch org pulse for ${args.org}:`,
+          err instanceof Error ? err.message : String(err),
+        );
         return errorRespond(classifyError(err));
       }