let5see/clawdbot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs(skill): add PR review safety rules for coding-agent

Browse Source 
- Never checkout branches in live Clawdis repo
- Clone to temp folder or use git worktree for reviews
- Added explicit examples for safe PR review workflow
This commit is contained in:
Peter Steinberger
2026-01-03 12:49:03 +00:00
parent 0647d56555
commit 861e1b33f5
1 changed files with 21 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
skills/coding-agent/SKILL.md
View File

@@ -47,10 +47,29 @@ bash workdir:~/project background:true command:"codex exec --full-auto \"Build a
``` ```
### Reviewing PRs (vanilla, no flags) ### Reviewing PRs (vanilla, no flags)
**⚠️ CRITICAL: Never review PRs in Clawdis's own project folder!**
- Either use the project where the PR is submitted (if it's NOT ~/Projects/clawdis)
- Or clone to a temp folder first
```bash ```bash
bash workdir:~/project background:true command:"codex exec \"Review PR #115. Use: git diff origin/main...origin/pr/115\"" # Option 1: Review in the actual project (if NOT clawdis)
bash workdir:~/Projects/some-other-repo background:true command:"codex review --base main"
# Option 2: Clone to temp folder for safe review (REQUIRED for clawdis PRs!)
REVIEW_DIR=$(mktemp -d)
git clone https://github.com/steipete/clawdis.git $REVIEW_DIR
cd $REVIEW_DIR && gh pr checkout 130
bash workdir:$REVIEW_DIR background:true command:"codex review --base origin/main"
# Clean up after: rm -rf $REVIEW_DIR
# Option 3: Use git worktree (keeps main intact)
git worktree add /tmp/pr-130-review pr-130-branch
bash workdir:/tmp/pr-130-review background:true command:"codex review --base main"
``` ```
**Why?** Checking out branches in the running Clawdis repo can break the live instance!
### Batch PR Reviews (parallel army!) ### Batch PR Reviews (parallel army!)
```bash ```bash
# Fetch all PR refs first # Fetch all PR refs first
@@ -103,3 +122,4 @@ bash workdir:~/project background:true command:"opencode run \"Your task\""
5. **vanilla for reviewing** — no special flags needed 5. **vanilla for reviewing** — no special flags needed
6. **Parallel is OK** — run many Codex processes at once for batch work 6. **Parallel is OK** — run many Codex processes at once for batch work
7. **NEVER start Codex in ~/clawd/** — it'll read your soul docs and get weird ideas about the org chart! Use the target project dir or /tmp for blank slate chats 7. **NEVER start Codex in ~/clawd/** — it'll read your soul docs and get weird ideas about the org chart! Use the target project dir or /tmp for blank slate chats
8. **NEVER checkout branches in ~/Projects/clawdis/** — that's the LIVE Clawdis instance! Clone to /tmp or use git worktree for PR reviews