v0.2.0-alpha Add Live stat Tracking

.gitignore

@@ -24,4 +24,5 @@ dist-ssr
 *.sw?
 .env
 *.tsbuildinfo
-coverage/.tmp
+coverage/*
+!coverage/.tmp

coverage/.tmp

@@ -0,0 +1,2 @@
+# This file keeps the coverage folder in git while ignoring all coverage reports
+# Coverage reports are generated by running: npm run test:coverage

database/README.md

@@ -0,0 +1,46 @@
+# Database Scripts
+
+This folder contains SQL scripts and database-related files for the Derby Stat Tracker application.
+
+## Files
+
+### `supabase-rls-performance-fixes.sql`
+
+Comprehensive SQL script to fix Supabase performance issues identified by the database linter.
+
+**Includes:**
+- **RLS Performance Fixes** - Optimizes Row Level Security policies by wrapping `auth.uid()` calls in subqueries
+- **Foreign Key Indexes** - Adds missing indexes on foreign key columns for better query performance
+
+**How to use:**
+1. Open your Supabase Dashboard
+2. Navigate to the SQL Editor
+3. Copy and paste the contents of this file
+4. Execute the SQL commands
+
+**Performance Benefits:**
+- Eliminates RLS auth function re-evaluation per row (22 warnings fixed)
+- Improves join performance with proper foreign key indexing (4 warnings fixed)
+- Faster queries for live stat tracking operations
+- Better performance when loading team rosters and bout data
+
+**Tables Optimized:**
+- `teams` - RLS policies and foreign key references
+- `players` - RLS policies  
+- `player_teams` - RLS policies and team_id indexing
+- `bouts` - RLS policies and team foreign key indexing
+- `player_stats` - RLS policies and bout_id indexing
+
+## Future Database Changes
+
+When making database schema changes:
+1. Add new SQL scripts to this folder
+2. Use descriptive filenames with dates when relevant
+3. Document the purpose and usage in this README
+4. Test scripts in development before applying to production
+
+## Development Notes
+
+- These scripts are safe to run multiple times (uses `IF EXISTS` and `IF NOT EXISTS`)
+- All changes maintain existing security policies while improving performance
+- Foreign key indexes use standard naming convention: `idx_{table}_{column}`

database/supabase-rls-performance-fixes.sql

@@ -0,0 +1,133 @@
+-- Supabase RLS Performance Fixes
+-- Run these commands in your Supabase SQL editor to fix the auth RLS initialization plan warnings
+
+-- Fix teams table RLS policies
+DROP POLICY IF EXISTS "Allow authenticated users to view teams" ON public.teams;
+DROP POLICY IF EXISTS "Allow authenticated users to insert teams" ON public.teams;
+DROP POLICY IF EXISTS "Allow authenticated users to update teams" ON public.teams;
+DROP POLICY IF EXISTS "Allow authenticated users to delete teams" ON public.teams;
+
+CREATE POLICY "Allow authenticated users to view teams" ON public.teams
+    FOR SELECT USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to insert teams" ON public.teams
+    FOR INSERT WITH CHECK ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to update teams" ON public.teams
+    FOR UPDATE USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to delete teams" ON public.teams
+    FOR DELETE USING ((SELECT auth.uid()) IS NOT NULL);
+
+-- Fix players table RLS policies
+DROP POLICY IF EXISTS "Allow authenticated users to view players" ON public.players;
+DROP POLICY IF EXISTS "Allow authenticated users to insert players" ON public.players;
+DROP POLICY IF EXISTS "Allow authenticated users to update players" ON public.players;
+DROP POLICY IF EXISTS "Allow authenticated users to delete players" ON public.players;
+
+CREATE POLICY "Allow authenticated users to view players" ON public.players
+    FOR SELECT USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to insert players" ON public.players
+    FOR INSERT WITH CHECK ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to update players" ON public.players
+    FOR UPDATE USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to delete players" ON public.players
+    FOR DELETE USING ((SELECT auth.uid()) IS NOT NULL);
+
+-- Fix player_teams table RLS policies
+DROP POLICY IF EXISTS "Allow authenticated users to view player_teams" ON public.player_teams;
+DROP POLICY IF EXISTS "Allow authenticated users to insert player_teams" ON public.player_teams;
+DROP POLICY IF EXISTS "Allow authenticated users to update player_teams" ON public.player_teams;
+DROP POLICY IF EXISTS "Allow authenticated users to delete player_teams" ON public.player_teams;
+
+CREATE POLICY "Allow authenticated users to view player_teams" ON public.player_teams
+    FOR SELECT USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to insert player_teams" ON public.player_teams
+    FOR INSERT WITH CHECK ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to update player_teams" ON public.player_teams
+    FOR UPDATE USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to delete player_teams" ON public.player_teams
+    FOR DELETE USING ((SELECT auth.uid()) IS NOT NULL);
+
+-- Fix bouts table RLS policies
+DROP POLICY IF EXISTS "Allow authenticated users to view bouts" ON public.bouts;
+DROP POLICY IF EXISTS "Allow authenticated users to insert bouts" ON public.bouts;
+DROP POLICY IF EXISTS "Allow authenticated users to update bouts" ON public.bouts;
+DROP POLICY IF EXISTS "Allow authenticated users to delete bouts" ON public.bouts;
+
+CREATE POLICY "Allow authenticated users to view bouts" ON public.bouts
+    FOR SELECT USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to insert bouts" ON public.bouts
+    FOR INSERT WITH CHECK ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to update bouts" ON public.bouts
+    FOR UPDATE USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to delete bouts" ON public.bouts
+    FOR DELETE USING ((SELECT auth.uid()) IS NOT NULL);
+
+-- Fix player_stats table RLS policies
+DROP POLICY IF EXISTS "Allow authenticated users to view player_stats" ON public.player_stats;
+DROP POLICY IF EXISTS "Allow authenticated users to insert player_stats" ON public.player_stats;
+DROP POLICY IF EXISTS "Allow authenticated users to update player_stats" ON public.player_stats;
+DROP POLICY IF EXISTS "Allow authenticated users to delete player_stats" ON public.player_stats;
+
+CREATE POLICY "Allow authenticated users to view player_stats" ON public.player_stats
+    FOR SELECT USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to insert player_stats" ON public.player_stats
+    FOR INSERT WITH CHECK ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to update player_stats" ON public.player_stats
+    FOR UPDATE USING ((SELECT auth.uid()) IS NOT NULL);
+
+CREATE POLICY "Allow authenticated users to delete player_stats" ON public.player_stats
+    FOR DELETE USING ((SELECT auth.uid()) IS NOT NULL);
+
+-- Note: These fixes wrap auth.uid() in (SELECT auth.uid()) subqueries
+-- This ensures the authentication check is evaluated once per query instead of once per row
+-- This dramatically improves performance for queries that return multiple rows
+
+-- =============================================================================
+-- FOREIGN KEY INDEX PERFORMANCE FIXES
+-- =============================================================================
+-- These indexes improve performance for foreign key lookups and joins
+
+-- Index for bouts.home_team_id foreign key
+-- Improves performance when querying bouts by home team
+CREATE INDEX IF NOT EXISTS idx_bouts_home_team_id ON public.bouts(home_team_id);
+
+-- Index for bouts.away_team_id foreign key  
+-- Improves performance when querying bouts by away team
+CREATE INDEX IF NOT EXISTS idx_bouts_away_team_id ON public.bouts(away_team_id);
+
+-- Index for player_stats.bout_id foreign key
+-- Improves performance when querying player stats by bout
+-- This is particularly important for live stat tracking
+CREATE INDEX IF NOT EXISTS idx_player_stats_bout_id ON public.player_stats(bout_id);
+
+-- Index for player_teams.team_id foreign key
+-- Improves performance when querying player-team relationships by team
+-- This is important for loading team rosters
+CREATE INDEX IF NOT EXISTS idx_player_teams_team_id ON public.player_teams(team_id);
+
+-- Composite index for player_stats (bout_id, player_id)
+-- Optimizes the common query pattern of getting specific player stats for a bout
+CREATE INDEX IF NOT EXISTS idx_player_stats_bout_player ON public.player_stats(bout_id, player_id);
+
+-- Composite index for player_teams (team_id, is_active)
+-- Optimizes loading active players for a team (common in live tracking)
+CREATE INDEX IF NOT EXISTS idx_player_teams_team_active ON public.player_teams(team_id, is_active);
+
+-- Note: These indexes will significantly improve query performance for:
+-- 1. Loading bout details with team information
+-- 2. Fetching player stats for a specific bout (live tracking)
+-- 3. Getting team rosters and active players
+-- 4. Joining tables on foreign key relationships

src/App.tsx

@@ -6,21 +6,31 @@ import Dashboard from './components/Dashboard'
 import Teams from './components/Teams'
 import Players from './components/Players'
 import Bouts from './components/Bouts'
+import LiveStatTracker from './components/LiveStatTracker'
 import { Auth } from './components/Auth'
 import { ConfigurationError } from './components/ConfigurationError'
 import { useAuth } from './hooks/useAuth'
 import { isSupabaseConfigured } from './lib/supabase'
 import { Analytics } from "@vercel/analytics/react"
 import { SpeedInsights } from '@vercel/speed-insights/react'
-
-
-type ActiveView = 'dashboard' | 'players' | 'bouts' | 'teams' | 'settings'
+import { ActiveView } from './types'
 
 function App() {
   const [activeView, setActiveView] = useState<ActiveView>('dashboard')
+  const [selectedBoutId, setSelectedBoutId] = useState<string | null>(null)
 
   // Always call hooks first (React hooks rules)
   const { user, loading } = useAuth()
+
+  const handleStartLiveTracking = (boutId: string) => {
+    setSelectedBoutId(boutId)
+    setActiveView('live-track')
+  }
+
+  const handleNavigateBackToBouts = () => {
+    setSelectedBoutId(null)
+    setActiveView('bouts')
+  }
 
   // Check for configuration errors early - if Supabase is not configured, show error
   if (!isSupabaseConfigured) {
@@ -49,8 +59,14 @@ function App() {
         <main className="main-content">
           {activeView === 'dashboard' && <Dashboard />}
           {activeView === 'players' && <Players />}
-          {activeView === 'bouts' && <Bouts />}
+          {activeView === 'bouts' && <Bouts onStartLiveTracking={handleStartLiveTracking} />}
           {activeView === 'teams' && <Teams />}
+          {activeView === 'live-track' && (
+            <LiveStatTracker 
+              boutId={selectedBoutId} 
+              onNavigateBack={handleNavigateBackToBouts}
+            />
+          )}
           {activeView === 'settings' && <div className="view-placeholder">Settings - Coming Soon</div>}
           <Analytics />
           <SpeedInsights />