Address code review feedback: add CachedValue::set method and fix pet-poetry caching logic

Co-authored-by: karthiknadig <3840081+karthiknadig@users.noreply.github.com>

Author: Copilot

crates/pet-core/src/cache.rs

@@ -119,6 +119,11 @@ impl<V: Clone> CachedValue<V> {
         self.value.read().unwrap().clone()
     }
 
+    /// Sets the cached value.
+    pub fn set(&self, value: V) {
+        *self.value.write().unwrap() = Some(value);
+    }
+
     /// Returns the cached value if present, otherwise computes it using the
     /// provided function, caches it, and returns it.
     pub fn get_or_compute<F>(&self, f: F) -> V
@@ -272,4 +277,20 @@ mod tests {
         let result = cached.get_or_compute(|| 100);
         assert_eq!(result, 100);
     }
+
+    #[test]
+    fn test_cached_value_set() {
+        let cached: CachedValue<i32> = CachedValue::new();
+
+        // Initially empty
+        assert_eq!(cached.get(), None);
+
+        // Set a value
+        cached.set(42);
+        assert_eq!(cached.get(), Some(42));
+
+        // Set overwrites existing value
+        cached.set(100);
+        assert_eq!(cached.get(), Some(100));
+    }
 }

crates/pet-poetry/src/lib.rs

@@ -171,9 +171,8 @@ impl Poetry {
         let envs = list_environments(&self.env_vars, &workspace_dirs, manager).unwrap_or_default();
         result.environments.extend(envs.clone());
 
-        // Having a value in the search result means that we have already searched for environments
-        // We need to explicitly compute and cache since get_or_compute doesn't fit here
-        let _ = self.search_result.get_or_compute(|| result.clone());
+        // Cache the result (even if empty, to avoid recomputing)
+        self.search_result.set(result.clone());
 
         if result.managers.is_empty() && result.environments.is_empty() {
             None