puma-release 0.1.0

data/test/unit/git_repo_test.rb

@@ -0,0 +1,169 @@
+# frozen_string_literal: true
+
+require_relative "../test_helper"
+
+class GitRepoTest < Minitest::Test
+  def test_push_branch_uses_release_repo_remote_when_present
+    shell = FakeShell.new(
+      {
+        ["git", "remote"] => "origin\nmine\n",
+        ["git", "remote", "get-url", "origin"] => "https://github.com/puma/puma.git\n",
+        ["git", "remote", "get-url", "mine"] => "git@github.com:nateberkopec/puma.git\n"
+      }
+    )
+
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma", metadata_repo: "puma/puma")
+
+    PumaRelease::GitRepo.new(context).push_branch!("release-v7.3.0")
+
+    assert_includes shell.commands, ["git", "push", "-u", "mine", "release-v7.3.0"]
+  end
+
+  def test_push_branch_falls_back_to_release_repo_url_when_remote_is_missing
+    shell = FakeShell.new(
+      {
+        ["git", "remote"] => "origin\n",
+        ["git", "remote", "get-url", "origin"] => "git@github.com:puma/puma.git\n"
+      }
+    )
+
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma", metadata_repo: "puma/puma")
+
+    PumaRelease::GitRepo.new(context).push_branch!("release-v7.3.0")
+
+    assert_includes shell.commands, ["git", "push", "git@github.com:nateberkopec/puma.git", "release-v7.3.0"]
+  end
+
+  def test_push_branch_confirms_the_full_git_command_in_live_mode
+    shell = FakeShell.new(
+      {
+        ["git", "remote"] => "origin\nmine\n",
+        ["git", "remote", "get-url", "origin"] => "https://github.com/puma/puma.git\n",
+        ["git", "remote", "get-url", "mine"] => "git@github.com:nateberkopec/puma.git\n"
+      }
+    )
+    confirmations = []
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma", metadata_repo: "puma/puma")
+    context.define_singleton_method(:confirm_live_git_command!) { |*command| confirmations << command }
+
+    PumaRelease::GitRepo.new(context).push_branch!("release-v7.3.0")
+
+    assert_equal [["git", "push", "-u", "mine", "release-v7.3.0"]], confirmations
+  end
+
+  def test_ensure_clean_base_prefers_metadata_repo_remote_when_present
+    shell = FakeShell.new(
+      {
+        ["git", "rev-parse", "--abbrev-ref", "HEAD"] => "main\n",
+        ["git", "status", "--porcelain"] => "",
+        ["git", "remote"] => "origin\nupstream\n",
+        ["git", "remote", "get-url", "origin"] => "git@github.com:nateberkopec/puma.git\n",
+        ["git", "remote", "get-url", "upstream"] => "https://github.com/puma/puma.git\n",
+        ["git", "rev-parse", "upstream/main"] => "abc123\n",
+        ["git", "rev-parse", "HEAD"] => "abc123\n"
+      }
+    )
+
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma", metadata_repo: "puma/puma", base_branch: "main")
+
+    PumaRelease::GitRepo.new(context).ensure_clean_base!
+
+    assert_includes shell.commands, ["git", "fetch", "upstream", "--quiet"]
+  end
+
+  def test_commit_release_creates_a_signed_commit
+    temp_repo do |repo|
+      version_file = repo.join("lib/puma/const.rb")
+      history_file = repo.join("History.md")
+      version_file.write("")
+      history_file.write("")
+      shell = FakeShell.new
+      context = OpenStruct.new(shell:, version_file:, history_file:)
+
+      PumaRelease::GitRepo.new(context).commit_release!("7.3.0")
+
+      assert_includes shell.commands, ["git", "commit", "-S", "-m", "Release v7.3.0"]
+    end
+  end
+
+  def test_ensure_release_tag_pushed_creates_a_signed_tag
+    shell = FakeShell.new(
+      {
+        ["git", "rev-parse", "HEAD"] => "abc123\n",
+        ["git", "ls-remote", "--tags", "mine", "refs/tags/v7.3.0", "refs/tags/v7.3.0^{}"] => "",
+        ["git", "remote"] => "origin\nmine\n",
+        ["git", "remote", "get-url", "origin"] => "https://github.com/puma/puma.git\n",
+        ["git", "remote", "get-url", "mine"] => "git@github.com:nateberkopec/puma.git\n"
+      }
+    )
+
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma", metadata_repo: "puma/puma")
+
+    PumaRelease::GitRepo.new(context).ensure_release_tag_pushed!("v7.3.0")
+
+    assert_includes shell.commands, ["git", "tag", "-s", "v7.3.0", "-m", "Release v7.3.0"]
+  end
+
+  def test_remote_tag_sha_returns_the_peeled_commit_for_signed_tags
+    shell = FakeShell.new(
+      {
+        ["git", "ls-remote", "--tags", "mine", "refs/tags/v7.3.0", "refs/tags/v7.3.0^{}"] => [
+          "deadbeef\trefs/tags/v7.3.0",
+          "abc123\trefs/tags/v7.3.0^{}"
+        ].join("\n"),
+        ["git", "remote"] => "origin\nmine\n",
+        ["git", "remote", "get-url", "origin"] => "https://github.com/puma/puma.git\n",
+        ["git", "remote", "get-url", "mine"] => "git@github.com:nateberkopec/puma.git\n"
+      }
+    )
+
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma", metadata_repo: "puma/puma")
+
+    assert_equal "abc123", PumaRelease::GitRepo.new(context).remote_tag_sha("v7.3.0")
+  end
+
+  def test_ensure_release_tag_pushed_rejects_an_unsigned_local_tag
+    shell = FakeShell.new(
+      {
+        ["git", "rev-parse", "HEAD"] => "abc123\n",
+        ["git", "rev-parse", "-q", "--verify", "refs/tags/v7.3.0^{commit}"] => "abc123\n",
+        ["git", "ls-remote", "--tags", "mine", "refs/tags/v7.3.0", "refs/tags/v7.3.0^{}"] => "",
+        ["git", "cat-file", "-p", "refs/tags/v7.3.0"] => "tree deadbeef\nauthor Example <example@test> 0 +0000\n\nnot signed\n",
+        ["git", "remote"] => "origin\nmine\n",
+        ["git", "remote", "get-url", "origin"] => "https://github.com/puma/puma.git\n",
+        ["git", "remote", "get-url", "mine"] => "git@github.com:nateberkopec/puma.git\n"
+      }
+    )
+
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma", metadata_repo: "puma/puma")
+
+    error = assert_raises(PumaRelease::Error) { PumaRelease::GitRepo.new(context).ensure_release_tag_pushed!("v7.3.0") }
+
+    assert_includes error.message, "is not GPG-signed"
+  end
+
+  def test_ensure_release_tag_pushed_rejects_a_remote_tag_that_differs_from_the_local_signed_tag
+    shell = FakeShell.new(
+      {
+        ["git", "rev-parse", "HEAD"] => "abc123\n",
+        ["git", "rev-parse", "-q", "--verify", "refs/tags/v7.3.0^{commit}"] => "abc123\n",
+        ["git", "rev-parse", "-q", "--verify", "refs/tags/v7.3.0"] => "localtag\n",
+        ["git", "ls-remote", "--tags", "mine", "refs/tags/v7.3.0", "refs/tags/v7.3.0^{}"] => [
+          "remotetag\trefs/tags/v7.3.0",
+          "abc123\trefs/tags/v7.3.0^{}"
+        ].join("\n"),
+        ["git", "ls-remote", "--tags", "mine", "refs/tags/v7.3.0"] => "remotetag\trefs/tags/v7.3.0\n",
+        ["git", "cat-file", "-p", "refs/tags/v7.3.0"] => "object abc123\ntype commit\ntag v7.3.0\n\n-----BEGIN PGP SIGNATURE-----\n",
+        ["git", "remote"] => "origin\nmine\n",
+        ["git", "remote", "get-url", "origin"] => "https://github.com/puma/puma.git\n",
+        ["git", "remote", "get-url", "mine"] => "git@github.com:nateberkopec/puma.git\n"
+      }
+    )
+
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma", metadata_repo: "puma/puma")
+
+    error = assert_raises(PumaRelease::Error) { PumaRelease::GitRepo.new(context).ensure_release_tag_pushed!("v7.3.0") }
+
+    assert_includes error.message, "does not match the local signed tag"
+  end
+end

