feat(auth): add places api key management#140
Conversation
2f64a6b to
f1d9f49
Compare
- Use static error variable instead of dynamic errors.New() - Use scopeAll constant instead of "all" string literal - Fix variable shadowing in test files and handlers - Add required blank lines per wsl_v5 rules - Remove unused nolint directive Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
Thanks @salmonumbrella. I reviewed this against the current Places API (New) docs and landed a smaller replacement on main in f550ae5, with final CI/lint cleanup through 57080af. The original branch was stale/conflicting, so I did not merge it directly. The landed version adds direct Places Text Search / Place Details support without a new dependency, plus Live proof: resolved “Elysian Coffee Vancouver” with Places API and dry-ran a Calendar create showing the canonical location plus private place metadata. Docs/README/generated command docs/changelog updated. CI is green on main: 25601394551. Closing this PR as replaced by the landed implementation. |
Summary
gog auth places-keycommands (set/status/clear) for managing Google Places API key--location-searchand--place-idflagsgog auth manageweb UICloses #138
Test Plan
internal/places/apikey.go(MaskAPIKey, empty key validation)auth places-keycommands🤖 Generated with Claude Code