rspec-mergify 0.1.4 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e3b9e1b83fe9ad90add227b5618ac7b4594883cf39c0eb58ab150feb0de9d916
-  data.tar.gz: e801b0a101413dea6c0d723e58d79c4a978cffdf252ca35332a64e47c4b75493
+  metadata.gz: 2f6831a7105994678c61d541f3ca3840a618f93ed59561cecd23063d91063d7e
+  data.tar.gz: e6b1528a39efa7de31ef4ded5d44b2479f78521666cf48acb26edc6135580067
 SHA512:
-  metadata.gz: 1a30b290d0860751b9a8be05cc18615415c0f3dc8ae16fbdd9ea877b54688767625e4d931e88f901ecbf8ca2dddc2e02eee457958ca31e7fb8fb305088912b66
-  data.tar.gz: 643aa383dbaa5324534aea467d8220642f3240182e16d57447d73e4dd44e4a0337d4de9514233b13498802057638d4574311bb046d57cd93898ddf1747361b00
+  metadata.gz: acc68c52030c7cf29760e293f99a46faae5c012745f0cd6d9b81c36cbb5c29505b7d16662a1785fd641f3f4ba8e424b592782308ef06356c3694e9bbfe09bddc
+  data.tar.gz: c95fc1c14925f61b63d1888e51e47f88af3ced37565a4964e6c30b913f0cbc6261a212562578f41faef937c1a0665fc0a150b302fd83e270bc991bcefb76b663

data/lib/mergify/rspec/ci_insights.rb

@@ -3,13 +3,8 @@
 require 'securerandom'
 require 'opentelemetry-sdk'
 require_relative 'utils'
+require_relative 'native'
 require_relative 'synchronous_batch_span_processor'
-require_relative 'resources/ci'
-require_relative 'resources/git'
-require_relative 'resources/github_actions'
-require_relative 'resources/jenkins'
-require_relative 'resources/buildkite'
-require_relative 'resources/mergify'
 require_relative 'resources/rspec'
 
 module Mergify
@@ -26,7 +21,7 @@ module Mergify
       # rubocop:disable-next Metrics/MethodLength
       def initialize
         @token = ENV.fetch('MERGIFY_TOKEN', nil)
-        @repo_name = Utils.repository_name
+        @repo_name = Native.detect_repository_name
         @api_url = ENV.fetch('MERGIFY_API_URL', 'https://api.mergify.com')
         @test_run_id = SecureRandom.hex(8)
         @tracer_provider = nil
@@ -95,22 +90,19 @@ module Mergify
         [processor, exp]
       end
 
-      # rubocop:disable-next Metrics/MethodLength
+      # The cicd.* and vcs.* attributes come from the Rust core, which every
+      # Mergify test client shares, so a provider gains them everywhere at once.
+      # What stays here is what only Ruby knows: the test framework, and the id
+      # this run invented for itself.
       def build_resource
         resources = [
-          Resources::CI.detect,
-          Resources::Git.detect,
-          Resources::GitHubActions.detect,
-          Resources::Jenkins.detect,
-          Resources::Buildkite.detect,
-          Resources::Mergify.detect,
-          Resources::RSpec.detect
+          OpenTelemetry::SDK::Resources::Resource.create(Native.detect_attributes),
+          Resources::RSpec.detect,
+          OpenTelemetry::SDK::Resources::Resource.create('test.run.id' => @test_run_id)
         ]
-        base = resources.reduce(OpenTelemetry::SDK::Resources::Resource.create({})) do |merged, r|
+        resources.reduce(OpenTelemetry::SDK::Resources::Resource.create({})) do |merged, r|
           merged.merge(r)
         end
-        run_id_resource = OpenTelemetry::SDK::Resources::Resource.create('test.run.id' => @test_run_id)
-        base.merge(run_id_resource)
       end
 
       def extract_branch_name(resource)

data/lib/mergify/rspec/flaky_detection.rb

@@ -1,10 +1,9 @@
 # frozen_string_literal: true
 
-require 'net/http'
-require 'json'
-require 'uri'
 require 'set'
 require_relative 'utils'
+require_relative 'native'
+require_relative 'version'
 
 module Mergify
   module RSpec
@@ -74,34 +73,25 @@ module Mergify
         @tests_to_process = []
         @budget = 0.0
 
