fix(date-picker): eliminate infinite re-render on re-open with existing selection

The useEffect that syncs picker state on open had initialStart and
initialEnd — Date objects computed on every render — in its dependency
array. Because Object.is returns false for any two distinct Date
instances, the effect fired on every render when open=true, calling
setRangeStart/setRangeEnd and triggering another render, producing an
infinite loop that crashed the page.

Fix: compute start and end as local variables inside the effect and
use the stable string props (props.startDate, props.endDate) as deps
instead.

Also removes the redundant typeof fileSize === 'number' guard in the
multipart quota check — fileSize is z.number() (required) in the
contract so it can never be undefined at that point.

Author: waleedlatif1

apps/sim/app/api/files/multipart/route.ts

@@ -134,12 +134,9 @@ export const POST = withRouteHandler(async (request: NextRequest) => {
 
         const config = getStorageConfig(storageContext)
 
-        if (
-          !QUOTA_EXEMPT_STORAGE_CONTEXTS.has(context as StorageContext) &&
-          typeof fileSize === 'number'
-        ) {
+        if (!QUOTA_EXEMPT_STORAGE_CONTEXTS.has(context as StorageContext)) {
           const { checkStorageQuota } = await import('@/lib/billing/storage')
-          const quotaCheck = await checkStorageQuota(userId, fileSize)
+          const quotaCheck = await checkStorageQuota(userId, fileSize ?? 0)
           if (!quotaCheck.allowed) {
             return NextResponse.json(
               { error: quotaCheck.error || 'Storage limit exceeded' },

apps/sim/components/emcn/components/date-picker/date-picker.tsx

@@ -553,25 +553,33 @@ const DatePicker = React.forwardRef<HTMLDivElement, DatePickerProps>((props, ref
 
   React.useEffect(() => {
     if (open && isRangeMode) {
-      setRangeStart(initialStart)
-      setRangeEnd(initialEnd)
+      const start = parseDate(props.startDate)
+      const end = parseDate(props.endDate)
+      setRangeStart(start)
+      setRangeEnd(end)
       setSelectingEnd(false)
       if (showTime) {
-        const sd = isRangeMode ? props.startDate : undefined
-        const ed = isRangeMode ? props.endDate : undefined
-        setStartTime(typeof sd === 'string' && sd.includes('T') ? sd.slice(11, 16) : '00:00')
-        setEndTime(typeof ed === 'string' && ed.includes('T') ? ed.slice(11, 16) : '23:59')
+        setStartTime(
+          typeof props.startDate === 'string' && props.startDate.includes('T')
+            ? props.startDate.slice(11, 16)
+            : '00:00'
+        )
+        setEndTime(
+          typeof props.endDate === 'string' && props.endDate.includes('T')
+            ? props.endDate.slice(11, 16)
+            : '23:59'
+        )
       }
-      if (initialStart) {
-        setViewMonth(initialStart.getMonth())
-        setViewYear(initialStart.getFullYear())
+      if (start) {
+        setViewMonth(start.getMonth())
+        setViewYear(start.getFullYear())
       } else {
         const now = new Date()
         setViewMonth(now.getMonth())
         setViewYear(now.getFullYear())
       }
     }
-  }, [open, isRangeMode, initialStart, initialEnd, showTime, props.startDate, props.endDate])
+  }, [open, isRangeMode, props.startDate, props.endDate, showTime])
 
   const singleValueKey = !isRangeMode && selectedDate ? selectedDate.getTime() : undefined
   const [prevSingleValueKey, setPrevSingleValueKey] = React.useState(singleValueKey)