Skip to content

fix(cli): preserve chat -q answer by gating exit-summary screen clear (#60926) - #60968

Closed
HEBEI77 wants to merge 4 commits into
NousResearch:mainfrom
HEBEI77:fix/60926-chat-q-clear
Closed

fix(cli): preserve chat -q answer by gating exit-summary screen clear (#60926)#60968
HEBEI77 wants to merge 4 commits into
NousResearch:mainfrom
HEBEI77:fix/60926-chat-q-clear

Conversation

@HEBEI77

@HEBEI77 HEBEI77 commented Jul 8, 2026

Copy link
Copy Markdown

Fixes #60926 where running immediately erased the output because unconditionally cleared the terminal.

  • Modified to accept argument.
  • Gate the clear command in single-query mode.
  • Updated in regression tests to match the new signature.

@alt-glitch alt-glitch added type/bug Something isn't working comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery P2 Medium — degraded but workaround exists duplicate This issue or pull request already exists labels Jul 8, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Duplicate of #60926 — the maintainer salvage that gated _print_exit_summary's exit-clear behind a clear_screen kwarg (passing False on the chat -q path) was merged and is already on main (commit 58e1647). This PR re-implements the identical fix. Note it also bundles a leftover create_pr.sh artifact (whose body clones #60926's description) that shouldn't ship. Superseded by the merged #60926; related lineage #53025 (predecessor), #53009 (original issue).

@HEBEI77 HEBEI77 closed this Jul 8, 2026
@HEBEI77
HEBEI77 deleted the fix/60926-chat-q-clear branch July 8, 2026 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery duplicate This issue or pull request already exists P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants