fix(deploy): use the remote user's home for Brev sync

[WuKongAI-CMU]

Summary

• stop hardcoding /home/ubuntu/nemoclaw in nemoclaw deploy
• sync, upload .env, and reconnect through ~/nemoclaw so existing Brev instances work across non-Ubuntu usernames
• update the remote-deploy docs and generated skill snippets to match the new path

Verification

• npm test -- test/cli.test.js -t "deploy uses the remote user's home directory instead of /home/ubuntu"
• npm test -- test/cli.test.js
• python3 scripts/docs-to-skills.py docs/ .agents/skills/ --prefix nemoclaw
• make docs
• git diff --check

Closes #1319.

Summary by CodeRabbit

• Documentation

  • Updated SSH command paths across deployment and network policy documentation for consistency.

• Tests

  • Added comprehensive CLI test for the deploy command to validate remote path handling and command generation.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 75235be3-c25e-453e-8931-0e3fc5af1d69

📥 Commits

Reviewing files that changed from the base of the PR and between db82a7c and 94efcd8.

📒 Files selected for processing (5)

• .agents/skills/nemoclaw-deploy-remote/SKILL.md
• .agents/skills/nemoclaw-manage-policy/SKILL.md
• docs/deployment/deploy-to-remote-gpu.md
• docs/network-policy/approve-network-requests.md
• test/cli.test.js

✅ Files skipped from review due to trivial changes (5)

• docs/network-policy/approve-network-requests.md
• .agents/skills/nemoclaw-manage-policy/SKILL.md
• .agents/skills/nemoclaw-deploy-remote/SKILL.md
• docs/deployment/deploy-to-remote-gpu.md
• test/cli.test.js

---

📝 Walkthrough

Walkthrough

Documentation and test updates replacing hardcoded absolute paths with home directory shortcuts. SSH commands in deployment and skill documentation changed from /home/ubuntu/nemoclaw to ~/nemoclaw. Added CLI test validating that deployment commands use the remote user's home directory instead of hardcoded paths.

Changes

Cohort / File(s)	Summary
Documentation Path Updates .agents/skills/nemoclaw-deploy-remote/SKILL.md, .agents/skills/nemoclaw-manage-policy/SKILL.md, docs/deployment/deploy-to-remote-gpu.md, docs/network-policy/approve-network-requests.md	Updated SSH command working directory from absolute path /home/ubuntu/nemoclaw to home directory shorthand ~/nemoclaw in remote-sandbox commands.
CLI Test Coverage test/cli.test.js	Added new test case for deploy shade-box command that verifies deployment uses the remote user's home directory in generated commands and does not hardcode /home/ubuntu/nemoclaw paths. Test stubs SSH, rsync, and scp binaries to validate correct path resolution.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A rabbit hopped through paths so plain,
Where /home/ubuntu caused much pain,
Now ~/nemoclaw shortcuts the way,
And tests ensure it works each day! 🎉

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title 'fix(deploy): use the remote user's home for Brev sync' directly and clearly describes the main change: replacing hardcoded /home/ubuntu with dynamic remote user home directory.
Linked Issues check	✅ Passed	The PR addresses all coding requirements from issue #1319: removes hardcoded /home/ubuntu paths from deploy commands, uses remote user's home directory dynamically for mkdir/rsync/scp operations, and adds test coverage verifying this behavior works across varying usernames.
Out of Scope Changes check	✅ Passed	All changes are in-scope: documentation updates reflect the code fix, skill snippets are regenerated per verification steps, and the new test validates the exact issue from #1319 without introducing unrelated functionality.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[wscurran]

✨ Thanks for submitting this pull request, which proposes a way to improve Brev compatibility by using the user's home directory instead of a hardcoded path.

---

Possibly related open issues:

• #1319 [NemoClaw][Brev] nemoclaw deploy — hardcoded /home/ubuntu path fails on non-ubuntu cloud providers