From d2419ac423ea1bb7891be271fd07ef788da3a88c Mon Sep 17 00:00:00 2001
From: Bryant Park <bryantp@palantir.com>
Date: Wed, 24 Jul 2024 17:02:10 -0400
Subject: [PATCH] Handle StringLong in AggregationFromType<T> and
 MultipleAggregationFromType<T>

---
 .../legacy-client/src/client/interfaces/aggregations.ts    | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/packages/legacy-client/src/client/interfaces/aggregations.ts b/packages/legacy-client/src/client/interfaces/aggregations.ts
index 32e8d6f64..17722de87 100644
--- a/packages/legacy-client/src/client/interfaces/aggregations.ts
+++ b/packages/legacy-client/src/client/interfaces/aggregations.ts
@@ -34,6 +34,7 @@ import type {
   StringGroupBy,
   TimestampGroupBy,
 } from "../objectSets/aggregations/index.js";
+import type { StringLong } from "../utils/StringLong.js";
 import type { OmitMetadataProperties } from "./utils/OmitProperties.js";
 
 export declare type ObjectTypesGroupByFunction<
@@ -111,15 +112,17 @@ export type GroupByFromType<T, N extends string> = NonNullable<T> extends number
   : T extends Array<infer U> ? GroupByFromType<U, N>
   : never;
 
-export type AggregationFromType<T> = NonNullable<T> extends number
+export type AggregationFromType<T> = NonNullable<T> extends StringLong
   ? AggregatableProperty<Double>
+  : NonNullable<T> extends number ? AggregatableProperty<Double>
   : NonNullable<T> extends boolean ? AggregatableProperty<Double>
   : NonNullable<T> extends LocalDate ? AggregatableProperty<LocalDate>
   : NonNullable<T> extends Timestamp ? AggregatableProperty<Timestamp>
   : AggregatableProperty<never>;
 
-export type MultipleAggregationFromType<T> = NonNullable<T> extends number
+export type MultipleAggregationFromType<T> = NonNullable<T> extends StringLong
   ? MultipleAggregatableProperty<Double>
+  : NonNullable<T> extends number ? MultipleAggregatableProperty<Double>
   : NonNullable<T> extends boolean ? MultipleAggregatableProperty<Double>
   : NonNullable<T> extends LocalDate ? MultipleAggregatableProperty<LocalDate>
   : NonNullable<T> extends Timestamp ? MultipleAggregatableProperty<Timestamp>