This repository was archived by the owner on Oct 21, 2025. It is now read-only.
Precommit #9
Merged
Precommit #9
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add .pre-commit-config.yaml with exact command matching - Use local hooks to ensure consistency with pyproject.toml settings - Run checks in order: ty → ruff check → ruff format - Add pre-commit setup instructions to CLAUDE.md - Include general file quality checks (trailing whitespace, etc.) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Remove multi_turn parameters from all test classes - Delete unused experimental files (exploit_simple.py, exploit_unified.py) - Delete simplified_test.py - Use Any type for flexible client handling (ANN401 already ignored) - Fix trailing whitespace and end-of-file issues - Update base class to accept any client type - Remove unused imports
- Enable auto-fixing: ruff check --fix and ruff format (no --check) - Remove redundant hooks (trailing-whitespace, end-of-file-fixer, etc) - Keep only essential validators: YAML, JSON, TOML, merge-conflict - Ruff already handles Python formatting/whitespace issues
- Undo end-of-file changes to generated JSON files - These files should not be modified by pre-commit hooks
- Document uv tool install with pre-commit-uv plugin - List what hooks do automatically (type check, lint, format) - Clarify that hooks use same commands/settings as manual runs - Note that ruff auto-fixes safe issues
jakozaur
approved these changes
Aug 18, 2025
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.