gitlab-housekeeper 0.1.0.codechallenge.3c718d.pre

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: a8df72541c3c16730fc0e2b3537ca0eab3ee820fc4039e662b098567c6a1dea2
+  data.tar.gz: 37893d59abb0a19eb9fc03528273f2b1b475e47af40923f372dcdcaa50869de4
+SHA512:
+  metadata.gz: a6d10df326d70b9b9001c3b101a5c4f877159d99d7aeb8774b0d820045ab4352b2869551e528abb51c01d8d8ba9e7db67b572848af54faffbecfb206f28e4ad9
+  data.tar.gz: a2c679dda31d5aceffa7b45ef9a7c8921b9f893f7a1fcf120fef244643477d737a229a6d9e8d7193921ad4a02eac70e45b2a52874261ba70815ca19c71380ede

data/bin/gitlab-housekeeper

@@ -0,0 +1,36 @@
+#!/usr/bin/env ruby
+
+require "optparse"
+require 'gitlab/housekeeper'
+
+options = {}
+
+OptionParser.new do |opts|
+  opts.banner = 'Creates merge requests that can be inferred from the current state of the codebase'
+
+  opts.on('-b=BRANCH', '--target-branch=BRANCH', String, 'Target branch to use. Defaults to master.') do |branch|
+    options[:target_branch] = branch
+  end
+
+  opts.on('-m=M', '--max-mrs=M', Integer, 'Limit of MRs to create. Defaults to 1.') do |m|
+    options[:max_mrs] = m
+  end
+
+  opts.on('-d', '--dry-run', 'Dry-run only. Print the MR titles, descriptions and diffs') do
+    options[:dry_run] = true
+  end
+
+  opts.on('-k OverdueFinalizeBackgroundMigration,AnotherKeep', '--keeps OverdueFinalizeBackgroundMigration,AnotherKeep', Array, 'Require keeps specified') do |k|
+    options[:keeps] = k
+  end
+
+  opts.on('--filter-identifiers some-identifier-regex,another-regex', Array, 'Skip any changes where none of the identifiers match these regexes. The identifiers is an array, so at least one element must match at least one regex.') do |filters|
+    options[:filter_identifiers] = filters.map { |f| Regexp.new(f) }
+  end
+
+  opts.on('-h', '--help', 'Prints this help') do
+    abort opts.to_s
+  end
+end.parse!
+
+Gitlab::Housekeeper::Runner.new(**options).run

data/lib/gitlab/housekeeper/change.rb

