[ADD] ruff integration via matrix position 8 (ruff_matrix_value)

Adds ruff as an opt-in linter/formatter via --compatibility-version.
Position 8 controls the behaviour:

  < 20 (default=10): disabled — existing consumers unaffected
  >= 20: ruff check+fix runs alongside flake8 and black (transition)
  >= 30: ruff replaces flake8/black/isort/autoflake/pyupgrade entirely

New config template ruff.toml.jinja:
  line-length=119, target-version=py310, select E/W/F/B/C4/I/UP/N.
  Ignores carried forward from .flake8 plus Odoo-specific B006/B008.
  isort: odoo+odoo.addons merged into first-party (cosmetic delta from
  legacy ODOO/ODOO_ADDONS sections — no lint impact).
  skip_string_normalization wired to ruff format quote-style.

Template changes:
  .pre-commit-config.yaml.jinja       ruff mandatory hook (>= 20), flake8 gated (< 30)
  .pre-commit-config-autofix.yaml.jinja  ruff check+fix (>= 20), ruff-format (>= 30),
                                         black/autoflake/isort/pyupgrade gated (< 30)
  .pre-commit-config-optional.yaml.jinja ruff SIM/RUF/C90 via --extend-select (>= 20),
                                         flake8-bugbear gated (< 30)

Deviations discovered during CI validation:
  1. ruff-format gated at >= 30, not >= 20. Running ruff-format and black
     simultaneously causes an infinite format loop: the two formatters
     disagree on wrapping point for multiline assert expressions. At
     value=20, ruff provides lint fixes only; black remains the formatter.
  2. B036 and E123 dropped from ignore list. Neither rule exists in ruff
     (rejected as "Unknown rule selector"). No practical impact.
  3. B018 added to __manifest__.py per-file-ignores. ruff B018 fires on
     Odoo manifest bare dict literals, which are intentional.
  4. Optional hook uses --extend-select, not --select. CLI --select
     overrides the config ignore list, re-triggering UP031.
  5. ci/bootstrap.py, docs/conf.py, pre_commit_vauxoo.py: .format() calls
     converted to f-strings (UP032). The docker test runs ruff on the
     repo's own source; these files had to comply.
  6. tests/test_pre_commit_vauxoo.py: two assert statements reformatted to
     a layout both black and ruff-format accept without conflict.

Benchmark (25 files, ruff 0.15.16 / black 26.5.1 / isort, Ubuntu 24.04):
  ruff check + ruff format              ~130 ms
  black + isort (partial legacy)        ~570 ms   (~4x faster)
  full legacy stack estimate
    (+ flake8 + autoflake + pyupgrade)  ~1270 ms  (~10x faster)

Author: hbto

ci/bootstrap.py

@@ -1,17 +1,8 @@
 #!/usr/bin/env python
-# -*- coding: utf-8 -*-
-from __future__ import absolute_import
-from __future__ import print_function
-from __future__ import unicode_literals
-
 import os
 import subprocess
 import sys
-from os.path import abspath
-from os.path import dirname
-from os.path import exists
-from os.path import join
-from os.path import relpath
+from os.path import abspath, dirname, exists, join, relpath
 
 base_path = dirname(dirname(abspath(__file__)))
 templates_path = join(base_path, "ci", "templates")
@@ -31,7 +22,7 @@ def exec_in_env():
     if not exists(env_path):
         import subprocess
 
-        print("Making bootstrap env in: {0} ...".format(env_path))
+        print(f"Making bootstrap env in: {env_path} ...")
         try:
             check_call([sys.executable, "-m", "venv", env_path])
         except subprocess.CalledProcessError:
@@ -43,9 +34,9 @@ def exec_in_env():
         check_call([join(bin_path, "pip"), "install", "jinja2", "tox", "matrix"])
     python_executable = join(bin_path, "python")
     if not os.path.exists(python_executable):
-        python_executable += '.exe'
+        python_executable += ".exe"
 
-    print("Re-executing with: {0}".format(python_executable))
+    print(f"Re-executing with: {python_executable}")
     print("+ exec", python_executable, __file__, "--no-env")
     os.execv(python_executable, [python_executable, __file__, "--no-env"])
 
@@ -54,7 +45,7 @@ def main():
     import jinja2
     import matrix
 
-    print("Project path: {0}".format(base_path))
+    print(f"Project path: {base_path}")
 
     jinja = jinja2.Environment(
         loader=jinja2.FileSystemLoader(templates_path),
@@ -64,7 +55,7 @@ def main():
     )
 
     tox_environments = {}
