refactor: improve CLI refresh error handling and validation

- Add directory type validation to catch incorrect paths early
- Improve exception handling with specific exception types
- Distinguish between expected and unexpected errors in logging
- Better error categorization aids debugging and monitoring

Author: Gvolexe

src/tagstudio/core/cli_driver.py

@@ -35,6 +35,10 @@ def refresh_library(self, library_path: str) -> int:
             logger.error("Library path does not exist", path=path)
             return 1
 
+        if not path.is_dir():
+            logger.error("Library path is not a directory", path=path)
+            return 1
+
         logger.info("Opening library", path=path)
         open_status = self.lib.open_library(path)
 
@@ -84,8 +88,15 @@ def refresh_library(self, library_path: str) -> int:
             )
             return 0
 
-        except Exception as e:
-            logger.exception("Error during library refresh", error=str(e))
+        except (OSError, ValueError, RuntimeError) as e:
+            logger.error(
+                "Expected error during library refresh",
+                error_type=type(e).__name__,
+                error=str(e),
+            )
+            return 1
+        except Exception:
+            logger.exception("Unexpected error during library refresh")
             return 1
 
         finally: