Clean up

Author: ericmj

lib/hex/api/client.ex

@@ -9,10 +9,10 @@ defmodule Hex.API.Client do
         http_user_agent_fragment: user_agent_fragment()
     }
 
-    config = maybe_put_api_key(config, opts)
-    config = maybe_put_organization(config, opts)
-    config = maybe_put_repository(config, opts)
     config
+    |> maybe_put_api_key(opts)
+    |> maybe_put_organization(opts)
+    |> maybe_put_repository(opts)
   end
 
   defp maybe_put_api_key(config, opts) do

lib/hex/http.ex

@@ -6,6 +6,7 @@ defmodule Hex.HTTP do
   @request_timeout 15_000
   @request_redirects 3
   @request_retries 2
+  @chunk_size 10_000
 
   @spec config() :: :mix_hex_core.config()
   def config do
@@ -383,15 +384,11 @@ defmodule Hex.HTTP do
     end
   end
 
-  @chunk_size 10_000
-
   defp wrap_body_with_progress(body, progress_callback) do
     case body do
       {content_type, binary_body}
       when is_binary(binary_body) and is_function(progress_callback, 1) ->
-        # Create streaming function for httpc chunked upload - match old API pattern exactly
         total_size = byte_size(binary_body)
-        # Initialize progress bar
         progress_callback.(0)
 
         body_fn = fn
@@ -405,13 +402,11 @@ defmodule Hex.HTTP do
             :eof
         end
 
-        # Return chunked body AND content-length header (required for chunked uploads)
         body_result = {content_type, {body_fn, 0}}
         headers = %{"content-length" => Integer.to_string(total_size)}
         {body_result, headers}
 
       _other ->
-        # No progress callback or not a binary body, use as-is
         {body, %{}}
     end
   end

lib/mix/tasks/hex.user.ex

@@ -280,22 +280,7 @@ defmodule Mix.Tasks.Hex.User do
 
     case Hex.API.Key.delete_all(auth) do
       {:ok, {code, _headers, body}} when code in 200..299 ->
-        auth_key_deleted =
-          cond do
-            is_list(body) ->
-              Enum.any?(body, fn
-                item when is_map(item) -> Map.get(item, "authing_key") == true
-                _ -> false
-              end)
-
-            is_map(body) ->
-              Map.get(body, "authing_key") == true
-
-            true ->
-              false
-          end
-
-        if auth_key_deleted do
+        if Map.get(body, "authing_key") == true do
           Mix.Tasks.Hex.User.run(["deauth"])
         end