Stop duplicating "Press Ctrl+C to exit." in holding messages

[umair-ably]

Summary

• formatListening (src/utils/output.ts:18) auto-appends "Press Ctrl+C to exit." to the message it gets, and logHolding/logListening (src/base-command.ts) both route through it.
• Every logHolding call site — and one logListening site in channels/presence/enter — passed a message that already ended with the same phrase, so the rendered output doubled it: Holding presence. Press Ctrl+C to exit. Press Ctrl+C to exit.
• Stripped the manual suffix from the six affected call sites. The helper still appends it once, and the holding-style sites now match the convention logListening callers already follow elsewhere (e.g. "Listening for messages.").

Test plan

• pnpm prepare builds clean
• pnpm exec eslint . — 0 errors
• pnpm test:unit — 2295 passed, 1 skipped, 1 todo
• Manual: run ably rooms presence enter chat and confirm the holding line reads Holding presence. Press Ctrl+C to exit. (single suffix)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
cli-web-cli	Ready	Preview, Comment	Apr 27, 2026 1:14pm

[claude-code-ably-assistant]

Walkthrough

formatListening in src/utils/output.ts already appends "Press Ctrl+C to exit." to every message it renders, but six logHolding / logListening call sites were passing that phrase as part of their message string — causing the suffix to appear twice in terminal output (e.g. Holding presence. Press Ctrl+C to exit. Press Ctrl+C to exit.). This PR strips the manual suffix from all six offending sites so the helper appends it exactly once, matching the convention already followed by other logListening callers.

Changes

Area	Files	Summary
Commands – Channels	src/commands/channels/presence/enter.ts	Remove duplicate "Press Ctrl+C to exit." from logListening and logHolding call
Commands – Rooms	src/commands/rooms/presence/enter.ts	Remove duplicate suffix from logHolding call
Commands – Spaces	src/commands/spaces/cursors/set.ts	Remove suffix from both branches of the simulate/hold ternary passed to logHolding
Commands – Spaces	src/commands/spaces/locations/set.ts	Remove suffix from logHolding call
Commands – Spaces	src/commands/spaces/locks/acquire.ts	Remove suffix from logHolding call
Commands – Spaces	src/commands/spaces/members/enter.ts	Remove suffix from logHolding call

Review Notes

• Behavioural change (display only): Terminal output for all six commands changes from <message> Press Ctrl+C to exit. Press Ctrl+C to exit. to <message> Press Ctrl+C to exit. — intentional fix, no functional side-effects.
• No new dependencies.
• Test coverage: Unit tests pass (2 295 passed). A manual smoke-test of one holding command (e.g. ably rooms presence enter) is recommended to confirm the rendered line looks correct; no automated test currently asserts the exact holding message text.
• No migration or deployment considerations — pure display fix.

[sacOO7]

LGTM