Add repository count diagnostics for GitHub App installations

Co-authored-by: MattG57 <[email protected]>

Author: Copilot

backend/src/controllers/setup.controller.ts

@@ -32,6 +32,7 @@ interface InstallationDiagnostic {
   octokitTest: OctokitTestResult | null;
   isValid: boolean;
   validationErrors: string[];
+  repositoryCount: number;
 }
 
 interface AppInfo {
@@ -55,6 +56,8 @@ interface DiagnosticsResponse {
     invalidInstallations: number;
     organizationNames: string[];
     accountTypes: Record<string, number>;
+    repositoryCounts: Record<string, number>;
+    totalRepositories: number;
   };
 }
 
@@ -180,7 +183,9 @@ class SetupController {
           validInstallations: 0,
           invalidInstallations: 0,
           organizationNames: [],
-          accountTypes: {}
+          accountTypes: {},
+          repositoryCounts: {},
+          totalRepositories: 0
         }
       };
 
@@ -213,7 +218,8 @@ class SetupController {
           hasOctokit: !!octokit,
           octokitTest: null,
           isValid: true,
-          validationErrors: []
+          validationErrors: [],
+          repositoryCount: 0
         };
 
         // Validate required fields
@@ -232,7 +238,7 @@ class SetupController {
           installationDiag.validationErrors.push('Missing account.type');
         }
 
-        // Test Octokit functionality
+        // Test Octokit functionality and fetch repository count
         if (octokit) {
           try {
             // Test basic API call with the installation's octokit
@@ -243,17 +249,29 @@ class SetupController {
               appOwner: (authTest.data?.owner && 'login' in authTest.data.owner) ? authTest.data.owner.login : 'Unknown',
               permissions: authTest.data?.permissions || {}
             };
+
+            // Fetch repositories for this installation
+            try {
+              const repos = await octokit.request(installation.repositories_url);
+              installationDiag.repositoryCount = repos.data.repositories?.length || 0;
+            } catch (repoError) {
+              // If repository fetching fails, log it but don't mark installation as invalid
+              installationDiag.validationErrors.push(`Failed to fetch repositories: ${repoError instanceof Error ? repoError.message : 'Unknown error'}`);
+              installationDiag.repositoryCount = 0;
+            }
           } catch (error) {
             installationDiag.octokitTest = {
               success: false,
               error: error instanceof Error ? error.message : 'Unknown error'
             };
             installationDiag.isValid = false;
             installationDiag.validationErrors.push(`Octokit API test failed: ${error instanceof Error ? error.message : 'Unknown error'}`);
+            installationDiag.repositoryCount = 0;
           }
         } else {
           installationDiag.isValid = false;
           installationDiag.validationErrors.push('Octokit instance is missing');
+          installationDiag.repositoryCount = 0;
         }
 
         // Update summary
@@ -270,6 +288,11 @@ class SetupController {
         const accountType = installation.account?.type || 'Unknown';
         diagnostics.summary.accountTypes[accountType] = (diagnostics.summary.accountTypes[accountType] || 0) + 1;
 
+        // Track repository counts per organization
+        const orgName = installation.account?.login || 'Unknown';
+        diagnostics.summary.repositoryCounts[orgName] = installationDiag.repositoryCount;
+        diagnostics.summary.totalRepositories += installationDiag.repositoryCount;
+
         diagnostics.installations.push(installationDiag);
       }
 

frontend/src/app/main/diagnostics/main-diagnostics.component.ts

@@ -41,6 +41,7 @@ import { DiagnosticsResponse } from '../../types/diagnostics.types';
             <li>Validating that all installation accounts have proper data</li>
             <li>Testing Octokit authentication for each installation</li>
             <li>Listing all organization names (account.login) available</li>
+            <li>Counting repositories for each GitHub App installation</li>
             <li>Verifying account types and permissions</li>
             <li>Providing detailed error information for troubleshooting</li>
           </ul>
