feat: add limit parameter to activity totals queries and remove console logs

Author: austenstone

backend/src/services/seats.service.ts

@@ -441,9 +441,11 @@ class SeatsService {
     org?: string;
     since?: string;
     until?: string;
+    limit?: number;
   }) {
     const ActivityTotals = mongoose.model('ActivityTotals');
     const { org, since, until } = params;
+    const limit = typeof params.limit === 'string' ? parseInt(params.limit) : (params.limit || 100);
 
     const match: mongoose.FilterQuery<MemberActivityType> = {};
     if (org) match.org = org;
@@ -472,6 +474,7 @@ class SeatsService {
         }
       },
       { $sort: { total_time: -1 } },
+      { $limit: limit },
       {
         $project: {
           _id: 0,

frontend/src/app/main/copilot/copilot-metrics/copilot-metrics.component.ts

@@ -88,10 +88,10 @@ export class CopilotMetricsComponent implements OnInit {
     this.subscriptions.push(
       this.seatService.getActivityTotals({
         org: this.installation?.account?.login,
-        since: startModified.toISOString(),
-        until: endModified.toISOString()
+        since: event.start.toISOString(),
+        until: event.end.toISOString(),
+        limit: 10
       }).subscribe(totals => {
-        Object.keys(totals).forEach((key, index) => index > 10 ? delete totals[key] : null);
         this.activityTotals = totals;
         this.cdr.detectChanges();
       })

frontend/src/app/services/api/seat.service.ts

@@ -75,6 +75,7 @@ export class SeatService {
     org?: string | undefined;
     since?: string;
     until?: string;
+    limit?: number;
   }) {
     if (!queryParams?.org) delete queryParams?.org;
     return this.http.get<Record<string, number>>(`${this.apiUrl}/activity/totals`, {