fix: let move_across_dir to deal with non-exist dst

Author: MiyakoMeow

cli/src/fs/moving.rs

@@ -652,6 +652,47 @@ mod tests {
         });
     }
 
+    #[test]
+    fn test_move_elements_across_dir_nonexistent_target() {
+        smol::block_on(async {
+            let temp_dir = tempdir().expect("Failed to create temp directory");
+            let src_dir = temp_dir.path().join("src");
+            let dst_dir = temp_dir.path().join("nonexistent_dst");
+
+            // Create source directory structure
+            fs::create_dir_all(&src_dir)
+                .await
+                .expect("Failed to create source directory");
+            create_test_structure(&src_dir)
+                .await
+                .expect("Failed to create test structure");
+
+            let replace_options = ReplaceOptions::default();
+            let result = move_elements_across_dir(&src_dir, &dst_dir, replace_options).await;
+            assert!(
+                result.is_ok(),
+                "Moving to non-existent target should succeed"
+            );
+
+            // Verify the entire directory was moved (renamed)
+            assert!(!src_dir.exists(), "Source directory should not exist");
+            assert!(dst_dir.exists(), "Target directory should exist");
+
+            // Verify all files were moved
+            let expected_files = [
+                ("file1.txt", "content1"),
+                ("file2.bms", "content2"),
+                ("subdir/file3.txt", "content3"),
+                ("subdir/nested/file4.txt", "content4"),
+            ];
+            verify_structure(&dst_dir, &expected_files)
+                .await
+                .expect("Failed to verify structure");
+
+            cleanup_test_dir(&temp_dir).await;
+        });
+    }
+
     #[test]
     fn test_move_elements_across_dir_with_ext_specific_rules() {
         smol::block_on(async {

cli/src/options/rawpack.rs

@@ -92,10 +92,6 @@ pub async fn unzip_numeric_to_bms_folder(
             root_dir.join(id_str)
         };
 
-        if !target_dir_path.exists() {
-            fs::create_dir_all(&target_dir_path).await?;
-        }
-
         // Move cache to bms dir
         info!(
             " > Moving files in {} to {}",
@@ -200,11 +196,6 @@ pub async fn unzip_with_name_to_bms_folder(
 
         let target_dir_path = root_dir.join(file_name_without_ext);
 
-        // Create New Target dir
-        if !target_dir_path.exists() {
-            fs::create_dir_all(&target_dir_path).await?;
-        }
-
         // Move cache to bms dir
         info!(
             " > Moving files in {} to {}",

cli/src/options/work.rs

@@ -138,10 +138,6 @@ pub async fn set_name_by_bms(
         target_work_dir.display()
     );
     if !dry_run {
-        fs::DirBuilder::new()
-            .recursive(true)
-            .create(&target_work_dir)
-            .await?;
         move_elements_across_dir(
             work_dir,
             target_work_dir,