dev_metrics 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 18955085e7be6c6a43726926c4b77a93039d26ffb1433dc9d7487d30c0a345ff
-  data.tar.gz: 5e9b1bbe91561e5643399645f24f438f447ef060d85a5499e1a068c51e05d577
+  metadata.gz: 61eeda6933db54b48b7f7ca3941b035da3d04f9f9a8e6d414b8ef9db87a265ea
+  data.tar.gz: c2477155037f8f078dc7ffcc044c7cba1767915a0334434af365ddf967fcca06
 SHA512:
-  metadata.gz: fa1b4fd17cf2720ce19b49026e4f726252ae386cf153b6d9140c41f24da2efd56108eed7d8905b7d4b47891d111f8e2c28f768426d3b6a75ddeafdb37ad37668
-  data.tar.gz: 30c988f1a026536d4eff28d9365aa9e3d335fe3d70b7033a2b6101d40a8091aaa2816c9c30667cc050f98c2201befbcbeb4ddabbdee3d5189239eec9a75e53bf
+  metadata.gz: 3ed77728ffae77e7e4ff2f84affc023902fabbd2c9cd2adce81cc3b40b88eb02d97f1c6ced8e7bf7323452234274b29229f3ea775618ab3324ecc3752f4130e3
+  data.tar.gz: b7d5c47d5d486014d03ab9829b4f325377aa0f5c07c7e64b5f1692d0d308333352f45ea9eaf3b89f6120a374be8ddfec7fb36d7054b037887ee1d8f9c74a5505

data/lib/dev_metrics/client.rb

@@ -3,6 +3,7 @@ require 'uri'
 require 'date'
 require 'json'
 require 'time'
+require_relative 'query_builder'
 
 module DevMetrics
   class Client
@@ -18,10 +19,17 @@ module DevMetrics
 
     def process(period: Date.today)
       uri = URI.parse(GITHUB_GRAPHQL_API)
-      request = build_request(uri, period)
-      response = execute_request(uri, request)
+      builder = DevMetrics::QueryBuilder.new(@repo_name)
+
+      auth_request = build_request(uri, builder.access_auth_check)
+      auth_response = execute_request(uri, auth_request)
+      check_auth_errors!(auth_response)
+
+      pr_request = build_request(uri, builder.pull_requests_for(period))
+      pr_response = execute_request(uri, pr_request)
+
+      pr_data = parse_response(pr_response)
 
-      pr_data = parse_response(response)
       filtered_prs = exclude_bots(pr_data)
       correction_pr_count = count_correction_prs(filtered_prs)
 
@@ -31,11 +39,10 @@ module DevMetrics
 
     private
 
-    def build_request(uri, period)
+    def build_request(uri, body)
       request = Net::HTTP::Post.new(uri)
-      request["Authorization"] = "Bearer #{@access_token}"
-      request.body = graphql_query(period)
-
+      request['Authorization'] = "Bearer #{@access_token}"
+      request.body = body
       request
     end
 
@@ -44,12 +51,21 @@ module DevMetrics
       Net::HTTP.start(uri.hostname, uri.port, options) { |http| http.request(request) }
     end
 
+    def check_auth_errors!(response)
+      if response.code.to_i == 403 || response.body.include?('FORBIDDEN')
+        raise "Access denied: ensure the access token has permission to access #{@repo_name}"
+      end
+    end
+
     def parse_response(response)
       unless response.is_a?(Net::HTTPSuccess)
         raise "Failed to fetch data: #{response.message} (#{response.code})"
       end
 
-      JSON.parse(response.body).dig('data', 'search', 'edges')
+      data = JSON.parse(response.body)
+      raise "GraphQL error: #{data['errors']}" if data['errors']
+
+      data.dig('data', 'search', 'edges') || []
     end
 
     def exclude_bots(prs)
@@ -58,7 +74,7 @@ module DevMetrics
     end
 
     def count_correction_prs(prs)