-        fetch_context
-        validate!
+        @context = fetch_context
+        raise FlakyDetectionDisabledError unless Native::Budget.should_run(@context, @mode)
       end
 
-      # rubocop:disable-next Metrics/MethodLength,Metrics/AbcSize
+      # Which tests this session reruns, and how long it may spend doing it,
+      # both come from the shared budget engine -- so a Ruby suite and a Python
+      # one facing the same context spend the same time.
+      #
+      # The engine sizes the budget from the existing tests *in this session*,
+      # where this class counted every existing test the context knew about. A
+      # session running part of a suite was handed the whole suite's budget; it
+      # now gets its own.
       def prepare_for_session(test_ids)
-        existing = Set.new(@context[:existing_test_names])
-        unhealthy = Set.new(@context[:unhealthy_test_names])
-
-        @tests_to_process =
-          if @mode == 'new'
-            test_ids.reject { |id| existing.include?(id) }
-          else
-            test_ids.select { |id| unhealthy.include?(id) }
-          end
+        plan = Native::Budget.compute(@context, @mode, test_ids, [])
 
-        budget_ratio = if @mode == 'new'
-                         @context[:budget_ratio_for_new_tests]
-                       else
-                         @context[:budget_ratio_for_unhealthy_tests]
-                       end
-
-        mean_duration_s = @context[:existing_tests_mean_duration_ms] / 1000.0
-        existing_count = @context[:existing_test_names].size
-        min_budget_s = @context[:min_budget_duration_ms] / 1000.0
-
-        ratio_budget = budget_ratio * mean_duration_s * existing_count
-        @budget = [ratio_budget, min_budget_s].max
+        @tests_to_process = plan['tests_to_process']
+        # The engine works in milliseconds; everything downstream compares
+        # against RSpec's durations, which are seconds.
+        @budget = plan['available_budget_ms'] / 1000.0
       end
 
       # rubocop:disable-next Metrics/MethodLength
@@ -113,7 +103,7 @@ module Mergify
 
         return unless @tests_to_process.include?(test_id)
 
-        if test_id.length > @context[:max_test_name_length]
+        if test_id.length > @context['max_test_name_length']
           @over_length_tests.add(test_id)
           return
         end
@@ -153,7 +143,7 @@ module Mergify
         return false unless @metrics.key?(test_id)
 
         metrics = @metrics[test_id]
-        min_exec = @context[:min_test_execution_count]
+        min_exec = @context['min_test_execution_count']
         (metrics.initial_duration * min_exec) > metrics.remaining_time
       end
 
@@ -161,7 +151,7 @@ module Mergify
         return false unless @metrics.key?(test_id)
 
         metrics = @metrics[test_id]
-        metrics.will_exceed_deadline? || metrics.rerun_count >= @context[:max_test_execution_count]
+        metrics.will_exceed_deadline? || metrics.rerun_count >= @context['max_test_execution_count']
       end
 
       def test_metrics(test_id)
@@ -196,54 +186,16 @@ module Mergify
 
       private
 
-      # rubocop:disable-next Metrics/AbcSize,Metrics/MethodLength
+      # A nil context means the repository has not opted into flaky detection,
+      # which is the expected default rather than a failure.
       def fetch_context
-        owner, repo = Utils.split_full_repo_name(@full_repository_name)
-        uri = URI("#{@url}/v1/ci/#{owner}/repositories/#{repo}/flaky-detection-context")
-
-        http = Net::HTTP.new(uri.host, uri.port)
-        http.use_ssl = uri.scheme == 'https'
-        http.open_timeout = 10
-        http.read_timeout = 10
-
-        request = Net::HTTP::Get.new(uri)
-        request['Authorization'] = "Bearer #{@token}"
-
-        response = http.request(request)
-        case response.code.to_i
-        when 200
-          parse_context(response.body)
-        when 404
-          # A 404 means the repository has not opted into flaky detection; this
-          # is the expected default, not an error.
-          raise FlakyDetectionDisabledError
-        else
-          raise "Mergify API returned HTTP #{response.code}"
-        end
-      end
-
-      # rubocop:disable-next Metrics/MethodLength,Metrics/AbcSize
-      def parse_context(body)
-        data = JSON.parse(body, symbolize_names: true)
-        @context = {
-          budget_ratio_for_new_tests: data[:budget_ratio_for_new_tests].to_f,
-          budget_ratio_for_unhealthy_tests: data[:budget_ratio_for_unhealthy_tests].to_f,
-          existing_test_names: Array(data[:existing_test_names]),
-          existing_tests_mean_duration_ms: data[:existing_tests_mean_duration_ms].to_f,
-          unhealthy_test_names: Array(data[:unhealthy_test_names]),
-          max_test_execution_count: data[:max_test_execution_count].to_i,
-          max_test_name_length: data[:max_test_name_length].to_i,
-          min_budget_duration_ms: data[:min_budget_duration_ms].to_f,
-          min_test_execution_count: data[:min_test_execution_count].to_i
-        }
-      end
+        raise FlakyDetectionDisabledError unless Native.available?
 
