Skip to content

Hide console status in quiet mode - #1375

Open
daandemeyer wants to merge 1 commit into
facebook:mainfrom
daandemeyer:push-monyqsutowvk
Open

Hide console status in quiet mode#1375
daandemeyer wants to merge 1 commit into
facebook:mainfrom
daandemeyer:push-monyqsutowvk

Conversation

@daandemeyer

Copy link
Copy Markdown
Contributor

Make simple-console command start and final remote execution summaries respect
status verbosity. Extend the same behavior to the superconsole final status
frame and the post-superconsole Build ID reprint.

This allows -v 0,actions to report completed actions in the simple console
while remaining silent when a command performs no work. The live superconsole
still renders while a command is running, preserving interactive progress
without leaving status-only output in scrollback.

Add regression coverage that verifies quiet superconsole status remains visible
during normal rendering but is omitted from the final frame.

Validated with the buck2_client_ctx superconsole unit tests, a buck2_client
Cargo check, and rustfmt.

Signed-off-by: Daan De Meyer [email protected]

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 17, 2026
@meta-codesync

meta-codesync Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

This pull request has been imported. If you are a Meta employee, you can view this in D112616169. (Because this pull request was imported automatically, there will not be any future comments.)

Make simple-console command start and final remote execution summaries respect
status verbosity. Extend the same behavior to the superconsole final status
frame and the post-superconsole Build ID reprint.

This allows -v 0,actions to report completed actions in the simple console
while remaining silent when a command performs no work. The live superconsole
still renders while a command is running, preserving interactive progress
without leaving status-only output in scrollback.

Declare buck2_memfrag as a recognized workspace cfg so the PR continues to
build after the daemon dump watcher landed on main and CI began denying unknown
cfg names.

Add regression coverage that verifies quiet superconsole status remains visible
during normal rendering but is omitted from the final frame.

Validated with the buck2_client_ctx superconsole unit tests, a buck2_client
Cargo check, rustfmt, and targeted buck2_daemon clippy against current main.

Signed-off-by: Daan De Meyer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant