fix(cli): improve namespace inspect member list display#6324
Merged
Conversation
Sort members alphabetically by email to match the UI ordering. Replace fixed-width username padding with tabwriter for proper column alignment regardless of email length. Update the Members section header to include the count inline as "Members (N):".
|
Claude finished @geovannewashington's task in 32s —— View job Code Review CompleteReviewed 1 file across code quality, security, testing, Go patterns, and architecture — no issues found. The code looks good as-is. The changes are clean and well-scoped: alphabetical sorting by email via If you push additional changes and want a new review, tag |
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.
What changed?
Improved the member list display in
namespace inspect:%-12spadding withtabwriterfor properalignment with emails of any length
Members: NtoMembers (N):for consistency with theother section headers
Why
With real client data, members with long emails or usernames caused
the role column to misalign. The ordering was also inconsistent with
the UI, which sorts members alphabetically by email.
How to test
./bin/docker-compose up -dcli namespace inspect <namespace>on a namespace withmultiple members