Add commit stats fetching and update logic to Package model; enhance sole maintainer view with additional issue metadata

Author: andrew

app/models/package.rb

@@ -633,6 +633,26 @@ def ping_commits
     return unless repo_metadata.present?
     ecosystems_api_get("#{commits_api_url}/ping")
   end
+
+  def fetch_commit_stats
+    return unless commits_api_url.present?
+    ecosystems_api_get("#{commits_api_url}/stats")
+  rescue
+    nil
+  end
+
+  # Fetches commit stats from the commits service and stores in repo_metadata['commit_stats']
+  # This provides an alternative to relying on the repos service to load commit data
+  def update_commit_stats
+    return unless repo_metadata.present?
+    
+    commit_stats = fetch_commit_stats
+    if commit_stats.present?
+      updated_metadata = repo_metadata.deep_dup
+      updated_metadata['commit_stats'] = commit_stats
+      update(repo_metadata: updated_metadata)
+    end
+  end
 
   def sync_maintainers
     registry.sync_maintainers(self)

app/views/critical/_sole_maintainer_package.html.erb

@@ -26,10 +26,10 @@
       <% end %>
     </small>
 
-    <% if package.issue_metadata.present? || package.commit_stats.present? %>
+    <% if package.issue_metadata.present? || package.commit_stats.present? || package.repo_metadata&.dig('commit_stats').present? %>
       <br>
       <small class='text-muted'>
-        <% committers_count = package.commit_stats&.dig('total_committers') %>
+        <% committers_count = package.commit_stats&.dig('total_committers') || package.repo_metadata&.dig('commit_stats', 'total_committers') %>
         <% issue_authors_count = package.issue_metadata&.dig('past_year_issue_authors_count') %>
         <% pr_authors_count = package.issue_metadata&.dig('past_year_pull_request_authors_count') %>
         <% maintainers_count = package.issue_metadata&.dig('maintainers')&.length %>
@@ -61,7 +61,7 @@
           <% any_stats = true %>
         <% end %>
 
-        <% dds_score = package.issue_metadata&.dig('dds') || package.commit_stats&.dig('dds') %>
+        <% dds_score = package.issue_metadata&.dig('dds') || package.commit_stats&.dig('dds') || package.repo_metadata&.dig('commit_stats', 'dds') %>
         <% if dds_score && dds_score > 0 %>
           <%= '- ' if any_stats %><%= number_with_delimiter(dds_score) %> DDS
         <% end %>

test/controllers/critical_controller_test.rb

@@ -12,7 +12,13 @@ class CriticalControllerTest < ActionDispatch::IntegrationTest
       maintainers_count: 1,
       downloads: 1000,
       dependent_packages_count: 5,
-      dependent_repos_count: 10
+      dependent_repos_count: 10,
+      issue_metadata: {
+        'past_year_issue_authors_count' => 25,
+        'past_year_pull_request_authors_count' => 12,
+        'maintainers' => [{'login' => 'test-maintainer'}],
+        'active_maintainers' => [{'login' => 'test-maintainer'}]
+      }
     )
 
     @critical_package.maintainers << @maintainer
@@ -44,7 +50,10 @@ class CriticalControllerTest < ActionDispatch::IntegrationTest
       ecosystem: 'pypi', 
       name: 'other-package', 
       critical: true,
-      maintainers_count: 1
+      maintainers_count: 1,
+      issue_metadata: {
+        'past_year_issue_authors_count' => 15
+      }
     )
 
     get critical_sole_maintainers_path(registry: @registry.name)
@@ -59,7 +68,10 @@ class CriticalControllerTest < ActionDispatch::IntegrationTest
       name: 'high-download-package', 
       critical: true,
       maintainers_count: 1,
-      downloads: 5000
+      downloads: 5000,
+      issue_metadata: {
+        'past_year_issue_authors_count' => 30
+      }
     )
 
     get critical_sole_maintainers_path