fix: Resolve CI failures - Credo naming and Dialyzer type specs

- Add Credo inline directive for arrayBuffer camelCase (Browser API requirement)
- Fix alias ordering in test file (alphabetical)
- Change read_all spec from String.t() to binary() for correct type
- Fix text/1 function to pass full response struct to read_all
- All CI checks now passing: Credo, Dialyzer, Format, Compile

Author: GSMLG-BOT

lib/http/response.ex

@@ -217,9 +217,9 @@ defmodule HTTP.Response do
   @spec text(t()) :: String.t()
   def text(%__MODULE__{body: body, stream: nil}), do: body
 
-  def text(%__MODULE__{body: body, stream: stream}) do
+  def text(%__MODULE__{body: body, stream: stream} = response) do
     if is_nil(body) and is_pid(stream) do
-      read_all(%__MODULE__{body: body, stream: stream})
+      read_all(response)
     else
       body || ""
     end
@@ -236,7 +236,7 @@ defmodule HTTP.Response do
       iex> HTTP.Response.read_all(response)
       "Hello World"
   """
-  @spec read_all(t()) :: String.t()
+  @spec read_all(t()) :: binary()
   def read_all(%__MODULE__{body: body, stream: nil}), do: body || ""
 
   def read_all(%__MODULE__{body: _body, stream: stream}) do
@@ -410,6 +410,7 @@ defmodule HTTP.Response do
       <<1, 2, 3, 4>>
   """
   @spec arrayBuffer(t()) :: binary()
+  # credo:disable-for-next-line Credo.Check.Readability.FunctionNames
   def arrayBuffer(%__MODULE__{} = response) do
     # arrayBuffer is essentially the same as read_all for binary data
     read_all(response)

test/http/response_browser_api_test.exs

@@ -1,9 +1,9 @@
 defmodule HTTP.ResponseBrowserAPITest do
   use ExUnit.Case, async: true
 
-  alias HTTP.Response
-  alias HTTP.Headers
   alias HTTP.Blob
+  alias HTTP.Headers
+  alias HTTP.Response
 
   describe "Browser Fetch API properties" do
     test "ok is true for 200-299 status codes" do
@@ -41,6 +41,7 @@ defmodule HTTP.ResponseBrowserAPITest do
 
       for {status, expected_text} <- test_cases do
         response = Response.new(status: status)
+
         assert response.status_text == expected_text,
                "Status #{status} should have status_text='#{expected_text}', got '#{response.status_text}'"
       end
@@ -92,7 +93,8 @@ defmodule HTTP.ResponseBrowserAPITest do
 
       # Multiple reads work because response is immutable
       assert Response.text(response) == "Hello"
-      assert Response.text(response) == "Hello"  # Works in Elixir
+      # Works in Elixir
+      assert Response.text(response) == "Hello"
     end
 
     test "body_used flag can be manually checked" do
@@ -253,7 +255,8 @@ defmodule HTTP.ResponseBrowserAPITest do
   describe "Response.blob/1" do
     test "returns blob with correct data and type" do
       headers = Headers.new([{"content-type", "image/png"}])
-      data = <<137, 80, 78, 71>>  # PNG magic bytes
+      # PNG magic bytes
+      data = <<137, 80, 78, 71>>
       response = Response.new(status: 200, body: data, headers: headers)
 
       blob = Response.blob(response)