Add ESLint with CI enforcement

[ambv]

We'll be adding more bona fide tests to the three components of the system, but let's at least start with sensible linting. Frontend first.

[Copilot]

Pull request overview

Adds frontend ESLint + CI enforcement and updates existing frontend code to comply with lint/typecheck rules.

Changes:

• Add ESLint dependencies to the frontend and enforce lint + typecheck in GitHub Actions CI.
• Clean up frontend code to satisfy linting (unused imports/vars, prefer-const, JSX apostrophe escaping, etc.).
• Minor type/style refactors (e.g., type aliases, additional UI exports).

Reviewed changes

Copilot reviewed 27 out of 28 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
frontend/src/types/components.ts	Removes unused type imports to satisfy linting.
frontend/src/types/api.ts	Changes TrendQueryParams to a type alias for backward compatibility.
frontend/src/lib/api.ts	Removes unused imported API types.
frontend/src/hooks/use-toast.ts	Refactors action type definitions to avoid unused constants.
frontend/src/components/ui/textarea.tsx	Simplifies props type to a type alias.
frontend/src/components/ui/dialog.tsx	Exports DialogClose from the dialog UI module.
frontend/src/components/ui/collapsible.tsx	Removes unused React import.
frontend/src/components/diff/CommitTooltipContent.tsx	Removes unused UI imports.
frontend/src/app/version-comparison/page.tsx	Removes unused imports/variables; escapes apostrophes for JSX.
frontend/src/app/trends/page.tsx	Applies prefer-const in key parsing logic.
frontend/src/app/page.tsx	Removes unused index parameter in .map.
frontend/src/app/diff/page.tsx	Removes unused imports/helpers; minor whitespace cleanup.
frontend/src/app/build-comparison/page.tsx	Lint-driven refactors (consts, escaping); adjusts memoization usage.
frontend/src/app/binaries/[id]/page.tsx	Prefixes unused prop with _ to satisfy no-unused-vars rule.
frontend/src/app/admin/page.tsx	Removes unused constants/imports; simplifies catch blocks.
frontend/src/app/admin/components/TokensManager.tsx	Simplifies catch blocks; escapes apostrophes in UI text.
frontend/src/app/admin/components/QueryConsole.tsx	Reworks dynamic imports; removes unused imports; formatting cleanup.
frontend/src/app/admin/components/MemrayFailuresManager.tsx	Removes unused API_BASE.
frontend/src/app/admin/components/EnvironmentsManager.tsx	Simplifies catch blocks; formatting cleanup.
frontend/src/app/admin/components/BinariesManager.tsx	Removes unused imports; simplifies catch blocks.
frontend/src/app/admin/components/BenchmarkResultsManager.tsx	Removes unused icon imports; simplifies catch blocks; formatting cleanup.
frontend/src/app/admin/components/AdminUsersManager.tsx	Removes unused imports/types; formatting cleanup.
frontend/src/app/about/page.tsx	Simplifies catch blocks; escapes apostrophes; removes unused index.
frontend/package.json	Adds eslint and eslint-config-next devDependencies.
docker-README.md	Documents linting and typecheck commands via Docker.
README.md	Updates testing instructions and notes CI enforcement.
.github/workflows/ci.yml	Replaces dummy CI with frontend lint + typecheck job.

Comments suppressed due to low confidence (1)

frontend/src/app/build-comparison/page.tsx:316

• This useMemo computes and returns an array but the return value is discarded (not assigned/used). This still runs the expensive aggregation/sort work when deps change, but has no effect on rendering. Either remove this hook entirely, or assign it to a variable and use it (or switch to useEffect if side effects are intended).

 useMemo(() => {
// For "All Benchmarks" mode, don't require selectedBenchmarks
const requiresBenchmarkSelection =
benchmarkMode === 'specific' && selectedBenchmarks.length === 0;

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.