Fix os.getlogin() crash in ns setup

[gwarmstrong]

Replace os.getlogin() with os.path.expanduser('~') (mount path default) and getpass.getuser() (SSH username default). Both work in containers/VMs without a login session.

Closes #1269

Summary by CodeRabbit

• Bug Fixes
  • Improved reliability of default configuration values in pipeline setup by using more robust methods for determining home directory and SSH username defaults.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aa47923 and c364d71.

📒 Files selected for processing (1)

• nemo_skills/pipeline/setup.py

---

📝 Walkthrough

Walkthrough

Changed the method of determining home directory and SSH username in nemo_skills/pipeline/setup.py from using os.getlogin() to os.path.expanduser('~') for the mount path default and getpass.getuser() for the SSH username default. Added corresponding import for getpass.

Changes

Cohort / File(s)	Summary
Home directory resolution nemo_skills/pipeline/setup.py	Replaced os.getlogin() with os.path.expanduser('~') for local config mount path default. Added getpass.getuser() for SSH username default instead of os.getlogin(). Added getpass import.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 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 title accurately summarizes the main change: fixing os.getlogin() crashes in the ns setup by replacing it with container-safe alternatives.
Linked Issues check	✅ Passed	The PR implements all required fixes from issue #1269: replacing os.getlogin() with os.path.expanduser('~') for mount paths and getpass.getuser() for SSH usernames.
Out of Scope Changes check	✅ Passed	All changes are directly related to fixing the os.getlogin() issue in nemo_skills/pipeline/setup.py; no out-of-scope modifications were introduced.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch georgea/fix-issue-1269

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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