Skip to content

Improve CLI logout account prompt#1607

Merged
ChiragAgg5k merged 1 commit into
mainfrom
fix/cli-logout-prompt
Jun 23, 2026
Merged

Improve CLI logout account prompt#1607
ChiragAgg5k merged 1 commit into
mainfrom
fix/cli-logout-prompt

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

Summary

  • Improves the CLI logout account checkbox labels so each row clearly shows email, current status, and endpoint.
  • Uses Inquirer short labels so selected accounts echo without long endpoint wrapping.
  • Adds CLI e2e coverage for logout choice ordering and labels.

Testing

  • php example.php cli
  • composer lint-twig
  • composer refactor:check
  • vendor/bin/phpunit tests/e2e/CLIBun13Test.php reached and passed auth:logout-question-choices before the expected-output list was updated; after updating it, rerun failed before test execution because Docker reported network mockapi not found.

@ChiragAgg5k ChiragAgg5k force-pushed the fix/cli-logout-prompt branch from 3e14be8 to c20c65f Compare June 23, 2026 11:59
@ChiragAgg5k ChiragAgg5k merged commit 81e1bfe into main Jun 23, 2026
59 checks passed
@ChiragAgg5k ChiragAgg5k deleted the fix/cli-logout-prompt branch June 23, 2026 12:01
@greptile-apps

greptile-apps Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR improves the CLI logout account prompt. The main changes are:

  • Clearer logout prompt wording.
  • Logout choices that show email, current status, and endpoint.
  • Short selected-account labels for Inquirer output.
  • E2E coverage for logout choice ordering and labels.

Confidence Score: 5/5

The changes are limited to CLI prompt labeling and related end-to-end expectations, with no unresolved code issues identified.

The modified behavior is narrow in scope, covered by targeted CLI test updates, and does not introduce risky control-flow, persistence, or security-sensitive changes.

T-Rex T-Rex Logs

What T-Rex did

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "Improve CLI logout account prompt" | Re-trigger Greptile

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