feat: office install runs install.sh - #71
Conversation
The same verb as zyx and murmurflow: set it up, start nothing. `office install [--theme]` runs ./install.sh, which is idempotent. Before this the word fell through to the fuzzy repo match and looked for a repo called "install". CI's second fresh-HOME install now goes through `office install`. All eight probes in bin/ pass locally. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe PR adds an ChangesOffice install command
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Sequence Diagram(s)sequenceDiagram
participant Office as office install
participant InstallScript as install.sh
participant ShellTmux as zsh and tmux
Office->>InstallScript: Invoke with trailing arguments
InstallScript->>ShellTmux: Rewire office
Merge Risk: ⚪ Minimal · up to The new install command is documented, forwards installer options, and is exercised by CI without identified merge-blocking risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
Adds
office install [--theme]: runs./install.sh(idempotent). It starts nothing.Why: zyx, murmurflow and office now share five verbs with one meaning each:
install(set up, start nothing) ·on·off·update·doctor.Office already had
on,off,update,doctor;installwas the gap. Before this,office installfell through to the fuzzy repo match.installis in the unzoom-exemption list (it moves no pane).bin/office install.bin/pass (each run withenv -u TMUX);office installrun twice on a throwaway HOME + TMUX_TMPDIR gives one line in.zshrc, one in.tmux.conf, and the~/.local/bin/officelink. No tmux version guard needed (no tmux call added).🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
office install [--theme]command to re-run installation, rewire office into zsh and tmux, and safely accept installation options.Documentation