Skip to content

Commit dee1175

Browse files
committed
chore: remove simplify step from Claude PR review and add prek rule
Drop the /simplify step that caused unprompted refactors and scope creep in PR reviews. Also add prek pre-commit rule to project config so the PR bot and all contributors see it.
1 parent c611617 commit dee1175

2 files changed

Lines changed: 2 additions & 7 deletions

File tree

.claude/rules/code-style.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,4 @@
99
- **Naming**: NEVER use leading underscores (`_function_name`) - Python has no true private functions, use public names
1010
- **Paths**: Always use absolute paths
1111
- **Encoding**: Always pass `encoding="utf-8"` to `open()`, `read_text()`, `write_text()`, etc. in new or changed code — Windows defaults to `cp1252` which breaks on non-ASCII content. Don't flag pre-existing code that lacks it unless you're already modifying that line.
12+
- **Pre-commit**: Run `uv run prek` before committing — fix any issues before creating the commit

.github/workflows/claude.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -192,12 +192,6 @@ jobs:
192192
*Last updated: <timestamp>*
193193
</step>
194194
195-
<step name="simplify">
196-
Run /simplify to review recently changed code for reuse, quality, and efficiency opportunities.
197-
If improvements are found, commit with "refactor: simplify <description>" and push.
198-
Only make behavior-preserving changes.
199-
</step>
200-
201195
<step name="merge_optimization_prs">
202196
Check for open PRs from codeflash-ai[bot]:
203197
`gh pr list --author "codeflash-ai[bot]" --state open --json number,title,headRefName,createdAt,mergeable`
@@ -219,7 +213,7 @@ jobs:
219213
- All findings are in a single summary comment (no inline review comments were created)
220214
- If fixes were made, they were verified with prek
221215
</verification>
222-
claude_args: '--model us.anthropic.claude-sonnet-4-6 --max-turns 25 --allowedTools "Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*),Bash(gh pr checks:*),Bash(gh pr merge:*),Bash(gh issue view:*),Bash(gh issue list:*),Bash(gh api:*),Bash(uv run prek *),Bash(uv run mypy *),Bash(uv run coverage *),Bash(uv run pytest *),Bash(git status*),Bash(git add *),Bash(git commit *),Bash(git push*),Bash(git diff *),Bash(git checkout *),Read,Glob,Grep,Edit,Skill"'
216+
claude_args: '--model us.anthropic.claude-sonnet-4-6 --allowedTools "Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*),Bash(gh pr checks:*),Bash(gh pr merge:*),Bash(gh issue view:*),Bash(gh issue list:*),Bash(gh api:*),Bash(uv run prek *),Bash(uv run mypy *),Bash(uv run coverage *),Bash(uv run pytest *),Bash(git status*),Bash(git add *),Bash(git commit *),Bash(git push*),Bash(git diff *),Bash(git checkout *),Read,Glob,Grep,Edit"'
223217
additional_permissions: |
224218
actions: read
225219

0 commit comments

Comments
 (0)