-      prs.count { |pr| pr.dig('node', 'headRefName')&.match?(/^#{@fix_branch_names}/) }
+      prs.count { |pr| pr.dig('node', 'headRefName')&.match?(/^#{@fix_branch_names.join('|')}/) }
     end
 
     def calculate_lead_time(prs)
@@ -81,30 +97,6 @@ module DevMetrics
       Time.at(remaining).utc.strftime("#{days}d %H:%M:%S")
     end
 
-    def graphql_query(period)
-      query_string = <<-GRAPHQL
-    {
-      search(query: "repo:#{@repo_name} is:pr merged:#{period}", type: ISSUE, first: 100) {
-        edges {
-          node {
-            ... on PullRequest {
-              url
-              title
-              author {
-                login
-              }
-              mergedAt
-              headRefName
-              publishedAt
-            }
-          }
-        }
-      }
-    }
-      GRAPHQL
-      { "query" => query_string.strip }.to_json
-    end
-
     def output_metrics(period, prs, correction_pr_count)
       formatted_data = format_data(period, prs, correction_pr_count)
 

data/lib/dev_metrics/markdown.rb

@@ -8,12 +8,16 @@ module DevMetrics
     def format_data(period, prs, correction_pr_count)
       output = ""
       output << "| #{period} | #{prs.count} | #{correction_pr_count} | "
-      output << "#{((correction_pr_count.to_f / prs.count) * 100).round(2)}% | "
+      output << "#{correction_calc(correction_pr_count, prs)}% | "
       output << "#{calculate_lead_time(prs)} | "
       output << "[PRs for #{period}](https://github.com/#{@repo_name}/pulls?q=is%3Apr+merged%3A#{period}) |\n"
       output
     end
 
+    def correction_calc(correction_pr_count, prs)
+      return "-" if prs.empty?
+      ((correction_pr_count.to_f / prs.count) * 100).round(2)
+    end
     def output_filename
       "metrics_report.md"
     end

data/lib/dev_metrics/query_builder.rb

@@ -0,0 +1,44 @@
+module DevMetrics
+  class QueryBuilder
+
+    def initialize(repo)
+      @repo = repo
+    end
+    def access_auth_check
+      to_body <<~GRAPHQL
+      {
+         repository(owner: "#{@repo.split('/')[0]}", name: "#{@repo.split('/')[1]}"){
+          id
+        }
+      }
+    GRAPHQL
+    end
+
+    def pull_requests_for(period)
+      to_body <<~GRAPHQL
+      {
+        search(query: "repo:#{@repo} is:pr merged:#{period}", type: ISSUE, first: 100) {
+          edges {
+            node {
+              ... on PullRequest {
+                url
+                title
+                author { login }
+                mergedAt
+                headRefName
+                publishedAt
+              }
+            }
+          }
+        }
+      }
+    GRAPHQL
+    end
+
+    private
+
+    def to_body(query_string)
+      { "query" => query_string.strip }.to_json
+    end
+  end
+end

data/lib/dev_metrics.rb

@@ -23,7 +23,7 @@ module DevMetrics
   end
 
   def self.run(period:, format:)
-    markdown_processor = format.new(@configuration)
-    markdown_processor.process(period: period)
+    client = format.new(@configuration)
+    client.process(period: period)
   end
 end

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: dev_metrics
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - sean2121
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-09-04 00:00:00.000000000 Z
+date: 2025-04-17 00:00:00.000000000 Z
 dependencies: []
-description:
-email:
+description: 
+email: 
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -19,11 +19,12 @@ files:
 - lib/dev_metrics.rb
 - lib/dev_metrics/client.rb
 - lib/dev_metrics/markdown.rb
-homepage:
+- lib/dev_metrics/query_builder.rb
+homepage: 
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -39,7 +40,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.2.33
-signing_key:
+signing_key: 
 specification_version: 4
 summary: A RubyGem for tracking and analyzing development metrics from various repositories.
 test_files: []