fix: preserve quantile level in dashboard UI export (#1735)

Reorder RootValueExpressionSchema Zod union branches so the quantile/histogram branch (with required `level` field) is checked before the general branch. Previously, `aggFn: 'quantile'` matched the general branch first, causing `level` to be stripped during schema parsing.

Fixes #1734

Author: alex-clickhouse

packages/common-utils/src/__tests__/utils.test.ts

@@ -683,6 +683,60 @@ describe('utils', () => {
         ],
       });
     });
+
+    it('should preserve level property for quantile aggFn in select', () => {
+      const dashboard: z.infer<typeof DashboardSchema> = {
+        id: 'dashboard1',
+        name: 'Quantile Dashboard',
+        tags: [],
+        tiles: [
+          {
+            id: 'tile1',
+            config: {
+              name: 'P95 Latency',
+              source: 'source1',
+              select: [
+                {
+                  aggFn: 'quantile',
+                  level: 0.95,
+                  aggCondition: '',
+                  aggConditionLanguage: 'lucene',
+                  valueExpression: 'Duration',
+                },
+              ],
+              where: '',
+            },
+            x: 0,
+            y: 0,
+            w: 6,
+            h: 6,
+          },
+        ],
+      };
+
+      const sources: TSourceUnion[] = [
+        {
+          id: 'source1',
+          name: 'Logs',
+          connection: 'connection1',
+          kind: SourceKind.Log,
+          from: {
+            databaseName: 'db1',
+            tableName: 'logs_table',
+          },
+          timestampValueExpression: 'Timestamp',
+          defaultTableSelectExpression: '',
+        },
+      ];
+
+      const template = convertToDashboardTemplate(dashboard, sources);
+      const selectList = template.tiles[0].config.select;
+      expect(Array.isArray(selectList)).toBe(true);
+      expect((selectList as any[])[0]).toMatchObject({
+        aggFn: 'quantile',
+        level: 0.95,
+      });
+    });
   });
 
   describe('isJsonExpression', () => {

packages/common-utils/src/types.ts

@@ -77,9 +77,12 @@ export const AggregateFunctionWithCombinatorsSchema = z
 export const RootValueExpressionSchema = z
   .object({
     aggFn: z.union([
-      AggregateFunctionSchema,
-      AggregateFunctionWithCombinatorsSchema,
+      z.literal('quantile'),
+      z.literal('quantileMerge'),
+      z.literal('histogram'),
+      z.literal('histogramMerge'),
     ]),
+    level: z.number(),
     aggCondition: SearchConditionSchema,
     aggConditionLanguage: SearchConditionLanguageSchema,
     valueExpression: z.string(),
@@ -89,12 +92,9 @@ export const RootValueExpressionSchema = z
   .or(
     z.object({
       aggFn: z.union([
-        z.literal('quantile'),
-        z.literal('quantileMerge'),
-        z.literal('histogram'),
-        z.literal('histogramMerge'),
+        AggregateFunctionSchema,
+        AggregateFunctionWithCombinatorsSchema,
       ]),
-      level: z.number(),
       aggCondition: SearchConditionSchema,
       aggConditionLanguage: SearchConditionLanguageSchema,
       valueExpression: z.string(),