data/test/unit/github_client_test.rb

@@ -0,0 +1,90 @@
+# frozen_string_literal: true
+
+require_relative "../test_helper"
+
+class GitHubClientTest < Minitest::Test
+  def test_edit_release_target_passes_target_to_gh
+    shell = FakeShell.new(
+      {
+        ["gh", "release", "edit", "v7.3.0", "--repo", "nateberkopec/puma", "--target", "abc123"] => FakeShell::Result.new(stdout: "", stderr: "", success?: true, exitstatus: 0),
+        ["gh", "release", "view", "v7.3.0", "--repo", "nateberkopec/puma", "--json", "tagName,name,isDraft,body,url,assets,targetCommitish"] => FakeShell::Result.new(stdout: '{"tagName":"v7.3.0","targetCommitish":"abc123"}', stderr: "", success?: true, exitstatus: 0)
+      }
+    )
+
+    confirmations = []
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma")
+    context.define_singleton_method(:confirm_live_gh_command!) { |*command| confirmations << command }
+    release = PumaRelease::GitHubClient.new(context).edit_release_target("v7.3.0", "abc123")
+
+    assert_equal [["gh", "release", "edit", "v7.3.0", "--repo", "nateberkopec/puma", "--target", "abc123"]], confirmations
+    assert_equal "abc123", release.fetch("targetCommitish")
+  end
+
+  def test_edit_release_title_passes_title_to_gh
+    shell = FakeShell.new(
+      {
+        ["gh", "release", "edit", "v7.3.0", "--repo", "nateberkopec/puma", "--title", "v7.3.0 - INSERT CODENAME HERE"] => FakeShell::Result.new(stdout: "", stderr: "", success?: true, exitstatus: 0),
+        ["gh", "release", "view", "v7.3.0", "--repo", "nateberkopec/puma", "--json", "tagName,name,isDraft,body,url,assets,targetCommitish"] => FakeShell::Result.new(stdout: '{"tagName":"v7.3.0","name":"v7.3.0 - INSERT CODENAME HERE"}', stderr: "", success?: true, exitstatus: 0)
+      }
+    )
+
+    confirmations = []
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma")
+    context.define_singleton_method(:confirm_live_gh_command!) { |*command| confirmations << command }
+    release = PumaRelease::GitHubClient.new(context).edit_release_title("v7.3.0", "v7.3.0 - INSERT CODENAME HERE")
+
+    assert_equal [["gh", "release", "edit", "v7.3.0", "--repo", "nateberkopec/puma", "--title", "v7.3.0 - INSERT CODENAME HERE"]], confirmations
+    assert_equal "v7.3.0 - INSERT CODENAME HERE", release.fetch("name")
+  end
+
+  def test_create_release_pr_confirms_before_writing
+    shell = FakeShell.new(
+      {
+        ["gh", "pr", "create", "--repo", "puma/puma", "--base", "main", "--head", "release-v7.3.0", "--title", "Release v7.3.0", "--body", "compare"] => "https://github.com/puma/puma/pull/1\n"
+      }
+    )
+
+    confirmations = []
+    context = OpenStruct.new(shell:, release_repo: "puma/puma", base_branch: "main")
+    context.define_singleton_method(:confirm_live_gh_command!) { |*command| confirmations << command }
+
+    pr_url = PumaRelease::GitHubClient.new(context).create_release_pr("Release v7.3.0", "release-v7.3.0", body: "compare")
+
+    assert_equal [["gh", "pr", "create", "--repo", "puma/puma", "--base", "main", "--head", "release-v7.3.0", "--title", "Release v7.3.0", "--body", "compare"]], confirmations
+    assert_equal "https://github.com/puma/puma/pull/1", pr_url
+  end
+
+  def test_retag_release_updates_the_release_tag_name_via_api
+    shell = FakeShell.new(
+      {
+        ["gh", "api", "repos/nateberkopec/puma/releases/tags/v7.3.0-proposal"] => FakeShell::Result.new(stdout: '{"id":123}', stderr: "", success?: true, exitstatus: 0),
+        ["gh", "api", "-X", "PATCH", "repos/nateberkopec/puma/releases/123", "-f", "tag_name=v7.3.0", "-f", "target_commitish=abc123"] => FakeShell::Result.new(stdout: "", stderr: "", success?: true, exitstatus: 0),
+        ["gh", "release", "view", "v7.3.0", "--repo", "nateberkopec/puma", "--json", "tagName,name,isDraft,body,url,assets,targetCommitish"] => FakeShell::Result.new(stdout: '{"tagName":"v7.3.0","targetCommitish":"abc123"}', stderr: "", success?: true, exitstatus: 0)
+      }
+    )
+
+    confirmations = []
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma")
+    context.define_singleton_method(:confirm_live_gh_command!) { |*command| confirmations << command }
+
+    release = PumaRelease::GitHubClient.new(context).retag_release("v7.3.0-proposal", "v7.3.0", target: "abc123")
+
+    assert_equal [["gh", "api", "-X", "PATCH", "repos/nateberkopec/puma/releases/123", "-f", "tag_name=v7.3.0", "-f", "target_commitish=abc123"]], confirmations
+    assert_equal "v7.3.0", release.fetch("tagName")
+  end
+
+  def test_delete_tag_ref_deletes_a_remote_tag_ref_via_api
+    shell = FakeShell.new(
+      {
+        ["gh", "api", "-X", "DELETE", "repos/nateberkopec/puma/git/refs/tags/v7.3.0-proposal"] => FakeShell::Result.new(stdout: "", stderr: "", success?: true, exitstatus: 0)
+      }
+    )
+
+    confirmations = []
+    context = OpenStruct.new(shell:, release_repo: "nateberkopec/puma")
+    context.define_singleton_method(:confirm_live_gh_command!) { |*command| confirmations << command }
+
+    assert PumaRelease::GitHubClient.new(context).delete_tag_ref("v7.3.0-proposal")
+    assert_equal [["gh", "api", "-X", "DELETE", "repos/nateberkopec/puma/git/refs/tags/v7.3.0-proposal"]], confirmations
+  end
+end

