Skip to content

feat(auth): show a tighter default set of columns in auth status and auth login - #362

Open
jpage-godaddy wants to merge 1 commit into
mainfrom
custom-auth-view
Open

jpage-godaddy wants to merge 1 commit into
mainfrom
custom-auth-view

Conversation

@jpage-godaddy

Copy link
Copy Markdown
Collaborator

Summary

  • gddy auth status no longer shows the granted scopes by default. Sessions now get the full scope set, so the column was always truncated in human output.
  • gddy auth login still shows the scopes it just granted.
  • The provider column is dropped from both commands, since there is only one provider.
  • Nothing is removed: --fields provider,scopes brings both columns back, and gddy auth scopes lists the full scope set.

Default columns are now:

Command Columns
auth status identity, expires_at, env, expired, refreshable
auth login identity, expires_at, env, refreshable, scopes

Also in this PR

  • Bumps cli-engine to 0.10.4, which provides the default-field settings used above.

Testing

  • cargo check --workspace, cargo clippy --workspace -- -D warnings, cargo fmt --check, ./rust/scripts/check-module-size.sh and cargo test --workspace all pass.

🤖 Generated with Claude Code

…auth login

`auth status` no longer lists the granted scopes by default. Now that
sessions are granted the full scope set, the column is always truncated
in human output. `auth login` still shows the scopes it just granted.

The provider column is also dropped from both commands, since there is
only one provider to choose from.

Everything stays available with `--fields provider,scopes`, and the full
scope list is still shown by `auth scopes`.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues are recorded for the auth output defaults and dependency update.

No open issues 🎉

What changed in this PR

This PR simplifies the default auth output by configuring command-specific columns through cli-engine 0.10.4.

Changes:

  • Hides provider from both auth status and auth login, and scopes from auth status only.
  • Keeps hidden fields available through --fields, while login continues to display granted scopes.
  • Updates the cli-engine dependency and lockfile to 0.10.4.
File Description
rust/​src/​main.rs Configures default fields for auth status and login.
rust/​Cargo.toml Bumps cli-engine to 0.10.4 for default-field support.
rust/​Cargo.lock Locks the updated dependency version and checksum.

Review effort: Balanced

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