Refactor: Improve type safety and test coverage

Co-authored-by: erik <erik@veefriends.com>

Author: cursoragent

src/core/Gracket.test.ts

@@ -464,7 +464,7 @@ describe('Gracket', () => {
 
       it('should trigger onScoreUpdate callback', () => {
         let callbackCalled = false;
-        let callbackData: any = null;
+        let callbackData: { r: number; g: number; t: number; score: number } | null = null;
 
         const gracketWithCallback = new Gracket(container, {
           src: JSON.parse(JSON.stringify(mutableData)),
@@ -765,7 +765,7 @@ describe('Gracket', () => {
     });
 
     it('should pass correct parameters to onScoreUpdate', () => {
-      const capturedParams: any[] = [];
+      const capturedParams: Array<{ r: number; g: number; t: number; score: number }> = [];
 
       const data: TournamentData = [
         [

src/integration.test.ts

@@ -213,22 +213,7 @@ describe('Integration Tests - Complete Workflows', () => {
       const tournamentData = generateTournamentWithByes(teams, 'top-seeds');
       const gracket = new Gracket(container, { src: tournamentData });
 
-      let roundsCompleted = 0;
-
-      // Monitor round completion
-      const onRoundComplete = (r: number) => {
-        roundsCompleted++;
-        
-        // Auto-advance when round completes
-        if (gracket.isRoundComplete(r)) {
-          gracket.advanceRound(r, {
-            tieBreaker: 'higher-seed',
-            createRounds: true
-          });
-        }
-      };
-
-      // Manually attach callback logic
+      // Get initial data
       const data = gracket.getData();
 
       // Score round 1

src/utils/byes.test.ts

@@ -423,8 +423,8 @@ describe('byes utilities', () => {
       const tournament = generateTournamentWithByes(teams, 'top-seeds');
 
       // Each round should have placeholder teams with IDs
-      tournament.forEach((round, rIdx) => {
-        round.forEach((game, gIdx) => {
+      tournament.forEach((round) => {
+        round.forEach((game) => {
           game.forEach(team => {
             expect(team.name).toBeDefined();
             expect(team.seed).toBeDefined();

src/utils/tournament.test.ts

@@ -234,7 +234,7 @@ describe('tournament utilities', () => {
 
     it('should throw error for unknown strategy', () => {
       expect(() => {
-        applyTieBreaker(team1, team2, 'invalid' as any);
+        applyTieBreaker(team1, team2, 'invalid' as 'higher-seed' | 'lower-seed' | 'callback');
       }).toThrow('Unknown tie-breaker strategy');
     });