git 5.1.0 → 5.2.0

data/lib/git/repository/branching.rb

@@ -41,7 +41,7 @@ module Git
 
       # Option keys accepted by {#checkout}
       #
-      CHECKOUT_ALLOWED_OPTS = %i[force f new_branch b start_point].freeze
+      CHECKOUT_ALLOWED_OPTS = %i[force f new_branch b start_point orphan].freeze
       private_constant :CHECKOUT_ALLOWED_OPTS
 
       # Option keys accepted by {#checkout_index}
@@ -130,6 +130,9 @@ module Git
       # @example Create a new branch with a name different from the start point
       #   repo.checkout('main', new_branch: 'new-feature')
       #
+      # @example Create and check out an unborn branch with no history
+      #   repo.checkout('gh-pages', orphan: true)
+      #
       # @example Force checkout discarding local changes
       #   repo.checkout('main', force: true)
       #
@@ -151,13 +154,25 @@ module Git
       #
       # @option opts [Boolean, nil] :f (nil) alias for `:force`
       #
+      # @option opts [Boolean, String, nil] :orphan (nil) when `true`, creates a
+      #   new unborn branch named `branch` whose first commit has no parents
+      #
+      #   When a `String`, creates an unborn branch with that name, using
+      #   `branch` as the start point for the working tree and index.
+      #
+      #   `false` and `nil` are both treated as unset. A blank branch name is
+      #   rejected rather than ignored.
+      #
       # @option opts [String, nil] :start_point (nil) the commit or branch to
-      #   start the new branch from; used together with `new_branch: true`
+      #   start the new branch from; used together with `new_branch: true` or
+      #   `orphan: true`
       #
       # @return [String] git's stdout from the checkout
       #
       # @raise [ArgumentError] if unsupported options are provided
       #
+      # @raise [ArgumentError] if `:orphan` is given a blank or missing branch name
+      #
       # @raise [Git::FailedError] if git exits with a non-zero exit status
       #
       def checkout(branch = nil, opts = {})
@@ -680,19 +695,24 @@ module Git
           :unborn
         end
 
-        # Translates legacy checkout options to the new command interface
+        # Translates {#checkout} options to the new command interface
         #
         # Legacy callers passed combinations like:
         #   checkout('branch', new_branch: true, start_point: 'main')
         # which should map to:
         #   checkout('main', b: 'branch')
         #
+        # `orphan: true` follows the same shape, naming the unborn branch:
+        #   checkout('branch', orphan: true, start_point: 'main')
+        # maps to:
+        #   checkout('main', orphan: 'branch')
+        #
         # @param branch [String, nil] the branch argument passed to {#checkout}
         #
         # @param checkout_options [Hash] the raw options passed to {#checkout}
         #
-        # @return [Array] a two-element tuple `[target, options]` containing the
-        #   translated checkout arguments
+        # @return [Array((String, nil), Hash)] a two-element tuple
+        #   `[target, options]` containing the translated checkout arguments
         #
         #   `target` (`String` or `nil`) is the branch or commit to check out.
         #   `options` is a `Hash` of keyword arguments for
@@ -701,15 +721,69 @@ module Git
         # @api private
         #
         def translate_checkout_opts(branch, checkout_options)
+          checkout_options = normalize_orphan_option(checkout_options)
+
           if checkout_options[:new_branch] == true || checkout_options[:b] == true
             [checkout_options[:start_point], checkout_options.except(:new_branch, :b, :start_point).merge(b: branch)]
           elsif checkout_options[:new_branch].is_a?(String)
             [branch, checkout_options.except(:new_branch).merge(b: checkout_options[:new_branch])]
+          elsif checkout_options[:orphan] == true
+            translate_orphan_opts(branch, checkout_options)
           else
             [branch, checkout_options]
           end
         end
 
