refactor: reorganize solve files to avoid name redundancy

- Rename files to remove 'solve' prefix:
  - solve_mode.jl → mode.jl
  - solve_canonical.jl → canonical.jl
  - solve_explicit.jl → explicit.jl
  - solve_descriptive.jl → descriptive.jl

- Delete redundant test files:
  - test_solve_dispatch.jl (functionality moved to test_explicit.jl)
  - test_solve_mode.jl (functionality moved to test_mode.jl)

- Update includes in src/OptimalControl.jl

This makes the file structure cleaner and avoids naming redundancy
while maintaining the same functionality.

Author: ocots

src/OptimalControl.jl

@@ -9,6 +9,7 @@ module OptimalControl
 
 using DocStringExtensions
 using Reexport
+using CTBase
 
 # Imports
 include(joinpath(@__DIR__, "imports", "ctbase.jl"))
@@ -29,11 +30,11 @@ include(joinpath(@__DIR__, "helpers", "strategy_builders.jl"))
 include(joinpath(@__DIR__, "helpers", "component_completion.jl"))
 
 # solve
-include(joinpath(@__DIR__, "solve", "solve_mode.jl"))
+include(joinpath(@__DIR__, "solve", "mode.jl"))
 include(joinpath(@__DIR__, "solve", "mode_detection.jl"))
-include(joinpath(@__DIR__, "solve", "solve_dispatch.jl"))
-include(joinpath(@__DIR__, "solve", "solve_canonical.jl"))
-include(joinpath(@__DIR__, "solve", "solve_explicit.jl"))
+include(joinpath(@__DIR__, "solve", "canonical.jl"))
+include(joinpath(@__DIR__, "solve", "explicit.jl"))
+include(joinpath(@__DIR__, "solve", "descriptive.jl"))
 
 export methods # non useful since it is already in Base
 export get_strategy_registry

src/solve/solve_canonical.jl src/solve/canonical.jlsrc/solve/solve_canonical.jl renamed to src/solve/canonical.jl

@@ -0,0 +1,42 @@
+"""
+$(TYPEDSIGNATURES)
+
+Stub for descriptive mode resolution.
+
+Raises [`CTBase.NotImplemented`](@ref) until `solve_descriptive` is implemented.
+This stub allows testing the orchestration layer (mode detection, dispatch routing)
+before the descriptive mode handler exists.
+
+The `description` vararg will be forwarded to `solve_descriptive` when implemented.
+
+# Arguments
+- `ocp`: The optimal control problem to solve
+- `description`: Symbolic description tokens (e.g., `:collocation`, `:adnlp`, `:ipopt`)
+- `initial_guess`: Normalized initial guess (processed by Layer 1)
+- `display`: Whether to display configuration information
+- `registry`: Strategy registry
+- `kwargs...`: Strategy-specific options
+
+# Throws
+- `CTBase.NotImplemented`: Always — descriptive mode is not yet implemented
+
+# See Also
+- [`DescriptiveMode`](@ref): The dispatch sentinel type
+- [`CommonSolve.solve`](@ref): The entry point that dispatches here
+"""
+function _solve(
+    ::DescriptiveMode,
+    ocp::CTModels.AbstractModel,
+    description::Symbol...;
+    initial_guess::CTModels.AbstractInitialGuess,
+    display::Bool,
+    registry::CTSolvers.Strategies.StrategyRegistry,
+    kwargs...
+)::CTModels.AbstractSolution
+
+    throw(CTBase.NotImplemented(
+        "Descriptive mode is not yet implemented",
+        suggestion="Use explicit mode: solve(ocp; discretizer=..., modeler=..., solver=...)",
+        context="_solve(::DescriptiveMode, ...)"
+    ))
+end

src/solve/descriptive.jl

@@ -1,5 +1,3 @@
-using CTBase
-
 """
 $(TYPEDSIGNATURES)
 
@@ -53,47 +51,4 @@ function _solve(
         components.solver;
         display=display
     )
-end
-
-"""
-$(TYPEDSIGNATURES)
-
-Stub for descriptive mode resolution.
-
-Raises [`CTBase.NotImplemented`](@ref) until `solve_descriptive` is implemented.
-This stub allows testing the orchestration layer (mode detection, dispatch routing)
-before the descriptive mode handler exists.
-
-The `description` vararg will be forwarded to `solve_descriptive` when implemented.
-
-# Arguments
-- `ocp`: The optimal control problem to solve
-- `description`: Symbolic description tokens (e.g., `:collocation`, `:adnlp`, `:ipopt`)
-- `initial_guess`: Normalized initial guess (processed by Layer 1)
-- `display`: Whether to display configuration information
-- `registry`: Strategy registry
-- `kwargs...`: Strategy-specific options
-
-# Throws
-- `CTBase.NotImplemented`: Always — descriptive mode is not yet implemented
-
-# See Also
-- [`DescriptiveMode`](@ref): The dispatch sentinel type
-- [`CommonSolve.solve`](@ref): The entry point that dispatches here
-"""
-function _solve(
-    ::DescriptiveMode,
-    ocp::CTModels.AbstractModel,
-    description::Symbol...;
-    initial_guess::CTModels.AbstractInitialGuess,
-    display::Bool,
-    registry::CTSolvers.Strategies.StrategyRegistry,
-    kwargs...
-)::CTModels.AbstractSolution
-
-    throw(CTBase.NotImplemented(
-        "Descriptive mode is not yet implemented",
-        suggestion="Use explicit mode: solve(ocp; discretizer=..., modeler=..., solver=...)",
-        context="_solve(::DescriptiveMode, ...)"
-    ))
-end
+end