fix(test): remove unused beforeEach imports to satisfy ESLint no-unused-vars

The games and discord controller test files imported beforeEach from
vitest but never called it. TypeScript-ESLint's no-unused-vars rule
treats unused named imports as errors in this project's flat config.

https://claude.ai/code/session_013coUXcACPpVfmbK9i2D86Z

Author: claude

app/packages/server/src/controllers/discord.controller.test.ts

@@ -1,5 +1,5 @@
 import 'reflect-metadata';
-import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import { BadRequestException } from '@nestjs/common';
 import { DiscordController } from './discord.controller.js';
 import type { DiscordConfigService, DiscordAction } from '../services/DiscordConfigService.js';

app/packages/server/src/controllers/games.controller.test.ts

@@ -1,5 +1,5 @@
 import 'reflect-metadata';
-import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import { GamesController } from './games.controller.js';
 import type { ConfigService, TfOutputs } from '../services/ConfigService.js';
 import type { EcsService } from '../services/EcsService.js';