Added the --gitrepo and --gitdiff switches to .check.rb

[dearblue]

Overview: This allows checking against a Git repository and a specified branch. For pull requests, only modified or added “.gem” files are checked.

[gemini-code-assist]

Code Review

This pull request adds command-line options (--gitdiff, --gitrepo, and --interval) to .check.rb to allow verifying files with changes between Git refs or checking a Git repository and its branch. Feedback includes improving check_git_repository with defensive checks, idiomatic array methods, and cleaner string formatting, as well as optimizing the script to avoid executing git ls-files twice when --gitdiff is disabled.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

There are a few improvements we can make to check_git_repository:

1. Defensive Programming: If repository is missing or empty in the YAML file, repo will be nil or empty. Calling git_get_remote_refs with nil will run a failing git command. We should guard against this and return early.
2. Idiomatic Membership Check: Use Array#include? instead of Array#any? with an argument for checking exact string matching. It is more standard and compatible.
3. Cleaner String Manipulation: Instead of joining the array with newlines, performing a regex replacement, and then translating newlines back to spaces, we can map over the array using delete_prefix and join them with spaces directly.

def check_git_repository(gemname, tree, errinfo) # rubocop:disable Metrics/MethodLength
  repo = tree["repository"]
  return if repo.to_s.empty?

  branch = tree["branch"] || "master"
  refs = git_get_remote_refs(repo)

  if refs.nil?
    errinfo.push <<~INFO
      #{gemname}: unable to access the repository or locate a valid branch name.
    INFO
  elsif refs.include?("refs/heads/#{branch}")
    # OK
  elsif refs.include?("refs/tags/#{branch}")
    errinfo.push <<~INFO
      WARN: #{gemname}: instead of branches, the tag "#{branch}" is used.
    INFO
  else
    candidates = refs.grep(%r{\Arefs/heads/}).map { |r| r.delete_prefix("refs/heads/") }.join(" ")
    errinfo.push <<~INFO
      #{gemname}: branch "#{branch}" is missing.
      \tcandidates: #{candidates}
    INFO
  end
end

[gemini-code-assist]

When --gitdiff is not specified, git ls-files is executed twice: once to populate gemfiles and once to populate entries. We can optimize this by running git ls-files once, storing it in entries, and then assigning gemfiles = entries in the else branch.

entries = run_and_split_nul(%w[git ls-files -z :^*/*])

if gitdiff
  case ARGV.size
  when 1, 2
    # do nothing
  else
    warn banner
    exit 1
  end
  cmd = %w[git diff -z --name-only --diff-filter=ACMR]
  cmd.concat ARGV
  cmd.concat %w[-- :^*/*]
  gemfiles = run_and_split_nul(cmd)
else
  gemfiles = entries
end