gemini review

Author: FRICK-ELDY

apps/core/lib/mix/tasks/alchemy.ci.ex

@@ -150,21 +150,15 @@ defmodule Mix.Tasks.Alchemy.Ci do
   defp elixir_test(failed, _, _), do: failed
 
   # MIX_ENV=test で子プロセスの mix を起動。[D] の test は GitHub elixir-test と同様。
+  # System.cmd の :env は親プロセスの環境を継承しつつ上書き（PATH 等は維持される）。
   defp mix_cmd(args, root) when is_list(args) do
     System.cmd("elixir", ["-S", "mix"] ++ args,
       cd: root,
       stderr_to_stdout: true,
-      env: env_with(%{"MIX_ENV" => "test"})
+      env: [{"MIX_ENV", "test"}]
     )
   end
 
-  # System.cmd の :env は「全環境の置き換え」のため、PATH 等を落とさないよう現在の環境にマージする。
-  defp env_with(overrides) when is_map(overrides) do
-    System.get_env()
-    |> Map.merge(overrides)
-    |> Map.to_list()
-  end
-
   defp run_step(failed, name, fun) do
     case fun.() do
       {_, 0} ->

apps/network/lib/network/udp/server.ex

@@ -112,8 +112,16 @@ defmodule Network.UDP do
           case port do
             0 ->
               case :inet.port(socket) do
-                {:ok, p} -> p
-                {:error, _} -> port
+                {:ok, p} ->
+                  p
+
+                {:error, reason} ->
+                  Logger.warning(
+                    "[Network.UDP] :inet.port/1 failed after bind on port 0: #{inspect(reason)}; " <>
+                      "state.port may be incorrect"
+                  )
+
+                  port
               end
 
             _ ->