fix lint warnings

app/components/AudioStream.tsx

@@ -98,7 +98,7 @@ function AudioTrack({
 			mediaStream.removeTrack(audioTrack)
 			onTrackRemovedRef.current(track, audioTrack)
 		}
-	}, [audioTrack])
+	}, [audioTrack, mediaStream, track])
 
 	return null
 }

app/hooks/rxjsHooks.ts

@@ -1,5 +1,6 @@
 import { useEffect, useRef, useState } from 'react'
-import { BehaviorSubject, Observable } from 'rxjs'
+import type { Observable } from 'rxjs'
+import { BehaviorSubject } from 'rxjs'
 
 export function useSubscribedState<T>(observable: Observable<T>): T | undefined
 export function useSubscribedState<T>(

app/hooks/useDeadPulledTrackMonitor.tsx

@@ -37,7 +37,7 @@ export function useDeadPulledTrackMonitor(
 			// reset this to -1 for the check above
 			timeoutRef.current = -1
 		}
-	}, [deadTrack, enabled, feedbackEnabled, peer, track])
+	}, [deadTrack, enabled, feedbackEnabled, peer, peerConnection, track])
 
 	useEffect(() => {
 		if (!sessionId || !deadTrack || !feedbackEnabled) return
@@ -65,7 +65,6 @@ export function useDeadPulledTrackMonitor(
 		name,
 		sessionId,
 		room.identity?.name,
-		sessionId,
 		traceLink,
 		trackInfo,
 	])

app/routes/_room.$roomName.room.tsx

@@ -278,6 +278,7 @@ function JoinedRoom({ bugReportsEnabled }: { bugReportsEnabled: boolean }) {
 							userMedia.videoStreamTrack &&
 							fakeUsers.map((uid) => (
 								<PullVideoTrack
+									key={identity.id}
 									video={identity.tracks.video}
 									audio={identity.tracks.audio}
 								>

app/routes/_room.tsx

@@ -145,7 +145,7 @@ function Room() {
 		useStateObservable<RTCRtpEncodingParameters[]>(videoEncodingParams)
 	const pushedVideoTrack$ = useMemo(
 		() => peer.pushTrack(userMedia.videoTrack$, videoTrackEncodingParams$),
-		[videoTrackEncodingParams$]
+		[peer, userMedia.videoTrack$, videoTrackEncodingParams$]
 	)
 
 	const pushedVideoTrack = useSubscribedState(pushedVideoTrack$)
@@ -160,7 +160,7 @@ function Room() {
 					},
 				])
 			),
-		[]
+		[peer, userMedia.publicAudioTrack$]
 	)
 	const pushedAudioTrack = useSubscribedState(pushedAudioTrack$)
 

app/routes/demo.rxjs.tsx

@@ -1,6 +1,7 @@
 import type { ComponentProps, ElementRef } from 'react'
 import { useMemo, useRef, useState } from 'react'
-import { Observable, map, shareReplay } from 'rxjs'
+import type { Observable } from 'rxjs'
+import { map, shareReplay } from 'rxjs'
 import { useObservableEffect, useSubscribedState } from '~/hooks/rxjsHooks'
 import { useIsServer } from '~/hooks/useIsServer'
 import { RxjsPeer } from '~/utils/rxjs/RxjsPeer.client'

app/utils/rxjs/devicePrioritization.ts

@@ -1,4 +1,5 @@
-import { BehaviorSubject, Observable, merge, of } from 'rxjs'
+import type { Observable } from 'rxjs'
+import { BehaviorSubject, merge, of } from 'rxjs'
 
 export interface GetDeviceListOptions {
 	prioritizeList?: Array<Partial<MediaDeviceInfo>>

app/utils/rxjs/getPacketLossStats$.ts

@@ -1,11 +1,5 @@
-import {
-	combineLatest,
-	interval,
-	map,
-	Observable,
-	pairwise,
-	switchMap,
-} from 'rxjs'
+import type { Observable } from 'rxjs'
+import { combineLatest, interval, map, pairwise, switchMap } from 'rxjs'
 import Ewma from '~/utils/ewma'
 
 export interface PacketLossStats {

app/utils/rxjs/getUserMediaTrack$.ts

@@ -1,6 +1,6 @@
+import type { Subscriber } from 'rxjs'
 import {
 	Observable,
-	Subscriber,
 	combineLatest,
 	concat,
 	distinctUntilChanged,

package.json

@@ -5,7 +5,7 @@
 	"sideEffects": false,
 	"scripts": {
 		"build": "remix build",
-		"lint": "prettier --check . && eslint --ignore-path .gitignore --cache --cache-location ./node_modules/.cache/eslint .",
+		"lint": "prettier --check . && eslint --ignore-path .gitignore --cache --cache-location ./node_modules/.cache/eslint --max-warnings=0 .",
 		"dev": "remix dev --manual -c \"npm start\"",
 		"start": "wrangler dev ./build/index.js",
 		"prebuild": "npm run clean",