-      def validate!
-        return unless @mode == 'new' && @context[:existing_test_names].empty?
+        owner, repo = Utils.split_full_repo_name(@full_repository_name)
+        context = Native::Client.new(@url, @token, owner, repo, VERSION).fetch_flaky_context
+        raise FlakyDetectionDisabledError if context.nil?
 
-        # Without a baseline, `new` mode would treat every test as new and rerun
-        # the whole suite. Skip instead of surfacing an error.
-        raise FlakyDetectionDisabledError
+        context
       end
 
       def remaining_budget

data/lib/mergify/rspec/native.rb

@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+module Mergify
+  module RSpec
+    # The Rust extension over mergify-ci-core: CI detection shared with the
+    # pytest and TypeScript clients.
+    #
+    # Loading is best-effort by design. A precompiled gem carries one extension
+    # per Ruby under `<ruby>/`, the source gem compiles one alongside this file,
+    # and a platform we publish neither for gets neither. Detection is one input
+    # to telemetry, not the point of the gem, so an absent extension degrades
+    # what we can report rather than breaking the suite under test -- the same
+    # fail-open posture the napi binding takes when a platform has no prebuilt
+    # binary. `load_error` keeps the reason, for callers that want to say so.
+    module Native
+      # Raised when a Mergify API call fails outright.
+      #
+      # Distinct from StandardError on purpose: callers degrade on an API
+      # failure, and a bare rescue there would swallow genuine bugs in the
+      # binding as though the backend were down.
+      class ApiError < StandardError; end
+
+      class << self
+        # The LoadError that prevented the extension loading, or nil.
+        attr_accessor :load_error
+
+        def available?
+          load_error.nil?
+        end
+      end
+    end
+  end
+end
+
+begin
+  # Precompiled gems ship lib/mergify/rspec/<ruby>/mergify_ci.<dlext>.
+  require_relative "#{RUBY_VERSION.to_f}/mergify_ci"
+rescue LoadError
+  begin
+    # Source gem, and any local `rake compile`.
+    require_relative 'mergify_ci'
+  rescue LoadError => e
+    Mergify::RSpec::Native.load_error = e
+  end
+end
+
+unless Mergify::RSpec::Native.available?
+  # Stand in for the extension so callers can just ask, and get the same answer
+  # they would get from a machine that is not in CI. Branching on availability
+  # at every call site would only spread the same nil back through the caller.
+  module Mergify
+    module RSpec
+      module Native
+        class << self
+          def detect_provider
+            nil
+          end
+
+          def detect_repository_name
+            nil
+          end
+
+          def detect_attributes
+            {}
+          end
+        end
+      end
+    end
+  end
+end

data/lib/mergify/rspec/quarantine.rb

@@ -1,14 +1,19 @@
 # frozen_string_literal: true
 
-require 'net/http'
-require 'json'
-require 'uri'
 require 'set'
 require_relative 'utils'
+require_relative 'native'
+require_relative 'version'
 
 module Mergify
   module RSpec
     # Fetches quarantined test names from the Mergify API and tracks which are used.
+    #
+    # The fetch itself -- pagination, the RFC 8288 `next` links, the status
+    # codes that mean "not subscribed" rather than "broken" -- belongs to the
+    # shared Rust client now, so every Mergify test client reads a quarantine
+    # list the same way. What stays here is what RSpec cares about: which of
+    # those tests this session actually ran, and the report at the end.
     class Quarantine
       attr_reader :quarantined_tests, :init_error_msg
 
