From 830de45bfab8387c1c898c9087a2ccdb92406328 Mon Sep 17 00:00:00 2001
From: dal <dallin@buster.so>
Date: Fri, 2 May 2025 14:45:27 -0600
Subject: [PATCH] llm clarification on metric yml and lock metric yml to an
 enum

---
 api/libs/agents/src/tools/categories/file_tools/common.rs | 2 +-
 api/libs/database/src/types/metric_yml.rs                 | 8 ++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/api/libs/agents/src/tools/categories/file_tools/common.rs b/api/libs/agents/src/tools/categories/file_tools/common.rs
index 24ff48c9e..02636c9cb 100644
--- a/api/libs/agents/src/tools/categories/file_tools/common.rs
+++ b/api/libs/agents/src/tools/categories/file_tools/common.rs
@@ -445,7 +445,7 @@ definitions:
         description: Custom display name for the column
       numberSeparatorStyle:
         type: string
-        description: Style for number separators. Your option is ',' or null.  Null results in no separator.
+        description: Style for number separators. Your option is ',' or a null value.  Not null wrapped in quotes, a null value.
       minimumFractionDigits:
         type: integer
         description: Minimum number of fraction digits to display
diff --git a/api/libs/database/src/types/metric_yml.rs b/api/libs/database/src/types/metric_yml.rs
index 320ff5852..67f5d6bdd 100644
--- a/api/libs/database/src/types/metric_yml.rs
+++ b/api/libs/database/src/types/metric_yml.rs
@@ -322,6 +322,14 @@ pub struct BaseChartConfig {
     pub y2_axis_config: Option<Y2AxisConfig>,
 }
 
+#[derive(Debug, Serialize, Deserialize, Clone)]
+#[serde(rename_all = "camelCase")]
+pub enum NumberSeparatorStyle {
+    #[serde(rename = ",")]
+    Comma,
+    None,
+}
+
 #[derive(Debug, Serialize, Deserialize, Clone)]
 #[serde(rename_all = "camelCase")]
 pub struct ColumnLabelFormat {