Merge pull request #1031 from imageworks/hyphens-in-env-var-names

Change any hyphens in SPK_OPT_* env var names

crates/spk-schema/crates/foundation/src/name/mod.rs

@@ -12,6 +12,8 @@ pub use error::{Error, Result};
 use miette::Diagnostic;
 use thiserror::Error;
 
+use crate::spec_ops::EnvName;
+
 #[cfg(test)]
 #[path = "./name_test.rs"]
 mod name_test;
@@ -264,6 +266,12 @@ impl OptNameBuf {
     }
 }
 
+impl EnvName for OptNameBuf {
+    fn env_name(&self) -> String {
+        self.0.replace('-', "_")
+    }
+}
+
 /// Ensure that the provided string is a valid option name.
 ///
 /// This is for checking option names with or without any leading

crates/spk-schema/crates/foundation/src/option_map/mod.rs

@@ -14,6 +14,7 @@ use once_cell::sync::Lazy;
 use serde::{Deserialize, Serialize};
 
 use crate::name::{OptName, OptNameBuf, PkgName};
+use crate::spec_ops::EnvName;
 
 mod error;
 mod format;
@@ -186,7 +187,7 @@ impl OptionMap {
     pub fn to_environment(&self) -> HashMap<String, String> {
         let mut out = HashMap::default();
         for (name, value) in self.iter() {
-            let var_name = format!("SPK_OPT_{name}");
+            let var_name = format!("SPK_OPT_{}", name.env_name());
             out.insert(var_name, value.into());
         }
         out