feat: configure dynamic plugins through CLI

README.md

@@ -70,9 +70,10 @@ nemo-relay plugins edit --project
 ```
 
 The editor creates or updates the nearest project plugin file at
-`.nemo-relay/plugins.toml`. In the menu:
+`.nemo-relay/plugins.toml`. In the top-level menu, select **Observability**,
+then configure these sections:
 
-1. Enable the `Observability` component.
+1. Toggle the Observability component on.
 2. Open `ATOF`, toggle the section `[on]`
 
    Optionally set:
@@ -84,7 +85,7 @@ The editor creates or updates the nearest project plugin file at
    Optionally set:
    - `output_directory` to `.nemo-relay/atif`
    - `filename_template` to `trajectory-{session_id}.json`
-4. Press `p` to preview the generated TOML.
+4. Return to the top-level menu and press `p` to preview the generated TOML.
 5. Press `s` to save.
 
 > [!NOTE]

about.toml

@@ -11,6 +11,7 @@ accepted = [
     "BSL-1.0",
     "ISC",
     "MIT",
+    "MIT-0",
     "MPL-2.0",
     "Unicode-3.0",
     "Zlib",

crates/cli/Cargo.toml

@@ -38,8 +38,11 @@ console = "0.16"
 futures-util = "0.3"
 http = "1"
 http-body-util = "0.1"
-dialoguer = { version = "0.11", default-features = false }
+dialoguer = { version = "0.11", default-features = false, features = ["password"] }
+jsonschema = { version = "0.46.6", default-features = false }
+percent-encoding = "2"
 reqwest = { version = "0.12", default-features = false, features = ["charset", "http2", "json", "rustls-tls-native-roots", "stream"] }
+regex = "1"
 ring = "0.17"
 serde = { version = "1", features = ["derive"] }
 serde_json = "1"

crates/cli/README.md

@@ -110,6 +110,11 @@ plugin config with:
 nemo-relay plugins edit
 ```
 
+The top-level editor menu contains one entry per supported built-in, followed by
+the dynamic plugin references in the selected physical `plugins.toml`. Dynamic
+plugins with a manifest-declared JSON Schema provide structured field controls.
+Other dynamic plugins use a raw JSON object editor.
+
 The canonical plugin file is `plugins.toml`; user config lives at
 `~/.config/nemo-relay/plugins.toml` or
 `$XDG_CONFIG_HOME/nemo-relay/plugins.toml`. Project config lives at

crates/cli/src/config.rs

