v0.2.12-pre.007

crates/ksp-app-solprices-desk/src/app_state.rs

@@ -1,5 +1,5 @@
 // file: crates/ksp-app-solprices-desk/src/app_state.rs
-// version: 5
+// version: 6
 
 //! Shared backend state owned by the SOL Prices Desk Tauri application.
 
@@ -129,7 +129,7 @@ impl crate::AppState {
             fallback_logging_active: runtime.fallback_active,
             provider_count,
             ready_provider_count,
-            shell_phase: "pre.006-multi-refresh".to_owned(),
+            shell_phase: "pre.007-ux-instrumentation".to_owned(),
             startup_diagnostic: runtime.startup_diagnostic.clone(),
             unavailable_provider_count,
         });

crates/ksp-app-solprices-desk/src/tauri.rs

@@ -1,5 +1,5 @@
 // file: crates/ksp-app-solprices-desk/src/tauri.rs
-// version: 5
+// version: 6
 
 //! Tauri runtime assembly for the KSP SOL prices desktop application.
 
@@ -98,12 +98,24 @@ fn configure_setup(builder: tauri::Builder<tauri::Wry>) -> tauri::Builder<tauri:
     });
 }
 
+fn project_command_error(command: &'static str, domain: &'static str, error: &ksp_core_lib::Error) -> crate::CommandErrorDto {
+    ksp_logging_lib::warn!(
+        target: crate::TRACING_TARGET,
+        domain = domain,
+        command = command,
+        error_domain = error.code().domain(),
+        error_code = error.code().code(),
+        "SOL Prices Desk Tauri command failed"
+    );
+    return crate::CommandErrorDto::from_error(error);
+}
+
 #[tauri::command]
 fn emit_frontend_log(payload: crate::FrontendLogPayloadDto) -> std::result::Result<(), crate::CommandErrorDto> {
     let result = crate::emit_frontend_log_event(payload);
     return match result {
         std::result::Result::Ok(()) => std::result::Result::Ok(()),
-        std::result::Result::Err(error) => std::result::Result::Err(crate::CommandErrorDto::from_error(&error)),
+        std::result::Result::Err(error) => std::result::Result::Err(project_command_error("emit_frontend_log", crate::TRACING_DOMAIN_FRONTEND, &error)),
     };
 }
 
@@ -112,7 +124,7 @@ fn get_runtime_status(state: tauri::State<'_, crate::AppState>) -> std::result::
     let result = state.runtime_status();
     return match result {
         std::result::Result::Ok(value) => std::result::Result::Ok(value),
-        std::result::Result::Err(error) => std::result::Result::Err(crate::CommandErrorDto::from_error(&error)),
+        std::result::Result::Err(error) => std::result::Result::Err(project_command_error("get_runtime_status", crate::TRACING_DOMAIN_SHELL, &error)),
     };
 }
 
@@ -123,10 +135,17 @@ fn list_market_prices(
     let result = state.list_market_prices();
     return match result {
         std::result::Result::Ok(value) => {
-            ksp_logging_lib::trace!(target: crate::TRACING_TARGET, domain = crate::TRACING_DOMAIN_OFFCHAIN_TRANSPORT, row_count = value.len(), "projected SOL Prices Desk market-price registry rows");
+            ksp_logging_lib::trace!(
+                target: crate::TRACING_TARGET,
+                domain = crate::TRACING_DOMAIN_OFFCHAIN_TRANSPORT,
+                row_count = value.len(),
+                "projected SOL Prices Desk market-price registry rows"
+            );
             std::result::Result::Ok(value)
         },
-        std::result::Result::Err(error) => std::result::Result::Err(crate::CommandErrorDto::from_error(&error)),
+        std::result::Result::Err(error) => {
+            std::result::Result::Err(project_command_error("list_market_prices", crate::TRACING_DOMAIN_OFFCHAIN_TRANSPORT, &error))
+        },
     };
 }
 
@@ -137,7 +156,9 @@ async fn refresh_all_market_prices(
     let result = state.refresh_all_market_prices().await;
     return match result {
         std::result::Result::Ok(value) => std::result::Result::Ok(value),
-        std::result::Result::Err(error) => std::result::Result::Err(crate::CommandErrorDto::from_error(&error)),
+        std::result::Result::Err(error) => {
+            std::result::Result::Err(project_command_error("refresh_all_market_prices", crate::TRACING_DOMAIN_OFFCHAIN_TRANSPORT, &error))
+        },
     };
 }
 
@@ -149,7 +170,9 @@ async fn refresh_market_price(
     let result = state.refresh_market_price(provider_id).await;
     return match result {
         std::result::Result::Ok(value) => std::result::Result::Ok(value),
-        std::result::Result::Err(error) => std::result::Result::Err(crate::CommandErrorDto::from_error(&error)),
+        std::result::Result::Err(error) => {
+            std::result::Result::Err(project_command_error("refresh_market_price", crate::TRACING_DOMAIN_OFFCHAIN_TRANSPORT, &error))
+        },
     };
 }
 
@@ -161,7 +184,9 @@ async fn refresh_market_prices(
     let result = state.refresh_market_prices(request).await;
     return match result {
         std::result::Result::Ok(value) => std::result::Result::Ok(value),
-        std::result::Result::Err(error) => std::result::Result::Err(crate::CommandErrorDto::from_error(&error)),
+        std::result::Result::Err(error) => {
+            std::result::Result::Err(project_command_error("refresh_market_prices", crate::TRACING_DOMAIN_OFFCHAIN_TRANSPORT, &error))
+        },
     };
 }
 
@@ -174,6 +199,6 @@ async fn splash_frontend_ready(
     let result = crate::splash_frontend_ready_service(app, webview_window, &state).await;
     return match result {
         std::result::Result::Ok(()) => std::result::Result::Ok(()),
-        std::result::Result::Err(error) => std::result::Result::Err(crate::CommandErrorDto::from_error(&error)),
+        std::result::Result::Err(error) => std::result::Result::Err(project_command_error("splash_frontend_ready", crate::TRACING_DOMAIN_WINDOWS, &error)),
     };
 }