data/test/unit/github_command_test.rb

@@ -0,0 +1,153 @@
+# frozen_string_literal: true
+
+require_relative "../test_helper"
+
+class GithubCommandTest < Minitest::Test
+  def test_checks_git_and_gh_and_requires_a_clean_main_checkout
+    calls = []
+    context = Object.new
+    context.define_singleton_method(:check_dependencies!) { |*commands| calls << commands }
+    context.define_singleton_method(:announce_live_mode!) {}
+    context.define_singleton_method(:ensure_release_writes_allowed!) {}
+
+    git_repo = Object.new
+    git_repo.define_singleton_method(:ensure_clean_base!) { calls << :clean_main }
+
+    repo_files = Object.new
+    repo_files.define_singleton_method(:current_version) { raise PumaRelease::Error, "stop" }
+
+    command = PumaRelease::Commands::Github.allocate
+    command.instance_variable_set(:@context, context)
+    command.instance_variable_set(:@git_repo, git_repo)
+    command.instance_variable_set(:@repo_files, repo_files)
+    command.instance_variable_set(:@github, Object.new)
+
+    error = assert_raises(PumaRelease::Error) { command.call }
+
+    assert_equal "stop", error.message
+    assert_includes calls, ["git", "gh"]
+    assert_includes calls, :clean_main
+  end
+
+  def test_ensures_the_signed_tag_is_pushed_before_creating_a_release
+    Dir.mktmpdir do |dir|
+      repo_dir = Pathname(dir)
+      context = OpenStruct.new(
+        repo_dir:,
+        history_file: repo_dir.join("History.md"),
+        events: Object.new,
+        ui: Object.new
+      )
+      context.history_file.write("## 7.2.0 / 2026-01-20\n\n* Bugfixes\n  * One fix ([#1])\n")
+      context.define_singleton_method(:check_dependencies!) { |_git, _gh| }
+      context.define_singleton_method(:announce_live_mode!) {}
+      context.define_singleton_method(:ensure_release_writes_allowed!) {}
+      context.events.define_singleton_method(:publish) { |_name, _payload| }
+      context.ui.define_singleton_method(:info) { |_message| }
+
+      git_repo = Object.new
+      git_repo.define_singleton_method(:ensure_clean_base!) {}
+      git_repo.define_singleton_method(:release_tag) { |_version| "v7.2.0" }
+      git_repo.define_singleton_method(:proposal_tag) { |_version| "v7.2.0-proposal" }
+      git_repo.define_singleton_method(:ensure_release_tag_pushed!) { |_tag| raise PumaRelease::Error, "stop" }
+
+      repo_files = Object.new
+      repo_files.define_singleton_method(:current_version) { "7.2.0" }
+      repo_files.define_singleton_method(:extract_history_section) { |_version| "* Bugfixes\n  * One fix ([#1])" }
+      repo_files.define_singleton_method(:release_name) { |_version| "v7.2.0" }
+
+      github = Object.new
+      github.define_singleton_method(:create_release) { |_tag, _body, title:, draft:| flunk "create_release should not be called before the tag is ensured" }
+      github.define_singleton_method(:release) { |_tag| nil }
+
+      command = PumaRelease::Commands::Github.allocate
+      command.instance_variable_set(:@context, context)
+      command.instance_variable_set(:@git_repo, git_repo)
+      command.instance_variable_set(:@repo_files, repo_files)
+      command.instance_variable_set(:@github, github)
+
+      repo_dir.join("pkg").mkpath
+      repo_dir.join("pkg/puma-7.2.0.gem").write("")
+      repo_dir.join("pkg/puma-7.2.0-java.gem").write("")
+
+      error = assert_raises(PumaRelease::Error) { command.call }
+
+      assert_equal "stop", error.message
+    end
+  end
+
+  def test_promotes_the_proposal_release_to_the_final_tag
+    Dir.mktmpdir do |dir|
+      repo_dir = Pathname(dir)
+      context = OpenStruct.new(
+        repo_dir:,
+        history_file: repo_dir.join("History.md"),
+        events: Object.new,
+        ui: Object.new
+      )
+      context.history_file.write("## 7.2.0 / 2026-01-20\n\n* Bugfixes\n  * One fix ([#1])\n")
+      published = []
+      infos = []
+      context.define_singleton_method(:check_dependencies!) { |_git, _gh| }
+      context.define_singleton_method(:announce_live_mode!) {}
+      context.define_singleton_method(:ensure_release_writes_allowed!) {}
+      context.events.define_singleton_method(:publish) { |name, payload| published << [name, payload] }
+      context.ui.define_singleton_method(:info) { |message| infos << message }
+
+      git_repo = Object.new
+      git_repo.define_singleton_method(:ensure_clean_base!) {}
+      git_repo.define_singleton_method(:release_tag) { |_version| "v7.2.0" }
+      git_repo.define_singleton_method(:proposal_tag) { |_version| "v7.2.0-proposal" }
+      git_repo.define_singleton_method(:ensure_release_tag_pushed!) { |_tag| }
+      git_repo.define_singleton_method(:local_tag_sha) { |_tag| "abc123" }
+      git_repo.define_singleton_method(:remote_tag_sha) { |_tag| "proposal123" }
+
+      repo_files = Object.new
+      repo_files.define_singleton_method(:current_version) { "7.2.0" }
+      repo_files.define_singleton_method(:extract_history_section) { |_version| "* Bugfixes\n  * One fix ([#1])" }
+      repo_files.define_singleton_method(:release_name) { |_version| "v7.2.0" }
+
+      calls = []
+      github = Object.new
+      github.define_singleton_method(:release) do |tag|
+        case tag
+        when "v7.2.0" then nil
+        when "v7.2.0-proposal" then {"isDraft" => true, "targetCommitish" => "release-v7.2.0", "name" => "v7.2.0", "body" => "* Bugfixes\n  * One fix ([#1])", "url" => "https://example.test/release"}
+        end
+      end
+      github.define_singleton_method(:retag_release) do |old_tag, new_tag, target:|
+        calls << [:retag_release, old_tag, new_tag, target]
+        {"isDraft" => true, "targetCommitish" => target, "name" => "v7.2.0", "body" => "* Bugfixes\n  * One fix ([#1])", "url" => "https://example.test/release"}
+      end
+      github.define_singleton_method(:edit_release_target) { |_tag, _target| flunk "edit_release_target should not be called when retag_release already set the final target" }
+      github.define_singleton_method(:edit_release_title) { |_tag, _title| flunk "edit_release_title should not be called when the release title already matches" }
+      github.define_singleton_method(:edit_release_notes) { |_tag, _body| flunk "edit_release_notes should not be called when the release notes already match" }
+      github.define_singleton_method(:upload_release_assets) { |tag, *paths| calls << [:upload_release_assets, tag, paths.map { |path| File.basename(path) }] }
+      github.define_singleton_method(:publish_release) do |tag|
+        calls << [:publish_release, tag]
+        {"isDraft" => false, "url" => "https://example.test/release"}
+      end
+      github.define_singleton_method(:delete_release) { |_tag, allow_failure:| calls << [:delete_release, allow_failure] }
+      github.define_singleton_method(:delete_tag_ref) { |tag, allow_failure:| calls << [:delete_tag_ref, tag, allow_failure] }
+      github.define_singleton_method(:create_release) { |_tag, _body, title:, draft:| flunk "create_release should not be called when a proposal release already exists" }
+
+      command = PumaRelease::Commands::Github.allocate
+      command.instance_variable_set(:@context, context)
+      command.instance_variable_set(:@git_repo, git_repo)
+      command.instance_variable_set(:@repo_files, repo_files)
+      command.instance_variable_set(:@github, github)
+
+      repo_dir.join("pkg").mkpath
+      repo_dir.join("pkg/puma-7.2.0.gem").write("")
+      repo_dir.join("pkg/puma-7.2.0-java.gem").write("")
+
+      assert_equal :complete, command.call
+      assert_includes infos, "Promoting draft release from v7.2.0-proposal to v7.2.0..."
+      assert_includes calls, [:retag_release, "v7.2.0-proposal", "v7.2.0", "abc123"]
+      assert_includes calls, [:upload_release_assets, "v7.2.0", ["puma-7.2.0.gem", "puma-7.2.0-java.gem"]]
+      assert_includes calls, [:publish_release, "v7.2.0"]
+      assert_includes calls, [:delete_tag_ref, "v7.2.0-proposal", true]
+      assert_equal [[:release_published, {tag: "v7.2.0", url: "https://example.test/release"}]], published
+    end
+  end
+end