+        # Normalizes the `:orphan` option, rejecting names that git would never see
+        #
+        # `:orphan` is a value option on the underlying command, so a literal
+        # `false` would be emitted as `--orphan false` and create a branch named
+        # "false". Flag options such as `:force` already ignore `false`; this
+        # gives `:orphan` the same behavior.
+        #
+        # A blank name is rejected rather than dropped: the argument DSL omits
+        # empty values, so `orphan: ''` would otherwise degrade silently into a
+        # plain checkout.
+        #
+        # @param checkout_options [Hash] the raw options passed to {#checkout}
+        #
+        # @return [Hash] the options with a `false` `:orphan` key removed
+        #
+        # @raise [ArgumentError] if `:orphan` is given a blank branch name
+        #
+        # @api private
+        #
+        def normalize_orphan_option(checkout_options)
+          orphan = checkout_options[:orphan]
+          return checkout_options.except(:orphan) if orphan == false
+          raise ArgumentError, 'orphan requires a non-empty branch name' if orphan.is_a?(String) && orphan.strip.empty?
+
+          checkout_options
+        end
+
+        # Translates `orphan: true` into the command's `:orphan` value option
+        #
+        # `orphan: true` names the unborn branch from the positional argument and
+        # takes its start point from `:start_point`, mirroring `new_branch: true`.
+        #
+        # @param branch [String, nil] the branch argument passed to {#checkout}
+        #
+        # @param checkout_options [Hash] the raw options passed to {#checkout}
+        #
+        # @return [Array((String, nil), Hash)] a two-element tuple
+        #   `[target, options]` containing the translated checkout arguments
+        #
+        # @raise [ArgumentError] if `branch` is blank (`nil`, empty, or whitespace
+        #   only), since the unborn branch would otherwise have no name
+        #
+        # @api private
+        #
+        def translate_orphan_opts(branch, checkout_options)
+          raise ArgumentError, 'orphan: true requires a branch name' if branch.to_s.strip.empty?
+
+          [checkout_options[:start_point], checkout_options.except(:start_point).merge(orphan: branch)]
+        end
+
         # Normalizes path specifications for Git commands
         #
         # @param pathspecs [String, Pathname, Array<String, Pathname>, nil]

data/lib/git/version.rb

@@ -4,7 +4,7 @@ module Git
   # The current gem version
   #
   # @return [String] the current gem version
-  VERSION = '5.1.0'
+  VERSION = '5.2.0'
 
   # Represents a git version with major, minor, and patch components
   #

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: git
 version: !ruby/object:Gem::Version
-  version: 5.1.0
+  version: 5.2.0
 platform: ruby
 authors:
 - Scott Chacon and others
@@ -43,14 +43,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '4.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '4.1'
 - !ruby/object:Gem::Dependency
   name: rchardet
   requirement: !ruby/object:Gem::Requirement
@@ -298,80 +298,16 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".claude/commands/address-copilot-reviews.md"
-- ".claude/settings.json"
-- ".claude/skills"
-- ".commitlintrc.yml"
-- ".dockerignore"
-- ".github/copilot-instructions.md"
-- ".github/hooks/bin-setup-on-worktree.json"
-- ".github/hooks/run-bin-setup-once.sh"
-- ".github/issue_template.md"
-- ".github/prompts/iteratively-address-copilot-reviews.prompt.md"
-- ".github/pull_request_template.md"
-- ".github/skills-deprecated/README.md"
-- ".github/skills-deprecated/extract-command-from-lib/SKILL.md"
-- ".github/skills-deprecated/extract-facade-from-base-lib/KEYWORD_ARG_REMEDIATION.md"
-- ".github/skills-deprecated/extract-facade-from-base-lib/SKILL.md"
-- ".github/skills-deprecated/review-backward-compatibility/SKILL.md"
-- ".github/skills/breaking-change-analysis/SKILL.md"
-- ".github/skills/ci-cd-troubleshooting/SKILL.md"
-- ".github/skills/command-implementation/REFERENCE.md"
-- ".github/skills/command-implementation/SKILL.md"
-- ".github/skills/command-test-conventions/SKILL.md"
-- ".github/skills/command-yard-documentation/SKILL.md"
-- ".github/skills/dependency-management/SKILL.md"
-- ".github/skills/development-workflow/SKILL.md"
-- ".github/skills/facade-implementation/REFERENCE.md"
-- ".github/skills/facade-implementation/SKILL.md"
-- ".github/skills/facade-test-conventions/SKILL.md"
-- ".github/skills/facade-yard-documentation/SKILL.md"
-- ".github/skills/make-skill-template/SKILL.md"
-- ".github/skills/pr-readiness-review/SKILL.md"
-- ".github/skills/project-context/SKILL.md"
-- ".github/skills/pull-request-review/SKILL.md"
-- ".github/skills/rebase/SKILL.md"
-- ".github/skills/refactor-command-to-commandlineresult/SKILL.md"
-- ".github/skills/release-management/SKILL.md"
-- ".github/skills/resolve-feedback/SKILL.md"
-- ".github/skills/review-arguments-dsl/CHECKLIST.md"
-- ".github/skills/review-arguments-dsl/SKILL.md"
-- ".github/skills/review-cross-command-consistency/SKILL.md"
-- ".github/skills/reviewing-skills/SKILL.md"
-- ".github/skills/rspec-unit-testing-standards/SKILL.md"
-- ".github/skills/tdd-refactor-step/SKILL.md"
-- ".github/skills/test-debugging/SKILL.md"
-- ".github/skills/yard-documentation/SKILL.md"
-- ".github/skills/yard-documentation/element-rules.md"
-- ".github/workflows/continuous_integration.yml"
-- ".github/workflows/enforce_conventional_commits.yml"
-- ".github/workflows/experimental_continuous_integration.yml"
-- ".github/workflows/release.yml"
-- ".github/workflows/warm_bundler_caches.yml"
-- ".gitignore"
-- ".husky/commit-msg"
-- ".husky/pre-commit"
-- ".release-please-config.json"
-- ".release-please-manifest.json"
-- ".rspec"
-- ".rubocop.yml"
-- ".rubocop_todo.yml"
-- ".yard-lint.yml"
 - ".yardopts"
 - AI_POLICY.md
 - CHANGELOG.md
