fix: usability items #5, #6, #8#25
Conversation
#5 — Suppress npm deprecation spam: - Set NODE_NO_WARNINGS=1 when spawning gateway subprocess - All install commands already use stdio:'pipe' (hide npm stderr) #6 — Whisper/STT setup improvements: - Default voice=false on macOS (whisper pip install is heavy) - Add whisper check in postflight with install hint - Linux keeps voice=true (auto-install on first message) #8 — Simplify getting started: - 'roundhouse install' now deprecated with redirect to setup - Removed 60+ lines of redundant install logic from cli.ts - README Quick Start: 3 lines (npm install, setup, start) - Cleaned up unused imports (mkdir, writeFile, envQuote, etc.)
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 88980a20b8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| console.log(" Recommended:"); | ||
| console.log(" roundhouse setup --telegram\n"); | ||
| console.log(" This sets up config, installs packages, pairs Telegram,"); | ||
| console.log(" and installs the systemd service — all in one command.\n"); |
There was a problem hiding this comment.
On Linux, roundhouse install now only prints guidance and exits successfully without creating/updating the service, config, or env file. That creates a functional dead end for users and automation because the CLI still points people to install in other flows (start, status, update), but following that advice does nothing while returning exit code 0. Either delegate to setup or return a non-zero exit so callers don’t treat this as a successful install.
Useful? React with 👍 / 👎.
| notifyChatIds: [], | ||
| systemd: platform() === "linux", | ||
| voice: true, | ||
| voice: platform() === "linux", // Default off on macOS (whisper install is heavy) |
There was a problem hiding this comment.
Changing the default to voice: platform() === "linux" makes macOS setup runs always set opts.voice to false, and there is no --voice flag to opt back in. Because setup later writes voice.stt.enabled=false whenever opts.voice === false, rerunning setup on macOS will forcibly disable STT even for users who previously enabled it, turning a “default off” change into an effective “always off” behavior.
Useful? React with 👍 / 👎.
1 participant
#5 Suppress npm deprecation spam (NODE_NO_WARNINGS)
#6 STT default off on macOS, whisper check in postflight
#8 Deprecate install command, setup is single entry point