Fix review feedback: remove unused imports and redundant test conditions

[Copilot]

Addresses code quality issues identified in PR #5 review.

Changes

• worker/seed.sql: Updated test team URL from legacy format to match isValidELTTLUrl validation pattern (elttl.interactive.co.uk/teams/view/{id})
• worker/src/database.ts: Removed unused getAvailabilityKey import
• worker/src/database.integration.test.ts: Removed unused afterEach import
• worker/src/validation.test.ts:
  • Removed unused gamesPlayed variable
  • Eliminated redundant conditional checks where totalGames > 0 was always true/false
  • Made test intent explicit by using literal values instead of derived assignments

// Before: always-true condition
const totalGames = gamesPlayed + gamesScheduled; // 8
const selectionRate = totalGames > 0 
  ? Math.round((gamesScheduled / totalGames) * 100) 
  : 0;

// After: simplified
const totalGames = gamesPlayed + gamesScheduled; // 8
const selectionRate = Math.round((gamesScheduled / totalGames) * 100);

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Preview URL	Updated (UTC)
✅ Deployment successful! View logs	tabletennis	f022e34	Commit Preview URL Branch Preview URL	Dec 31 2025, 12:45 AM

[cloudflare-workers-and-pages]

Deploying tabletennis with    Cloudflare Pages

Latest commit:	f022e34
Status:	✅  Deploy successful!
Preview URL:	https://f86187e8.tabletennis.pages.dev
Branch Preview URL:	https://copilot-sub-pr-5.tabletennis.pages.dev

View logs