hensei-web/middleware.ts
Justin Edmund e4b7f0c356
Fix TypeScript errors for production build (#436)
## Summary
Fixed multiple TypeScript errors that were preventing the production
build from completing on Railway.

## Changes Made

### Nullable Type Fixes
- Fixed `searchParams.toString()` calls with optional chaining (`?.`)
and fallback values
- Fixed `pathname` nullable access in UpdateToastClient
- Added fallbacks for undefined values in translation interpolations

### Type Consistency Fixes
- Fixed recency parameter handling (string from URL, converted to number
internally)
- Removed duplicate local interface definitions for Party and User types
- Fixed Party type mismatches by using global type definitions

### API Route Error Handling
- Fixed error type checking in catch blocks for login/signup routes
- Added proper type guards for axios error objects

### Component Props Fixes
- Fixed RadixSelect.Trigger by removing invalid placeholder prop
- Fixed Toast and Tooltip components by using Omit to exclude
conflicting content type
- Added missing onAdvancedFilter prop to FilterBar components
- Fixed PartyFooter props with required parameters

## Test Plan
- [x] Fixed all TypeScript compilation errors locally
- [ ] Production build should complete successfully on Railway
- [ ] All affected components should function correctly

🤖 Generated with [Claude Code](https://claude.ai/code)

---------

Co-authored-by: Claude <noreply@anthropic.com>
2025-09-04 01:47:10 -07:00

73 lines
No EOL
2.3 KiB
TypeScript

import createMiddleware from 'next-intl/middleware'
import {routing} from './i18n/routing'
import {locales, type Locale} from './i18n.config'
import {NextResponse} from 'next/server'
import type {NextRequest} from 'next/server'
const intl = createMiddleware(routing)
const PROTECTED_PATHS = ['/saved', '/profile'] as const
const MIXED_AUTH_PATHS = ['/api/parties', '/p/'] as const
export default function middleware(request: NextRequest) {
const {pathname} = request.nextUrl
// Skip intl middleware for API routes
if (!pathname.startsWith('/api/')) {
// Run next-intl for non-API routes (handles locale detection, redirects, etc.)
const intlResponse = intl(request)
if (intlResponse) return intlResponse
}
const seg = pathname.split('/')[1]
const pathWithoutLocale = locales.includes(seg as Locale)
? pathname.slice(seg.length + 1) || '/'
: pathname
const isProtectedPath = PROTECTED_PATHS.some(
(p) => pathWithoutLocale === p || pathWithoutLocale.startsWith(p + '/')
)
const isMixedAuthPath = MIXED_AUTH_PATHS.some(
(p) => pathWithoutLocale === p || pathWithoutLocale.startsWith(p)
)
const needsAuth =
isProtectedPath || (isMixedAuthPath && ['POST', 'PUT', 'DELETE'].includes(request.method))
if (!needsAuth) return NextResponse.next()
const accountCookie = request.cookies.get('account')
if (!accountCookie?.value) {
if (pathWithoutLocale.startsWith('/api/')) {
return NextResponse.json({error: 'Authentication required'}, {status: 401})
}
// Preserve locale in redirect
const url = request.nextUrl.clone()
url.pathname = '/teams'
return NextResponse.redirect(url)
}
try {
const account = JSON.parse(accountCookie.value)
if (!account.token) {
if (pathWithoutLocale.startsWith('/api/')) {
return NextResponse.json({error: 'Authentication required'}, {status: 401})
}
const url = request.nextUrl.clone()
url.pathname = '/teams'
return NextResponse.redirect(url)
}
} catch {
if (pathWithoutLocale.startsWith('/api/')) {
return NextResponse.json({error: 'Authentication required'}, {status: 401})
}
const url = request.nextUrl.clone()
url.pathname = '/teams'
return NextResponse.redirect(url)
}
return NextResponse.next()
}
export const config = {
matcher: ['/((?!_next|_vercel|.*\\..*).*)']
}