The Rust workflow lint job failed on master:
- cargo fmt --check: imports in src/main.rs were misordered
(mcp_server/models/pipeline lines out of alphabetical order,
McpError/McpRequest/McpResponse tuple unsorted) from b843146.
- cargo clippy -D warnings: clippy::new_without_default now fires on
DealAnalyzer, ReportExtractor, OptimizationParser, StageExecutor
and McpServer because the lib target (src/lib.rs) makes their
pub constructors part of the public API.
Fix: reorder main.rs imports per rustfmt; add Default impls
delegating to new() for the five types.
Devid HW
608b07cead