code-quality-check

name	code-quality-check
description	This skill should be used before committing code to ensure it meets Breenix quality standards. Use for running clippy checks, fixing compiler warnings, verifying no log side-effects, checking for dead code, and enforcing project coding standards from CLAUDE.md.

Code Quality Checks for Breenix

Pre-commit code quality verification for Breenix kernel development.

Purpose

Breenix enforces strict code quality standards. This skill provides the checks and fixes required before committing code.

Core Quality Standards (from CLAUDE.md)

1. Fix ALL compiler warnings before committing
2. Fix ALL clippy warnings
3. Use proper patterns (e.g., Once) to avoid unsafe warnings
4. Only #[allow(dead_code)] for legitimate API functions

Pre-Commit Checklist

Before every commit:

# 1. Build kernel and check for warnings
cd kernel
cargo build --target x86_64-unknown-none 2>&1 | grep warning

# 2. Run clippy
cargo clippy --target x86_64-unknown-none

# 3. Run tests (if modifying core subsystems)
cd ..
cargo test

# 4. Check for log side-effects (manual)
grep -R "log::trace!.*(" kernel/src/ | grep -vE '\".*\"' | grep -vE '\.(as_|to_|into_|len|is_|get)'

Clippy Configuration

Project-Specific Clippy Flags

cd kernel
RUSTFLAGS= \
cargo clippy --target x86_64-unknown-none \
  -- -Dclippy::debug_assert_with_mut_call \
     -Dclippy::print_stdout \
     -Wclippy::suspicious_operation_groupings