Move error reporting to prevent missed reports

This commit is contained in:
canvrno-oai
2026-03-13 15:04:53 -07:00
parent bb831401ca
commit fd4b37f969

View File

@@ -521,17 +521,6 @@ impl PluginsManager {
.map(|_| ())
.map_err(PluginInstallError::from)?;
let analytics_events_client = match self.analytics_events_client.read() {
Ok(client) => client.clone(),
Err(err) => err.into_inner().clone(),
};
if let Some(analytics_events_client) = analytics_events_client {
analytics_events_client.track_plugin_installed(plugin_telemetry_metadata_from_root(
&result.plugin_id,
result.installed_path.as_path(),
));
}
for app in load_plugin_apps(result.installed_path.as_path()) {
if app.0 == RESERVED_APP_DEFAULT_KEY {
warn!(
@@ -558,6 +547,17 @@ impl PluginsManager {
.map_err(PluginInstallError::from)?;
}
let analytics_events_client = match self.analytics_events_client.read() {
Ok(client) => client.clone(),
Err(err) => err.into_inner().clone(),
};
if let Some(analytics_events_client) = analytics_events_client {
analytics_events_client.track_plugin_installed(plugin_telemetry_metadata_from_root(
&result.plugin_id,
result.installed_path.as_path(),
));
}
Ok(PluginInstallOutcome {
plugin_id: result.plugin_id,
plugin_version: result.plugin_version,