Move skills to .claude/skills/ for Claude Code#444
Conversation
- Move test_bot.md to .claude/skills/ where Claude Code picks it up - Add remote-gpu-testing.md skill for SSH+tmux workflow on GPU machines - Update .gitignore to track .claude/skills/ while ignoring other .claude/ files - Update CLAUDE.md reference to new path
There was a problem hiding this comment.
Pull request overview
Moves Claude Code “skills” documentation into the location Claude Code reads and adds guidance for running GPU workflows remotely.
Changes:
- Update
CLAUDE.mdto reference the new skills path under.claude/skills/ - Add a new skill doc for running GPU experiments/tests remotely via SSH + tmux
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| CLAUDE.md | Updates the skills reference to the new .claude/skills/ location |
| .claude/skills/remote-gpu-testing.md | Adds a documented SSH + tmux workflow for remote GPU testing |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| Never run GPU code locally. Always use this pattern: | ||
|
|
||
| 1. **Set up a tmux session** on the remote machine (idempotent): |
There was a problem hiding this comment.
Using || true makes the command appear successful even when tmux fails for real reasons (e.g., tmux not installed, permission issues), which can mask problems and lead to confusing follow-on failures. Prefer an idempotent check that only suppresses the 'session already exists' case (e.g., tmux has-session ... || tmux new-session ...) so genuine failures still surface.
|
|
||
| 2. **Set up the environment** (mirrors the GH Action): | ||
| ```bash | ||
| ssh host "tmux send-keys -t work 'cd /home/user/kernelbot && pip install -r requirements-dev.txt && pip install -e . 2>&1 | tee /tmp/setup.log' Enter" |
There was a problem hiding this comment.
Shell redirection/piping precedence here means 2>&1 | tee /tmp/setup.log applies only to the final pip install -e ., so output from pip install -r requirements-dev.txt will not be captured in /tmp/setup.log. If the intent is to log the entire setup sequence, group the commands so the redirection/pipe wraps both installs.
| ssh host "tmux send-keys -t work 'cd /home/user/kernelbot && pip install -r requirements-dev.txt && pip install -e . 2>&1 | tee /tmp/setup.log' Enter" | |
| ssh host "tmux send-keys -t work 'cd /home/user/kernelbot && (pip install -r requirements-dev.txt && pip install -e .) 2>&1 | tee /tmp/setup.log' Enter" |
Coverage report
Click to see where and how coverage changed
This report was generated by python-coverage-comment-action |
||||||||||||||||||||||||
1 participant
Summary
SKILLS/test_bot.mdto.claude/skills/test_bot.mdwhere Claude Code actually reads itremote-gpu-testing.mdskill documenting the SSH + tmux pattern for running GPU experiments on remote machines.gitignoreto track.claude/skills/while keeping other.claude/files ignoredCLAUDE.mdreference to point to new pathTest plan
.claude/skills/files are checked in and visible on the branch.claude/files (e.g.settings.local.json) are still gitignored