Skip to content

fix(host): say what say, stop, rename, archive and other commands did, and ask before deleting a busy channel - #661

Merged
iamnbutler merged 2 commits into
mainfrom
fix-cli-says
Oct 22, 2026
Merged

iamnbutler merged 2 commits into
mainfrom
fix-cli-says

Conversation

@iamnbutler

Copy link
Copy Markdown
Contributor

Several CLI commands printed nothing on success, and ace delete removed a channel with a run in progress without asking.

  • One line on success:
    • say → Posted to <channel>.
    • ask --detach → Asked <channel>'s agent; the run continues. Follow it with: ace watch <channel>
    • answer → Answered the question in <channel>.
    • stop → Stopped the run in <channel>. or Nothing was running in <channel>., based on the chat's state before the stop
    • rename → Renamed <old> to <new>.
    • archive → Archived <channel>. Resume it with: ace unarchive <channel>
    • unarchive → Unarchived <channel>.
    • desktop … off → Desktop tools are off for <channel>. Its agents' new desktop calls are refused.
  • ace delete on a channel with a run in progress asks <channel> has a run in progress. Delete it anyway? [y/N] in a terminal. Without a terminal it refuses and names --yes, which skips the question.
  • ace ls and ace ls --team start with a header row.

Not in this PR: ace ls still shows running for a channel whose worker is up with no run. docs/terms.md only defines Dormant and avoids "idle", so a word for that state needs a terms decision. Also not here: --team's column order, and the serve, demo and --version items in #625. Part of #625. Map: ^422, ^449.

Smoke: loopback source host with its own ACE_HOME, real model.

  • new/say/stop/rename/archive/unarchive/desktop off on a new channel printed each line above. stop with nothing running said so.
  • ace ls and ace ls --team printed their header rows.
  • ask --detach "Run 'sleep 60'…" printed its line. ace delete then refused without a terminal (exit 1). stop printed Stopped the run in clisaid. and delete --yes deleted a busy channel.
  • On a pseudo-terminal, answering y to the delete question deleted the channel.
  • answer was not exercised (it needs a waiting question).

Built in Ace

No: made from Claude Code (a fix-train worker), using the CLI for the smoke test. Dogfooding gap tracked in #5.

@iamnbutler
iamnbutler merged commit 542cee9 into main Oct 22, 2026
1 check passed
@iamnbutler
iamnbutler deleted the fix-cli-says branch October 22, 2026 21:31
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.

1 participant