refactor: add more logic for local dependencies (#506)

.vscode/launch.json

@@ -388,7 +388,7 @@
       "type": "lldb",
       "request": "launch",
       "program": "${workspaceFolder}/out/linux/x64/ten_manager/bin/tman",
-      "cwd": "${workspaceFolder}/out/linux/x64/tests/ten_manager/install_all/install_all_in_local_file_system/test_app",
+      "cwd": "${workspaceFolder}/out/linux/x64/tests/ten_manager/install/mock_app/mock_app",
       "args": [
         "--config-file=${workspaceFolder}/out/linux/x64/tests/local_registry/config.json",
         "install",

core/src/ten_manager/Cargo.toml

@@ -19,6 +19,7 @@ clap = { version = "4.5" }
 clingo = { version = "0.8" }
 console = { version = "0.15" }
 dirs = { version = "5.0" }
+fs_extra = "1.3.0"
 futures-util = { version = "0.3" }
 globset = { version = "0.4" }
 handlebars = { version = "6.2" }

core/src/ten_manager/src/cmd/cmd_check/cmd_check_manifest_json.rs

@@ -9,7 +9,7 @@ use clap::{Arg, ArgMatches, Command};
 use console::Emoji;
 use ten_rust::json_schema::ten_validate_manifest_json_string;
 
-use crate::{config::TmanConfig, utils::read_file_to_string};
+use crate::{config::TmanConfig, fs::read_file_to_string};
 
 #[derive(Debug)]
 pub struct CheckManifestJsonCommand {

core/src/ten_manager/src/cmd/cmd_check/cmd_check_property_json.rs

@@ -9,7 +9,7 @@ use clap::{Arg, ArgMatches, Command};
 use console::Emoji;
 use ten_rust::json_schema::ten_validate_property_json_string;
 
-use crate::utils::read_file_to_string;
+use crate::fs::read_file_to_string;
 
 #[derive(Debug)]
 pub struct CheckPropertyJsonCommand {

core/src/ten_manager/src/cmd/cmd_designer.rs

@@ -19,8 +19,8 @@ use console::Emoji;
 use crate::{
     config::TmanConfig,
     designer::{configure_routes, DesignerState},
+    fs::{check_is_app_folder, get_cwd},
     log::tman_verbose_println,
-    utils::{check_is_app_folder, get_cwd},
 };
 
 #[derive(Clone, Debug)]

core/src/ten_manager/src/cmd/cmd_install.rs

@@ -39,6 +39,7 @@ use crate::{
     constants::{APP_DIR_IN_DOT_TEN_DIR, DOT_TEN_DIR, MANIFEST_JSON_FILENAME},
     dep_and_candidate::get_all_candidates_from_deps,
     error::TmanError,
+    fs::{check_is_app_folder, check_is_package_folder},
     install::PkgIdentityMapping,
     log::tman_verbose_println,
     manifest_lock::{
@@ -56,11 +57,11 @@ use crate::{
             install_solver_results_in_standalone_mode,
         },
     },
-    utils::{check_is_app_folder, check_is_package_folder},
 };
 
 #[derive(Debug, Clone, Copy)]
 pub enum LocalInstallMode {
+    Invalid,
     Copy,
     Link,
 }
@@ -145,7 +146,7 @@ pub fn create_sub_cmd(args_cfg: &crate::cmd_line::ArgsCfg) -> Command {
                 .long("local-install-mode")
                 .help("Local install mode: copy or link")
                 .value_parser(["copy", "link"])
-                .default_value("copy")
+                .default_value("link")
                 .required(false),
         )
 }
@@ -166,7 +167,8 @@ pub fn parse_sub_cmd(sub_cmd_args: &ArgMatches) -> Result<InstallCommand> {
             .get_one::<bool>("TEMPLATE_MODE")
             .unwrap_or(&false),
         template_data: HashMap::new(),
-        local_install_mode: LocalInstallMode::Copy,
+
+        local_install_mode: LocalInstallMode::Invalid,
     };
 
     let _ = cmd.support.set_defaults();
@@ -291,7 +293,7 @@ fn update_package_manifest(
                         break;
                     }
                 }
-                ManifestDependency::LocalDependency { path } => {
+                ManifestDependency::LocalDependency { path, .. } => {
                     let manifest_dependency_pkg_info =
                         match get_pkg_info_from_path(Path::new(&path), false) {
                             Ok(info) => info,
@@ -553,7 +555,7 @@ pub async fn execute_cmd(
 
     let started = Instant::now();
 
-    let cwd = crate::utils::get_cwd()?;
+    let cwd = crate::fs::get_cwd()?;
 
     // The package affected by tman install command, which is the root declared
     // in the resolver.
@@ -692,6 +694,7 @@ pub async fn execute_cmd(
                     },
                     version_req: desired_pkg_src_version_.clone(),
                     path: None,
+                    base_dir: None,
                 },
             };
             extra_dependency_relationships.push(extra_dependency_relationship);

core/src/ten_manager/src/cmd/cmd_modify/cmd_modify_graph.rs

@@ -13,7 +13,7 @@ use serde_json::Value;
 
 use crate::{
     cmd::cmd_modify::jq_util::jq_run, config::TmanConfig,
-    constants::PROPERTY_JSON_FILENAME, utils::read_file_to_string,
+    constants::PROPERTY_JSON_FILENAME, fs::read_file_to_string,
 };
 
 #[derive(Debug)]

core/src/ten_manager/src/cmd/cmd_package.rs

@@ -43,7 +43,7 @@ pub async fn execute_cmd(
 
     let started = Instant::now();
 
-    let cwd = crate::utils::get_cwd()?;
+    let cwd = crate::fs::get_cwd()?;
 
     let pkg_info = get_pkg_info_from_path(&cwd, true)?;
     let output_zip_file_name = get_package_zip_file_name(&pkg_info)?;

core/src/ten_manager/src/cmd/cmd_publish.rs

@@ -43,7 +43,7 @@ pub async fn execute_cmd(
 
     let started = Instant::now();
 
-    let cwd = crate::utils::get_cwd()?;
+    let cwd = crate::fs::get_cwd()?;
 
     let pkg_info = get_pkg_info_from_path(&cwd, true)?;
     let output_zip_file_name = get_package_zip_file_name(&pkg_info)?;

core/src/ten_manager/src/cmd/cmd_run.rs

@@ -67,7 +67,7 @@ pub async fn execute_cmd(
     tman_verbose_println!(tman_config, "Executing run command: {:?}", cmd);
 
     // Read `manifest.json` in the current working directory.
-    let cwd = crate::utils::get_cwd()?;
+    let cwd = crate::fs::get_cwd()?;
     let manifest_path = cwd.join(MANIFEST_JSON_FILENAME);
     if !manifest_path.exists() {
         return Err(anyhow!(
@@ -78,10 +78,10 @@ pub async fn execute_cmd(
     }
 
     // Parse `manifest.json`.
-    let manifest_json_str = crate::utils::read_file_to_string(&manifest_path)
+    let manifest_json_str = crate::fs::read_file_to_string(&manifest_path)
         .map_err(|e| {
-        anyhow!("Failed to read {}: {}", MANIFEST_JSON_FILENAME, e)
-    })?;
+            anyhow!("Failed to read {}: {}", MANIFEST_JSON_FILENAME, e)
+        })?;
     let manifest_value: serde_json::Value =
         serde_json::from_str(&manifest_json_str).map_err(|e| {
             anyhow!("Failed to parse {}: {}", MANIFEST_JSON_FILENAME, e)

core/src/ten_manager/src/cmd/cmd_uninstall.rs

@@ -23,9 +23,9 @@ use crate::{
         DOT_TEN_DIR, INSTALLED_PATHS_JSON_FILENAME, INSTALL_PATHS_APP_PREFIX,
         PACKAGE_INFO_DIR_IN_DOT_TEN_DIR,
     },
+    fs::check_is_app_folder,
     install::installed_paths::InstalledPaths,
     log::tman_verbose_println,
-    utils::check_is_app_folder,
 };
 
 #[derive(Debug)]
@@ -149,7 +149,7 @@ pub async fn execute_cmd(
 
     let started = Instant::now();
 
-    let cwd = crate::utils::get_cwd()?;
+    let cwd = crate::fs::get_cwd()?;
     check_is_app_folder(&cwd)?;
 
     remove_installed_paths(&cwd, &command_data).await?;