-- CLAUDE.md
 - CODE_OF_CONDUCT.md
 - CONTRIBUTING.md
 - GOVERNANCE.md
-- Gemfile
 - LICENSE
 - MAINTAINERS.md
 - README.md
-- Rakefile
 - UPGRADING.md
-- docker/test/Dockerfile
-- docker/test/docker-compose.yml
 - git.gemspec
 - lib/git.rb
 - lib/git/author.rb
@@ -600,39 +536,14 @@ files:
 - lib/git/version_constraint.rb
 - lib/git/worktree.rb
 - lib/git/worktrees.rb
-- package.json
-- redesign/1_architecture_existing.md
-- redesign/2_architecture_redesign.md
-- redesign/3_architecture_implementation.md
-- redesign/Phase 4 - Step A.md
-- redesign/Phase 4 - Step B.md
-- redesign/Phase 4 - Step C.md
-- redesign/beta_release.md
-- redesign/branch_parse_refactor_plan.md
-- redesign/c1a-public-api-scope.tsv
-- redesign/c1c2_audit.md
-- redesign/c1c2_bucket6_lib_orphans.md
-- redesign/config_design.rb
-- redesign/index.md
-- redesign/info_object_migration_plan.md
-- redesign/integration_test_analysis.md
-- redesign/phase-4-step-b-test-audit.tsv
-- redesign/remote_refactor_plan.md
-- redesign/reverse_dependencies.sql
-- tasks/gem_tasks.rake
-- tasks/npm_tasks.rake
-- tasks/rspec.rake
-- tasks/rubocop.rake
-- tasks/test_gem.rake
-- tasks/yard.rake
 homepage: http://github.com/ruby-git/ruby-git
 licenses:
 - MIT
 metadata:
   homepage_uri: http://github.com/ruby-git/ruby-git
   source_code_uri: http://github.com/ruby-git/ruby-git
-  changelog_uri: https://rubydoc.info/gems/git/5.1.0/file/CHANGELOG.md
-  documentation_uri: https://rubydoc.info/gems/git/5.1.0
+  changelog_uri: https://rubydoc.info/gems/git/5.2.0/file/CHANGELOG.md
+  documentation_uri: https://rubydoc.info/gems/git/5.2.0
   rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths:

data/.claude/commands/address-copilot-reviews.md

@@ -1,14 +0,0 @@
----
-description: Address all unresolved Copilot review threads on the active pull request
-argument-hint: "[PR number — defaults to the PR for the current branch]"
----
-
-Follow the shared agent prompt in
-`.github/prompts/iteratively-address-copilot-reviews.prompt.md` exactly. That file
-is the canonical instructions for this task and is shared with other AI agents; read
-it before doing anything else.
-
-@.github/prompts/iteratively-address-copilot-reviews.prompt.md
-
-If `$ARGUMENTS` is non-empty, treat it as the pull request number to operate on
-instead of detecting the PR from the current branch.