@@ -19,10 +24,7 @@ module Mergify
         @used_tests = Set.new
         @init_error_msg = nil
 
-        owner, repo = Utils.split_full_repo_name(repo_name)
-        fetch_quarantined_tests(api_url, token, owner, repo, branch_name)
-      rescue Utils::InvalidRepositoryFullNameError => e
-        @init_error_msg = e.message
+        fetch(api_url, token, branch_name)
       end
 
       def include?(example_id)
@@ -53,80 +55,21 @@ module Mergify
 
       private
 
-      def fetch_quarantined_tests(api_url, token, owner, repo, branch_name)
-        uri = URI("#{api_url}/v1/ci/#{owner}/repositories/#{repo}/quarantines")
-        uri.query = URI.encode_www_form(branch: branch_name, per_page: 100)
-        collected = walk_paginated_quarantines(uri, token)
-        @quarantined_tests = collected if collected
-      rescue Net::OpenTimeout, Net::ReadTimeout, Errno::ECONNREFUSED, SocketError => e
-        @init_error_msg = "Failed to connect to Mergify API: #{e.message}"
-      rescue JSON::ParserError => e
-        @init_error_msg = "Mergify API returned a malformed quarantine list: #{e.message}"
-      end
-
-      # Follows the RFC 5988 `next` link until exhausted. Returns the full list
-      # on success, or `nil` when the run was aborted (subscription missing or
-      # an error already recorded in @init_error_msg).
-      # rubocop:disable-next Metrics/MethodLength,Metrics/AbcSize
-      def walk_paginated_quarantines(uri, token)
-        collected = []
-        # Guard against a server returning a `next` link that loops back to a
-        # URL we have already fetched.
-        seen = Set.new
-        while uri
-          if seen.include?(uri.to_s)
-            @init_error_msg = 'Mergify API returned a cyclic `next` link, aborting.'
-            return nil
-          end
-          seen.add(uri.to_s)
-
-          response = perform_request(uri, token)
-          case response.code.to_i
-          when 200
-            data = JSON.parse(response.body)
-            collected.concat(data.fetch('quarantined_tests', []).map { |t| t['test_name'] })
-            next_url = parse_next_link(response['Link'])
-            uri = next_url ? URI(next_url) : nil
-          when 402
-            return nil
-          else
-            @init_error_msg = "Mergify API returned HTTP #{response.code}"
-            return nil
-          end
-        end
-        collected
-      end
-
-      def perform_request(uri, token)
-        http = Net::HTTP.new(uri.host, uri.port)
-        http.use_ssl = uri.scheme == 'https'
-        http.open_timeout = 10
-        http.read_timeout = 10
-
-        request = Net::HTTP::Get.new(uri)
-        request['Authorization'] = "Bearer #{token}"
-        http.request(request)
-      end
-
-      # Parses RFC 8288 Link headers tolerantly: accepts both quoted
-      # (`rel="next"`) and token (`rel=next`) forms, and matches when `next`
-      # is one of several space-separated rel-types (`rel="next prev"`).
-      def parse_next_link(link_header)
-        return nil if link_header.nil? || link_header.empty?
-
-        link_header.split(',').each do |part|
-          match = part.strip.match(/\A<([^>]+)>\s*;\s*(.+)\z/)
-          next unless match && next_rel?(match[2])
-
-          return match[1]
+      # A nil list means the repository has no quarantine subscription, which is
+      # not an error: the session simply quarantines nothing. Anything that went
+      # genuinely wrong is recorded and the suite carries on -- this plugin has
+      # never let the backend fail a test run.
+      def fetch(api_url, token, branch_name)
+        unless Native.available?
+          @init_error_msg = "Mergify native extension unavailable: #{Native.load_error}"
+          return
         end
-        nil
-      end
 
-      def next_rel?(params)
-        params.scan(/rel\s*=\s*(?:"([^"]+)"|([^\s;,]+))/i).any? do |quoted, token|
-          (quoted || token).split.include?('next')
-        end
+        owner, repo = Utils.split_full_repo_name(@repo_name)
+        client = Native::Client.new(api_url, token, owner, repo, VERSION)
+        @quarantined_tests = client.fetch_quarantine(branch_name) || []
+      rescue Utils::InvalidRepositoryFullNameError, Native::ApiError => e
+        @init_error_msg = e.message
       end
     end
   end

