refactor: use SmartDefault for RegistryConfig, default auto_discover to true

Author: markovejnovic

Cargo.lock

@@ -35,6 +35,7 @@ uuid        = { version = "1", features = ["serde", "v4"] }
 chrono      = { version = "0.4", features = ["serde"] }
 thiserror   = "2"
 derive_more = { version = "1", default-features = false, features = ["full"] }
+smart-default = "0.7"
 tokio       = { version = "1", features = ["full"] }
 tokio-util  = { version = "0.7", features = ["rt"] }
 extism      = "1"

Cargo.toml

@@ -71,6 +71,7 @@ hm-plugin-protocol = { workspace = true }
 hm-util = { workspace = true }
 schemars = { workspace = true }
 semver = { workspace = true }
+smart-default = { workspace = true }
 once_cell = "1"
 hex = "0.4"
 

crates/hm/Cargo.toml

@@ -50,10 +50,7 @@ pub async fn run(cmd: PluginCommand) -> Result<()> {
 
 #[allow(clippy::unused_async)]
 async fn list() -> Result<()> {
-    let reg = PluginRegistry::load(RegistryConfig {
-        auto_discover: true,
-        ..Default::default()
-    })?;
+    let reg = PluginRegistry::load(RegistryConfig::default())?;
     if reg.manifests().count() == 0 {
         println!("No plugins installed.");
         println!();
@@ -78,10 +75,7 @@ async fn list() -> Result<()> {
 
 #[allow(clippy::unused_async)]
 async fn info(name: &str) -> Result<()> {
-    let reg = PluginRegistry::load(RegistryConfig {
-        auto_discover: true,
-        ..Default::default()
-    })?;
+    let reg = PluginRegistry::load(RegistryConfig::default())?;
     let m = reg
         .manifests()
         .find(|m| m.name == name)

crates/hm/src/cli/plugin.rs

@@ -10,10 +10,7 @@ use crate::plugin::{PluginRegistry, RegistryConfig};
 ///
 /// Returns an error if the plugin registry cannot be loaded.
 pub async fn run() -> Result<()> {
-    let reg = PluginRegistry::load(RegistryConfig {
-        auto_discover: true,
-        ..Default::default()
-    })?;
+    let reg = PluginRegistry::load(RegistryConfig::default())?;
     println!("hm {}", env!("CARGO_PKG_VERSION"));
     println!("plugin api version: {HM_PLUGIN_API_VERSION}");
     let count = reg.manifests().count();

crates/hm/src/cli/version.rs

@@ -27,10 +27,11 @@ use super::manifest::{ManifestError, validate_standalone};
 use super::paths;
 use crate::error::HmError;
 
-#[derive(Debug, Default)]
+#[derive(Debug, smart_default::SmartDefault)]
 pub struct RegistryConfig {
     /// If `false`, skip discovery and only registers explicitly added
     /// plugins. Used by integration tests.
+    #[default = true]
     pub auto_discover: bool,
     /// Extra plugin paths to load (in addition to discovery). Used by
     /// tests to load fixture plugins.