Add whoami command using GET /v3/me - #326
Merged
Merged
Conversation
Show the authenticated user and workspace from the new me endpoint so users can verify which account the CLI is using.
sergey-borovkov
left a comment
Contributor
There was a problem hiding this comment.
A few small things inline, nothing blocking.
Use N/A only in table output and empty cells in CSV so missing data is not confused with real values, and cover empty names / missing URL in tests.
Keep a single rows() list so adding or reordering fields cannot drift between output formats.
The mock server test does not touch HOME or the filesystem, so the tempdir setup was dead code.
Keep nightly rustfmt check green after the Option-field review changes.
Log failed API bodies with debug! like the other HTTP helpers so JSON/CSV output on stdout stays clean.
sergey-borovkov
approved these changes
Sep 23, 2026
sergey-borovkov
left a comment
Contributor
There was a problem hiding this comment.
All three comments addressed, plus the stdout fix in commands::get. Checked out the branch: 222 tests pass, clippy and cargo fmt --check are clean, and with a bad token stdout is now empty so -o json stays parseable.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
screenly whoamicommand that callsGET /v3/mein a single requestLocal build test