Merge pull request #16 from PRODYNA/feature/15-repositories-of-contributors-are-only-listed-once

Attaching mutliple repositories work

Author: dkrizic

action.yaml

@@ -25,7 +25,7 @@ inputs:
     default: 1
 runs:
   using: 'docker'
-  image: 'docker://ghcr.io/prodyna/github-users:v1.1'
+  image: 'docker://ghcr.io/prodyna/github-users:v1.2'
   env:
     ACTION: ${{ inputs.action }}
     ENTERPRISE: ${{ inputs.enterprise }}

template/collaborators.tpl

@@ -6,6 +6,5 @@ Last updated: {{ .Updated }}
 | ------ | ---- | ------------- | ------------ | ---------- |
 {{ range $user := .Users }}{{ range $org := $user.Organizations }}{{ range $repo := $org.Repositories }}| {{ $user.Number }} | [{{ $user.Login }}](https://github.com/{{ $user.Login }}) | {{if $user.Contributions}}:green_square:{{else}}:red_square:{{end}} {{ $user.Contributions }} | [{{ $org.Name }}](https://github.com/{{ $org.Login }}) | [{{ $repo.Name }}](https://github.com/{{ $org.Login }}/{{ $repo.Name }}) |
 {{ end }}{{ end }}{{ end }}
-
 ---
 Generated with :heart: by [github-users](https://github.com/prodyna/github-users)

userlist/collaborators.go

@@ -143,23 +143,32 @@ func (c *UserListConfig) loadCollaborators() error {
 			slog.DebugContext(ctx, "Processing repository", "repository", repo.Name, "collaborator.count", len(repo.Collaborators.Nodes))
 			for _, collaborator := range repo.Collaborators.Nodes {
 				slog.DebugContext(ctx, "Processing collaborator", "login", collaborator.Login, "name", collaborator.Name, "contributions", collaborator.ContributionsCollection.ContributionCalendar.TotalContributions)
+
+				// User
 				user := c.userList.findUser(collaborator.Login)
 				if user == nil {
 					user = c.userList.createUser(userNumber+1, collaborator.Login, collaborator.Name, "", collaborator.ContributionsCollection.ContributionCalendar.TotalContributions)
 					userNumber++
 				} else {
 					slog.Info("Found existing user", "login", user.Login)
 				}
-				organization := Organization{
-					Login:        org.Login,
-					Name:         org.Name,
-					Repositories: new([]Repository),
+
+				// Organization
+				organization := user.findOrganization(org.Login)
+				if organization == nil {
+					organization = user.createOrganization(org.Login, org.Name)
+				} else {
+					slog.Info("Found existing organization", "organization", organization.Name)
 				}
-				user.upsertOrganization(organization)
-				repository := Repository{
-					Name: repo.Name,
+
+				// Repository
+				repository := organization.findRepository(repo.Name)
+				if repository == nil {
+					repository = organization.createRepository(repo.Name)
+				} else {
+					slog.Info("Found existing repository", "repository", repository.Name)
 				}
-				organization.upsertRepository(repository)
+				organization.upsertRepository(*repository)
 			}
 		}
 

userlist/userlist.go

@@ -208,3 +208,39 @@ func (o *Organization) upsertRepository(repo Repository) {
 	slog.Debug("Upserting repository", "name", repo.Name, "organization", o.Name)
 	*o.Repositories = append(*o.Repositories, repo)
 }
+
+func (u *User) findOrganization(login string) *Organization {
+	for _, o := range *u.Organizations {
+		if o.Login == login {
+			return &o
+		}
+	}
+	return nil
+}
+
+func (u *User) createOrganization(login string, name string) *Organization {
+	org := &Organization{
+		Login:        login,
+		Name:         name,
+		Repositories: new([]Repository),
+	}
+	u.upsertOrganization(*org)
+	return org
+}
+
+func (o *Organization) findRepository(name string) *Repository {
+	for _, r := range *o.Repositories {
+		if r.Name == name {
+			return &r
+		}
+	}
+	return nil
+}
+
+func (o *Organization) createRepository(name string) *Repository {
+	repo := &Repository{
+		Name: name,
+	}
+	o.upsertRepository(*repo)
+	return repo
+}