style(router): cargo fmt + clippy clean (hero_router & hero_router_test) #122

Merged

mahmoud merged 1 commit from development_fix_fmt into development

2026-06-14 16:44:11 +00:00

mahmoud commented

2026-06-14 16:31:51 +00:00

Owner

Follow-up to #121 (SSE alignment), which merged the logic but left the formatting uncommitted in the working tree. This brings both crates to fmt-clean + clippy-clean.

Changes (no behavior change)

cargo fmt across hero_router + hero_router_test (31 files)
clippy: drop redundant let lines = lines; rebind (routes.rs)
clippy: checked_div for avg-latency / percent (runner.rs, scenarios.rs)
clippy: &Path instead of &PathBuf (hero_router_test main.rs)
clippy: fix doc-list overindent / leading-+ list markers (fake_service.rs, main.rs parse_bind doc)

Verification

cargo fmt --check → clean
cargo clippy -p hero_router -p hero_router_test --all-targets → 0 warnings
cargo test -p hero_router → 143 + 5 + 5 pass

🤖 Generated with Claude Code

Follow-up to #121 (SSE alignment), which merged the logic but left the formatting uncommitted in the working tree. This brings both crates to **fmt-clean + clippy-clean**. ### Changes (no behavior change) - `cargo fmt` across `hero_router` + `hero_router_test` (31 files) - clippy: drop redundant `let lines = lines;` rebind (`routes.rs`) - clippy: `checked_div` for avg-latency / percent (`runner.rs`, `scenarios.rs`) - clippy: `&Path` instead of `&PathBuf` (`hero_router_test` `main.rs`) - clippy: fix doc-list overindent / leading-`+` list markers (`fake_service.rs`, `main.rs` `parse_bind` doc) ### Verification - `cargo fmt --check` → clean - `cargo clippy -p hero_router -p hero_router_test --all-targets` → 0 warnings - `cargo test -p hero_router` → 143 + 5 + 5 pass 🤖 Generated with [Claude Code](https://claude.com/claude-code)

mahmoud added 1 commit

2026-06-14 16:31:51 +00:00

style(router): cargo fmt workspace + clippy fixes

Build & Test / check (pull_request) Successful in 9m38s

Details

72ea40cda0

PR #121 (SSE alignment) merged the logic but not the formatting that was
left uncommitted in the working tree. This brings hero_router and
hero_router_test to fmt-clean + clippy-clean state.

- cargo fmt across both crates (31 files)
- clippy: drop redundant `let lines = lines;` rebind (routes.rs)
- clippy: checked_div for avg latency / percent (runner.rs, scenarios.rs)
- clippy: &Path instead of &PathBuf (hero_router_test main.rs)
- clippy: fix doc-list overindent / leading-`+` list markers (fake_service.rs,
  main.rs parse_bind doc)

No behavior change. `cargo fmt --check`, `cargo clippy --all-targets`, and
`cargo test -p hero_router` (143+5+5) all clean.

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