Skip to content

🔧 Blanket #![allow(dead_code)] masks genuinely unused code #79

Description

@ajianaz

Description

src/main.rs:3 has #![allow(dead_code)] for the entire crate. Several public functions have no callers outside of tests:

  • config/providers.rs:first_detected_preset()
  • git/files.rs:list_tracked_files(), list_all_changed_files(), filter_by_globs(), is_gitignored()
  • git/diff.rs:is_inside_git_repo()
  • engine/review.rs:review_diff() (non-stream variant), scan_project()

Suggested Fix

Remove the blanket allow. Add targeted #[allow(dead_code)] only where needed. Remove genuinely unused functions.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions