Skip to content

Commit ce0e956

Browse files
drnclaude
andauthored
Replace find with ls/git-ls-files in rereview skill
The dynamic context block used find with complex args and pipes, which triggered Bash permission checks and broke skill loading. Replaced with simpler ls and git ls-files alternatives. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 64eb1ff commit ce0e956

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

agents/skills/rereview/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@ Re-review all branch changes from scratch with independent competing reviewers.
1616
- Current branch: !`git branch --show-current`
1717
- Base ref: !`git branch -r 2>/dev/null | grep -oE 'origin/(main|master)' | head -1`
1818
- Git status: !`git status --short`
19-
- Project type: !`find . -maxdepth 1 \( -name go.mod -o -name Gemfile -o -name package.json -o -name Cargo.toml -o -name pyproject.toml \) 2>/dev/null | head -5`
20-
- Test framework: !`find . -maxdepth 4 -name "*_test.*" -o -name "*.test.*" -o -name "*_spec.*" 2>/dev/null | head -10`
19+
- Project type: !`ls go.mod Gemfile package.json Cargo.toml pyproject.toml 2>/dev/null | head -5`
20+
- Test framework: !`git ls-files 2>/dev/null | grep -E '_test\.|\.test\.|_spec\.' | head -10`
2121
- Changes vs main: !`git diff --stat HEAD...origin/main 2>/dev/null | head -50`
2222
- Changes vs master: !`git diff --stat HEAD...origin/master 2>/dev/null | head -50`
2323

0 commit comments

Comments
 (0)