Skip to content

Standardize CLI terminal output: separator widths and missing emojis#84

Open
Copilot wants to merge 2 commits intomainfrom
copilot/analyze-cli-ux-tui
Open

Standardize CLI terminal output: separator widths and missing emojis#84
Copilot wants to merge 2 commits intomainfrom
copilot/analyze-cli-ux-tui

Conversation

Copy link
Contributor

Copilot AI commented Mar 3, 2026

Audit of CLI output found two classes of inconsistency: summary separators used 50-char while printBanner() uses 40-char , and several section headers in cleanup/deploy flows lacked emoji prefixes unlike the rest of the CLI.

  • Separator width → 40 across all summary blocks in configure_full, configure_connection_{add,update,delete}, configure_scope_{add,delete}, configure_project{s,_delete}, status
  • Add emojis to plain section headers in cleanup.go (📋 Deployment found, 🗑️ Resources to delete, 🌐 Endpoints that will be removed) and deploy_azure.go (📋 Configuration)

Output-only changes — no logic, API, or data flow modifications.

Original prompt

Can you please do a detailed analysis and generate a report of any improvement we can make to the CLI UX/TUI? Is spacing and indentation appropritate? Anywhere we can safely add emojis? How to make it more readable?

Custom agent used: prettify
Terminal UX specialist — enforces output formatting rules, improves readability, and ensures consistent visual rhythm across all CLI commands.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: ewega <26189114+ewega@users.noreply.github.com>
Copilot AI changed the title [WIP] Analyze CLI UX and TUI for improvements Standardize CLI terminal output: separator widths and missing emojis Mar 3, 2026
Copilot finished work on behalf of ewega March 3, 2026 20:24
@ewega ewega requested a review from Copilot March 3, 2026 21:24
@ewega ewega marked this pull request as ready for review March 3, 2026 21:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Standardizes the CLI’s terminal output formatting to better match the repository’s terminal UX conventions by aligning summary separator widths and adding emoji-prefixed section headers where they were missing.

Changes:

  • Normalize summary separators from 50 to 40 characters across multiple configure/status flows.
  • Add emoji prefixes to previously plain section headers in Azure deploy/cleanup output.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
cmd/status.go Adjusts the status section separator width to 40.
cmd/deploy_azure.go Adds 📋 emoji to the “Configuration” header.
cmd/configure_scope_delete.go Updates summary separator width to 40.
cmd/configure_scope_add.go Updates summary separator width to 40.
cmd/configure_projects.go Updates project finalization summary separator width to 40.
cmd/configure_project_delete.go Updates delete summary separator width to 40.
cmd/configure_full.go Updates connection-setup summary separator widths to 40.
cmd/configure_connection_update.go Updates update summary separator width to 40.
cmd/configure_connection_delete.go Updates delete summary separator width to 40.
cmd/configure_connection_add.go Updates add summary separator width to 40.
cmd/cleanup.go Adds emoji-prefixed headers for deployment/resources/endpoints sections.

fmt.Printf(" Plugin: %s\n", pn)
}
fmt.Println(strings.Repeat("\u2500", 50))
fmt.Println(strings.Repeat("\u2500", 40))
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

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

This summary block ends with a closing separator and then returns without printing a trailing blank line. Terminal Output Standards recommend a blank line after separators so the shell prompt (or subsequent output) doesn’t jam against the separator. Consider printing a final blank line after the closing separator here.

Suggested change
fmt.Println(strings.Repeat("\u2500", 40))
fmt.Println(strings.Repeat("\u2500", 40))
fmt.Println()

Copilot uses AI. Check for mistakes.
fmt.Println("\n" + strings.Repeat("─", 40))
fmt.Println("⚠️ No connections were created.")
fmt.Println(strings.Repeat("─", 50))
fmt.Println(strings.Repeat("─", 40))
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

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

In the len(results) == 0 branch, the summary separator block ends and the function returns an error immediately. Per Terminal Output Standards (blank line after separators), this will cause the subsequent error message printed by Execute() to appear directly under the separator with no breathing room. Add an extra blank line after the closing separator (or ensure the next output starts with a blank line) before returning.

Suggested change
fmt.Println(strings.Repeat("─", 40))
fmt.Println(strings.Repeat("─", 40))
fmt.Println()

Copilot uses AI. Check for mistakes.
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.

3 participants