feat: support kebab-case and snake_case for fields in namespace config

Author: jhl-aa

README.md

@@ -75,7 +75,7 @@ set -a; source .env
 curl -v -X POST 127.0.0.1:8081/execute_skill \
 -H "Authorization: Bearer $PHARIA_AI_TOKEN" \
 -H 'Content-Type: application/json' \
--d '{"skill":"pharia_kernel_team/greet_skill", "input":"Homer"}'
+-d '{"skill":"pharia-kernel-team/greet_skill", "input":"Homer"}'
 ```
 
 ## User manual

operator-config.toml

@@ -1,11 +1,11 @@
-[namespaces.pharia_kernel_team]
+[namespaces.pharia-kernel-team]
 # The URL to the configuration listing the skills of this namespace
 # Pharia kernel will use the contents of the `NAMESPACES__PHARIA_KERNEL_TEAM__CONFIG_ACCESS_TOKEN` environment variable to access (authorize) the config
 config_url = "https://gitlab.aleph-alpha.de/api/v4/projects/966/repository/files/config.toml/raw?ref=main"
 
 # Registry to load skills from
 # Pharia kernel will use the contents of the `NAMESPACES__PHARIA_KERNEL_TEAM__REGISTRY__USER` and `NAMESPACES__PHARIA_KERNEL_TEAM__REGISTRY__PASSWORD` environment variables to access (authorize) the registry
-[namespaces.pharia_kernel_team.registry]
+[namespaces.pharia-kernel-team.registry]
 type = "oci"
 name = "registry.gitlab.aleph-alpha.de"
 repository = "engineering/pharia-skills/skills"

src/namespace_watcher/actor.rs

@@ -34,9 +34,9 @@ impl NamespaceDescriptionLoaders {
                 config
                     .loader()
                     .with_context(|| {
-                        format!("Unable to load configuration of namespace: '{namespace}'")
+                        format!("Unable to load configuration of namespace: '{namespace:?}'")
                     })
-                    .map(|loader| (namespace, loader))
+                    .map(|loader| (namespace.into_string(), loader))
             })
             .collect::<anyhow::Result<HashMap<_, _>>>()?;
         Ok(Self { namespaces })
@@ -264,6 +264,7 @@ pub mod tests {
     use tokio::sync::{mpsc, Mutex};
     use tokio::time::timeout;
 
+    use crate::namespace_watcher::config::Namespace;
     use crate::namespace_watcher::tests::NamespaceConfig;
     use crate::skill_store::tests::SkillStoreMessage;
     use crate::skills::SkillPath;
@@ -407,7 +408,7 @@ pub mod tests {
     async fn watch_skills_in_empty_directory() {
         let temp_dir = tempdir().unwrap();
         let namespaces = [(
-            "local".to_owned(),
+            Namespace::new("local"),
             NamespaceConfig::Watch {
                 directory: temp_dir.path().to_owned(),
             },
@@ -430,7 +431,7 @@ pub mod tests {
         fs::File::create(directory.join("skill_1.wasm")).unwrap();
         fs::File::create(directory.join("skill_2.wasm")).unwrap();
         let namespaces = [(
-            "local".to_owned(),
+            Namespace::new("local"),
             NamespaceConfig::Watch {
                 directory: directory.to_owned(),
             },