fix: remove spurious blank lines from status output#848
Merged
Conversation
Both the short and detailed status paths printed two empty lines before the actual status line, making the output look padded. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 4123b5a6edb5
computermode
approved these changes
Apr 3, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR tidies the entire status command output by removing two unintended leading blank lines that were printed before the first status line in both short and --detailed modes.
Changes:
- Removed two
fmt.Fprintln(w)calls that produced extra leading blank lines in short status output. - Removed the same extra blank lines from the detailed status path, keeping the single intentional separator blank line after the effective status line.
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
entire status(both short and--detailedmodes) printed two blank lines before the actual status line, making the output look unnecessarily paddedfmt.Fprintln(w)calls from both code pathsTest plan
go test ./cmd/entire/cli/ -run Status)🤖 Generated with Claude Code
Note
Low Risk
Low risk: output formatting-only change in the
statuscommand; no settings/session logic is modified.Overview
Removes the two leading blank lines previously printed before the first status line in
entire status.This applies to both the default (short) output and
--detailedmode, so status output starts immediately with the effective settings summary.Written by Cursor Bugbot for commit c7a7b2c. Configure here.