data/lib/mergify/rspec/utils.rb

@@ -1,24 +1,21 @@
 # frozen_string_literal: true
 
-require 'open3'
-
 module Mergify
   module RSpec
     # Utility methods shared across the rspec-mergify gem.
+    #
+    # CI detection used to live here -- the provider table, the git shelling
+    # out, the per-provider environment mappings. It is the Rust core's now, via
+    # the extension in Mergify::RSpec::Native, so that every Mergify test client
+    # detects identically. What is left is what is genuinely Ruby's: parsing a
+    # repository name the API needs split, and deciding whether the plugin
+    # should switch itself on at all.
     module Utils
       module_function
 
       # Raised when a repository full name (owner/repo) is malformed.
       class InvalidRepositoryFullNameError < StandardError; end
 
-      SUPPORTED_CIS = {
-        'GITHUB_ACTIONS' => :github_actions,
-        'CIRCLECI' => :circleci,
-        'JENKINS_URL' => :jenkins,
-        'BUILDKITE' => :buildkite,
-        '_RSPEC_MERGIFY_TEST' => :rspec_mergify_suite
-      }.freeze
-
       TRUTHY_STRINGS = %w[y yes t true on 1].freeze
       FALSY_STRINGS  = %w[n no f false off 0].freeze
 
@@ -40,44 +37,14 @@ module Mergify
 
       # Returns true when the suite is running inside CI or when
       # RSPEC_MERGIFY_ENABLE is set to a truthy value.
+      #
+      # Deliberately not the core's provider detection: this asks whether the
+      # plugin should run, which an unrecognised CI or a developer setting
+      # RSPEC_MERGIFY_ENABLE both answer yes to.
       def in_ci?
         env_truthy?('CI') || env_truthy?('RSPEC_MERGIFY_ENABLE')
       end
 
