don't add non-sensitive repos

Author: nicholasyan

dist/index.js

@@ -34479,9 +34479,8 @@ async function runOnOwner(context) {
         return;
     }
     try {
-        const { sensitiveRepos, nonSensitiveRepos } = await fetchOrgRepos(context);
+        const { sensitiveRepos } = await fetchOrgRepos(context);
         await updateInstalledRepos({
-            addRepos: nonSensitiveRepos,
             removeRepos: sensitiveRepos,
             context,
         });
@@ -34517,45 +34516,24 @@ async function fetchOrgRepos(context) {
         .sort(repo_utils_js_1.sortRepos);
     const sensitiveReposSet = new Set();
     sensitiveRepos.forEach((sr) => sensitiveReposSet.add(sr.id));
-    const nonSensitiveRepos = Object.values(repoInfo)
-        .filter((repo) => !sensitiveReposSet.has(repo.id))
-        .sort(repo_utils_js_1.sortRepos);
     context.log.info(`All '${context.login}' repos (visible to supplied token)`);
     context.log.info("------------------------------");
     context.log.info("");
     context.log.info(`Sensitive [${sensitiveRepos.length}]:`);
     context.log.repos(sensitiveRepos);
     context.log.info("");
-    context.log.info(`Non-sensitive [${nonSensitiveRepos.length}]:`);
-    context.log.repos(nonSensitiveRepos);
-    context.log.info("");
     return {
         sensitiveRepos,
-        nonSensitiveRepos,
     };
 }
-async function updateInstalledRepos({ addRepos, removeRepos, context, }) {
-    if (addRepos.length === 0 && removeRepos.length === 0) {
+async function updateInstalledRepos({ removeRepos, context, }) {
+    if (removeRepos.length === 0) {
+        context.log.info(`No repo adjustments needed!`);
+        context.log.info(``);
         return;
     }
     context.log.info(`Applying adjustments...`);
     context.log.info(``);
-    for (const repo of addRepos) {
-        try {
-            const res = await context.github
-                .classicPat()
-                .rest.apps.addRepoToInstallationForAuthenticatedUser({
-                installation_id: context.installId,
-                repository_id: repo.id,
-            });
-            context.log.info(`+ ${repo.fullName} (status: ${res.status})`);
-        }
-        catch (err) {
-            context.log.error(`Failed to add ${repo.fullName} (id: ${repo.id})`, err);
-            context.setStatus("FAILURE");
-        }
-    }
-    context.log.info(``);
     for (const repo of removeRepos) {
         try {
             const res = await context.github

dist/index.js.map

@@ -31,9 +31,8 @@ async function runOnOwner(context: OwnerContext) {
   }
 
   try {
-    const { sensitiveRepos, nonSensitiveRepos } = await fetchOrgRepos(context);
+    const { sensitiveRepos } = await fetchOrgRepos(context);
     await updateInstalledRepos({
-      addRepos: nonSensitiveRepos,
       removeRepos: sensitiveRepos,
       context,
     });
@@ -45,7 +44,6 @@ async function runOnOwner(context: OwnerContext) {
 
 async function fetchOrgRepos(context: OwnerContext): Promise<{
   sensitiveRepos: Repo[];
-  nonSensitiveRepos: Repo[];
 }> {
   const repoInfo: Record<
     string,
@@ -89,9 +87,6 @@ async function fetchOrgRepos(context: OwnerContext): Promise<{
 
   const sensitiveReposSet = new Set();
   sensitiveRepos.forEach((sr) => sensitiveReposSet.add(sr.id));
-  const nonSensitiveRepos = Object.values(repoInfo)
-    .filter((repo) => !sensitiveReposSet.has(repo.id))
-    .sort(sortRepos);
 
   context.log.info(`All '${context.login}' repos (visible to supplied token)`);
   context.log.info("------------------------------");
@@ -101,48 +96,27 @@ async function fetchOrgRepos(context: OwnerContext): Promise<{
   context.log.repos(sensitiveRepos);
   context.log.info("");
 
-  context.log.info(`Non-sensitive [${nonSensitiveRepos.length}]:`);
-  context.log.repos(nonSensitiveRepos);
-  context.log.info("");
-
   return {
     sensitiveRepos,
-    nonSensitiveRepos,
   };
 }
 
 async function updateInstalledRepos({
-  addRepos,
   removeRepos,
   context,
 }: {
-  addRepos: Repo[];
   removeRepos: Repo[];
   context: OwnerContext;
 }) {
-  if (addRepos.length === 0 && removeRepos.length === 0) {
+  if (removeRepos.length === 0) {
+    context.log.info(`No repo adjustments needed!`);
+    context.log.info(``);
     return;
   }
 
   context.log.info(`Applying adjustments...`);
   context.log.info(``);
 
-  for (const repo of addRepos) {
-    try {
-      const res = await context.github
-        .classicPat()
-        .rest.apps.addRepoToInstallationForAuthenticatedUser({
-          installation_id: context.installId,
-          repository_id: repo.id,
-        });
-      context.log.info(`+ ${repo.fullName} (status: ${res.status})`);
-    } catch (err) {
-      context.log.error(`Failed to add ${repo.fullName} (id: ${repo.id})`, err);
-      context.setStatus("FAILURE");
-    }
-  }
-  context.log.info(``);
-
   for (const repo of removeRepos) {
     try {
       const res = await context.github