fix: autoswitching now occurs on install, resolve package into id instead of alias

Author: Taxrosdev

src/repo/versions.rs

@@ -19,7 +19,15 @@ fn hash_package(package_manifest: &PackageManifest, hash_kind: HashKind) -> Resu
 ///
 /// - Filesystem errors (Out of space, Permissions)
 /// - Invalid Repository/Package manifest
-pub fn install_version(repo_path: &Path, package_id: &str, chunk_store_path: &Path) -> Result<()> {
+///
+/// # Returns
+///
+/// Returns the hash of the installed package
+pub fn install_version(
+    repo_path: &Path,
+    package_id: &str,
+    chunk_store_path: &Path,
+) -> Result<String> {
     let repo_manifest = read_manifest(repo_path)?;
 
     let package_manifest = get_package(&repo_manifest, package_id)
@@ -37,14 +45,22 @@ pub fn install_version(repo_path: &Path, package_id: &str, chunk_store_path: &Pa
         serde_yaml::to_string(&package_manifest)?,
     )?;
 
-    Ok(())
+    Ok(package_hash)
 }
 
-pub fn switch_version(repo_path: &Path, hash: String, package_id: &str) -> Result<()> {
+/// Switch to an older version/package hash.
+///
+/// # Errors
+///
+/// - Filesystem error during symlink (Within repo directory)
+pub fn switch_version(repo_path: &Path, hash: &str, package_id: &str) -> Result<()> {
     let target_parent_path = repo_path.join("installed");
     let target_path = target_parent_path.join(package_id);
+    fs::create_dir(target_parent_path)?;
     symlink(format!("../versions/{package_id}-{hash}"), target_path)?;
 
+    println!("{}", repo_path.display());
+
     Ok(())
 }
 

src/run/mod.rs

@@ -10,7 +10,10 @@ use std::{
 
 #[cfg(feature = "network")]
 use crate::chunks::install_tree;
-use crate::repo::{PackageManifest, get_package, read_manifest, versions::install_version};
+use crate::repo::{
+    PackageManifest, get_package, read_manifest,
+    versions::{install_version, switch_version},
+};
 
 /// Starts a package from an entrypoint
 ///
@@ -85,6 +88,8 @@ pub async fn install_package(
 
     let package_manifest = get_package(&repo_manifest, package_id)
         .with_context(|| "Failed to get package from Repository.")?;
+    // Don't just use an alias but actually resolve into a correct package id
+    let package_id = &package_manifest.id;
 
     // Get any chunks that are not installed
     #[cfg(feature = "network")]
@@ -97,7 +102,11 @@ pub async fn install_package(
     .await
     .with_context(|| "Failed to install package.")?;
 
-    install_version(repo_path, package_id, chunk_store_path)
+    let hash = install_version(repo_path, package_id, chunk_store_path)?;
+
+    switch_version(repo_path, &hash, package_id)?;
+
+    Ok(())
 }
 
 #[cfg(test)]