fix(installer,workflow): silence /dev/tty noise; bump action-gh-release to v3#35
Conversation
…se to v3 - install.sh: replace `[ -r /dev/tty ]` (which returns true in non-interactive contexts where /dev/tty exists in the namespace but isn't actually openable) with a real `can_read_tty` try-open helper. Eliminates the four "environment: line N: /dev/tty: No such device or address" warnings during `bash -c "$(curl ...)"` installs. - .github/workflows/release.yml: bump softprops/action-gh-release from v2 (Node 20, deprecated) to v3 (Node 24). v3.0.0 is API-compatible per upstream release notes; only runtime is bumped. Bump to 0.1.3. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThis PR centralizes TTY readability detection in the install script by extracting a ChangesTTY Detection Refactor and Release
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Brings in installer bugfixes (PRs #33-#35), .maestro/ untrack (#36), and the docs split (#37) that landed on main after rc was branched. Conflict resolutions: - Kept rc's queue error-hiding (PR #39) and per-message logger.error. - Kept rc's WAL-checkpoint shutdown alongside main's split DB import. - Combined main's docs split with rc's Slack provider additions in README.md, AGENTS.md, .env.example, install.sh, providers.ts, migrations.ts. - Extended config_complete in maestro-relay-ctl.sh to cover the Slack key set when ENABLED_PROVIDERS=slack.
Summary
Cosmetic patches found during the v0.1.2 fresh-install dry-run:
Bumps to `0.1.3` (patch).
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit
Chores
Improvements