data/.claude/settings.json

@@ -1,16 +0,0 @@
-{
-  "$schema": "https://json.schemastore.org/claude-code-settings.json",
-  "hooks": {
-    "SessionStart": [
-      {
-        "hooks": [
-          {
-            "type": "command",
-            "command": "\"$CLAUDE_PROJECT_DIR/.github/hooks/run-bin-setup-once.sh\"",
-            "timeout": 900
-          }
-        ]
-      }
-    ]
-  }
-}

data/.claude/skills

@@ -1 +0,0 @@
-../.github/skills

data/.commitlintrc.yml

@@ -1,38 +0,0 @@
----
-extends: '@commitlint/config-conventional'
-
-rules:
-  # See: https://commitlint.js.org/reference/rules.html
-  #
-  # Rules are made up by a name and a configuration array. The configuration
-  # array contains:
-  #
-  # * Severity [0..2]: 0 disable rule, 1 warning if violated, or 2 error if
-  #   violated
-  # * Applicability [always|never]: never inverts the rule
-  # * Value: value to use for this rule (if applicable)
-  #
-  # Run `npx commitlint --print-config` to see the current setting for all
-  # rules.
-  #
-  header-max-length:      [2, always, 100]        # Header can not exceed 100 chars
-
-  type-case:              [2, always, lower-case] # Type must be lower case
-  type-empty:             [2, never]              # Type must not be empty
-
-  # Supported conventional commit types
-  type-enum:              [2, always, [build, ci, chore, docs, feat, fix, perf, refactor, revert, style, test]]
-
-  scope-case:             [2, always, lower-case] # Scope must be lower case
-
-  # Error if subject is one of these cases (encourages lower-case)
-  subject-case:           [2, never, [sentence-case, start-case, pascal-case, upper-case]]
-  subject-empty:          [2, never]              # Subject must not be empty
-  subject-full-stop:      [2, never, "."]         # Subject must not end with a period
-
-  body-leading-blank:     [2, always]             # Body must have a blank line before it
-  body-max-line-length:   [2, always, 100]        # Body lines can not exceed 100 chars
-
-  footer-leading-blank:   [2, always]             # Footer must have a blank line before it
-  footer-max-line-length: [2, always, 100]        # Footer lines can not exceed 100 chars
-  

data/.dockerignore

@@ -1,27 +0,0 @@
-# Keep the Docker build context free of host-generated artifacts so the
-# image always builds gems fresh instead of picking up stale or
-# host-specific state (see .gitignore for the equivalent git-tracked list).
-#
-# .git is also excluded: the Dockerfile initializes its own throwaway git
-# repo instead of relying on the host's, since the host's `.git` may be a
-# linked-worktree pointer file referencing a path that doesn't exist in the
-# image (see docker/test/Dockerfile for details).
-.git
-*.gem
-*.kpf
-*.sw?
-.DS_Store
-.bundle/
-vendor/bundle/
-coverage
-doc
-.yardoc
-pkg
-rdoc
-Gemfile.lock
-node_modules
-package-lock.json
-ai-prompt.erb
-rubocop-report.json
-.rspec_status
-git-reference/

data/.github/copilot-instructions.md

@@ -1,30 +0,0 @@
-# GitHub Copilot Instructions for ruby-git
-
-## Project Overview
-
-A Ruby gem providing an interface to Git repositories by wrapping system calls to the
-`git` CLI.
-
-Read the "Project Context" skill (`.github/skills/project-context/SKILL.md`) for
-design philosophy, technical details, and compatibility requirements.
-
-This project enforces Conventional Commits. See `.commitlintrc.yml` for allowed types
-and scopes. Never use `#` in the commit message body. Doing so will cause
-commitlint to incorrectly parse the commit message body as a footer. If you need to
-refer to an issue in the body, use `issue 999` instead of `issue #999`. It is still
-fine to use `#` in footer values such as `Closes: #999` or `Refs: #999`.
-
-## Terminology & Writing Style
-
-- Use American English.
-- **Version strings:** Use `vN.x` for major series compatibility (e.g., `v4.x`) and
-  `vN.0.0` for specific releases (e.g., `v5.0.0`). Never use the ambiguous `vN.0`.
-- **RuboCop:** Use `RuboCop` for prose and `rubocop` for CLI or gem names. Never use
-  `Rubocop`.
-
-## Branch & PR Strategy
-
-| Target | When |
-| --- | --- |
-| `main` | New features, breaking changes, all active development |
-| `4.x` | Security fixes and backward-compatible bug fixes for the v4.x series |

