feat: remove fallback to "dev" namespace for operator-config.toml

Author: jhl-aa

doc/src/local-setup.md

@@ -33,14 +33,15 @@ In order to run Pharia Kernel, you need to provide a namespace configuration:
         mkdir skills
     ```
 
-    All skills in this folder are exposed in the namespace "dev".
+    All skills in this folder are exposed in the namespace "dev" with the environment variable `NAMESPACES__DEV__DIRECTORY`.
     Any changes in this folder will be picked up by the Pharia Kernel automatically. The `operator-config.toml` and `namespace.toml` should not be provided.
 
 2. Start the container:
 
     ```shell
         podman run \
             -v ./skills:/app/skills \
+            -e NAMESPACES__DEV__DIRECTORY = "/app/skills"
             -e PHARIA_AI_TOKEN=$PHARIA_AI_TOKEN \
             -e NAMESPACE_UPDATE_INTERVAL=1s \
             -e LOG_LEVEL="pharia_kernel=debug" \

src/config.rs

@@ -1,6 +1,5 @@
 use anyhow::anyhow;
-use config::ConfigError;
-use std::{env, io, net::SocketAddr, time::Duration};
+use std::{env, net::SocketAddr, time::Duration};
 
 use crate::namespace_watcher::OperatorConfig;
 
@@ -72,40 +71,8 @@ impl AppConfig {
             .unwrap_or("10s")
             .parse()?;
 
-        let operator_config = match OperatorConfig::new("operator-config.toml") {
-            Ok(operator_config) => operator_config,
-            Err(config_error) => match config_error.downcast::<ConfigError>() {
-                Ok(ConfigError::Foreign(foreign_error)) => {
-                    match foreign_error.downcast::<io::Error>() {
-                        Ok(io_error) if io_error.kind() == io::ErrorKind::NotFound => {
-                            // println! as the logger is not yet instantiated
-                            println!("Info: The 'operator-config.toml' is not found, fallback to the namespace 'dev' with the path 'skills' as the registry.");
-                            OperatorConfig::dev()
-                        }
-                        Ok(err) => {
-                            return Err(anyhow!(
-                                "The provided operator configuration must be valid: {err}"
-                            ))
-                        }
-                        Err(err) => {
-                            return Err(anyhow!(
-                                "The provided operator configuration must be valid: {err}"
-                            ))
-                        }
-                    }
-                }
-                Ok(err) => {
-                    return Err(anyhow!(
-                        "The provided operator configuration must be valid: {err}"
-                    ))
-                }
-                Err(err) => {
-                    return Err(anyhow!(
-                        "The provided operator configuration must be valid: {err}"
-                    ))
-                }
-            },
-        };
+        let operator_config = OperatorConfig::new("operator-config.toml")
+            .map_err(|err| anyhow!("The provided operator configuration must be valid: {err}"))?;
 
         let use_pooling_allocator = env::var("USE_POOLING_ALLOCATOR")
             .as_deref()

src/namespace_watcher/config.rs

@@ -48,7 +48,7 @@ impl OperatorConfig {
     /// # Errors
     /// Cannot parse operator config from the provided file or the environment variables.
     pub fn new(config_file: &str) -> anyhow::Result<Self> {
-        let file = File::with_name(config_file);
+        let file = File::with_name(config_file).required(false);
         let env = Self::environment();
         Self::from_sources(file, env)
     }