Review comments

crates/bws/src/main.rs

@@ -328,7 +328,7 @@ async fn process_commands() -> Result<()> {
     let state_file_path = state::get_state_file_path(
         profile.and_then(|p| p.state_file_dir).map(Into::into),
         access_token_obj.access_token_id.to_string(),
-    );
+    )?;
 
     let mut client = bitwarden::Client::new(settings);
 

crates/bws/src/state.rs

@@ -1,18 +1,20 @@
 use std::path::PathBuf;
 
+use color_eyre::eyre::Result;
+
 pub(crate) fn get_state_file_path(
     state_file_dir: Option<PathBuf>,
     access_token_id: String,
-) -> Option<PathBuf> {
+) -> Result<Option<PathBuf>> {
     if let Some(mut state_file_path) = state_file_dir {
         state_file_path.push(access_token_id);
 
         if let Some(parent_folder) = state_file_path.parent() {
-            std::fs::create_dir_all(parent_folder).expect("Parent directory should be writable");
+            std::fs::create_dir_all(parent_folder)?;
         }
 
-        return Some(state_file_path);
+        return Ok(Some(state_file_path));
     }
 
-    None
+    Ok(None)
 }