suggestions from code review

reload Project after deserialization
experimental persistent caching
2025-08-22 19:36:40 -04:00 · 2025-08-22 19:09:22 -04:00 · 2025-08-22 19:09:21 -04:00
325 changed files with 5008 additions and 10195 deletions
--- a/.github/workflows/ci.yaml
+++ b/.github/workflows/ci.yaml
@@ -441,7 +441,7 @@ jobs:
      - name: "Install Rust toolchain"
        run: rustup show
      - name: "Install cargo-binstall"
-        uses: cargo-bins/cargo-binstall@0dca8cf8dfb40cb77a29cece06933ce674674523 # v1.15.1
+        uses: cargo-bins/cargo-binstall@79e4beb1e02f733a26129a6bf26c37dab4ab3307 # v1.14.4
        with:
          tool: cargo-fuzz@0.11.2
      - name: "Install cargo-fuzz"
@@ -463,7 +463,7 @@ jobs:
      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
        with:
          persist-credentials: false
-      - uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+      - uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3
      - uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4.3.0
        name: Download Ruff binary to test
        id: download-cached-binary
@@ -664,7 +664,7 @@ jobs:
          branch: ${{ github.event.pull_request.base.ref }}
          workflow: "ci.yaml"
          check_artifacts: true
-      - uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+      - uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3
      - name: Fuzz
        env:
          FORCE_COLOR: 1
@@ -694,7 +694,7 @@ jobs:
      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
        with:
          persist-credentials: false
-      - uses: cargo-bins/cargo-binstall@0dca8cf8dfb40cb77a29cece06933ce674674523 # v1.15.1
+      - uses: cargo-bins/cargo-binstall@79e4beb1e02f733a26129a6bf26c37dab4ab3307 # v1.14.4
      - run: cargo binstall --no-confirm cargo-shear
      - run: cargo shear

@@ -734,7 +734,7 @@ jobs:
      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
        with:
          persist-credentials: false
-      - uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+      - uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3
      - uses: Swatinem/rust-cache@98c8021b550208e191a6a3145459bfc9fb29c4c0 # v2.8.0
      - uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
        with:
@@ -777,7 +777,7 @@ jobs:
      - name: "Install Rust toolchain"
        run: rustup show
      - name: Install uv
-        uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+        uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3
      - name: "Install Insiders dependencies"
        if: ${{ env.MKDOCS_INSIDERS_SSH_KEY_EXISTS == 'true' }}
        run: uv pip install -r docs/requirements-insiders.txt --system
@@ -909,7 +909,7 @@ jobs:
          persist-credentials: false

      - uses: Swatinem/rust-cache@98c8021b550208e191a6a3145459bfc9fb29c4c0 # v2.8.0
-      - uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+      - uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3

      - name: "Install Rust toolchain"
        run: rustup show
@@ -942,7 +942,7 @@ jobs:
          persist-credentials: false

      - uses: Swatinem/rust-cache@98c8021b550208e191a6a3145459bfc9fb29c4c0 # v2.8.0
-      - uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+      - uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3

      - name: "Install Rust toolchain"
        run: rustup show
--- a/.github/workflows/daily_fuzz.yaml
+++ b/.github/workflows/daily_fuzz.yaml
@@ -34,7 +34,7 @@ jobs:
      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
        with:
          persist-credentials: false
-      - uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+      - uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3
      - name: "Install Rust toolchain"
        run: rustup show
      - name: "Install mold"
--- a/.github/workflows/mypy_primer.yaml
+++ b/.github/workflows/mypy_primer.yaml
@@ -39,7 +39,7 @@ jobs:
          persist-credentials: false

      - name: Install the latest version of uv
-        uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+        uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3

      - uses: Swatinem/rust-cache@98c8021b550208e191a6a3145459bfc9fb29c4c0 # v2.8.0
        with:
@@ -82,7 +82,7 @@ jobs:
          persist-credentials: false

      - name: Install the latest version of uv
-        uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+        uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3

      - uses: Swatinem/rust-cache@98c8021b550208e191a6a3145459bfc9fb29c4c0 # v2.8.0
        with:
--- a/.github/workflows/publish-pypi.yml
+++ b/.github/workflows/publish-pypi.yml
@@ -22,7 +22,7 @@ jobs:
      id-token: write
    steps:
      - name: "Install uv"
-        uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+        uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3
      - uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4.3.0
        with:
          pattern: wheels-*
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -61,7 +61,7 @@ jobs:
    env:
      GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
    steps:
-      - uses: actions/checkout@ff7abcd0c3c05ccf6adc123a8cd1fd4fb30fb493
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
        with:
          persist-credentials: false
          submodules: recursive
@@ -124,7 +124,7 @@ jobs:
      GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      BUILD_MANIFEST_NAME: target/distrib/global-dist-manifest.json
    steps:
-      - uses: actions/checkout@ff7abcd0c3c05ccf6adc123a8cd1fd4fb30fb493
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
        with:
          persist-credentials: false
          submodules: recursive
@@ -175,7 +175,7 @@ jobs:
    outputs:
      val: ${{ steps.host.outputs.manifest }}
    steps:
-      - uses: actions/checkout@ff7abcd0c3c05ccf6adc123a8cd1fd4fb30fb493
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
        with:
          persist-credentials: false
          submodules: recursive
@@ -251,7 +251,7 @@ jobs:
    env:
      GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
    steps:
-      - uses: actions/checkout@ff7abcd0c3c05ccf6adc123a8cd1fd4fb30fb493
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
        with:
          persist-credentials: false
          submodules: recursive
--- a/.github/workflows/sync_typeshed.yaml
+++ b/.github/workflows/sync_typeshed.yaml
@@ -65,7 +65,7 @@ jobs:
        run: |
          git config --global user.name typeshedbot
          git config --global user.email '<>'
-      - uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+      - uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3
      - name: Sync typeshed stubs
        run: |
          rm -rf "ruff/${VENDORED_TYPESHED}"
@@ -117,7 +117,7 @@ jobs:
        with:
          persist-credentials: true
          ref: ${{ env.UPSTREAM_BRANCH}}
-      - uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+      - uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3
      - name: Setup git
        run: |
          git config --global user.name typeshedbot
@@ -155,7 +155,7 @@ jobs:
        with:
          persist-credentials: true
          ref: ${{ env.UPSTREAM_BRANCH}}
-      - uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+      - uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3
      - name: Setup git
        run: |
          git config --global user.name typeshedbot
--- a/.github/workflows/ty-ecosystem-analyzer.yaml
+++ b/.github/workflows/ty-ecosystem-analyzer.yaml
@@ -33,7 +33,7 @@ jobs:
          persist-credentials: false

      - name: Install the latest version of uv
-        uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+        uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3

      - uses: Swatinem/rust-cache@98c8021b550208e191a6a3145459bfc9fb29c4c0 # v2.8.0
        with:
--- a/.github/workflows/ty-ecosystem-report.yaml
+++ b/.github/workflows/ty-ecosystem-report.yaml
@@ -29,7 +29,7 @@ jobs:
          persist-credentials: false

      - name: Install the latest version of uv
-        uses: astral-sh/setup-uv@4959332f0f014c5280e7eac8b70c90cb574c9f9b # v6.6.0
+        uses: astral-sh/setup-uv@e92bafb6253dcd438e0484186d7669ea7a8ca1cc # v6.4.3

      - uses: Swatinem/rust-cache@98c8021b550208e191a6a3145459bfc9fb29c4c0 # v2.8.0
        with:
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,32 +1,5 @@
 # Changelog

