Run a more exhasutive clippy check

Signed-off-by: Mark Rossett <[email protected]>

Author: marosset

Justfile

@@ -176,6 +176,46 @@ clippy-apply-fix-unix:
 clippy-apply-fix-windows:
     cargo clippy --target x86_64-pc-windows-msvc --fix --all 
 
+# Generate and test all possible feature combinations for a package
+clippy-feature-combinations package target=default-target:
+    #!/usr/bin/env bash
+    set -euo pipefail
+    
+    # Get all features for the package
+    features=$(cargo metadata --format-version 1 --no-deps | jq -r '.packages[] | select(.name == "{{package}}") | .features | keys[]' | grep -v default || true)
+    
+    echo "Testing {{package}} with no features..."
+    cargo clippy -p {{package}} --all-targets --no-default-features --profile={{ if target == "debug" { "dev" } else { target } }} -- -D warnings
+    
+    echo "Testing {{package}} with default features..."
+    cargo clippy -p {{package}} --all-targets --profile={{ if target == "debug" { "dev" } else { target } }} -- -D warnings
+    
+    # Test each feature individually
+    for feature in $features; do
+        echo "Testing {{package}} with feature: $feature"
+        cargo clippy -p {{package}} --all-targets --no-default-features --features "$feature" --profile={{ if target == "debug" { "dev" } else { target } }} -- -D warnings || echo "Feature $feature failed for {{package}}"
+    done
+    
+    # Test all features together
+    if [ -n "$features" ]; then
+        all_features=$(echo $features | tr '\n' ',' | sed 's/,$//')
+        echo "Testing {{package}} with all features: $all_features"
+        cargo clippy -p {{package}} --all-targets --no-default-features --features "$all_features" --profile={{ if target == "debug" { "dev" } else { target } }} -- -D warnings || echo "All features failed for {{package}}"
+    fi
+
+# Run clippy with feature combinations for all packages
+clippy-exhaustive target=default-target: (witguest-wit)
+    ./hack/clippy-package-features.sh hyperlight-host {{ target }}
+    ./hack/clippy-package-features.sh hyperlight-guest {{ target }}
+    ./hack/clippy-package-features.sh hyperlight-guest-bin {{ target }}
+    ./hack/clippy-package-features.sh hyperlight-common {{ target }}
+    ./hack/clippy-package-features.sh hyperlight-testing {{ target }}
+    just clippy-guests {{ target }}
+
+# Test a specific package with all feature combinations
+clippy-package package target=default-target: (witguest-wit)
+    ./hack/clippy-package-features.sh {{ package }} {{ target }}
+
 # Verify Minimum Supported Rust Version
 verify-msrv:
     ./dev/verify-msrv.sh hyperlight-host hyperlight-guest hyperlight-guest-lib hyperlight-common

hack/clippy-package-features.sh

@@ -0,0 +1,63 @@
+#!/usr/bin/env bash
+# filepath: /home/marosset/src/github.com/marosset/hyperlight/hack/clippy-package-features.sh
+
+set -euo pipefail
+
+# Check for required arguments
+if [[ $# -lt 2 ]]; then
+    echo "Usage: $0 <package> <target>" >&2
+    echo "Example: $0 hyperlight-host debug" >&2
+    exit 1
+fi
+
+PACKAGE="$1"
+TARGET="$2"
+
+# Convert target for cargo profile
+PROFILE=$([ "$TARGET" = "debug" ] && echo "dev" || echo "$TARGET")
+
+# Required features for compilation (only for hyperlight-host)
+if [[ "$PACKAGE" == "hyperlight-host" ]]; then
+    REQUIRED_FEATURES="kvm,mshv3"
+    # Get all features for the package (excluding default and required features)
+    features=$(cargo metadata --format-version 1 --no-deps | jq -r ".packages[] | select(.name == \"$PACKAGE\") | .features | keys[]" | grep -v -E "^(default|kvm|mshv3)$" || true)
+else
+    REQUIRED_FEATURES=""
+    # Get all features for the package (excluding default)
+    features=$(cargo metadata --format-version 1 --no-deps | jq -r ".packages[] | select(.name == \"$PACKAGE\") | .features | keys[]" | grep -v "^default$" || true)
+fi
+
+# Test with minimal features
+if [[ -n "$REQUIRED_FEATURES" ]]; then
+    echo "Testing $PACKAGE with required features only ($REQUIRED_FEATURES)..."
+    cargo clippy -p "$PACKAGE" --all-targets --no-default-features --features "$REQUIRED_FEATURES" --profile="$PROFILE" -- -D warnings
+else
+    echo "Testing $PACKAGE with no features..."
+    cargo clippy -p "$PACKAGE" --all-targets --no-default-features --profile="$PROFILE" -- -D warnings
+fi
+
+echo "Testing $PACKAGE with default features..."
+cargo clippy -p "$PACKAGE" --all-targets --profile="$PROFILE" -- -D warnings
+
+# Test each additional feature individually
+for feature in $features; do
+    if [[ -n "$REQUIRED_FEATURES" ]]; then
+        echo "Testing $PACKAGE with feature: $REQUIRED_FEATURES,$feature"
+        cargo clippy -p "$PACKAGE" --all-targets --no-default-features --features "$REQUIRED_FEATURES,$feature" --profile="$PROFILE" -- -D warnings || echo "Feature $feature failed for $PACKAGE"
+    else
+        echo "Testing $PACKAGE with feature: $feature"
+        cargo clippy -p "$PACKAGE" --all-targets --no-default-features --features "$feature" --profile="$PROFILE" -- -D warnings || echo "Feature $feature failed for $PACKAGE"
+    fi
+done
+
+# Test all features together
+if [[ -n "$features" ]]; then
+    all_features=$(echo $features | tr '\n' ',' | sed 's/,$//')
+    if [[ -n "$REQUIRED_FEATURES" ]]; then
+        echo "Testing $PACKAGE with all features: $REQUIRED_FEATURES,$all_features"
+        cargo clippy -p "$PACKAGE" --all-targets --no-default-features --features "$REQUIRED_FEATURES,$all_features" --profile="$PROFILE" -- -D warnings || echo "All features failed for $PACKAGE"
+    else
+        echo "Testing $PACKAGE with all features: $all_features"
+        cargo clippy -p "$PACKAGE" --all-targets --no-default-features --features "$all_features" --profile="$PROFILE" -- -D warnings || echo "All features failed for $PACKAGE"
+    fi
+fi