Skip to content

fix: auth robustness and docs consistency#194

Open
bmdavis419 wants to merge 1 commit intocodex/pr-d-config-inheritance-fixfrom
codex/pr-e-auth-robustness-docs
Open

fix: auth robustness and docs consistency#194
bmdavis419 wants to merge 1 commit intocodex/pr-d-config-inheritance-fixfrom
codex/pr-e-auth-robustness-docs

Conversation

@bmdavis419
Copy link
Collaborator

@bmdavis419 bmdavis419 commented Feb 28, 2026

Summary

  • treat empty auth.json as valid empty state
  • reduce noisy auth read warnings
  • align runtime + docs auth matrix for providers

Issues

Greptile Summary

Improves auth robustness by treating empty auth.json as valid empty state and aligns documentation with runtime behavior for Google provider authentication.

  • Added empty file handling in auth file readers to prevent JSON parsing errors
  • Removed Google OAuth support from provider auth matrix (API key only)
  • Updated all documentation to consistently reflect Google requires API key authentication
  • Reduced noisy warnings when reading empty auth files

Confidence Score: 5/5

  • Safe to merge with no breaking changes to existing features
  • All changes are consistent, well-tested patterns for handling edge cases. The Google OAuth removal only affects documentation since OAuth wasn't actually implemented for Google provider. Empty auth.json handling prevents errors without changing existing behavior.
  • No files require special attention

Important Files Changed

Filename Overview
apps/cli/src/lib/copilot-oauth.ts Added empty auth.json handling to prevent JSON parsing errors
apps/cli/src/lib/opencode-oauth.ts Added empty auth.json handling to prevent JSON parsing errors
apps/server/src/providers/auth.ts Added empty auth.json handling and removed Google OAuth support (API key only)

Last reviewed commit: f403623

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