@@ -197,7 +197,7 @@ pub(crate) struct PluginJsonContext<'a> {
 /// Plugin configuration subcommands.
 #[derive(Debug, Clone, Subcommand)]
 pub(crate) enum PluginsSubcommand {
-    /// Interactively create or edit built-in plugin configuration in `plugins.toml`.
+    /// Interactively create or edit built-in and dynamic plugin configuration.
     Edit(PluginsEditCommand),
     /// Register a manifest-backed dynamic plugin in `plugins.toml`.
     Add(PluginsAddCommand),
@@ -811,28 +811,28 @@ fn load_shared_config(explicit: Option<&PathBuf>) -> Result<ResolvedConfig, CliE
     let mut merged = toml::Value::Table(toml::map::Map::new());
     let mut config_toml_plugin_sources = Vec::new();
     for path in config_paths(explicit) {
-        if path.exists() {
-            let raw = std::fs::read_to_string(&path)?;
-            let parsed = raw
-                .parse::<toml::Table>()
-                .map(toml::Value::Table)
-                .map_err(|error| {
-                    CliError::Config(format!("invalid TOML in {}: {error}", path.display()))
-                })?;
-            let legacy_observability = legacy_observability_sections(&parsed);
-            if !legacy_observability.is_empty() {
-                return Err(CliError::Config(format!(
-                    "legacy observability config in {} is no longer supported: {}; configure \
-                     observability in plugins.toml with `nemo-relay plugins edit`",
-                    path.display(),
-                    legacy_observability.join(", ")
-                )));
-            }
-            if has_config_toml_plugin_config(&parsed) {
-                config_toml_plugin_sources.push(path.clone());
-            }
-            merge_toml(&mut merged, parsed);
+        let Some(raw) = read_config_file(&path, explicit.is_some(), "configuration")? else {
+            continue;
+        };
+        let parsed = raw
+            .parse::<toml::Table>()
+            .map(toml::Value::Table)
+            .map_err(|error| {
+                CliError::Config(format!("invalid TOML in {}: {error}", path.display()))
+            })?;
+        let legacy_observability = legacy_observability_sections(&parsed);
+        if !legacy_observability.is_empty() {
+            return Err(CliError::Config(format!(
+                "legacy observability config in {} is no longer supported: {}; configure \
+                 observability in plugins.toml with `nemo-relay plugins edit`",
+                path.display(),
+                legacy_observability.join(", ")
+            )));
         }
+        if has_config_toml_plugin_config(&parsed) {
+            config_toml_plugin_sources.push(path.clone());
+        }
+        merge_toml(&mut merged, parsed);
     }
     if config_toml_plugin_sources.len() > 1 {
         return Err(CliError::Config(format!(
@@ -856,6 +856,30 @@ fn load_shared_config(explicit: Option<&PathBuf>) -> Result<ResolvedConfig, CliE
     Ok(resolved)
 }
 
+fn read_config_file(
+    path: &Path,
+    required: bool,
+    description: &str,
+) -> Result<Option<String>, CliError> {
+    match path.try_exists() {
+        Ok(false) if !required => Ok(None),
+        Ok(false) => Err(CliError::Config(format!(
+            "explicit {description} file {} does not exist",
+            path.display()
+        ))),
+        Err(error) => Err(CliError::Config(format!(
+            "failed to inspect {description} file {}: {error}",
+            path.display()
+        ))),
+        Ok(true) => std::fs::read_to_string(path).map(Some).map_err(|error| {
+            CliError::Config(format!(
+                "failed to read {description} file {}: {error}",
+                path.display()
+            ))
+        }),
+    }
+}
+
 /// Returns true if any of the implicit config file locations exists on disk. Used by the
 /// easy-path dispatcher to decide whether to launch setup (no config found) or proceed
 /// with config-driven settings. Mirrors `config_paths(None)` but only checks existence.
@@ -1045,10 +1069,9 @@ where
     let mut runtime_documents = Vec::new();
 
     for path in &paths {
-        if !path.exists() {
+        let Some(raw) = read_config_file(path, false, "plugin configuration")? else {
             continue;
-        }
-        let raw = std::fs::read_to_string(path)?;
+        };
         let mut parsed = raw
             .parse::<toml::Table>()
             .map(toml::Value::Table)

crates/cli/src/model_pricing.rs

@@ -17,9 +17,7 @@ use crate::config::{
     PricingValidateCommand, ServerArgs, resolve_server_config,
 };
 use crate::error::CliError;
-use crate::plugins::config_io::{
-    TargetScope, read_plugin_config, target_path, validate_config, write_plugin_config,
-};
+use crate::plugins::config_io::{PluginConfigDocument, TargetScope, target_path, validate_config};
 
 const PRICING_PLUGIN_KIND: &str = "pricing";
 
@@ -37,13 +35,13 @@ pub(crate) fn validate(command: PricingValidateCommand) -> Result<(), CliError>
 pub(crate) fn init(command: PricingInitCommand) -> Result<(), CliError> {
     let scope = target_pricing_scope(&command.scope)?;
     let path = target_path(scope)?;
-    let mut plugin_config = read_plugin_config(&path)?;
-    let index = ensure_pricing_component(&mut plugin_config)?;
-    let pricing_config = pricing_config_from_component(&plugin_config.components[index])?;
-    store_pricing_config(&mut plugin_config.components[index], &pricing_config)?;
-    plugin_config.components[index].enabled = true;
-    validate_config(&plugin_config)?;
-    write_plugin_config(&path, &plugin_config)?;
+    update_plugin_config_document(&path, |plugin_config| {
+        let index = ensure_pricing_component(plugin_config)?;
+        let pricing_config = pricing_config_from_component(&plugin_config.components[index])?;
+        store_pricing_config(&mut plugin_config.components[index], &pricing_config)?;
+        plugin_config.components[index].enabled = true;
+        Ok(())
+    })?;
     println!("Initialized model pricing config: {}", path.display());
     Ok(())
 }
@@ -58,23 +56,22 @@ pub(crate) fn add_source(command: PricingAddSourceCommand) -> Result<(), CliErro
     read_pricing_catalog(&source_path)?;
     let scope = target_pricing_scope(&command.scope)?;
     let path = target_path(scope)?;
-    let mut plugin_config = read_plugin_config(&path)?;
-    let index = ensure_pricing_component(&mut plugin_config)?;
-    let mut pricing_config = pricing_config_from_component(&plugin_config.components[index])?;
     let source = PricingSourceConfig::File { path: source_path };
 
-    if !pricing_config.sources.contains(&source) {
-        if command.append {
-            pricing_config.sources.push(source);
-        } else {
-            pricing_config.sources.insert(0, source);
+    update_plugin_config_document(&path, |plugin_config| {
+        let index = ensure_pricing_component(plugin_config)?;
+        let mut pricing_config = pricing_config_from_component(&plugin_config.components[index])?;
+        if !pricing_config.sources.contains(&source) {
+            if command.append {
+                pricing_config.sources.push(source);
+            } else {
+                pricing_config.sources.insert(0, source);
+            }
         }
-    }
-
-    store_pricing_config(&mut plugin_config.components[index], &pricing_config)?;
-    plugin_config.components[index].enabled = true;
-    validate_config(&plugin_config)?;
-    write_plugin_config(&path, &plugin_config)?;
+        store_pricing_config(&mut plugin_config.components[index], &pricing_config)?;
+        plugin_config.components[index].enabled = true;
+        Ok(())
+    })?;
     println!(
         "Added model pricing source: {} -> {}",
         command.path.display(),
@@ -83,6 +80,16 @@ pub(crate) fn add_source(command: PricingAddSourceCommand) -> Result<(), CliErro
     Ok(())
 }
 
+fn update_plugin_config_document(
+    path: &Path,
+    update: impl FnOnce(&mut PluginConfig) -> Result<(), CliError>,
+) -> Result<(), CliError> {
+    let mut document = PluginConfigDocument::read(path)?;
+    update(document.config_mut())?;
+    validate_config(document.config())?;
+    document.write()
+}
+
 pub(crate) fn resolve(command: PricingResolveCommand) -> Result<(), CliError> {
     let sources = pricing_catalog_sources_from_current_config()?;
     if sources.is_empty() {