Skip to content

fix: eliminate compiler warnings and apply code formatting#547

Merged
echobt merged 1 commit intomasterfrom
fix/code-quality-1769998022
Feb 2, 2026
Merged

fix: eliminate compiler warnings and apply code formatting#547
echobt merged 1 commit intomasterfrom
fix/code-quality-1769998022

Conversation

@echobt
Copy link
Contributor

@echobt echobt commented Feb 2, 2026

Summary

This PR addresses code quality issues across the codebase:

Changes

  • Warning fixes: Eliminated all compiler warnings in cortex-tui, cortex-feedback, and cortex-app-server
  • Dead code cleanup: Prefixed unused fields and variables with underscore, added #[allow(dead_code)] for deserialization-only fields
  • Deprecation fixes: Replaced deprecated Buffer::get_mut with cell_mut
  • Pattern fixes: Fixed unreachable pattern in dispatch.rs
  • Code formatting: Applied rustfmt formatting (import ordering, whitespace cleanup, standardized patterns)

Verification

  • cargo check --workspace passes with no errors
  • cargo fmt --check passes
  • cargo clippy --workspace passes

Files Changed

224 files modified with formatting and warning fixes across multiple crates.

- Fix unused import warnings across cortex-tui, cortex-feedback, and cortex-app-server
- Remove dead code and unused variables with underscore prefix
- Replace deprecated Buffer::get_mut with cell_mut
- Fix unreachable pattern in dispatch.rs
- Add #[allow(dead_code)] for deserialization-only fields
- Apply rustfmt formatting: reorder imports, fix whitespace, standardize patterns
@echobt echobt force-pushed the fix/code-quality-1769998022 branch from 720379c to 3d30cfd Compare February 2, 2026 02:12
@echobt echobt merged commit 30b18f9 into master Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant