Updates error handling when finding path providers

Signed-off-by: David Gilligan-Cook <dcook@imageworks.com>

crates/spk-cli/group4/src/cmd_view.rs

@@ -197,7 +197,15 @@ impl View {
         // contain this file. Each of them should contain at least a
         // layer and the filepath entry, but might contain other spfs
         // objects.
-        let (in_a_runtime, found) = spk_storage::find_path_providers(filepath).await?;
+        let mut in_a_runtime = true;
+        let found = match spk_storage::find_path_providers(filepath).await {
+            Ok(f) => f,
+            Err(spk_storage::Error::SPFS(spfs::Error::NoActiveRuntime)) => {
+                in_a_runtime = false;
+                Vec::new()
+            }
+            Err(err) => return Err(err.into()),
+        };
 
         if found.is_empty() {
             println!("{filepath}: {}", "not found".yellow());
@@ -279,7 +287,7 @@ impl View {
                     println!(
                         " {}",
                         solved_request.request.format_request(
-                            &solved_request.repo_name(),
+                            solved_request.repo_name().as_ref(),
                             &solved_request.request.pkg.name,
                             &FormatChangeOptions {
                                 verbosity: DONT_SHOW_DETAILED_SETTINGS,

crates/spk-storage/src/storage/runtime.rs

@@ -8,7 +8,7 @@ use std::sync::Arc;
 
 use itertools::Itertools;
 use relative_path::RelativePathBuf;
-use spfs::find_path::ObjectPathEntry;
+use spfs::find_path::ObjectPath;
 use spfs::io::DigestFormat;
 use spfs::prelude::*;
 use spk_schema::foundation::ident_build::parse_build;
@@ -579,7 +579,7 @@ async fn find_layers_by_filenames<S: AsRef<str>>(
 
 /// Return a list of spfs object lists that lead to the given
 /// filepath in the runtime repo.
-pub async fn find_path_providers(filepath: &str) -> Result<(bool, Vec<Vec<ObjectPathEntry>>)> {
+pub async fn find_path_providers(filepath: &str) -> Result<Vec<ObjectPath>> {
     let config = spfs::get_config()?;
     let repo = config.get_local_repository_handle().await?;
 
@@ -589,10 +589,7 @@ pub async fn find_path_providers(filepath: &str) -> Result<(bool, Vec<Vec<Object
 }
 
 /// Print out a spfs object list down to the given filepath
-pub async fn pretty_print_filepath(
-    filepath: &str,
-    objectpath: &Vec<ObjectPathEntry>,
-) -> Result<()> {
+pub async fn pretty_print_filepath(filepath: &str, objectpath: &ObjectPath) -> Result<()> {
     let config = spfs::get_config()?;
     let repo = config.get_local_repository_handle().await?;