Skip to content

feat(cli): add type column to namespace list output#6197

Merged
gustavosbarreto merged 2 commits intomasterfrom
feature/namespace-ls-type-column
Apr 16, 2026
Merged

feat(cli): add type column to namespace list output#6197
gustavosbarreto merged 2 commits intomasterfrom
feature/namespace-ls-type-column

Conversation

@geovannewashington
Copy link
Copy Markdown
Member

@geovannewashington geovannewashington commented Apr 16, 2026

What changed?

Added a TYPE column to the namespace ls table output. Also added
the ls alias to the user list command for consistency.

Why

Users need to quickly identify and categorize namespaces (e.g. team
vs personal) at a glance without having to inspect each one. The
ls alias on user list was missing despite being available on
namespace list.

Notes

-q type was intentionally not added. Quiet mode is designed for
command substitution targeting a specific namespace (e.g. piping
into inspect), which requires a unique identifier. Since type is
shared across many namespaces, it doesn't serve that purpose.

How to test

  • Run ./bin/cli namespace list or ./bin/cli namespace ls to
    verify the TYPE column appears in the table output
  • Run ./bin/cli user list or ./bin/cli user ls to verify the
    alias works

Closes #6190

Add a TYPE column to the namespace list table output, displaying
each namespace's type (e.g. team, personal) alongside its name
and tenant ID.

This allows users to categorically identify namespaces at a
glance without having to inspect each one individually.
@geovannewashington geovannewashington self-assigned this Apr 16, 2026
@geovannewashington geovannewashington requested a review from a team as a code owner April 16, 2026 13:51
@geovannewashington geovannewashington added go Pull requests that update Go code area/cli labels Apr 16, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 16, 2026

Claude finished @geovannewashington's task in 2m 31s —— View job


Code Review Complete

Reviewed 2 files across code quality, security, testing, Go patterns, and architecture — 1 inline issue posted (medium severity).

The TYPE column addition to namespace list and the ls alias on user list both look correct. The one thing to address is that the quiet-mode field selector (-q) doesn't yet support extracting the new type field, which creates a UX gap for scripting users.


If you've addressed the feedback and want a new review, tag @shellhub-io/admin and a team member can trigger it.

Add the ls alias to the user list command for consistency with
the namespace list command, which already exposes this alias.
@geovannewashington geovannewashington force-pushed the feature/namespace-ls-type-column branch from 1620d5a to 7e40665 Compare April 16, 2026 14:02
@gustavosbarreto
Copy link
Copy Markdown
Member

/review

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 16, 2026

Claude finished @gustavosbarreto's task in 1m 56s —— View job


Code Review Complete

Reviewed 2 files across code quality, security, testing, Go patterns, and architecture — no issues found. The code looks good as-is.


If you push additional changes and want a new review, tag @shellhub-io/admin and a team member can trigger it.

@shellhub-io shellhub-io deleted a comment from github-actions bot Apr 16, 2026
@gustavosbarreto gustavosbarreto merged commit 7e292a5 into master Apr 16, 2026
21 checks passed
@gustavosbarreto gustavosbarreto deleted the feature/namespace-ls-type-column branch April 16, 2026 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing 'type' column for ./bin/cli namespace list

2 participants