From 9877f2640762713485b4b1d63c37c0836572096d Mon Sep 17 00:00:00 2001
From: Jens <1742418+1cu@users.noreply.github.com>
Date: Fri, 5 Dec 2025 20:39:10 +0100
Subject: [PATCH] chore: Improve CodeRabbit path filters configuration (#712)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

## ℹ️ Description

This PR improves the CodeRabbit configuration to ensure all important
project files are reviewed while excluding only build artifacts and
temporary files.

The previous configuration used a blanket `!**/.*` exclusion that was
unintentionally filtering out the entire `.github` directory, including
workflows, dependabot config, issue templates, and CODEOWNERS files.

## 📋 Changes Summary

- **Added** `.github/**` to include all GitHub automation files
(workflows, dependabot, templates, CODEOWNERS)
- **Added** root config files (`pyproject.toml`, `*.yaml`, `*.yml`,
`**/*.md`)
- **Removed** overly broad `!**/.*` exclusion pattern
- **Added** specific exclusions for Python cache directories
(`.pytest_cache`, `.mypy_cache`, `.ruff_cache`)
- **Added** explicit IDE file exclusions (`.vscode`, `.idea`,
`.DS_Store`)
- **Added** `pdm.lock` exclusion to reduce noise

### ⚙️ Type of Change
- [x] 🐞 Bug fix (non-breaking change which fixes an issue)
- [ ] ✨ New feature (adds new functionality without breaking existing
usage)
- [ ] 💥 Breaking change (changes that might break existing user setups,
scripts, or configurations)

## ✅ Checklist
- [x] I have reviewed my changes to ensure they meet the project's
standards.
- [x] I have tested my changes and ensured that all tests pass (`pdm run
test`).
- [x] I have formatted the code (`pdm run format`).
- [x] I have verified that linting passes (`pdm run lint`).
- [x] I have updated documentation where necessary.

By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Chores**
  * Updated internal code review configuration and automation settings.

<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
---
 .coderabbit.yaml | 56 ++++++++++++++++++++++++++++++++++--------------
 1 file changed, 40 insertions(+), 16 deletions(-)

diff --git a/.coderabbit.yaml b/.coderabbit.yaml
index 9cd13db..d96b773 100644
--- a/.coderabbit.yaml
+++ b/.coderabbit.yaml
@@ -15,7 +15,7 @@ tone_instructions: "Be strict about English-only code and translation system usa
 reviews:
   profile: "assertive"  # More feedback to catch complexity
   high_level_summary: true
-  review_status: true
+  review_status: false
   commit_status: true
   changed_files_summary: true
   sequence_diagrams: true
@@ -29,37 +29,61 @@ reviews:
 
   # Path filters to focus on important files
   path_filters:
+    # Source code
     - "src/**/*.py"
     - "tests/**/*.py"
     - "scripts/**/*.py"
+
+    # GitHub automation - workflows, dependabot, templates, etc.
+    - ".github/**"
+
+    # Root config files
+    - "pyproject.toml"
+    - "*.yaml"
+    - "*.yml"
+    - "**/*.md"
+
+    # Exclude build/cache artifacts
     - "!**/__pycache__/**"
-    - "!**/.*"
+    - "!**/.pytest_cache/**"
+    - "!**/.mypy_cache/**"
+    - "!**/.ruff_cache/**"
     - "!dist/**"
     - "!build/**"
+    - "!*.egg-info/**"
+
+    # Exclude IDE-specific files
+    - "!.vscode/**"
+    - "!.idea/**"
+    - "!.DS_Store"
+
+    # Exclude temporary files
     - "!*.log"
     - "!*.tmp"
     - "!*.temp"
 
+    # Exclude lock files (too noisy)
+    - "!pdm.lock"
+
   # Path-specific instructions for different file types
   path_instructions:
     - path: "src/kleinanzeigen_bot/**/*.py"
       instructions: |
         CRITICAL RULES FOR KLEINANZEIGEN BOT:
         1. ALL code, comments, and text MUST be in English
-        2. User-facing messages MUST use translation system (_()) function
-        3. NEVER access live website in tests (bot detection risk)
-        4. Use WebScrapingMixin for browser automation
-        5. Handle TimeoutError for all web operations
-        6. Use ensure() for critical validations
-        7. Don't add features until explicitly needed
-        8. Keep solutions simple and straightforward
-        9. Use async/await for I/O operations
-        10. Follow Pydantic model patterns
-        11. Use proper error handling and logging
-        12. Test business logic separately from web scraping
-        13. Include SPDX license headers on all Python files
-        14. Use type hints for all function parameters and return values
-        15. Use structured logging with context
+        2. NEVER access live website in tests (bot detection risk)
+        3. Use WebScrapingMixin for browser automation
+        4. Handle TimeoutError for all web operations
+        5. Use ensure() for critical validations
+        6. Don't add features until explicitly needed
+        7. Keep solutions simple and straightforward
+        8. Use async/await for I/O operations
+        9. Follow Pydantic model patterns
+        10. Use proper error handling and logging
+        11. Test business logic separately from web scraping
+        12. Include SPDX license headers on all Python files
+        13. Use type hints for all function parameters and return values
+        14. Use structured logging with context
     - path: "tests/**/*.py"
       instructions: |
         TESTING RULES: