Skip to content

Commit e71fcbd

Browse files
authored
chore: replace review subagents with warden (#151)
1 parent e3d5419 commit e71fcbd

File tree

9 files changed

+13
-311
lines changed

9 files changed

+13
-311
lines changed

.claude/agents/perf-review.md

Lines changed: 0 additions & 89 deletions
This file was deleted.

.claude/agents/react-review.md

Lines changed: 0 additions & 50 deletions
This file was deleted.

.claude/agents/security-review.md

Lines changed: 0 additions & 83 deletions
This file was deleted.

.claude/skills/create-pr/SKILL.md

Lines changed: 6 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -45,28 +45,14 @@ That's it. No "Test Plan", no "Screenshots", no checklists unless truly needed.
4545
git diff --name-only main...HEAD
4646
```
4747

48-
2. **Run code-simplifier first** (if available):
49-
50-
Run the `code-simplifier:code-simplifier` agent to simplify and clean up the code.
51-
This step modifies code, so it must run before reviews. Commit any changes it makes.
52-
53-
3. **Run validation + reviews in parallel**:
54-
55-
After code-simplifier is done, run these concurrently:
56-
- `bun run validate` (background)
57-
- Review agents based on changed files:
58-
59-
| Changed files | Agent to spawn |
60-
|---------------|----------------|
61-
| `src/agent/`, auth, user input, data handling | `security-review` |
62-
| Loops, data fetching, DB queries, heavy computation | `perf-review` |
63-
| `web/` or `mobile/` (.tsx/.jsx files) | `react-review` |
64-
65-
Spawn all applicable review agents in parallel using the Task tool.
48+
2. **Run validation**:
49+
```bash
50+
bun run validate
51+
```
6652

67-
4. **Fix any issues** found by validation or review agents before proceeding
53+
3. **Fix any issues** found by validation before proceeding
6854

69-
5. **Create PR** (only after validation passes and reviews are addressed):
55+
4. **Create PR** (only after validation passes):
7056
```bash
7157
gh pr create --title "<type>: <description>" --body "$(cat <<'EOF'
7258
## Summary

.claude/skills/validate/SKILL.md

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -49,17 +49,11 @@ Run the appropriate validation for your changes.
4949

5050
3. If validation fails, fix issues and re-run
5151

52-
## Run Review Agents in Parallel
53-
54-
While validation runs, spawn review subagents in parallel based on what changed:
55-
56-
| Changed files | Run agent |
57-
|---------------|-----------|
58-
| Auth, user input, data handling | `security-review` |
59-
| Loops, data fetching, DB queries | `perf-review` |
60-
| `web/` or `mobile/` (.tsx/.jsx) | `react-review` |
61-
62-
These agents review your local changes and report issues. Run them in parallel with validation to save time.
52+
4. Run warden to get code review feedback locally (security, react best practices, code simplification):
53+
```bash
54+
warden -v
55+
```
56+
The `-v` flag streams findings in real-time. Fix any issues warden finds before creating a PR.
6357

6458
## Notes
6559

.opencode/agent/perf-review.md

Lines changed: 0 additions & 15 deletions
This file was deleted.

.opencode/agent/react-review.md

Lines changed: 0 additions & 16 deletions
This file was deleted.

.opencode/agent/security-review.md

Lines changed: 0 additions & 15 deletions
This file was deleted.

AGENTS.md

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -13,19 +13,9 @@ Use the `skill` tool for common workflows:
1313

1414
If validation fails due to missing tools, install dependencies first with `bun install`.
1515

16-
## Code Review Agents
16+
## Code Review
1717

18-
Before submitting changes, run review agents in parallel with validation:
19-
20-
| Agent | When to use |
21-
|-------|-------------|
22-
| `security-review` | After auth, data handling, or user input changes |
23-
| `perf-review` | After adding loops, data fetching, or heavy computation |
24-
| `react-review` | After React component changes (web/ or mobile/) |
25-
26-
These agents review your local changes (uncommitted, branch diff, or last commit) and report issues. They will be honest - if the code is good, they'll say so. They may also suggest lint rules or tests to prevent similar issues in the future.
27-
28-
**Location**: Claude Code uses `.claude/agents/`, OpenCode uses `.opencode/agent/` (OpenCode agents reference the Claude Code files for shared instructions).
18+
Code reviews (security, react best practices, code simplification) are handled automatically by [Warden](https://github.com/getsentry/warden) on pull requests. See `warden.toml` for the configured triggers.
2919

3020
## Architecture
3121

0 commit comments

Comments
 (0)