data/.github/hooks/bin-setup-on-worktree.json

@@ -1,11 +0,0 @@
-{
-  "hooks": {
-    "SessionStart": [
-      {
-        "type": "command",
-        "command": ".github/hooks/run-bin-setup-once.sh",
-        "timeout": 900
-      }
-    ]
-  }
-}

data/.github/hooks/run-bin-setup-once.sh

@@ -1,20 +0,0 @@
-#!/usr/bin/env bash
-set -euo pipefail
-
-# This hook is shared by every agent configured for this repo (GitHub Copilot via
-# .github/hooks/, Claude Code via .claude/settings.json), so do not assume the
-# caller's working directory.
-command -v git >/dev/null 2>&1 || exit 0
-cd "$(dirname "${BASH_SOURCE[0]}")/../.." || exit 0
-
-# Skip if this repo does not use bin/setup.
-[[ -x "./bin/setup" ]] || exit 0
-
-# Keep setup idempotent per-worktree. The marker is shared across agents so that
-# cloning and opening the repo in two different tools does not run setup twice.
-marker="$(git rev-parse --git-path agent-bin-setup.done 2>/dev/null)" || exit 0
-[[ -f "$marker" ]] && exit 0
-
-./bin/setup
-mkdir -p "$(dirname "$marker")"
-touch "$marker"

data/.github/issue_template.md

@@ -1,15 +0,0 @@
-### Subject of the issue
-Describe your issue here.
-
-### Your environment
-* version of git and ruby-git
-* version of ruby
-
-### Steps to reproduce
-Tell us how to reproduce this issue. 
-
-### Expected behaviour
-What did you expect to happen?
-
-### Actual behaviour
-What actually happened?

data/.github/prompts/iteratively-address-copilot-reviews.prompt.md

