chore(cleanup): delete dead code (commented-out blocks + orphaned exports) (#193)

Two known scraps + four exports that ts-prune flagged as unreferenced
(verified by hand). Nothing here changes runtime behavior.

Commented-out scaffolding (from the original cleanup plan):
- src/app/layout.tsx — Geist font imports + const declarations sat
  commented with a "temporarily disabled for build" note. Even fully
  uncommented they wouldn't have re-enabled the font (the body className
  uses `font-sans antialiased`, not the geist variables). Re-adding
  Geist later is a fresh task; the commented placeholder was just rot.
- src/app/stations/[id]/edit/page.tsx — `// const [testingLotw,
  setTestingLotw] = useState(false); // Unused - for future LoTW
  validation`. The "future" comment had no path; if it ever lands,
  add the state then.

Orphaned exports (zero references in src/, verified with git grep):
- src/lib/qrz.ts `validateQRZCredentials` — explicitly marked "(legacy)"
  in its own comment; superseded by validateQRZApiKey. Delete the legacy.
- src/components/LotwSyncIndicator.tsx `LotwStatusBadge` — sibling
  helper appended after the main component, never imported anywhere.
- src/components/QRZSyncIndicator.tsx `QRZStatusBadge` — same pattern.
- src/components/LotwSettings.tsx — entire file unused (no import sites).

Removing the two badge helpers freed up the `Badge` import in both
LotwSyncIndicator.tsx and QRZSyncIndicator.tsx; dropped those too.

Intentionally kept (ts-prune flagged, but architecturally meaningful):
- Auth helpers: requireAuth, requireAdmin, withApiAuth
- Permission helpers: isAdmin, isModerator, getRolePermissions
- API utilities: successWithRateLimit, errorResponse, corsJsonResponse
- Settings CRUD: getSetting, getSettingsByCategory, createSetting,
  deleteSetting, SettingValue
- Type exports across types/awards.ts and models/

These are utility-belt/scaffolding code with clear architectural intent
(the auth and settings systems are partly wired). Deleting them removes
affordances; if they stay unused indefinitely they can be revisited.

Verification: lint baseline drops 52→51 (one LotwSettings React hooks
warning is gone), typecheck clean, build succeeds.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: patrickrb

src/app/layout.tsx

@@ -1,22 +1,10 @@
 import type { Metadata } from "next";
-// Font loading temporarily disabled for build in restricted environment
-// import { Geist, Geist_Mono } from "next/font/google";
 import "./globals.css";
 import { UserProvider } from "@/contexts/UserContext";
 import { ThemeProvider } from "@/contexts/ThemeContext";
 import Footer from "@/components/Footer";
 import InstallationChecker from "@/components/InstallationChecker";
 
-// const geistSans = Geist({
-//   variable: "--font-geist-sans",
-//   subsets: ["latin"],
-// });
-
-// const geistMono = Geist_Mono({
-//   variable: "--font-geist-mono",
-//   subsets: ["latin"],
-// });
-
 export const metadata: Metadata = {
   title: "Nextlog - Amateur Radio Logging",
   description: "Modern amateur radio contact logging and station management",

src/app/stations/[id]/edit/page.tsx

@@ -132,7 +132,6 @@ export default function EditStationPage({ params }: { params: Promise<{ id: stri
   const [validatingQrz, setValidatingQrz] = useState(false);
   const [qrzValidation, setQrzValidation] = useState<{ valid: boolean; message: string } | null>(null);
   const [showLotwPassword, setShowLotwPassword] = useState(false);
-  // const [testingLotw, setTestingLotw] = useState(false); // Unused - for future LoTW validation
   const [certFile, setCertFile] = useState<File | null>(null);
   const [certName, setCertName] = useState('');
   const [certPassword, setCertPassword] = useState('');