[*] refactor: video editor

Author: heng30

lib/video-editor/examples/media_basic_demo.rs

@@ -27,7 +27,7 @@ fn main() -> Result<()> {
     for file_path in &test_files {
         match library.add_file(PathBuf::from(file_path)) {
             Ok(id) => {
-                if let Some(item) = library.get_item(&id) {
+                if let Some(item) = library.get_item(id) {
                     log::info!("  ✓ Added: {} (ID: {})", item.name, id);
                     log::info!("    Type: {}", item.media_type.as_str());
                     log::info!("    Duration: {}", item.format_duration());
@@ -114,10 +114,6 @@ fn main() -> Result<()> {
             if !item.tags.is_empty() {
                 log::info!("    Tags: {}", item.tags.join(", "));
             }
-
-            if let Some(folder_id) = &item.folder_id {
-                log::info!("    Folder: {}", folder_id);
-            }
         }
     }
 
@@ -138,12 +134,12 @@ fn main() -> Result<()> {
     log::info!("=== Tag Management Demo ===");
     let items = library.all_items();
     if let Some(first_item) = items.first() {
-        let item_id = first_item.id.clone();
+        let item_id = first_item.id;
         let first_item_name = first_item.name.clone();
 
         // Add tags to the first item
         let tags = vec!["demo".to_string(), "example".to_string()];
-        if let Some(item) = library.get_item_mut(&item_id) {
+        if let Some(item) = library.get_item_mut(item_id) {
             *item = item.clone().with_tags(tags.clone());
             log::info!("Added tags to item '{}': {:?}", first_item_name, tags);
         }
@@ -153,31 +149,7 @@ fn main() -> Result<()> {
         log::info!("Items tagged with 'demo': {}", tagged_items.len());
     }
 
-    // Step 9: Demonstrate folder functionality
-    log::info!("=== Folder Organization Demo ===");
-    let folder_id = "test_folder".to_string();
-
-    // Add some items to a folder
-    let all_items = library.all_items();
-    let item_ids: Vec<String> = all_items
-        .iter()
-        .take(2)
-        .map(|item| item.id.clone())
-        .collect();
-
-    for item_id in item_ids {
-        if let Some(item_mut) = library.get_item_mut(&item_id) {
-            *item_mut = item_mut.clone().with_folder(folder_id.clone());
-        }
-    }
-
-    let folder_items = library.items_by_folder(&folder_id);
-    log::info!("Items in folder '{}': {}", folder_id, folder_items.len());
-    for item in folder_items {
-        log::info!("  - {}", item.name);
-    }
-
-    // Step 10: Demonstrate offline/online status
+    // Step 9: Demonstrate offline/online status
     log::info!("=== File Status Demo ===");
     library.check_all_status();
     let offline_items = library.offline_items();

lib/video-editor/examples/media_library_demo.rs

@@ -1,5 +1,5 @@
 use std::{path::PathBuf, time::Duration};
-use video_editor::{media::MediaType, playlist::Playlist};
+use video_editor::media::{MediaType, playlist::Playlist};
 
 fn main() -> Result<(), Box<dyn std::error::Error>> {
     env_logger::init();
@@ -89,9 +89,10 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
     match Playlist::from_json(&modified_json) {
         Ok(restored_playlist) => {
             log::info!("  Successfully restored playlist from JSON");
+            let count: usize = restored_playlist.item_count();
             log::info!(
                 "  Restored playlist has {} items",
-                restored_playlist.item_count()
+                count
             );
             log::info!("  Cache is configured: {}", restored_playlist.has_cache());
 

lib/video-editor/examples/playlist_demo.rs

@@ -4,7 +4,6 @@ pub mod filters;
 pub mod font;
 pub mod media;
 pub mod metadata;
-pub mod playlist;
 pub mod preview;
 pub mod project;
 pub mod tracks;

lib/video-editor/src/lib.rs

@@ -1,7 +1,10 @@
 pub mod cache;
 pub mod import;
 pub mod library;
+pub mod media_type;
+pub mod playlist;
 
 pub use cache::{MediaCache, MediaThumbnail};
 pub use import::{ImportOptions, ImportProgress, MediaImporter};
-pub use library::{MediaItem, MediaItemStatus, MediaLibrary, MediaType};
+pub use library::{MediaItem, MediaItemStatus, MediaLibrary};
+pub use media_type::MediaType;