Enforce pre-commit code quality checks for the Breenix kernel.
npx skills add https://github.com/ryanbreen/breenix --skill code-quality-check
Please help me install this Agent Skill. Skill: code-quality-check Source: https://github.com/ryanbreen/breenix/tree/main/breenix-code-quality-check Command: npx skills add https://github.com/ryanbreen/breenix --skill code-quality-check
Maintaining high code quality in a kernel project is crucial to prevent subtle bugs and ensure long-term maintainability. This Skill provides a pre-commit checklist and tools to enforce Breenix's strict coding standards, catching issues like compiler warnings, Clippy lints, and log side-effects before they enter the codebase.
log::trace! statement with a side-effect, allowing you to fix these issues proactively and maintain a pristine codebase.cd kernel cargo build --target x86_64-unknown-none 2>&1 | grep warning
cargo clippy --target x86_64-unknown-none
-- -Dclippy::debug_assert_with_mut_call
-Dclippy::print_stdout
High-intent search queries and answers about installing and using this skill.
Pre-commit code quality checks catch compiler warnings, Clippy lints, and log side-effects before commits. Run `cargo build --target x86_64-unknown-none` and `cargo clippy --target x86_64-unknown-none` to enforce zero-warning builds and project-specific coding standards, maintaining code quality in kernel development workflows.
Run a full code quality check: `cargo build --target x86_64-unknown-none` for compiler warnings, `cargo clippy --target x86_64-unknown-none` for linting, `cargo test` for correctness, and manual log-scan checks for side-effects. This identifies unused variables, redundant closures, and problematic log statements before they enter the codebase.
Yes. Pre-commit checks scan for unintended function calls within log statements that impact performance and identify dead code. Use `#[allow(dead_code)]` explicitly for legitimate API items, with compliance verified against CLAUDE.md standards.
Yes. The Skill applies project-specific Clippy flags like `-Dclippy::debug_assert_with_mut_call` and `-Dclippy::print_stdout` with the `x86_64-unknown-none` target, enforcing coding standards tailored to Breenix kernel development across local workflows, pre-commit hooks, and CI pipelines.
Mark legitimate API items with `#[allow(dead_code)]` and ensure compliance with CLAUDE.md. The pre-commit checks distinguish between dead code that should be removed and intentional public API that remains unused by current code.
Pre-commit checks prevent subtle bugs and maintain long-term kernel maintainability by catching compiler warnings, Clippy violations, log side-effects, and dead code early. This ensures a pristine codebase and reduces debugging burden downstream.
OpenZeppelinSet up Stylus smart contract projects with OpenZeppelin dependencies for Arbitrum deployment.
korchasaReview code changes for quality, QA, and documentation compliance before committing.
d-o-hubFormat, lint, and audit Rust code with cargo fmt, Clippy, and cargo-audit.
edgeandnodeValidate and lint Rust and shell scripts with cargo check, clippy, and shellcheck.