From e366cade07b5408f95a0e6c268ee123e014e261c Mon Sep 17 00:00:00 2001
From: 魏曹先生 <1992414357@qq.com>
Date: Fri, 26 Sep 2025 11:08:47 +0800
Subject: refactor: convert InputPackage from tuple to named struct

- Replace tuple-based InputPackage with named struct for better type safety
- Add proper field names and documentation
- Fix typo in InputRelativePathBuf type name
- Update add_input and remove_input methods to work with new struct
- Maintain serialization compatibility with derive attributes
---
 crates/vcs/src/data/vault/sheets.rs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'crates/vcs/src/data/vault/sheets.rs')

diff --git a/crates/vcs/src/data/vault/sheets.rs b/crates/vcs/src/data/vault/sheets.rs
index ede4077..f7f6665 100644
--- a/crates/vcs/src/data/vault/sheets.rs
+++ b/crates/vcs/src/data/vault/sheets.rs
@@ -54,9 +54,9 @@ impl Vault {
 
             // Check if it's a YAML file
             if path.is_file() && path.extension().map_or(false, |ext| ext == "yaml") {
-                if let Some(_file_stem) = path.file_stem().and_then(|s| s.to_str()) {
+                if let Some(file_stem) = path.file_stem().and_then(|s| s.to_str()) {
                     // Create a new SheetName and add it to the result list
-                    sheet_names.push(SheetName::new());
+                    sheet_names.push(file_stem.to_string());
                 }
             }
         }
@@ -128,7 +128,7 @@ impl Vault {
 
         // Create the sheet file
         let sheet_data = SheetData {
-            holder: sheet_name.clone(),
+            holder: holder.clone(),
             inputs: Vec::new(),
             mapping: HashMap::new(),
         };
-- 
cgit 


From 9c44621f13f27e7f2a82fa5ab2fc8e27381f3e39 Mon Sep 17 00:00:00 2001
From: 魏曹先生 <1992414357@qq.com>
Date: Fri, 26 Sep 2025 11:18:08 +0800
Subject: Fixed by clippy

---
 crates/vcs/src/data/vault/sheets.rs | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

(limited to 'crates/vcs/src/data/vault/sheets.rs')

diff --git a/crates/vcs/src/data/vault/sheets.rs b/crates/vcs/src/data/vault/sheets.rs
index f7f6665..dfad862 100644
--- a/crates/vcs/src/data/vault/sheets.rs
+++ b/crates/vcs/src/data/vault/sheets.rs
@@ -53,12 +53,11 @@ impl Vault {
             let path = entry.path();
 
             // Check if it's a YAML file
-            if path.is_file() && path.extension().map_or(false, |ext| ext == "yaml") {
-                if let Some(file_stem) = path.file_stem().and_then(|s| s.to_str()) {
+            if path.is_file() && path.extension().is_some_and(|ext| ext == "yaml")
+                && let Some(file_stem) = path.file_stem().and_then(|s| s.to_str()) {
                     // Create a new SheetName and add it to the result list
                     sheet_names.push(file_stem.to_string());
                 }
-            }
         }
 
         Ok(sheet_names)
@@ -220,7 +219,7 @@ impl Vault {
         if !trash_dir.exists() {
             return Err(Error::new(
                 std::io::ErrorKind::NotFound,
-                format!("Trash directory does not exist!"),
+                "Trash directory does not exist!".to_string(),
             ));
         }
 
@@ -229,15 +228,14 @@ impl Vault {
             let entry = entry?;
             let path = entry.path();
 
-            if path.is_file() {
-                if let Some(file_name) = path.file_stem().and_then(|s| s.to_str()) {
+            if path.is_file()
+                && let Some(file_name) = path.file_stem().and_then(|s| s.to_str()) {
                     // Check if the filename starts with the sheet name
                     if file_name.starts_with(&sheet_name) {
                         found_path = Some(path);
                         break;
                     }
                 }
-            }
         }
 
         let trash_path = found_path.ok_or_else(|| {
-- 
cgit