Avoid double counting outside collaborators, preventing deletions

The 'direct' affiliation of collaborators includes the 'outside' collaborators,
and in testing when the resulting source contained duplicates the test
function was not deleting collaborators.

Signed-off-by: Kyle Harding <[email protected]>

Author: klutchell

lib/plugins/collaborators.js

@@ -15,9 +15,12 @@ module.exports = class Collaborators extends Diffable {
   }
 
   find () {
-    // this.log.debug(`Finding collaborators for { repo: ${this.repo.repo}, owner: ${this.repo.owner}, affiliation: 'direct', 'outside', and 'pending invites' }`)
+    // https://docs.github.com/en/rest/collaborators/collaborators?apiVersion=2022-11-28
+    // 'outside' means all outside collaborators of an organization-owned repository.
+    // 'direct' means all collaborators with permissions to an organization-owned repository, regardless of organization membership status. (includes outside collaborators)
+    // 'all' means all collaborators the authenticated user can see.
+    // We are using 'direct' to avoid double listing users outside collaborators and team members.
     return Promise.all([this.github.repos.listCollaborators({ repo: this.repo.repo, owner: this.repo.owner, affiliation: 'direct' }),
-      this.github.repos.listCollaborators({ repo: this.repo.repo, owner: this.repo.owner, affiliation: 'outside' }),
       this.github.repos.listInvitations({ repo: this.repo.repo, owner: this.repo.owner })])
       .then(res => {
         const mapCollaborator = user => {