-      # Evaluates whether a CI environment variable should be considered enabled.
-      # rubocop:disable-next Metrics/MethodLength
-      def ci_provider
-        SUPPORTED_CIS.each do |envvar, name|
-          next unless ENV.key?(envvar)
-
-          enabled =
-            begin
-              strtobool(ENV.fetch(envvar, ''))
-            rescue ArgumentError
-              !ENV.fetch(envvar, '').strip.empty?
-            end
-
-          return name if enabled
-        end
-        nil
-      end
-
-      # Parse a git remote URL (SSH or HTTPS) into "owner/repo" form.
-      # Returns nil when the URL cannot be recognised.
-      def repository_name_from_url(url)
-        # SSH: git@github.com:owner/repo.git
-        if (m = url.match(%r{\Agit@[\w.-]+:(?<full_name>[\w.-]+/[\w.-]+?)(?:\.git)?/?$}))
-          return m[:full_name]
-        end
-
-        # HTTPS/HTTP with optional host (and optional port)
-        if (m = url.match(%r{\A(?:https?://[\w.-]+(?::\d+)?/)?(?<full_name>[\w.-]+/[\w.-]+)/?\z}))
-          return m[:full_name]
-        end
-
-        nil
-      end
-
       # Split "owner/repo" into [owner, repo].
       # Raises InvalidRepositoryFullNameError when the format is wrong.
       def split_full_repo_name(full_repo_name)
@@ -86,55 +53,6 @@ module Mergify
 
         raise InvalidRepositoryFullNameError, "Invalid repository name: #{full_repo_name}"
       end
-
-      # Run a git subcommand via Open3.
-      # Returns stripped stdout on success, nil on failure.
-      def git(*args)
-        stdout, status = Open3.capture2('git', *args, err: File::NULL)
-        status.success? ? stdout.strip : nil
-      rescue StandardError
-        nil
-      end
-
-      # Build an attribute hash from a mapping of
-      # { attr_name => [cast_method_symbol, env_var_name_or_callable] }.
-      # Attributes whose env var is unset or whose callable returns nil are omitted.
-      def get_attributes(mapping)
-        mapping.each_with_object({}) do |(attr, (cast, env_or_callable)), result|
-          value = env_or_callable.respond_to?(:call) ? env_or_callable.call : ENV.fetch(env_or_callable, nil)
-
-          next if value.nil?
-          next if value.respond_to?(:empty?) && value.empty?
-
-          result[attr] = value.public_send(cast)
-        end
-      end
-
-      # Detect the repository name using CI environment variables or a git
-      # remote fallback.
-      # rubocop:disable-next Metrics/MethodLength,Metrics/CyclomaticComplexity
-      def repository_name
-        provider = ci_provider
-
-        case provider
-        when :jenkins
-          url = ENV.fetch('GIT_URL', nil)
-          return repository_name_from_url(url) if url
-        when :github_actions
-          return ENV.fetch('GITHUB_REPOSITORY', nil)
-        when :circleci
-          url = ENV.fetch('CIRCLE_REPOSITORY_URL', nil)
-          return repository_name_from_url(url) if url
-        when :buildkite
-          url = ENV.fetch('BUILDKITE_REPO', nil)
-          return repository_name_from_url(url) if url
-        when :rspec_mergify_suite
-          return 'Mergifyio/rspec-mergify'
-        end
-
-        url = git('config', '--get', 'remote.origin.url')
-        repository_name_from_url(url) if url
-      end
     end
   end
 end

data/lib/mergify/rspec/version.rb

@@ -8,6 +8,6 @@ module Mergify
     # which resolved against whatever repository happened to be the working
     # directory at load time -- the monorepo's namespaced tags here, the user's
     # own tags once the gem was installed.
-    VERSION = '0.1.4'
+    VERSION = '0.2.0'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rspec-mergify
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.2.0
 platform: ruby
 authors:
 - Mergify
@@ -66,13 +66,8 @@ files:
 - lib/mergify/rspec/configuration.rb
 - lib/mergify/rspec/flaky_detection.rb
 - lib/mergify/rspec/formatter.rb
+- lib/mergify/rspec/native.rb
 - lib/mergify/rspec/quarantine.rb
-- lib/mergify/rspec/resources/buildkite.rb
-- lib/mergify/rspec/resources/ci.rb
-- lib/mergify/rspec/resources/git.rb
-- lib/mergify/rspec/resources/github_actions.rb
-- lib/mergify/rspec/resources/jenkins.rb
-- lib/mergify/rspec/resources/mergify.rb
 - lib/mergify/rspec/resources/rspec.rb
 - lib/mergify/rspec/synchronous_batch_span_processor.rb
 - lib/mergify/rspec/utils.rb

data/lib/mergify/rspec/resources/buildkite.rb

@@ -1,54 +0,0 @@
-# frozen_string_literal: true
-
-require 'opentelemetry-sdk'
-require_relative '../utils'
-require_relative 'git'
-
-module Mergify
-  module RSpec
-    module Resources
-      # Detects OpenTelemetry Resource attributes for Buildkite.
-      module Buildkite
-        module_function
-
-        BUILDKITE_MAPPING = {
-          'cicd.pipeline.name' => [:to_s, 'BUILDKITE_PIPELINE_SLUG'],
-          'cicd.pipeline.task.name' => [
-            :to_s,
-            lambda {
-              label = ENV.fetch('BUILDKITE_LABEL', nil)
-              label && !label.empty? ? label : ENV.fetch('BUILDKITE_STEP_KEY', nil)
-            }
-          ],
-          'cicd.pipeline.run.id' => [:to_s, 'BUILDKITE_BUILD_ID'],
-          'cicd.pipeline.run.url' => [:to_s, 'BUILDKITE_BUILD_URL'],
-          'cicd.pipeline.run.attempt' => [
-            :to_i,
-            -> { ENV.fetch('BUILDKITE_RETRY_COUNT', '0').to_i + 1 }
-          ],
-          'cicd.pipeline.runner.name' => [:to_s, 'BUILDKITE_AGENT_NAME'],
-          'vcs.ref.head.name' => [:to_s, 'BUILDKITE_BRANCH'],
-          'vcs.ref.base.name' => [:to_s, 'BUILDKITE_PULL_REQUEST_BASE_BRANCH'],
-          'vcs.ref.head.revision' => [:to_s, 'BUILDKITE_COMMIT'],
-          'vcs.repository.url.full' => [:to_s, 'BUILDKITE_REPO'],
-          'vcs.repository.name' => [
-            :to_s,
-            lambda {
-              url = ENV.fetch('BUILDKITE_REPO', nil)
-              Utils.repository_name_from_url(url) if url
-            }
-          ]
-        }.freeze
-
-        def detect
-          return OpenTelemetry::SDK::Resources::Resource.create({}) if Utils.ci_provider != :buildkite
-
-          git_attrs = Utils.get_attributes(Git::GIT_MAPPING)
-          buildkite_attrs = Utils.get_attributes(BUILDKITE_MAPPING)
-          merged = git_attrs.merge(buildkite_attrs)
-          OpenTelemetry::SDK::Resources::Resource.create(merged)
-        end
-      end
-    end
-  end
-end

data/lib/mergify/rspec/resources/ci.rb

@@ -1,24 +0,0 @@
-# frozen_string_literal: true
-
-require 'opentelemetry-sdk'
-require_relative '../utils'
-
-module Mergify
-  module RSpec
-    module Resources
-      # Detects OpenTelemetry Resource attributes for the CI provider.
-      module CI
-        module_function
-
-        def detect
-          provider = Utils.ci_provider
-          return OpenTelemetry::SDK::Resources::Resource.create({}) if provider.nil?
-
-          OpenTelemetry::SDK::Resources::Resource.create(
-            'cicd.provider.name' => provider.to_s
-          )
-        end
-      end
-    end
-  end
-end

data/lib/mergify/rspec/resources/git.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-require 'opentelemetry-sdk'
-require_relative '../utils'
-
-module Mergify
-  module RSpec
-    module Resources
-      # Detects OpenTelemetry Resource attributes from git.
-      module Git
-        module_function
-
-        GIT_MAPPING = {
-          'vcs.ref.head.name' => [:to_s, -> { Utils.git('rev-parse', '--abbrev-ref', 'HEAD') }],
-          'vcs.ref.head.revision' => [:to_s, -> { Utils.git('rev-parse', 'HEAD') }],
-          'vcs.repository.url.full' => [:to_s, -> { Utils.git('config', '--get', 'remote.origin.url') }],
-          'vcs.repository.name' => [
-            :to_s,
-            lambda {
-              url = Utils.git('config', '--get', 'remote.origin.url')
-              Utils.repository_name_from_url(url) if url
-            }
-          ]
-        }.freeze
-
-        def detect
-          return OpenTelemetry::SDK::Resources::Resource.create({}) if Utils.ci_provider.nil?
-
-          attributes = Utils.get_attributes(GIT_MAPPING)
-          OpenTelemetry::SDK::Resources::Resource.create(attributes)
-        end
-      end
-    end
-  end
-end

data/lib/mergify/rspec/resources/github_actions.rb

@@ -1,60 +0,0 @@
-# frozen_string_literal: true
-
-require 'json'
-require 'opentelemetry-sdk'
-require_relative '../utils'
-
-module Mergify
-  module RSpec
-    module Resources
-      # Detects OpenTelemetry Resource attributes for GitHub Actions.
-      module GitHubActions
-        module_function
-
-        def detect
-          return OpenTelemetry::SDK::Resources::Resource.create({}) if Utils.ci_provider != :github_actions
-
-          attributes = Utils.get_attributes(GHA_MAPPING)
-          OpenTelemetry::SDK::Resources::Resource.create(attributes)
-        end
-
-        GHA_MAPPING = {
-          'cicd.pipeline.name' => [:to_s, 'GITHUB_WORKFLOW'],
-          'cicd.pipeline.task.name' => [:to_s, 'GITHUB_JOB'],
-          'cicd.pipeline.run.id' => [:to_i, 'GITHUB_RUN_ID'],
-          'cicd.pipeline.run.attempt' => [:to_i, 'GITHUB_RUN_ATTEMPT'],
-          'cicd.pipeline.runner.name' => [:to_s, 'RUNNER_NAME'],
-          'vcs.ref.head.name' => [:to_s, -> { head_ref_name }],
-          'vcs.ref.head.type' => [:to_s, 'GITHUB_REF_TYPE'],
-          'vcs.ref.base.name' => [:to_s, 'GITHUB_BASE_REF'],
-          'vcs.repository.name' => [:to_s, 'GITHUB_REPOSITORY'],
-          'vcs.repository.id' => [:to_i, 'GITHUB_REPOSITORY_ID'],
-          'vcs.repository.url.full' => [:to_s, -> { repository_url }],
-          'vcs.ref.head.revision' => [:to_s, -> { head_sha }]
-        }.freeze
-
-        def head_ref_name
-          ref = ENV.fetch('GITHUB_HEAD_REF', '')
-          ref.empty? ? ENV.fetch('GITHUB_REF_NAME', nil) : ref
-        end
-
-        def repository_url
-          server = ENV.fetch('GITHUB_SERVER_URL', nil)
-          repo = ENV.fetch('GITHUB_REPOSITORY', nil)
-          "#{server}/#{repo}" if server && repo
-        end
-
-        def head_sha
-          if ENV.fetch('GITHUB_EVENT_NAME', nil) == 'pull_request'
-            event_path = ENV.fetch('GITHUB_EVENT_PATH', nil)
-            if event_path && File.file?(event_path)
-              event = JSON.parse(File.read(event_path))
-              return event.dig('pull_request', 'head', 'sha').to_s
-            end
-          end
-          ENV.fetch('GITHUB_SHA', nil)
-        end
-      end
-    end
-  end
-end

data/lib/mergify/rspec/resources/jenkins.rb

@@ -1,55 +0,0 @@
-# frozen_string_literal: true
-
-require 'opentelemetry-sdk'
-require_relative '../utils'
-require_relative 'git'
-
-module Mergify
-  module RSpec
-    module Resources
-      # Detects OpenTelemetry Resource attributes for Jenkins.
-      module Jenkins
-        module_function
-
-        GIT_BRANCH_PREFIXES = %w[origin/ refs/heads/].freeze
-
-        JENKINS_MAPPING = {
-          'cicd.pipeline.name' => [:to_s, 'JOB_NAME'],
-          'cicd.pipeline.task.name' => [:to_s, 'JOB_NAME'],
-          'cicd.pipeline.run.id' => [:to_s, 'BUILD_ID'],
-          'cicd.pipeline.run.url' => [:to_s, 'BUILD_URL'],
-          'cicd.pipeline.runner.name' => [:to_s, 'NODE_NAME'],
-          'vcs.ref.head.name' => [:to_s, -> { branch }],
-          'vcs.ref.head.revision' => [:to_s, 'GIT_COMMIT'],
-          'vcs.repository.url.full' => [:to_s, 'GIT_URL'],
-          'vcs.repository.name' => [
-            :to_s,
-            lambda {
-              url = ENV.fetch('GIT_URL', nil)
-              Utils.repository_name_from_url(url) if url
-            }
-          ]
-        }.freeze
-
-        def detect
-          return OpenTelemetry::SDK::Resources::Resource.create({}) if Utils.ci_provider != :jenkins
-
-          git_attrs = Utils.get_attributes(Git::GIT_MAPPING)
-          jenkins_attrs = Utils.get_attributes(JENKINS_MAPPING)
-          merged = git_attrs.merge(jenkins_attrs)
-          OpenTelemetry::SDK::Resources::Resource.create(merged)
-        end
-
-        def branch
-          raw = ENV.fetch('GIT_BRANCH', nil)
-          return nil unless raw
-
-          GIT_BRANCH_PREFIXES.each do |prefix|
-            return raw[prefix.length..] if raw.start_with?(prefix)
-          end
-          raw
-        end
-      end
-    end
-  end
-end

data/lib/mergify/rspec/resources/mergify.rb

@@ -1,24 +0,0 @@
-# frozen_string_literal: true
-
-require 'opentelemetry-sdk'
-require_relative '../utils'
-
-module Mergify
-  module RSpec
-    module Resources
-      # Detects OpenTelemetry Resource attributes for Mergify-specific fields.
-      module Mergify
-        module_function
-
-        MERGIFY_MAPPING = {
-          'mergify.test.job.name' => [:to_s, 'MERGIFY_TEST_JOB_NAME']
-        }.freeze
-
-        def detect
-          attributes = Utils.get_attributes(MERGIFY_MAPPING)
-          OpenTelemetry::SDK::Resources::Resource.create(attributes)
-        end
-      end
-    end
-  end
-end