fix(ci): run Rust linting on Windows and improve pre-commit hook

- Move lint-rust job to windows-latest (app is Windows-only)
- Add --all-targets --all-features to pre-commit clippy
- Fix conditional import of AppSettings for Windows-only code
- Fix clippy warning in test (case-sensitive extension comparison)

Author: kbzowski

.github/workflows/ci.yml

@@ -35,7 +35,7 @@ jobs:
 
   lint-rust:
     name: Lint Rust
-    runs-on: ubuntu-latest
+    runs-on: windows-latest
     steps:
       - uses: actions/checkout@v4
 
@@ -49,11 +49,6 @@ jobs:
         with:
           workspaces: src-tauri
 
-      - name: Install Tauri dependencies
-        run: |
-          sudo apt-get update
-          sudo apt-get install -y libwebkit2gtk-4.1-dev libappindicator3-dev librsvg2-dev
-
       - name: Check formatting
         run: cargo fmt --manifest-path src-tauri/Cargo.toml -- --check
 

.husky/pre-commit

@@ -1,6 +1,6 @@
 # Format and lint Rust code
 cargo fmt --manifest-path src-tauri/Cargo.toml
-cargo clippy --manifest-path src-tauri/Cargo.toml -- -D warnings
+cargo clippy --manifest-path src-tauri/Cargo.toml --all-targets --all-features -- -D warnings
 
 # Format and lint UI code (auto-fix)
 cd ui && pnpm biome check --write && cd ..

src-tauri/src/services/connection_manager.rs

@@ -20,8 +20,10 @@ use crate::config::connection::{JITTER_MAX_SECS, RETRY_BACKOFF_SECS};
 use crate::db::Database;
 use crate::error::AppError;
 use crate::models::{
-    normalize_url, AppSettings, Notification, NotificationDisplayMethod, NtfyMessage, Subscription,
+    normalize_url, Notification, NotificationDisplayMethod, NtfyMessage, Subscription,
 };
+#[cfg(windows)]
+use crate::models::AppSettings;
 use crate::services::TrayManager;
 
 /// Connection entry storing both the shutdown sender and a unique connection ID.

src-tauri/src/services/image_cache.rs

@@ -280,9 +280,13 @@ mod tests {
 
     #[test]
     fn test_cache_filename() {
+        use std::path::Path;
+
         let url = "https://example.com/path/to/image.jpg";
         let filename = get_cache_filename(url);
-        assert!(filename.ends_with(".jpg"));
+        assert!(Path::new(&filename)
+            .extension()
+            .is_some_and(|ext| ext.eq_ignore_ascii_case("jpg")));
 
         let url2 = "https://example.com/image.png?query=1";
         let filename2 = get_cache_filename(url2);