fix: address code review feedback for Phase 5.02 (ContextBuilder) master
Important Issue 1: Formatting - Fixed import order in src/agent/runtime.rs:2 (ContextBudget, ContextBuilder) - Ran cargo fmt to fix line-length and struct literal formatting issues Minor Issue 1: Test Mock Duplication - Extracted TestGraphStore trait mock (~90 lines) to module level (src/context/mod.rs:356-447) - Created create_test_context() helper function to reduce duplication across 6 test functions - Refactored test functions to use shared helper and removed ~540 lines of duplication: - test_build_system_prompt_output_format - test_v2_phase5_ac3_1_dependency_status_done - test_v2_phase5_ac3_2_previous_attempt_present - test_v2_phase5_ac3_3_previous_attempt_absent - test_v2_phase5_ac4_1_budget_aware_trimming - test_v2_phase5_ac4_2_required_sections_never_trimmed Verification: - cargo fmt --check: passed - cargo test --lib: 62 tests passed - cargo clippy: no new warnings - cargo build: success Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>