feat(medium): Improve Bluetooth Connection Retry & State Management

app/client/connect/SignalQualityIndicator.test.tsx

@@ -3,6 +3,18 @@ import { render, screen } from '@testing-library/react'
 import { SignalQualityIndicator } from './SignalQualityIndicator'
 import '@testing-library/jest-dom'
 
+// Mock Tooltip to ensure title is rendered as an attribute for easy testing
+jest.mock('@mui/material', () => ({
+  ...jest.requireActual('@mui/material'),
+  Tooltip: ({
+    children,
+    title,
+  }: {
+    children: React.ReactElement<{ title?: string }>
+    title: string
+  }) => React.cloneElement(children, { title }),
+}))
+
 // Mock MUI icons
 jest.mock('@mui/icons-material/SignalCellularAlt', () => ({
   __esModule: true,

app/client/connect/UserSettings.tsx

@@ -120,6 +120,7 @@ const UserSettings: React.FC<UserSettingsProps> = ({
               }
             }}
             onBlur={onHeightBlur}
+            error={!!heightError}
           />
           <TextField
             fullWidth
@@ -133,6 +134,8 @@ const UserSettings: React.FC<UserSettingsProps> = ({
               }
             }}
             onBlur={onHeightBlur}
+            error={!!heightError}
+            helperText={heightError}
           />
         </Stack>
       )}

app/client/connect/page.test.tsx

@@ -12,11 +12,12 @@ jest.mock('@/hooks/useBluetoothHRM')
 // Mock the WebSocket context
 jest.mock('@/context/WebSocketContext')
 
-const mockAutoConnect = jest.fn()
-const mockConnectAndStream = jest.fn()
+const mockAutoConnect = jest.fn().mockResolvedValue(undefined)
+const mockConnectAndStream = jest.fn().mockResolvedValue(undefined)
 
 describe('ConnectPage', () => {
   beforeEach(() => {
+    jest.useFakeTimers()
     jest.clearAllMocks()
     jest.spyOn(BluetoothHRMHook, 'default').mockReturnValue({
       connectAndStream: mockConnectAndStream,
@@ -26,10 +27,17 @@ describe('ConnectPage', () => {
       deviceStatus: 'Disconnected',
       batteryLevel: null,
       isConnected: false,
+      isDataStale: false,
       isSupported: true,
+      signalPeriodMs: 0,
+      connectionAttempted: false,
     })
   })
 
+  afterEach(() => {
+    jest.useRealTimers()
+  })
+
   it('should call autoConnect on mount when WebSocket is connected', () => {
     jest.spyOn(WebSocketContext, 'useWebSocket').mockReturnValue({
       connectionStatus: 'Connected',
@@ -55,6 +63,7 @@ describe('ConnectPage', () => {
       </UserSettingsProvider>
     )
 
+    jest.advanceTimersByTime(100)
     expect(mockAutoConnect).toHaveBeenCalledTimes(1)
   })
 
@@ -95,7 +104,10 @@ describe('ConnectPage', () => {
       deviceStatus: 'Connected',
       batteryLevel: null,
       isConnected: true,
+      isDataStale: false,
       isSupported: true,
+      signalPeriodMs: 1000,
+      connectionAttempted: false,
     })
 
     jest.spyOn(WebSocketContext, 'useWebSocket').mockReturnValue({

constants/bluetooth-config.ts

@@ -0,0 +1,24 @@
+/**
+ * @file bluetooth-config.ts
+ * @description Centralized constants for Bluetooth HRM configuration.
+ */
+
+export const HR_SERVICE_UUID = 'heart_rate'
+export const HR_CHARACTERISTIC_UUID = 'heart_rate_measurement'
+export const BATTERY_SERVICE_UUID = 'battery_service'
+export const BATTERY_LEVEL_CHARACTERISTIC_UUID = 'battery_level'
+
+export const ROLLING_AVG_HISTORY_LENGTH = 5
+export const MISSED_PACKET_THRESHOLD_BUFFER_MS = 500
+export const MIN_MISSED_PACKET_THRESHOLD_MS = 1500
+
+export const HEARTBEAT_INTERVAL_MS_TEST = 500
+export const HEARTBEAT_INTERVAL_MS_PROD = 1000
+
+export const HEARTBEAT_INTERVAL_MS =
+  typeof process !== 'undefined' && process.env.NODE_ENV === 'test'
+    ? HEARTBEAT_INTERVAL_MS_TEST
+    : HEARTBEAT_INTERVAL_MS_PROD
+
+// Connection timeout in milliseconds
+export const CONNECTION_TIMEOUT_MS = 30000

constants/bluetooth-messages.ts

@@ -11,6 +11,7 @@ export const BLUETOOTH_MESSAGES = {
   connecting: 'Connecting...',
   connected: 'Connected',
   reconnecting: 'Reconnecting...',
+  disconnecting: 'Disconnecting...',
 
   // Custom Status Messages
   unstableConnection: 'Connection unstable. Reconnecting...',

constants/bluetooth-reconnection.ts

@@ -11,9 +11,9 @@ export const BLUETOOTH_MAX_RECONNECT_ATTEMPTS =
   typeof process !== 'undefined' &&
   process.env.NEXT_PUBLIC_BLUETOOTH_MAX_RECONNECT_ATTEMPTS
     ? parseInt(process.env.NEXT_PUBLIC_BLUETOOTH_MAX_RECONNECT_ATTEMPTS, 10)
-    : 5
+    : 15
 
 // Reconnection Delay Parameters
-export const RECONNECT_BASE_DELAY_MS = 1000
-export const RECONNECT_DELAY_INCREMENT_MS = 500
-export const RECONNECT_RANDOM_DELAY_MS = 1000
+export const RECONNECT_BASE_DELAY_MS = 2000
+export const RECONNECT_EXPONENTIAL_ATTEMPTS = 3
+export const RECONNECT_LINEAR_DELAY_MS = 5000