Skip to content

fix(host): print a sign-in link from ace serve and one-line errors for bad flags and demo scripts - #652

Merged
iamnbutler merged 4 commits into
mainfrom
fix-cli-serve
Oct 22, 2026
Merged

iamnbutler merged 4 commits into
mainfrom
fix-cli-serve

Conversation

@iamnbutler

Copy link
Copy Markdown
Contributor

Map ^888 ^890 ^897 ^4 (part of #625).

  • ace serve prints Sign in at http://127.0.0.1:<port>/#token=… after Ace on …. Ace Helper doesn't print it, because its console is its log file and that file must not hold the token.
  • --demo reads and checks the script before the host listens. A missing file, invalid JSON, a person without login/github, or a step without at, a known who, exactly one action or a channel each fail with one line that points to docs/demo.md.
  • Unknown flags and a missing flag value print one sentence plus Run ace --help for usage. instead of a Bun stack trace. --version prints the installed version, and ace help prints the help.
  • The help uses the command's installed name (ace-canary through Canary's link) and this host's port, rather than ace and 4140.

Smoked against a source host on an isolated ACE_HOME:

  • --bogus, serve --demo with no value, and {}, missing-login, unknown-who, bad-JSON and missing demo scripts each printed the expected single line.
  • A valid script started demo mode.
  • Opening the printed sign-in link in a headless browser loaded the app signed in, with no "Disconnected".
  • Running the CLI with the command set to ace-canary, as helper.ts now does, printed ace-canary … help with port 4142 and ace-canary 0.0.54.1.

Not changed (still in #625): silent success for say/stop/kill and the others, ace delete without confirmation, and the ace ls header and idle state (^422, ^449).

Built in Ace

Made from Claude Code outside Ace (a fix-train worker running CLI and headless-browser smoke tests against an isolated host); see #5.

…r bad flags and demo scripts

Help and --version name the command as installed, such as ace-canary.

Why: apps/host/src/gateway.ts#function serve | Only a command-line host prints the sign-in link; Ace Helper's console is its log file, which must never hold the token
@iamnbutler
iamnbutler merged commit 5bf32b8 into main Oct 22, 2026
1 check passed
@iamnbutler
iamnbutler deleted the fix-cli-serve branch October 22, 2026 21:23
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