👔 Use regex to not override existing YAML type declarations

FCP-INDI · Sep 5, 2024 · de18553 · de18553
1 parent 9c34a1e
commit de18553
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/CPAC/utils/bids_utils.py b/CPAC/utils/bids_utils.py
@@ -1165,8 +1165,8 @@ def _check_value_type(
 def coerce_data_config_strings(contents: str) -> str:
     """Coerge `subject_id` and `unique_id` to be strings."""
     for key in ["subject_id: ", "unique_id: "]:
-        contents = contents.replace(key, f"{key}!!str ")
-    return contents
+        contents = re.sub(f"{key}(?!!!)", f"{key}!!str ", contents)
+    return contents.replace(": !!str !!", ": !!")
 
 
 def load_cpac_data_config(data_config_file, participant_labels, aws_input_creds):

diff --git a/CPAC/utils/tests/configs/github_2144.yml b/CPAC/utils/tests/configs/github_2144.yml
@@ -2,3 +2,7 @@
   subject_id: 01
   unique_id: 02
   derivatives_dir: /fprep/sub-0151
+- site: site-1
+  subject_id: !!str 02
+  unique_id: 02
+  derivatives_dir: /fprep/sub-0151