chore: drop VPS identifier from deploy script#62
Conversation
'founder-private artifacts' -> 'internal artifacts' in the deploy script's ship-list comment. Removes a name that referenced an unpublished directory.
Five places named the operator's mainnet host alias 'vps6'. The script header, the env doc comment, and the MAINNET_HOST default are all reworded to use 'the mainnet host' generically. MAINNET_HOST default removed — operators must now set it explicitly (error if unset). Anyone running this script already had to point at their own ssh alias; the previous default was leak surface for one specific operator's local config.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe PR updates Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
🚥 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)
Comment |
Five places named the operator's mainnet host alias 'vps6'. The script header, the env doc comment, and the MAINNET_HOST default are all reworded to use 'the mainnet host' generically.
MAINNET_HOST default removed — operators must now set it explicitly (error if unset). Anyone running this script already had to point at their own ssh alias; the previous default was leak surface for one specific operator's local config.
Found by a follow-up audit pass for
vps[1-9]patterns across public repos.Summary by CodeRabbit