Standardize CLI terminal output: separator widths and missing emojis#84
Standardize CLI terminal output: separator widths and missing emojis#84
Conversation
Co-authored-by: ewega <26189114+ewega@users.noreply.github.com>
There was a problem hiding this comment.
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)) |
There was a problem hiding this comment.
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.
| fmt.Println(strings.Repeat("\u2500", 40)) | |
| fmt.Println(strings.Repeat("\u2500", 40)) | |
| fmt.Println() |
| fmt.Println("\n" + strings.Repeat("─", 40)) | ||
| fmt.Println("⚠️ No connections were created.") | ||
| fmt.Println(strings.Repeat("─", 50)) | ||
| fmt.Println(strings.Repeat("─", 40)) |
There was a problem hiding this comment.
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.
| fmt.Println(strings.Repeat("─", 40)) | |
| fmt.Println(strings.Repeat("─", 40)) | |
| fmt.Println() |
Audit of CLI output found two classes of inconsistency: summary separators used 50-char
─whileprintBanner()uses 40-char═, and several section headers in cleanup/deploy flows lacked emoji prefixes unlike the rest of the CLI.configure_full,configure_connection_{add,update,delete},configure_scope_{add,delete},configure_project{s,_delete},statuscleanup.go(📋 Deployment found,🗑️ Resources to delete,🌐 Endpoints that will be removed) anddeploy_azure.go(📋 Configuration)Output-only changes — no logic, API, or data flow modifications.
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.