feat(platform/github): Skip archived repos at earliest point when ret…

…rieving repo listings (#22057)
renovatebot · May 10, 2023 · 06b8dcd · 06b8dcd
1 parent 2e2570c
commit 06b8dcd
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 3 deletions.
diff --git a/lib/modules/platform/github/index.spec.ts b/lib/modules/platform/github/index.spec.ts
@@ -174,9 +174,15 @@ describe('modules/platform/github/index', () => {
         .reply(200, [
           {
             full_name: 'a/b',
+            archived: false,
           },
           {
             full_name: 'c/d',
+            archived: false,
+          },
+          {
+            full_name: 'e/f',
+            archived: true,
           },
           null,
         ]);
@@ -227,9 +233,15 @@ describe('modules/platform/github/index', () => {
           repositories: [
             {
               full_name: 'a/b',
+              archived: false,
             },
             {
               full_name: 'c/d',
+              archived: false,
+            },
+            {
+              full_name: 'e/f',
+              archived: true,
             },
             null,
           ],

diff --git a/lib/modules/platform/github/index.ts b/lib/modules/platform/github/index.ts
@@ -191,20 +191,24 @@ export async function getRepos(): Promise<string[]> {
   try {
     if (platformConfig.isGHApp) {
       const res = await githubApi.getJson<{
-        repositories: { full_name: string }[];
+        repositories: GhRestRepo[];
       }>(`installation/repositories?per_page=100`, {
         paginationField: 'repositories',
         paginate: 'all',
       });
       return res.body.repositories
         .filter(is.nonEmptyObject)
+        .filter((repo) => !repo.archived)
         .map((repo) => repo.full_name);
     } else {
-      const res = await githubApi.getJson<{ full_name: string }[]>(
+      const res = await githubApi.getJson<GhRestRepo[]>(
         `user/repos?per_page=100`,
         { paginate: 'all' }
       );
-      return res.body.filter(is.nonEmptyObject).map((repo) => repo.full_name);
+      return res.body
+        .filter(is.nonEmptyObject)
+        .filter((repo) => !repo.archived)
+        .map((repo) => repo.full_name);
     }
   } catch (err) /* istanbul ignore next */ {
     logger.error({ err }, `GitHub getRepos error`);

diff --git a/lib/modules/platform/github/types.ts b/lib/modules/platform/github/types.ts
@@ -26,6 +26,7 @@ export interface GhRestRepo {
   owner: {
     login: string;
   };
+  archived: boolean;
 }
 
 export interface GhRestPr {