Skip to content

chore: drop the stale NEXT_PUBLIC_GOOGLE_MAPS_API_KEY docs - #589

Merged
Aymericr merged 1 commit into
mainfrom
chore/drop-stale-maps-var
Aug 4, 2026
Merged

chore: drop the stale NEXT_PUBLIC_GOOGLE_MAPS_API_KEY docs#589
Aymericr merged 1 commit into
mainfrom
chore/drop-stale-maps-var

Conversation

@Aymericr

@Aymericr Aymericr commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

NEXT_PUBLIC_GOOGLE_MAPS_API_KEY is documented in two places and read by none.

$ grep -rn "GOOGLE_MAPS" apps packages --include='*.ts' --include='*.tsx'
(no matches)

The only references are SETUP.md:26 and .env.example:4, and there's no geocoder anywhere in the tree — no Nominatim, Mapbox, Geoapify or Places call — so the "address search" the variable promises doesn't exist in this app. Setting it does nothing.

Docs that advertise a non-existent knob cost real time: someone provisions a Google Cloud key, sets it, sees no address search, and goes looking for the bug in their setup.

Noticed while reviewing #268, where I said I'd clean it up separately.

Docs only — no code changes.

🤖 Generated with Claude Code


Note

Low Risk
Documentation-only change with no runtime, config, or security impact.

Overview
Removes documentation only for NEXT_PUBLIC_GOOGLE_MAPS_API_KEY, which is not read anywhere in the app and does not enable address search.

.env.example no longer lists the commented Google Maps key block. SETUP.md drops the variable from the optional env table so setup docs match what the editor actually uses (PORT remains the only documented optional var).

Reviewed by Cursor Bugbot for commit ec501fc. Bugbot is set up for automated code reviews on this repo. Configure here.

Documented in SETUP.md and .env.example, referenced by zero lines of
apps/ or packages/ — there is no geocoder in this app, so the address
search it promises does not exist. Noticed while reviewing #268.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@Aymericr
Aymericr merged commit 7f3caca into main Aug 4, 2026
2 checks passed
@Aymericr
Aymericr deleted the chore/drop-stale-maps-var branch August 4, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant