Skip to content

Clarify login status stored credential wording#372

Closed
GreyforgeLabs wants to merge 1 commit into
CortexLM:mainfrom
GreyforgeLabs:fix-login-status-api-credential
Closed

Clarify login status stored credential wording#372
GreyforgeLabs wants to merge 1 commit into
CortexLM:mainfrom
GreyforgeLabs:fix-login-status-api-credential

Conversation

@GreyforgeLabs
Copy link
Copy Markdown

@GreyforgeLabs GreyforgeLabs commented May 20, 2026

Summary

  • change stored AuthMode::ApiKey login status wording from "API key" to neutral "stored API credential"
  • avoids misleading output after cortex login --token, which shares the same stored auth shape as --with-api-key

Fixes PlatformNetwork/bounty-challenge#53145

Tests

  • cargo fmt --package cortex-cli --check
  • git diff --check
  • cargo check -p cortex-cli --no-default-features --features cortex-tui

Note: package unit tests are currently blocked on main by an unrelated stale private import in src/cortex-cli/src/agent_cmd/tests.rs, so I kept validation to the compiling no-default-features profile used for this change.

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

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.

[CLI][v0.0.7] cortex login status says “API key” after cortex login --token, not “token”

1 participant