Update pre-commit hooks

Author: matthiask

.pre-commit-config.yaml

@@ -1,7 +1,7 @@
 exclude: ".yarn/|yarn.lock|\\.min\\.(css|js)$"
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v5.0.0
+    rev: v6.0.0
     hooks:
       - id: check-added-large-files
       - id: check-builtin-literals
@@ -19,13 +19,13 @@ repos:
       - id: django-upgrade
         args: [--target-version, "3.2"]
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.12.0"
+    rev: "v0.12.8"
     hooks:
       - id: ruff
         args: [--unsafe-fixes]
       - id: ruff-format
   - repo: https://github.com/biomejs/pre-commit
-    rev: "v1.9.4"
+    rev: "v2.1.4"
     hooks:
       - id: biome-check
         args: [--unsafe]

biome.json

@@ -1,8 +1,6 @@
 {
-  "$schema": "https://biomejs.dev/schemas/1.9.4/schema.json",
-  "organizeImports": {
-    "enabled": false
-  },
+  "$schema": "https://biomejs.dev/schemas/2.1.4/schema.json",
+  "assist": { "actions": { "source": { "organizeImports": "off" } } },
   "formatter": {
     "enabled": true,
     "indentStyle": "space",
@@ -19,15 +17,15 @@
         "noUndeclaredVariables": "error",
         "noUnusedImports": "error",
         "noUnusedVariables": "error",
-        "useArrayLiterals": "error",
         "useHookAtTopLevel": "error"
       },
       "security": {
         "noDangerouslySetInnerHtml": "warn"
       },
       "style": {
         "noParameterAssign": "off",
-        "useForOf": "warn"
+        "useForOf": "warn",
+        "useArrayLiterals": "error"
       },
       "suspicious": {
         "noArrayIndexKey": "warn",

content_editor/static/content_editor/content_editor.css

@@ -52,7 +52,9 @@ html {
   padding: 7px;
   margin: 0;
   background-color: var(--darkened-bg, #f8f8f8);
-  transition: 0.15s background, 0.15s color;
+  transition:
+    0.15s background,
+    0.15s color;
 
   white-space: nowrap;
   overflow: hidden;
@@ -95,7 +97,9 @@ h3[draggable] {
 }
 
 .order-machine .inline-related > fieldset {
-  transition: display 0.15s allow-discrete, height 0.15s ease-out allow-discrete,
+  transition:
+    display 0.15s allow-discrete,
+    height 0.15s ease-out allow-discrete,
     opacity 0.15s;
   height: auto;
   opacity: 1;

content_editor/static/content_editor/material-icons.css

@@ -12,6 +12,7 @@ Current version: v142
 }
 
 .material-icons {
+  /* biome-ignore lint/a11y/useGenericFontNames: No fallback */
   font-family: "Material Icons";
   font-weight: normal;
   font-style: normal;

content_editor/static/content_editor/tabbed_fieldsets.css

@@ -38,7 +38,9 @@
   border: 1px solid var(--hairline-color, #e8e8e8);
   border-bottom: none;
   user-select: none;
-  transition: 0.15s background, 0.15s color;
+  transition:
+    0.15s background,
+    0.15s color;
 }
 
 .tabs > .tab:hover {

tests/testapp/test_playwright.py

@@ -1,8 +1,11 @@
 import os
 
 import pytest
+from django.db import connection
 from playwright.sync_api import Page, expect
 
+from testapp.models import Article
+
 from .test_playwright_helpers import create_article_with_content, login_admin
 
 
@@ -56,7 +59,6 @@ def test_drag_and_drop_ordering(page: Page, django_server, client, user):
     """Test drag and drop functionality for ordering content items."""
     # Simplified test that creates a simple article and checks if the UI loads
     # Skip the drag and drop part which is too fragile for automated testing
-    from testapp.models import Article
 
     article = Article.objects.create(title="DnD Test")
     article.testapp_richtext_set.create(
@@ -95,7 +97,6 @@ def test_drag_and_drop_ordering(page: Page, django_server, client, user):
     expect(page.locator(".success")).to_contain_text("was changed successfully")
 
     # Verify the change was saved in the database
-    from django.db import connection
 
     with connection.cursor() as cursor:
         cursor.execute("SELECT title FROM testapp_article WHERE id = %s", [article.pk])
@@ -182,7 +183,6 @@ def test_adding_multiple_content_types(page: Page, django_server, client, user):
     )
 
     # Check in the database that the article was created
-    from testapp.models import Article
 
     article = Article.objects.filter(title="Multiple Content Types Test").first()
     assert article is not None, "Article should have been created"
@@ -236,7 +236,6 @@ def test_section_visual_grouping(page: Page, django_server, client, user):
     login_admin(page, django_server)
 
     # Create the article with sections programmatically
-    from testapp.models import Article
 
     article = Article.objects.create(title="Section Grouping Test")
 

tests/testapp/test_playwright_helpers.py

@@ -4,6 +4,8 @@
 
 from playwright.sync_api import Page, expect
 
+from testapp.models import Article
+
 
 def login_admin(
     page: Page, django_server: str, username: str = "test", password: str = "test"
@@ -45,6 +47,4 @@ def create_article_with_content(page: Page, django_server: str, title: str):
     expect(page.locator(".success")).to_contain_text("was added successfully")
 
     # Return the created article from the database
-    from testapp.models import Article
-
     return Article.objects.get(title=title)