[CodeCleanup] Replace individual optimization calls with recipeData:Optimize

[Copilot]

RecipeData:Optimize is the canonical async method for orchestrating recipe optimization steps, but several call sites were still manually chaining OptimizeGear/OptimizeReagents/OptimizeConcentration/OptimizeFinishingReagents with nested callbacks and conditional branching.

Changes

• CraftQueue:QueueOpenRecipe — replaced conditional chain of OptimizeGear → OptimizeReagents → OptimizeConcentration (with split if/else for async concentration path) with a single recipeData:Optimize call; button is now consistently disabled for the full duration of optimization
• CraftQueue/UI.lua optimizeProfitButton — resolved the existing TODO: Refactor using RecipeData:Optimize; replaced OptimizeGear + RunNextFrame(OptimizeReagents → OptimizeConcentration → OptimizeFinishingReagents) nested callback pyramid with a single recipeData:Optimize call
• Work order reagent optimization — replaced two RunNextFrame(OptimizeReagents + queueRecipe) branches (patron vs. normal order) with a single recipeData:Optimize { optimizeReagentOptions = ..., finally = queueRecipe }

Before (UI optimize button):

if optimizeProfessionGear then
    recipeData:OptimizeGear(...)
end
RunNextFrame(function()
    recipeData:OptimizeReagents { highestProfit = ... }
    if recipeData.supportsQualities and optimizeConcentration then
        optimizeButton:SetEnabled(false)
        recipeData:OptimizeConcentration {
            finally = function()
                if optimizeFinishingReagents then
                    recipeData:OptimizeFinishingReagents { ... finally = finalizeOptimize }
                else finalizeOptimize() end
            end, ...
        }
    else
        if optimizeFinishingReagents then
            optimizeButton:SetEnabled(false)
            recipeData:OptimizeFinishingReagents { ... finally = finalizeOptimize }
        else finalizeOptimize() end
    end
end)

After:

optimizeButton:SetEnabled(false)
recipeData:Optimize {
    optimizeGear = optimizeProfessionGear,
    optimizeReagentOptions = { highestProfit = optimizeTopProfit },
    optimizeConcentration = optimizeConcentration,
    optimizeConcentrationProgressCallback = function(p) optimizeButton:SetText(...) end,
    optimizeFinishingReagentsOptions = optimizeFinishingReagents and { ... } or nil,
    finally = function() finalizeOptimize() end,
}

Original prompt

---

This section details on the original issue you should resolve

<issue_title>[CodeCleanup] Optimize calls replacement</issue_title>
<issue_description>RecipeData.Optimize is now the best way to call for recipe optimizations

check where recipedata is optimized in the code and if not already using it, refactor the specific optimization calls to use the recipeData.Optimize async method with the correct optimization options configurations</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes [CodeCleanup] Optimize calls replacement #1103

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.