From 6989d4110ad30925081c4f805cfacbf2965de41a Mon Sep 17 00:00:00 2001
From: Michael Ramos <mdramos8@gmail.com>
Date: Sat, 29 Aug 2026 08:21:58 -0700
Subject: [PATCH] fix(last): hermes_home as two cfg functions (windows clippy
 needless_return)

---
 crates/plannotator-tui/src/last/locate.rs | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/crates/plannotator-tui/src/last/locate.rs b/crates/plannotator-tui/src/last/locate.rs
index 66d90f5..fc7636d 100644
--- a/crates/plannotator-tui/src/last/locate.rs
+++ b/crates/plannotator-tui/src/last/locate.rs
@@ -151,15 +151,16 @@ pub(crate) fn screen_fallback(env: &crate::herdr::context::HerdrEnv) -> Option<D
 
 /// Hermes' platform default (`hermes_constants.py`): `%LOCALAPPDATA%\hermes` on Windows,
 /// `~/.hermes` elsewhere.
+#[cfg(windows)]
+fn hermes_home() -> PathBuf {
+    std::env::var_os("LOCALAPPDATA")
+        .filter(|v| !v.is_empty())
+        .map_or_else(|| home().join("AppData").join("Local"), PathBuf::from)
+        .join("hermes")
+}
+
+#[cfg(not(windows))]
 fn hermes_home() -> PathBuf {
-    #[cfg(windows)]
-    {
-        let base = std::env::var_os("LOCALAPPDATA")
-            .filter(|v| !v.is_empty())
-            .map_or_else(|| home().join("AppData").join("Local"), PathBuf::from);
-        return base.join("hermes");
-    }
-    #[cfg(not(windows))]
     home().join(".hermes")
 }