@@ -1,188 +0,0 @@
----
-agent: agent
-model: claude-sonnet-4.6
-description: Address all unresolved Copilot review threads on the active pull request until there are no remaining unresolved review threads
----
-
-Address unresolved **Copilot** review threads on the active pull request. Ignore threads opened by human reviewers. Ask me for clarification or decisions as needed.
-
-If any terminal script command fails, stop immediately. Do not continue the loop, do not resolve additional threads, and do not request a new review. Report: the failed command, exit code, and the most relevant stderr output. **Exception**: the reply POST in Step 3 is intentionally non-fatal — a 404 after a force-push is expected and the resolve step must still run.
-
-## Terminology
-
-- **Review** — a top-level review submission by `copilot-pull-request-reviewer`, with a `submittedAt` timestamp and an optional summary body. A single review may contain zero or more Review Threads.
-- **Review Thread** — an inline comment thread attached to a specific code location. Key fields: `id` (GraphQL node ID, e.g. `PRRT_…`), `isResolved` (manually resolved by a maintainer), `isOutdated` (the underlying code changed since the thread was created). Each thread has one or more comments; the first comment is Copilot's suggestion.
-- **Check Run** — a standard CI status object on the HEAD commit. Note: Copilot Reviews do **not** create a Check Run; use the Reviews API to detect completion instead.
-
-## Before the Loop
-
-Run in terminal to establish `OWNER`, `REPO`, and `PR_NUMBER` for use throughout:
-
-```bash
-set -euo pipefail
-OWNER=$(gh repo view --json owner --jq '.owner.login')
-REPO=$(gh repo view --json name --jq '.name')
-PR_NUMBER=$(gh pr view --json number --jq '.number')
-: "${OWNER:?failed to resolve OWNER}"
-: "${REPO:?failed to resolve REPO}"
-: "${PR_NUMBER:?failed to resolve PR_NUMBER}"
-```
-
-Fetch unresolved, non-outdated Copilot Review Threads:
-
-```bash
-set -euo pipefail
-: "${OWNER:?missing OWNER}"
-: "${REPO:?missing REPO}"
-: "${PR_NUMBER:?missing PR_NUMBER}"
-threads_json=$(gh api graphql -f query='
-  query($owner:String!,$repo:String!,$pr:Int!){
-    repository(owner:$owner,name:$repo){
-      pullRequest(number:$pr){
-        reviewThreads(first:100){nodes{id isResolved isOutdated path
-          comments(first:1){nodes{databaseId author{login} createdAt body}}}}}}
-  }' -f owner="$OWNER" -f repo="$REPO" -F pr="$PR_NUMBER" \
-  --jq '.data.repository.pullRequest.reviewThreads.nodes')
-if [[ $(echo "$threads_json" | jq 'length') -ge 100 ]]; then
-  echo "Error: the PR already has 100 threads or more. Aborting."
-  exit 1
-fi
-echo "$threads_json" | jq '[.[] |
-  select(.isResolved==false) |
-  select(.isOutdated==false) |
-  select(.comments.nodes[0].author.login=="copilot-pull-request-reviewer")]'
-```
-
-- **Results non-empty** → proceed directly to the iteration loop.
-- **Results empty** → record `REVIEW_REQUESTED_AT` (current UTC, `YYYY-MM-DDTHH:MM:SSZ`, e.g. `2026-06-16T12:00:00Z`), request a new Copilot Review using the `mcp_github_mcp_se_request_copilot_review` tool (owner, repo, pullNumber), then jump to the **Wait for Review** section below.
-
-## Iteration Loop
-
-Repeat up to **${input:maxIterations:5}** iterations:
-
-### 1. Address threads
-
-Re-fetch unresolved, non-outdated Copilot Review Threads using the same `gh api graphql` query from Before the Loop. Group by file. For each file, read it once and address all its threads in that single pass:
-- Validate each suggestion before accepting it.
-- If a suggestion is invalid or out of scope: reply explaining why, then resolve the thread without changing code.
-- Otherwise: implement the change using TDD where possible; ensure test coverage.
-
-Skip any thread where `isOutdated` is true — the code it references has already changed; Copilot will re-evaluate it in the next Review.
-
-After all threads are addressed, run `rake`. If it fails, capture `rake 2>&1 | tail -n 50` and fix the failure before continuing.
-
-### 2. Commit and push
-
-Amend each change into the most relevant existing commit on the branch based on file name. If a change spans multiple commits or doesn't map clearly to one, ask me which commit to amend into (or whether to create a new commit). Confirm the working tree is clean and `rake` passes, then force push.
-
-### 3. Reply and resolve
-
-For each addressed thread object from the unresolved-threads query, in the same `run_in_terminal` script block that performs reply/resolve, export:
-
-- `COMMENT_DBID=.comments.nodes[0].databaseId`
-- `THREAD_ID=.id`
-- `EXPLANATION` to your plain-language fix summary for that thread
-
-Set the per-thread variables (`COMMENT_DBID`, `THREAD_ID`, `EXPLANATION`) immediately before running the commands below — do not rely on them surviving from a prior terminal invocation.
-
-Then post a reply and resolve it:
-
-```bash
-set -euo pipefail
-: "${OWNER:?missing OWNER}"
-: "${REPO:?missing REPO}"
-: "${COMMENT_DBID:?missing COMMENT_DBID}"
-: "${EXPLANATION:?missing EXPLANATION}"
-
-# Reply (COMMENT_DBID = databaseId of the thread's first comment)
-# Build JSON via jq to safely handle quotes/newlines/special chars in EXPLANATION.
-# EXPLANATION should contain the full reply text (e.g. "Fixed: ..." or "Not addressing this because...").
-BODY_JSON=$(jq -n --arg body "$EXPLANATION" '{body:$body}')
-# Non-fatal: a force-push can mark threads as outdated, causing the REST reply to return 404.
-# Always continue to the GraphQL resolve step regardless.
-gh api "repos/$OWNER/$REPO/pulls/comments/$COMMENT_DBID/replies" \
-  -X POST --input - <<<"$BODY_JSON" \
-  || echo "Warning: reply POST failed (thread may be outdated after force-push) — skipping reply, will still resolve"
-
-: "${THREAD_ID:?missing THREAD_ID}"
-
-# Resolve (THREAD_ID = GraphQL node id, e.g. PRRT_...)
-gh api graphql \
-  -f query='mutation($id:ID!){resolveReviewThread(input:{threadId:$id}){thread{isResolved}}}' \
-  -f id="$THREAD_ID"
-```
-
-### 4. Request review
-
-**If this was the ${input:maxIterations:5}th iteration**, skip steps 4 and 6 entirely — go directly to step 5 (Report) and then produce the Final Report.
-
-Otherwise, capture `REVIEW_REQUESTED_AT` by running `date -u +%Y-%m-%dT%H:%M:%SZ` in the terminal immediately before requesting the review. Then request a new Copilot Review using the `mcp_github_mcp_se_request_copilot_review` tool (owner, repo, pullNumber).
-
-### 5. Report
-
-List what was addressed and how each issue was resolved.
-
-### 6. Wait
-
-Jump to the **Wait for Review** section below. Return here to begin the next iteration once the new Copilot Review has been submitted.
-
-## Wait for Review
-
-**[BLOCKING — do not proceed until complete]** Poll for a new Copilot Review submission using the Reviews API. A Review with `submittedAt >= REVIEW_REQUESTED_AT` is the authoritative completion signal — it fires even when Copilot produces zero Review Threads.
-
-Run the following script via `run_in_terminal` (sync mode, timeout 750000 ms). Set the four variables on the first line to their actual values. On success, capture the script's last output line; on failure, follow the global failure-reporting rule (failed command, exit code, relevant stderr):
-
-```bash
-set -euo pipefail
-# Replace with actual values. REVIEW_REQUESTED_AT = output of `date -u +%Y-%m-%dT%H:%M:%SZ` captured just before requesting the review.
-OWNER="ruby-git"; REPO="ruby-git"; PR_NUMBER="1439"; REVIEW_REQUESTED_AT="2026-06-16T12:00:00Z"
-: "${OWNER:?missing OWNER}"
-: "${REPO:?missing REPO}"
-: "${PR_NUMBER:?missing PR_NUMBER}"
-: "${REVIEW_REQUESTED_AT:?missing REVIEW_REQUESTED_AT}"
-START=$(date +%s)
-for i in $(seq 1 60); do
-  new_review=$(gh pr view "$PR_NUMBER" --repo "$OWNER/$REPO" --json reviews \
-    --jq "[.reviews[] | select(.author.login==\"copilot-pull-request-reviewer\") | select(.submittedAt != null) | select(.submittedAt | fromdateiso8601 >= (\"$REVIEW_REQUESTED_AT\" | fromdateiso8601))] | length") \
-    || { rc=$?; echo "Error: gh pr view failed (exit $rc)"; exit $rc; }
-  if [[ -n "$new_review" && "$new_review" -gt 0 ]]; then
-    raw_nodes=$(gh api graphql -f query='
-      query($owner:String!,$repo:String!,$pr:Int!){
-        repository(owner:$owner,name:$repo){
-          pullRequest(number:$pr){
-            reviewThreads(first:100){nodes{isResolved isOutdated comments(first:1){nodes{author{login}}}}}
-          }
-        }
-      }' -f owner="$OWNER" -f repo="$REPO" -F pr="$PR_NUMBER" \
-      --jq '.data.repository.pullRequest.reviewThreads.nodes') \
-      || { rc=$?; echo "Error: gh api graphql failed (exit $rc)"; exit $rc; }
-    if [[ $(echo "$raw_nodes" | jq 'length') -ge 100 ]]; then
-      echo "Error: the PR already has 100 threads or more. Aborting."
-      exit 1
-    fi
-    count=$(echo "$raw_nodes" | jq "[.[] |
-      select(.isResolved==false) |
-      select(.isOutdated==false) |
-      select(.comments.nodes[0].author.login==\"copilot-pull-request-reviewer\")] | length")
-    echo "done: $count threads"  # count of all unresolved non-outdated Copilot Review Threads
-    exit 0
-  fi
-  elapsed=$(( $(date +%s) - START ))
-  echo "Waiting for Copilot Review to complete... (${elapsed}s elapsed)"
-  sleep 10
-done
-echo "timed out after 60 polls"
-exit 1
-```
-
-**If `timed out after 60 polls`**: stop immediately and ask me whether to re-request the review and retry, or abort.
-
-**If `done: 0 threads`**: the loop is complete — exit.
-
-**If `done: N threads`** (N > 0): begin the next iteration.
-
-## Final Report
-
-- Total iterations completed
-- Total threads addressed
-- Unresolved threads in the final Copilot review (should be 0 if the loop exited cleanly)