Fetch PR data once and allow whitespace after # in Redmine pattern

Address review feedback: extract the PR API call into the review method
and pass it to review_non_fork_branch and review_redmine_links to avoid
a duplicate API request. Also allow optional whitespace between # and
the ticket number in the Redmine ticket pattern.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: hsbt

tool/auto_review_pr.rb

@@ -38,7 +38,7 @@ class AutoReviewPR
   UPSTREAM_COMMENT_PREFIX = 'The following files are maintained in the following upstream repositories:'
   UPSTREAM_COMMENT_SUFFIX = 'Please file a pull request to the above instead. Thank you!'
 
-  REDMINE_TICKET_PATTERN = /\[(Bug|Feature|Misc)\s*#(\d+)\]/
+  REDMINE_TICKET_PATTERN = /\[(Bug|Feature|Misc)\s*#\s*(\d+)\]/
   REDMINE_COMMENT_PREFIX = 'This pull request references the following Redmine tickets:'
 
   FORK_COMMENT_PREFIX = 'It looks like this pull request was filed from a branch in ruby/ruby.'
@@ -60,9 +60,10 @@ def initialize(client)
 
   def review(pr_number)
     existing_comments = fetch_existing_comments(pr_number)
-    review_non_fork_branch(pr_number, existing_comments)
+    pr = @client.get("/repos/#{REPO}/pulls/#{pr_number}")
+    review_non_fork_branch(pr_number, pr, existing_comments)
     review_upstream_repos(pr_number, existing_comments)
-    review_redmine_links(pr_number, existing_comments)
+    review_redmine_links(pr_number, pr, existing_comments)
   end
 
   private
@@ -82,13 +83,12 @@ def post_comment(pr_number, comment)
   end
 
   # Suggest re-filing from a fork if the PR branch is in ruby/ruby itself
-  def review_non_fork_branch(pr_number, existing_comments)
+  def review_non_fork_branch(pr_number, pr, existing_comments)
     if already_commented?(existing_comments, FORK_COMMENT_PREFIX)
       puts "Skipped: The PR ##{pr_number} already has a fork branch comment."
       return
     end
 
-    pr = @client.get("/repos/#{REPO}/pulls/#{pr_number}")
     head_repo = pr.dig(:head, :repo, :full_name)
     if head_repo != REPO
       puts "Skipped: The PR ##{pr_number} is already from a fork (#{head_repo})."
@@ -124,13 +124,12 @@ def review_upstream_repos(pr_number, existing_comments)
     post_comment(pr_number, format_upstream_comment(upstream_repos))
   end
 
-  def review_redmine_links(pr_number, existing_comments)
+  def review_redmine_links(pr_number, pr, existing_comments)
     if already_commented?(existing_comments, REDMINE_COMMENT_PREFIX)
       puts "Skipped: The PR ##{pr_number} already has a Redmine links comment."
       return
     end
 
-    pr = @client.get("/repos/#{REPO}/pulls/#{pr_number}")
     text = "#{pr[:title]}\n#{pr[:body]}"
 
     tickets = text.scan(REDMINE_TICKET_PATTERN).uniq