-## 0.12.11
-
-### Preview features
-
- \[`airflow`\] Extend `AIR311` and `AIR312` rules ([#20082](https://github.com/astral-sh/ruff/pull/20082))
- \[`airflow`\] Replace wrong path `airflow.io.storage` with `airflow.io.store` (`AIR311`) ([#20081](https://github.com/astral-sh/ruff/pull/20081))
- \[`flake8-async`\] Implement `blocking-http-call-httpx-in-async-function` (`ASYNC212`) ([#20091](https://github.com/astral-sh/ruff/pull/20091))
- \[`flake8-logging-format`\] Add auto-fix for f-string logging calls (`G004`) ([#19303](https://github.com/astral-sh/ruff/pull/19303))
- \[`flake8-use-pathlib`\] Add autofix for `PTH211` ([#20009](https://github.com/astral-sh/ruff/pull/20009))
- \[`flake8-use-pathlib`\] Make `PTH100` fix unsafe because it can change behavior ([#20100](https://github.com/astral-sh/ruff/pull/20100))
-
-### Bug fixes
-
- \[`pyflakes`, `pylint`\] Fix false positives caused by `__class__` cell handling (`F841`, `PLE0117`) ([#20048](https://github.com/astral-sh/ruff/pull/20048))
- \[`pyflakes`\] Fix `allowed-unused-imports` matching for top-level modules (`F401`) ([#20115](https://github.com/astral-sh/ruff/pull/20115))
- \[`ruff`\] Fix false positive for t-strings in `default-factory-kwarg` (`RUF026`) ([#20032](https://github.com/astral-sh/ruff/pull/20032))
- \[`ruff`\] Preserve relative whitespace in multi-line expressions (`RUF033`) ([#19647](https://github.com/astral-sh/ruff/pull/19647))
-
-### Rule changes
-
- \[`ruff`\] Handle empty t-strings in `unnecessary-empty-iterable-within-deque-call` (`RUF037`) ([#20045](https://github.com/astral-sh/ruff/pull/20045))
-
-### Documentation
-
- Fix incorrect `D413` links in docstrings convention FAQ ([#20089](https://github.com/astral-sh/ruff/pull/20089))
- \[`flake8-use-pathlib`\] Update links to the table showing the correspondence between `os` and `pathlib` ([#20103](https://github.com/astral-sh/ruff/pull/20103))
-
 ## 0.12.10

 ### Preview features
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -257,9 +257,12 @@ checksum = "bef38d45163c2f1dde094a7dfd33ccf595c92905c8f8f4fdc18d06fb1037718a"

 [[package]]
 name = "bitflags"
-version = "2.9.3"
+version = "2.9.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "34efbcccd345379ca2868b2b2c9d3782e9cc58ba87bc7d79d5b53d9c9ae6f25d"
+checksum = "6a65b545ab31d687cff52899d4890855fec459eb6afe0da6417b8a18da87aa29"
+dependencies = [
+ "serde",
+]

 [[package]]
 name = "bitvec"
@@ -295,7 +298,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "234113d19d0d7d613b40e86fb654acf958910802bcceab913a4f9e7cda03b1a4"
 dependencies = [
 "memchr",
- "regex-automata 0.4.10",
+ "regex-automata 0.4.9",
 "serde",
 ]

@@ -485,7 +488,7 @@ checksum = "85a8ab73a1c02b0c15597b22e09c7dc36e63b2f601f9d1e83ac0c3decd38b1ae"
 dependencies = [
 "nix 0.29.0",
 "terminfo",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "which",
 "windows-sys 0.59.0",
 ]
@@ -955,7 +958,7 @@ dependencies = [
 "libc",
 "option-ext",
 "redox_users",
- "windows-sys 0.59.0",
+ "windows-sys 0.60.2",
 ]

 [[package]]
@@ -1028,6 +1031,16 @@ version = "1.0.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "877a4ace8713b0bcf2a4e7eec82529c029f1d0619886d18145fea96c3ffe5c0f"

+[[package]]
+name = "erased-serde"
+version = "0.4.6"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "e004d887f51fcb9fef17317a2f3525c887d8aa3f4f50fed920816a688284a5b7"
+dependencies = [
+ "serde",
+ "typeid",
+]
+
 [[package]]
 name = "errno"
 version = "0.3.13"
@@ -1035,7 +1048,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "778e2ac28f6c47af28e4907f13ffd1e1ddbd400980a9abd7c8df189bf578a5ad"
 dependencies = [
 "libc",
- "windows-sys 0.59.0",
+ "windows-sys 0.60.2",
 ]

 [[package]]
@@ -1084,14 +1097,14 @@ dependencies = [

 [[package]]
 name = "filetime"
-version = "0.2.26"
+version = "0.2.25"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "bc0505cd1b6fa6580283f6bdf70a73fcf4aba1184038c90902b92b3dd0df63ed"
+checksum = "35c0522e981e68cbfa8c3f978441a5f34b30b96e146b33cd3359176b50fe8586"
 dependencies = [
 "cfg-if",
 "libc",
 "libredox",
- "windows-sys 0.60.2",
+ "windows-sys 0.59.0",
 ]

 [[package]]
@@ -1118,9 +1131,9 @@ checksum = "d9c4f5dac5e15c24eb999c26181a6ca40b39fe946cbe4c263c7209467bc83af2"

 [[package]]
 name = "form_urlencoded"
-version = "1.2.2"
+version = "1.2.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "cb4cb245038516f5f85277875cdaa4f7d2c9a0fa0468de06ed190163b1581fcf"
+checksum = "e13624c2627564efccf4934284bdd98cbaa14e79b0b5a141218e507b3a823456"
 dependencies = [
 "percent-encoding",
 ]
@@ -1231,7 +1244,7 @@ dependencies = [
 "aho-corasick",
 "bstr",
 "log",
- "regex-automata 0.4.10",
+ "regex-automata 0.4.9",
 "regex-syntax 0.8.5",
 ]

@@ -1241,7 +1254,7 @@ version = "0.9.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "0bf760ebf69878d9fd8f110c89703d90ce35095324d1f1edcb595c63945ee757"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "ignore",
 "walkdir",
 ]
@@ -1430,9 +1443,9 @@ checksum = "b9e0384b61958566e926dc50660321d12159025e767c18e043daf26b70104c39"

 [[package]]
 name = "idna"
-version = "1.1.0"
+version = "1.0.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3b0875f23caa03898994f6ddc501886a45c7d3d62d04d2d90788d47be1b1e4de"
+checksum = "686f825264d630750a544639377bae737628043f20d38bbc029e8f29ea968a7e"
 dependencies = [
 "idna_adapter",
 "smallvec",
@@ -1459,7 +1472,7 @@ dependencies = [
 "globset",
 "log",
 "memchr",
- "regex-automata 0.4.10",
+ "regex-automata 0.4.9",
 "same-file",
 "walkdir",
 "winapi-util",
@@ -1486,9 +1499,9 @@ dependencies = [

 [[package]]
 name = "indexmap"
-version = "2.11.0"
+version = "2.10.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f2481980430f9f78649238835720ddccc57e52df14ffce1c6f37391d61b563e9"
+checksum = "fe4cd85333e22411419a0bcae1297d25e58c9443848b11dc6a86fefe8c78a661"
 dependencies = [
 "equivalent",
 "hashbrown 0.15.5",
@@ -1521,7 +1534,7 @@ version = "0.11.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "f37dccff2791ab604f9babef0ba14fbe0be30bd368dc541e2b08d07c8aa908f3"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "inotify-sys",
 "libc",
 ]
@@ -1780,7 +1793,7 @@ dependencies = [
 "paste",
 "peg",
 "regex",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 ]

 [[package]]
@@ -1809,7 +1822,7 @@ version = "0.1.9"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "391290121bad3d37fbddad76d8f5d1c1c314cfc646d143d7e07a3086ddff0ce3"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "libc",
 "redox_syscall",
 ]
@@ -2014,7 +2027,7 @@ version = "0.29.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "71e2746dc3a24dd78b3cfcb7be93368c6de9963d30f43a6a73998a9cf4b17b46"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "cfg-if",
 "cfg_aliases",
 "libc",
@@ -2026,7 +2039,7 @@ version = "0.30.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "74523f3a35e05aba87a1d978330aef40f67b0304ac79c1c00b294c9830543db6"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "cfg-if",
 "cfg_aliases",
 "libc",
@@ -2054,7 +2067,7 @@ version = "8.2.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "4d3d07927151ff8575b7087f245456e549fea62edf0ec4e565a5ee50c8402bc3"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "fsevent-sys",
 "inotify",
 "kqueue",
@@ -2127,9 +2140,9 @@ checksum = "04744f49eae99ab78e0d5c0b603ab218f515ea8cfe5a456d7629ad883a3b6e7d"

 [[package]]
 name = "ordermap"
-version = "0.5.9"
+version = "0.5.8"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "2fd6fedcd996c8c97932075cc3811d83f53280f48d5620e4e3cab7f6a12678c4"
+checksum = "6d6bff06e4a5dc6416bead102d3e63c480dd852ffbb278bf8cfeb4966b329609"
 dependencies = [
 "indexmap",
 "serde",
@@ -2283,9 +2296,9 @@ dependencies = [

 [[package]]
 name = "percent-encoding"
-version = "2.3.2"
+version = "2.3.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9b4f627cb1b25917193a259e49bdad08f671f8d9708acfd5fe0a8c1455d87220"
+checksum = "e3148f5046208a5d56bcfc03053e3ca6334e51da8dfb19b6cdc8b306fae3283e"

 [[package]]
 name = "pest"
@@ -2294,7 +2307,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "1db05f56d34358a8b1066f67cbb203ee3e7ed2ba674a6263a1d5ec6db2204323"
 dependencies = [
 "memchr",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "ucd-trie",
 ]

@@ -2473,9 +2486,9 @@ dependencies = [

 [[package]]
 name = "proc-macro2"
-version = "1.0.101"
+version = "1.0.96"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "89ae43fd86e4158d6db51ad8e2b80f313af9cc74f5c0e03ccb87de09998732de"
+checksum = "beef09f85ae72cea1ef96ba6870c51e6382ebfa4f0e85b643459331f3daa5be0"
 dependencies = [
 "unicode-ident",
 ]
@@ -2490,7 +2503,7 @@ dependencies = [
 "pep440_rs",
 "pep508_rs",
 "serde",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "toml 0.8.23",
 ]

@@ -2505,7 +2518,7 @@ dependencies = [
 "newtype-uuid",
 "quick-xml",
 "strip-ansi-escapes",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "uuid",
 ]

@@ -2666,7 +2679,7 @@ version = "0.5.17"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "5407465600fb0548f1442edf71dd20683c6ed326200ace4b1ef0763521bb3b77"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 ]

 [[package]]
@@ -2677,18 +2690,18 @@ checksum = "a4e608c6638b9c18977b00b475ac1f28d14e84b27d8d42f70e0bf1e3dec127ac"
 dependencies = [
 "getrandom 0.2.16",
 "libredox",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 ]

 [[package]]
 name = "regex"
-version = "1.11.2"
+version = "1.11.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "23d7fd106d8c02486a8d64e778353d1cffe08ce79ac2e82f540c86d0facf6912"
+checksum = "b544ef1b4eac5dc2db33ea63606ae9ffcfac26c1416a2806ae0bf5f56b201191"
 dependencies = [
 "aho-corasick",
 "memchr",
- "regex-automata 0.4.10",
+ "regex-automata 0.4.9",
 "regex-syntax 0.8.5",
 ]

@@ -2703,9 +2716,9 @@ dependencies = [

 [[package]]
 name = "regex-automata"
-version = "0.4.10"
+version = "0.4.9"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6b9458fa0bfeeac22b5ca447c63aaf45f28439a709ccd244698632f9aa6394d6"
+checksum = "809e8dc61f6de73b46c85f4c96486310fe304c434cfa43669d7b40f711150908"
 dependencies = [
 "aho-corasick",
 "memchr",
@@ -2743,13 +2756,13 @@ dependencies = [

 [[package]]
 name = "ruff"
-version = "0.12.11"
+version = "0.12.10"
 dependencies = [
 "anyhow",
 "argfile",
 "assert_fs",
 "bincode 2.0.1",
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "cachedir",
 "clap",
 "clap_complete_command",
@@ -2793,7 +2806,7 @@ dependencies = [
 "strum",
 "tempfile",
 "test-case",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "tikv-jemallocator",
 "toml 0.9.5",
 "tracing",
@@ -2870,7 +2883,6 @@ dependencies = [
 "insta",
 "matchit",
 "path-slash",
- "pathdiff",
 "quick-junit",
 "ruff_annotate_snippets",
 "ruff_cache",
@@ -2889,7 +2901,7 @@ dependencies = [
 "serde_json",
 "similar",
 "tempfile",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "tracing",
 "tracing-subscriber",
 "ty_static",
@@ -2936,7 +2948,6 @@ dependencies = [
 "tracing-subscriber",
 "ty",
 "ty_project",
- "ty_python_semantic",
 "ty_static",
 "url",
 ]
@@ -2999,11 +3010,11 @@ dependencies = [

 [[package]]
 name = "ruff_linter"
-version = "0.12.11"
+version = "0.12.10"
 dependencies = [
 "aho-corasick",
 "anyhow",
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "clap",
 "colored 3.0.0",
 "fern",
@@ -3021,6 +3032,7 @@ dependencies = [
 "memchr",
 "natord",
 "path-absolutize",
+ "pathdiff",
 "pep440_rs",
 "pyproject-toml",
 "regex",
@@ -3049,7 +3061,7 @@ dependencies = [
 "strum_macros",
 "tempfile",
 "test-case",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "toml 0.9.5",
 "typed-arena",
 "unicode-normalization",
@@ -3092,7 +3104,7 @@ dependencies = [
 "serde_json",
 "serde_with",
 "test-case",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "uuid",
 ]

@@ -3108,7 +3120,7 @@ name = "ruff_python_ast"
 version = "0.0.0"
 dependencies = [
 "aho-corasick",
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "compact_str",
 "get-size2",
 "is-macro",
@@ -3123,7 +3135,7 @@ dependencies = [
 "salsa",
 "schemars",
 "serde",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 ]

 [[package]]
@@ -3177,7 +3189,7 @@ dependencies = [
 "similar",
 "smallvec",
 "static_assertions",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "tracing",
 ]

@@ -3196,7 +3208,7 @@ dependencies = [
 name = "ruff_python_literal"
 version = "0.0.0"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "itertools 0.14.0",
 "ruff_python_ast",
 "unic-ucd-category",
@@ -3207,7 +3219,7 @@ name = "ruff_python_parser"
 version = "0.0.0"
 dependencies = [
 "anyhow",
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "bstr",
 "compact_str",
 "get-size2",
@@ -3232,7 +3244,7 @@ dependencies = [
 name = "ruff_python_semantic"
 version = "0.0.0"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "insta",
 "is-macro",
 "ruff_cache",
@@ -3253,7 +3265,7 @@ dependencies = [
 name = "ruff_python_stdlib"
 version = "0.0.0"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "unicode-ident",
 ]

@@ -3307,7 +3319,7 @@ dependencies = [
 "serde",
 "serde_json",
 "shellexpand",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "toml 0.9.5",
 "tracing",
 "tracing-log",
@@ -3337,7 +3349,7 @@ dependencies = [

 [[package]]
 name = "ruff_wasm"
-version = "0.12.11"
+version = "0.12.10"
 dependencies = [
 "console_error_panic_hook",
 "console_log",
@@ -3430,11 +3442,11 @@ version = "1.0.8"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "11181fbabf243db407ef8df94a6ce0b2f9a733bd8be4ad02b4eda9602296cac8"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "errno",
 "libc",
 "linux-raw-sys",
- "windows-sys 0.59.0",
+ "windows-sys 0.60.2",
 ]

 [[package]]
@@ -3452,12 +3464,13 @@ checksum = "28d3b2b1366ec20994f1fd18c3c594f05c5dd4bc44d8bb0c1c632c8d6829481f"
 [[package]]
 name = "salsa"
 version = "0.23.0"
-source = "git+https://github.com/salsa-rs/salsa.git?rev=a3ffa22cb26756473d56f867aedec3fd907c4dd9#a3ffa22cb26756473d56f867aedec3fd907c4dd9"
+source = "git+https://github.com/salsa-rs/salsa.git?rev=a0e7a06#a0e7a0660c93136f23bf08b4f1604eee3d1f6b11"
 dependencies = [
 "boxcar",
 "compact_str",
 "crossbeam-queue",
 "crossbeam-utils",
+ "erased-serde",
 "hashbrown 0.15.5",
 "hashlink",
 "indexmap",
@@ -3468,6 +3481,7 @@ dependencies = [
 "rustc-hash",
 "salsa-macro-rules",
 "salsa-macros",
+ "serde",
 "smallvec",
 "thin-vec",
 "tracing",
@@ -3476,12 +3490,12 @@ dependencies = [
 [[package]]
 name = "salsa-macro-rules"
 version = "0.23.0"
-source = "git+https://github.com/salsa-rs/salsa.git?rev=a3ffa22cb26756473d56f867aedec3fd907c4dd9#a3ffa22cb26756473d56f867aedec3fd907c4dd9"
+source = "git+https://github.com/salsa-rs/salsa.git?rev=a0e7a06#a0e7a0660c93136f23bf08b4f1604eee3d1f6b11"

 [[package]]
 name = "salsa-macros"
 version = "0.23.0"
-source = "git+https://github.com/salsa-rs/salsa.git?rev=a3ffa22cb26756473d56f867aedec3fd907c4dd9#a3ffa22cb26756473d56f867aedec3fd907c4dd9"
+source = "git+https://github.com/salsa-rs/salsa.git?rev=a0e7a06#a0e7a0660c93136f23bf08b4f1604eee3d1f6b11"
 dependencies = [
 "proc-macro2",
 "quote",
@@ -3578,9 +3592,9 @@ dependencies = [

 [[package]]
 name = "serde_json"
-version = "1.0.143"
+version = "1.0.142"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d401abef1d108fbd9cbaebc3e46611f4b1021f714a0597a71f41ee463f5f4a5a"
+checksum = "030fedb782600dcbd6f02d479bf0d817ac3bb40d644745b769d6a96bc3afc5a7"
 dependencies = [
 "itoa",
 "memchr",
@@ -3701,6 +3715,9 @@ name = "smallvec"
 version = "1.15.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "67b1b7a3b5fe4f1376887184045fcf45c69e92af734b7aaddc05fb777b6fbd03"
+dependencies = [
+ "serde",
+]

 [[package]]
 name = "snapbox"
@@ -3791,9 +3808,9 @@ dependencies = [

 [[package]]
 name = "syn"
-version = "2.0.106"
+version = "2.0.104"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ede7c438028d4436d71104916910f5bb611972c5cfd7f89b8300a8186e6fada6"
+checksum = "17b6f705963418cdb9927482fa304bc562ece2fdd4f616084c50b7023b435a40"
 dependencies = [
 "proc-macro2",
 "quote",
@@ -3905,6 +3922,9 @@ name = "thin-vec"
 version = "0.2.14"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "144f754d318415ac792f9d69fc87abbbfc043ce2ef041c60f16ad828f638717d"
+dependencies = [
+ "serde",
+]

 [[package]]
 name = "thiserror"
@@ -3917,11 +3937,11 @@ dependencies = [

 [[package]]
 name = "thiserror"
-version = "2.0.16"
+version = "2.0.12"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3467d614147380f2e4e374161426ff399c91084acd2363eaf549172b3d5e60c0"
+checksum = "567b8a2dae586314f7be2a752ec7474332959c6460e02bde30d702a66d488708"
 dependencies = [
- "thiserror-impl 2.0.16",
+ "thiserror-impl 2.0.12",
 ]

 [[package]]
@@ -3937,9 +3957,9 @@ dependencies = [

 [[package]]
 name = "thiserror-impl"
-version = "2.0.16"
+version = "2.0.12"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6c5e1be1c48b9172ee610da68fd9cd2770e7a4056cb3fc98710ee6906f0c7960"
+checksum = "7f7cf42b4507d8ea322120659672cf1b9dbb93f8f2d4ecfd6e51350ff5b17a1d"
 dependencies = [
 "proc-macro2",
 "quote",
@@ -4138,9 +4158,9 @@ dependencies = [

 [[package]]
 name = "tracing-indicatif"
-version = "0.3.13"
+version = "0.3.12"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "04d4e11e0e27acef25a47f27e9435355fecdc488867fa2bc90e75b0700d2823d"
+checksum = "e1983afead46ff13a3c93581e0cec31d20b29efdd22cbdaa8b9f850eccf2c352"
 dependencies = [
 "indicatif",
 "tracing",
@@ -4240,22 +4260,17 @@ dependencies = [
 name = "ty_ide"
 version = "0.0.0"
 dependencies = [
- "bitflags 2.9.3",
- "get-size2",
+ "bitflags 2.9.2",
 "insta",
 "itertools 0.14.0",
- "rayon",
 "regex",
 "ruff_db",
- "ruff_index",
- "ruff_memory_usage",
 "ruff_python_ast",
 "ruff_python_parser",
 "ruff_python_trivia",
 "ruff_source_file",
 "ruff_text_size",
 "rustc-hash",
- "salsa",
 "smallvec",
 "tracing",
 "ty_project",
@@ -4267,6 +4282,7 @@ name = "ty_project"
 version = "0.0.0"
 dependencies = [
 "anyhow",
+ "bincode 2.0.1",
 "camino",
 "colored 3.0.0",
 "crossbeam",
@@ -4278,7 +4294,7 @@ dependencies = [
 "pep440_rs",
 "rayon",
 "regex",
- "regex-automata 0.4.10",
+ "regex-automata 0.4.9",
 "ruff_cache",
 "ruff_db",
 "ruff_macros",
@@ -4291,11 +4307,12 @@ dependencies = [
 "salsa",
 "schemars",
 "serde",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "toml 0.9.5",
 "tracing",
 "ty_combine",
 "ty_python_semantic",
+ "ty_static",
 "ty_vendored",
 ]

@@ -4304,7 +4321,7 @@ name = "ty_python_semantic"
 version = "0.0.0"
 dependencies = [
 "anyhow",
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "bitvec",
 "camino",
 "colored 3.0.0",
@@ -4344,7 +4361,7 @@ dependencies = [
 "strum_macros",
 "tempfile",
 "test-case",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "tracing",
 "ty_python_semantic",
 "ty_static",
@@ -4357,7 +4374,7 @@ name = "ty_server"
 version = "0.0.0"
 dependencies = [
 "anyhow",
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "crossbeam",
 "dunce",
 "insta",
@@ -4378,7 +4395,7 @@ dependencies = [
 "serde_json",
 "shellexpand",
 "tempfile",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "tracing",
 "tracing-subscriber",
 "ty_combine",
@@ -4400,7 +4417,7 @@ name = "ty_test"
 version = "0.0.0"
 dependencies = [
 "anyhow",
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 "camino",
 "colored 3.0.0",
 "insta",
@@ -4419,7 +4436,7 @@ dependencies = [
 "serde",
 "smallvec",
 "tempfile",
- "thiserror 2.0.16",
+ "thiserror 2.0.12",
 "toml 0.9.5",
 "tracing",
 "ty_python_semantic",
@@ -4467,6 +4484,12 @@ version = "2.0.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "6af6ae20167a9ece4bcb41af5b80f8a1f1df981f6391189ce00fd257af04126a"

+[[package]]
+name = "typeid"
+version = "1.0.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "bc7d623258602320d5c55d1bc22793b57daff0ec7efc270ea7d55ce1d5f5471c"
+
 [[package]]
 name = "typenum"
 version = "1.18.0"
@@ -4596,9 +4619,9 @@ checksum = "6d49784317cd0d1ee7ec5c716dd598ec5b4483ea832a2dced265471cc0f690ae"

 [[package]]
 name = "url"
-version = "2.5.7"
+version = "2.5.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "08bc136a29a3d1758e07a9cca267be308aeebf5cfd5a10f3f67ab2097683ef5b"
+checksum = "32f8b686cadd1473f4bd0117a5d28d36b1ade384ea9b5069a1c40aefed7fda60"
 dependencies = [
 "form_urlencoded",
 "idna",
@@ -5150,7 +5173,7 @@ version = "0.39.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "6f42320e61fe2cfd34354ecb597f86f413484a798ba44a8ca1165c58d42da6c1"
 dependencies = [
- "bitflags 2.9.3",
+ "bitflags 2.9.2",
 ]

 [[package]]
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -57,8 +57,8 @@ anyhow = { version = "1.0.80" }
 arc-swap = { version = "1.7.1" }
 assert_fs = { version = "1.1.0" }
 argfile = { version = "0.2.0" }
-bincode = { version = "2.0.0" }
-bitflags = { version = "2.5.0" }
+bincode = { version = "2.0.0", features = ["serde"] }
+bitflags = { version = "2.5.0", features = ["serde"] }
 bitvec = { version = "1.0.1", default-features = false, features = [
    "alloc",
 ] }
@@ -126,7 +126,7 @@ memchr = { version = "2.7.1" }
 mimalloc = { version = "0.1.39" }
 natord = { version = "1.0.9" }
 notify = { version = "8.0.0" }
-ordermap = { version = "0.5.0" }
+ordermap = { version = "0.5.0", features = ["serde"] }
 path-absolutize = { version = "3.1.1" }
 path-slash = { version = "0.2.1" }
 pathdiff = { version = "0.2.1" }
@@ -143,24 +143,25 @@ regex-automata = { version = "0.4.9" }
 rustc-hash = { version = "2.0.0" }
 rustc-stable-hash = { version = "0.1.2" }
 # When updating salsa, make sure to also update the revision in `fuzz/Cargo.toml`
-salsa = { git = "https://github.com/salsa-rs/salsa.git", rev = "a3ffa22cb26756473d56f867aedec3fd907c4dd9", default-features = false, features = [
+salsa = { git = "https://github.com/salsa-rs/salsa.git", rev = "a0e7a06", default-features = false, features = [
    "compact_str",
    "macros",
    "salsa_unstable",
    "inventory",
+    "persistence",
 ] }
 schemars = { version = "0.8.16" }
 seahash = { version = "4.1.0" }
-serde = { version = "1.0.197", features = ["derive"] }
+serde = { version = "1.0.197", features = ["derive", "rc"] }
 serde-wasm-bindgen = { version = "0.6.4" }
-serde_json = { version = "1.0.113" }
+serde_json = { version = "1.0.142" }
 serde_test = { version = "1.0.152" }
 serde_with = { version = "3.6.0", default-features = false, features = [
    "macros",
 ] }
 shellexpand = { version = "3.0.0" }
 similar = { version = "2.4.0", features = ["inline"] }
-smallvec = { version = "1.13.2", features = ["union", "const_generics", "const_new"] }
+smallvec = { version = "1.13.2", features = ["union", "const_generics", "const_new", "serde"] }
 snapbox = { version = "0.6.0", features = [
    "diff",
    "term-svg",
--- a/README.md
+++ b/README.md
@@ -148,8 +148,8 @@ curl -LsSf https://astral.sh/ruff/install.sh | sh
 powershell -c "irm https://astral.sh/ruff/install.ps1 | iex"

 # For a specific version.
-curl -LsSf https://astral.sh/ruff/0.12.11/install.sh | sh
-powershell -c "irm https://astral.sh/ruff/0.12.11/install.ps1 | iex"
+curl -LsSf https://astral.sh/ruff/0.12.10/install.sh | sh
+powershell -c "irm https://astral.sh/ruff/0.12.10/install.ps1 | iex"
 ```

 You can also install Ruff via [Homebrew](https://formulae.brew.sh/formula/ruff), [Conda](https://anaconda.org/conda-forge/ruff),
@@ -182,7 +182,7 @@ Ruff can also be used as a [pre-commit](https://pre-commit.com/) hook via [`ruff
 ```yaml
 - repo: https://github.com/astral-sh/ruff-pre-commit
  # Ruff version.
-  rev: v0.12.11
+  rev: v0.12.10
  hooks:
    # Run the linter.
    - id: ruff-check
--- a/crates/ruff/Cargo.toml
+++ b/crates/ruff/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "ruff"
-version = "0.12.11"
+version = "0.12.10"
 publish = true
 authors = { workspace = true }
 edition = { workspace = true }
@@ -31,7 +31,7 @@ ruff_workspace = { workspace = true }

 anyhow = { workspace = true }
 argfile = { workspace = true }
-bincode = { workspace = true, features = ["serde"] }
+bincode = { workspace = true }
 bitflags = { workspace = true }
 cachedir = { workspace = true }
 clap = { workspace = true, features = ["derive", "env", "wrap_help"] }
--- a/crates/ruff/src/printer.rs
+++ b/crates/ruff/src/printer.rs
@@ -15,7 +15,8 @@ use ruff_db::diagnostic::{
 use ruff_linter::fs::relativize_path;
 use ruff_linter::logging::LogLevel;
 use ruff_linter::message::{
-    Emitter, EmitterContext, GithubEmitter, GroupedEmitter, SarifEmitter, TextEmitter,
+    Emitter, EmitterContext, GithubEmitter, GitlabEmitter, GroupedEmitter, SarifEmitter,
+    TextEmitter,
 };
 use ruff_linter::notify_user;
 use ruff_linter::settings::flags::{self};
@@ -295,11 +296,7 @@ impl Printer {
                GithubEmitter.emit(writer, &diagnostics.inner, &context)?;
            }
            OutputFormat::Gitlab => {
-                let config = DisplayDiagnosticConfig::default()
-                    .format(DiagnosticFormat::Gitlab)
-                    .preview(preview);
-                let value = DisplayDiagnostics::new(&context, &config, &diagnostics.inner);
-                write!(writer, "{value}")?;
+                GitlabEmitter::default().emit(writer, &diagnostics.inner, &context)?;
            }
            OutputFormat::Pylint => {
                let config = DisplayDiagnosticConfig::default()
--- a/crates/ruff/tests/snapshots/lint__output_format_gitlab.snap
+++ b/crates/ruff/tests/snapshots/lint__output_format_gitlab.snap
@@ -20,59 +20,59 @@ exit_code: 1
  {
    "check_name": "F401",
    "description": "F401: `os` imported but unused",
-    "severity": "major",
    "fingerprint": "4dbad37161e65c72",
    "location": {
      "path": "input.py",
      "positions": {
        "begin": {
-          "line": 1,
-          "column": 8
+          "column": 8,
+          "line": 1
        },
        "end": {
-          "line": 1,
-          "column": 10
+          "column": 10,
+          "line": 1
        }
      }
-    }
+    },
+    "severity": "major"
  },
  {
    "check_name": "F821",
    "description": "F821: Undefined name `y`",
-    "severity": "major",
    "fingerprint": "7af59862a085230",
    "location": {
      "path": "input.py",
      "positions": {
        "begin": {
-          "line": 2,
-          "column": 5
+          "column": 5,
+          "line": 2
        },
        "end": {
-          "line": 2,
-          "column": 6
+          "column": 6,
+          "line": 2
        }
      }
-    }
+    },
+    "severity": "major"
  },
  {
    "check_name": "invalid-syntax",
    "description": "invalid-syntax: Cannot use `match` statement on Python 3.9 (syntax was added in Python 3.10)",
-    "severity": "major",
    "fingerprint": "e558cec859bb66e8",
    "location": {
      "path": "input.py",
      "positions": {
        "begin": {
-          "line": 3,
-          "column": 1
+          "column": 1,
+          "line": 3
        },
        "end": {
-          "line": 3,
-          "column": 6
+          "column": 6,
+          "line": 3
        }
      }
-    }
+    },
+    "severity": "major"
  }
 ]
 ----- stderr -----
--- a/crates/ruff_benchmark/benches/ty.rs
+++ b/crates/ruff_benchmark/benches/ty.rs
@@ -450,6 +450,9 @@ fn benchmark_complex_constrained_attributes_2(criterion: &mut Criterion) {
                    r#"
                    class C:
                        def f(self: "C"):
+                            self.a = ""
+                            self.b = ""
+
                            if isinstance(self.a, str):
                                return

@@ -463,56 +466,6 @@ fn benchmark_complex_constrained_attributes_2(criterion: &mut Criterion) {
                                return
                            if isinstance(self.b, str):
                                return
-                            if isinstance(self.b, str):
-                                return
-                            if isinstance(self.b, str):
-                                return
-
-                            self.a = ""
-                            self.b = ""
-                    "#,
-                )
-            },
-            |case| {
-                let Case { db, .. } = case;
-                let result = db.check();
-                assert_eq!(result.len(), 0);
-            },
-            BatchSize::SmallInput,
-        );
-    });
-}
-
-fn benchmark_complex_constrained_attributes_3(criterion: &mut Criterion) {
-    setup_rayon();
-
-    criterion.bench_function("ty_micro[complex_constrained_attributes_3]", |b| {
-        b.iter_batched_ref(
-            || {
-                // This is a regression test for https://github.com/astral-sh/ty/issues/758
-                setup_micro_case(
-                    r#"
-                    class GridOut:
-                        def __init__(self: "GridOut") -> None:
-                            self._buffer = b""
-
-                        def _read_size_or_line(self: "GridOut", size: int = -1):
-                            if size > self._position:
-                                size = self._position
-                                pass
-                            if size == 0:
-                                return bytes()
-
-                            while size > 0:
-                                if self._buffer:
-                                    buf = self._buffer
-                                    self._buffer = b""
-                                else:
-                                    buf = b""
-
-                                if len(buf) > size:
-                                    self._buffer = buf
-                                    self._position -= len(self._buffer)
                    "#,
                )
            },
@@ -715,7 +668,6 @@ criterion_group!(
    benchmark_tuple_implicit_instance_attributes,
    benchmark_complex_constrained_attributes_1,
    benchmark_complex_constrained_attributes_2,
-    benchmark_complex_constrained_attributes_3,
    benchmark_many_enum_members,
 );
 criterion_group!(project, anyio, attrs, hydra, datetype);
--- a/crates/ruff_db/Cargo.toml
+++ b/crates/ruff_db/Cargo.toml
@@ -34,7 +34,6 @@ glob = { workspace = true }
 ignore = { workspace = true, optional = true }
 matchit = { workspace = true }
 path-slash = { workspace = true }
-pathdiff = { workspace = true }
 quick-junit = { workspace = true, optional = true }
 rustc-hash = { workspace = true }
 salsa = { workspace = true }
@@ -54,7 +53,7 @@ web-time = { version = "1.1.0" }
 etcetera = { workspace = true, optional = true }

 [dev-dependencies]
-insta = { workspace = true, features = ["filters"] }
+insta = { workspace = true }
 tempfile = { workspace = true }

 [features]
--- a/crates/ruff_db/src/diagnostic/mod.rs
+++ b/crates/ruff_db/src/diagnostic/mod.rs
@@ -22,6 +22,7 @@ mod stylesheet;
 /// a characteristic is a deficiency. An example of a characteristic that is
 /// _not_ a deficiency is the `reveal_type` diagnostic for our type checker.
 #[derive(Debug, Clone, Eq, PartialEq, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 pub struct Diagnostic {
    /// The actual diagnostic.
    ///
@@ -325,11 +326,6 @@ impl Diagnostic {
        self.inner.fix.as_ref()
    }

-    #[cfg(test)]
-    pub(crate) fn fix_mut(&mut self) -> Option<&mut Fix> {
-        Arc::make_mut(&mut self.inner).fix.as_mut()
-    }
-
    /// Set the fix for this diagnostic.
    pub fn set_fix(&mut self, fix: Fix) {
        debug_assert!(
@@ -505,6 +501,7 @@ impl Diagnostic {
 }

 #[derive(Debug, Clone, Eq, PartialEq, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 struct DiagnosticInner {
    id: DiagnosticId,
    severity: Severity,
@@ -581,6 +578,7 @@ impl Eq for RenderingSortKey<'_> {}
 /// another (for a single parent diagnostic) is the order in which they were
 /// attached to the diagnostic.
 #[derive(Debug, Clone, Eq, PartialEq, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 pub struct SubDiagnostic {
    /// Like with `Diagnostic`, we box the `SubDiagnostic` to make it
    /// pointer-sized.
@@ -690,6 +688,7 @@ impl SubDiagnostic {
 }

 #[derive(Debug, Clone, Eq, PartialEq, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 struct SubDiagnosticInner {
    severity: SubDiagnosticSeverity,
    message: DiagnosticMessage,
@@ -718,6 +717,7 @@ struct SubDiagnosticInner {
 /// Messages attached to annotations should also be as brief and specific as
 /// possible. Long messages could negative impact the quality of rendering.
 #[derive(Debug, Clone, Eq, PartialEq, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 pub struct Annotation {
    /// The span of this annotation, corresponding to some subsequence of the
    /// user's input that we want to highlight.
@@ -860,6 +860,7 @@ impl Annotation {
 /// These tags are used to provide additional information about the annotation.
 /// and are passed through to the language server protocol.
 #[derive(Debug, Clone, Eq, PartialEq, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 pub enum DiagnosticTag {
    /// Unused or unnecessary code. Used for unused parameters, unreachable code, etc.
    Unnecessary,
@@ -874,6 +875,7 @@ pub enum DiagnosticTag {
 ///
 /// Rules use kebab case, e.g. `no-foo`.
 #[derive(Debug, Copy, Clone, PartialOrd, Ord, PartialEq, Eq, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Serialize), serde(transparent))]
 pub struct LintName(&'static str);

 impl LintName {
@@ -886,6 +888,66 @@ impl LintName {
    }
 }

+#[cfg(feature = "serde")]
+pub use lint_name_serde::LintRegistryGuard;
+
+#[cfg(feature = "serde")]
+mod lint_name_serde {
+    use super::LintName;
+
+    use std::cell::RefCell;
+
+    thread_local! {
+        /// Serde doesn't provide any easy means to pass a value to a [`Deserialize`] implementation,
+        /// but we need a way to retrieve static [`LintName`]s from the lint registry when deserializing.
+        ///
+        /// Use the [`LintRegistryGuard`] to initialize the thread local before calling into any
+        /// deserialization code. It ensures that the thread local variable gets cleaned up
+        /// once deserialization is done (once the guard gets dropped).
+        static LINT_REGISTRY: RefCell<Option<LintRegistry>> = const { RefCell::new(None) };
+    }
+
+    type LintRegistry = fn(&str) -> Option<LintName>;
+
+    /// Guard to safely change the lint registry for the current thread.
+    #[must_use]
+    pub struct LintRegistryGuard {
+        prev_value: Option<LintRegistry>,
+    }
+
+    impl LintRegistryGuard {
+        pub fn new(registry: LintRegistry) -> Self {
+            let prev = LINT_REGISTRY.replace(Some(registry));
+            Self { prev_value: prev }
+        }
+    }
+
+    impl Drop for LintRegistryGuard {
+        fn drop(&mut self) {
+            LINT_REGISTRY.set(self.prev_value.take());
+        }
+    }
+
+    #[cfg(feature = "serde")]
+    impl<'de> serde::Deserialize<'de> for LintName {
+        fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
+        where
+            D: serde::Deserializer<'de>,
+        {
+            let name: &str = serde::Deserialize::deserialize(deserializer)?;
+
+            LINT_REGISTRY.with_borrow(|registry| {
+                let registry = registry
+                    .expect("must set the `LintRegistryGuard` when deserializing a `LintName`");
+
+                registry(name).ok_or(serde::de::Error::custom(format!(
+                    "invalid `LintName` {name}"
+                )))
+            })
+        }
+    }
+}
+
 impl std::ops::Deref for LintName {
    type Target = str;

@@ -914,6 +976,7 @@ impl PartialEq<&str> for LintName {

 /// Uniquely identifies the kind of a diagnostic.
 #[derive(Debug, Copy, Clone, Eq, PartialEq, PartialOrd, Ord, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 pub enum DiagnosticId {
    Panic,

@@ -1102,6 +1165,30 @@ impl UnifiedFile {
    }
 }

+#[cfg(feature = "serde")]
+impl serde::Serialize for UnifiedFile {
+    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
+    where
+        S: serde::Serializer,
+    {
+        match self {
+            UnifiedFile::Ty(file) => serde::Serialize::serialize(file, serializer),
+            // Persistent caching is only used in ty.
+            UnifiedFile::Ruff(..) => panic!("Ruff files are not persistable"),
+        }
+    }
+}
+
+#[cfg(feature = "serde")]
+impl<'de> serde::Deserialize<'de> for UnifiedFile {
+    fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
+    where
+        D: serde::Deserializer<'de>,
+    {
+        serde::Deserialize::deserialize(deserializer).map(UnifiedFile::Ty)
+    }
+}
+
 /// A unified wrapper for types that can be converted to a [`SourceCode`].
 ///
 /// As with [`UnifiedFile`], ruff and ty use slightly different representations for source code.
@@ -1133,6 +1220,7 @@ impl DiagnosticSource {
 /// range isn't present, it semantically implies that the diagnostic refers to
 /// the entire file. For example, when the file should be executable but isn't.
 #[derive(Debug, Clone, PartialEq, Eq, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 pub struct Span {
    file: UnifiedFile,
    range: Option<TextRange>,
@@ -1211,6 +1299,7 @@ impl From<crate::files::FileRange> for Span {
 }

 #[derive(Debug, Clone, Copy, PartialEq, Eq, Ord, PartialOrd, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 pub enum Severity {
    Info,
    Warning,
@@ -1246,6 +1335,7 @@ impl Severity {
 /// used for main diagnostics. If we want to add `Severity::Help` in the future, this type could be
 /// deleted and the two combined again.
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Ord, PartialOrd, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 pub enum SubDiagnosticSeverity {
    Help,
    Info,
@@ -1435,11 +1525,6 @@ pub enum DiagnosticFormat {
    /// Print diagnostics in the format expected by JUnit.
    #[cfg(feature = "junit")]
    Junit,
-    /// Print diagnostics in the JSON format used by GitLab [Code Quality] reports.
-    ///
-    /// [Code Quality]: https://docs.gitlab.com/ee/ci/testing/code_quality.html#implement-a-custom-tool
-    #[cfg(feature = "serde")]
-    Gitlab,
 }

 /// A representation of the kinds of messages inside a diagnostic.
@@ -1499,6 +1584,7 @@ impl std::fmt::Display for ConciseMessage<'_> {
 /// a blanket trait implementation for `IntoDiagnosticMessage` for
 /// anything that implements `std::fmt::Display`.
 #[derive(Clone, Debug, Eq, PartialEq, Hash, get_size2::GetSize)]
+#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 pub struct DiagnosticMessage(Box<str>);

 impl DiagnosticMessage {
@@ -1562,7 +1648,11 @@ impl<T: std::fmt::Display> IntoDiagnosticMessage for T {
 ///
 /// For Ruff rules this means the noqa code.
 #[derive(Clone, Debug, PartialEq, Eq, PartialOrd, Ord, Default, Hash, get_size2::GetSize)]
-#[cfg_attr(feature = "serde", derive(serde::Serialize), serde(transparent))]
+#[cfg_attr(
+    feature = "serde",
+    derive(serde::Serialize, serde::Deserialize),
+    serde(transparent)
+)]
 pub struct SecondaryCode(String);

 impl SecondaryCode {
--- a/crates/ruff_db/src/diagnostic/render.rs
+++ b/crates/ruff_db/src/diagnostic/render.rs
@@ -31,8 +31,6 @@ mod azure;
 mod concise;
 mod full;
 #[cfg(feature = "serde")]
-mod gitlab;
-#[cfg(feature = "serde")]
 mod json;
 #[cfg(feature = "serde")]
 mod json_lines;
@@ -138,10 +136,6 @@ impl std::fmt::Display for DisplayDiagnostics<'_> {
            DiagnosticFormat::Junit => {
                junit::JunitRenderer::new(self.resolver).render(f, self.diagnostics)?;
            }
-            #[cfg(feature = "serde")]
-            DiagnosticFormat::Gitlab => {
-                gitlab::GitlabRenderer::new(self.resolver).render(f, self.diagnostics)?;
-            }
        }

        Ok(())
@@ -2628,13 +2622,6 @@ watermelon
            self.config = config;
        }

-        /// Show a diff for the fix when rendering.
-        pub(super) fn show_fix_diff(&mut self, yes: bool) {
-            let mut config = std::mem::take(&mut self.config);
-            config = config.show_fix_diff(yes);
-            self.config = config;
-        }
-
        /// The lowest fix applicability to show when rendering.
        pub(super) fn fix_applicability(&mut self, applicability: Applicability) {
            let mut config = std::mem::take(&mut self.config);
--- a/crates/ruff_db/src/diagnostic/render/full.rs
+++ b/crates/ruff_db/src/diagnostic/render/full.rs
@@ -2,13 +2,12 @@ use std::borrow::Cow;
 use std::num::NonZeroUsize;

 use anstyle::Style;
-use ruff_notebook::NotebookIndex;
 use similar::{ChangeTag, TextDiff};

 use ruff_annotate_snippets::Renderer as AnnotateRenderer;
 use ruff_diagnostics::{Applicability, Fix};
 use ruff_source_file::OneIndexed;
-use ruff_text_size::{Ranged, TextLen, TextRange, TextSize};
+use ruff_text_size::{Ranged, TextRange, TextSize};

 use crate::diagnostic::render::{FileResolver, Resolved};
 use crate::diagnostic::stylesheet::{DiagnosticStylesheet, fmt_styled};
@@ -82,7 +81,6 @@ impl<'a> FullRenderer<'a> {
 struct Diff<'a> {
    fix: &'a Fix,
    diagnostic_source: DiagnosticSource,
-    notebook_index: Option<NotebookIndex>,
    stylesheet: &'a DiagnosticStylesheet,
 }

@@ -92,11 +90,12 @@ impl<'a> Diff<'a> {
        stylesheet: &'a DiagnosticStylesheet,
        resolver: &'a dyn FileResolver,
    ) -> Option<Diff<'a>> {
-        let file = &diagnostic.primary_span_ref()?.file;
        Some(Diff {
            fix: diagnostic.fix()?,
-            diagnostic_source: file.diagnostic_source(resolver),
-            notebook_index: resolver.notebook_index(file),
+            diagnostic_source: diagnostic
+                .primary_span_ref()?
+                .file
+                .diagnostic_source(resolver),
            stylesheet,
        })
    }
@@ -107,24 +106,19 @@ impl std::fmt::Display for Diff<'_> {
        let source_code = self.diagnostic_source.as_source_code();
        let source_text = source_code.text();

-        // Partition the source code into end offsets for each cell. If `self.notebook_index` is
-        // `None`, indicating a regular script file, all the lines will be in one "cell" under the
-        // `None` key.
-        let cells = if let Some(notebook_index) = &self.notebook_index {
-            let mut last_cell = OneIndexed::MIN;
-            let mut cells: Vec<(Option<OneIndexed>, TextSize)> = Vec::new();
-            for (row, cell) in notebook_index.iter() {
-                if cell != last_cell {
-                    let offset = source_code.line_start(row);
-                    cells.push((Some(last_cell), offset));
-                    last_cell = cell;
-                }
-            }
-            cells.push((Some(last_cell), source_text.text_len()));
-            cells
-        } else {
-            vec![(None, source_text.text_len())]
-        };
+        // TODO(dhruvmanila): Add support for Notebook cells once it's user-facing
+        let mut output = String::with_capacity(source_text.len());
+        let mut last_end = TextSize::default();
+
+        for edit in self.fix.edits() {
+            output.push_str(source_code.slice(TextRange::new(last_end, edit.start())));
+            output.push_str(edit.content().unwrap_or_default());
+            last_end = edit.end();
+        }
+
+        output.push_str(&source_text[usize::from(last_end)..]);
+
+        let diff = TextDiff::from_lines(source_text, &output);

        let message = match self.fix.applicability() {
            // TODO(zanieb): Adjust this messaging once it's user-facing
@@ -139,98 +133,60 @@ impl std::fmt::Display for Diff<'_> {
        // tests, which is the only place these are currently used.
        writeln!(f, "ℹ {}", fmt_styled(message, self.stylesheet.separator))?;

-        let mut last_end = TextSize::ZERO;
-        for (cell, offset) in cells {
-            let range = TextRange::new(last_end, offset);
-            last_end = offset;
-            let input = source_code.slice(range);
+        let (largest_old, largest_new) = diff
+            .ops()
+            .last()
+            .map(|op| (op.old_range().start, op.new_range().start))
+            .unwrap_or_default();

-            let mut output = String::with_capacity(input.len());
-            let mut last_end = range.start();
+        let digit_with = OneIndexed::from_zero_indexed(largest_new.max(largest_old)).digits();

-            let mut applied = 0;
-            for edit in self.fix.edits() {
-                if range.contains_range(edit.range()) {
-                    output.push_str(source_code.slice(TextRange::new(last_end, edit.start())));
-                    output.push_str(edit.content().unwrap_or_default());
-                    last_end = edit.end();
-                    applied += 1;
-                }
+        for (idx, group) in diff.grouped_ops(3).iter().enumerate() {
+            if idx > 0 {
+                writeln!(f, "{:-^1$}", "-", 80)?;
            }
+            for op in group {
+                for change in diff.iter_inline_changes(op) {
+                    let sign = match change.tag() {
+                        ChangeTag::Delete => "-",
+                        ChangeTag::Insert => "+",
+                        ChangeTag::Equal => " ",
+                    };

-            // No edits were applied, so there's no need to diff.
-            if applied == 0 {
-                continue;
-            }
+                    let line_style = LineStyle::from(change.tag(), self.stylesheet);

-            output.push_str(&source_text[usize::from(last_end)..usize::from(range.end())]);
+                    let old_index = change.old_index().map(OneIndexed::from_zero_indexed);
+                    let new_index = change.new_index().map(OneIndexed::from_zero_indexed);

-            let diff = TextDiff::from_lines(input, &output);
+                    write!(
+                        f,
+                        "{} {} |{}",
+                        Line {
+                            index: old_index,
+                            width: digit_with
+                        },
+                        Line {
+                            index: new_index,
+                            width: digit_with
+                        },
+                        fmt_styled(line_style.apply_to(sign), self.stylesheet.emphasis),
+                    )?;

-            let (largest_old, largest_new) = diff
-                .ops()
-                .last()
-                .map(|op| (op.old_range().start, op.new_range().start))
-                .unwrap_or_default();
-
-            let digit_with = OneIndexed::from_zero_indexed(largest_new.max(largest_old)).digits();
-
-            if let Some(cell) = cell {
-                // Room for 2 digits, 2 x 1 space before each digit, 1 space, and 1 `|`. This
-                // centers the three colons on the pipe.
-                writeln!(f, "{:>1$} cell {cell}", ":::", 2 * digit_with.get() + 4)?;
-            }
-
-            for (idx, group) in diff.grouped_ops(3).iter().enumerate() {
-                if idx > 0 {
-                    writeln!(f, "{:-^1$}", "-", 80)?;
-                }
-                for op in group {
-                    for change in diff.iter_inline_changes(op) {
-                        let sign = match change.tag() {
-                            ChangeTag::Delete => "-",
-                            ChangeTag::Insert => "+",
-                            ChangeTag::Equal => " ",
-                        };
-
-                        let line_style = LineStyle::from(change.tag(), self.stylesheet);
-
-                        let old_index = change.old_index().map(OneIndexed::from_zero_indexed);
-                        let new_index = change.new_index().map(OneIndexed::from_zero_indexed);
-
-                        write!(
-                            f,
-                            "{} {} |{}",
-                            Line {
-                                index: old_index,
-                                width: digit_with,
-                            },
-                            Line {
-                                index: new_index,
-                                width: digit_with,
-                            },
-                            fmt_styled(line_style.apply_to(sign), self.stylesheet.emphasis),
-                        )?;
-
-                        for (emphasized, value) in change.iter_strings_lossy() {
-                            let value = show_nonprinting(&value);
-                            if emphasized {
-                                write!(
-                                    f,
-                                    "{}",
-                                    fmt_styled(
-                                        line_style.apply_to(&value),
-                                        self.stylesheet.underline
-                                    )
-                                )?;
-                            } else {
-                                write!(f, "{}", line_style.apply_to(&value))?;
-                            }
-                        }
-                        if change.missing_newline() {
-                            writeln!(f)?;
+                    for (emphasized, value) in change.iter_strings_lossy() {
+                        let value = show_nonprinting(&value);
+                        if emphasized {
+                            write!(
+                                f,
+                                "{}",
+                                fmt_styled(line_style.apply_to(&value), self.stylesheet.underline)
+                            )?;
+                        } else {
+                            write!(f, "{}", line_style.apply_to(&value))?;
                        }
                    }
+                    if change.missing_newline() {
+                        writeln!(f)?;
+                    }
                }
            }
        }
@@ -297,7 +253,7 @@ fn show_nonprinting(s: &str) -> Cow<'_, str> {

 #[cfg(test)]
 mod tests {
-    use ruff_diagnostics::{Applicability, Fix};
+    use ruff_diagnostics::Applicability;
    use ruff_text_size::{TextLen, TextRange, TextSize};

    use crate::diagnostic::{
@@ -698,107 +654,6 @@ print()
        ");
    }

-    /// Test that we remap notebook cell line numbers in the diff as well as the main diagnostic.
-    #[test]
-    fn notebook_output_with_diff() {
-        let (mut env, diagnostics) = create_notebook_diagnostics(DiagnosticFormat::Full);
-        env.show_fix_diff(true);
-        insta::assert_snapshot!(env.render_diagnostics(&diagnostics), @r"
-        error[unused-import][*]: `os` imported but unused
-         --> notebook.ipynb:cell 1:2:8
-          |
-        1 | # cell 1
-        2 | import os
-          |        ^^
-          |
-        help: Remove unused import: `os`
-
-        ℹ Safe fix
-           ::: cell 1
-        1 1 | # cell 1
-        2   |-import os
-
-        error[unused-import][*]: `math` imported but unused
-         --> notebook.ipynb:cell 2:2:8
-          |
-        1 | # cell 2
-        2 | import math
-          |        ^^^^
-        3 |
-        4 | print('hello world')
-          |
-        help: Remove unused import: `math`
-
-        ℹ Safe fix
-           ::: cell 2
-        1 1 | # cell 2
-        2   |-import math
-        3 2 | 
-        4 3 | print('hello world')
-
-        error[unused-variable]: Local variable `x` is assigned to but never used
-         --> notebook.ipynb:cell 3:4:5
-          |
-        2 | def foo():
-        3 |     print()
-        4 |     x = 1
-          |     ^
-          |
-        help: Remove assignment to unused variable `x`
-
-        ℹ Unsafe fix
-           ::: cell 3
-        1 1 | # cell 3
-        2 2 | def foo():
-        3 3 |     print()
-        4   |-    x = 1
-        5 4 |
-        ");
-    }
-
-    #[test]
-    fn notebook_output_with_diff_spanning_cells() {
-        let (mut env, mut diagnostics) = create_notebook_diagnostics(DiagnosticFormat::Full);
-        env.show_fix_diff(true);
-
-        // Move all of the edits from the later diagnostics to the first diagnostic to simulate a
-        // single diagnostic with edits in different cells.
-        let mut diagnostic = diagnostics.swap_remove(0);
-        let fix = diagnostic.fix_mut().unwrap();
-        let mut edits = fix.edits().to_vec();
-        for diag in diagnostics {
-            edits.extend_from_slice(diag.fix().unwrap().edits());
-        }
-        *fix = Fix::unsafe_edits(edits.remove(0), edits);
-
-        insta::assert_snapshot!(env.render(&diagnostic), @r"
-        error[unused-import]: `os` imported but unused
-         --> notebook.ipynb:cell 1:2:8
-          |
-        1 | # cell 1
-        2 | import os
-          |        ^^
-          |
-        help: Remove unused import: `os`
-
-        ℹ Unsafe fix
-           ::: cell 1
-        1 1 | # cell 1
-        2   |-import os
-           ::: cell 2
-        1 1 | # cell 2
-        2   |-import math
-        3 2 | 
-        4 3 | print('hello world')
-           ::: cell 3
-        1 1 | # cell 3
-        2 2 | def foo():
-        3 3 |     print()
-        4   |-    x = 1
-        5 4 |
-        ");
-    }
-
    /// Carriage return (`\r`) is a valid line-ending in Python, so we should normalize this to a
    /// line feed (`\n`) for rendering. Otherwise we report a single long line for this case.
    #[test]
--- a/crates/ruff_db/src/diagnostic/render/gitlab.rs
+++ b/crates/ruff_db/src/diagnostic/render/gitlab.rs
@@ -1,205 +0,0 @@
-use std::{
-    collections::HashSet,
-    hash::{DefaultHasher, Hash, Hasher},
-    path::Path,
-};
-
-use ruff_source_file::LineColumn;
-use serde::{Serialize, Serializer, ser::SerializeSeq};
-
-use crate::diagnostic::{Diagnostic, Severity};
-
-use super::FileResolver;
-
-pub(super) struct GitlabRenderer<'a> {
-    resolver: &'a dyn FileResolver,
-}
-
-impl<'a> GitlabRenderer<'a> {
-    pub(super) fn new(resolver: &'a dyn FileResolver) -> Self {
-        Self { resolver }
-    }
-}
-
-impl GitlabRenderer<'_> {
-    pub(super) fn render(
-        &self,
-        f: &mut std::fmt::Formatter,
-        diagnostics: &[Diagnostic],
-    ) -> std::fmt::Result {
-        write!(
-            f,
-            "{}",
-            serde_json::to_string_pretty(&SerializedMessages {
-                diagnostics,
-                resolver: self.resolver,
-                #[expect(
-                    clippy::disallowed_methods,
-                    reason = "We don't have access to a `System` here, \
-                              and this is only intended for use by GitLab CI, \
-                              which runs on a real `System`."
-                )]
-                project_dir: std::env::var("CI_PROJECT_DIR").ok().as_deref(),
-            })
-            .unwrap()
-        )
-    }
-}
-
-struct SerializedMessages<'a> {
-    diagnostics: &'a [Diagnostic],
-    resolver: &'a dyn FileResolver,
-    project_dir: Option<&'a str>,
-}
-
-impl Serialize for SerializedMessages<'_> {
-    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
-    where
-        S: Serializer,
-    {
-        let mut s = serializer.serialize_seq(Some(self.diagnostics.len()))?;
-        let mut fingerprints = HashSet::<u64>::with_capacity(self.diagnostics.len());
-
-        for diagnostic in self.diagnostics {
-            let location = diagnostic
-                .primary_span()
-                .map(|span| {
-                    let file = span.file();
-                    let positions = if self.resolver.is_notebook(file) {
-                        // We can't give a reasonable location for the structured formats,
-                        // so we show one that's clearly a fallback
-                        Default::default()
-                    } else {
-                        let diagnostic_source = file.diagnostic_source(self.resolver);
-                        let source_code = diagnostic_source.as_source_code();
-                        span.range()
-                            .map(|range| Positions {
-                                begin: source_code.line_column(range.start()),
-                                end: source_code.line_column(range.end()),
-                            })
-                            .unwrap_or_default()
-                    };
-
-                    let path = self.project_dir.as_ref().map_or_else(
-                        || file.relative_path(self.resolver).display().to_string(),
-                        |project_dir| relativize_path_to(file.path(self.resolver), project_dir),
-                    );
-
-                    Location { path, positions }
-                })
-                .unwrap_or_default();
-
-            let mut message_fingerprint = fingerprint(diagnostic, &location.path, 0);
-
-            // Make sure that we do not get a fingerprint that is already in use
-            // by adding in the previously generated one.
-            while fingerprints.contains(&message_fingerprint) {
-                message_fingerprint = fingerprint(diagnostic, &location.path, message_fingerprint);
-            }
-            fingerprints.insert(message_fingerprint);
-
-            let description = diagnostic.body();
-            let check_name = diagnostic.secondary_code_or_id();
-            let severity = match diagnostic.severity() {
-                Severity::Info => "info",
-                Severity::Warning => "minor",
-                Severity::Error => "major",
-                // Another option here is `blocker`
-                Severity::Fatal => "critical",
-            };
-
-            let value = Message {
-                check_name,
-                // GitLab doesn't display the separate `check_name` field in a Code Quality report,
-                // so prepend it to the description too.
-                description: format!("{check_name}: {description}"),
-                severity,
-                fingerprint: format!("{:x}", message_fingerprint),
-                location,
-            };
-
-            s.serialize_element(&value)?;
-        }
-
-        s.end()
-    }
-}
-
-#[derive(Serialize)]
-struct Message<'a> {
-    check_name: &'a str,
-    description: String,
-    severity: &'static str,
-    fingerprint: String,
-    location: Location,
-}
-
-/// The place in the source code where the issue was discovered.
-///
-/// According to the CodeClimate report format [specification] linked from the GitLab [docs], this
-/// field is required, so we fall back on a default `path` and position if the diagnostic doesn't
-/// have a primary span.
-///
-/// [specification]: https://github.com/codeclimate/platform/blob/master/spec/analyzers/SPEC.md#data-types
-/// [docs]: https://docs.gitlab.com/ci/testing/code_quality/#code-quality-report-format
-#[derive(Default, Serialize)]
-struct Location {
-    path: String,
-    positions: Positions,
-}
-
-#[derive(Default, Serialize)]
-struct Positions {
-    begin: LineColumn,
-    end: LineColumn,
-}
-
-/// Generate a unique fingerprint to identify a violation.
-fn fingerprint(diagnostic: &Diagnostic, project_path: &str, salt: u64) -> u64 {
-    let mut hasher = DefaultHasher::new();
-
-    salt.hash(&mut hasher);
-    diagnostic.name().hash(&mut hasher);
-    project_path.hash(&mut hasher);
-
-    hasher.finish()
-}
-
-/// Convert an absolute path to be relative to the specified project root.
-fn relativize_path_to<P: AsRef<Path>, R: AsRef<Path>>(path: P, project_root: R) -> String {
-    format!(
-        "{}",
-        pathdiff::diff_paths(&path, project_root)
-            .expect("Could not diff paths")
-            .display()
-    )
-}
-
-#[cfg(test)]
-mod tests {
-    use crate::diagnostic::{
-        DiagnosticFormat,
-        render::tests::{create_diagnostics, create_syntax_error_diagnostics},
-    };
-
-    const FINGERPRINT_FILTERS: [(&str, &str); 1] = [(
-        r#""fingerprint": "[a-z0-9]+","#,
-        r#""fingerprint": "<redacted>","#,
-    )];
-
-    #[test]
-    fn output() {
-        let (env, diagnostics) = create_diagnostics(DiagnosticFormat::Gitlab);
-        insta::with_settings!({filters => FINGERPRINT_FILTERS}, {
-            insta::assert_snapshot!(env.render_diagnostics(&diagnostics));
-        });
-    }
-
-    #[test]
-    fn syntax_errors() {
-        let (env, diagnostics) = create_syntax_error_diagnostics(DiagnosticFormat::Gitlab);
-        insta::with_settings!({filters => FINGERPRINT_FILTERS}, {
-            insta::assert_snapshot!(env.render_diagnostics(&diagnostics));
-        });
-    }
-}
--- a/crates/ruff_db/src/file_revision.rs
+++ b/crates/ruff_db/src/file_revision.rs
@@ -9,7 +9,17 @@ use crate::system::file_time_now;
 /// * The last modification time of the file.
 /// * The hash of the file's content.
 /// * The revision as it comes from an external system, for example the LSP.
-#[derive(Copy, Clone, Debug, Eq, PartialEq, Default, get_size2::GetSize)]
+#[derive(
+    Copy,
+    Clone,
+    Debug,
+    Eq,
+    PartialEq,
+    Default,
+    get_size2::GetSize,
+    serde::Serialize,
+    serde::Deserialize,
+)]
 pub struct FileRevision(u128);

 impl FileRevision {
--- a/crates/ruff_db/src/files.rs
+++ b/crates/ruff_db/src/files.rs
@@ -14,7 +14,7 @@ use crate::diagnostic::{Span, UnifiedFile};
 use crate::file_revision::FileRevision;
 use crate::files::file_root::FileRoots;
 use crate::files::private::FileStatus;
-use crate::system::{SystemPath, SystemPathBuf, SystemVirtualPath, SystemVirtualPathBuf};
+use crate::system::{FileType, SystemPath, SystemPathBuf, SystemVirtualPath, SystemVirtualPathBuf};
 use crate::vendored::{VendoredPath, VendoredPathBuf};
 use crate::{Db, FxDashMap, vendored};

@@ -139,6 +139,7 @@ impl Files {
                };

                tracing::trace!("Adding vendored file `{}`", path);
+
                let file = File::builder(FilePath::Vendored(path.to_path_buf()))
                    .permissions(Some(0o444))
                    .revision(metadata.revision())
@@ -200,7 +201,15 @@ impl Files {
        let mut roots = self.inner.roots.write().unwrap();

        let absolute = SystemPath::absolute(path, db.system().current_directory());
-        roots.try_add(db, absolute, kind)
+
+        let (Ok(root) | Err(root)) = roots.try_add(db, absolute, |absolute| {
+            FileRoot::builder(absolute, kind, FileRevision::now())
+                .durability(Durability::HIGH)
+                .revision_durability(kind.durability())
+                .new(db)
+        });
+
+        root
    }

    /// Updates the revision of the root for `path`.
@@ -259,6 +268,51 @@ impl Files {
            root.set_revision(db).to(FileRevision::now());
        }
    }
+
+    /// Seed the files with an existing [`File`] instance.
+    pub fn seed(&self, file: File, db: &dyn Db) {
+        let seeded = match file.path(db) {
+            FilePath::System(path) => self
+                .inner
+                .system_by_path
+                .insert(path.clone(), file)
+                .is_none(),
+            FilePath::SystemVirtual(path) => self
+                .inner
+                .system_virtual_by_path
+                .insert(path.clone(), VirtualFile(file))
+                .is_none(),
+            FilePath::Vendored(path) => self
+                .inner
+                .vendored_by_path
+                .insert(path.clone(), file)
+                .is_none(),
+        };
+
+        // Recreating a `File` input means the persisted queries depending on that file
+        // will be invalidated.
+        assert!(
+            seeded,
+            "unexpected `File` input recreated for path `{}`",
+            file.path(db)
+        );
+    }
+
+    /// Seed the files with an existing [`FileRoot`] instance.
+    pub fn seed_root(&self, root: FileRoot, db: &dyn Db) {
+        let mut roots = self.inner.roots.write().unwrap();
+        let seeded = roots
+            .try_add(db, root.path(db).to_path_buf(), |_| root)
+            .is_ok();
+
+        // Recreating a `FileRoot` input means the persisted queries depending on that file
+        // root will be invalidated.
+        assert!(
+            seeded,
+            "unexpected `FileRoot` input recreated for path `{}`",
+            root.path(db)
+        );
+    }
 }

 impl fmt::Debug for Files {
@@ -290,7 +344,7 @@ impl std::panic::RefUnwindSafe for Files {}
 /// # Ordering
 /// Ordering is based on the file's salsa-assigned id and not on its values.
 /// The id may change between runs.
-#[salsa::input(heap_size=ruff_memory_usage::heap_size)]
+#[salsa::input(persist, heap_size=ruff_memory_usage::heap_size)]
 #[derive(PartialOrd, Ord)]
 pub struct File {
    /// The path of the file (immutable).
@@ -414,6 +468,15 @@ impl File {
        }
    }

+    /// Loads all existing [`File`]s in the database.
+    pub fn load_all(db: &dyn Db) -> Vec<File> {
+        // TODO: Prune deleted paths.
+        File::ingredient(db)
+            .entries(db.zalsa())
+            .map(|entry| entry.as_struct())
+            .collect()
+    }
+
    /// Private method providing the implementation for [`Self::sync_path`] and [`Self::sync`] for
    /// system paths.
    fn sync_system_path(db: &mut dyn Db, path: &SystemPath, file: Option<File>) {
@@ -522,7 +585,17 @@ impl VirtualFile {
 // The types in here need to be public because they're salsa ingredients but we
 // don't want them to be publicly accessible. That's why we put them into a private module.
 mod private {
-    #[derive(Copy, Clone, Debug, Eq, PartialEq, Default, get_size2::GetSize)]
+    #[derive(
+        Copy,
+        Clone,
+        Debug,
+        Eq,
+        PartialEq,
+        Default,
+        get_size2::GetSize,
+        serde::Serialize,
+        serde::Deserialize,
+    )]
    pub enum FileStatus {
        /// The file exists.
        #[default]
@@ -536,6 +609,16 @@ mod private {
    }
 }

+impl From<FileType> for FileStatus {
+    fn from(value: FileType) -> Self {
+        match value {
+            FileType::File => FileStatus::Exists,
+            FileType::Symlink => FileStatus::Exists,
+            FileType::Directory => FileStatus::IsADirectory,
+        }
+    }
+}
+
 #[derive(Copy, Clone, Debug, Eq, PartialEq)]
 pub enum FileError {
    IsADirectory,
--- a/crates/ruff_db/src/files/file_root.rs
+++ b/crates/ruff_db/src/files/file_root.rs
@@ -16,7 +16,7 @@ use crate::system::{SystemPath, SystemPathBuf};
 /// The main usage of file roots is to determine a file's durability. But it can also be used
 /// to make a salsa query dependent on whether a file in a root has changed without writing any
 /// manual invalidation logic.
-#[salsa::input(debug, heap_size=ruff_memory_usage::heap_size)]
+#[salsa::input(persist, debug, heap_size=ruff_memory_usage::heap_size)]
 pub struct FileRoot {
    /// The path of a root is guaranteed to never change.
    #[returns(deref)]
@@ -35,9 +35,20 @@ impl FileRoot {
    pub fn durability(self, db: &dyn Db) -> salsa::Durability {
        self.kind_at_time_of_creation(db).durability()
    }
+
+    /// Loads all existing [`FileRoot`]s in the database.
+    pub fn load_all(db: &dyn Db) -> Vec<FileRoot> {
+        // TODO: Prune deleted paths.
+        FileRoot::ingredient(db)
+            .entries(db.zalsa())
+            .map(|entry| entry.as_struct())
+            .collect()
+    }
 }

-#[derive(Copy, Clone, Debug, Eq, PartialEq, get_size2::GetSize)]
+#[derive(
+    Copy, Clone, Debug, Eq, PartialEq, get_size2::GetSize, serde::Serialize, serde::Deserialize,
+)]
 pub enum FileRootKind {
    /// The root of a project.
    Project,
@@ -47,7 +58,7 @@ pub enum FileRootKind {
 }

 impl FileRootKind {
-    const fn durability(self) -> Durability {
+    pub const fn durability(self) -> Durability {
        match self {
            FileRootKind::Project => Durability::LOW,
            FileRootKind::LibrarySearchPath => Durability::HIGH,
@@ -62,34 +73,34 @@ pub(super) struct FileRoots {
 }

 impl FileRoots {
-    /// Tries to add a new root for `path` and returns the root.
+    /// Tries to add a new root for `path`.
    ///
    /// The root isn't added nor is the file root's kind updated if a root for `path` already exists.
+    ///
+    /// Returns `Ok(root)` if the `FileRoot` was successfully added, and returns `Err(root)` with
+    /// the previous root if one already existed at that path.
    pub(super) fn try_add(
        &mut self,
        db: &dyn Db,
        path: SystemPathBuf,
-        kind: FileRootKind,
-    ) -> FileRoot {
+        create_root: impl FnOnce(SystemPathBuf) -> FileRoot,
+    ) -> Result<FileRoot, FileRoot> {
        // SAFETY: Guaranteed to succeed because `path` is a UTF-8 that only contains Unicode characters.
        let normalized_path = path.as_std_path().to_slash().unwrap();

        if let Ok(existing) = self.by_path.at(&normalized_path) {
            // Only if it is an exact match
            if existing.value.path(db) == &*path {
-                return *existing.value;
+                return Err(*existing.value);
            }
        }

-        // normalize the path to use `/` separators and escape the '{' and '}' characters,
-        // which matchit uses for routing parameters
+        // Normalize the path to use `/` separators and escape the '{' and '}' characters,
+        // which `matchit` uses for routing parameters.
        let mut route = normalized_path.replace('{', "{{").replace('}', "}}");

        // Insert a new source root
-        let root = FileRoot::builder(path, kind, FileRevision::now())
-            .durability(Durability::HIGH)
-            .revision_durability(kind.durability())
-            .new(db);
+        let root = create_root(path);

        // Insert a path that matches the root itself
        self.by_path.insert(route.clone(), root).unwrap();
@@ -100,7 +111,7 @@ impl FileRoots {
        self.by_path.insert(route, root).unwrap();
        self.roots.push(root);

-        root
+        Ok(root)
    }

    /// Returns the closest root for `path` or `None` if no root contains `path`.
--- a/crates/ruff_db/src/files/path.rs
+++ b/crates/ruff_db/src/files/path.rs
@@ -11,7 +11,9 @@ use std::fmt::{Display, Formatter};
 /// * a file stored on the [host system](crate::system::System).
 /// * a virtual file stored on the [host system](crate::system::System).
 /// * a vendored file stored in the [vendored file system](crate::vendored::VendoredFileSystem).
-#[derive(Clone, Debug, Eq, PartialEq, Hash, get_size2::GetSize)]
+#[derive(
+    Clone, Debug, Eq, PartialEq, Hash, get_size2::GetSize, serde::Serialize, serde::Deserialize,
+)]
 pub enum FilePath {
    /// Path to a file on the [host system](crate::system::System).
    System(SystemPathBuf),
--- a/crates/ruff_db/src/source.rs
+++ b/crates/ruff_db/src/source.rs
@@ -148,7 +148,16 @@ impl From<Notebook> for SourceTextKind {
    }
 }

-#[derive(Debug, thiserror::Error, PartialEq, Eq, Clone, get_size2::GetSize)]
+#[derive(
+    Debug,
+    thiserror::Error,
+    PartialEq,
+    Eq,
+    Clone,
+    get_size2::GetSize,
+    serde::Serialize,
+    serde::Deserialize,
+)]
 pub enum SourceTextError {
    #[error("Failed to read notebook: {0}`")]
    FailedToReadNotebook(String),
--- a/crates/ruff_db/src/system.rs
+++ b/crates/ruff_db/src/system.rs
@@ -66,6 +66,9 @@ pub trait System: Debug + Sync + Send {
    /// See [dunce::canonicalize] for more information.
    fn canonicalize_path(&self, path: &SystemPath) -> Result<SystemPathBuf>;

+    /// Reads the content of the file at `path` into a bytes buffer.
+    fn read_to_end(&self, path: &SystemPath) -> Result<Vec<u8>>;
+
    /// Reads the content of the file at `path` into a [`String`].
    fn read_to_string(&self, path: &SystemPath) -> Result<String>;

@@ -242,7 +245,7 @@ pub trait WritableSystem: System {
    fn create_new_file(&self, path: &SystemPath) -> Result<()>;

    /// Writes the given content to the file at the given path.
-    fn write_file(&self, path: &SystemPath, content: &str) -> Result<()>;
+    fn write_file(&self, path: &SystemPath, content: &[u8]) -> Result<()>;

    /// Creates a directory at `path` as well as any intermediate directories.
    fn create_directory_all(&self, path: &SystemPath) -> Result<()>;
@@ -278,7 +281,7 @@ pub trait WritableSystem: System {
        // ensures that only one thread/process ever attempts to write to it to avoid corrupting
        // the cache.
        self.create_new_file(&cache_path)?;
-        self.write_file(&cache_path, &contents)?;
+        self.write_file(&cache_path, contents.as_bytes())?;

        Ok(Some(cache_path))
    }
--- a/crates/ruff_db/src/system/memory_fs.rs
+++ b/crates/ruff_db/src/system/memory_fs.rs
@@ -114,8 +114,8 @@ impl MemoryFileSystem {
        matches!(by_path.get(&normalized), Some(Entry::Directory(_)))
    }

-    pub fn read_to_string(&self, path: impl AsRef<SystemPath>) -> Result<String> {
-        fn read_to_string(fs: &MemoryFileSystem, path: &SystemPath) -> Result<String> {
+    pub fn read_to_end(&self, path: impl AsRef<SystemPath>) -> Result<Vec<u8>> {
+        fn read_to_end(fs: &MemoryFileSystem, path: &SystemPath) -> Result<Vec<u8>> {
            let by_path = fs.inner.by_path.read().unwrap();
            let normalized = fs.normalize_path(path);

@@ -127,13 +127,18 @@ impl MemoryFileSystem {
            }
        }

-        read_to_string(self, path.as_ref())
+        read_to_end(self, path.as_ref())
    }

-    pub(crate) fn read_virtual_path_to_string(
+    pub fn read_to_string(&self, path: impl AsRef<SystemPath>) -> Result<String> {
+        self.read_to_end(path)
+            .and_then(|bytes| String::from_utf8(bytes).map_err(io::Error::other))
+    }
+
+    pub(crate) fn read_virtual_path_to_end(
        &self,
        path: impl AsRef<SystemVirtualPath>,
-    ) -> Result<String> {
+    ) -> Result<Vec<u8>> {
        let virtual_files = self.inner.virtual_files.read().unwrap();
        let file = virtual_files
            .get(&path.as_ref().to_path_buf())
@@ -142,6 +147,14 @@ impl MemoryFileSystem {
        Ok(file.content.clone())
    }

+    pub(crate) fn read_virtual_path_to_string(
+        &self,
+        path: impl AsRef<SystemVirtualPath>,
+    ) -> Result<String> {
+        self.read_virtual_path_to_end(path)
+            .and_then(|bytes| String::from_utf8(bytes).map_err(io::Error::other))
+    }
+
    pub fn exists(&self, path: &SystemPath) -> bool {
        let by_path = self.inner.by_path.read().unwrap();
        let normalized = self.normalize_path(path);
@@ -161,7 +174,7 @@ impl MemoryFileSystem {
        match by_path.entry(normalized) {
            btree_map::Entry::Vacant(entry) => {
                entry.insert(Entry::File(File {
-                    content: String::new(),
+                    content: Vec::new(),
                    last_modified: file_time_now(),
                }));

@@ -177,13 +190,17 @@ impl MemoryFileSystem {
    /// Stores a new file in the file system.
    ///
    /// The operation overrides the content for an existing file with the same normalized `path`.
-    pub fn write_file(&self, path: impl AsRef<SystemPath>, content: impl ToString) -> Result<()> {
+    pub fn write_file(
+        &self,
+        path: impl AsRef<SystemPath>,
+        content: impl Into<Vec<u8>>,
+    ) -> Result<()> {
        let mut by_path = self.inner.by_path.write().unwrap();

        let normalized = self.normalize_path(path.as_ref());

        let file = get_or_create_file(&mut by_path, &normalized)?;
-        file.content = content.to_string();
+        file.content = content.into();
        file.last_modified = file_time_now();

        Ok(())
@@ -214,7 +231,7 @@ impl MemoryFileSystem {
    pub fn write_file_all(
        &self,
        path: impl AsRef<SystemPath>,
-        content: impl ToString,
+        content: impl Into<Vec<u8>>,
    ) -> Result<()> {
        let path = path.as_ref();

@@ -228,19 +245,23 @@ impl MemoryFileSystem {
    /// Stores a new virtual file in the file system.
    ///
    /// The operation overrides the content for an existing virtual file with the same `path`.
-    pub fn write_virtual_file(&self, path: impl AsRef<SystemVirtualPath>, content: impl ToString) {
+    pub fn write_virtual_file(
+        &self,
+        path: impl AsRef<SystemVirtualPath>,
+        content: impl Into<Vec<u8>>,
+    ) {
        let path = path.as_ref();
        let mut virtual_files = self.inner.virtual_files.write().unwrap();

        match virtual_files.entry(path.to_path_buf()) {
            std::collections::hash_map::Entry::Vacant(entry) => {
                entry.insert(File {
-                    content: content.to_string(),
+                    content: content.into(),
                    last_modified: file_time_now(),
                });
            }
            std::collections::hash_map::Entry::Occupied(mut entry) => {
-                entry.get_mut().content = content.to_string();
+                entry.get_mut().content = content.into();
            }
        }
    }
@@ -468,7 +489,7 @@ impl Entry {

 #[derive(Debug)]
 struct File {
-    content: String,
+    content: Vec<u8>,
    last_modified: FileTime,
 }

@@ -533,7 +554,7 @@ fn get_or_create_file<'a>(

    let entry = paths.entry(normalized.to_path_buf()).or_insert_with(|| {
        Entry::File(File {
-            content: String::new(),
+            content: Vec::new(),
            last_modified: file_time_now(),
        })
    });
--- a/crates/ruff_db/src/system/os.rs
+++ b/crates/ruff_db/src/system/os.rs
@@ -93,6 +93,10 @@ impl System for OsSystem {
        })
    }

+    fn read_to_end(&self, path: &SystemPath) -> Result<Vec<u8>> {
+        std::fs::read(path.as_std_path())
+    }
+
    fn read_to_string(&self, path: &SystemPath) -> Result<String> {
        std::fs::read_to_string(path.as_std_path())
    }
@@ -357,7 +361,7 @@ impl WritableSystem for OsSystem {
        std::fs::File::create_new(path).map(drop)
    }

-    fn write_file(&self, path: &SystemPath, content: &str) -> Result<()> {
+    fn write_file(&self, path: &SystemPath, content: &[u8]) -> Result<()> {
        std::fs::write(path.as_std_path(), content)
    }

--- a/crates/ruff_db/src/system/path.rs
+++ b/crates/ruff_db/src/system/path.rs
@@ -762,7 +762,17 @@ impl SystemVirtualPath {
 }

 /// An owned, virtual path on [`System`](`super::System`) (akin to [`String`]).
-#[derive(Eq, PartialEq, Clone, Hash, PartialOrd, Ord, get_size2::GetSize)]
+#[derive(
+    Eq,
+    PartialEq,
+    Clone,
+    Hash,
+    PartialOrd,
+    Ord,
+    get_size2::GetSize,
+    serde::Serialize,
+    serde::Deserialize,
+)]
 pub struct SystemVirtualPathBuf(String);

 impl SystemVirtualPathBuf {
--- a/crates/ruff_db/src/system/test.rs
+++ b/crates/ruff_db/src/system/test.rs
@@ -75,6 +75,10 @@ impl System for TestSystem {
        self.system().canonicalize_path(path)
    }

+    fn read_to_end(&self, path: &SystemPath) -> Result<Vec<u8>> {
+        self.system().read_to_end(path)
+    }
+
    fn read_to_string(&self, path: &SystemPath) -> Result<String> {
        self.system().read_to_string(path)
    }
@@ -165,7 +169,7 @@ impl WritableSystem for TestSystem {
        self.system().create_new_file(path)
    }

-    fn write_file(&self, path: &SystemPath, content: &str) -> Result<()> {
+    fn write_file(&self, path: &SystemPath, content: &[u8]) -> Result<()> {
        self.system().write_file(path, content)
    }

@@ -185,7 +189,9 @@ pub trait DbWithWritableSystem: Db + Sized {
    /// Writes the content of the given file and notifies the Db about the change.
    fn write_file(&mut self, path: impl AsRef<SystemPath>, content: impl AsRef<str>) -> Result<()> {
        let path = path.as_ref();
-        match self.writable_system().write_file(path, content.as_ref()) {
+        let content = content.as_ref();
+
+        match self.writable_system().write_file(path, content.as_bytes()) {
            Ok(()) => {
                File::sync_path(self, path);
                Ok(())
@@ -198,7 +204,8 @@ pub trait DbWithWritableSystem: Db + Sized {
                        File::sync_path(self, ancestor);
                    }

-                    self.writable_system().write_file(path, content.as_ref())?;
+                    self.writable_system()
+                        .write_file(path, content.as_bytes())?;
                    File::sync_path(self, path);

                    Ok(())
@@ -243,8 +250,14 @@ pub trait DbWithTestSystem: Db + Sized {
    ///
    /// ## Panics
    /// If the db isn't using the [`InMemorySystem`].
-    fn write_virtual_file(&mut self, path: impl AsRef<SystemVirtualPath>, content: impl ToString) {
+    fn write_virtual_file(
+        &mut self,
+        path: impl AsRef<SystemVirtualPath>,
+        content: impl Into<Vec<u8>>,
+    ) {
        let path = path.as_ref();
+        let content = content.into();
+
        self.test_system()
            .memory_file_system()
            .write_virtual_file(path, content);
@@ -322,6 +335,10 @@ impl System for InMemorySystem {
        self.memory_fs.canonicalize(path)
    }

+    fn read_to_end(&self, path: &SystemPath) -> Result<Vec<u8>> {
+        self.memory_fs.read_to_end(path)
+    }
+
    fn read_to_string(&self, path: &SystemPath) -> Result<String> {
        self.memory_fs.read_to_string(path)
    }
@@ -412,7 +429,7 @@ impl WritableSystem for InMemorySystem {
        self.memory_fs.create_new_file(path)
    }

-    fn write_file(&self, path: &SystemPath, content: &str) -> Result<()> {
+    fn write_file(&self, path: &SystemPath, content: &[u8]) -> Result<()> {
        self.memory_fs.write_file(path, content)
    }

--- a/crates/ruff_db/src/vendored/path.rs
+++ b/crates/ruff_db/src/vendored/path.rs
@@ -88,7 +88,7 @@ impl ToOwned for VendoredPath {
 }

 #[repr(transparent)]
-#[derive(Debug, Eq, PartialEq, Clone, Hash)]
+#[derive(Debug, Eq, PartialEq, Clone, Hash, serde::Serialize, serde::Deserialize)]
 pub struct VendoredPathBuf(Utf8PathBuf);

 impl get_size2::GetSize for VendoredPathBuf {
--- a/crates/ruff_dev/Cargo.toml
+++ b/crates/ruff_dev/Cargo.toml
@@ -13,7 +13,6 @@ license = { workspace = true }
 [dependencies]
 ty = { workspace = true }
 ty_project = { workspace = true, features = ["schemars"] }
-ty_python_semantic = { workspace = true }
 ty_static = { workspace = true }
 ruff = { workspace = true }
 ruff_formatter = { workspace = true }
--- a/crates/ruff_dev/src/generate_ty_rules.rs
+++ b/crates/ruff_dev/src/generate_ty_rules.rs
@@ -52,7 +52,7 @@ pub(crate) fn main(args: &Args) -> Result<()> {
 }

 fn generate_markdown() -> String {
-    let registry = ty_python_semantic::default_lint_registry();
+    let registry = &*ty_project::DEFAULT_LINT_REGISTRY;

    let mut output = String::new();

--- a/crates/ruff_linter/Cargo.toml
+++ b/crates/ruff_linter/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "ruff_linter"
-version = "0.12.11"
+version = "0.12.10"
 publish = false
 authors = { workspace = true }
 edition = { workspace = true }
@@ -51,6 +51,7 @@ path-absolutize = { workspace = true, features = [
    "once_cell_cache",
    "use_unix_paths_on_wasm",
 ] }
+pathdiff = { workspace = true }
 pep440_rs = { workspace = true }
 pyproject-toml = { workspace = true }
 regex = { workspace = true }
--- a/crates/ruff_linter/resources/test/fixtures/airflow/AIR311_names.py
+++ b/crates/ruff_linter/resources/test/fixtures/airflow/AIR311_names.py
@@ -34,7 +34,7 @@ task_group()
 setup()
 from airflow.decorators import teardown
 from airflow.io.path import ObjectStoragePath
-from airflow.io.store import attach
+from airflow.io.storage import attach
 from airflow.models import DAG as DAGFromModel
 from airflow.models import (
    Connection,
@@ -74,36 +74,3 @@ DatasetOrTimeSchedule()

 # airflow.utils.dag_parsing_context
 get_parsing_context()
-
-from airflow.decorators.base import (
-    DecoratedMappedOperator,
-    DecoratedOperator,
-    TaskDecorator,
-    get_unique_task_id,
-    task_decorator_factory,
-)
-
-# airflow.decorators.base
-DecoratedMappedOperator()
-DecoratedOperator()
-TaskDecorator()
-get_unique_task_id()
-task_decorator_factory()
-
-
-from airflow.models import Param
-
-# airflow.models
-Param()
-
-
-from airflow.sensors.base import (
-    BaseSensorOperator,
-    PokeReturnValue,
-    poke_mode_only,
-)
-
-# airflow.sensors.base
-BaseSensorOperator()
-PokeReturnValue()
-poke_mode_only()
--- a/crates/ruff_linter/resources/test/fixtures/airflow/AIR312.py
+++ b/crates/ruff_linter/resources/test/fixtures/airflow/AIR312.py
@@ -9,6 +9,7 @@ from airflow.operators.empty import EmptyOperator
 from airflow.operators.latest_only import LatestOnlyOperator
 from airflow.operators.trigger_dagrun import TriggerDagRunOperator
 from airflow.operators.weekday import BranchDayOfWeekOperator
+from airflow.sensors.date_time import DateTimeSensor

 FSHook()
 PackageIndexHook()
@@ -21,6 +22,7 @@ EmptyOperator()

 LatestOnlyOperator()
 BranchDayOfWeekOperator()
+DateTimeSensor()

 from airflow.operators.python import (
    BranchPythonOperator,
@@ -28,23 +30,16 @@ from airflow.operators.python import (
    PythonVirtualenvOperator,
    ShortCircuitOperator,
 )
-from airflow.sensors.bash import BashSensor
-from airflow.sensors.date_time import DateTimeSensor
-
-BranchPythonOperator()
-PythonOperator()
-PythonVirtualenvOperator()
-ShortCircuitOperator()
-
-BashSensor()
-DateTimeSensor()
 from airflow.sensors.date_time import DateTimeSensorAsync
 from airflow.sensors.external_task import (
    ExternalTaskMarker,
    ExternalTaskSensor,
 )
+from airflow.sensors.time_sensor import (
+    TimeSensor,
+    TimeSensorAsync,
+)
 from airflow.sensors.filesystem import FileSensor
-from airflow.sensors.python import PythonSensor

 BranchPythonOperator()
 PythonOperator()
@@ -54,13 +49,6 @@ DateTimeSensorAsync()
 ExternalTaskMarker()
 ExternalTaskSensor()
 FileSensor()
-PythonSensor()
-
-from airflow.sensors.time_sensor import (
-    TimeSensor,
-    TimeSensorAsync,
-)
-
 TimeSensor()
 TimeSensorAsync()

--- a/crates/ruff_linter/resources/test/fixtures/flake8_async/ASYNC212.py
+++ b/crates/ruff_linter/resources/test/fixtures/flake8_async/ASYNC212.py
@@ -1,75 +0,0 @@
-from typing import Optional
-
-import httpx
-
-
-def foo():
-    client = httpx.Client()
-    client.close()  # Ok
-    client.delete()  # Ok
-    client.get()  # Ok
-    client.head()  # Ok
-    client.options()  # Ok
-    client.patch()  # Ok
-    client.post()  # Ok
-    client.put()  # Ok
-    client.request()  # Ok
-    client.send()  # Ok
-    client.stream()  # Ok
-
-    client.anything()  # Ok
-    client.build_request()  # Ok
-    client.is_closed  # Ok
-
-
-async def foo():
-    client = httpx.Client()
-    client.close()  # ASYNC212
-    client.delete()  # ASYNC212
-    client.get()  # ASYNC212
-    client.head()  # ASYNC212
-    client.options()  # ASYNC212
-    client.patch()  # ASYNC212
-    client.post()  # ASYNC212
-    client.put()  # ASYNC212
-    client.request()  # ASYNC212
-    client.send()  # ASYNC212
-    client.stream()  # ASYNC212
-
-    client.anything()  # Ok
-    client.build_request()  # Ok
-    client.is_closed  # Ok
-
-
-async def foo(client: httpx.Client):
-    client.request()  # ASYNC212
-    client.anything()  # Ok
-
-
-async def foo(client: httpx.Client | None):
-    client.request()  # ASYNC212
-    client.anything()  # Ok
-
-
-async def foo(client: Optional[httpx.Client]):
-    client.request()  # ASYNC212
-    client.anything()  # Ok
-
-
-async def foo():
-    client: httpx.Client = ...
-    client.request()  # ASYNC212
-    client.anything()  # Ok
-
-
-global_client = httpx.Client()
-
-
-async def foo():
-    global_client.request()  # ASYNC212
-    global_client.anything()  # Ok
-
-
-async def foo():
-    async with httpx.AsyncClient() as client:
-        await client.get()  # Ok
--- a/crates/ruff_linter/resources/test/fixtures/flake8_logging_format/G004.py
+++ b/crates/ruff_linter/resources/test/fixtures/flake8_logging_format/G004.py
@@ -17,50 +17,3 @@ info(f"{__name__}")
 # Don't trigger for t-strings
 info(t"{name}")
 info(t"{__name__}")
-
-count = 5
-total = 9
-directory_path = "/home/hamir/ruff/crates/ruff_linter/resources/test/"
-logging.info(f"{count} out of {total} files in {directory_path} checked")
-
-
-
-x = 99
-fmt = "08d"
-logger.info(f"{x:{'08d'}}")
-logger.info(f"{x:>10} {x:{fmt}}")
-
-logging.info(f"")
-logging.info(f"This message doesn't have any variables.")
-
-obj = {"key": "value"}
-logging.info(f"Object: {obj!r}")
-
-items_count = 3
-logging.warning(f"Items: {items_count:d}")
-
-data = {"status": "active"}
-logging.info(f"Processing {len(data)} items")
-logging.info(f"Status: {data.get('status', 'unknown').upper()}")
-
-
-result = 123
-logging.info(f"Calculated result: {result + 100}")
-
-temperature = 123
-logging.info(f"Temperature: {temperature:.1f}°C")
-
-class FilePath:
-    def __init__(self, name: str):
-        self.name = name
-
-logging.info(f"No changes made to {file_path.name}.")
-
-user = "tron"
-balance = 123.45
-logging.error(f"Error {404}: User {user} has insufficient balance ${balance:.2f}")
-
-import logging
-
-x = 1
-logging.error(f"{x} -> %s", x)
--- a/crates/ruff_linter/resources/test/fixtures/flake8_logging_format/G004_arg_order.py
+++ b/crates/ruff_linter/resources/test/fixtures/flake8_logging_format/G004_arg_order.py
@@ -1,10 +0,0 @@
-"""Test f-string argument order."""
-
-import logging
-
-logger = logging.getLogger(__name__)
-
-X = 1
-Y = 2
-logger.error(f"{X} -> %s", Y)
-logger.error(f"{Y} -> %s", X)
--- a/crates/ruff_linter/resources/test/fixtures/pyflakes/F401_35.py
+++ b/crates/ruff_linter/resources/test/fixtures/pyflakes/F401_35.py
@@ -1,16 +0,0 @@
-"""
-Test: allowed-unused-imports-top-level-module
-"""
-
-# No errors
-
-def f():
-    import hvplot
-def f():
-    import hvplot.pandas
-def f():
-    import hvplot.pandas.plots
-def f():
-    from hvplot.pandas import scatter_matrix
-def f():
-    from hvplot.pandas.plots import scatter_matrix
--- a/crates/ruff_linter/resources/test/fixtures/pyflakes/F841_0.py
+++ b/crates/ruff_linter/resources/test/fixtures/pyflakes/F841_0.py
@@ -151,39 +151,3 @@ def f():
        pass
    except Exception as _:
        pass
-
-
-# OK, `__class__` in this case is not the special `__class__` cell, so we don't
-# emit a diagnostic. (It has its own special semantics -- see
-# https://github.com/astral-sh/ruff/pull/20048#discussion_r2298338048 -- but
-# those aren't relevant here.)
-class A:
-    __class__ = 1
-
-
-# The following three cases are flagged because they declare local `__class__`
-# variables that don't refer to the special `__class__` cell.
-class A:
-    def set_class(self, cls):
-        __class__ = cls  # F841
-
-
-class A:
-    class B:
-        def set_class(self, cls):
-            __class__ = cls  # F841
-
-
-class A:
-    def foo():
-        class B:
-            print(__class__)
-            def set_class(self, cls):
-                __class__ = cls  # F841
-
-
-# OK, the `__class__` cell is nonlocal and declared as such.
-class NonlocalDunderClass:
-    def foo():
-        nonlocal __class__
-        __class__ = 1
--- a/crates/ruff_linter/resources/test/fixtures/pylint/nonlocal_without_binding.py
+++ b/crates/ruff_linter/resources/test/fixtures/pylint/nonlocal_without_binding.py
@@ -44,8 +44,3 @@ def f():
    def g():
        nonlocal x
        x = 2
-
-# OK
-class A:
-    def method(self):
-        nonlocal __class__
--- a/crates/ruff_linter/resources/test/fixtures/ruff/RUF033.py
+++ b/crates/ruff_linter/resources/test/fixtures/ruff/RUF033.py
@@ -124,19 +124,3 @@ def fun_with_python_syntax():
            ...

    return Foo
-
-
-@dataclass
-class C:
-    def __post_init__(self, x: tuple[int, ...] = (
-        1,
-        2,
-    )) -> None:
-        self.x = x
-
-
-@dataclass
-class D:
-    def __post_init__(self, x: int = """
-    """) -> None:
-        self.x = x
--- a/crates/ruff_linter/src/checkers/ast/analyze/expression.rs
+++ b/crates/ruff_linter/src/checkers/ast/analyze/expression.rs
@@ -660,9 +660,6 @@ pub(crate) fn expression(expr: &Expr, checker: &Checker) {
            if checker.is_rule_enabled(Rule::BlockingHttpCallInAsyncFunction) {
                flake8_async::rules::blocking_http_call(checker, call);
            }
-            if checker.is_rule_enabled(Rule::BlockingHttpCallHttpxInAsyncFunction) {
-                flake8_async::rules::blocking_http_call_httpx(checker, call);
-            }
            if checker.is_rule_enabled(Rule::BlockingOpenCallInAsyncFunction) {
                flake8_async::rules::blocking_open_call(checker, call);
            }
@@ -1049,6 +1046,7 @@ pub(crate) fn expression(expr: &Expr, checker: &Checker) {
                Rule::PyPath,
                Rule::Glob,
                Rule::OsListdir,
+                Rule::OsSymlink,
            ]) {
                flake8_use_pathlib::rules::replaceable_by_pathlib(checker, call);
            }
--- a/crates/ruff_linter/src/checkers/ast/mod.rs
+++ b/crates/ruff_linter/src/checkers/ast/mod.rs
@@ -703,10 +703,7 @@ impl SemanticSyntaxContext for Checker<'_> {
            match scope.kind {
                ScopeKind::Class(_) | ScopeKind::Lambda(_) => return false,
                ScopeKind::Function(ast::StmtFunctionDef { is_async, .. }) => return *is_async,
-                ScopeKind::Generator { .. }
-                | ScopeKind::Module
-                | ScopeKind::Type
-                | ScopeKind::DunderClassCell => {}
+                ScopeKind::Generator { .. } | ScopeKind::Module | ScopeKind::Type => {}
            }
        }
        false
@@ -717,10 +714,7 @@ impl SemanticSyntaxContext for Checker<'_> {
            match scope.kind {
                ScopeKind::Class(_) => return false,
                ScopeKind::Function(_) | ScopeKind::Lambda(_) => return true,
-                ScopeKind::Generator { .. }
-                | ScopeKind::Module
-                | ScopeKind::Type
-                | ScopeKind::DunderClassCell => {}
+                ScopeKind::Generator { .. } | ScopeKind::Module | ScopeKind::Type => {}
            }
        }
        false
@@ -731,7 +725,7 @@ impl SemanticSyntaxContext for Checker<'_> {
            match scope.kind {
                ScopeKind::Class(_) | ScopeKind::Generator { .. } => return false,
                ScopeKind::Function(_) | ScopeKind::Lambda(_) => return true,
-                ScopeKind::Module | ScopeKind::Type | ScopeKind::DunderClassCell => {}
+                ScopeKind::Module | ScopeKind::Type => {}
            }
        }
        false
@@ -1098,24 +1092,6 @@ impl<'a> Visitor<'a> for Checker<'a> {
                    }
                }

-                // Here we add the implicit scope surrounding a method which allows code in the
-                // method to access `__class__` at runtime. See the `ScopeKind::DunderClassCell`
-                // docs for more information.
-                let added_dunder_class_scope = if self.semantic.current_scope().kind.is_class() {
-                    self.semantic.push_scope(ScopeKind::DunderClassCell);
-                    let binding_id = self.semantic.push_binding(
-                        TextRange::default(),
-                        BindingKind::DunderClassCell,
-                        BindingFlags::empty(),
-                    );
-                    self.semantic
-                        .current_scope_mut()
-                        .add("__class__", binding_id);
-                    true
-                } else {
-                    false
-                };
-
                self.semantic.push_scope(ScopeKind::Type);

                if let Some(type_params) = type_params {
@@ -1179,9 +1155,6 @@ impl<'a> Visitor<'a> for Checker<'a> {
                self.semantic.pop_scope(); // Function scope
                self.semantic.pop_definition();
                self.semantic.pop_scope(); // Type parameter scope
-                if added_dunder_class_scope {
-                    self.semantic.pop_scope(); // `__class__` cell closure scope
-                }
                self.add_binding(
                    name,
                    stmt.identifier(),
--- a/crates/ruff_linter/src/codes.rs
+++ b/crates/ruff_linter/src/codes.rs
@@ -336,7 +336,6 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<(RuleGroup, Rule)> {
        (Flake8Async, "115") => (RuleGroup::Stable, rules::flake8_async::rules::AsyncZeroSleep),
        (Flake8Async, "116") => (RuleGroup::Preview, rules::flake8_async::rules::LongSleepNotForever),
        (Flake8Async, "210") => (RuleGroup::Stable, rules::flake8_async::rules::BlockingHttpCallInAsyncFunction),
-        (Flake8Async, "212") => (RuleGroup::Preview, rules::flake8_async::rules::BlockingHttpCallHttpxInAsyncFunction),
        (Flake8Async, "220") => (RuleGroup::Stable, rules::flake8_async::rules::CreateSubprocessInAsyncFunction),
        (Flake8Async, "221") => (RuleGroup::Stable, rules::flake8_async::rules::RunProcessInAsyncFunction),
        (Flake8Async, "222") => (RuleGroup::Stable, rules::flake8_async::rules::WaitForProcessInAsyncFunction),
--- a/crates/ruff_linter/src/fs.rs
+++ b/crates/ruff_linter/src/fs.rs
@@ -58,3 +58,13 @@ pub fn relativize_path<P: AsRef<Path>>(path: P) -> String {
    }
    format!("{}", path.display())
 }
+
+/// Convert an absolute path to be relative to the specified project root.
+pub fn relativize_path_to<P: AsRef<Path>, R: AsRef<Path>>(path: P, project_root: R) -> String {
+    format!(
+        "{}",
+        pathdiff::diff_paths(&path, project_root)
+            .expect("Could not diff paths")
+            .display()
+    )
+}
--- a/crates/ruff_linter/src/message/gitlab.rs
+++ b/crates/ruff_linter/src/message/gitlab.rs
@@ -0,0 +1,174 @@
+use std::collections::HashSet;
+use std::collections::hash_map::DefaultHasher;
+use std::hash::{Hash, Hasher};
+use std::io::Write;
+
+use serde::ser::SerializeSeq;
+use serde::{Serialize, Serializer};
+use serde_json::json;
+
+use ruff_db::diagnostic::Diagnostic;
+
+use crate::fs::{relativize_path, relativize_path_to};
+use crate::message::{Emitter, EmitterContext};
+
+/// Generate JSON with violations in GitLab CI format
+//  https://docs.gitlab.com/ee/ci/testing/code_quality.html#implement-a-custom-tool
+pub struct GitlabEmitter {
+    project_dir: Option<String>,
+}
+
+impl Default for GitlabEmitter {
+    fn default() -> Self {
+        Self {
+            project_dir: std::env::var("CI_PROJECT_DIR").ok(),
+        }
+    }
+}
+
+impl Emitter for GitlabEmitter {
+    fn emit(
+        &mut self,
+        writer: &mut dyn Write,
+        diagnostics: &[Diagnostic],
+        context: &EmitterContext,
+    ) -> anyhow::Result<()> {
+        serde_json::to_writer_pretty(
+            writer,
+            &SerializedMessages {
+                diagnostics,
+                context,
+                project_dir: self.project_dir.as_deref(),
+            },
+        )?;
+
+        Ok(())
+    }
+}
+
+struct SerializedMessages<'a> {
+    diagnostics: &'a [Diagnostic],
+    context: &'a EmitterContext<'a>,
+    project_dir: Option<&'a str>,
+}
+
+impl Serialize for SerializedMessages<'_> {
+    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
+    where
+        S: Serializer,
+    {
+        let mut s = serializer.serialize_seq(Some(self.diagnostics.len()))?;
+        let mut fingerprints = HashSet::<u64>::with_capacity(self.diagnostics.len());
+
+        for diagnostic in self.diagnostics {
+            let filename = diagnostic.expect_ruff_filename();
+
+            let (start_location, end_location) = if self.context.is_notebook(&filename) {
+                // We can't give a reasonable location for the structured formats,
+                // so we show one that's clearly a fallback
+                Default::default()
+            } else {
+                (
+                    diagnostic.expect_ruff_start_location(),
+                    diagnostic.expect_ruff_end_location(),
+                )
+            };
+
+            let path = self.project_dir.as_ref().map_or_else(
+                || relativize_path(&filename),
+                |project_dir| relativize_path_to(&filename, project_dir),
+            );
+
+            let mut message_fingerprint = fingerprint(diagnostic, &path, 0);
+
+            // Make sure that we do not get a fingerprint that is already in use
+            // by adding in the previously generated one.
+            while fingerprints.contains(&message_fingerprint) {
+                message_fingerprint = fingerprint(diagnostic, &path, message_fingerprint);
+            }
+            fingerprints.insert(message_fingerprint);
+
+            let description = diagnostic.body();
+            let check_name = diagnostic.secondary_code_or_id();
+
+            let value = json!({
+                "check_name": check_name,
+                // GitLab doesn't display the separate `check_name` field in a Code Quality report,
+                // so prepend it to the description too.
+                "description": format!("{check_name}: {description}"),
+                "severity": "major",
+                "fingerprint": format!("{:x}", message_fingerprint),
+                "location": {
+                    "path": path,
+                    "positions": {
+                        "begin": start_location,
+                        "end": end_location,
+                    },
+                },
+            });
+
+            s.serialize_element(&value)?;
+        }
+
+        s.end()
+    }
+}
+
+/// Generate a unique fingerprint to identify a violation.
+fn fingerprint(message: &Diagnostic, project_path: &str, salt: u64) -> u64 {
+    let mut hasher = DefaultHasher::new();
+
+    salt.hash(&mut hasher);
+    message.name().hash(&mut hasher);
+    project_path.hash(&mut hasher);
+
+    hasher.finish()
+}
+
+#[cfg(test)]
+mod tests {
+    use insta::assert_snapshot;
+
+    use crate::message::GitlabEmitter;
+    use crate::message::tests::{
+        capture_emitter_output, create_diagnostics, create_syntax_error_diagnostics,
+    };
+
+    #[test]
+    fn output() {
+        let mut emitter = GitlabEmitter::default();
+        let content = capture_emitter_output(&mut emitter, &create_diagnostics());
+
+        assert_snapshot!(redact_fingerprint(&content));
+    }
+
+    #[test]
+    fn syntax_errors() {
+        let mut emitter = GitlabEmitter::default();
+        let content = capture_emitter_output(&mut emitter, &create_syntax_error_diagnostics());
+
+        assert_snapshot!(redact_fingerprint(&content));
+    }
+
+    // Redact the fingerprint because the default hasher isn't stable across platforms.
+    fn redact_fingerprint(content: &str) -> String {
+        static FINGERPRINT_HAY_KEY: &str = r#""fingerprint": ""#;
+
+        let mut output = String::with_capacity(content.len());
+        let mut last = 0;
+
+        for (start, _) in content.match_indices(FINGERPRINT_HAY_KEY) {
+            let fingerprint_hash_start = start + FINGERPRINT_HAY_KEY.len();
+            output.push_str(&content[last..fingerprint_hash_start]);
+            output.push_str("<redacted>");
+            last = fingerprint_hash_start
+                + content[fingerprint_hash_start..]
+                    .find('"')
+                    .expect("Expected terminating quote");
+        }
+
+        output.push_str(&content[last..]);
+
+        output
+    }
+}
--- a/crates/ruff_linter/src/message/mod.rs
+++ b/crates/ruff_linter/src/message/mod.rs
@@ -10,6 +10,7 @@ use ruff_db::diagnostic::{
 use ruff_db::files::File;

 pub use github::GithubEmitter;
+pub use gitlab::GitlabEmitter;
 pub use grouped::GroupedEmitter;
 use ruff_notebook::NotebookIndex;
 use ruff_source_file::SourceFile;
@@ -21,6 +22,7 @@ use crate::Fix;
 use crate::registry::Rule;

 mod github;
+mod gitlab;
 mod grouped;
 mod sarif;
 mod text;
--- a/crates/ruff_db/src/diagnostic/render/snapshots/ruff_dbdiagnosticrendergitlabtests__output.snap
+++ b/crates/ruff_db/src/diagnostic/render/snapshots/ruff_dbdiagnosticrendergitlabtests__output.snap
@@ -1,63 +1,63 @@
 ---
-source: crates/ruff_db/src/diagnostic/render/gitlab.rs
-expression: env.render_diagnostics(&diagnostics)
+source: crates/ruff_linter/src/message/gitlab.rs
+expression: redact_fingerprint(&content)
 ---
 [
  {
    "check_name": "F401",
    "description": "F401: `os` imported but unused",
-    "severity": "major",
    "fingerprint": "<redacted>",
    "location": {
      "path": "fib.py",
      "positions": {
        "begin": {
-          "line": 1,
-          "column": 8
+          "column": 8,
+          "line": 1
        },
        "end": {
-          "line": 1,
-          "column": 10
+          "column": 10,
+          "line": 1
        }
      }
-    }
+    },
+    "severity": "major"
  },
  {
    "check_name": "F841",
    "description": "F841: Local variable `x` is assigned to but never used",
-    "severity": "major",
    "fingerprint": "<redacted>",
    "location": {
      "path": "fib.py",
      "positions": {
        "begin": {
-          "line": 6,
-          "column": 5
+          "column": 5,
+          "line": 6
        },
        "end": {
-          "line": 6,
-          "column": 6
+          "column": 6,
+          "line": 6
        }
      }
-    }
+    },
+    "severity": "major"
  },
  {
    "check_name": "F821",
    "description": "F821: Undefined name `a`",
-    "severity": "major",
    "fingerprint": "<redacted>",
    "location": {
      "path": "undef.py",
      "positions": {
        "begin": {
-          "line": 1,
-          "column": 4
+          "column": 4,
+          "line": 1
        },
        "end": {
-          "line": 1,
-          "column": 5
+          "column": 5,
+          "line": 1
        }
      }
-    }
+    },
+    "severity": "major"
  }
 ]
--- a/crates/ruff_db/src/diagnostic/render/snapshots/ruff_dbdiagnosticrendergitlabtests__syntax_errors.snap
+++ b/crates/ruff_db/src/diagnostic/render/snapshots/ruff_dbdiagnosticrendergitlabtests__syntax_errors.snap
@@ -1,44 +1,44 @@
 ---
-source: crates/ruff_db/src/diagnostic/render/gitlab.rs
-expression: env.render_diagnostics(&diagnostics)
+source: crates/ruff_linter/src/message/gitlab.rs
+expression: redact_fingerprint(&content)
 ---
 [
  {
    "check_name": "invalid-syntax",
    "description": "invalid-syntax: Expected one or more symbol names after import",
-    "severity": "major",
    "fingerprint": "<redacted>",
    "location": {
      "path": "syntax_errors.py",
      "positions": {
        "begin": {
-          "line": 1,
-          "column": 15
+          "column": 15,
+          "line": 1
        },
        "end": {
-          "line": 2,
-          "column": 1
+          "column": 1,
+          "line": 2
        }
      }
-    }
+    },
+    "severity": "major"
  },
  {
    "check_name": "invalid-syntax",
    "description": "invalid-syntax: Expected ')', found newline",
-    "severity": "major",
    "fingerprint": "<redacted>",
    "location": {
      "path": "syntax_errors.py",
      "positions": {
        "begin": {
-          "line": 3,
-          "column": 12
+          "column": 12,
+          "line": 3
        },
        "end": {
-          "line": 4,
-          "column": 1
+          "column": 1,
+          "line": 4
        }
      }
-    }
+    },
+    "severity": "major"
  }
 ]
--- a/crates/ruff_linter/src/preview.rs
+++ b/crates/ruff_linter/src/preview.rs
@@ -40,11 +40,6 @@ pub(crate) const fn is_bad_version_info_in_non_stub_enabled(settings: &LinterSet
    settings.preview.is_enabled()
 }

-/// <https://github.com/astral-sh/ruff/pull/19303>
-pub(crate) const fn is_fix_f_string_logging_enabled(settings: &LinterSettings) -> bool {
-    settings.preview.is_enabled()
-}
-
 // https://github.com/astral-sh/ruff/pull/16719
 pub(crate) const fn is_fix_manual_dict_comprehension_enabled(settings: &LinterSettings) -> bool {
    settings.preview.is_enabled()
--- a/crates/ruff_linter/src/renamer.rs
+++ b/crates/ruff_linter/src/renamer.rs
@@ -354,10 +354,7 @@ impl Renamer {
                ))
            }
            // Avoid renaming builtins and other "special" bindings.
-            BindingKind::FutureImport
-            | BindingKind::Builtin
-            | BindingKind::Export(_)
-            | BindingKind::DunderClassCell => None,
+            BindingKind::FutureImport | BindingKind::Builtin | BindingKind::Export(_) => None,
            // By default, replace the binding's name with the target name.
            BindingKind::Annotation
            | BindingKind::Argument
--- a/crates/ruff_linter/src/rules/airflow/rules/suggested_to_move_to_provider_in_3.rs
+++ b/crates/ruff_linter/src/rules/airflow/rules/suggested_to_move_to_provider_in_3.rs
@@ -215,12 +215,6 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
                version: "0.0.1",
            }
        }
-        ["airflow", "sensors", "bash", "BashSensor"] => ProviderReplacement::AutoImport {
-            module: "airflow.providers.standard.sensor.bash",
-            name: "BashSensor",
-            provider: "standard",
-            version: "0.0.1",
-        },
        [
            "airflow",
            "sensors",
@@ -249,12 +243,6 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
            provider: "standard",
            version: "0.0.2",
        },
-        ["airflow", "sensors", "python", "PythonSensor"] => ProviderReplacement::AutoImport {
-            module: "airflow.providers.standard.sensors.python",
-            name: "PythonSensor",
-            provider: "standard",
-            version: "0.0.1",
-        },
        [
            "airflow",
            "sensors",
--- a/crates/ruff_linter/src/rules/airflow/rules/suggested_to_update_3_0.rs
+++ b/crates/ruff_linter/src/rules/airflow/rules/suggested_to_update_3_0.rs
@@ -227,26 +227,13 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
            module: "airflow.sdk",
            name: (*rest).to_string(),
        },
-        [
-            "airflow",
-            "decorators",
-            "base",
-            rest @ ("DecoratedMappedOperator"
-            | "DecoratedOperator"
-            | "TaskDecorator"
-            | "get_unique_task_id"
-            | "task_decorator_factory"),
-        ] => Replacement::SourceModuleMoved {
-            module: "airflow.sdk.bases.decorator",
-            name: (*rest).to_string(),
-        },

        // airflow.io
        ["airflow", "io", "path", "ObjectStoragePath"] => Replacement::SourceModuleMoved {
            module: "airflow.sdk",
            name: "ObjectStoragePath".to_string(),
        },
-        ["airflow", "io", "store", "attach"] => Replacement::SourceModuleMoved {
+        ["airflow", "io", "storage", "attach"] => Replacement::SourceModuleMoved {
            module: "airflow.sdk.io",
            name: "attach".to_string(),
        },
@@ -258,10 +245,6 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
                name: (*rest).to_string(),
            }
        }
-        ["airflow", "models", "Param"] => Replacement::AutoImport {
-            module: "airflow.sdk.definitions.param",
-            name: "Param",
-        },

        // airflow.models.baseoperator
        [
@@ -277,30 +260,16 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
            module: "airflow.sdk",
            name: "BaseOperatorLink",
        },
-
        // airflow.model..DAG
        ["airflow", "models", .., "DAG"] => Replacement::SourceModuleMoved {
            module: "airflow.sdk",
            name: "DAG".to_string(),
        },
-
-        // airflow.sensors.base
-        [
-            "airflow",
-            "sensors",
-            "base",
-            rest @ ("BaseSensorOperator" | "PokeReturnValue" | "poke_mode_only"),
-        ] => Replacement::SourceModuleMoved {
-            module: "airflow.sdk",
-            name: (*rest).to_string(),
-        },
-
        // airflow.timetables
        ["airflow", "timetables", "datasets", "DatasetOrTimeSchedule"] => Replacement::AutoImport {
            module: "airflow.timetables.assets",
            name: "AssetOrTimeSchedule",
        },
-
        // airflow.utils
        [
            "airflow",
--- a/crates/ruff_linter/src/rules/airflow/snapshots/ruff_linterrulesairflowtestsAIR311_AIR311_names.py.snap
+++ b/crates/ruff_linter/src/rules/airflow/snapshots/ruff_linterrulesairflowtestsAIR311_AIR311_names.py.snap
@@ -312,7 +312,7 @@ help: Use `teardown` from `airflow.sdk` instead.
 34 34 | setup()
 35    |-from airflow.decorators import teardown
 36 35 | from airflow.io.path import ObjectStoragePath
-37 36 | from airflow.io.store import attach
+37 36 | from airflow.io.storage import attach
 38 37 | from airflow.models import DAG as DAGFromModel
 --------------------------------------------------------------------------------
 43 42 | from airflow.models.baseoperator import chain, chain_linear, cross_downstream
@@ -338,7 +338,7 @@ help: Use `ObjectStoragePath` from `airflow.sdk` instead.
 34 34 | setup()
 35 35 | from airflow.decorators import teardown
 36    |-from airflow.io.path import ObjectStoragePath
-37 36 | from airflow.io.store import attach
+37 36 | from airflow.io.storage import attach
 38 37 | from airflow.models import DAG as DAGFromModel
 39 38 | from airflow.models import (
 --------------------------------------------------------------------------------
@@ -350,7 +350,7 @@ help: Use `ObjectStoragePath` from `airflow.sdk` instead.
 47 47 | # airflow.decorators
 48 48 | teardown()

-AIR311 [*] `airflow.io.store.attach` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
+AIR311 [*] `airflow.io.storage.attach` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
  --> AIR311_names.py:52:1
   |
 50 | # # airflow.io
@@ -366,7 +366,7 @@ help: Use `attach` from `airflow.sdk.io` instead.
 34 34 | setup()
 35 35 | from airflow.decorators import teardown
 36 36 | from airflow.io.path import ObjectStoragePath
-37    |-from airflow.io.store import attach
+37    |-from airflow.io.storage import attach
 38 37 | from airflow.models import DAG as DAGFromModel
 39 38 | from airflow.models import (
 40 39 |     Connection,
@@ -391,7 +391,7 @@ AIR311 [*] `airflow.models.Connection` is removed in Airflow 3.0; It still works
 help: Use `Connection` from `airflow.sdk` instead.

 ℹ Unsafe fix
-37 37 | from airflow.io.store import attach
+37 37 | from airflow.io.storage import attach
 38 38 | from airflow.models import DAG as DAGFromModel
 39 39 | from airflow.models import (
 40    |-    Connection,
@@ -614,8 +614,6 @@ AIR311 [*] `airflow.utils.dag_parsing_context.get_parsing_context` is removed in
 75 | # airflow.utils.dag_parsing_context
 76 | get_parsing_context()
   | ^^^^^^^^^^^^^^^^^^^
-77 |
-78 | from airflow.decorators.base import (
   |
 help: Use `get_parsing_context` from `airflow.sdk` instead.

@@ -628,211 +626,3 @@ help: Use `get_parsing_context` from `airflow.sdk` instead.
 71 71 | 
 72 72 | # airflow.timetables.datasets
 73 73 | DatasetOrTimeSchedule()
-
-AIR311 [*] `airflow.decorators.base.DecoratedMappedOperator` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
-  --> AIR311_names.py:87:1
-   |
-86 | # airflow.decorators.base
-87 | DecoratedMappedOperator()
-   | ^^^^^^^^^^^^^^^^^^^^^^^
-88 | DecoratedOperator()
-89 | TaskDecorator()
-   |
-help: Use `DecoratedMappedOperator` from `airflow.sdk.bases.decorator` instead.
-
-ℹ Unsafe fix
-76 76 | get_parsing_context()
-77 77 | 
-78 78 | from airflow.decorators.base import (
-79    |-    DecoratedMappedOperator,
-80 79 |     DecoratedOperator,
-81 80 |     TaskDecorator,
-82 81 |     get_unique_task_id,
-83 82 |     task_decorator_factory,
-84 83 | )
-   84 |+from airflow.sdk.bases.decorator import DecoratedMappedOperator
-85 85 | 
-86 86 | # airflow.decorators.base
-87 87 | DecoratedMappedOperator()
-
-AIR311 [*] `airflow.decorators.base.DecoratedOperator` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
-  --> AIR311_names.py:88:1
-   |
-86 | # airflow.decorators.base
-87 | DecoratedMappedOperator()
-88 | DecoratedOperator()
-   | ^^^^^^^^^^^^^^^^^
-89 | TaskDecorator()
-90 | get_unique_task_id()
-   |
-help: Use `DecoratedOperator` from `airflow.sdk.bases.decorator` instead.
-
-ℹ Unsafe fix
-77 77 | 
-78 78 | from airflow.decorators.base import (
-79 79 |     DecoratedMappedOperator,
-80    |-    DecoratedOperator,
-81 80 |     TaskDecorator,
-82 81 |     get_unique_task_id,
-83 82 |     task_decorator_factory,
-84 83 | )
-   84 |+from airflow.sdk.bases.decorator import DecoratedOperator
-85 85 | 
-86 86 | # airflow.decorators.base
-87 87 | DecoratedMappedOperator()
-
-AIR311 [*] `airflow.decorators.base.TaskDecorator` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
-  --> AIR311_names.py:89:1
-   |
-87 | DecoratedMappedOperator()
-88 | DecoratedOperator()
-89 | TaskDecorator()
-   | ^^^^^^^^^^^^^
-90 | get_unique_task_id()
-91 | task_decorator_factory()
-   |
-help: Use `TaskDecorator` from `airflow.sdk.bases.decorator` instead.
-
-ℹ Unsafe fix
-78 78 | from airflow.decorators.base import (
-79 79 |     DecoratedMappedOperator,
-80 80 |     DecoratedOperator,
-81    |-    TaskDecorator,
-82 81 |     get_unique_task_id,
-83 82 |     task_decorator_factory,
-84 83 | )
-   84 |+from airflow.sdk.bases.decorator import TaskDecorator
-85 85 | 
-86 86 | # airflow.decorators.base
-87 87 | DecoratedMappedOperator()
-
-AIR311 [*] `airflow.decorators.base.get_unique_task_id` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
-  --> AIR311_names.py:90:1
-   |
-88 | DecoratedOperator()
-89 | TaskDecorator()
-90 | get_unique_task_id()
-   | ^^^^^^^^^^^^^^^^^^
-91 | task_decorator_factory()
-   |
-help: Use `get_unique_task_id` from `airflow.sdk.bases.decorator` instead.
-
-ℹ Unsafe fix
-79 79 |     DecoratedMappedOperator,
-80 80 |     DecoratedOperator,
-81 81 |     TaskDecorator,
-82    |-    get_unique_task_id,
-83 82 |     task_decorator_factory,
-84 83 | )
-   84 |+from airflow.sdk.bases.decorator import get_unique_task_id
-85 85 | 
-86 86 | # airflow.decorators.base
-87 87 | DecoratedMappedOperator()
-
-AIR311 [*] `airflow.decorators.base.task_decorator_factory` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
-  --> AIR311_names.py:91:1
-   |
-89 | TaskDecorator()
-90 | get_unique_task_id()
-91 | task_decorator_factory()
-   | ^^^^^^^^^^^^^^^^^^^^^^
-   |
-help: Use `task_decorator_factory` from `airflow.sdk.bases.decorator` instead.
-
-ℹ Unsafe fix
-80 80 |     DecoratedOperator,
-81 81 |     TaskDecorator,
-82 82 |     get_unique_task_id,
-83    |-    task_decorator_factory,
-84 83 | )
-   84 |+from airflow.sdk.bases.decorator import task_decorator_factory
-85 85 | 
-86 86 | # airflow.decorators.base
-87 87 | DecoratedMappedOperator()
-
-AIR311 [*] `airflow.models.Param` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
-  --> AIR311_names.py:97:1
-   |
-96 | # airflow.models
-97 | Param()
-   | ^^^^^
-   |
-help: Use `Param` from `airflow.sdk.definitions.param` instead.
-
-ℹ Unsafe fix
-91 91 | task_decorator_factory()
-92 92 | 
-93 93 | 
-94    |-from airflow.models import Param
-   94 |+from airflow.sdk.definitions.param import Param
-95 95 | 
-96 96 | # airflow.models
-97 97 | Param()
-
-AIR311 [*] `airflow.sensors.base.BaseSensorOperator` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
-   --> AIR311_names.py:107:1
-    |
-106 | # airflow.sensors.base
-107 | BaseSensorOperator()
-    | ^^^^^^^^^^^^^^^^^^
-108 | PokeReturnValue()
-109 | poke_mode_only()
-    |
-help: Use `BaseSensorOperator` from `airflow.sdk` instead.
-
-ℹ Unsafe fix
-98  98  | 
-99  99  | 
-100 100 | from airflow.sensors.base import (
-101     |-    BaseSensorOperator,
-102 101 |     PokeReturnValue,
-103 102 |     poke_mode_only,
-104 103 | )
-    104 |+from airflow.sdk import BaseSensorOperator
-105 105 | 
-106 106 | # airflow.sensors.base
-107 107 | BaseSensorOperator()
-
-AIR311 [*] `airflow.sensors.base.PokeReturnValue` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
-   --> AIR311_names.py:108:1
-    |
-106 | # airflow.sensors.base
-107 | BaseSensorOperator()
-108 | PokeReturnValue()
-    | ^^^^^^^^^^^^^^^
-109 | poke_mode_only()
-    |
-help: Use `PokeReturnValue` from `airflow.sdk` instead.
-
-ℹ Unsafe fix
-99  99  | 
-100 100 | from airflow.sensors.base import (
-101 101 |     BaseSensorOperator,
-102     |-    PokeReturnValue,
-103 102 |     poke_mode_only,
-104 103 | )
-    104 |+from airflow.sdk import PokeReturnValue
-105 105 | 
-106 106 | # airflow.sensors.base
-107 107 | BaseSensorOperator()
-
-AIR311 [*] `airflow.sensors.base.poke_mode_only` is removed in Airflow 3.0; It still works in Airflow 3.0 but is expected to be removed in a future version.
-   --> AIR311_names.py:109:1
-    |
-107 | BaseSensorOperator()
-108 | PokeReturnValue()
-109 | poke_mode_only()
-    | ^^^^^^^^^^^^^^
-    |
-help: Use `poke_mode_only` from `airflow.sdk` instead.
-
-ℹ Unsafe fix
-100 100 | from airflow.sensors.base import (
-101 101 |     BaseSensorOperator,
-102 102 |     PokeReturnValue,
-103     |-    poke_mode_only,
-104 103 | )
-    104 |+from airflow.sdk import poke_mode_only
-105 105 | 
-106 106 | # airflow.sensors.base
-107 107 | BaseSensorOperator()
--- a/crates/ruff_linter/src/rules/airflow/snapshots/ruff_linterrulesairflowtestsAIR312_AIR312.py.snap
+++ b/crates/ruff_linter/src/rules/airflow/snapshots/ruff_linterrulesairflowtestsAIR312_AIR312.py.snap
--- a/crates/ruff_linter/src/rules/flake8_async/mod.rs
+++ b/crates/ruff_linter/src/rules/flake8_async/mod.rs
@@ -23,7 +23,6 @@ mod tests {
    #[test_case(Rule::AsyncZeroSleep, Path::new("ASYNC115.py"))]
    #[test_case(Rule::LongSleepNotForever, Path::new("ASYNC116.py"))]
    #[test_case(Rule::BlockingHttpCallInAsyncFunction, Path::new("ASYNC210.py"))]
-    #[test_case(Rule::BlockingHttpCallHttpxInAsyncFunction, Path::new("ASYNC212.py"))]
    #[test_case(Rule::CreateSubprocessInAsyncFunction, Path::new("ASYNC22x.py"))]
    #[test_case(Rule::RunProcessInAsyncFunction, Path::new("ASYNC22x.py"))]
    #[test_case(Rule::WaitForProcessInAsyncFunction, Path::new("ASYNC22x.py"))]
--- a/crates/ruff_linter/src/rules/flake8_async/rules/blocking_http_call_httpx.rs
+++ b/crates/ruff_linter/src/rules/flake8_async/rules/blocking_http_call_httpx.rs
@@ -1,145 +0,0 @@
-use ruff_python_ast::{self as ast, Expr, ExprCall};
-
-use ruff_macros::{ViolationMetadata, derive_message_formats};
-use ruff_python_semantic::analyze::typing::{TypeChecker, check_type, traverse_union_and_optional};
-use ruff_text_size::Ranged;
-
-use crate::Violation;
-use crate::checkers::ast::Checker;
-
-/// ## What it does
-/// Checks that async functions do not use blocking httpx clients.
-///
-/// ## Why is this bad?
-/// Blocking an async function via a blocking HTTP call will block the entire
-/// event loop, preventing it from executing other tasks while waiting for the
-/// HTTP response, negating the benefits of asynchronous programming.
-///
-/// Instead of using the blocking `httpx` client, use the asynchronous client.
-///
-/// ## Example
-/// ```python
-/// import httpx
-///
-///
-/// async def fetch():
-///     client = httpx.Client()
-///     response = client.get(...)
-/// ```
-///
-/// Use instead:
-/// ```python
-/// import httpx
-///
-///
-/// async def fetch():
-///     async with httpx.AsyncClient() as client:
-///         response = await client.get(...)
-/// ```
-#[derive(ViolationMetadata)]
-pub(crate) struct BlockingHttpCallHttpxInAsyncFunction {
-    name: String,
-    call: String,
-}
-
-impl Violation for BlockingHttpCallHttpxInAsyncFunction {
-    #[derive_message_formats]
-    fn message(&self) -> String {
-        format!(
-            "Blocking httpx method {name}.{call}() in async context, use httpx.AsyncClient",
-            name = self.name,
-            call = self.call,
-        )
-    }
-}
-
-struct HttpxClientChecker;
-
-impl TypeChecker for HttpxClientChecker {
-    fn match_annotation(
-        annotation: &ruff_python_ast::Expr,
-        semantic: &ruff_python_semantic::SemanticModel,
-    ) -> bool {
-        // match base annotation directly
-        if semantic
-            .resolve_qualified_name(annotation)
-            .is_some_and(|qualified_name| matches!(qualified_name.segments(), ["httpx", "Client"]))
-        {
-            return true;
-        }
-
-        // otherwise traverse any union or optional annotation
-        let mut found = false;
-        traverse_union_and_optional(
-            &mut |inner_expr, _| {
-                if semantic
-                    .resolve_qualified_name(inner_expr)
-                    .is_some_and(|qualified_name| {
-                        matches!(qualified_name.segments(), ["httpx", "Client"])
-                    })
-                {
-                    found = true;
-                }
-            },
-            semantic,
-            annotation,
-        );
-        found
-    }
-
-    fn match_initializer(
-        initializer: &ruff_python_ast::Expr,
-        semantic: &ruff_python_semantic::SemanticModel,
-    ) -> bool {
-        let Expr::Call(ExprCall { func, .. }) = initializer else {
-            return false;
-        };
-
-        semantic
-            .resolve_qualified_name(func)
-            .is_some_and(|qualified_name| matches!(qualified_name.segments(), ["httpx", "Client"]))
-    }
-}
-
-/// ASYNC212
-pub(crate) fn blocking_http_call_httpx(checker: &Checker, call: &ExprCall) {
-    let semantic = checker.semantic();
-    if !semantic.in_async_context() {
-        return;
-    }
-
-    let Some(ast::ExprAttribute { value, attr, .. }) = call.func.as_attribute_expr() else {
-        return;
-    };
-    let Some(name) = value.as_name_expr() else {
-        return;
-    };
-    let Some(binding) = semantic.only_binding(name).map(|id| semantic.binding(id)) else {
-        return;
-    };
-
-    if check_type::<HttpxClientChecker>(binding, semantic) {
-        if matches!(
-            attr.id.as_str(),
-            "close"
-                | "delete"
-                | "get"
-                | "head"
-                | "options"
-                | "patch"
-                | "post"
-                | "put"
-                | "request"
-                | "send"
-                | "stream"
-        ) {
-            checker.report_diagnostic(
-                BlockingHttpCallHttpxInAsyncFunction {
-                    name: name.id.to_string(),
-                    call: attr.id.to_string(),
-                },
-                call.func.range(),
-            );
-        }
-    }
-}
--- a/crates/ruff_linter/src/rules/flake8_async/rules/mod.rs
+++ b/crates/ruff_linter/src/rules/flake8_async/rules/mod.rs
@@ -2,7 +2,6 @@ pub(crate) use async_busy_wait::*;
 pub(crate) use async_function_with_timeout::*;
 pub(crate) use async_zero_sleep::*;
 pub(crate) use blocking_http_call::*;
-pub(crate) use blocking_http_call_httpx::*;
 pub(crate) use blocking_open_call::*;
 pub(crate) use blocking_process_invocation::*;
 pub(crate) use blocking_sleep::*;
@@ -14,7 +13,6 @@ mod async_busy_wait;
 mod async_function_with_timeout;
 mod async_zero_sleep;
 mod blocking_http_call;
-mod blocking_http_call_httpx;
 mod blocking_open_call;
 mod blocking_process_invocation;
 mod blocking_sleep;
--- a/crates/ruff_linter/src/rules/flake8_async/snapshots/ruff_linterrulesflake8_asynctestsASYNC212_ASYNC212.py.snap
+++ b/crates/ruff_linter/src/rules/flake8_async/snapshots/ruff_linterrulesflake8_asynctestsASYNC212_ASYNC212.py.snap
@@ -1,168 +0,0 @@
---
-source: crates/ruff_linter/src/rules/flake8_async/mod.rs
---
-ASYNC212 Blocking httpx method client.close() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:27:5
-   |
-25 | async def foo():
-26 |     client = httpx.Client()
-27 |     client.close()  # ASYNC212
-   |     ^^^^^^^^^^^^
-28 |     client.delete()  # ASYNC212
-29 |     client.get()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.delete() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:28:5
-   |
-26 |     client = httpx.Client()
-27 |     client.close()  # ASYNC212
-28 |     client.delete()  # ASYNC212
-   |     ^^^^^^^^^^^^^
-29 |     client.get()  # ASYNC212
-30 |     client.head()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.get() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:29:5
-   |
-27 |     client.close()  # ASYNC212
-28 |     client.delete()  # ASYNC212
-29 |     client.get()  # ASYNC212
-   |     ^^^^^^^^^^
-30 |     client.head()  # ASYNC212
-31 |     client.options()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.head() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:30:5
-   |
-28 |     client.delete()  # ASYNC212
-29 |     client.get()  # ASYNC212
-30 |     client.head()  # ASYNC212
-   |     ^^^^^^^^^^^
-31 |     client.options()  # ASYNC212
-32 |     client.patch()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.options() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:31:5
-   |
-29 |     client.get()  # ASYNC212
-30 |     client.head()  # ASYNC212
-31 |     client.options()  # ASYNC212
-   |     ^^^^^^^^^^^^^^
-32 |     client.patch()  # ASYNC212
-33 |     client.post()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.patch() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:32:5
-   |
-30 |     client.head()  # ASYNC212
-31 |     client.options()  # ASYNC212
-32 |     client.patch()  # ASYNC212
-   |     ^^^^^^^^^^^^
-33 |     client.post()  # ASYNC212
-34 |     client.put()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.post() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:33:5
-   |
-31 |     client.options()  # ASYNC212
-32 |     client.patch()  # ASYNC212
-33 |     client.post()  # ASYNC212
-   |     ^^^^^^^^^^^
-34 |     client.put()  # ASYNC212
-35 |     client.request()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.put() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:34:5
-   |
-32 |     client.patch()  # ASYNC212
-33 |     client.post()  # ASYNC212
-34 |     client.put()  # ASYNC212
-   |     ^^^^^^^^^^
-35 |     client.request()  # ASYNC212
-36 |     client.send()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.request() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:35:5
-   |
-33 |     client.post()  # ASYNC212
-34 |     client.put()  # ASYNC212
-35 |     client.request()  # ASYNC212
-   |     ^^^^^^^^^^^^^^
-36 |     client.send()  # ASYNC212
-37 |     client.stream()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.send() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:36:5
-   |
-34 |     client.put()  # ASYNC212
-35 |     client.request()  # ASYNC212
-36 |     client.send()  # ASYNC212
-   |     ^^^^^^^^^^^
-37 |     client.stream()  # ASYNC212
-   |
-
-ASYNC212 Blocking httpx method client.stream() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:37:5
-   |
-35 |     client.request()  # ASYNC212
-36 |     client.send()  # ASYNC212
-37 |     client.stream()  # ASYNC212
-   |     ^^^^^^^^^^^^^
-38 |
-39 |     client.anything()  # Ok
-   |
-
-ASYNC212 Blocking httpx method client.request() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:45:5
-   |
-44 | async def foo(client: httpx.Client):
-45 |     client.request()  # ASYNC212
-   |     ^^^^^^^^^^^^^^
-46 |     client.anything()  # Ok
-   |
-
-ASYNC212 Blocking httpx method client.request() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:50:5
-   |
-49 | async def foo(client: httpx.Client | None):
-50 |     client.request()  # ASYNC212
-   |     ^^^^^^^^^^^^^^
-51 |     client.anything()  # Ok
-   |
-
-ASYNC212 Blocking httpx method client.request() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:55:5
-   |
-54 | async def foo(client: Optional[httpx.Client]):
-55 |     client.request()  # ASYNC212
-   |     ^^^^^^^^^^^^^^
-56 |     client.anything()  # Ok
-   |
-
-ASYNC212 Blocking httpx method client.request() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:61:5
-   |
-59 | async def foo():
-60 |     client: httpx.Client = ...
-61 |     client.request()  # ASYNC212
-   |     ^^^^^^^^^^^^^^
-62 |     client.anything()  # Ok
-   |
-
-ASYNC212 Blocking httpx method global_client.request() in async context, use httpx.AsyncClient
-  --> ASYNC212.py:69:5
-   |
-68 | async def foo():
-69 |     global_client.request()  # ASYNC212
-   |     ^^^^^^^^^^^^^^^^^^^^^
-70 |     global_client.anything()  # Ok
-   |
--- a/crates/ruff_linter/src/rules/flake8_logging_format/mod.rs
+++ b/crates/ruff_linter/src/rules/flake8_logging_format/mod.rs
@@ -22,7 +22,6 @@ mod tests {
    #[test_case(Path::new("G002.py"))]
    #[test_case(Path::new("G003.py"))]
    #[test_case(Path::new("G004.py"))]
-    #[test_case(Path::new("G004_arg_order.py"))]
    #[test_case(Path::new("G010.py"))]
    #[test_case(Path::new("G101_1.py"))]
    #[test_case(Path::new("G101_2.py"))]
@@ -49,24 +48,4 @@ mod tests {
        assert_diagnostics!(snapshot, diagnostics);
        Ok(())
    }
-
-    #[test_case(Rule::LoggingFString, Path::new("G004.py"))]
-    #[test_case(Rule::LoggingFString, Path::new("G004_arg_order.py"))]
-    fn preview_rules(rule_code: Rule, path: &Path) -> Result<()> {
-        let snapshot = format!(
-            "preview__{}_{}",
-            rule_code.noqa_code(),
-            path.to_string_lossy()
-        );
-        let diagnostics = test_path(
-            Path::new("flake8_logging_format").join(path).as_path(),
-            &settings::LinterSettings {
-                logger_objects: vec!["logging_setup.logger".to_string()],
-                preview: settings::types::PreviewMode::Enabled,
-                ..settings::LinterSettings::for_rule(rule_code)
-            },
-        )?;
-        assert_diagnostics!(snapshot, diagnostics);
-        Ok(())
-    }
 }
--- a/crates/ruff_linter/src/rules/flake8_logging_format/rules/logging_call.rs
+++ b/crates/ruff_linter/src/rules/flake8_logging_format/rules/logging_call.rs
@@ -1,11 +1,9 @@
-use ruff_python_ast::InterpolatedStringElement;
-use ruff_python_ast::{self as ast, Arguments, Expr, Keyword, Operator, StringFlags};
+use ruff_python_ast::{self as ast, Arguments, Expr, Keyword, Operator};
 use ruff_python_semantic::analyze::logging;
 use ruff_python_stdlib::logging::LoggingLevel;
 use ruff_text_size::Ranged;

 use crate::checkers::ast::Checker;
-use crate::preview::is_fix_f_string_logging_enabled;
 use crate::registry::Rule;
 use crate::rules::flake8_logging_format::violations::{
    LoggingExcInfo, LoggingExtraAttrClash, LoggingFString, LoggingPercentFormat,
@@ -13,87 +11,6 @@ use crate::rules::flake8_logging_format::violations::{
 };
 use crate::{Edit, Fix};

-fn logging_f_string(
-    checker: &Checker,
-    msg: &Expr,
-    f_string: &ast::ExprFString,
-    arguments: &Arguments,
-    msg_pos: usize,
-) {
-    // Report the diagnostic up-front so we can attach a fix later only when preview is enabled.
-    let mut diagnostic = checker.report_diagnostic(LoggingFString, msg.range());
-
-    // Preview gate for the automatic fix.
-    if !is_fix_f_string_logging_enabled(checker.settings()) {
-        return;
-    }
-
-    // If there are existing positional arguments after the message, bail out.
-    // This could indicate a mistake or complex usage we shouldn't try to fix.
-    if arguments.args.len() > msg_pos + 1 {
-        return;
-    }
-
-    let mut format_string = String::new();
-    let mut args: Vec<&str> = Vec::new();
-
-    // Try to reuse the first part's quote style when building the replacement.
-    // Default to double quotes if we can't determine it.
-    let quote_str = f_string
-        .value
-        .f_strings()
-        .next()
-        .map(|f| f.flags.quote_str())
-        .unwrap_or("\"");
-
-    for f in f_string.value.f_strings() {
-        for element in &f.elements {
-            match element {
-                InterpolatedStringElement::Literal(lit) => {
-                    // If the literal text contains a '%' placeholder, bail out: mixing
-                    // f-string interpolation with '%' placeholders is ambiguous for our
-                    // automatic conversion, so don't offer a fix for this case.
-                    if lit.value.as_ref().contains('%') {
-                        return;
-                    }
-                    format_string.push_str(lit.value.as_ref());
-                }
-                InterpolatedStringElement::Interpolation(interpolated) => {
-                    if interpolated.format_spec.is_some()
-                        || !matches!(
-                            interpolated.conversion,
-                            ruff_python_ast::ConversionFlag::None
-                        )
-                    {
-                        return;
-                    }
-                    match interpolated.expression.as_ref() {
-                        Expr::Name(name) => {
-                            format_string.push_str("%s");
-                            args.push(name.id.as_str());
-                        }
-                        _ => return,
-                    }
-                }
-            }
-        }
-    }
-
-    if args.is_empty() {
-        return;
-    }
-
-    let replacement = format!(
-        "{q}{format_string}{q}, {args}",
-        q = quote_str,
-        format_string = format_string,
-        args = args.join(", ")
-    );
-
-    let fix = Fix::safe_edit(Edit::range_replacement(replacement, msg.range()));
-    diagnostic.set_fix(fix);
-}
-
 /// Returns `true` if the attribute is a reserved attribute on the `logging` module's `LogRecord`
 /// class.
 fn is_reserved_attr(attr: &str) -> bool {
@@ -125,7 +42,7 @@ fn is_reserved_attr(attr: &str) -> bool {
 }

 /// Check logging messages for violations.
-fn check_msg(checker: &Checker, msg: &Expr, arguments: &Arguments, msg_pos: usize) {
+fn check_msg(checker: &Checker, msg: &Expr) {
    match msg {
        // Check for string concatenation and percent format.
        Expr::BinOp(ast::ExprBinOp { op, .. }) => match op {
@@ -138,10 +55,8 @@ fn check_msg(checker: &Checker, msg: &Expr, arguments: &Arguments, msg_pos: usiz
            _ => {}
        },
        // Check for f-strings.
-        Expr::FString(f_string) => {
-            if checker.is_rule_enabled(Rule::LoggingFString) {
-                logging_f_string(checker, msg, f_string, arguments, msg_pos);
-            }
+        Expr::FString(_) => {
+            checker.report_diagnostic_if_enabled(LoggingFString, msg.range());
        }
        // Check for .format() calls.
        Expr::Call(ast::ExprCall { func, .. }) => {
@@ -253,7 +168,7 @@ pub(crate) fn logging_call(checker: &Checker, call: &ast::ExprCall) {
    // G001, G002, G003, G004
    let msg_pos = usize::from(matches!(logging_call_type, LoggingCallType::LogCall));
    if let Some(format_arg) = call.arguments.find_argument_value("msg", msg_pos) {
-        check_msg(checker, format_arg, &call.arguments, msg_pos);
+        check_msg(checker, format_arg);
    }

    // G010
--- a/crates/ruff_linter/src/rules/flake8_logging_format/snapshots/ruff_linterrulesflake8_logging_formattestsG004.py.snap
+++ b/crates/ruff_linter/src/rules/flake8_logging_format/snapshots/ruff_linterrulesflake8_logging_formattestsG004.py.snap
@@ -9,7 +9,6 @@ G004 Logging statement uses f-string
  |              ^^^^^^^^^^^^^^^
 5 | logging.log(logging.INFO, f"Hello {name}")
  |
-help: Convert to lazy `%` formatting

 G004 Logging statement uses f-string
 --> G004.py:5:27
@@ -21,7 +20,6 @@ G004 Logging statement uses f-string
 6 |
 7 | _LOGGER = logging.getLogger()
  |
-help: Convert to lazy `%` formatting

 G004 Logging statement uses f-string
  --> G004.py:8:14
@@ -32,7 +30,6 @@ G004 Logging statement uses f-string
 9 |
 10 | logging.getLogger().info(f"{name}")
   |
-help: Convert to lazy `%` formatting

 G004 Logging statement uses f-string
  --> G004.py:10:26
@@ -44,7 +41,6 @@ G004 Logging statement uses f-string
 11 |
 12 | from logging import info
   |
-help: Convert to lazy `%` formatting

 G004 Logging statement uses f-string
  --> G004.py:14:6
@@ -55,7 +51,6 @@ G004 Logging statement uses f-string
   |      ^^^^^^^^^
 15 | info(f"{__name__}")
   |
-help: Convert to lazy `%` formatting

 G004 Logging statement uses f-string
  --> G004.py:15:6
@@ -66,156 +61,3 @@ G004 Logging statement uses f-string
 16 |
 17 | # Don't trigger for t-strings
   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:24:14
-   |
-22 | total = 9
-23 | directory_path = "/home/hamir/ruff/crates/ruff_linter/resources/test/"
-24 | logging.info(f"{count} out of {total} files in {directory_path} checked")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:30:13
-   |
-28 | x = 99
-29 | fmt = "08d"
-30 | logger.info(f"{x:{'08d'}}")
-   |             ^^^^^^^^^^^^^^
-31 | logger.info(f"{x:>10} {x:{fmt}}")
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:31:13
-   |
-29 | fmt = "08d"
-30 | logger.info(f"{x:{'08d'}}")
-31 | logger.info(f"{x:>10} {x:{fmt}}")
-   |             ^^^^^^^^^^^^^^^^^^^^
-32 |
-33 | logging.info(f"")
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:33:14
-   |
-31 | logger.info(f"{x:>10} {x:{fmt}}")
-32 |
-33 | logging.info(f"")
-   |              ^^^
-34 | logging.info(f"This message doesn't have any variables.")
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:34:14
-   |
-33 | logging.info(f"")
-34 | logging.info(f"This message doesn't have any variables.")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-35 |
-36 | obj = {"key": "value"}
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:37:14
-   |
-36 | obj = {"key": "value"}
-37 | logging.info(f"Object: {obj!r}")
-   |              ^^^^^^^^^^^^^^^^^^
-38 |
-39 | items_count = 3
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:40:17
-   |
-39 | items_count = 3
-40 | logging.warning(f"Items: {items_count:d}")
-   |                 ^^^^^^^^^^^^^^^^^^^^^^^^^
-41 |
-42 | data = {"status": "active"}
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:43:14
-   |
-42 | data = {"status": "active"}
-43 | logging.info(f"Processing {len(data)} items")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-44 | logging.info(f"Status: {data.get('status', 'unknown').upper()}")
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:44:14
-   |
-42 | data = {"status": "active"}
-43 | logging.info(f"Processing {len(data)} items")
-44 | logging.info(f"Status: {data.get('status', 'unknown').upper()}")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:48:14
-   |
-47 | result = 123
-48 | logging.info(f"Calculated result: {result + 100}")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-49 |
-50 | temperature = 123
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:51:14
-   |
-50 | temperature = 123
-51 | logging.info(f"Temperature: {temperature:.1f}°C")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-52 |
-53 | class FilePath:
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:57:14
-   |
-55 |         self.name = name
-56 |
-57 | logging.info(f"No changes made to {file_path.name}.")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-58 |
-59 | user = "tron"
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:61:15
-   |
-59 | user = "tron"
-60 | balance = 123.45
-61 | logging.error(f"Error {404}: User {user} has insufficient balance ${balance:.2f}")
-   |               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-62 |
-63 | import logging
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:66:15
-   |
-65 | x = 1
-66 | logging.error(f"{x} -> %s", x)
-   |               ^^^^^^^^^^^^
-   |
-help: Convert to lazy `%` formatting
--- a/crates/ruff_linter/src/rules/flake8_logging_format/snapshots/ruff_linterrulesflake8_logging_formattestsG004_arg_order.py.snap
+++ b/crates/ruff_linter/src/rules/flake8_logging_format/snapshots/ruff_linterrulesflake8_logging_formattestsG004_arg_order.py.snap
@@ -1,23 +0,0 @@
---
-source: crates/ruff_linter/src/rules/flake8_logging_format/mod.rs
---
-G004 Logging statement uses f-string
-  --> G004_arg_order.py:9:14
-   |
- 7 | X = 1
- 8 | Y = 2
- 9 | logger.error(f"{X} -> %s", Y)
-   |              ^^^^^^^^^^^^
-10 | logger.error(f"{Y} -> %s", X)
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004_arg_order.py:10:14
-   |
- 8 | Y = 2
- 9 | logger.error(f"{X} -> %s", Y)
-10 | logger.error(f"{Y} -> %s", X)
-   |              ^^^^^^^^^^^^
-   |
-help: Convert to lazy `%` formatting
--- a/crates/ruff_linter/src/rules/flake8_logging_format/snapshots/ruff_linterrulesflake8_logging_formattestspreview__G004_G004.py.snap
+++ b/crates/ruff_linter/src/rules/flake8_logging_format/snapshots/ruff_linterrulesflake8_logging_formattestspreview__G004_G004.py.snap
@@ -1,291 +0,0 @@
---
-source: crates/ruff_linter/src/rules/flake8_logging_format/mod.rs
---
-G004 [*] Logging statement uses f-string
- --> G004.py:4:14
-  |
-3 | name = "world"
-4 | logging.info(f"Hello {name}")
-  |              ^^^^^^^^^^^^^^^
-5 | logging.log(logging.INFO, f"Hello {name}")
-  |
-help: Convert to lazy `%` formatting
-
-ℹ Safe fix
-1 1 | import logging
-2 2 | 
-3 3 | name = "world"
-4   |-logging.info(f"Hello {name}")
-  4 |+logging.info("Hello %s", name)
-5 5 | logging.log(logging.INFO, f"Hello {name}")
-6 6 | 
-7 7 | _LOGGER = logging.getLogger()
-
-G004 [*] Logging statement uses f-string
- --> G004.py:5:27
-  |
-3 | name = "world"
-4 | logging.info(f"Hello {name}")
-5 | logging.log(logging.INFO, f"Hello {name}")
-  |                           ^^^^^^^^^^^^^^^
-6 |
-7 | _LOGGER = logging.getLogger()
-  |
-help: Convert to lazy `%` formatting
-
-ℹ Safe fix
-2 2 | 
-3 3 | name = "world"
-4 4 | logging.info(f"Hello {name}")
-5   |-logging.log(logging.INFO, f"Hello {name}")
-  5 |+logging.log(logging.INFO, "Hello %s", name)
-6 6 | 
-7 7 | _LOGGER = logging.getLogger()
-8 8 | _LOGGER.info(f"{__name__}")
-
-G004 [*] Logging statement uses f-string
-  --> G004.py:8:14
-   |
- 7 | _LOGGER = logging.getLogger()
- 8 | _LOGGER.info(f"{__name__}")
-   |              ^^^^^^^^^^^^^
- 9 |
-10 | logging.getLogger().info(f"{name}")
-   |
-help: Convert to lazy `%` formatting
-
-ℹ Safe fix
-5 5 | logging.log(logging.INFO, f"Hello {name}")
-6 6 | 
-7 7 | _LOGGER = logging.getLogger()
-8   |-_LOGGER.info(f"{__name__}")
-  8 |+_LOGGER.info("%s", __name__)
-9 9 | 
-10 10 | logging.getLogger().info(f"{name}")
-11 11 | 
-
-G004 [*] Logging statement uses f-string
-  --> G004.py:10:26
-   |
- 8 | _LOGGER.info(f"{__name__}")
- 9 |
-10 | logging.getLogger().info(f"{name}")
-   |                          ^^^^^^^^^
-11 |
-12 | from logging import info
-   |
-help: Convert to lazy `%` formatting
-
-ℹ Safe fix
-7  7  | _LOGGER = logging.getLogger()
-8  8  | _LOGGER.info(f"{__name__}")
-9  9  | 
-10    |-logging.getLogger().info(f"{name}")
-   10 |+logging.getLogger().info("%s", name)
-11 11 | 
-12 12 | from logging import info
-13 13 | 
-
-G004 [*] Logging statement uses f-string
-  --> G004.py:14:6
-   |
-12 | from logging import info
-13 |
-14 | info(f"{name}")
-   |      ^^^^^^^^^
-15 | info(f"{__name__}")
-   |
-help: Convert to lazy `%` formatting
-
-ℹ Safe fix
-11 11 | 
-12 12 | from logging import info
-13 13 | 
-14    |-info(f"{name}")
-   14 |+info("%s", name)
-15 15 | info(f"{__name__}")
-16 16 | 
-17 17 | # Don't trigger for t-strings
-
-G004 [*] Logging statement uses f-string
-  --> G004.py:15:6
-   |
-14 | info(f"{name}")
-15 | info(f"{__name__}")
-   |      ^^^^^^^^^^^^^
-16 |
-17 | # Don't trigger for t-strings
-   |
-help: Convert to lazy `%` formatting
-
-ℹ Safe fix
-12 12 | from logging import info
-13 13 | 
-14 14 | info(f"{name}")
-15    |-info(f"{__name__}")
-   15 |+info("%s", __name__)
-16 16 | 
-17 17 | # Don't trigger for t-strings
-18 18 | info(t"{name}")
-
-G004 [*] Logging statement uses f-string
-  --> G004.py:24:14
-   |
-22 | total = 9
-23 | directory_path = "/home/hamir/ruff/crates/ruff_linter/resources/test/"
-24 | logging.info(f"{count} out of {total} files in {directory_path} checked")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-   |
-help: Convert to lazy `%` formatting
-
-ℹ Safe fix
-21 21 | count = 5
-22 22 | total = 9
-23 23 | directory_path = "/home/hamir/ruff/crates/ruff_linter/resources/test/"
-24    |-logging.info(f"{count} out of {total} files in {directory_path} checked")
-   24 |+logging.info("%s out of %s files in %s checked", count, total, directory_path)
-25 25 | 
-26 26 | 
-27 27 | 
-
-G004 Logging statement uses f-string
-  --> G004.py:30:13
-   |
-28 | x = 99
-29 | fmt = "08d"
-30 | logger.info(f"{x:{'08d'}}")
-   |             ^^^^^^^^^^^^^^
-31 | logger.info(f"{x:>10} {x:{fmt}}")
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:31:13
-   |
-29 | fmt = "08d"
-30 | logger.info(f"{x:{'08d'}}")
-31 | logger.info(f"{x:>10} {x:{fmt}}")
-   |             ^^^^^^^^^^^^^^^^^^^^
-32 |
-33 | logging.info(f"")
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:33:14
-   |
-31 | logger.info(f"{x:>10} {x:{fmt}}")
-32 |
-33 | logging.info(f"")
-   |              ^^^
-34 | logging.info(f"This message doesn't have any variables.")
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:34:14
-   |
-33 | logging.info(f"")
-34 | logging.info(f"This message doesn't have any variables.")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-35 |
-36 | obj = {"key": "value"}
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:37:14
-   |
-36 | obj = {"key": "value"}
-37 | logging.info(f"Object: {obj!r}")
-   |              ^^^^^^^^^^^^^^^^^^
-38 |
-39 | items_count = 3
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:40:17
-   |
-39 | items_count = 3
-40 | logging.warning(f"Items: {items_count:d}")
-   |                 ^^^^^^^^^^^^^^^^^^^^^^^^^
-41 |
-42 | data = {"status": "active"}
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:43:14
-   |
-42 | data = {"status": "active"}
-43 | logging.info(f"Processing {len(data)} items")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-44 | logging.info(f"Status: {data.get('status', 'unknown').upper()}")
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:44:14
-   |
-42 | data = {"status": "active"}
-43 | logging.info(f"Processing {len(data)} items")
-44 | logging.info(f"Status: {data.get('status', 'unknown').upper()}")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:48:14
-   |
-47 | result = 123
-48 | logging.info(f"Calculated result: {result + 100}")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-49 |
-50 | temperature = 123
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:51:14
-   |
-50 | temperature = 123
-51 | logging.info(f"Temperature: {temperature:.1f}°C")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-52 |
-53 | class FilePath:
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:57:14
-   |
-55 |         self.name = name
-56 |
-57 | logging.info(f"No changes made to {file_path.name}.")
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-58 |
-59 | user = "tron"
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:61:15
-   |
-59 | user = "tron"
-60 | balance = 123.45
-61 | logging.error(f"Error {404}: User {user} has insufficient balance ${balance:.2f}")
-   |               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-62 |
-63 | import logging
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004.py:66:15
-   |
-65 | x = 1
-66 | logging.error(f"{x} -> %s", x)
-   |               ^^^^^^^^^^^^
-   |
-help: Convert to lazy `%` formatting
--- a/crates/ruff_linter/src/rules/flake8_logging_format/snapshots/ruff_linterrulesflake8_logging_formattestspreview__G004_G004_arg_order.py.snap
+++ b/crates/ruff_linter/src/rules/flake8_logging_format/snapshots/ruff_linterrulesflake8_logging_formattestspreview__G004_G004_arg_order.py.snap
@@ -1,23 +0,0 @@
---
-source: crates/ruff_linter/src/rules/flake8_logging_format/mod.rs
---
-G004 Logging statement uses f-string
-  --> G004_arg_order.py:9:14
-   |
- 7 | X = 1
- 8 | Y = 2
- 9 | logger.error(f"{X} -> %s", Y)
-   |              ^^^^^^^^^^^^
-10 | logger.error(f"{Y} -> %s", X)
-   |
-help: Convert to lazy `%` formatting
-
-G004 Logging statement uses f-string
-  --> G004_arg_order.py:10:14
-   |
- 8 | Y = 2
- 9 | logger.error(f"{X} -> %s", Y)
-10 | logger.error(f"{Y} -> %s", X)
-   |              ^^^^^^^^^^^^
-   |
-help: Convert to lazy `%` formatting
--- a/crates/ruff_linter/src/rules/flake8_logging_format/violations.rs
+++ b/crates/ruff_linter/src/rules/flake8_logging_format/violations.rs
@@ -327,16 +327,10 @@ impl Violation for LoggingStringConcat {
 pub(crate) struct LoggingFString;

 impl Violation for LoggingFString {
-    const FIX_AVAILABILITY: crate::FixAvailability = crate::FixAvailability::Sometimes;
-
    #[derive_message_formats]
    fn message(&self) -> String {
        "Logging statement uses f-string".to_string()
    }
-
-    fn fix_title(&self) -> Option<String> {
-        Some("Convert to lazy `%` formatting".to_string())
-    }
 }

 /// ## What it does
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_chmod.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_chmod.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.chmod`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.chmod)
 /// - [Python documentation: `os.chmod`](https://docs.python.org/3/library/os.html#os.chmod)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_getcwd.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_getcwd.rs
@@ -43,7 +43,7 @@ use ruff_text_size::Ranged;
 /// - [Python documentation: `os.getcwd`](https://docs.python.org/3/library/os.html#os.getcwd)
 /// - [Python documentation: `os.getcwdb`](https://docs.python.org/3/library/os.html#os.getcwdb)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_makedirs.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_makedirs.rs
@@ -46,7 +46,7 @@ use crate::{FixAvailability, Violation};
 /// - [Python documentation: `Path.mkdir`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.mkdir)
 /// - [Python documentation: `os.makedirs`](https://docs.python.org/3/library/os.html#os.makedirs)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_mkdir.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_mkdir.rs
@@ -47,7 +47,7 @@ use crate::{FixAvailability, Violation};
 /// - [Python documentation: `Path.mkdir`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.mkdir)
 /// - [Python documentation: `os.mkdir`](https://docs.python.org/3/library/os.html#os.mkdir)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_abspath.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_abspath.rs
@@ -1,15 +1,9 @@
-use ruff_diagnostics::{Edit, Fix};
+use crate::checkers::ast::Checker;
+use crate::preview::is_fix_os_path_abspath_enabled;
+use crate::rules::flake8_use_pathlib::helpers::check_os_pathlib_single_arg_calls;
+use crate::{FixAvailability, Violation};
 use ruff_macros::{ViolationMetadata, derive_message_formats};
 use ruff_python_ast::ExprCall;
-use ruff_text_size::Ranged;
-
-use crate::checkers::ast::Checker;
-use crate::importer::ImportRequest;
-use crate::preview::is_fix_os_path_abspath_enabled;
-use crate::rules::flake8_use_pathlib::helpers::{
-    has_unknown_keywords_or_starred_expr, is_pathlib_path_call,
-};
-use crate::{FixAvailability, Violation};

 /// ## What it does
 /// Checks for uses of `os.path.abspath`.
@@ -40,18 +34,13 @@ use crate::{FixAvailability, Violation};
 /// especially on older versions of Python.
 ///
 /// ## Fix Safety
-/// This rule's fix is always marked as unsafe because `Path.resolve()` resolves symlinks, while
-/// `os.path.abspath()` does not. If resolving symlinks is important, you may need to use
-/// `Path.absolute()`. However, `Path.absolute()` also does not remove any `..` components in a
-/// path, unlike `os.path.abspath()` and `Path.resolve()`, so if that specific combination of
-/// behaviors is required, there's no existing `pathlib` alternative. See CPython issue
-/// [#69200](https://github.com/python/cpython/issues/69200).
+/// This rule's fix is marked as unsafe if the replacement would remove comments attached to the original expression.
 ///
 /// ## References
 /// - [Python documentation: `Path.resolve`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.resolve)
 /// - [Python documentation: `os.path.abspath`](https://docs.python.org/3/library/os.path.html#os.path.abspath)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
@@ -74,44 +63,12 @@ pub(crate) fn os_path_abspath(checker: &Checker, call: &ExprCall, segments: &[&s
    if segments != ["os", "path", "abspath"] {
        return;
    }
-
-    if call.arguments.len() != 1 {
-        return;
-    }
-
-    let Some(arg) = call.arguments.find_argument_value("path", 0) else {
-        return;
-    };
-
-    let arg_code = checker.locator().slice(arg.range());
-    let range = call.range();
-
-    let mut diagnostic = checker.report_diagnostic(OsPathAbspath, call.func.range());
-
-    if has_unknown_keywords_or_starred_expr(&call.arguments, &["path"]) {
-        return;
-    }
-
-    if !is_fix_os_path_abspath_enabled(checker.settings()) {
-        return;
-    }
-
-    diagnostic.try_set_fix(|| {
-        let (import_edit, binding) = checker.importer().get_or_import_symbol(
-            &ImportRequest::import("pathlib", "Path"),
-            call.start(),
-            checker.semantic(),
-        )?;
-
-        let replacement = if is_pathlib_path_call(checker, arg) {
-            format!("{arg_code}.resolve()")
-        } else {
-            format!("{binding}({arg_code}).resolve()")
-        };
-
-        Ok(Fix::unsafe_edits(
-            Edit::range_replacement(replacement, range),
-            [import_edit],
-        ))
-    });
+    check_os_pathlib_single_arg_calls(
+        checker,
+        call,
+        "resolve()",
+        "path",
+        is_fix_os_path_abspath_enabled(checker.settings()),
+        OsPathAbspath,
+    );
 }
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_basename.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_basename.rs
@@ -40,7 +40,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `PurePath.name`](https://docs.python.org/3/library/pathlib.html#pathlib.PurePath.name)
 /// - [Python documentation: `os.path.basename`](https://docs.python.org/3/library/os.path.html#os.path.basename)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_dirname.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_dirname.rs
@@ -40,7 +40,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `PurePath.parent`](https://docs.python.org/3/library/pathlib.html#pathlib.PurePath.parent)
 /// - [Python documentation: `os.path.dirname`](https://docs.python.org/3/library/os.path.html#os.path.dirname)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_exists.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_exists.rs
@@ -40,7 +40,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.exists`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.exists)
 /// - [Python documentation: `os.path.exists`](https://docs.python.org/3/library/os.path.html#os.path.exists)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_expanduser.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_expanduser.rs
@@ -40,7 +40,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.expanduser`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.expanduser)
 /// - [Python documentation: `os.path.expanduser`](https://docs.python.org/3/library/os.path.html#os.path.expanduser)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_getatime.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_getatime.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.stat`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.stat)
 /// - [Python documentation: `os.path.getatime`](https://docs.python.org/3/library/os.path.html#os.path.getatime)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_getctime.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_getctime.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.stat`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.stat)
 /// - [Python documentation: `os.path.getctime`](https://docs.python.org/3/library/os.path.html#os.path.getctime)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_getmtime.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_getmtime.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.stat`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.stat)
 /// - [Python documentation: `os.path.getmtime`](https://docs.python.org/3/library/os.path.html#os.path.getmtime)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_getsize.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_getsize.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.stat`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.stat)
 /// - [Python documentation: `os.path.getsize`](https://docs.python.org/3/library/os.path.html#os.path.getsize)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_isabs.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_isabs.rs
@@ -39,7 +39,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `PurePath.is_absolute`](https://docs.python.org/3/library/pathlib.html#pathlib.PurePath.is_absolute)
 /// - [Python documentation: `os.path.isabs`](https://docs.python.org/3/library/os.path.html#os.path.isabs)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_isdir.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_isdir.rs
@@ -40,7 +40,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.is_dir`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.is_dir)
 /// - [Python documentation: `os.path.isdir`](https://docs.python.org/3/library/os.path.html#os.path.isdir)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_isfile.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_isfile.rs
@@ -40,7 +40,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.is_file`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.is_file)
 /// - [Python documentation: `os.path.isfile`](https://docs.python.org/3/library/os.path.html#os.path.isfile)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_islink.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_islink.rs
@@ -40,7 +40,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.is_symlink`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.is_symlink)
 /// - [Python documentation: `os.path.islink`](https://docs.python.org/3/library/os.path.html#os.path.islink)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_samefile.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_path_samefile.rs
@@ -40,7 +40,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.samefile`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.samefile)
 /// - [Python documentation: `os.path.samefile`](https://docs.python.org/3/library/os.path.html#os.path.samefile)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_readlink.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_readlink.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::{ExprCall, PythonVersion};
 /// - [Python documentation: `Path.readlink`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.readline)
 /// - [Python documentation: `os.readlink`](https://docs.python.org/3/library/os.html#os.readlink)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_remove.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_remove.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.unlink`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.unlink)
 /// - [Python documentation: `os.remove`](https://docs.python.org/3/library/os.html#os.remove)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_rename.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_rename.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.rename`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.rename)
 /// - [Python documentation: `os.rename`](https://docs.python.org/3/library/os.html#os.rename)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_replace.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_replace.rs
@@ -45,7 +45,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.replace`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.replace)
 /// - [Python documentation: `os.replace`](https://docs.python.org/3/library/os.html#os.replace)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_rmdir.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_rmdir.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.rmdir`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.rmdir)
 /// - [Python documentation: `os.rmdir`](https://docs.python.org/3/library/os.html#os.rmdir)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_symlink.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_symlink.rs
@@ -44,7 +44,7 @@ use crate::{FixAvailability, Violation};
 /// ## References
 /// - [Python documentation: `Path.symlink_to`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.symlink_to)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_unlink.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/rules/os_unlink.rs
@@ -42,7 +42,7 @@ use ruff_python_ast::ExprCall;
 /// - [Python documentation: `Path.unlink`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.unlink)
 /// - [Python documentation: `os.unlink`](https://docs.python.org/3/library/os.html#os.unlink)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/snapshots/ruff_linterrulesflake8_use_pathlibtestspreview_full_name.py.snap
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/snapshots/ruff_linterrulesflake8_use_pathlibtestspreview_full_name.py.snap
@@ -13,7 +13,7 @@ PTH100 [*] `os.path.abspath()` should be replaced by `Path.resolve()`
  |
 help: Replace with `Path(...).resolve()`

-ℹ Unsafe fix
+ℹ Safe fix
 1 1 | import os
 2 2 | import os.path
  3 |+import pathlib
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/snapshots/ruff_linterrulesflake8_use_pathlibtestspreview_import_as.py.snap
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/snapshots/ruff_linterrulesflake8_use_pathlibtestspreview_import_as.py.snap
@@ -13,7 +13,7 @@ PTH100 [*] `os.path.abspath()` should be replaced by `Path.resolve()`
  |
 help: Replace with `Path(...).resolve()`

-ℹ Unsafe fix
+ℹ Safe fix
 1 1 | import os as foo
 2 2 | import os.path as foo_p
  3 |+import pathlib
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/snapshots/ruff_linterrulesflake8_use_pathlibtestspreview_import_from.py.snap
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/snapshots/ruff_linterrulesflake8_use_pathlibtestspreview_import_from.py.snap
@@ -13,7 +13,7 @@ PTH100 [*] `os.path.abspath()` should be replaced by `Path.resolve()`
   |
 help: Replace with `Path(...).resolve()`

-ℹ Unsafe fix
+ℹ Safe fix
 2  2  | from os import remove, unlink, getcwd, readlink, stat
 3  3  | from os.path import abspath, exists, expanduser, isdir, isfile, islink
 4  4  | from os.path import isabs, join, basename, dirname, samefile, splitext
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/snapshots/ruff_linterrulesflake8_use_pathlibtestspreview_import_from_as.py.snap
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/snapshots/ruff_linterrulesflake8_use_pathlibtestspreview_import_from_as.py.snap
@@ -13,7 +13,7 @@ PTH100 [*] `os.path.abspath()` should be replaced by `Path.resolve()`
   |
 help: Replace with `Path(...).resolve()`

-ℹ Unsafe fix
+ℹ Safe fix
 7  7  | from os.path import isfile as xisfile, islink as xislink, isabs as xisabs
 8  8  | from os.path import join as xjoin, basename as xbasename, dirname as xdirname
 9  9  | from os.path import samefile as xsamefile, splitext as xsplitext
--- a/crates/ruff_linter/src/rules/flake8_use_pathlib/violations.rs
+++ b/crates/ruff_linter/src/rules/flake8_use_pathlib/violations.rs
@@ -43,7 +43,7 @@ use crate::Violation;
 /// - [Python documentation: `Path.owner`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.owner)
 /// - [Python documentation: `os.stat`](https://docs.python.org/3/library/os.html#os.stat)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
@@ -89,7 +89,7 @@ impl Violation for OsStat {
 /// - [Python documentation: `PurePath.joinpath`](https://docs.python.org/3/library/pathlib.html#pathlib.PurePath.joinpath)
 /// - [Python documentation: `os.path.join`](https://docs.python.org/3/library/os.path.html#os.path.join)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
@@ -160,7 +160,7 @@ pub(crate) enum Joiner {
 /// - [Python documentation: `Path.suffixes`](https://docs.python.org/3/library/pathlib.html#pathlib.PurePath.suffixes)
 /// - [Python documentation: `os.path.splitext`](https://docs.python.org/3/library/os.path.html#os.path.splitext)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
@@ -205,7 +205,7 @@ impl Violation for OsPathSplitext {
 /// - [Python documentation: `Path.open`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.open)
 /// - [Python documentation: `open`](https://docs.python.org/3/library/functions.html#open)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
@@ -298,7 +298,7 @@ impl Violation for PyPath {
 /// - [Python documentation: `Path.iterdir`](https://docs.python.org/3/library/pathlib.html#pathlib.Path.iterdir)
 /// - [Python documentation: `os.listdir`](https://docs.python.org/3/library/os.html#os.listdir)
 /// - [PEP 428 – The pathlib module – object-oriented filesystem paths](https://peps.python.org/pep-0428/)
-/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#corresponding-tools)
+/// - [Correspondence between `os` and `pathlib`](https://docs.python.org/3/library/pathlib.html#correspondence-to-tools-in-the-os-module)
 /// - [Why you should be using pathlib](https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/)
 /// - [No really, pathlib is great](https://treyhunner.com/2019/01/no-really-pathlib-is-great/)
 #[derive(ViolationMetadata)]
--- a/crates/ruff_linter/src/rules/pyflakes/mod.rs
+++ b/crates/ruff_linter/src/rules/pyflakes/mod.rs
@@ -376,22 +376,6 @@ mod tests {
        Ok(())
    }

-    #[test_case(Rule::UnusedImport, Path::new("F401_35.py"))]
-    fn f401_allowed_unused_imports_top_level_module(rule_code: Rule, path: &Path) -> Result<()> {
-        let diagnostics = test_path(
-            Path::new("pyflakes").join(path).as_path(),
-            &LinterSettings {
-                pyflakes: pyflakes::settings::Settings {
-                    allowed_unused_imports: vec!["hvplot".to_string()],
-                    ..pyflakes::settings::Settings::default()
-                },
-                ..LinterSettings::for_rule(rule_code)
-            },
-        )?;
-        assert_diagnostics!(diagnostics);
-        Ok(())
-    }
-
    #[test]
    fn f841_dummy_variable_rgx() -> Result<()> {
        let diagnostics = test_path(
@@ -753,7 +737,6 @@ mod tests {

    /// A re-implementation of the Pyflakes test runner.
    /// Note that all tests marked with `#[ignore]` should be considered TODOs.
-    #[track_caller]
    fn flakes(contents: &str, expected: &[Rule]) {
        let contents = dedent(contents);
        let source_type = PySourceType::default();
--- a/Show More
+++ b/Show More
Author	SHA1	Message	Date
Ibraheem Ahmed	992e77e4d0	suggestions from code review	2025-08-22 19:36:40 -04:00
Ibraheem Ahmed	8d05367d60	reload `Project` after deserialization	2025-08-22 19:09:22 -04:00
Ibraheem Ahmed	2402831223	experimental persistent caching	2025-08-22 19:09:21 -04:00