astro-check (warning): Remove unused imports — components & layouts

[lsr-explore]

Remove unused imports — components & layouts

Labels: cleanup, tech-debt, good-first-issue
Estimated PR size: ~5 files

Note

Ticket description was generated using Claude Code. Review before starting on a fix.

Note

The following issue adds support for npm run astro:check

• Add astro check support #561

Summary

A handful of components and layouts have unused imports / parameters / destructured names. Same mechanical cleanup as issue #19, but separated to keep PRs scoped.

astro check warnings

src/components/HeroWithBreadcrumbs.astro:3:1 - warning ts(6133): 'isEmpty' is declared but its value is never read.

src/components/TipRegistration.astro:12:3 - warning ts(6133): 'newsletters' is declared but its value is never read.
src/components/TipRegistration.astro:13:3 - warning ts(6133): 'announcements' is declared but its value is never read.

src/components/forms/GoogleForm.astro:67:14 - warning ts(6133): 'response' is declared but its value is never read.

src/layouts/DAFLayout.astro:19:19 - warning ts(6133): 'title' is declared but its value is never read.

Suggested fix

Delete each unused import / variable. For TipRegistration — the newsletters and announcements props look like they were once meant to drive form behavior. Verify with git blame whether they were stripped during a refactor and should be re-wired, or whether they're genuinely orphaned.

For the (response) => callback in GoogleForm.astro — prefix with underscore ((_response) =>) since it's a callback signature.

Acceptance criteria

• All 5 warnings clear.
• npm run build from astro/ still succeeds.
• Forms (registration, donate flow) still submit correctly.

Notes

If TipRegistration's newsletters / announcements were lost during a refactor and should be wired up, file that as a separate bug — don't hide the gap by deleting them.