fix: resolve clippy type complexity and serde derive issues

- Add type aliases to fix clippy type complexity warnings in examples
- Fix serde derive imports with conditional compilation guards
- Resolve CI build failures from missing derive macros
- Improve code readability with simplified type signatures

Technical changes:
- examples/bbl_crate_test.rs: Add PidValues, PidWithFf type aliases
- src/export.rs: Add conditional serde imports for feature gates
- Ensure all clippy and formatting checks pass

Author: nerdCopter

.github/copilot-instructions.md

@@ -34,10 +34,11 @@
 
 ## Committing Rules
 - **Commit Conditions:** Only commit if:
-  - `cargo clippy -- -D warnings` passes.
+  - `cargo clippy --all-targets --all-features -- -D warnings` passes.
   - `cargo fmt --all -- --check` passes.
   - `cargo test --verbose` passes.
   - `cargo test --features=cli --verbose` passes.
+  - `cargo build --release` passes with no errors or warnings.
 - **Files to Commit:**
   - Only `src/**/*.rs`, `Cargo.*`, `README.md`, `OVERVIEW.md` and `.gitignore` — never `git add .` or `git add -A`.
   - Follow `.gitignore`.
@@ -46,3 +47,11 @@
   - Check `git diff --cached` before committing.
   - Use concise commit messages and descriptions.
   - Use `feat:`, `fix:`, `docs:` where applicable.
+
+## Mandatory Checks
+- **BEFORE ANY CODE CHANGES:** Always run `cargo clippy --all-targets --all-features -- -D warnings` to catch ALL issues.
+- **BEFORE ANY CODE CHANGES:** Always run `cargo fmt --all -- --check` to ensure formatting compliance.
+- **NO OPTIONAL FEATURES ERRORS:** All feature combinations must compile without errors.
+- **NO FORMATTING VIOLATIONS:** Code must pass `cargo fmt --all -- --check` without any formatting issues.
+- **STRICT COMPLIANCE:** Never skip clippy checks or formatting checks. Never allow warnings to pass.
+- **IMMEDIATE FORMATTING:** Apply `cargo fmt --all` immediately if formatting check fails.

examples/bbl_crate_test.rs

@@ -4,6 +4,12 @@ use clap::Parser;
 use glob::glob;
 use std::path::{Path, PathBuf};
 
+// Type aliases to simplify complex PID tuples
+type PidValues = (i32, i32, i32);
+type PidWithFf = (i32, i32, i32, i32);
+type ThreePidValues = (PidValues, PidValues, PidValues);
+type ThreePidWithFf = (PidWithFf, PidWithFf, PidWithFf);
+
 #[derive(Parser)]
 #[command(name = "bbl_crate_test")]
 #[command(about = "Test program demonstrating BBL parser crate usage")]
@@ -179,13 +185,7 @@ fn display_pid_settings(all_headers: &[String]) {
 }
 
 /// Parse PID values with feedforward from raw header lines (for iNav 4-value format)
-fn parse_pid_with_ff_from_headers(
-    all_headers: &[String],
-) -> Option<(
-    (i32, i32, i32, i32),
-    (i32, i32, i32, i32),
-    (i32, i32, i32, i32),
-)> {
+fn parse_pid_with_ff_from_headers(all_headers: &[String]) -> Option<ThreePidWithFf> {
     let mut roll_pid = None;
     let mut pitch_pid = None;
     let mut yaw_pid = None;
@@ -214,9 +214,7 @@ fn parse_pid_with_ff_from_headers(
 }
 
 /// Parse PID values from raw header lines (for Betaflight rollPID/pitchPID/yawPID format)
-fn parse_pid_from_headers(
-    all_headers: &[String],
-) -> Option<((i32, i32, i32), (i32, i32, i32), (i32, i32, i32))> {
+fn parse_pid_from_headers(all_headers: &[String]) -> Option<ThreePidValues> {
     let mut roll_pid = None;
     let mut pitch_pid = None;
     let mut yaw_pid = None;
@@ -245,7 +243,7 @@ fn parse_pid_from_headers(
 }
 
 /// Parse feedforward values from raw header lines (Betaflight ff_weight format)
-fn parse_feedforward_from_headers(all_headers: &[String]) -> Option<(i32, i32, i32)> {
+fn parse_feedforward_from_headers(all_headers: &[String]) -> Option<PidValues> {
     for header in all_headers {
         if header.starts_with("H ff_weight:") {
             if let Some(value_str) = header.strip_prefix("H ff_weight:") {
@@ -259,7 +257,7 @@ fn parse_feedforward_from_headers(all_headers: &[String]) -> Option<(i32, i32, i
 }
 
 /// Parse PID string in format "P,I,D,FF" and return (P, I, D, FF) tuple
-fn parse_pid_with_ff_string(pid_value: &str) -> Option<(i32, i32, i32, i32)> {
+fn parse_pid_with_ff_string(pid_value: &str) -> Option<PidWithFf> {
     // Remove quotes if present
     let cleaned = pid_value.trim_matches('"');
 
@@ -278,7 +276,7 @@ fn parse_pid_with_ff_string(pid_value: &str) -> Option<(i32, i32, i32, i32)> {
 }
 
 /// Parse PID string in format "P,I,D" or "P,I,D,FF" and return (P, I, D) tuple
-fn parse_pid_string(pid_value: &str) -> Option<(i32, i32, i32)> {
+fn parse_pid_string(pid_value: &str) -> Option<PidValues> {
     // Remove quotes if present
     let cleaned = pid_value.trim_matches('"');
 

src/export.rs

@@ -7,6 +7,9 @@ use crate::types::*;
 use crate::Result;
 use std::path::Path;
 
+#[cfg(feature = "serde")]
+use serde::{Deserialize, Serialize};
+
 /// Export options for various output formats
 #[derive(Debug, Clone, Default)]
 #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]