fix(home): handle GitHub API failures gracefully in releases/changelogs (#3292)

* fix(home): handle GitHub API failures gracefully in releases/changelogs (#3291)

Wrap GitHub API calls in try/catch and return empty array on failure.
Prevents 422 errors when rate-limited or repos are unreachable.

* fix(home): align error tests with graceful GitHub API fallback

Tests now expect 200 with empty data when GitHub API fails, matching
the service's catch-and-return-empty-array behavior.

* fix(home): update JSDoc for releases/changelogs to reflect graceful degradation

Author: PierreBrisorgueil

modules/home/services/home.service.js

@@ -28,44 +28,52 @@ const page = async (name) => {
 };
 
 /**
- * @desc Function to get all versions
- * @return {Promise} All versions
+ * @desc Fetch releases from configured GitHub repos. Returns an empty array on API failure (graceful degradation).
+ * @return {Promise<Array<{title: string, list: Array}>>} Releases grouped by repo, or [] on error
  */
 const releases = async () => {
-  const requests = config.repos.map((item) =>
-    axios.get(`https://api.github.com/repos/${item.owner}/${item.repo}/releases`, {
-      headers: item.token ? { Authorization: `token ${item.token}` } : {},
-    }),
-  );
-  let results = await axios.all(requests);
-  results = results.map((result, i) => ({
-    title: config.repos[i].title,
-    list: result.data.map((release) => ({
-      name: release.name,
-      prerelease: release.prerelease,
-      published_at: release.published_at,
-    })),
-  }));
-  return Promise.resolve(results);
+  try {
+    const requests = config.repos.map((item) =>
+      axios.get(`https://api.github.com/repos/${item.owner}/${item.repo}/releases`, {
+        headers: item.token ? { Authorization: `token ${item.token}` } : {},
+      }),
+    );
+    let results = await axios.all(requests);
+    results = results.map((result, i) => ({
+      title: config.repos[i].title,
+      list: result.data.map((release) => ({
+        name: release.name,
+        prerelease: release.prerelease,
+        published_at: release.published_at,
+      })),
+    }));
+    return results;
+  } catch (_err) {
+    return [];
+  }
 };
 
 /**
- * @desc Function to get all changelogs
- * @return {Promise} All changelogs
+ * @desc Fetch changelogs from configured GitHub repos. Returns an empty array on API failure (graceful degradation).
+ * @return {Promise<Array<{title: string, markdown: string}>>} Changelogs grouped by repo, or [] on error
  */
 const changelogs = async () => {
-  const repos = _.filter(config.repos, (repo) => repo.changelog);
-  const requests = repos.map((item) =>
-    axios.get(`https://api.github.com/repos/${item.owner}/${item.repo}/contents/${item.changelog}`, {
-      headers: item.token ? { Authorization: `token ${item.token}` } : {},
-    }),
-  );
-  let results = await axios.all(requests);
-  results = results.map((result, i) => ({
-    title: config.repos[i].title,
-    markdown: Base64.decode(result.data.content),
-  }));
-  return Promise.resolve(results);
+  try {
+    const repos = _.filter(config.repos, (repo) => repo.changelog);
+    const requests = repos.map((item) =>
+      axios.get(`https://api.github.com/repos/${item.owner}/${item.repo}/contents/${item.changelog}`, {
+        headers: item.token ? { Authorization: `token ${item.token}` } : {},
+      }),
+    );
+    let results = await axios.all(requests);
+    results = results.map((result, i) => ({
+      title: repos[i].title,
+      markdown: Base64.decode(result.data.content),
+    }));
+    return results;
+  } catch (_err) {
+    return [];
+  }
 };
 
 /**

modules/home/tests/home.integration.tests.js

@@ -91,20 +91,20 @@ describe('Home integration tests:', () => {
       }
     });
 
-    test('should return 422 when GitHub API fails for releases', async () => {
+    test('should return empty releases gracefully when GitHub API fails', async () => {
       axios.get.mockRejectedValueOnce(new Error('GitHub API unavailable'));
-      const result = await agent.get('/api/home/releases').expect(422);
-      expect(result.body.type).toBe('error');
-      expect(result.body.message).toBe('Unprocessable Entity');
-      expect(result.body.description).toBe('GitHub API unavailable.');
+      const result = await agent.get('/api/home/releases').expect(200);
+      expect(result.body.type).toBe('success');
+      expect(result.body.message).toBe('releases');
+      expect(result.body.data).toEqual([]);
     });
 
-    test('should return 422 when GitHub API fails for changelogs', async () => {
+    test('should return empty changelogs gracefully when GitHub API fails', async () => {
       axios.get.mockRejectedValueOnce(new Error('GitHub API unavailable'));
-      const result = await agent.get('/api/home/changelogs').expect(422);
-      expect(result.body.type).toBe('error');
-      expect(result.body.message).toBe('Unprocessable Entity');
-      expect(result.body.description).toBe('GitHub API unavailable.');
+      const result = await agent.get('/api/home/changelogs').expect(200);
+      expect(result.body.type).toBe('success');
+      expect(result.body.message).toBe('changelogs');
+      expect(result.body.data).toEqual([]);
     });
 
     test('should use Authorization header when a token is configured for releases', async () => {