Fix race condition

Fix race condition in get_mcli_path() at src/util/misc.rs:15. When tests ran in parallel,
multiple threads could attempt to create the .mcli directory simultaneously,
causing one to fail with "File exists".

  The fix:
  - Replaced expect() with proper error handling using pattern matching
  - Gracefully ignores ErrorKind::AlreadyExists (safe when another thread created it)
  - Preserves panic behavior for genuine errors (permissions, etc.)
  - Only prints "Directory created" when actually successful

  This makes the function thread-safe and eliminates the flaky test failure in GitHub Actions.

Author: grunch

src/util/misc.rs

@@ -12,8 +12,13 @@ pub fn get_mcli_path() -> String {
     let home_dir = dirs::home_dir().expect("Couldn't get home directory");
     let mcli_path = format!("{}/.mcli", home_dir.display());
     if !Path::new(&mcli_path).exists() {
-        fs::create_dir(&mcli_path).expect("Couldn't create mostro-cli directory in HOME");
-        println!("Directory {} created.", mcli_path);
+        match fs::create_dir(&mcli_path) {
+            Ok(_) => println!("Directory {} created.", mcli_path),
+            Err(e) if e.kind() == std::io::ErrorKind::AlreadyExists => {
+                // Directory was created by another thread/process, which is fine
+            }
+            Err(e) => panic!("Couldn't create mostro-cli directory in HOME: {}", e),
+        }
     }
     mcli_path
 }