chore: major pkg updates

app/components/progress-bar.tsx

@@ -22,7 +22,7 @@ function EpicProgress() {
 			.getAnimations()
 			.map(({ finished }) => finished)
 
-		Promise.allSettled(animationPromises).then(() => {
+		void Promise.allSettled(animationPromises).then(() => {
 			if (!delayedPending) setAnimationComplete(true)
 		})
 	}, [delayedPending])

app/components/ui/icon.tsx

@@ -1,7 +1,7 @@
 import { type SVGProps } from 'react'
 import { cn } from '#app/utils/misc.tsx'
-import { type IconName } from '@/icon-name'
 import href from './icons/sprite.svg'
+import { type IconName } from '@/icon-name'
 
 export { href }
 export { IconName }

app/entry.client.tsx

@@ -3,7 +3,7 @@ import { startTransition } from 'react'
 import { hydrateRoot } from 'react-dom/client'
 
 if (ENV.MODE === 'production' && ENV.SENTRY_DSN) {
-	import('./utils/monitoring.client.tsx').then(({ init }) => init())
+	void import('./utils/monitoring.client.tsx').then(({ init }) => init())
 }
 
 startTransition(() => {

app/entry.server.tsx

@@ -7,6 +7,7 @@ import {
 } from '@remix-run/node'
 import { RemixServer } from '@remix-run/react'
 import * as Sentry from '@sentry/remix'
+import chalk from 'chalk'
 import { isbot } from 'isbot'
 import { renderToPipeableStream } from 'react-dom/server'
 import { getEnv, init } from './utils/env.server.ts'
@@ -19,10 +20,6 @@ const ABORT_DELAY = 5000
 init()
 global.ENV = getEnv()
 
-if (ENV.MODE === 'production' && ENV.SENTRY_DSN) {
-	import('./utils/monitoring.server.ts').then(({ init }) => init())
-}
-
 type DocRequestArgs = Parameters<HandleDocumentRequestFunction>
 
 export default async function handleRequest(...args: DocRequestArgs) {
@@ -39,11 +36,15 @@ export default async function handleRequest(...args: DocRequestArgs) {
 	responseHeaders.set('fly-primary-instance', primaryInstance)
 	responseHeaders.set('fly-instance', currentInstance)
 
+	if (process.env.NODE_ENV === 'production' && process.env.SENTRY_DSN) {
+		responseHeaders.append('Document-Policy', 'js-profiling')
+	}
+
 	const callbackName = isbot(request.headers.get('user-agent'))
 		? 'onAllReady'
 		: 'onShellReady'
 
-	const nonce = String(loadContext.cspNonce) ?? undefined
+	const nonce = loadContext.cspNonce?.toString() ?? ''
 	return new Promise(async (resolve, reject) => {
 		let didError = false
 		// NOTE: this timing will only include things that are rendered in the shell
@@ -70,10 +71,8 @@ export default async function handleRequest(...args: DocRequestArgs) {
 				onShellError: (err: unknown) => {
 					reject(err)
 				},
-				onError: (error: unknown) => {
+				onError: () => {
 					didError = true
-
-					console.error(error)
 				},
 				nonce,
 			},
@@ -97,9 +96,21 @@ export function handleError(
 	error: unknown,
 	{ request }: LoaderFunctionArgs | ActionFunctionArgs,
 ): void {
+	// Skip capturing if the request is aborted as Remix docs suggest
+	// Ref: https://remix.run/docs/en/main/file-conventions/entry.server#handleerror
+	if (request.signal.aborted) {
+		return
+	}
 	if (error instanceof Error) {
-		Sentry.captureRemixServerException(error, 'remix.server', request)
+		console.error(chalk.red(error.stack))
+		void Sentry.captureRemixServerException(
+			error,
+			'remix.server',
+			request,
+			true,
+		)
 	} else {
+		console.error(chalk.red(error))
 		Sentry.captureException(error)
 	}
 }

app/routes/_auth+/auth.$provider.callback.test.ts

@@ -26,7 +26,7 @@ afterEach(async () => {
 test('a new user goes to onboarding', async () => {
 	const request = await setupRequest()
 	const response = await loader({ request, params: PARAMS, context: {} }).catch(
-		e => e,
+		(e) => e,
 	)
 	expect(response).toHaveRedirect('/onboarding/github')
 })
@@ -40,7 +40,7 @@ test('when auth fails, send the user to login with a toast', async () => {
 	)
 	const request = await setupRequest()
 	const response = await loader({ request, params: PARAMS, context: {} }).catch(
-		e => e,
+		(e) => e,
 	)
 	invariant(response instanceof Response, 'response should be a Response')
 	expect(response).toHaveRedirect('/login')
@@ -98,7 +98,7 @@ test(`when a user is logged in and has already connected, it doesn't do anything
 	})
 	const response = await loader({ request, params: PARAMS, context: {} })
 	expect(response).toHaveRedirect('/settings/profile/connections')
-	expect(response).toSendToast(
+	await expect(response).toSendToast(
 		expect.objectContaining({
 			title: 'Already Connected',
 			description: expect.stringContaining(githubUser.profile.login),

app/routes/bucnation.tsx

@@ -1,6 +1,4 @@
 import {
-	type LoaderFunctionArgs,
-	type ActionFunctionArgs,
 	json,
 } from '@remix-run/node'
 import {
@@ -73,7 +71,7 @@ export const meta: MetaFunction = ({ data }) => [
 	},
 ]
 
-export const loader = async ({ request }: LoaderFunctionArgs) => {
+export const loader = async () => {
 	const hoopsApi = process.env.HOOPS_API;
 	const responsePromises = await Promise.all([
 		fetch(`${hoopsApi}/api/etsu/roster`),
@@ -84,7 +82,6 @@ export const loader = async ({ request }: LoaderFunctionArgs) => {
 	const responseData = (await Promise.all(
 		responsePromises.map(response => response.json()),
 	)) as [RosterApiRes, ScheduleApiRes, StatsApiRes]
-	// console.log('LOG: loaderData', responseData)
 
 	return json({
 		roster: responseData[0],
@@ -93,10 +90,6 @@ export const loader = async ({ request }: LoaderFunctionArgs) => {
 	})
 }
 
-export const action = async ({ request }: ActionFunctionArgs) => {
-	return null
-}
-
 export const heightFromInches = (total: number) => {
 	const feet = Math.floor(total / 12)
 	const inches = total % 12

app/utils/cache.server.ts

@@ -100,7 +100,7 @@ export const cache: CachifiedCache = {
 		return { metadata, value }
 	},
 	async set(key, entry) {
-		// eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+		 
 		const { currentIsPrimary, primaryInstance } = await getInstanceInfo()
 		if (currentIsPrimary) {
 			cacheDb

app/utils/db.server.ts

@@ -31,6 +31,6 @@ export const prisma = remember('prisma', () => {
 		const dur = chalk[color](`${e.duration}ms`)
 		console.info(`prisma:query - ${dur} - ${e.query}`)
 	})
-	client.$connect()
+	void client.$connect()
 	return client
 })

app/utils/monitoring.client.tsx

@@ -1,9 +1,14 @@
 import { useLocation, useMatches } from '@remix-run/react'
-import * as Sentry from '@sentry/remix'
+import {
+	init as sentryInit,
+	browserTracingIntegration,
+	replayIntegration,
+	browserProfilingIntegration,
+} from '@sentry/remix'
 import { useEffect } from 'react'
 
 export function init() {
-	Sentry.init({
+	sentryInit({
 		dsn: ENV.SENTRY_DSN,
 		environment: ENV.MODE,
 		beforeSend(event) {
@@ -20,16 +25,13 @@ export function init() {
 			return event
 		},
 		integrations: [
-			new Sentry.BrowserTracing({
-				routingInstrumentation: Sentry.remixRouterInstrumentation(
-					useEffect,
-					useLocation,
-					useMatches,
-				),
+			browserTracingIntegration({
+				useEffect,
+				useLocation,
+				useMatches,
 			}),
-			// Replay is only available in the client
-			new Sentry.Replay(),
-			new Sentry.BrowserProfilingIntegration(),
+			replayIntegration(),
+			browserProfilingIntegration(),
 		],
 
 		// Set tracesSampleRate to 1.0 to capture 100%

app/utils/providers/constants.ts

@@ -0,0 +1,3 @@
+export const MOCK_CODE_GITHUB = 'MOCK_CODE_GITHUB_KODY'
+
+export const MOCK_CODE_GITHUB_HEADER = 'x-mock-code-github'

app/utils/timing.server.ts

@@ -30,7 +30,7 @@ function createTimer(type: string, desc?: string) {
 			let timingType = timings[type]
 
 			if (!timingType) {
-				// eslint-disable-next-line no-multi-assign
+				 
 				timingType = timings[type] = []
 			}
 			timingType.push({ desc, time: performance.now() - start })

eslint.config.js

@@ -0,0 +1,7 @@
+import { default as defaultConfig } from '@epic-web/config/eslint'
+
+/** @type {import("eslint").Linter.Config} */
+export default [
+	...defaultConfig,
+	// add custom config objects here:
+]