refactor!: Rename reasoning param to reasoning_effort

Author: ycastorium

lib/sycophant.ex

@@ -51,7 +51,7 @@ defmodule Sycophant do
   are applied automatically.
 
   Common shared params: `:temperature`, `:max_tokens`, `:top_p`, `:top_k`,
-  `:stop`, `:reasoning`, `:reasoning_summary`, `:service_tier`, `:tool_choice`,
+  `:stop`, `:reasoning_effort`, `:reasoning_summary`, `:service_tier`, `:tool_choice`,
   `:parallel_tool_calls`.
 
   Wire-specific params are passed as flat keywords alongside shared ones. For

lib/sycophant/param_defs.ex

@@ -37,7 +37,7 @@ defmodule Sycophant.ParamDefs do
       stop:
         Zoi.list(Zoi.string(), description: "Stop sequences")
         |> Zoi.optional(),
-      reasoning:
+      reasoning_effort:
         Zoi.enum([:none, :minimal, :low, :medium, :high, :xhigh],
           description: "Extended thinking effort level"
         )

lib/sycophant/reasoning.ex

@@ -2,7 +2,7 @@ defmodule Sycophant.Reasoning do
   @moduledoc """
   Reasoning output from an LLM response.
 
-  When a model supports extended thinking (e.g. with the `:reasoning` parameter),
+  When a model supports extended thinking (e.g. with the `:reasoning_effort` parameter),
   the chain-of-thought is available in `response.reasoning.content` as a list
   of `Content.Thinking` structs. Each thinking block may carry `:text` (raw
   chain-of-thought), `:summary` (condensed summary), or both, depending on

lib/sycophant/wire_protocol/anthropic_messages.ex

@@ -512,11 +512,11 @@ defmodule Sycophant.WireProtocol.AnthropicMessages do
 
   defp maybe_put_tool_choice(payload, _), do: payload
 
-  defp maybe_put_thinking(payload, %{reasoning: :none}) do
+  defp maybe_put_thinking(payload, %{reasoning_effort: :none}) do
     Map.put(payload, "thinking", %{"type" => "disabled"})
   end
 
-  defp maybe_put_thinking(payload, %{reasoning: level})
+  defp maybe_put_thinking(payload, %{reasoning_effort: level})
        when is_map_key(@reasoning_budgets, level) do
     budget = Map.fetch!(@reasoning_budgets, level)
     Map.put(payload, "thinking", %{"type" => "enabled", "budget_tokens" => budget})

lib/sycophant/wire_protocol/bedrock_converse.ex

@@ -44,7 +44,7 @@ defmodule Sycophant.WireProtocol.BedrockConverse do
                     :stop,
                     :tool_choice,
                     :parallel_tool_calls,
-                    :reasoning
+                    :reasoning_effort
                   ])
                 )
 
@@ -522,13 +522,13 @@ defmodule Sycophant.WireProtocol.BedrockConverse do
 
   # --- Private: Thinking ---
 
-  defp maybe_put_thinking(payload, %{reasoning: :none}) do
+  defp maybe_put_thinking(payload, %{reasoning_effort: :none}) do
     Map.put(payload, "additionalModelRequestFields", %{
       "thinking" => %{"type" => "disabled"}
     })
   end
 
-  defp maybe_put_thinking(payload, %{reasoning: level})
+  defp maybe_put_thinking(payload, %{reasoning_effort: level})
        when is_map_key(@reasoning_budgets, level) do
     budget = Map.fetch!(@reasoning_budgets, level)
 

lib/sycophant/wire_protocol/google_gemini.ex

@@ -37,7 +37,7 @@ defmodule Sycophant.WireProtocol.GoogleGemini do
                       :top_p,
                       :top_k,
                       :stop,
-                      :reasoning,
+                      :reasoning_effort,
                       :reasoning_summary,
                       :tool_choice,
                       :parallel_tool_calls
@@ -540,9 +540,10 @@ defmodule Sycophant.WireProtocol.GoogleGemini do
       else: Map.put(config, "thinkingConfig", thinking_config)
   end
 
-  defp build_thinking_level(%{reasoning: :none}, _model), do: %{"thinkingBudget" => 0}
+  defp build_thinking_level(%{reasoning_effort: :none}, _model),
+    do: %{"thinkingBudget" => 0}
 
-  defp build_thinking_level(%{reasoning: level}, model)
+  defp build_thinking_level(%{reasoning_effort: level}, model)
        when is_map_key(@thinking_levels, level) do
     if legacy_thinking_model?(model) do
       %{"thinkingBudget" => Map.fetch!(@thinking_budgets, level)}

lib/sycophant/wire_protocol/openai_completions.ex

@@ -400,7 +400,7 @@ defmodule Sycophant.WireProtocol.OpenAICompletions do
     |> maybe_put_reasoning_summary(params)
   end
 
-  defp maybe_put_reasoning_effort(payload, %{reasoning: level}) when not is_nil(level),
+  defp maybe_put_reasoning_effort(payload, %{reasoning_effort: level}) when not is_nil(level),
     do: Map.put(payload, "reasoning_effort", stringify_atom(level))
 
   defp maybe_put_reasoning_effort(payload, _), do: payload

lib/sycophant/wire_protocol/openai_responses.ex

@@ -563,7 +563,7 @@ defmodule Sycophant.WireProtocol.OpenAIResponses do
   defp maybe_put_reasoning(payload, params) do
     reasoning =
       %{}
-      |> maybe_put("effort", Map.get(params, :reasoning))
+      |> maybe_put("effort", Map.get(params, :reasoning_effort))
       |> maybe_put("summary", Map.get(params, :reasoning_summary))
 
     if map_size(reasoning) > 0 do