fix: join multi-line commit bodies into single body field

Previously `build_commit` split body lines into separate `body` and
`footer` fields, losing content beyond the second paragraph. Now all
non-header lines are joined into `body`. Also disables GitHub API
lookups in release tests to prevent rate-limit-induced timeouts.

Author: jimsynz

lib/git_ops/commit.ex

@@ -96,7 +96,7 @@ defmodule GitOps.Commit do
       end
 
     footer_text =
-      if breaking? && String.starts_with?(body || "", "BREAKING CHANGE:") do
+      if breaking? && footer && String.starts_with?(body || "", "BREAKING CHANGE:") do
         "\n\n" <> footer
       end
 
@@ -221,22 +221,25 @@ defmodule GitOps.Commit do
   end
 
   defp build_commit(parsed, body_lines, author_info, hash) do
-    remaining =
+    body =
       body_lines
       |> Enum.map(&String.trim/1)
       |> Enum.reject(&(&1 == ""))
+      |> Enum.join("\n\n")
+      |> case do
+        "" -> nil
+        text -> text
+      end
 
-    body = Enum.at(remaining, 0)
-    footer = Enum.at(remaining, 1)
     {author_name, author_email} = author_info || {nil, nil}
 
     %__MODULE__{
       type: Enum.at(parsed[:type], 0),
       scope: scopes(parsed[:scope]),
       message: Enum.at(parsed[:message], 0),
       body: body,
-      footer: footer,
-      breaking?: breaking?(parsed[:breaking?], body, footer),
+      footer: nil,
+      breaking?: breaking?(parsed[:breaking?], body, nil),
       author_name: author_name,
       author_email: author_email,
       hash: hash
@@ -248,6 +251,5 @@ defmodule GitOps.Commit do
 
   defp breaking?(breaking, _, _) when not is_nil(breaking), do: true
   defp breaking?(_, "BREAKING CHANGE:" <> _, _), do: true
-  defp breaking?(_, _, "BREAKING CHANGE:" <> _), do: true
   defp breaking?(_, _, _), do: false
 end

test/commit_test.exs

@@ -105,13 +105,13 @@ defmodule GitOps.Test.CommitTest do
     assert [
              %Commit{
                message: "fixed a bug",
-               body: "some text about it",
-               footer: "some even more data about it"
+               body: "some text about it\n\nsome even more data about it",
+               footer: nil
              },
              %Commit{
                message: "improved a thing",
-               body: "some other text about it",
-               footer: "some even more text about it"
+               body: "some other text about it\n\nsome even more text about it",
+               footer: nil
              }
            ] = parse_many!(text)
   end
@@ -215,8 +215,9 @@ defmodule GitOps.Test.CommitTest do
 
       assert %Commit{
                message: "resolve memory leak",
-               body: "The connection pool was not being drained on shutdown.",
-               footer: "This caused OOM errors in production."
+               body:
+                 "The connection pool was not being drained on shutdown.\n\nThis caused OOM errors in production.",
+               footer: nil
              } = parse_one!(text)
     end
 
@@ -231,7 +232,8 @@ defmodule GitOps.Test.CommitTest do
 
       commit = parse_one!(text)
       assert commit.breaking?
-      assert commit.body == "BREAKING CHANGE: The v1 API has been removed."
+      assert commit.body ==
+               "BREAKING CHANGE: The v1 API has been removed.\n\nUsers must migrate to v2."
     end
 
     test "non-conventional first line returns error" do

test/release_test.exs

@@ -17,6 +17,7 @@ defmodule GitOps.Mix.Tasks.Test.ReleaseTest do
       Application.put_env(:git_ops, :manage_readme_version, true)
       Application.put_env(:git_ops, :types, custom: [header: "Custom"], docs: [hidden?: false])
       Application.put_env(:git_ops, :version_tag_prefix, "v")
+      Application.put_env(:git_ops, :github_handle_lookup?, false)
 
       on_exit(fn -> File.rm!(changelog) end)
 
@@ -40,6 +41,7 @@ defmodule GitOps.Mix.Tasks.Test.ReleaseTest do
       Application.put_env(:git_ops, :changelog_file, changelog)
       Application.put_env(:git_ops, :manage_readme_version, true)
       Application.put_env(:git_ops, :types, custom: [header: "Custom"], docs: [hidden?: false])
+      Application.put_env(:git_ops, :github_handle_lookup?, false)
 
       on_exit(fn -> File.rm!(changelog) end)
 
@@ -64,6 +66,7 @@ defmodule GitOps.Mix.Tasks.Test.ReleaseTest do
       Application.put_env(:git_ops, :manage_readme_version, true)
       Application.put_env(:git_ops, :types, custom: [header: "Custom"], docs: [hidden?: false])
       Application.put_env(:git_ops, :version_tag_prefix, "")
+      Application.put_env(:git_ops, :github_handle_lookup?, false)
 
       on_exit(fn -> File.rm!(changelog) end)