Add detection for version update loops

Author: Taucher2003

lib/pyxis/github_client.rb

@@ -23,6 +23,14 @@ def octokit(instance = :release_tools)
         CLIENT_CONFIGS[instance][:octokit] ||= create_octokit(instance)
       end
 
+      def without_auto_pagination(octokit)
+        current_auto_paginate = octokit.instance_variable_get(:@auto_paginate)
+        octokit.instance_variable_set(:@auto_paginate, false)
+        yield octokit
+      ensure
+        octokit.instance_variable_set(:@auto_paginate, current_auto_paginate)
+      end
+
       private
 
       def create_octokit(instance)

lib/pyxis/managed_versioning/component_updater.rb

@@ -61,10 +61,12 @@ def update_component_version(current_version, new_version)
           GithubClient.octokit.branch(Project::Reticulum.github_path, Project::Reticulum.default_branch).commit.sha
         )
 
+        update_title = "Update #{component.component_name} version to #{present_sha(new_version)}"
+
         GithubClient.octokit.update_contents(
           Project::Reticulum.github_path,
           version_file,
-          "Update #{component.component_name} version to #{present_sha(new_version)}",
+          update_title,
           version_file_content.sha,
           new_version,
           branch: update_branch
@@ -76,7 +78,7 @@ def update_component_version(current_version, new_version)
           Project::Reticulum.github_path,
           Project::Reticulum.default_branch,
           update_branch,
-          "Update #{component.component_name} version to #{present_sha(new_version)}",
+          update_title,
           <<~DESCRIPTION
             Update #{component.component_name} to #{new_version_link} as part of managed versioning
 
@@ -85,7 +87,16 @@ def update_component_version(current_version, new_version)
         )
         logger.info('Created pull request', pull_request_url: pr.html_url)
 
-        Pyxis::Services::AutoMergeService.new(Project::Reticulum, pr).execute
+        if version_update_loop?(update_title)
+          GithubClient.octokit.add_comment(
+            Project::Reticulum.github_path,
+            pr.number,
+            '@code0-tech/delivery This component had already been updated to this version in the past. ' \
+            'Please check why this component is getting updated to this version again.'
+          )
+        else
+          Pyxis::Services::AutoMergeService.new(Project::Reticulum, pr).execute
+        end
       end
 
       def update_branch_exists?
@@ -124,6 +135,16 @@ def filter_for_passing_checks(commits)
         filtered_commits
       end
 
+      def version_update_loop?(update_title)
+        commits = GithubClient.without_auto_pagination(GithubClient.octokit) do |octokit|
+          octokit.list_commits(
+            Project::Reticulum.github_path,
+            path: version_file
+          )
+        end
+        commits.reverse.any? { |commit| commit.commit.message == update_title }
+      end
+
       def present_sha(sha)
         Presenter::CommitSha.new(sha).as_short
       end