@terreno/ui 0.13.3 → 0.14.0

package/src/Common.ts

@@ -4,6 +4,7 @@ import type {ReactElement, ReactNode} from "react";
 import type {
   ImageStyle,
   ListRenderItemInfo,
+  ScrollView,
   StyleProp,
   TextInput,
   TextStyle,
@@ -463,7 +464,7 @@ export interface BoxPropsBase {
   lgColumn?: UnsignedUpTo12;
   dangerouslySetInlineStyle?: {
     __style: {
-      // noExplicitAny: escape hatch for arbitrary inline style values
+      // biome-ignore lint/suspicious/noExplicitAny: escape hatch for arbitrary inline style values that users may need to set
       [key: string]: any;
     };
   };
@@ -549,7 +550,7 @@ export interface BoxPropsBase {
 
   avoidKeyboard?: boolean;
   keyboardOffset?: number;
-  scrollRef?: React.RefObject<any>;
+  scrollRef?: React.RefObject<ScrollView | null>;
   onScroll?: (offsetY: number) => void;
   onLayout?: (event: LayoutChangeEvent) => void;
   testID?: string;
@@ -858,17 +859,17 @@ export interface SplitPageProps {
   loading?: boolean;
   color?: SurfaceColor;
   keyboardOffset?: number;
-  // noExplicitAny: ListRenderItemInfo generic type depends on the consumer's data shape
+  // biome-ignore lint/suspicious/noExplicitAny: ListRenderItemInfo generic type depends on the consumer's data shape
   renderListViewItem: (itemInfo: ListRenderItemInfo<any>) => ReactElement | null;
   renderListViewHeader?: () => ReactElement | null;
   renderContent?: (index?: number) => ReactElement | ReactElement[] | null;
-  // noExplicitAny: list data type varies by consumer's data model
+  // biome-ignore lint/suspicious/noExplicitAny: list data type varies by consumer's data model
   listViewData: any[];
   listViewExtraData?: unknown;
   listViewWidth?: number;
   listViewMaxWidth?: number;
   renderChild?: () => ReactChild;
-  // noExplicitAny: callback value type varies by consumer's data model
+  // biome-ignore lint/suspicious/noExplicitAny: callback value type varies by consumer's data model
   onSelectionChange?: (value?: any) => void | Promise<void>;
 }
 
@@ -1500,6 +1501,8 @@ export interface BodyProps {
   children?: ReactNode;
 }
 
+export type ButtonPressAnimation = "scale" | "opacity" | "none";
+
 export interface ButtonProps {
   /**
    * The text content of the confirmation modal.
@@ -1538,6 +1541,11 @@ export interface ButtonProps {
    * The subtitle of the confirmation modal.
    */
   modalSubTitle?: string;
+  /**
+   * The press animation to use when the button is touched.
+   * @default "scale"
+   */
+  pressAnimation?: ButtonPressAnimation;
   /**
    * The test ID for the button, used for testing purposes.
    */
@@ -1675,6 +1683,7 @@ export interface DateTimeActionSheetProps {
   type?: "date" | "time" | "datetime";
   // Returns an ISO 8601 string. If mode is "time", the date portion is today.
   onChange: OnChangeCallback;
+  // biome-ignore lint/suspicious/noExplicitAny: ActionSheet class lives in ActionSheet.tsx which imports from Common.ts; typing this would create a circular import
   actionSheetRef: React.RefObject<any>;
   visible: boolean;
   onDismiss: () => void;
@@ -1686,6 +1695,7 @@ export interface DecimalRangeActionSheetProps {
   min: number;
   max: number;
   onChange: OnChangeCallback;
+  // biome-ignore lint/suspicious/noExplicitAny: ActionSheet class lives in ActionSheet.tsx which imports from Common.ts; typing this would create a circular import
   actionSheetRef: React.RefObject<any>;
 }
 
@@ -1745,6 +1755,7 @@ export type FieldProps =
 export interface HeightActionSheetProps {
   value?: string;
   onChange: OnChangeCallback;
+  // biome-ignore lint/suspicious/noExplicitAny: ActionSheet class lives in ActionSheet.tsx which imports from Common.ts; typing this would create a circular import
   actionSheetRef: React.RefObject<any>;
   /** Minimum height in total inches */
   min?: number;
@@ -1756,14 +1767,17 @@ export interface HeightActionSheetProps {
 
 export interface HyperlinkProps {
   linkDefault?: boolean;
-  // noExplicitAny: type comes from external linkify-it library which lacks an exported type
+  // biome-ignore lint/suspicious/noExplicitAny: linkify-it library's main export lacks a TypeScript type definition
   linkify?: any;
+  // biome-ignore lint/suspicious/noExplicitAny: StyleProp's generic is heterogeneous (TextStyle | ViewStyle) for link contexts
   linkStyle?: StyleProp<any>;
   linkText?: string | ((url: string) => string);
   onPress?: (url: string) => void;
   onLongPress?: (url: string, text: string) => void;
+  // biome-ignore lint/suspicious/noExplicitAny: returned view props are spread onto a heterogeneous View; consumers pass arbitrary props
   injectViewProps?: (url: string) => any;
   children?: React.ReactNode;
+  // biome-ignore lint/suspicious/noExplicitAny: StyleProp's generic is heterogeneous for the container which holds mixed Text/View children
   style?: StyleProp<any>;
 }
 
@@ -1913,12 +1927,12 @@ export interface ModalProps {
   /**
    * The function to call when the primary button is clicked.
    */
-  // noExplicitAny: callback value type varies by consumer context
+  // biome-ignore lint/suspicious/noExplicitAny: callback value type varies by consumer context
   primaryButtonOnClick?: (value?: any) => void | Promise<void>;
   /**
    * The function to call when the secondary button is clicked.
    */
-  // noExplicitAny: callback value type varies by consumer context
+  // biome-ignore lint/suspicious/noExplicitAny: callback value type varies by consumer context
   secondaryButtonOnClick?: (value?: any) => void | Promise<void>;
 }
 
@@ -1927,11 +1941,12 @@ export interface NumberPickerActionSheetProps {
   min: number;
   max: number;
   onChange: OnChangeCallback;
+  // biome-ignore lint/suspicious/noExplicitAny: ActionSheet class lives in ActionSheet.tsx which imports from Common.ts; typing this would create a circular import
   actionSheetRef: React.RefObject<any>;
 }
 
 export interface PageProps {
-  // noExplicitAny: React Navigation type varies by navigation stack configuration
+  // biome-ignore lint/suspicious/noExplicitAny: React Navigation type varies by navigation stack configuration
   navigation?: any;
   scroll?: boolean;
   loading?: boolean;
@@ -2252,6 +2267,7 @@ export interface TextFieldPickerActionSheetProps {
   value?: string;
   mode?: "date" | "time";
   onChange: OnChangeCallback;
+  // biome-ignore lint/suspicious/noExplicitAny: ActionSheet class lives in ActionSheet.tsx which imports from Common.ts; typing this would create a circular import
   actionSheetRef: React.RefObject<any>;
 }
 
@@ -2371,19 +2387,19 @@ export type TapToEditProps =
 
 export interface BaseTapToEditProps extends Omit<FieldProps, "onChange" | "value"> {
   title: string;
-  // noExplicitAny: value type varies across TapToEdit field types (text, number, date, etc.)
+  // biome-ignore lint/suspicious/noExplicitAny: value type varies across TapToEdit field types (text, number, date, etc.)
   value: any;
 
   /**
    * Not required if not editable.
    */
-  // noExplicitAny: value type varies across TapToEdit field types
+  // biome-ignore lint/suspicious/noExplicitAny: value type varies across TapToEdit field types
   setValue?: (value: any) => void;
 
   /**
    * Not required if not editable.
    */
-  // noExplicitAny: value type varies across TapToEdit field types
+  // biome-ignore lint/suspicious/noExplicitAny: value type varies across TapToEdit field types
   onSave?: (value: any) => void | Promise<void>;
 
   /**
@@ -2396,7 +2412,7 @@ export interface BaseTapToEditProps extends Omit<FieldProps, "onChange" | "value
    * Enable edit mode from outside the component.
    */
   isEditing?: boolean;
-  // noExplicitAny: input value type varies across TapToEdit field types
+  // biome-ignore lint/suspicious/noExplicitAny: input value type varies across TapToEdit field types
   transform?: (value: any) => string;
   /**
    * Show a confirmation modal before saving the value.
@@ -2485,11 +2501,12 @@ export interface ModelFields {
 
 export interface OpenAPISpec {
   paths: {
-    // noExplicitAny: OpenAPI path items are deeply accessed with chained property lookups
+    // biome-ignore lint/suspicious/noExplicitAny: OpenAPI path items are deeply accessed with chained property lookups
     [key: string]: any;
   };
 }
 
+// biome-ignore lint/suspicious/noExplicitAny: ModelFieldConfig is a passthrough for arbitrary field configuration objects from various model contexts
 export type ModelFieldConfig = any;
 
 export interface OpenAPIProviderProps {
@@ -2518,7 +2535,7 @@ export interface ModelAdminFieldConfig {
 
 // The props for a custom column component for ModelAdmin.
 export interface ModelAdminCustomComponentProps extends Omit<FieldProps, "name"> {
-  // noExplicitAny: document shape varies by model used with ModelAdmin
+  // biome-ignore lint/suspicious/noExplicitAny: document shape varies by model used with ModelAdmin
   doc: any;
   fieldKey: string; // Dot notation representation of the field.
   // user: User;

package/src/ConsentFormScreen.test.tsx

@@ -208,6 +208,108 @@ describe("ConsentFormScreen", () => {
     expect(queryByTestId("consent-form-required-legend")).toBeNull();
   });
 
+  it("confirms the modal and toggles the checkbox on", () => {
+    const form: ConsentFormPublic = {
+      ...baseForm,
+      checkboxes: [{confirmationPrompt: "Are you sure?", label: "Tricky", required: true}],
+    };
+    const {getByTestId, getByText, queryByTestId} = renderWithTheme(
+      <ConsentFormScreen form={form} locale="en" onAgree={() => {}} />
+    );
+    act(() => {
+      fireEvent.press(getByTestId("consent-form-checkbox-0"));
+    });
+    expect(getByText("Are you sure?")).toBeTruthy();
+    // Press the "Confirm" button inside the modal
+    act(() => {
+      fireEvent.press(getByText("Confirm"));
+    });
+    return new Promise<void>((resolve) => {
+      setTimeout(() => {
+        // Checkbox hint should be gone because the required checkbox was toggled on
+        expect(queryByTestId("consent-footer-checkboxes-hint")).toBeNull();
+        resolve();
+      }, 600);
+    });
+  });
+
+  it("dismisses the modal without toggling the checkbox", () => {
+    const form: ConsentFormPublic = {
+      ...baseForm,
+      checkboxes: [{confirmationPrompt: "Are you sure?", label: "Tricky", required: true}],
+    };
+    const {getByTestId, getByText} = renderWithTheme(
+      <ConsentFormScreen form={form} locale="en" onAgree={() => {}} />
+    );
+    act(() => {
+      fireEvent.press(getByTestId("consent-form-checkbox-0"));
+    });
+    expect(getByText("Are you sure?")).toBeTruthy();
+    // Press the "Cancel" button inside the modal
+    act(() => {
+      fireEvent.press(getByText("Cancel"));
+    });
+    return new Promise<void>((resolve) => {
+      setTimeout(() => {
+        // The checkbox hint should still show because the checkbox was not toggled
+        expect(getByTestId("consent-footer-checkboxes-hint")).toBeTruthy();
+        resolve();
+      }, 600);
+    });
+  });
+
+  it("auto-satisfies scroll requirement when content fits the viewport via contentSizeChange", () => {
+    const form = {...baseForm, requireScrollToBottom: true};
+    const {getByTestId, queryByTestId} = renderWithTheme(
+      <ConsentFormScreen form={form} locale="en" onAgree={() => {}} />
+    );
+    const scroll = getByTestId("consent-form-scroll-view");
+    // First set the layout height, then content size smaller than layout
+    act(() => {
+      fireEvent(scroll, "layout", {nativeEvent: {layout: {height: 500}}});
+    });
+    act(() => {
+      fireEvent(scroll, "contentSizeChange", 0, 400);
+    });
+    expect(queryByTestId("consent-form-scroll-hint")).toBeNull();
+  });
+
+  it("auto-satisfies scroll requirement when content fits the viewport via layout", () => {
+    const form = {...baseForm, requireScrollToBottom: true};
+    const {getByTestId, queryByTestId} = renderWithTheme(
+      <ConsentFormScreen form={form} locale="en" onAgree={() => {}} />
+    );
+    const scroll = getByTestId("consent-form-scroll-view");
+    // First set the content size, then layout height larger than content
+    act(() => {
+      fireEvent(scroll, "contentSizeChange", 0, 300);
+    });
+    act(() => {
+      fireEvent(scroll, "layout", {nativeEvent: {layout: {height: 500}}});
+    });
+    expect(queryByTestId("consent-form-scroll-hint")).toBeNull();
+  });
+
+  it("handleScroll returns early when already scrolled to bottom", () => {
+    const form = {...baseForm, requireScrollToBottom: false};
+    const {getByTestId} = renderWithTheme(
+      <ConsentFormScreen form={form} locale="en" onAgree={() => {}} />
+    );
+    const scroll = getByTestId("consent-form-scroll-view");
+    // requireScrollToBottom is false so hasScrolledToBottom starts as true.
+    // Firing scroll should hit the early return at line 81.
+    act(() => {
+      fireEvent(scroll, "scroll", {
+        nativeEvent: {
+          contentOffset: {y: 0},
+          contentSize: {height: 1000},
+          layoutMeasurement: {height: 500},
+        },
+      });
+    });
+    expect(scroll).toBeTruthy();
+  });
+
   it("shows the checkbox footer hint when a required checkbox is unchecked", () => {
     const form: ConsentFormPublic = {
       ...baseForm,

package/src/ConsentFormScreen.tsx

@@ -1,5 +1,11 @@
 import React, {useState} from "react";
-import {Pressable, ScrollView} from "react-native";
+import {
+  type LayoutChangeEvent,
+  type NativeScrollEvent,
+  type NativeSyntheticEvent,
+  Pressable,
+  ScrollView,
+} from "react-native";
 
 import {Box} from "./Box";
 import {Button} from "./Button";
@@ -62,7 +68,7 @@ export const ConsentFormScreen: React.FC<ConsentFormScreenProps> = ({
     }
   };
 
-  const handleLayout = (event: any) => {
+  const handleLayout = (event: LayoutChangeEvent) => {
     const h = event.nativeEvent.layout.height;
     setLayoutHeight(h);
     if (!hasScrolledToBottom && contentHeight > 0 && h > 0 && contentHeight <= h) {
@@ -70,7 +76,7 @@ export const ConsentFormScreen: React.FC<ConsentFormScreenProps> = ({
     }
   };
 
-  const handleScroll = (event: any) => {
+  const handleScroll = (event: NativeSyntheticEvent<NativeScrollEvent>) => {
     if (hasScrolledToBottom) {
       return;
     }

package/src/ConsentNavigator.test.tsx

@@ -1,3 +1,4 @@
+// biome-ignore-all lint/suspicious/noExplicitAny: test mock typing
 import {describe, expect, it, mock} from "bun:test";
 import React from "react";
 import {Pressable} from "react-native";

package/src/ConsentNavigator.tsx

@@ -10,11 +10,12 @@ import type {SubmitConsentBody} from "./useSubmitConsent";
 import {useSubmitConsent} from "./useSubmitConsent";
 
 interface ConsentNavigatorProps {
+  // biome-ignore lint/suspicious/noExplicitAny: RTK Query api instance is a complex generic type that varies per consumer
   api: any;
   baseUrl?: string;
   children: React.ReactNode;
   extraScreens?: React.ReactNode[];
-  onError?: (error: any) => void;
+  onError?: (error: unknown) => void;
   variables?: Record<string, string>;
 }
 
@@ -48,7 +49,8 @@ export const ConsentNavigator: React.FC<ConsentNavigatorProps> = ({
   }
 
   if (error) {
-    const status = (error as any)?.status ?? (error as any)?.originalStatus;
+    const errorObj = error as {status?: number; originalStatus?: number};
+    const status = errorObj?.status ?? errorObj?.originalStatus;
     console.warn("[ConsentNavigator] Error fetching pending consents:", {error, status});
     // On auth errors, pass through to let the app handle re-authentication
     if (status === 401 || status === 403) {
@@ -80,7 +82,7 @@ export const ConsentNavigator: React.FC<ConsentNavigatorProps> = ({
         `[ConsentNavigator] Showing extra screen ${extraScreenIndex + 1}/${validExtraScreens.length}`
       );
       if (React.isValidElement(currentScreen)) {
-        return React.cloneElement(currentScreen as React.ReactElement<any>, {
+        return React.cloneElement(currentScreen as React.ReactElement<{onNext?: () => void}>, {
           onNext: () => setExtraScreenIndex((i) => i + 1),
         });
       }

package/src/CustomSelectField.tsx

@@ -101,7 +101,9 @@ export const CustomSelectField: FC<CustomSelectFieldProps> = ({
           <TextField
             disabled={disabled}
             id="customOptions"
-            inputRef={(ref: any) => (textInputRef.current = ref)}
+            inputRef={(ref: TextInput | null) => {
+              textInputRef.current = ref;
+            }}
             onChange={onChange}
             placeholder="None selected"
             type="text"

package/src/DataTable.test.tsx

@@ -1,3 +1,4 @@
+// biome-ignore-all lint/suspicious/noExplicitAny: test mock typing
 import {describe, expect, it, mock} from "bun:test";
 
 import {DataTable} from "./DataTable";

package/src/DataTable.tsx

@@ -264,7 +264,7 @@ const DataTableHeaderCell: FC<DataTableHeaderCellProps> = ({
       }}
     >
       {[
-        Boolean(column.title) ? (
+        column.title ? (
           <TableTitle align="left" key="data-table-header-title" title={column.title!} />
         ) : null,
         <View key="data-table-header-tools" style={{alignItems: "center", flexDirection: "row"}}>

package/src/DateTimeActionSheet.tsx

@@ -369,7 +369,11 @@ const DateCalendar = ({
   const {theme} = useTheme();
 
   const markedDates: {
-    [id: string]: {selected: boolean; selectedColor: string; customStyles?: any};
+    [id: string]: {
+      selected: boolean;
+      selectedColor: string;
+      customStyles?: {container?: {backgroundColor?: string; borderRadius?: number}};
+    };
   } = {};
 
   // Check if the date is T00:00:00.000Z (it should be), otherwise treat it as a date in the
@@ -471,7 +475,7 @@ export const DateTimeActionSheet = ({
 
   // If the value changes in the props, update the state for the date and time.
   useEffect(() => {
-    let datetime;
+    let datetime: DateTime;
     if (value) {
       if (type === "date") {
         datetime = DateTime.fromISO(value).toUTC().set({millisecond: 0, second: 0});
@@ -495,7 +499,7 @@ export const DateTimeActionSheet = ({
     setHour(h);
     setMinute(datetime.minute);
     setAmPm(datetime.toFormat("a") === "AM" ? "am" : "pm");
-    setDate(datetime.toISO());
+    setDate(datetime.toISO() ?? "");
     // Reset timezone when the sent date changes.
     setTimezone(originalTimezone);
   }, [value, originalTimezone, type]);

package/src/DateTimeField.test.tsx

@@ -1,3 +1,4 @@
+// biome-ignore-all lint/suspicious/noExplicitAny: test mock typing
 import {afterEach, beforeEach, describe, expect, it, type mock} from "bun:test";
 import {act, userEvent} from "@testing-library/react-native";
 import {DateTime} from "luxon";

package/src/DateTimeField.tsx

@@ -418,6 +418,7 @@ export const DateTimeField: FC<DateTimeFieldProps> = ({
   helperText,
 }): React.ReactElement => {
   const {theme} = useTheme();
+  // biome-ignore lint/suspicious/noExplicitAny: ActionSheet class is defined in ActionSheet.tsx which imports from Common.ts indirectly; using its type here would create a circular dependency
   const dateActionSheetRef: React.RefObject<any> = React.createRef();
   const [amPm, setAmPm] = useState<"am" | "pm">("am");
   const [showDate, setShowDate] = useState(false);
@@ -567,7 +568,7 @@ export const DateTimeField: FC<DateTimeFieldProps> = ({
       const dayVal = override?.day ?? day;
       const yearVal = override?.year ?? year;
       const hourVal = override?.hour ?? hour;
-      let date;
+      let date: DateTime;
       if (type === "datetime") {
         if (!monthVal || !dayVal || !yearVal || !hour || !minuteVal) {
           return undefined;
@@ -635,7 +636,7 @@ export const DateTimeField: FC<DateTimeFieldProps> = ({
 
       if (date.isValid) {
         // Always return UTC ISO string
-        return date.toUTC().toISO();
+        return date.toUTC().toISO() ?? undefined;
       }
       return undefined;
     },

package/src/DateUtilities.test.ts

@@ -1,5 +1,8 @@
 import {beforeEach, describe, expect, it, mock} from "bun:test";
 import {
+  convertNullToUndefined,
+  getIsoDate,
+  getTimezoneOptions,
   humanDate,
   humanDateAndTime,
   printDate,
@@ -454,5 +457,113 @@ describe("DateUtilities", () => {
       // print without ago
       expect(printSince("2019-12-23T11:00:00.000Z", {showAgo: false})).toBe("3 years");
     });
+
+    it("should throw for invalid date", () => {
+      expect(() => printSince("not-a-date")).toThrow("printSince: Invalid date: not-a-date");
+    });
+  });
+
+  describe("humanDate – non-string input", () => {
+    it("should throw for non-string date", () => {
+      expect(() => humanDate(123 as unknown as string)).toThrow(
+        "humanDate: Invalid date type: number"
+      );
+    });
+  });
+
+  describe("humanDateAndTime – showTimezone false", () => {
+    it("should format time without timezone abbreviation", () => {
+      const result = humanDateAndTime("2022-12-24T12:00:00.000Z", {showTimezone: false});
+      expect(result).toBe("7:00 AM");
+    });
+
+    it("should format tomorrow without timezone abbreviation", () => {
+      const result = humanDateAndTime("2022-12-25T12:00:00.000Z", {showTimezone: false});
+      expect(result).toBe("Tomorrow 7:00 AM");
+    });
+  });
+
+  describe("printDate – showTimezone warning", () => {
+    it("should still return the date when showTimezone is true", () => {
+      const result = printDate("2022-12-24T12:00:00.000Z", {showTimezone: true});
+      expect(result).toBe("12/24/2022");
+    });
+  });
+
+  describe("printDateRange – timeOnly with different dates", () => {
+    it("should warn but still return time range when dates differ", () => {
+      const result = printDateRange("2022-12-24T12:00:00.000Z", "2022-12-25T18:00:00.000Z", {
+        timeOnly: true,
+        timezone: "America/New_York",
+      });
+      expect(result).toBe("7:00 AM - 1:00 PM EST");
+    });
+  });
+
+  describe("convertNullToUndefined", () => {
+    it("should return the string when given a string", () => {
+      expect(convertNullToUndefined("hello")).toBe("hello");
+    });
+
+    it("should return undefined when given null", () => {
+      expect(convertNullToUndefined(null)).toBeUndefined();
+    });
+  });
+
+  describe("getIsoDate", () => {
+    it("should return undefined for undefined input", () => {
+      expect(getIsoDate(undefined)).toBeUndefined();
+    });
+
+    it("should return undefined for empty string", () => {
+      expect(getIsoDate("")).toBeUndefined();
+    });
+
+    it("should return ISO string for valid date", () => {
+      const result = getIsoDate("2022-12-24T12:00:00.000Z");
+      expect(result).toBe("2022-12-24T12:00:00.000Z");
+    });
+  });
+
+  describe("humanDate – non-Error thrown", () => {
+    it("should stringify a non-Error value thrown during date parsing", () => {
+      expect(() => humanDate(42 as unknown as string)).toThrow(
+        "humanDate: Invalid date type: number"
+      );
+    });
+  });
+
+  describe("getTimezoneOptions", () => {
+    it("returns US timezone options with full labels", () => {
+      const options = getTimezoneOptions("USA");
+      expect(options.length).toBe(7);
+      const labels = options.map((o) => o.label);
+      expect(labels).toContain("Eastern");
+      expect(labels).toContain("Pacific");
+      expect(labels).toContain("Arizona");
+    });
+
+    it("returns US timezone options with short labels", () => {
+      const options = getTimezoneOptions("USA", true);
+      expect(options.length).toBe(7);
+      const azOption = options.find((o) => o.value === "America/Phoenix");
+      expect(azOption?.label).toBe("AZ");
+    });
+
+    it("returns worldwide timezone options", () => {
+      const options = getTimezoneOptions("Worldwide");
+      expect(options.length).toBeGreaterThan(7);
+      const values = options.map((o) => o.value);
+      expect(values).toContain("America/New_York");
+    });
+
+    it("returns worldwide timezone options with short labels", () => {
+      const options = getTimezoneOptions("Worldwide", true);
+      expect(options.length).toBeGreaterThan(7);
+      options.forEach((o) => {
+        expect(typeof o.label).toBe("string");
+        expect(typeof o.value).toBe("string");
+      });
+    });
   });
 });

package/src/DateUtilities.tsx

@@ -61,7 +61,7 @@ export function humanDate(
   date: string,
   {timezone, dontShowTime}: {timezone?: string; dontShowTime?: boolean} = {}
 ): string {
-  let clonedDate;
+  let clonedDate: DateTime;
   try {
     clonedDate = getDate(date, {timezone});
   } catch (error: unknown) {
@@ -95,7 +95,7 @@ export function humanDateAndTime(
   date: string,
   {timezone, showTimezone = true}: {timezone?: string; showTimezone?: boolean} = {}
 ): string {
-  let clonedDate;
+  let clonedDate: DateTime;
   try {
     clonedDate = getDate(date, {timezone});
   } catch (error: unknown) {
@@ -162,7 +162,7 @@ export const printDate = (
     return justDate.startOf("day").toFormat("M/d/yyyy");
   }
 
-  let clonedDate;
+  let clonedDate: DateTime;
   try {
     clonedDate = getDate(date, {timezone});
   } catch (error: unknown) {
@@ -214,7 +214,7 @@ export function printTime(
   if (!date) {
     return defaultValue ?? "Invalid Date";
   }
-  let clonedDate;
+  let clonedDate: DateTime;
   if (!timezone) {
     throw new Error("printTime: timezone is required");
   }
@@ -250,7 +250,7 @@ export function printDateAndTime(
   if (!date) {
     return defaultValue ?? "Invalid Datetime";
   }
-  let clonedDate;
+  let clonedDate: DateTime;
   try {
     clonedDate = getDate(date, {timezone});
   } catch (error: unknown) {
@@ -307,7 +307,7 @@ export function printSince(
   date: string,
   {timezone, showAgo = true}: {timezone?: string; showAgo?: boolean} = {}
 ): string {
-  let clonedDate;
+  let clonedDate: DateTime;
   const ago = showAgo ? " ago" : "";
   try {
     clonedDate = getDate(date, {timezone});

package/src/DecimalRangeActionSheet.test.tsx

@@ -94,4 +94,32 @@ describe("DecimalRangeActionSheet", () => {
     });
     expect(handleChange).toHaveBeenCalled();
   });
+
+  it("invokes actionSheetRef.setModalVisible(false) when Close is pressed", () => {
+    const actionSheetRef = createRef<ActionSheet>();
+    const {UNSAFE_getAllByProps} = render(
+      <ThemeProvider>
+        <DecimalRangeActionSheet
+          actionSheetRef={actionSheetRef}
+          max={10}
+          min={0}
+          onChange={() => {}}
+          value="5.5"
+        />
+      </ThemeProvider>
+    );
+
+    // Spy on the ActionSheet instance's setModalVisible after React assigns the ref
+    const spy = mock(() => {});
+    if (actionSheetRef.current) {
+      actionSheetRef.current.setModalVisible = spy;
+    }
+
+    const closeButtons = UNSAFE_getAllByProps({text: "Close"});
+    const buttonOnClick = closeButtons[0].props.onClick as () => void;
+    act(() => {
+      buttonOnClick();
+    });
+    expect(spy).toHaveBeenCalledWith(false);
+  });
 });