Skip to content

feat: add auth status command and improve login output#25

Merged
SachaMorard merged 2 commits intomainfrom
feat/auth-status
Mar 20, 2026
Merged

feat: add auth status command and improve login output#25
SachaMorard merged 2 commits intomainfrom
feat/auth-status

Conversation

@SachaMorard
Copy link
Copy Markdown
Member

Summary

  • Adds edgee auth status command: shows logged-in email, config file path, and Claude Code mode (plan/api); gracefully degrades if email is unavailable
  • Stores email and user_id in credentials at login time (extracted from callback URL params, percent-decoded) — no separate API call needed
  • Login confirmation now prints "Logged in as <email>" instead of the masked API key
  • Adds reqwest as a direct dependency (was previously only indirect via self_update)

Test plan

  • edgee auth login — completes, prints Logged in as <email>
  • edgee auth status — shows email, config path, and Claude Code mode
  • edgee auth status when not logged in — shows ✗ Not logged in prompt
  • edgee auth --help — lists login and status subcommands
  • Credentials file contains email and user_id after login

🤖 Generated with Claude Code

- Add `edgee auth status` command: shows logged-in email, config path,
  and Claude Code mode (plan/api); gracefully degrades if email is unavailable
- Store email and user_id in credentials at login time (passed via callback
  URL params, percent-decoded) instead of making a separate API call
- Show "Logged in as <email>" after successful login
- Add reqwest as a direct dependency for future authenticated API calls

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@SachaMorard SachaMorard requested a review from a team as a code owner March 20, 2026 09:25
@SachaMorard SachaMorard merged commit abeddf6 into main Mar 20, 2026
3 checks passed
@SachaMorard SachaMorard deleted the feat/auth-status branch March 20, 2026 09:37
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.

2 participants