wboayue / realflight-bridge / #24

92%

push

refactor: eliminate DRY violations (#26)

* refactor: eliminate DRY violations in SimulatorState and decoders

- Add unit_types module with conditional type aliases for uom/non-uom
- Single SimulatorState definition using type aliases (was duplicated)
- Unify decode_state_field with converter functions (to_velocity, etc.)
- Replace 10+ as_* functions with generic parse_with helper
- Add parse_fuel for conditional fuel unit conversion

Reduces lib.rs by ~80 lines, decoders/mod.rs by ~155 lines.

* refactor: reduce boilerplate in process_request

- Add Response helper methods: success(), success_with(), error(), from_result()
- Refactor process_request to use helpers (66 -> 16 lines)
- Simplify process_request_stubbed with pattern matching

Reduces remote.rs by ~40 lines.

* refactor: centralize DEFAULT_SIMULATOR_HOST constant

- Add DEFAULT_SIMULATOR_HOST to lib.rs
- Update Configuration::default() to use central constant
- Simplify handle_client to use RealFlightLocalBridge::new()
- Remove redundant SIMULATOR_HOST constant from remote.rs

* refactor: use generic Fn trait in parse_with, add missing doc

* chore: ignore plans directory

83 of 101 new or added lines in 3 files covered. (82.18%)

468 of 545 relevant lines covered (85.87%)

83.54 hits per line