ci: Replace black and flake8 with ruff. (#4866)

### Description
Replace `black` and `flake8` with `ruff`.

With this change also reformat the code base (only minor changes,
because the configuration is basically the same.) The commits for the
reformatting are not included in `git blame` calls (see
`.git-blame-ignore-revs`).

Also updates the `pre-commit` config so we have exactly the same
linting/formatting in CI and local dev.

We now have:
- Less dependencies 
- Less configuration
- Faster


#### Issues
* resolves: #2527

Author: antonpirker

.coveragerc36

@@ -3,12 +3,12 @@
 
 [run]
 branch = true
-omit = 
+omit =
     /tmp/*
     */tests/*
     */.venv/*
 
 
 [report]
-exclude_lines = 
+exclude_lines =
     if TYPE_CHECKING:

.cursor/rules/core-architecture.mdc

@@ -1,6 +1,6 @@
 ---
-description: 
-globs: 
+description:
+globs:
 alwaysApply: false
 ---
 # Core Architecture

.cursor/rules/integrations-guide.mdc

@@ -1,6 +1,6 @@
 ---
-description: 
-globs: 
+description:
+globs:
 alwaysApply: false
 ---
 # Integrations Guide
@@ -133,10 +133,10 @@ from sentry_sdk.integrations import Integration
 
 class MyIntegration(Integration):
     identifier = "my_integration"
-    
+
     def __init__(self, param=None):
         self.param = param
-    
+
     @staticmethod
     def setup_once():
         # Install hooks, monkey patches, etc.

.cursor/rules/project-overview.mdc

@@ -1,6 +1,6 @@
 ---
-description: 
-globs: 
+description:
+globs:
 alwaysApply: false
 ---
 # Sentry Python SDK - Project Overview

.cursor/rules/quick-reference.mdc

@@ -1,6 +1,6 @@
 ---
-description: 
-globs: 
+description:
+globs:
 alwaysApply: false
 ---
 # Quick Reference
@@ -44,7 +44,7 @@ tox -e py3.12-django-v5.2.3
 
 ### Code Quality
 
-Our `linters` tox environment runs `black` for formatting, `flake8` for linting and `mypy` for type checking.
+Our `linters` tox environment runs `ruff-format` for formatting, `ruff-check` for linting and `mypy` for type checking.
 
 ```bash
 tox -e linters

.cursor/rules/testing-guide.mdc

@@ -1,6 +1,6 @@
 ---
-description: 
-globs: 
+description:
+globs:
 alwaysApply: false
 ---
 # Testing Guide
@@ -65,10 +65,10 @@ def test_flask_integration(sentry_init, capture_events):
     # Test setup
     sentry_init(integrations=[FlaskIntegration()])
     events = capture_events()
-    
+
     # Test execution
     # ... test code ...
-    
+
     # Assertions
     assert len(events) == 1
     assert events[0]["exception"]["values"][0]["type"] == "ValueError"

.git-blame-ignore-revs

@@ -0,0 +1,3 @@
+# Formatting commits to ignore in git blame
+afea4a017bf13f78e82f725ea9d6a56a8e02cb34
+23a340a9dca60eea36de456def70c00952a33556

.pre-commit-config.yaml

@@ -1,31 +1,9 @@
 # See https://pre-commit.com for more information
 # See https://pre-commit.com/hooks.html for more hooks
 repos:
--   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+-   repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.13.2
     hooks:
-    -   id: trailing-whitespace
-    -   id: end-of-file-fixer
-
--   repo: https://github.com/psf/black
-    rev: 24.1.0
-    hooks:
-    -   id: black
-        exclude: ^(.*_pb2.py|.*_pb2_grpc.py)
-
--   repo: https://github.com/pycqa/flake8
-    rev: 5.0.4
-    hooks:
-    -   id: flake8
-        additional_dependencies:
-            [
-                flake8-pyproject,
-                flake8-bugbear,
-                pep8-naming,
-            ]
-
-# Disabled for now, because it lists a lot of problems.
-#-   repo: https://github.com/pre-commit/mirrors-mypy
-#    rev: 'v0.931'
-#    hooks:
-#    -   id: mypy
+    -   id: ruff-check
+        args: [--fix]
+    -   id: ruff-format

codecov.yml

@@ -19,9 +19,9 @@ comment:
   # Comments will only post when coverage changes. Furthermore, if a comment
   # already exists, and a newer commit results in no coverage change for the
   # entire pull, the comment will be deleted.
-  require_changes: true  
+  require_changes: true
   require_base: true  # must have a base report to post
   require_head: true  # must have a head report to post
 
 github_checks:
-  annotations: false
+  annotations: false

pyproject.toml

@@ -1,19 +1,3 @@
-#
-# Tool: Black
-#
-
-[tool.black]
-# 'extend-exclude' excludes files or directories in addition to the defaults
-extend-exclude = '''
-# A regex preceded with ^/ will apply only to files and directories
-# in the root of the project.
-(
-    .*_pb2.py  # exclude autogenerated Protocol Buffer files anywhere in the project
-    | .*_pb2_grpc.py  # exclude autogenerated Protocol Buffer files anywhere in the project
-)
-'''
-
-
 #
 # Tool: Coverage
 #
@@ -196,29 +180,48 @@ module = "agents.*"
 ignore_missing_imports = true
 
 #
-# Tool: Flake8
+# Tool: Ruff (linting and formatting)
 #
 
-[tool.flake8]
-extend-ignore = [
-  # Handled by black (Whitespace before ':' -- handled by black)
-  "E203",
-  # Handled by black (Line too long)
-  "E501",
-  # Sometimes not possible due to execution order (Module level import is not at top of file)
-  "E402",
-  # I don't care (Do not assign a lambda expression, use a def)
-  "E731",
-  # does not apply to Python 2 (redundant exception types by flake8-bugbear)
-  "B014",
-  # I don't care (Lowercase imported as non-lowercase by pep8-naming)
-  "N812",
-  # is a worse version of and conflicts with B902 (first argument of a classmethod should be named cls)
-  "N804",
+[tool.ruff]
+# Target Python 3.7+ (minimum version supported by ruff)
+target-version = "py37"
+
+# Exclude files and directories
+extend-exclude = [
+    "*_pb2.py",     # Protocol Buffer files (covers all pb2 files including grpc_test_service_pb2.py)
+    "*_pb2_grpc.py", # Protocol Buffer files (covers all pb2_grpc files including grpc_test_service_pb2_grpc.py)
+    "checkouts",    # From flake8
+    "lol*",         # From flake8
 ]
-extend-exclude = ["checkouts", "lol*"]
-exclude = [
-  # gRCP generated files
-  "grpc_test_service_pb2.py",
-  "grpc_test_service_pb2_grpc.py",
+
+[tool.ruff.lint]
+# Match flake8's default rule selection exactly
+# Flake8 by default only enables E and W (pycodestyle) + F (pyflakes)
+select = [
+    "E",   # pycodestyle errors (same as flake8 default)
+    "W",   # pycodestyle warnings (same as flake8 default)
+    "F",   # Pyflakes (same as flake8 default)
+    # Note: B and N rules are NOT enabled by default in flake8
+    # They were only active through the plugins, which may not have been fully enabled
 ]
+
+# Use ONLY the same ignores as the original flake8 config + compatibility for this codebase
+ignore = [
+    "E203", # Whitespace before ':'
+    "E501", # Line too long
+    "E402", # Module level import not at top of file
+    "E731", # Do not assign a lambda expression, use a def
+    "B014", # Redundant exception types
+    "N812", # Lowercase imported as non-lowercase
+    "N804", # First argument of classmethod should be named cls
+
+    # Additional ignores for codebase compatibility
+    "F401", # Unused imports - many in TYPE_CHECKING blocks used for type comments
+    "E721", # Use isinstance instead of type() == - existing pattern in this codebase
+]
+
+[tool.ruff.format]
+# ruff format already excludes the same files as specified in extend-exclude
+# Ensure Python 3.7 compatibility - avoid using Python 3.9+ syntax features
+skip-magic-trailing-comma = false