@@ -84,13 +85,19 @@ import { DiagnosticsResponse } from '../../types/diagnostics.types';
               <div class="stat-value">{{ getSuccessRate() }}%</div>
               <div class="stat-label">Success Rate</div>
             </div>
+            <div class="stat">
+              <div class="stat-value" [class.success]="lastResult.summary.totalRepositories > 0">
+                {{ lastResult.summary.totalRepositories }}
+              </div>
+              <div class="stat-label">Total Repositories</div>
+            </div>
           </div>
           
           <div class="organizations" *ngIf="lastResult.summary.organizationNames.length > 0">
             <h4>Organizations Found:</h4>
             <div class="org-list">
               <span class="org-chip" *ngFor="let org of lastResult.summary.organizationNames">
-                {{ org }}
+                {{ org }} ({{ getRepositoryCount(org) }})
               </span>
             </div>
           </div>
@@ -232,6 +239,11 @@ export class MainDiagnosticsComponent {
     return Math.round((this.lastResult.summary.validInstallations / this.lastResult.totalInstallations) * 100);
   }
 
+  getRepositoryCount(orgName: string): number {
+    if (!this.lastResult || !this.lastResult.summary.repositoryCounts) return 0;
+    return this.lastResult.summary.repositoryCounts[orgName] || 0;
+  }
+
   showFullDetails(): void {
     this.dialog.open(InstallationDiagnosticsDialogComponent, {
       width: '90vw',
@@ -282,6 +294,7 @@ export class MainDiagnosticsComponent {
               <p><strong>Valid:</strong> {{ data.summary.validInstallations }}</p>
               <p><strong>Invalid:</strong> {{ data.summary.invalidInstallations }}</p>
               <p><strong>Success Rate:</strong> {{ getSuccessRate() }}%</p>
+              <p><strong>Total Repositories:</strong> {{ data.summary.totalRepositories }}</p>
             </mat-card-content>
           </mat-card>
         </div>
@@ -304,7 +317,7 @@ export class MainDiagnosticsComponent {
           </mat-card-header>
           <mat-card-content>
             <mat-chip-set>
-              <mat-chip *ngFor="let org of data.summary.organizationNames">{{ org }}</mat-chip>
+              <mat-chip *ngFor="let org of data.summary.organizationNames">{{ org }} ({{ getRepositoryCount(org) }} repos)</mat-chip>
             </mat-chip-set>
           </mat-card-content>
         </mat-card>
@@ -361,6 +374,9 @@ export class MainDiagnosticsComponent {
                 <div class="detail-item">
                   <strong>Has Octokit:</strong> {{ installation.hasOctokit ? 'Yes' : 'No' }}
                 </div>
+                <div class="detail-item">
+                  <strong>Repository Count:</strong> {{ installation.repositoryCount }}
+                </div>
                 <div class="detail-item">
                   <strong>Created:</strong> {{ installation.createdAt | date:'medium' }}
                 </div>
@@ -469,6 +485,11 @@ export class InstallationDiagnosticsDialogComponent {
     return Math.round((this.data.summary.validInstallations / this.data.totalInstallations) * 100);
   }
 
+  getRepositoryCount(orgName: string): number {
+    if (!this.data.summary.repositoryCounts) return 0;
+    return this.data.summary.repositoryCounts[orgName] || 0;
+  }
+
   downloadDiagnostics(): void {
     const dataStr = JSON.stringify(this.data, null, 2);
     const dataBlob = new Blob([dataStr], { type: 'application/json' });

frontend/src/app/types/diagnostics.types.ts

@@ -27,6 +27,7 @@ export interface InstallationDiagnostic {
   octokitTest: OctokitTestResult | null;
   isValid: boolean;
   validationErrors: string[];
+  repositoryCount: number;
 }
 
 export interface AppInfo {
@@ -50,5 +51,7 @@ export interface DiagnosticsResponse {
     invalidInstallations: number;
     organizationNames: string[];
     accountTypes: Record<string, number>;
+    repositoryCounts: Record<string, number>;
+    totalRepositories: number;
   };
 }