diff --git a/.gitignore b/.gitignore index 6936990..d268494 100644 --- a/.gitignore +++ b/.gitignore @@ -1,3 +1,4 @@ /target **/*.rs.bk Cargo.lock +CODE_CLEANUP_TODO.md diff --git a/CODE_CLEANUP_TODO.md b/CODE_CLEANUP_TODO.md deleted file mode 100644 index d08bb37..0000000 --- a/CODE_CLEANUP_TODO.md +++ /dev/null @@ -1,38 +0,0 @@ -# Code Cleanup TODO - -## 🔥 Critical Issues - -### 1. Code Duplication -- [x] Extract cursor application logic from `select_cursor()` and `next_cursor()` -- [x] Extract cleanup logic from `quit_app()` and system tray quit handler - -### 2. Steam Integration -- [ ] Replace `unwrap()` with proper error handling in `Steam::new()`- Leave this on the backburner for now -- [ ] App should work fine even if Steam is offline (Steam remembers inventory, we pull from Steam even in offline mode) - Leave this on the backburner for now - -### 3. Unsafe Thread-Safety -- [ ] Document or fix `SyncHCURSOR` unsafe impls in `types.rs` - -## 🚨 High Priority - -### 4. Debug Logging Spam -- [ ] Remove 73 debug print statements from production code -- [ ] Add proper logging framework - -### 5. Mutex Unwraps -- [ ] Replace 9 `lock().unwrap()` calls with proper error handling - -### 6. Large Functions -- [ ] Break down `load_cursor_files()` (90+ lines) -- [ ] Break down `create_cursor_from_image()` (170+ lines) - -## 📋 Medium Priority - -### 7. Mock Data Fallbacks -- [ ] Remove hardcoded fallback cursors pointing to non-existent files - -### 8. Inline Event Handlers -- [ ] Extract system tray/window event handlers from Tauri builder - -### 9. Error Handling -- [ ] Standardize error types and handling patterns