Drop stale undefined_exports broken marker in OptimizationManopt QA

Aqua now reports "Unexpected Pass" for OptimizationManopt's undefined_exports
check: the `solve!` reexport clash that motivated `broken = true` no longer
produces an undefined export, so the broken marker errors the test. Remove it
and let the check run normally.

Verified the other libs with the same marker are still genuinely broken and
keep theirs: OptimizationNOMAD (`OptimizationNOMAD.solve` is undefined), plus
OptimizationMadNLP/OptimizationMetaheuristics (their QA jobs pass with the
marker in place).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: jClugstor

lib/OptimizationManopt/test/qa/qa.jl

@@ -2,15 +2,10 @@ using OptimizationManopt, Aqua, JET
 using Test
 
 @testset "Aqua" begin
-    # `@reexport using Manopt` exports `solve!`, which clashes with SciMLBase's `solve!`
-    # brought in transitively via OptimizationBase. The clash leaves OptimizationManopt's
-    # `solve!` export pointing at neither binding; mark broken until the reexport is
-    # restructured.
     # Manifolds is declared because the curvature analysis path may pull it in,
     # but no symbol from it is currently used in src — ignore it for now.
     Aqua.test_all(
         OptimizationManopt;
-        undefined_exports = (; broken = true),
         stale_deps = (; ignore = [:Manifolds])
     )
 end