chore: prune non-critical comments across src

[InnerDesires]

Comment lines in src/ drop from 785 to 333 (-58%) across 112 files. The bar for keeping a comment: a reader would be confused, or would likely break something, without it.

Removed

• JSX section markers — {/* Top bar */}, {/* stats */}, {/* certificates */}, {/* avatar with XP level ring */}
• Step/narration comments that restate the next statement — // Step 1: Email entry, // Find the verification record, // Increment attempt counter
• JSDoc that only repeats the symbol name — /** Total XP a course awards. */, and the /** Locale-aware base path ... */ prop doc duplicated across four components
• Payload/Next template boilerplate — the 27-line ImageMedia loader essay, @param blocks on deepMerge/getMediaUrl/formatAuthors, defaultPopulate notes, the shadcn preamble in ui.ts, and a stale note in Media.ts claiming uploads land in public/media (they go to Vercel Blob)
• ~10 near-identical "session-free so it stays in the ISR cache" headers — kept one authoritative copy on getCatalogData.ts plus the two documenting a specific trap (generateStaticParams in [locale]/page.tsx, useSearchParams in CourseCatalog.tsx)
• Commented-out code in VideoMedia — see below

Kept

Security invariants (collection access rationale, the role-escalation warning on Users, FK delete ordering), framework workarounds (Payload's empty-array minRows loophole in Courses.ts, the cyrillicSlugify NULL-vs-'' unique-constraint fix, better-auth's #fragment rejection, plugin-search's per-locale writes), magic-number provenance (every measured coordinate in the certificate pdf.tsx, rate-limit thresholds), CSS tricks (StepsList minmax(0,1fr), the -50% ticker keyframe), and all functional directives (eslint-disable, @ts-expect-error, @ts-nocheck, empty-catch markers).

About 20 surviving multi-line headers were shortened to their load-bearing sentence rather than deleted outright — rate-limit.ts, ukrainianAdmin.ts, revalidateCourse.ts, xp.ts, contentCounts.ts and others.

Reviewer note: one non-comment change

Removing the commented-out code in src/components/Media/VideoMedia/index.tsx left a useEffect registering an empty 'suspend' listener, so the dead effect and its now-unused useEffect import went too. Provably inert — the listener body was entirely commented out — but it is the only change in the diff that is not purely a comment.

Verification

• tsc --noEmit — clean
• eslint on all 112 changed files — clean
• Prettier — the repo already had 40 files failing --check at HEAD; the failing set is unchanged after these edits
• Integration tests were not run: this worktree has no .env, so there is no Neon branch to point at. For a comments-only diff the typecheck is the meaningful signal.

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
learning-platform	Ignored		Aug 6, 2026 3:53pm