chore: drop the stale NEXT_PUBLIC_GOOGLE_MAPS_API_KEY docs - #589
Merged
Conversation
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
NEXT_PUBLIC_GOOGLE_MAPS_API_KEYis documented in two places and read by none.The only references are
SETUP.md:26and.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.exampleno longer lists the commented Google Maps key block.SETUP.mddrops the variable from the optional env table so setup docs match what the editor actually uses (PORTremains 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.