fix: don't panic on missing shorebird.yaml

library/src/c_api.rs

@@ -83,14 +83,10 @@ fn log_on_error<F, R>(f: F, context: &str, error_result: R) -> R
 where
     F: FnOnce() -> Result<R, anyhow::Error>,
 {
-    let result = f();
-    match result {
-        Ok(r) => r,
-        Err(e) => {
-            error!("Error {}: {:?}", context, e);
-            error_result
-        }
-    }
+    f().unwrap_or_else(|e| {
+        error!("Error {}: {:?}", context, e);
+        error_result
+    })
 }
 
 /// Configures updater.  First parameter is a struct containing configuration
@@ -119,7 +115,7 @@ pub extern "C" fn shorebird_init(
 pub extern "C" fn shorebird_next_boot_patch_number() -> *mut c_char {
     log_on_error(
         || {
-            let patch = updater::next_boot_patch().context("failed to fetch patch info")?;
+            let patch = updater::next_boot_patch()?.context("failed to fetch patch info")?;
             allocate_c_string(&patch.number.to_string())
         },
         "fetching next_boot_patch_number",
@@ -133,7 +129,7 @@ pub extern "C" fn shorebird_next_boot_patch_number() -> *mut c_char {
 pub extern "C" fn shorebird_next_boot_patch_path() -> *mut c_char {
     log_on_error(
         || {
-            let patch = updater::next_boot_patch().context("failed to fetch patch info")?;
+            let patch = updater::next_boot_patch()?.context("failed to fetch patch info")?;
             allocate_c_string(&patch.path)
         },
         "fetching next_boot_patch_path",
@@ -155,13 +151,20 @@ pub extern "C" fn shorebird_free_string(c_string: *mut c_char) {
 /// Check for an update.  Returns true if an update is available.
 #[no_mangle]
 pub extern "C" fn shorebird_check_for_update() -> bool {
-    return updater::check_for_update();
+    log_on_error(updater::check_for_update, "checking for update", false)
 }
 
 /// Synchronously download an update if one is available.
 #[no_mangle]
 pub extern "C" fn shorebird_update() {
-    updater::update();
+    log_on_error(
+        || {
+            updater::update()?;
+            Ok(())
+        },
+        "downloading update",
+        (),
+    );
 }
 
 /// Start a thread to download an update if one is available.
@@ -177,11 +180,7 @@ pub extern "C" fn shorebird_start_update_thread() {
 /// shorebird_report_launch_success or shorebird_report_launch_failure.
 #[no_mangle]
 pub extern "C" fn shorebird_report_launch_start() {
-    log_on_error(
-        || updater::report_launch_start(),
-        "reporting launch start",
-        (),
-    );
+    log_on_error(updater::report_launch_start, "reporting launch start", ());
 }
 
 /// Report that the app failed to launch.  This will cause the updater to
@@ -190,10 +189,7 @@ pub extern "C" fn shorebird_report_launch_start() {
 #[no_mangle]
 pub extern "C" fn shorebird_report_launch_failure() {
     log_on_error(
-        || {
-            updater::report_launch_failure()?;
-            Ok(())
-        },
+        updater::report_launch_failure,
         "reporting launch failure",
         (),
     );
@@ -209,10 +205,7 @@ pub extern "C" fn shorebird_report_launch_failure() {
 #[no_mangle]
 pub extern "C" fn shorebird_report_launch_success() {
     log_on_error(
-        || {
-            updater::report_launch_success()?;
-            Ok(())
-        },
+        updater::report_launch_success,
         "reporting launch success",
         (),
     );

library/src/config.rs

@@ -1,4 +1,5 @@
 // This file handles the global config for the updater library.
+use anyhow::bail;
 
 #[cfg(test)]
 use crate::network::{DownloadFileFn, PatchCheckRequestFn};
@@ -7,6 +8,7 @@ use std::cell::RefCell;
 
 use crate::updater::AppConfig;
 use crate::yaml::YamlConfig;
+use crate::UpdateError;
 
 #[cfg(not(test))]
 use once_cell::sync::OnceCell;
@@ -60,14 +62,14 @@ pub fn testing_reset_config() {
     });
 }
 
-pub fn check_initialized_and_call<F, R>(f: F, config: &ResolvedConfig) -> R
+pub fn check_initialized_and_call<F, R>(f: F, config: &ResolvedConfig) -> anyhow::Result<R>
 where
     F: FnOnce(&ResolvedConfig) -> R,
 {
     if !config.is_initialized {
-        panic!("Must call shorebird_init() before using the updater.");
+        bail!(UpdateError::ConfigNotInitialized);
     }
-    return f(&config);
+    return Ok(f(&config));
 }
 
 #[cfg(test)]
@@ -93,7 +95,7 @@ where
 }
 
 #[cfg(not(test))]
-pub fn with_config<F, R>(f: F) -> R
+pub fn with_config<F, R>(f: F) -> anyhow::Result<R>
 where
     F: FnOnce(&ResolvedConfig) -> R,
 {
@@ -106,7 +108,7 @@ where
 }
 
 #[cfg(test)]
-pub fn with_config<F, R>(f: F) -> R
+pub fn with_config<F, R>(f: F) -> anyhow::Result<R>
 where
     F: FnOnce(&ResolvedConfig) -> R,
 {

library/src/lib.rs

@@ -5,7 +5,7 @@
 // C doesn't care about the namespaces, but Rust does.
 pub mod c_api;
 
-// Declare other .rs file/module exists, but make them public.
+// Declare other .rs file/module exists, but make them private.
 mod cache;
 mod config;
 mod logging;