fix: careful now

src/__tests__/utils/number-utils.test.ts

@@ -1,4 +1,11 @@
 import { clampToRange } from '../../utils/number-utils'
+import { logger } from '../../utils/logger'
+
+jest.mock('../../utils/logger', () => ({
+    logger: {
+        warn: jest.fn(),
+    },
+}))
 
 describe('number-utils', () => {
     describe('clampToRange', () => {
@@ -13,5 +20,10 @@ describe('number-utils', () => {
             const result = clampToRange(value, min, max, 'Test Label')
             expect(result).toBe(expected)
         })
+
+        it('logs a warning when min is greater than max', () => {
+            expect(clampToRange(50, 100, 10, 'Test Label')).toBe(10)
+            expect(logger.warn).toHaveBeenCalledWith('min cannot be greater than max.')
+        })
     })
 })

src/utils/number-utils.ts

@@ -2,6 +2,11 @@ import { isNumber } from './type-utils'
 import { logger } from './logger'
 
 export function clampToRange(value: unknown, min: number, max: number, label?: string): number {
+    if (min > max) {
+        logger.warn('min cannot be greater than max.')
+        min = max
+    }
+
     if (!isNumber(value)) {
         label && logger.warn(label + ' must be a number. Defaulting to max value:' + max)
         return max