build(deps): bump react and @types/react (#488)

* build(deps): bump react and @types/react

Bumps [react](https://github.com/facebook/react/tree/HEAD/packages/react) and [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react). These dependencies needed to be updated together.

Updates `react` from 18.3.1 to 19.0.0
- [Release notes](https://github.com/facebook/react/releases)
- [Changelog](https://github.com/facebook/react/blob/main/CHANGELOG.md)
- [Commits](https://github.com/facebook/react/commits/v19.0.0/packages/react)

Updates `@types/react` from 18.3.18 to 19.0.7
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react)

---
updated-dependencies:
- dependency-name: react
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: "@types/react"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

* resolve draggable component type error

* fix new JSX namespace error

* remove `use-between` library

* feat: update package.lock after react update

* fix: revert missing cast

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Fred <[email protected]>
Co-authored-by: freds-dev <[email protected]>
Co-authored-by: freds-dev <[email protected]>
Co-authored-by: Matthias Pfeil <[email protected]>
Co-authored-by: David Scheidt <[email protected]>

Author: 4 people

app/components/device-detail/device-detail-box.tsx

@@ -29,7 +29,6 @@ import {
 	useSearchParams,
 	Link,
 } from 'react-router'
-import { useBetween } from 'use-between'
 import SensorIcon from '../sensor-icon'
 import Spinner from '../spinner'
 import {
@@ -76,8 +75,9 @@ import { useToast } from '../ui/use-toast'
 import EntryLogs from './entry-logs'
 import ShareLink from './share-link'
 import { type loader } from '~/routes/explore.$deviceId'
-import  { type SensorWithLatestMeasurement } from '~/schema'
+import { type SensorWithLatestMeasurement } from '~/schema'
 import { getArchiveLink } from '~/utils/device'
+import { useGlobalCompareMode } from './useGlobalCompareMode'
 
 export interface MeasurementProps {
 	sensorId: string
@@ -87,13 +87,6 @@ export interface MeasurementProps {
 	max_value: string
 }
 
-const useCompareMode = () => {
-	const [compareMode, setCompareMode] = useState(false)
-	return { compareMode, setCompareMode }
-}
-
-export const useSharedCompareMode = () => useBetween(useCompareMode)
-
 export default function DeviceDetailBox() {
 	const navigation = useNavigation()
 	const navigate = useNavigate()
@@ -103,12 +96,12 @@ export default function DeviceDetailBox() {
 	const sensorIds = new Set()
 
 	const data = useLoaderData<typeof loader>()
-	const nodeRef = useRef(null)
+	const nodeRef = useRef<HTMLDivElement>(null)
 	// state variables
 	const [open, setOpen] = useState(true)
 	const [offsetPositionX, setOffsetPositionX] = useState(0)
 	const [offsetPositionY, setOffsetPositionY] = useState(0)
-	const { compareMode, setCompareMode } = useSharedCompareMode()
+	const [compareMode, setCompareMode] = useGlobalCompareMode()
 	const [refreshOn] = useState(false)
 	const [refreshSecond, setRefreshSecond] = useState(59)
 
@@ -195,7 +188,7 @@ export default function DeviceDetailBox() {
 		<>
 			{open && (
 				<Draggable
-					nodeRef={nodeRef}
+					nodeRef={nodeRef as React.RefObject<HTMLDivElement>}
 					defaultPosition={{ x: offsetPositionX, y: offsetPositionY }}
 					onDrag={handleDrag}
 					bounds="#osem"

app/components/device-detail/graph.tsx

@@ -13,7 +13,7 @@ import {
 import 'chartjs-adapter-date-fns'
 // import { de, enGB } from "date-fns/locale";
 import { Download, RefreshCcw, X } from 'lucide-react'
-import { useMemo, useRef, useState, useEffect, useContext } from 'react'
+import { useMemo, useRef, useState, useEffect, useContext, RefObject } from 'react'
 import { Scatter } from 'react-chartjs-2'
 import { isBrowser, isTablet } from 'react-device-detect'
 import Draggable, { type DraggableData } from 'react-draggable'
@@ -97,7 +97,7 @@ export default function Graph({
 	})
 	const isAggregated = aggregation !== 'raw'
 
-	const nodeRef = useRef(null)
+	const nodeRef = useRef<HTMLDivElement>(null)
 	const chartRef = useRef<ChartJS<'scatter'>>(null)
 
 	useEffect(() => {
@@ -518,7 +518,7 @@ export default function Graph({
 	return (
 		<>
 			<Draggable
-				nodeRef={nodeRef}
+				nodeRef={nodeRef as RefObject<HTMLDivElement>}
 				bounds="#osem"
 				handle="#graphTop"
 				defaultPosition={{ x: offsetPositionX, y: offsetPositionY }}

app/components/device-detail/useGlobalCompareMode.tsx

@@ -0,0 +1,30 @@
+import { useEffect, useState } from 'react'
+
+let globalValue = false // Shared global state
+let listeners: ((value: boolean) => void)[] = [] // Listeners for updates
+
+export const useGlobalCompareMode = () => {
+	const [compareMode, setCompareMode] = useState(globalValue)
+
+	// Function to update the global value and notify listeners
+	const updateValue = (newValue: boolean) => {
+		globalValue = newValue
+		listeners.forEach((listener) => listener(newValue))
+	}
+
+	// Subscribe to global state updates
+	const subscribe = (listener: (value: boolean) => void) => {
+		listeners.push(listener)
+		return () => {
+			listeners = listeners.filter((l) => l !== listener)
+		}
+	}
+
+	// Effect to keep the local state synced with the global state
+	useEffect(() => {
+		const unsubscribe = subscribe(setCompareMode)
+		return unsubscribe // Clean up on unmount
+	}, [])
+
+	return [compareMode, updateValue] as const
+}

app/components/error-boundary.tsx

@@ -1,10 +1,11 @@
+import React from "react";
 import { isRouteErrorResponse, useParams, useRouteError, type ErrorResponse  } from "react-router";
 import { getErrorMessage } from "~/utils/misc";
 
 type StatusHandler = (info: {
   error: ErrorResponse;
   params: Record<string, string | undefined>;
-}) => JSX.Element | null;
+}) => React.JSX.Element | null;
 
 export function GeneralErrorBoundary({
   defaultStatusHandler = ({ error }) => (
@@ -17,7 +18,7 @@ export function GeneralErrorBoundary({
 }: {
   defaultStatusHandler?: StatusHandler;
   statusHandlers?: Record<number, StatusHandler>;
-  unexpectedErrorHandler?: (error: unknown) => JSX.Element | null;
+  unexpectedErrorHandler?: (error: unknown) => React.JSX.Element | null;
 }) {
   const error = useRouteError();
   const params = useParams();

app/components/map/layers/cluster/box-marker.tsx

@@ -1,121 +1,121 @@
-import { AnimatePresence, motion } from "framer-motion";
-import { Box, Rocket } from "lucide-react";
-import { useState } from "react";
-import  { type MarkerProps, Marker, useMap  } from "react-map-gl";
-import { useMatches, useNavigate, useSearchParams } from "react-router";
-import { useSharedCompareMode } from "~/components/device-detail/device-detail-box";
-import { cn } from "~/lib/utils";
-import  { type Device } from "~/schema";
+import { AnimatePresence, motion } from 'framer-motion'
+import { Box, Rocket } from 'lucide-react'
+import { useState } from 'react'
+import { type MarkerProps, Marker, useMap } from 'react-map-gl'
+import { useMatches, useNavigate, useSearchParams } from 'react-router'
+import { useGlobalCompareMode } from '~/components/device-detail/useGlobalCompareMode'
+import { cn } from '~/lib/utils'
+import { type Device } from '~/schema'
 
 interface BoxMarkerProps extends MarkerProps {
-  device: Device;
+	device: Device
 }
 
 const getStatusColor = (device: Device) => {
-  if (device.status === "active") {
-    if (device.exposure === "mobile") {
-      return "bg-blue-100";
-    }
-    return "bg-green-300";
-  } else if (device.status === "inactive") {
-    return "bg-gray-100";
-  } else {
-    return "bg-gray-100 opacity-50";
-  }
-};
+	if (device.status === 'active') {
+		if (device.exposure === 'mobile') {
+			return 'bg-blue-100'
+		}
+		return 'bg-green-300'
+	} else if (device.status === 'inactive') {
+		return 'bg-gray-100'
+	} else {
+		return 'bg-gray-100 opacity-50'
+	}
+}
 
 export default function BoxMarker({ device, ...props }: BoxMarkerProps) {
-  const navigate = useNavigate();
-  const matches = useMatches();
-  const { osem } = useMap();
-  const { compareMode, setCompareMode } = useSharedCompareMode();
+	const navigate = useNavigate()
+	const matches = useMatches()
+	const { osem } = useMap()
+	const [compareMode, setCompareMode] = useGlobalCompareMode()
 
-  const isFullZoom = osem && osem?.getZoom() >= 14;
+	const isFullZoom = osem && osem?.getZoom() >= 14
 
-  const [isHovered, setIsHovered] = useState(false);
-  const [searchParams] = useSearchParams();
+	const [isHovered, setIsHovered] = useState(false)
+	const [searchParams] = useSearchParams()
 
-  // calculate zIndex based on device status and hover
-  const getZIndex = () => {
-    if (isHovered) {
-      return 30;
-    }
-    // priority to active devices
-    if (device.status === "active") {
-      return 20;
-    }
-    if (device.status === "inactive") {
-      return 10;
-    }
+	// calculate zIndex based on device status and hover
+	const getZIndex = () => {
+		if (isHovered) {
+			return 30
+		}
+		// priority to active devices
+		if (device.status === 'active') {
+			return 20
+		}
+		if (device.status === 'inactive') {
+			return 10
+		}
 
-    return 0;
-  };
+		return 0
+	}
 
-  return (
-    <Marker
-      {...props}
-      style={{
-        zIndex: getZIndex(),
-      }}
-    >
-      <AnimatePresence mode="popLayout">
-        <motion.div
-          className={cn(
-            "group absolute flex w-fit cursor-pointer items-center rounded-full bg-white p-1 text-sm shadow hover:z-10 hover:shadow-lg",
-            isFullZoom ? "-left-4 -top-4" : "-left-[10px] -top-[10px]",
-          )}
-          onClick={() => {
-            if (searchParams.has("sensor")) {
-              searchParams.delete("sensor");
-            }
-            if (compareMode) {
-              void navigate(
-                `/explore/${matches[2].params.deviceId}/compare/${device.id}`,
-              );
-              setCompareMode(false);
-              return;
-            }
-            void navigate({
-              pathname: `${device.id}`,
-              search: searchParams.toString(),
-            });
-          }}
-          onHoverStart={() => setIsHovered(true)}
-          onHoverEnd={() => setIsHovered(false)}
-        >
-          <span
-            className={cn(
-              "relative rounded-full transition-colors",
-              `${getStatusColor(device)} p-1`,
-            )}
-          >
-            {device.exposure === "mobile" ? (
-              <Rocket className="text-black h-4 w-4" />
-            ) : (
-              <Box className="text-black h-4 w-4" />
-            )}
-            {device.status === "active" ? (
-              <div
-                className={cn(
-                  "absolute left-0 top-0 h-full w-full animate-ping rounded-full opacity-50",
-                  getStatusColor(device),
-                )}
-              />
-            ) : null}
-          </span>
-          {isFullZoom ? (
-            <motion.span
-              layoutId={device.id}
-              className="text-black max-w-[100px] overflow-hidden overflow-ellipsis whitespace-nowrap px-1 group-hover:max-w-fit group-hover:overflow-auto"
-              initial={{ opacity: 0, translateX: -20 }}
-              animate={{ opacity: 1, translateX: 0 }}
-              exit={{ opacity: 0, translateX: -20 }}
-            >
-              {device.name}
-            </motion.span>
-          ) : null}
-        </motion.div>
-      </AnimatePresence>
-    </Marker>
-  );
+	return (
+		<Marker
+			{...props}
+			style={{
+				zIndex: getZIndex(),
+			}}
+		>
+			<AnimatePresence mode="popLayout">
+				<motion.div
+					className={cn(
+						'group absolute flex w-fit cursor-pointer items-center rounded-full bg-white p-1 text-sm shadow hover:z-10 hover:shadow-lg',
+						isFullZoom ? '-left-4 -top-4' : '-left-[10px] -top-[10px]',
+					)}
+					onClick={() => {
+						if (searchParams.has('sensor')) {
+							searchParams.delete('sensor')
+						}
+						if (compareMode) {
+							void navigate(
+								`/explore/${matches[2].params.deviceId}/compare/${device.id}`,
+							)
+							setCompareMode(false)
+							return
+						}
+						void navigate({
+							pathname: `${device.id}`,
+							search: searchParams.toString(),
+						})
+					}}
+					onHoverStart={() => setIsHovered(true)}
+					onHoverEnd={() => setIsHovered(false)}
+				>
+					<span
+						className={cn(
+							'relative rounded-full transition-colors',
+							`${getStatusColor(device)} p-1`,
+						)}
+					>
+						{device.exposure === 'mobile' ? (
+							<Rocket className="h-4 w-4 text-black" />
+						) : (
+							<Box className="h-4 w-4 text-black" />
+						)}
+						{device.status === 'active' ? (
+							<div
+								className={cn(
+									'absolute left-0 top-0 h-full w-full animate-ping rounded-full opacity-50',
+									getStatusColor(device),
+								)}
+							/>
+						) : null}
+					</span>
+					{isFullZoom ? (
+						<motion.span
+							layoutId={device.id}
+							className="max-w-[100px] overflow-hidden overflow-ellipsis whitespace-nowrap px-1 text-black group-hover:max-w-fit group-hover:overflow-auto"
+							initial={{ opacity: 0, translateX: -20 }}
+							animate={{ opacity: 1, translateX: 0 }}
+							exit={{ opacity: 0, translateX: -20 }}
+						>
+							{device.name}
+						</motion.span>
+					) : null}
+				</motion.div>
+			</AnimatePresence>
+		</Marker>
+	)
 }