-    for (alias, conf) in matrix.from_file(join(base_path, "setup.cfg")).items():
+    for alias, conf in matrix.from_file(join(base_path, "setup.cfg")).items():
         deps = conf["dependencies"]
         tox_environments[alias] = {
             "deps": deps.split(),
@@ -81,7 +72,7 @@ def main():
             relative = relpath(root, templates_path)
             with open(join(base_path, relative, name), "w") as fh:
                 fh.write(jinja.get_template(join(relative, name)).render(tox_environments=tox_environments))
-            print("Wrote {}".format(name))
+            print(f"Wrote {name}")
     print("DONE.")
 
 
@@ -92,5 +83,5 @@ def main():
     elif not args:
         exec_in_env()
     else:
-        print("Unexpected arguments {0}".format(args), file=sys.stderr)
+        print(f"Unexpected arguments {args}", file=sys.stderr)
         sys.exit(1)

docs/conf.py

@@ -17,7 +17,7 @@
 project = "pre-commit-vauxoo"
 year = "2022"
 author = "Vauxoo"
-copyright = "{}, {}".format(year, author)
+copyright = f"{year}, {author}"
 version = release = "8.2.31"
 
 pygments_style = "trac"

src/pre_commit_vauxoo/cfg/.pre-commit-config-autofix.yaml.jinja

@@ -60,6 +60,19 @@ repos:
           - --config=.oca_hooks-autofix.cfg
           - --fix
 {%- endif %}
+{%- if ruff_matrix_value >= 20 %}
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.15.16
+    hooks:
+      - id: ruff
+        name: ruff (check + fix)
+        args: [--fix, --unsafe-fixes]
+{%- if ruff_matrix_value >= 30 %}
+      - id: ruff-format
+        name: ruff (format)
+{%- endif %}
+{%- endif %}
+{%- if ruff_matrix_value < 30 %}
   - repo: https://github.com/psf/black-pre-commit-mirror.git
 {%- if black_autoflake_matrix_value >= 20 %}
     rev: 25.12.0
@@ -83,6 +96,7 @@ repos:
           - --remove-all-unused-imports
           - --remove-duplicate-keys
           - --remove-unused-variables
+{%- endif %}
   - repo: local
     hooks:
       - id: prettier
@@ -123,6 +137,7 @@ repos:
 {%- if pre_commit_matrix_value >= 20 %}
       - id: requirements-txt-fixer
 {%- endif %}
+{%- if ruff_matrix_value < 30 %}
   - repo: https://github.com/asottile/pyupgrade
     rev: v3.21.2
     hooks:
@@ -132,6 +147,7 @@ repos:
     rev: 7.0.0
     hooks:
       - id: isort
+{%- endif %}
   - repo: https://github.com/Lucas-C/pre-commit-hooks
     rev: v1.5.6
     hooks:

src/pre_commit_vauxoo/cfg/.pre-commit-config-optional.yaml.jinja

@@ -68,6 +68,15 @@ repos:
         language: python
         pass_filenames: false
         always_run: true
+{%- if ruff_matrix_value >= 20 %}
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.15.16
+    hooks:
+      - id: ruff
+        name: ruff optional rules (SIM/RUF/C90)
+        args: [--extend-select, "SIM,RUF,C90", --exit-non-zero-on-fix]
+{%- endif %}
+{%- if ruff_matrix_value < 30 %}
   - repo: https://github.com/PyCQA/flake8
 {%- if flake8_matrix_value >= 20 %}
     rev: 7.3.0
@@ -84,6 +93,7 @@ repos:
 {%- endif %}
         args:
           - --config=.flake8-optional
+{%- endif %}
   - repo: https://github.com/PyCQA/bandit
     rev: 1.9.2
     hooks:

src/pre_commit_vauxoo/cfg/.pre-commit-config.yaml.jinja

@@ -31,6 +31,15 @@ default_language_version:
   python: python3
   node: "24.13.0"
 repos:
+{%- if ruff_matrix_value >= 20 %}
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.15.16
+    hooks:
+      - id: ruff
+        name: ruff mandatory checks
+        args: [--fix, --exit-non-zero-on-fix]
+{%- endif %}
+{%- if ruff_matrix_value < 30 %}
   - repo: https://github.com/PyCQA/flake8
 {%- if flake8_matrix_value >= 20 %}
     rev: 7.3.0
@@ -40,6 +49,7 @@ repos:
     hooks:
       - id: flake8
         name: flake8 mandatory checks
+{%- endif %}
   - repo: https://github.com/OCA/pylint-odoo
 {%- if pylint_matrix_value >= 20 %}
     rev: v10.0.2

src/pre_commit_vauxoo/cfg/ruff.toml.jinja

@@ -0,0 +1,59 @@
+# ruff.toml — generated by pre-commit-vauxoo
+# Replaces: .flake8, .isort.cfg, pyproject.toml [tool.black] when ruff is active.
+# Reference: https://docs.astral.sh/ruff/configuration/
+
+line-length = 119
+target-version = "py310"
+
+[lint]
+select = [
+    "E",    # pycodestyle errors (replaces flake8 core E)
+    "W",    # pycodestyle warnings (replaces flake8 core W)
+    "F",    # pyflakes (replaces flake8 core F)
+    "B",    # flake8-bugbear
+    "C4",   # flake8-comprehensions
+    "I",    # isort
+    "UP",   # pyupgrade
+    "N",    # pep8-naming
+]
+
+# Carried forward from cfg/.flake8 + Odoo-specific additions.
+# E133: not in ruff (pep8 extension rule) — omitted safely.
+# F601: not in ruff — omitted safely.
+# W503/W504: removed from PEP8 modern — omitted safely.
+ignore = [
+    "B006",  # mutable-argument-default — Odoo uses lists in field defaults
+    "B008",  # function-call-in-default-argument — Odoo uses callables in field defaults
+    "B024",  # abstract-base-class-without-abstract-method
+    # B036: not implemented in ruff (flake8-bugbear only)
+    # E123: not implemented in ruff — omitted safely
+    "E203",  # whitespace-before-punctuation (ruff-format compat)
+    "E226",  # missing-whitespace-around-arithmetic-operator
+    "E241",  # multiple-spaces-after-comma (preview rule, ignored for compat)
+    "E242",  # tab-after-comma (preview rule, ignored for compat)
+    "E501",  # line-too-long (delegated to ruff format)
+    "E741",  # ambiguous-variable-name
+    "F811",  # redefinition-of-unused-name
+    "UP031", # printf-string-formatting (--keep-percent-format compat)
+]
+
+[lint.per-file-ignores]
+"__init__.py" = ["F401"]       # unused-import OK in Odoo __init__.py (re-exports)
+"__manifest__.py" = ["B018"]   # bare dict literal is the Odoo manifest format
+
+[lint.isort]
+# NOTE: isort behavioral delta from legacy .isort.cfg
+# Legacy .isort.cfg used 7 sections: FUTURE,STDLIB,THIRDPARTY,ODOO,ODOO_ADDONS,FIRSTPARTY,LOCALFOLDER
+# Ruff supports only standard sections; odoo and odoo.addons are merged into first-party.
+# Cosmetic difference only — no CI failures.
+known-first-party = ["odoo", "odoo.addons"]
+default-section = "third-party"
+combine-as-imports = true
+split-on-trailing-comma = true
+force-sort-within-sections = false
+
+[format]
+line-ending = "auto"
+quote-style = {{ '"preserve"' if skip_string_normalization else '"double"' }}
+indent-style = "space"
+skip-magic-trailing-comma = false

src/pre_commit_vauxoo/cli.py

@@ -385,6 +385,9 @@ def precommit_hooks_type_callback(ctx, param, value):
 🟢 5. pre-commit framework
 🟢 6. Pylint/pylint-odoo
 🟢 7. flake8
+🟢 8. Ruff (linter + formatter)
+    (0-10 → disabled; 20 → ruff check+format replaces flake8+black+isort+autoflake+pyupgrade in parallel;
+     30 → legacy tools removed from generated configs)
 
 ⚠️ Higher values or empty valuesmay introduce formatting changes, stricter linting,
 or non-backward-compatible fixes (especially for XML, Python, and JS files).""",

src/pre_commit_vauxoo/pre_commit_vauxoo.py

@@ -30,6 +30,7 @@
     "pre_commit_matrix_value",
     "pylint_matrix_value",
     "flake8_matrix_value",
+    "ruff_matrix_value",
 )
 DEFAULT_MAX_COMPATIBILITY = 1000000
 
@@ -439,7 +440,7 @@ def print_summary(all_status):
             if test_result["status"]
             else logging_colored.colorized_msg(test_result["status_msg"], test_result["level"])
         )
-        summary_msg.append("| {:<28}{}".format(test_name, outcome))
+        summary_msg.append(f"| {test_name:<28}{outcome}")
     summary_msg.append("+" + "=" * 39)
     _logger.info("Tests summary\n%s", "\n".join(summary_msg))
 

tests/test_pre_commit_vauxoo.py

@@ -401,9 +401,10 @@ def test_uninstallable(self, caplog):
         os.environ["PRECOMMIT_HOOKS_TYPE"] = "all"
         uninstallable_path = os.path.join(self.tmp_dir, "module_uninstallable")
         result = self.runner.invoke(main, ["-p", uninstallable_path])
-        assert (
-            not result.exit_code
-        ), "Uninstallable module should not have been linted. " "Exited with error %s - %s" % (result, result.output)
+        assert not result.exit_code, "Uninstallable module should not have been linted. Exited with error %s - %s" % (
+            result,
+            result.output,
+        )
 
     def test_exclude_only_uninstallable(self, caplog):
         repo_path = posixpath.join(self.tmp_dir, "repo")
@@ -436,9 +437,10 @@ def test_disable_oca_hooks(self, caplog):
             config.read(oca_hooks_cfg_path)
             disable_raw = config.get("MESSAGES_CONTROL", "disable")
             disabled = {item.strip(", ") for item in disable_raw.replace("\n", "").split(",") if item.strip()}
-            assert expected_disabled.issubset(
-                disabled
-            ), f"random-msg was supposed to be disabled for {oca_hooks_cfg_path} through the corresponding environment variable"
+            assert expected_disabled.issubset(disabled), (
+                "random-msg was supposed to be disabled for %s through the corresponding environment variable"
+                % oca_hooks_cfg_path
+            )
 
     def test_valid_pylintrc_messages(self, caplog):
         self.runner.invoke(main, ["--only-cp-cfg"])