Build warnings in Rust codebase

[tyler-jewell]

Summary

The Claudia project has several compilation warnings that should be addressed to maintain code quality and prevent potential issues.

Current Behavior

When building the project with bun run tauri build --debug, the following warnings are generated:

Unused Imports (6 warnings)

1. File: src/commands/claude.rs

   • Line 10: use crate::process::ProcessHandle;
   • Line 11: CheckpointResult, CheckpointDiff, SessionTimeline, Checkpoint

2. File: src/commands/agents.rs

   • Line 1: SandboxRule
   • Line 2: SerializedProfile, SerializedOperation
   • Line 11: Arc

3. File: src/commands/mcp.rs

   • Line 9: warn from log crate

Dead Code Warnings (10 warnings)

1. File: src/checkpoint/mod.rs

   • Methods never used in CheckpointPaths impl:
     • file_snapshot_path (line 247)
     • file_reference_path (line 252)

2. File: src/checkpoint/state.rs

   • Methods never used in CheckpointState impl:
     • get_manager (line 90)
     • clear_all (line 106)
     • has_active_manager (line 124)
     • clear_all_and_count (line 129)

3. File: src/process/registry.rs

   • Fields never read in ProcessHandle struct:
     • info (line 22)
     • child (line 23)
   • Methods never used in ProcessRegistry impl:
     • unregister_process (line 75)
     • get_running_processes (line 82)
     • get_process (line 88)
     • kill_process (line 94)
     • is_process_running (line 119)
     • cleanup_finished_processes (line 175)

Expected Behavior

The codebase should compile without warnings, with all imports and code being either used or removed.

Steps to Reproduce

1. Clone the repository
2. Run bun install
3. Run bun run tauri build --debug
4. Observe the warnings in the build output

Environment

• Platform: macOS (darwin)
• OS Version: Darwin 24.5.0
• Architecture: aarch64 (Apple Silicon)
• Build tool: Tauri
• Package manager: Bun v1.2.17

Additional Context

• Total warnings: 16 (6 unused imports + 10 dead code warnings)
• Cargo suggests running cargo fix --lib -p claudia to apply fixes for the unused imports
• Some of the unused code might be intended for future use or might be remnants from refactoring

Suggested Solutions

1. For unused imports: Run cargo fix --lib -p claudia or manually remove the unused imports
2. For dead code:
   • Review if the code is needed for future features
   • If needed, add #[allow(dead_code)] attribute with a comment explaining why
   • If not needed, remove the dead code
   • Consider if some methods should be made public or used in tests

Priority

Medium - These warnings don't affect functionality but impact code maintainability and build output clarity.