Code Quality

Usage

# Format check (fast)
./scripts/code-quality.sh fmt

# Lint with clippy
./scripts/code-quality.sh clippy

# Security audit
./scripts/code-quality.sh audit

# Run all quality gates
./scripts/code-quality.sh check

# Auto-fix common issues
./scripts/code-quality.sh clippy --fix

Quality Gates

./scripts/code-quality.sh fmt

./scripts/code-quality.sh clippy --workspace

cargo audit

./scripts/quality-gates.sh

cargo llvm-cov --html --output-dir coverage

cargo doc --no-deps

Rust Quality Dimensions

find . -name "*.rs" -exec wc -l {} +

rg "unwrap()" --glob "*.rs" --glob "!*/tests/*"

rg "async fn|spawn_blocking" --glob "*.rs"

./scripts/quality-gates.sh

cargo doc --no-deps

Rust-Specific Anti-Patterns

• Excessive Clone: Use borrowing or Arc
• Unnecessary Unwrap: Use

  ?

  operator
• Deep Nesting: Extract methods to flatten
• Large Functions: Split into smaller functions (< 50 LOC)
• Deadlocks: Release locks before

  .await

Best Practices Checklist

• Files <500 LOC
• Clear module hierarchy
• Custom Error enum with Result<T>
• No unwrap() in production code
• async fn for IO operations
• spawn_blocking for CPU work
• >=90% test coverage
• Public APIs documented
• SOLID principles applied
• No code duplication (DRY)

Dependency Monitoring (ADR-036)

# Count duplicate dependency roots (target: < 100)
cargo tree -d | grep -cE "^[a-z]"

# Find unused dependencies
cargo install --locked cargo-machete cargo-shear
cargo machete
cargo shear

# Find unused features
cargo install --locked cargo-unused-features
cargo unused-features analyze

References

• ADR-036: Dependency Deduplication
• ADR-032: Disk Space Optimization

_consolidated/

• rust-code-quality

  — Rust-specific quality dimensions, analysis commands, report format

• clean-code-developer

  — SOLID principles, refactoring techniques, anti-patterns