start refinements

Author: Taxrosdev

Cargo.lock

@@ -12,7 +12,6 @@ edition = "2024"
 anyhow = "1.0.99"
 blake3 = { version = "1.8.2", features = ["digest"] }
 clap = { version = "4.5.47", features = ["derive"] }
-clap-cargo = "0.17.0"
 comfy-table = "7.2.0"
 directories = "6.0.0"
 ed25519-dalek = { version = "=3.0.0-pre.0", features = [

Cargo.toml

@@ -8,7 +8,11 @@
 - [x] Building
 - [x] Running
 - [ ] Bundling
+  - [ ] Extraction
+  - [ ] Building
 - [ ] Downloads
+  - [ ] Downloading chunks
+  - [ ] Downloading/Pulling Repositories
 - [ ] Compression
 
 ### Far future

ROADMAP.md

@@ -117,8 +117,6 @@ pub fn remove_package(package_id: &str, repo_path: &Path) -> Result<()> {
 /// - Repository doesn't exist
 /// - ID doesn't exist inside the Repository
 pub fn get_package(repo_path: &Path, id: &str) -> Result<PackageManifest> {
-    println!("{repo_path:?}");
-
     let repo_manifest = read_manifest(repo_path)?;
 
     // Check ID's and aliases

src/repo/mod.rs

@@ -21,6 +21,7 @@ pub fn start<S: AsRef<OsStr>>(
     entrypoint: &str,
     args: Vec<S>,
 ) -> Result<ExitStatus> {
+    // This should use the `install.meta`, not the Repositories package
     let package_manifest =
         repo::get_package(repo_path, package_id).with_context(|| "Failed to get package")?;
     let installed_path = &repo_path.join("installed").join(package_id);
@@ -29,28 +30,29 @@ pub fn start<S: AsRef<OsStr>>(
     let matches: Vec<&PathBuf> = package_manifest
         .commands
         .iter()
-        .filter(|package| package.ends_with(entrypoint))
+        .filter(|command| command.ends_with(entrypoint))
         .collect();
 
     // Make sure theres at least a single match
-    if matches.is_empty() {
-        bail!("Entrypoint does not exist.")
-    }
-
-    // Install if not installed
-    if !installed_path.exists() {
-        install(repo_path, package_id)?;
-    }
+    if let Some(entrypoint) = matches.first() {
+        // Install if not installed
+        if !installed_path.exists() {
+            install(repo_path, package_id).with_context(|| "Failed to install package.")?;
+        }
 
-    // Allow build_manifests to have a / at the start of entrypoints, eg: /bin/bash
-    let entrypoint = entrypoint.trim_start_matches('/');
+        // Allow build_manifests to have a / at the start of entrypoints, eg: /bin/bash
+        let entrypoint = entrypoint.to_string_lossy();
+        let entrypoint: &str = entrypoint.trim_start_matches('/');
 
-    // Actually run the command
-    let status = Command::new(installed_path.join(entrypoint))
-        .args(args)
-        .status()?;
+        // Actually run the command
+        let status = Command::new(installed_path.join(entrypoint))
+            .args(args)
+            .status()?;
 
-    Ok(status)
+        Ok(status)
+    } else {
+        bail!("Entrypoint does not exist.")
+    }
 }
 
 /// Installs or Updates a Package.