data/test/unit/options_test.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require_relative "../test_helper"
+
+class OptionsTest < Minitest::Test
+  def test_parse_sets_live_flag
+    options = PumaRelease::Options.parse(["--live"])
+
+    assert_equal true, options.fetch(:live)
+  end
+
+  def test_parse_sets_skip_ci_check_flag
+    options = PumaRelease::Options.parse(["--skip-ci-check"])
+
+    assert_equal true, options.fetch(:skip_ci_check)
+  end
+end

data/test/unit/prepare_test.rb

@@ -0,0 +1,136 @@
+# frozen_string_literal: true
+
+require_relative "../test_helper"
+
+class PrepareTest < Minitest::Test
+  class FakeUI
+    attr_reader :infos, :warnings
+
+    def initialize
+      @infos = []
+      @warnings = []
+    end
+
+    def info(message)
+      infos << message
+    end
+
+    def warn(message)
+      warnings << message
+    end
+  end
+
+  def test_ensure_green_ci_skips_when_flag_is_set
+    ui = FakeUI.new
+    context = OpenStruct.new(skip_ci_check?: true, ui:)
+    git_repo = OpenStruct.new(head_sha: "abc123")
+    prepare = PumaRelease::Commands::Prepare.allocate
+    prepare.instance_variable_set(:@context, context)
+    prepare.instance_variable_set(:@git_repo, git_repo)
+
+    checker = Object.new
+    def checker.ensure_green!(_sha)
+      flunk "CiChecker should not be called when CI is skipped"
+    end
+    prepare.define_singleton_method(:ci_checker) { checker }
+
+    prepare.send(:ensure_green_ci!)
+
+    assert_equal ["Skipping CI check because --skip-ci-check was set."], ui.warnings
+    assert_empty ui.infos
+  end
+
+  def test_ensure_green_ci_checks_head_when_flag_is_not_set
+    ui = FakeUI.new
+    context = OpenStruct.new(skip_ci_check?: false, ui:)
+    git_repo = OpenStruct.new(head_sha: "abc123")
+    prepare = PumaRelease::Commands::Prepare.allocate
+    prepare.instance_variable_set(:@context, context)
+    prepare.instance_variable_set(:@git_repo, git_repo)
+
+    checker = Object.new
+    calls = []
+    checker.define_singleton_method(:ensure_green!) { |sha| calls << sha }
+    prepare.define_singleton_method(:ci_checker) { checker }
+
+    prepare.send(:ensure_green_ci!)
+
+    assert_equal ["Checking CI status for HEAD..."], ui.infos
+    assert_equal ["abc123"], calls
+    assert_empty ui.warnings
+  end
+
+  def test_pr_comment_starts_with_llm_attribution
+    context = Object.new
+    context.define_singleton_method(:comment_attribution) do |model_name|
+      "This comment was written by #{model_name} working on behalf of [puma-release](https://github.com/nateberkopec/puma-release)."
+    end
+    context.define_singleton_method(:comment_author_model_name) { "fallback-model" }
+    prepare = PumaRelease::Commands::Prepare.allocate
+    prepare.instance_variable_set(:@context, context)
+
+    comment = prepare.send(
+      :pr_comment,
+      {
+        "model_name" => "openai-codex/gpt-5.4",
+        "bump_type" => "minor",
+        "reasoning_markdown" => "Because of [this commit](https://github.com/puma/puma/commit/abc)."
+      },
+      nil
+    )
+
+    assert_match(%r{\AThis comment was written by openai-codex/gpt-5\.4 working on behalf of \[puma-release\]\(https://github.com/nateberkopec/puma-release\)\.}, comment)
+  end
+
+  def test_ensure_draft_release_uses_the_proposal_tag
+    git_repo = Object.new
+    git_repo.define_singleton_method(:proposal_tag) { |_version| "v7.3.0-proposal" }
+
+    repo_files = Object.new
+    repo_files.define_singleton_method(:release_name) { |_version| "v7.3.0 - Example" }
+    repo_files.define_singleton_method(:extract_history_section) { |_version| "* Features\n  * Example ([#1])" }
+
+    calls = []
+    github = Object.new
+    github.define_singleton_method(:release) { |_tag| nil }
+    github.define_singleton_method(:create_release) do |tag, body, title:, draft:, target:|
+      calls << [:create_release, tag, body, title, draft, target]
+      {"name" => title, "body" => body, "targetCommitish" => target}
+    end
+    github.define_singleton_method(:edit_release_target) { |_tag, _target| flunk "edit_release_target should not be called when the draft release target already matches" }
+    github.define_singleton_method(:edit_release_title) { |_tag, _title| flunk "edit_release_title should not be called when the title already matches" }
+    github.define_singleton_method(:edit_release_notes) { |_tag, _body| flunk "edit_release_notes should not be called when the notes already match" }
+
+    prepare = PumaRelease::Commands::Prepare.allocate
+    prepare.instance_variable_set(:@context, OpenStruct.new(history_file: Pathname("History.md")))
+    prepare.instance_variable_set(:@git_repo, git_repo)
+    prepare.instance_variable_set(:@repo_files, repo_files)
+    prepare.instance_variable_set(:@github, github)
+
+    prepare.send(:ensure_draft_release, "7.3.0", "release-v7.3.0")
+
+    assert_equal [[:create_release, "v7.3.0-proposal", "* Features\n  * Example ([#1])", "v7.3.0 - Example", true, "release-v7.3.0"]], calls
+  end
+
+  def test_show_version_recommendation_prints_reasoning_and_breaking_changes
+    ui = FakeUI.new
+    context = OpenStruct.new(ui:)
+    prepare = PumaRelease::Commands::Prepare.allocate
+    prepare.instance_variable_set(:@context, context)
+
+    output = capture_io do
+      prepare.send(
+        :show_version_recommendation,
+        {
+          "reasoning_markdown" => "Major because of [this commit](https://github.com/puma/puma/commit/abc123).",
+          "breaking_changes" => ["Dropped support for an older Rack integration"]
+        }
+      )
+    end.first
+
+    assert_equal ["Version bump recommendation:"], ui.infos
+    assert_equal ["Potential breaking changes:"], ui.warnings
+    assert_includes output, "Major because of [this commit](https://github.com/puma/puma/commit/abc123)."
+    assert_includes output, "- Dropped support for an older Rack integration"
+  end
+end