@@ -0,0 +1,61 @@
+# frozen_string_literal: true
+
+module Gitlab
+  module Housekeeper
+    class Change
+      attr_accessor :identifiers,
+        :title,
+        :description,
+        :changed_files,
+        :labels,
+        :keep_class,
+        :changelog_type,
+        :mr_web_url
+      attr_reader :reviewers
+
+      def initialize
+        @labels = []
+        @reviewers = []
+      end
+
+      def reviewers=(reviewers)
+        @reviewers = Array(reviewers)
+      end
+
+      def mr_description
+        <<~MARKDOWN
+        #{description}
+
+        This change was generated by
+        [gitlab-housekeeper](https://gitlab.com/gitlab-org/gitlab/-/tree/master/gems/gitlab-housekeeper)
+        using the #{keep_class} keep.
+
+        To provide feedback on your experience with `gitlab-housekeeper` please comment in
+        <https://gitlab.com/gitlab-org/gitlab/-/issues/442003>.
+        MARKDOWN
+      end
+
+      def commit_message
+        <<~MARKDOWN
+        #{title}
+
+        #{mr_description}
+
+        Changelog: #{changelog_type || 'other'}
+        MARKDOWN
+      end
+
+      def matches_filters?(filters)
+        filters.all? do |filter|
+          identifiers.any? do |identifier|
+            identifier.match?(filter)
+          end
+        end
+      end
+
+      def valid?
+        @identifiers && @title && @description && @changed_files
+      end
+    end
+  end
+end

data/lib/gitlab/housekeeper/git.rb

@@ -0,0 +1,89 @@
+# frozen_string_literal: true
+
+require 'logger'
+require 'gitlab/housekeeper/shell'
+
+module Gitlab
+  module Housekeeper
+    class Git
+      def initialize(logger:, branch_from: 'master')
+        @logger = logger
+        @branch_from = branch_from
+      end
+
+      def with_clean_state
+        result = Shell.execute('git', 'stash')
+        stashed = !result.include?('No local changes to save')
+
+        with_return_to_current_branch(stashed: stashed) do
+          checkout_branch(@branch_from)
+
+          yield
+        end
+      end
+
+      def create_branch(change)
+        branch_name = branch_name(change.identifiers)
+
+        Shell.execute("git", "branch", "-f", branch_name)
+
+        branch_name
+      end
+
+      def in_branch(branch_name)
+        with_return_to_current_branch do
+          checkout_branch(branch_name)
+
+          yield
+        end
+      end
+
+      def create_commit(change)
+        Shell.execute("git", "add", *change.changed_files)
+        Shell.execute("git", "commit", "-m", change.commit_message)
+      end
+
+      private
+
+      def create_and_checkout_branch(branch_name)
+        begin
+          Shell.execute("git", "branch", '-D', branch_name)
+        rescue Shell::Error # Might not exist yet
+        end
+
+        Shell.execute("git", "checkout", "-b", branch_name)
+      end
+
+      def checkout_branch(branch_name)
+        Shell.execute("git", "checkout", branch_name)
+      end
+
+      def branch_name(identifiers)
+        # Hyphen-case each identifier then join together with hyphens.
+        branch_name = identifiers
+          .map { |i| i.gsub(/[^\w]+/, '-') }
+          .map { |i| i.gsub(/[[:upper:]]/) { |w| "-#{w.downcase}" } }
+          .join('-')
+          .delete_prefix("-")
+
+        # Truncate if it's too long and add a digest
+        if branch_name.length > 240
+          branch_name = branch_name[0...200] + OpenSSL::Digest::SHA256.hexdigest(branch_name)[0...15]
+        end
+
+        branch_name
+      end
+
+      def with_return_to_current_branch(stashed: false)
+        current_branch = Shell.execute('git', 'branch', '--show-current').chomp
+
+        yield
+      ensure
+        # The `current_branch` won't be set in CI due to how the repo is cloned. Therefore we should only checkout
+        # `current_branch` if we actually have one.
+        checkout_branch(current_branch) if current_branch.present?
+        Shell.execute('git', 'stash', 'pop') if stashed
+      end
+    end
+  end
+end

data/lib/gitlab/housekeeper/gitlab_client.rb

@@ -0,0 +1,206 @@
+# frozen_string_literal: true
+
+require 'httparty'
+require 'json'
+
+module Gitlab
+  module Housekeeper
+    class GitlabClient
+      Error = Class.new(StandardError)
+
+      def initialize
+        @token = ENV.fetch("HOUSEKEEPER_GITLAB_API_TOKEN")
+        @base_uri = 'https://gitlab.com/api/v4'
+      end
+
+      # This looks at the system notes of the merge request to detect if it has been updated by anyone other than the
+      # current housekeeper user. If it has then it assumes that they did this for a reason and we can skip updating
+      # this detail of the merge request. Otherwise we assume we should generate it again using the latest output.
+      def non_housekeeper_changes(
+        source_project_id:,
+        source_branch:,
+        target_branch:,
+        target_project_id:
+      )
+
+        existing_merge_request = get_existing_merge_request(
+          source_project_id: source_project_id,
+          source_branch: source_branch,
+          target_branch: target_branch,
+          target_project_id: target_project_id
+        )
+
+        return [] if existing_merge_request.nil?
+
+        merge_request_notes = get_merge_request_notes(
+          target_project_id: target_project_id,
+          iid: existing_merge_request['iid']
+        )
+
+        changes = Set.new
+
+        merge_request_notes.each do |note|
+          next false unless note["system"]
+          next false if note["author"]["id"] == current_user_id
+
+          changes << :title if note['body'].start_with?("changed title from")
+          changes << :description if note['body'] == "changed the description"
+          changes << :code if note['body'].match?(/added \d+ commit/)
+
+          changes << :reviewers if note['body'].include?('requested review from')
+          changes << :reviewers if note['body'].include?('removed review request for')
+        end
+
+        resource_label_events = get_merge_request_resource_label_events(
+          target_project_id: target_project_id,
+          iid: existing_merge_request['iid']
+        )
+
+        resource_label_events.each do |event|
+          next if event["user"]["id"] == current_user_id
+
+          # Labels are routinely added by both humans and bots, so addition events aren't cause for concern.
+          # However, if labels have been removed it may mean housekeeper added an incorrect label, and we shouldn't
+          # re-add them.
+          #
+          # TODO: Inspect the actual labels housekeeper wants to add, and add if they haven't previously been removed.
+          changes << :labels if event["action"] == "remove"
+        end
+
+        changes.to_a
+      end
+
+      # rubocop:disable Metrics/ParameterLists
+      def create_or_update_merge_request(
+        change:,
+        source_project_id:,
+        source_branch:,
+        target_branch:,
+        target_project_id:,
+        update_title:,
+        update_description:,
+        update_labels:,
+        update_reviewers:
+      )
+        existing_merge_request = get_existing_merge_request(
+          source_project_id: source_project_id,
+          source_branch: source_branch,
+          target_branch: target_branch,
+          target_project_id: target_project_id
+        )
+
+        if existing_merge_request
+          update_existing_merge_request(
+            change: change,
+            existing_iid: existing_merge_request['iid'],
+            target_project_id: target_project_id,
+            update_title: update_title,
+            update_description: update_description,
+            update_labels: update_labels,
+            update_reviewers: update_reviewers
+          )
+        else
+          create_merge_request(
+            change: change,
+            source_project_id: source_project_id,
+            source_branch: source_branch,
+            target_branch: target_branch,
+            target_project_id: target_project_id
+          )
+        end
+      end
+      # rubocop:enable Metrics/ParameterLists
+
+      def get_existing_merge_request(source_project_id:, source_branch:, target_branch:, target_project_id:)
+        data = request(:get, "/projects/#{target_project_id}/merge_requests", query: {
+          state: :opened,
+          source_branch: source_branch,
+          target_branch: target_branch,
+          source_project_id: source_project_id
+        })
+
+        return nil if data.empty?
+
+        raise Error, "More than one matching MR exists: iids: #{data.pluck('iid').join(',')}" unless data.size == 1
+
+        data.first
+      end
+
+      private
+
+      def get_merge_request_notes(target_project_id:, iid:)
+        request(:get, "/projects/#{target_project_id}/merge_requests/#{iid}/notes", query: { per_page: 100 })
+      end
+
+      def get_merge_request_resource_label_events(target_project_id:, iid:)
+        request(:get, "/projects/#{target_project_id}/merge_requests/#{iid}/resource_label_events",
+          query: { per_page: 100 })
+      end
+
+      def current_user_id
+        @current_user_id ||= request(:get, "/user")['id']
+      end
+
+      def create_merge_request(
+        change:,
+        source_project_id:,
+        source_branch:,
+        target_branch:,
+        target_project_id:
+      )
+        request(:post, "/projects/#{source_project_id}/merge_requests", body: {
+          title: change.title,
+          description: change.mr_description,
+          labels: Array(change.labels).join(','),
+          source_branch: source_branch,
+          target_branch: target_branch,
+          target_project_id: target_project_id,
+          remove_source_branch: true,
+          reviewer_ids: usernames_to_ids(change.reviewers)
+        })
+      end
+
+      def update_existing_merge_request(
+        change:,
+        existing_iid:,
+        target_project_id:,
+        update_title:,
+        update_description:,
+        update_labels:,
+        update_reviewers:
+      )
+        body = {}
+
+        body[:title] = change.title if update_title
+        body[:description] = change.mr_description if update_description
+        body[:add_labels] = Array(change.labels).join(',') if update_labels
+        body[:reviewer_ids] = usernames_to_ids(change.reviewers) if update_reviewers
+
+        return if body.empty?
+
+        request(:put, "/projects/#{target_project_id}/merge_requests/#{existing_iid}", body: body)
+      end
+
+      def usernames_to_ids(usernames)
+        Array(usernames).map do |username|
+          data = request(:get, "/users", query: { username: username })
+          data[0]['id']
+        end
+      end
+
+      def request(method, path, query: {}, body: {})
+        response = HTTParty.public_send(method, "#{@base_uri}#{path}", query: query, body: body.to_json, headers: { # rubocop:disable GitlabSecurity/PublicSend
+          'Private-Token' => @token,
+          'Content-Type' => 'application/json'
+        })
+
+        unless (200..299).cover?(response.code)
+          raise Error,
+            "Failed with response code: #{response.code} and body:\n#{response.body}"
+        end
+
+        JSON.parse(response.body)
+      end
+    end
+  end
+end

data/lib/gitlab/housekeeper/keep.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+module Gitlab
+  module Housekeeper
+    # A Keep is analogous to a Cop in RuboCop. The Keep is responsible for:
+    # - Detecting a specific code change that should be made (eg. removing an old feature flag)
+    # - Making the code change (eg. delete the feature flag YML file)
+    # - Yielding a Change object that describes this change, For example:
+    #   ```
+    #   yield Gitlab::Housekeeper::Change.new(
+    #     identifiers: ['remove-old-ff', 'old_ff_name'], # Unique and stable identifier for branch name
+    #     title: "Remove old feature flag old_ff_name as it has been enabled since %15.0",
+    #     description: "This feature flag was enabled in 15.0 and is not needed anymore ...",
+    #     changed_files: ["config/feature_flags/ops/old_ff_name.yml", "app/models/user.rb"]
+    #   )
+    #   ```
+    class Keep
+      # The each_change method must update local working copy files and yield a Change object which describes the
+      # specific changed files and other data that will be used to generate a merge request. This is the core
+      # implementation details for a specific housekeeper keep. This does not need to commit the changes or create the
+      # merge request as that is handled by the gitlab-housekeeper gem.
+      #
+      # @yieldparam [Gitlab::Housekeeper::Change]
+      def each_change
+        raise NotImplementedError, "A Keep must implement each_change method"
+      end
+    end
+  end
+end

data/lib/gitlab/housekeeper/keeps/rubocop_fixer.rb

@@ -0,0 +1,91 @@
+# frozen_string_literal: true
+
+require 'yaml'
+
+module Gitlab
+  module Housekeeper
+    module Keeps
+      class RubocopFixer < Keep
+        LIMIT_FIXES = 20
+        RUBOCOP_TODO_DIR_PATTERN = ".rubocop_todo/**/*.yml"
+
+        def initialize(todo_dir_pattern: RUBOCOP_TODO_DIR_PATTERN, limit_fixes: LIMIT_FIXES)
+          @todo_dir_pattern = todo_dir_pattern
+          @limit_fixes = limit_fixes
+        end
+
+        def each_change
+          each_allowed_rubocop_rule do |rule, rule_file_path, violating_files|
+            remove_allow_rule = true
+
+            if violating_files.count > @limit_fixes
+              violating_files = violating_files.first(@limit_fixes)
+              remove_allow_rule = false
+            end
+
+            change = Change.new
+            change.title = "Fix #{violating_files.count} rubocop violations for #{rule}"
+            change.identifiers = [self.class.name, rule, violating_files.last]
+            change.description = <<~MARKDOWN
+            Fixes the #{violating_files.count} violations for the rubocop rule `#{rule}`
+            that were previously excluded in `#{rule_file_path}`.
+            The exclusions have now been removed.
+            MARKDOWN
+
+            if remove_allow_rule
+              FileUtils.rm(rule_file_path)
+            else
+              remove_first_exclusions(rule, rule_file_path, violating_files.count)
+            end
+
+            begin
+              ::Gitlab::Housekeeper::Shell.execute('rubocop', '--autocorrect', *violating_files)
+            rescue ::Gitlab::Housekeeper::Shell::Error
+              # Ignore when it cannot be automatically fixed. But we need to checkout any files we might have updated.
+              ::Gitlab::Housekeeper::Shell.execute('git', 'checkout', rule_file_path, *violating_files)
+              next
+            end
+
+            change.changed_files = [rule_file_path, *violating_files]
+
+            yield(change)
+          end
+        end
+
+        def each_allowed_rubocop_rule
+          Dir.glob(@todo_dir_pattern).each do |file|
+            content = File.read(file)
+            next unless content.include?('Cop supports --autocorrect')
+
+            data = YAML.safe_load(content)
+
+            # Assume one per file since that's how it is in gitlab-org/gitlab
+            next unless data.keys.count == 1
+
+            rule = data.keys[0]
+            violating_files = data[rule]['Exclude']
+            next unless violating_files&.count&.positive?
+
+            yield rule, file, violating_files
+          end
+        end
+
+        def remove_first_exclusions(_rule, file, remove_count)
+          content = File.read(file)
+          skipped = 0
+
+          output = content.each_line.filter do |line|
+            if skipped < remove_count && line.match?(/\s+-\s+/)
+              skipped += 1
+              false
+            else
+              true
+            end
+          end
+
+          File.write(file, output.join)
+        end
+      end
+    end
+  end
+end

data/lib/gitlab/housekeeper/keeps/upgrade_dependency.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module Gitlab
+  module Housekeeper
+    module Keeps
+      class UpgradeDependency < Keep
+        def initialize(
+          upgrade_command: ENV.fetch('HOUSEKEEPER_DEPENDENCY_UPGRADE_COMMAND'), # bundle update haml
+          changed_files: ENV.fetch('HOUSEKEEPER_DEPENDENCY_UPGRADE_CHANGED_FILES').split(',') # Gemfile.lock
+        )
+          @upgrade_command = upgrade_command
+          @changed_files = changed_files
+        end
+
+        def each_change
+          ::Gitlab::Housekeeper::Shell.execute(@upgrade_command)
+
+          change = Change.new
+          change.title = "Run #{@upgrade_command}"
+          change.identifiers = [self.class.name, @upgrade_command]
+          change.description = <<~MARKDOWN
+          Automatically upgrade a dependency using:
+
+          ```
+          #{@upgrade_command}
+          ```
+          MARKDOWN
+
+
+          change.changed_files = @changed_files
+
+          yield(change)
+        end
+      end
+    end
+  end
+end

data/lib/gitlab/housekeeper/runner.rb

@@ -0,0 +1,186 @@
+# frozen_string_literal: true
+
+require 'active_support'
+require 'active_support/core_ext'
+require 'gitlab/housekeeper/keep'
+require 'gitlab/housekeeper/keeps/rubocop_fixer'
+require 'gitlab/housekeeper/keeps/upgrade_dependency'
+require 'gitlab/housekeeper/gitlab_client'
+require 'gitlab/housekeeper/git'
+require 'gitlab/housekeeper/change'
+require 'gitlab/housekeeper/substitutor'
+require 'awesome_print'
+require 'digest'
+
+module Gitlab
+  module Housekeeper
+    class Runner
+      def initialize(max_mrs: 1, dry_run: false, keeps: nil, filter_identifiers: [], target_branch: 'master')
+        @max_mrs = max_mrs
+        @dry_run = dry_run
+        @logger = Logger.new($stdout)
+        @target_branch = target_branch
+        require_keeps
+
+        @keeps = if keeps
+                   keeps.map { |k| k.is_a?(String) ? k.constantize : k }
+                 else
+                   all_keeps
+                 end
+
+        @filter_identifiers = filter_identifiers
+      end
+
+      def run
+        created = 0
+
+        git.with_clean_state do
+          @keeps.each do |keep_class|
+            keep = keep_class.new
+            keep.each_change do |change|
+              unless change.valid?
+                puts "Ignoring invalid change from: #{keep_class}"
+                next
+              end
+
+              change.keep_class ||= keep_class
+
+              branch_name = git.create_branch(change)
+              add_standard_change_data(change)
+
+              unless change.matches_filters?(@filter_identifiers)
+                # At this point the keep has already run and edited files so we need to
+                # restore the local working copy. We could simply checkout all
+                # changed_files but this is very risky as it could mean losing work that
+                # cannot be recovered. Instead we commit all the work to the branch and
+                # move on without pushing the branch.
+                git.in_branch(branch_name) do
+                  git.create_commit(change)
+                end
+
+                puts "Skipping change: #{change.identifiers} due to not matching filter."
+                puts "Modified files have been committed to branch #{branch_name.yellowish}, but will not be pushed."
+                puts
+
+                next
+              end
+
+              # If no merge request exists yet, create an empty one to allow keeps to use the web URL.
+              unless @dry_run
+                merge_reqeust = get_existing_merge_request(branch_name) || create(change, branch_name)
+
+                change.mr_web_url = merge_reqeust['web_url']
+              end
+
+              git.in_branch(branch_name) do
+                Gitlab::Housekeeper::Substitutor.perform(change)
+
+                git.create_commit(change)
+              end
+
+              print_change_details(change, branch_name)
+
+              create(change, branch_name) unless @dry_run
+
+              created += 1
+              break if created >= @max_mrs
+            end
+            break if created >= @max_mrs
+          end
+        end
+
+        puts "Housekeeper created #{created} MRs"
+      end
+
+      def add_standard_change_data(change)
+        change.labels ||= []
+        change.labels << 'automation:gitlab-housekeeper-authored'
+      end
+
+      def git
+        @git ||= ::Gitlab::Housekeeper::Git.new(logger: @logger)
+      end
+
+      def require_keeps
+        Dir.glob("keeps/*.rb").each do |f|
+          require(Pathname(f).expand_path.to_s)
+        end
+      end
+
+      def print_change_details(change, branch_name)
+        puts "Merge request URL: #{change.mr_web_url || '(known after create)'}, on branch #{branch_name}".yellowish
+        puts "=> #{change.identifiers.join(': ')}".purple
+
+        puts '=> Title:'.purple
+        puts change.title.purple
+        puts
+
+        puts '=> Description:'
+        puts change.description
+        puts
+
+        if change.labels.present? || change.reviewers.present?
+          puts '=> Attributes:'
+          puts "Labels: #{change.labels.join(', ')}"
+          puts "Reviewers: #{change.reviewers.join(', ')}"
+          puts
+        end
+
+        puts '=> Diff:'
+        puts Shell.execute('git', '--no-pager', 'diff', '--color=always', @target_branch, branch_name, '--',
+          *change.changed_files)
+        puts
+      end
+
+      def create(change, branch_name)
+        non_housekeeper_changes = gitlab_client.non_housekeeper_changes(
+          source_project_id: housekeeper_fork_project_id,
+          source_branch: branch_name,
+          target_branch: @target_branch,
+          target_project_id: housekeeper_target_project_id
+        )
+
+        unless non_housekeeper_changes.include?(:code)
+          Shell.execute('git', 'push', '-f', 'housekeeper', "#{branch_name}:#{branch_name}")
+        end
+
+        gitlab_client.create_or_update_merge_request(
+          change: change,
+          source_project_id: housekeeper_fork_project_id,
+          source_branch: branch_name,
+          target_branch: @target_branch,
+          target_project_id: housekeeper_target_project_id,
+          update_title: !non_housekeeper_changes.include?(:title),
+          update_description: !non_housekeeper_changes.include?(:description),
+          update_labels: !non_housekeeper_changes.include?(:labels),
+          update_reviewers: !non_housekeeper_changes.include?(:reviewers)
+        )
+      end
+
+      def get_existing_merge_request(branch_name)
+        gitlab_client.get_existing_merge_request(
+          source_project_id: housekeeper_fork_project_id,
+          source_branch: branch_name,
+          target_branch: @target_branch,
+          target_project_id: housekeeper_target_project_id
+        )
+      end
+
+      def housekeeper_fork_project_id
+        ENV.fetch('HOUSEKEEPER_FORK_PROJECT_ID')
+      end
+
+      def housekeeper_target_project_id
+        ENV.fetch('HOUSEKEEPER_TARGET_PROJECT_ID')
+      end
+
+      def gitlab_client
+        @gitlab_client ||= GitlabClient.new
+      end
+
+      def all_keeps
+        @all_keeps ||= ObjectSpace.each_object(Class).select { |klass| klass < Keep }
+      end
+    end
+  end
+end

data/lib/gitlab/housekeeper/shell.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+require 'open3'
+
+module Gitlab
+  module Housekeeper
+    class Shell
+      Error = Class.new(StandardError)
+
+      def self.execute(*cmd)
+        stdin, stdout, stderr, wait_thr = Open3.popen3(*cmd)
+
+        stdin.close
+        out = stdout.read
+        stdout.close
+        err = stderr.read
+        stderr.close
+
+        exit_status = wait_thr.value
+
+        raise Error, "Failed with #{exit_status}\n#{out}\n#{err}\n" unless exit_status.success?
+
+        out + err
+      end
+    end
+  end
+end

data/lib/gitlab/housekeeper/substitutor.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module Gitlab
+  module Housekeeper
+    class Substitutor
+      MR_WEB_URL_PLACEHOLDER = '<HOUSEKEEPER_MR_WEB_URL>'
+
+      def self.perform(change)
+        return unless change.mr_web_url.present?
+
+        change.changed_files.each do |file|
+          next unless File.file?(file)
+
+          content = File.read(file)
+          content.gsub!(MR_WEB_URL_PLACEHOLDER, change.mr_web_url)
+
+          File.write(file, content)
+        end
+      end
+    end
+  end
+end

data/lib/gitlab/housekeeper/version.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module Gitlab
+  module Housekeeper
+    VERSION = "0.1.0"
+  end
+end

data/lib/gitlab/housekeeper.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+require "gitlab/housekeeper/version"
+require "gitlab/housekeeper/runner"
+
+module Gitlab
+  module Housekeeper
+    Error = Class.new(StandardError)
+    # Your code goes here...
+  end
+end

metadata

@@ -0,0 +1,169 @@
+--- !ruby/object:Gem::Specification
+name: gitlab-housekeeper
+version: !ruby/object:Gem::Version
+  version: 0.1.0.codechallenge.3c718d.pre
+platform: ruby
+authors:
+- group::tenant-scale
+autorequire:
+bindir: bin
+cert_chain: []
+date: 2024-03-14 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: awesome_print
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: httparty
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: gitlab-styles
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: Housekeeping following https://gitlab.com/gitlab-org/gitlab/-/merge_requests/134487
+email:
+- engineering@gitlab.com
+executables:
+- gitlab-housekeeper
+extensions: []
+extra_rdoc_files: []
+files:
+- bin/gitlab-housekeeper
+- lib/gitlab/housekeeper.rb
+- lib/gitlab/housekeeper/change.rb
+- lib/gitlab/housekeeper/git.rb
+- lib/gitlab/housekeeper/gitlab_client.rb
+- lib/gitlab/housekeeper/keep.rb
+- lib/gitlab/housekeeper/keeps/rubocop_fixer.rb
+- lib/gitlab/housekeeper/keeps/upgrade_dependency.rb
+- lib/gitlab/housekeeper/runner.rb
+- lib/gitlab/housekeeper/shell.rb
+- lib/gitlab/housekeeper/substitutor.rb
+- lib/gitlab/housekeeper/version.rb
+homepage: https://gitlab.com/gitlab-org/gitlab/-/tree/master/gems/gitlab-housekeeper
+licenses:
+- MIT
+metadata:
+  rubygems_mfa_required: 'true'
+post_install_message:
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '3.0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubygems_version: 3.5.6
+signing_key:
+specification_version: 4
+summary: Gem summary
+test_files: []