debug

.github/workflows/ci.yaml

@@ -452,7 +452,7 @@ jobs:
       - name: "Install Rust toolchain"
         run: rustup show
       - name: "Install cargo-binstall"
-        uses: cargo-bins/cargo-binstall@38e8f5e4c386b611d51e8aa997b9a06a3c8eb67a # v1.15.6
+        uses: cargo-bins/cargo-binstall@20aa316bab4942180bbbabe93237858e8d77f1ed # v1.15.5
       - name: "Install cargo-fuzz"
         # Download the latest version from quick install and not the github releases because github releases only has MUSL targets.
         run: cargo binstall cargo-fuzz --force  --disable-strategies crate-meta-data --no-confirm
@@ -703,7 +703,7 @@ jobs:
       - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
         with:
           persist-credentials: false
-      - uses: cargo-bins/cargo-binstall@38e8f5e4c386b611d51e8aa997b9a06a3c8eb67a # v1.15.6
+      - uses: cargo-bins/cargo-binstall@20aa316bab4942180bbbabe93237858e8d77f1ed # v1.15.5
       - run: cargo binstall --no-confirm cargo-shear
       - run: cargo shear
 
@@ -932,7 +932,7 @@ jobs:
           tool: cargo-codspeed
 
       - name: "Build benchmarks"
-        run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark --bench formatter --bench lexer --bench linter --bench parser
+        run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark formatter lexer linter parser
 
       - name: "Run benchmarks"
         uses: CodSpeedHQ/action@653fdc30e6c40ffd9739e40c8a0576f4f4523ca1 # v4.0.1
@@ -967,7 +967,7 @@ jobs:
           tool: cargo-codspeed
 
       - name: "Build benchmarks"
-        run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark --bench ty
+        run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark ty
 
       - name: "Run benchmarks"
         uses: CodSpeedHQ/action@653fdc30e6c40ffd9739e40c8a0576f4f4523ca1 # v4.0.1

.github/workflows/ty-ecosystem-analyzer.yaml

@@ -64,7 +64,7 @@ jobs:
 
           cd ..
 
-          uv tool install "git+https://github.com/astral-sh/ecosystem-analyzer@279f8a15b0e7f77213bf9096dbc2335a19ef89c5"
+          uv tool install "git+https://github.com/astral-sh/ecosystem-analyzer@fc0f612798710b0dd69bb7528bc9b361dc60bd43"
 
           ecosystem-analyzer \
             --repository ruff \

.github/workflows/ty-ecosystem-report.yaml

@@ -49,13 +49,12 @@ jobs:
 
           cd ..
 
-          uv tool install "git+https://github.com/astral-sh/ecosystem-analyzer@279f8a15b0e7f77213bf9096dbc2335a19ef89c5"
+          uv tool install "git+https://github.com/astral-sh/ecosystem-analyzer@fc0f612798710b0dd69bb7528bc9b361dc60bd43"
 
           ecosystem-analyzer \
             --verbose \
             --repository ruff \
             analyze \
-            --profile=release \
             --projects ruff/crates/ty_python_semantic/resources/primer/good.txt \
             --output ecosystem-diagnostics.json
 
@@ -63,7 +62,7 @@ jobs:
 
           ecosystem-analyzer \
             generate-report \
-            --max-diagnostics-per-project=1000 \
+            --max-diagnostics-per-project=1200 \
             ecosystem-diagnostics.json \
             --output dist/index.html
 

CHANGELOG.md

@@ -1,48 +1,5 @@
 # Changelog
 
-## 0.13.3
-
-Released on 2025-10-02.
-
-### Preview features
-
-- Display diffs for `ruff format --check` and add support for different output formats ([#20443](https://github.com/astral-sh/ruff/pull/20443))
-- \[`pyflakes`\] Handle some common submodule import situations for `unused-import` (`F401`) ([#20200](https://github.com/astral-sh/ruff/pull/20200))
-- \[`ruff`\] Do not flag `%r` + `repr()` combinations (`RUF065`) ([#20600](https://github.com/astral-sh/ruff/pull/20600))
-
-### Bug fixes
-
-- \[`cli`\] Add conflict between `--add-noqa` and `--diff` options ([#20642](https://github.com/astral-sh/ruff/pull/20642))
-- \[`pylint`\] Exempt required imports from `PLR0402` ([#20381](https://github.com/astral-sh/ruff/pull/20381))
-- \[`pylint`\] Fix missing `max-nested-blocks` in settings display ([#20574](https://github.com/astral-sh/ruff/pull/20574))
-- \[`pyupgrade`\] Prevent infinite loop with `I002` and `UP026` ([#20634](https://github.com/astral-sh/ruff/pull/20634))
-
-### Rule changes
-
-- \[`flake8-simplify`\] Improve help message clarity (`SIM105`) ([#20548](https://github.com/astral-sh/ruff/pull/20548))
-
-### Documentation
-
-- Add the *The Basics* title back to CONTRIBUTING.md ([#20624](https://github.com/astral-sh/ruff/pull/20624))
-- Fixed documentation for try_consider_else ([#20587](https://github.com/astral-sh/ruff/pull/20587))
-- \[`isort`\] Clarify dependency between `order-by-type` and `case-sensitive` settings ([#20559](https://github.com/astral-sh/ruff/pull/20559))
-- \[`pylint`\] Clarify fix safety to include left-hand hashability (`PLR6201`) ([#20518](https://github.com/astral-sh/ruff/pull/20518))
-
-### Other changes
-
-- \[`playground`\] Fix quick fixes for empty ranges in playground ([#20599](https://github.com/astral-sh/ruff/pull/20599))
-
-### Contributors
-
-- [@TaKO8Ki](https://github.com/TaKO8Ki)
-- [@ntBre](https://github.com/ntBre)
-- [@dylwil3](https://github.com/dylwil3)
-- [@MichaReiser](https://github.com/MichaReiser)
-- [@danparizher](https://github.com/danparizher)
-- [@LilMonk](https://github.com/LilMonk)
-- [@mgiovani](https://github.com/mgiovani)
-- [@IDrokin117](https://github.com/IDrokin117)
-
 ## 0.13.2
 
 Released on 2025-09-25.

CONTRIBUTING.md

@@ -37,8 +37,6 @@ exploration of new features, we will often close these pull requests immediately
 new feature to ruff creates a long-term maintenance burden and requires strong consensus from the ruff
 team before it is appropriate to begin work on an implementation.
 
-## The Basics
-
 ### Prerequisites
 
 Ruff is written in Rust. You'll need to install the

Cargo.lock

@@ -2738,7 +2738,7 @@ dependencies = [
 
 [[package]]
 name = "ruff"
-version = "0.13.3"
+version = "0.13.2"
 dependencies = [
  "anyhow",
  "argfile",
@@ -2994,7 +2994,7 @@ dependencies = [
 
 [[package]]
 name = "ruff_linter"
-version = "0.13.3"
+version = "0.13.2"
 dependencies = [
  "aho-corasick",
  "anyhow",
@@ -3348,7 +3348,7 @@ dependencies = [
 
 [[package]]
 name = "ruff_wasm"
-version = "0.13.3"
+version = "0.13.2"
 dependencies = [
  "console_error_panic_hook",
  "console_log",
@@ -4449,7 +4449,6 @@ dependencies = [
  "ruff_source_file",
  "ruff_text_size",
  "serde-wasm-bindgen",
- "tracing",
  "ty_ide",
  "ty_project",
  "ty_python_semantic",

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.13.3/install.sh | sh
-powershell -c "irm https://astral.sh/ruff/0.13.3/install.ps1 | iex"
+curl -LsSf https://astral.sh/ruff/0.13.2/install.sh | sh
+powershell -c "irm https://astral.sh/ruff/0.13.2/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.13.3
+  rev: v0.13.2
   hooks:
     # Run the linter.
     - id: ruff-check

_typos.toml

@@ -31,7 +31,6 @@ extend-ignore-re = [
     "typ",
     # TODO: Remove this once the `TYP` redirects are removed from `rule_redirects.rs`
     "TYP",
-    "ntBre"
 ]
 
 [default.extend-identifiers]

crates/ruff/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "ruff"
-version = "0.13.3"
+version = "0.13.2"
 publish = true
 authors = { workspace = true }
 edition = { workspace = true }

crates/ruff/src/args.rs

@@ -416,7 +416,6 @@ pub struct CheckCommand {
         conflicts_with = "stdin_filename",
         conflicts_with = "watch",
         conflicts_with = "fix",
-        conflicts_with = "diff",
     )]
     pub add_noqa: bool,
     /// See the files Ruff will be run against with the current settings.
@@ -538,14 +537,6 @@ pub struct FormatCommand {
     /// Exit with a non-zero status code if any files were modified via format, even if all files were formatted successfully.
     #[arg(long, help_heading = "Miscellaneous", alias = "exit-non-zero-on-fix")]
     pub exit_non_zero_on_format: bool,
-
-    /// Output serialization format for violations, when used with `--check`.
-    /// The default serialization format is "full".
-    ///
-    /// Note that this option is currently only respected in preview mode. A warning will be emitted
-    /// if this flag is used on stable.
-    #[arg(long, value_enum, env = "RUFF_OUTPUT_FORMAT")]
-    pub output_format: Option<OutputFormat>,
 }
 
 #[derive(Copy, Clone, Debug, clap::Parser)]
@@ -793,7 +784,6 @@ impl FormatCommand {
             target_version: self.target_version.map(ast::PythonVersion::from),
             cache_dir: self.cache_dir,
             extension: self.extension,
-            output_format: self.output_format,
             ..ExplicitConfigOverrides::default()
         };
 

crates/ruff/src/commands/check.rs

@@ -9,10 +9,11 @@ use ignore::Error;
 use log::{debug, warn};
 #[cfg(not(target_family = "wasm"))]
 use rayon::prelude::*;
-use ruff_linter::message::create_panic_diagnostic;
 use rustc_hash::FxHashMap;
 
-use ruff_db::diagnostic::Diagnostic;
+use ruff_db::diagnostic::{
+    Annotation, Diagnostic, DiagnosticId, Span, SubDiagnostic, SubDiagnosticSeverity,
+};
 use ruff_db::panic::catch_unwind;
 use ruff_linter::package::PackageRoot;
 use ruff_linter::registry::Rule;
@@ -194,7 +195,23 @@ fn lint_path(
     match result {
         Ok(inner) => inner,
         Err(error) => {
-            let diagnostic = create_panic_diagnostic(&error, Some(path));
+            let message = match error.payload.as_str() {
+                Some(summary) => format!("Fatal error while linting: {summary}"),
+                _ => "Fatal error while linting".to_owned(),
+            };
+            let mut diagnostic = Diagnostic::new(
+                DiagnosticId::Panic,
+                ruff_db::diagnostic::Severity::Fatal,
+                message,
+            );
+            let span = Span::from(SourceFileBuilder::new(path.to_string_lossy(), "").finish());
+            let mut annotation = Annotation::primary(span);
+            annotation.set_file_level(true);
+            diagnostic.annotate(annotation);
+            diagnostic.sub(SubDiagnostic::new(
+                SubDiagnosticSeverity::Info,
+                format!("{error}"),
+            ));
             Ok(Diagnostics::new(vec![diagnostic], FxHashMap::default()))
         }
     }
@@ -210,8 +227,7 @@ mod test {
     use rustc_hash::FxHashMap;
     use tempfile::TempDir;
 
-    use ruff_db::diagnostic::{DiagnosticFormat, DisplayDiagnosticConfig, DisplayDiagnostics};
-    use ruff_linter::message::EmitterContext;
+    use ruff_linter::message::{Emitter, EmitterContext, TextEmitter};
     use ruff_linter::registry::Rule;
     use ruff_linter::settings::types::UnsafeFixes;
     use ruff_linter::settings::{LinterSettings, flags};
@@ -264,16 +280,19 @@ mod test {
             UnsafeFixes::Enabled,
         )
         .unwrap();
+        let mut output = Vec::new();
 
-        let config = DisplayDiagnosticConfig::default()
-            .format(DiagnosticFormat::Concise)
-            .hide_severity(true);
-        let messages = DisplayDiagnostics::new(
-            &EmitterContext::new(&FxHashMap::default()),
-            &config,
-            &diagnostics.inner,
-        )
-        .to_string();
+        TextEmitter::default()
+            .with_show_fix_status(true)
+            .with_color(false)
+            .emit(
+                &mut output,
+                &diagnostics.inner,
+                &EmitterContext::new(&FxHashMap::default()),
+            )
+            .unwrap();
+
+        let messages = String::from_utf8(output).unwrap();
 
         insta::with_settings!({
             omit_expression => true,

crates/ruff/src/commands/format.rs

@@ -11,19 +11,13 @@ use itertools::Itertools;
 use log::{error, warn};
 use rayon::iter::Either::{Left, Right};
 use rayon::iter::{IntoParallelRefIterator, ParallelIterator};
-use ruff_db::diagnostic::{
-    Annotation, Diagnostic, DiagnosticId, DisplayDiagnosticConfig, Severity, Span,
-};
-use ruff_linter::message::{EmitterContext, create_panic_diagnostic, render_diagnostics};
-use ruff_linter::settings::types::OutputFormat;
-use ruff_notebook::NotebookIndex;
 use ruff_python_parser::ParseError;
-use rustc_hash::{FxHashMap, FxHashSet};
+use rustc_hash::FxHashSet;
 use thiserror::Error;
 use tracing::debug;
 
 use ruff_db::panic::{PanicError, catch_unwind};
-use ruff_diagnostics::{Edit, Fix, SourceMap};
+use ruff_diagnostics::SourceMap;
 use ruff_linter::fs;
 use ruff_linter::logging::{DisplayParseError, LogLevel};
 use ruff_linter::package::PackageRoot;
@@ -33,15 +27,14 @@ use ruff_linter::source_kind::{SourceError, SourceKind};
 use ruff_linter::warn_user_once;
 use ruff_python_ast::{PySourceType, SourceType};
 use ruff_python_formatter::{FormatModuleError, QuoteStyle, format_module_source, format_range};
-use ruff_source_file::{LineIndex, LineRanges, OneIndexed, SourceFileBuilder};
+use ruff_source_file::LineIndex;
 use ruff_text_size::{TextLen, TextRange, TextSize};
 use ruff_workspace::FormatterSettings;
-use ruff_workspace::resolver::{
-    PyprojectConfig, ResolvedFile, Resolver, match_exclusion, python_files_in_path,
-};
+use ruff_workspace::resolver::{ResolvedFile, Resolver, match_exclusion, python_files_in_path};
 
 use crate::args::{ConfigArguments, FormatArguments, FormatRange};
 use crate::cache::{Cache, FileCacheKey, PackageCacheMap, PackageCaches};
+use crate::resolve::resolve;
 use crate::{ExitStatus, resolve_default_files};
 
 #[derive(Debug, Copy, Clone, is_macro::Is)]
@@ -70,14 +63,11 @@ impl FormatMode {
 pub(crate) fn format(
     cli: FormatArguments,
     config_arguments: &ConfigArguments,
-    pyproject_config: &PyprojectConfig,
 ) -> Result<ExitStatus> {
+    let pyproject_config = resolve(config_arguments, cli.stdin_filename.as_deref())?;
     let mode = FormatMode::from_cli(&cli);
     let files = resolve_default_files(cli.files, false);
-    let (paths, resolver) = python_files_in_path(&files, pyproject_config, config_arguments)?;
-
-    let output_format = pyproject_config.settings.output_format;
-    let preview = pyproject_config.settings.formatter.preview;
+    let (paths, resolver) = python_files_in_path(&files, &pyproject_config, config_arguments)?;
 
     if paths.is_empty() {
         warn_user_once!("No Python files found under the given path(s)");
@@ -194,26 +184,17 @@ pub(crate) fn format(
     caches.persist()?;
 
     // Report on any errors.
-    //
-    // We only convert errors to `Diagnostic`s in `Check` mode with preview enabled, otherwise we
-    // fall back on printing simple messages.
-    if !(preview.is_enabled() && mode.is_check()) {
-        errors.sort_unstable_by(|a, b| a.path().cmp(&b.path()));
+    errors.sort_unstable_by(|a, b| a.path().cmp(&b.path()));
 
-        for error in &errors {
-            error!("{error}");
-        }
+    for error in &errors {
+        error!("{error}");
     }
 
     let results = FormatResults::new(results.as_slice(), mode);
     match mode {
         FormatMode::Write => {}
         FormatMode::Check => {
-            if preview.is_enabled() {
-                results.write_changed_preview(&mut stdout().lock(), output_format, &errors)?;
-            } else {
-                results.write_changed(&mut stdout().lock())?;
-            }
+            results.write_changed(&mut stdout().lock())?;
         }
         FormatMode::Diff => {
             results.write_diff(&mut stdout().lock())?;
@@ -225,7 +206,7 @@ pub(crate) fn format(
         if mode.is_diff() {
             // Allow piping the diff to e.g. a file by writing the summary to stderr
             results.write_summary(&mut stderr().lock())?;
-        } else if !preview.is_enabled() || output_format.is_human_readable() {
+        } else {
             results.write_summary(&mut stdout().lock())?;
         }
     }
@@ -314,7 +295,8 @@ pub(crate) fn format_path(
 
                 FormatResult::Formatted
             }
-            FormatMode::Check | FormatMode::Diff => FormatResult::Diff {
+            FormatMode::Check => FormatResult::Formatted,
+            FormatMode::Diff => FormatResult::Diff {
                 unformatted,
                 formatted,
             },
@@ -347,7 +329,7 @@ pub(crate) enum FormattedSource {
 impl From<FormattedSource> for FormatResult {
     fn from(value: FormattedSource) -> Self {
         match value {
-            FormattedSource::Formatted { .. } => FormatResult::Formatted,
+            FormattedSource::Formatted(_) => FormatResult::Formatted,
             FormattedSource::Unchanged => FormatResult::Unchanged,
         }
     }
@@ -495,10 +477,10 @@ pub(crate) fn format_source(
 /// The result of an individual formatting operation.
 #[derive(Debug, Clone, is_macro::Is)]
 pub(crate) enum FormatResult {
-    /// The file was formatted and written back to disk.
+    /// The file was formatted.
     Formatted,
 
-    /// The file needs to be formatted, as the `formatted` and `unformatted` contents differ.
+    /// The file was formatted, [`SourceKind`] contains the formatted code
     Diff {
         unformatted: SourceKind,
         formatted: SourceKind,
@@ -570,7 +552,7 @@ impl<'a> FormatResults<'a> {
             .results
             .iter()
             .filter_map(|result| {
-                if result.result.is_diff() {
+                if result.result.is_formatted() {
                     Some(result.path.as_path())
                 } else {
                     None
@@ -584,30 +566,6 @@ impl<'a> FormatResults<'a> {
         Ok(())
     }
 
-    /// Write a list of the files that would be changed and any errors to the given writer.
-    fn write_changed_preview(
-        &self,
-        f: &mut impl Write,
-        output_format: OutputFormat,
-        errors: &[FormatCommandError],
-    ) -> io::Result<()> {
-        let mut notebook_index = FxHashMap::default();
-        let diagnostics: Vec<_> = errors
-            .iter()
-            .map(Diagnostic::from)
-            .chain(self.to_diagnostics(&mut notebook_index))
-            .sorted_unstable_by(Diagnostic::ruff_start_ordering)
-            .collect();
-
-        let context = EmitterContext::new(&notebook_index);
-        let config = DisplayDiagnosticConfig::default()
-            .hide_severity(true)
-            .show_fix_diff(true)
-            .color(!cfg!(test) && colored::control::SHOULD_COLORIZE.should_colorize());
-
-        render_diagnostics(f, output_format, config, &context, &diagnostics)
-    }
-
     /// Write a summary of the formatting results to the given writer.
     fn write_summary(&self, f: &mut impl Write) -> io::Result<()> {
         // Compute the number of changed and unchanged files.
@@ -670,155 +628,6 @@ impl<'a> FormatResults<'a> {
             Ok(())
         }
     }
-
-    /// Convert formatted files into [`Diagnostic`]s.
-    fn to_diagnostics(
-        &self,
-        notebook_index: &mut FxHashMap<String, NotebookIndex>,
-    ) -> impl Iterator<Item = Diagnostic> {
-        /// The number of unmodified context lines rendered in diffs.
-        ///
-        /// Note that this should be kept in sync with the argument to `TextDiff::grouped_ops` in
-        /// the diff rendering in `ruff_db` (currently 3). The `similar` crate uses two times that
-        /// argument as a cutoff for rendering unmodified lines.
-        const CONTEXT_LINES: u32 = 6;
-
-        self.results.iter().filter_map(|result| {
-            let (unformatted, formatted) = match &result.result {
-                FormatResult::Skipped | FormatResult::Unchanged => return None,
-                FormatResult::Diff {
-                    unformatted,
-                    formatted,
-                } => (unformatted, formatted),
-                FormatResult::Formatted => {
-                    debug_assert!(
-                        false,
-                        "Expected `FormatResult::Diff` for changed files in check mode"
-                    );
-                    return None;
-                }
-            };
-
-            let mut diagnostic = Diagnostic::new(
-                DiagnosticId::Unformatted,
-                Severity::Error,
-                "File would be reformatted",
-            );
-
-            // Locate the first and last characters that differ to use as the diagnostic
-            // range and to narrow the `Edit` range.
-            let modified_range = ModifiedRange::new(unformatted, formatted);
-
-            let path = result.path.to_string_lossy();
-            // For scripts, this is a single `Edit` using the `ModifiedRange` above, but notebook
-            // edits must be split by cell in order to render them as diffs.
-            //
-            // We also attempt to estimate the line number width for aligning the
-            // annotate-snippets header. This is only an estimate because we don't actually know
-            // if the maximum line number present in the document will be rendered as part of
-            // the diff, either as a changed line or as an unchanged context line. For
-            // notebooks, we refine our estimate by checking the number of lines in each cell
-            // individually, otherwise we could use `formatted.source_code().count_lines(...)`
-            // in both cases.
-            let (fix, line_count) = if let SourceKind::IpyNotebook(formatted) = formatted
-                && let SourceKind::IpyNotebook(unformatted) = unformatted
-            {
-                notebook_index.insert(path.to_string(), unformatted.index().clone());
-
-                let mut edits = formatted
-                    .cell_offsets()
-                    .ranges()
-                    .zip(unformatted.cell_offsets().ranges())
-                    .filter_map(|(formatted_range, unformatted_range)| {
-                        // Filter out cells that weren't modified. We use `intersect` instead of
-                        // `contains_range` because the full modified range might start or end in
-                        // the middle of a cell:
-                        //
-                        // ```
-                        // | cell 1 | cell 2 | cell 3 |
-                        //     |----------------| modified range
-                        // ```
-                        //
-                        // The intersection will be `Some` for all three cells in this case.
-                        if modified_range
-                            .unformatted
-                            .intersect(unformatted_range)
-                            .is_some()
-                        {
-                            let formatted = &formatted.source_code()[formatted_range];
-                            let edit = if formatted.is_empty() {
-                                Edit::range_deletion(unformatted_range)
-                            } else {
-                                Edit::range_replacement(formatted.to_string(), unformatted_range)
-                            };
-                            Some(edit)
-                        } else {
-                            None
-                        }
-                    });
-
-                let fix = Fix::safe_edits(
-                    edits
-                        .next()
-                        .expect("Formatted files must have at least one edit"),
-                    edits,
-                );
-                let source = formatted.source_code();
-                let line_count = formatted
-                    .cell_offsets()
-                    .ranges()
-                    .filter_map(|range| {
-                        if modified_range.formatted.contains_range(range) {
-                            Some(source.count_lines(range))
-                        } else {
-                            None
-                        }
-                    })
-                    .max()
-                    .unwrap_or_default();
-                (fix, line_count)
-            } else {
-                let formatted_code = &formatted.source_code()[modified_range.formatted];
-                let edit = if formatted_code.is_empty() {
-                    Edit::range_deletion(modified_range.unformatted)
-                } else {
-                    Edit::range_replacement(formatted_code.to_string(), modified_range.unformatted)
-                };
-                let fix = Fix::safe_edit(edit);
-                let line_count = formatted
-                    .source_code()
-                    .count_lines(TextRange::up_to(modified_range.formatted.end()));
-                (fix, line_count)
-            };
-
-            let source_file = SourceFileBuilder::new(path, unformatted.source_code()).finish();
-            let span = Span::from(source_file).with_range(modified_range.unformatted);
-            let mut annotation = Annotation::primary(span);
-            annotation.hide_snippet(true);
-            diagnostic.annotate(annotation);
-            diagnostic.set_fix(fix);
-
-            // TODO(brent) this offset is a hack to get the header of the diagnostic message, which
-            // is rendered by our fork of `annotate-snippets`, to align with our manually-rendered
-            // diff. `annotate-snippets` computes the alignment of the arrow in the header based on
-            // the maximum line number width in its rendered snippet. However, we don't have a
-            // reasonable range to underline in an annotation, so we don't send `annotate-snippets`
-            // a snippet to measure. If we commit to staying on our fork, a more robust way of
-            // handling this would be to move the diff rendering in
-            // `ruff_db::diagnostic::render::full` into `annotate-snippets`, likely as another
-            // `DisplayLine` variant and update the `lineno_width` calculation in
-            // `DisplayList::fmt`. That would handle this offset "automatically."
-            let line_count = (line_count + CONTEXT_LINES).min(
-                formatted
-                    .source_code()
-                    .count_lines(TextRange::up_to(formatted.source_code().text_len())),
-            );
-            let lines = OneIndexed::new(line_count as usize).unwrap_or_default();
-            diagnostic.set_header_offset(lines.digits().get());
-
-            Some(diagnostic)
-        })
-    }
 }
 
 /// An error that can occur while formatting a set of files.
@@ -830,6 +639,7 @@ pub(crate) enum FormatCommandError {
     Read(Option<PathBuf>, SourceError),
     Format(Option<PathBuf>, FormatModuleError),
     Write(Option<PathBuf>, SourceError),
+    Diff(Option<PathBuf>, io::Error),
     RangeFormatNotebook(Option<PathBuf>),
 }
 
@@ -848,65 +658,12 @@ impl FormatCommandError {
             | Self::Read(path, _)
             | Self::Format(path, _)
             | Self::Write(path, _)
+            | Self::Diff(path, _)
             | Self::RangeFormatNotebook(path) => path.as_deref(),
         }
     }
 }
 
-impl From<&FormatCommandError> for Diagnostic {
-    fn from(error: &FormatCommandError) -> Self {
-        let annotation = error.path().map(|path| {
-            let file = SourceFileBuilder::new(path.to_string_lossy(), "").finish();
-            let span = Span::from(file);
-            let mut annotation = Annotation::primary(span);
-            annotation.hide_snippet(true);
-            annotation
-        });
-
-        let mut diagnostic = match error {
-            FormatCommandError::Ignore(error) => {
-                Diagnostic::new(DiagnosticId::Io, Severity::Error, error)
-            }
-            FormatCommandError::Parse(display_parse_error) => Diagnostic::new(
-                DiagnosticId::InvalidSyntax,
-                Severity::Error,
-                &display_parse_error.error().error,
-            ),
-            FormatCommandError::Panic(path, panic_error) => {
-                return create_panic_diagnostic(panic_error, path.as_deref());
-            }
-            FormatCommandError::Read(_, source_error)
-            | FormatCommandError::Write(_, source_error) => {
-                Diagnostic::new(DiagnosticId::Io, Severity::Error, source_error)
-            }
-            FormatCommandError::Format(_, format_module_error) => match format_module_error {
-                FormatModuleError::ParseError(parse_error) => Diagnostic::new(
-                    DiagnosticId::InternalError,
-                    Severity::Error,
-                    &parse_error.error,
-                ),
-                FormatModuleError::FormatError(format_error) => {
-                    Diagnostic::new(DiagnosticId::InternalError, Severity::Error, format_error)
-                }
-                FormatModuleError::PrintError(print_error) => {
-                    Diagnostic::new(DiagnosticId::InternalError, Severity::Error, print_error)
-                }
-            },
-            FormatCommandError::RangeFormatNotebook(_) => Diagnostic::new(
-                DiagnosticId::InvalidCliOption,
-                Severity::Error,
-                "Range formatting isn't supported for notebooks.",
-            ),
-        };
-
-        if let Some(annotation) = annotation {
-            diagnostic.annotate(annotation);
-        }
-
-        diagnostic
-    }
-}
-
 impl Display for FormatCommandError {
     fn fmt(&self, f: &mut Formatter<'_>) -> std::fmt::Result {
         match self {
@@ -974,6 +731,23 @@ impl Display for FormatCommandError {
                     write!(f, "{header} {err}", header = "Failed to format:".bold())
                 }
             }
+            Self::Diff(path, err) => {
+                if let Some(path) = path {
+                    write!(
+                        f,
+                        "{}{}{} {err}",
+                        "Failed to generate diff for ".bold(),
+                        fs::relativize_path(path).bold(),
+                        ":".bold()
+                    )
+                } else {
+                    write!(
+                        f,
+                        "{header} {err}",
+                        header = "Failed to generate diff:".bold(),
+                    )
+                }
+            }
             Self::RangeFormatNotebook(path) => {
                 if let Some(path) = path {
                     write!(
@@ -1018,54 +792,6 @@ impl Display for FormatCommandError {
     }
 }
 
-#[derive(Debug)]
-struct ModifiedRange {
-    unformatted: TextRange,
-    formatted: TextRange,
-}
-
-impl ModifiedRange {
-    /// Determine the range that differs between `unformatted` and `formatted`.
-    ///
-    /// If the two inputs are equal, the returned ranges will be empty.
-    fn new(unformatted: &SourceKind, formatted: &SourceKind) -> Self {
-        let unformatted = unformatted.source_code();
-        let formatted = formatted.source_code();
-
-        let mut prefix_length = TextSize::ZERO;
-        for (unformatted, formatted) in unformatted.chars().zip(formatted.chars()) {
-            if unformatted != formatted {
-                break;
-            }
-            prefix_length += unformatted.text_len();
-        }
-
-        // For the ends of the ranges, track the length of the common suffix and then subtract that
-        // from each total text length. Unlike for `start`, the character offsets are very unlikely
-        // to be equal, so they need to be treated separately.
-        let mut suffix_length = TextSize::ZERO;
-        for (old, new) in unformatted[prefix_length.to_usize()..]
-            .chars()
-            .rev()
-            .zip(formatted[prefix_length.to_usize()..].chars().rev())
-        {
-            if old != new {
-                break;
-            }
-            suffix_length += old.text_len();
-        }
-
-        let unformatted_range =
-            TextRange::new(prefix_length, unformatted.text_len() - suffix_length);
-        let formatted_range = TextRange::new(prefix_length, formatted.text_len() - suffix_length);
-
-        Self {
-            unformatted: unformatted_range,
-            formatted: formatted_range,
-        }
-    }
-}
-
 pub(super) fn warn_incompatible_formatter_settings(resolver: &Resolver) {
     // First, collect all rules that are incompatible regardless of the linter-specific settings.
     let mut incompatible_rules = FxHashSet::default();
@@ -1237,144 +963,3 @@ pub(super) fn warn_incompatible_formatter_settings(resolver: &Resolver) {
         }
     }
 }
-
-#[cfg(test)]
-mod tests {
-    use std::io;
-    use std::ops::Range;
-    use std::path::PathBuf;
-
-    use ignore::Error;
-    use insta::assert_snapshot;
-
-    use ruff_db::panic::catch_unwind;
-    use ruff_linter::logging::DisplayParseError;
-    use ruff_linter::source_kind::{SourceError, SourceKind};
-    use ruff_python_formatter::FormatModuleError;
-    use ruff_python_parser::{ParseError, ParseErrorType};
-    use ruff_text_size::{TextRange, TextSize};
-    use test_case::test_case;
-
-    use crate::commands::format::{FormatCommandError, FormatMode, FormatResults, ModifiedRange};
-
-    #[test]
-    fn error_diagnostics() -> anyhow::Result<()> {
-        let path = PathBuf::from("test.py");
-        let source_kind = SourceKind::Python("1".to_string());
-
-        let panic_error = catch_unwind(|| {
-            panic!("Test panic for FormatCommandError");
-        })
-        .unwrap_err();
-
-        let errors = [
-            FormatCommandError::Ignore(Error::WithPath {
-                path: path.clone(),
-                err: Box::new(Error::Io(io::Error::new(
-                    io::ErrorKind::PermissionDenied,
-                    "Permission denied",
-                ))),
-            }),
-            FormatCommandError::Parse(DisplayParseError::from_source_kind(
-                ParseError {
-                    error: ParseErrorType::UnexpectedIndentation,
-                    location: TextRange::default(),
-                },
-                Some(path.clone()),
-                &source_kind,
-            )),
-            FormatCommandError::Panic(Some(path.clone()), Box::new(panic_error)),
-            FormatCommandError::Read(
-                Some(path.clone()),
-                SourceError::Io(io::Error::new(io::ErrorKind::NotFound, "File not found")),
-            ),
-            FormatCommandError::Format(
-                Some(path.clone()),
-                FormatModuleError::ParseError(ParseError {
-                    error: ParseErrorType::EmptySlice,
-                    location: TextRange::default(),
-                }),
-            ),
-            FormatCommandError::Write(
-                Some(path.clone()),
-                SourceError::Io(io::Error::new(
-                    io::ErrorKind::PermissionDenied,
-                    "Cannot write to file",
-                )),
-            ),
-            FormatCommandError::RangeFormatNotebook(Some(path)),
-        ];
-
-        let results = FormatResults::new(&[], FormatMode::Check);
-        let mut buf = Vec::new();
-        results.write_changed_preview(
-            &mut buf,
-            ruff_linter::settings::types::OutputFormat::Full,
-            &errors,
-        )?;
-
-        let mut settings = insta::Settings::clone_current();
-        settings.add_filter(r"(Panicked at) [^:]+:\d+:\d+", "$1 <location>");
-        let _s = settings.bind_to_scope();
-
-        assert_snapshot!(str::from_utf8(&buf)?, @r"
-        io: test.py: Permission denied
-        --> test.py:1:1
-
-        invalid-syntax: Unexpected indentation
-        --> test.py:1:1
-
-        io: File not found
-        --> test.py:1:1
-
-        internal-error: Expected index or slice expression
-        --> test.py:1:1
-
-        io: Cannot write to file
-        --> test.py:1:1
-
-        invalid-cli-option: Range formatting isn't supported for notebooks.
-        --> test.py:1:1
-
-        panic: Panicked at <location> when checking `test.py`: `Test panic for FormatCommandError`
-        --> test.py:1:1
-        info: This indicates a bug in Ruff.
-        info: If you could open an issue at https://github.com/astral-sh/ruff/issues/new?title=%5Bpanic%5D, we'd be very appreciative!
-        info: run with `RUST_BACKTRACE=1` environment variable to show the full backtrace information
-        ");
-
-        Ok(())
-    }
-
-    #[test_case("abcdef", "abcXYdef", 3..3, 3..5; "insertion")]
-    #[test_case("abcXYdef", "abcdef", 3..5, 3..3; "deletion")]
-    #[test_case("abcXdef", "abcYdef", 3..4, 3..4; "modification")]
-    #[test_case("abc", "abcX", 3..3, 3..4; "strict_prefix")]
-    #[test_case("", "", 0..0, 0..0; "empty")]
-    #[test_case("abc", "abc", 3..3, 3..3; "equal")]
-    fn modified_range(
-        unformatted: &str,
-        formatted: &str,
-        expect_unformatted: Range<u32>,
-        expect_formatted: Range<u32>,
-    ) {
-        let mr = ModifiedRange::new(
-            &SourceKind::Python(unformatted.to_string()),
-            &SourceKind::Python(formatted.to_string()),
-        );
-        assert_eq!(
-            mr.unformatted,
-            TextRange::new(
-                TextSize::new(expect_unformatted.start),
-                TextSize::new(expect_unformatted.end)
-            )
-        );
-        assert_eq!(
-            mr.formatted,
-            TextRange::new(
-                TextSize::new(expect_formatted.start),
-                TextSize::new(expect_formatted.end)
-            )
-        );
-    }
-}

crates/ruff/src/commands/format_stdin.rs

@@ -4,10 +4,10 @@ use std::path::Path;
 use anyhow::Result;
 use log::error;
 
-use ruff_linter::source_kind::{SourceError, SourceKind};
+use ruff_linter::source_kind::SourceKind;
 use ruff_python_ast::{PySourceType, SourceType};
 use ruff_workspace::FormatterSettings;
-use ruff_workspace::resolver::{PyprojectConfig, Resolver, match_exclusion, python_file_at_path};
+use ruff_workspace::resolver::{Resolver, match_exclusion, python_file_at_path};
 
 use crate::ExitStatus;
 use crate::args::{ConfigArguments, FormatArguments, FormatRange};
@@ -15,15 +15,17 @@ use crate::commands::format::{
     FormatCommandError, FormatMode, FormatResult, FormattedSource, format_source,
     warn_incompatible_formatter_settings,
 };
+use crate::resolve::resolve;
 use crate::stdin::{parrot_stdin, read_from_stdin};
 
 /// Run the formatter over a single file, read from `stdin`.
 pub(crate) fn format_stdin(
     cli: &FormatArguments,
     config_arguments: &ConfigArguments,
-    pyproject_config: &PyprojectConfig,
 ) -> Result<ExitStatus> {
-    let mut resolver = Resolver::new(pyproject_config);
+    let pyproject_config = resolve(config_arguments, cli.stdin_filename.as_deref())?;
+
+    let mut resolver = Resolver::new(&pyproject_config);
     warn_incompatible_formatter_settings(&resolver);
 
     let mode = FormatMode::from_cli(cli);
@@ -122,9 +124,7 @@ fn format_source_code(
                     "{}",
                     source_kind.diff(formatted, path).unwrap()
                 )
-                .map_err(|err| {
-                    FormatCommandError::Write(path.map(Path::to_path_buf), SourceError::Io(err))
-                })?;
+                .map_err(|err| FormatCommandError::Diff(path.map(Path::to_path_buf), err))?;
             }
         },
         FormattedSource::Unchanged => {

crates/ruff/src/lib.rs

@@ -205,18 +205,12 @@ pub fn run(
 }
 
 fn format(args: FormatCommand, global_options: GlobalConfigArgs) -> Result<ExitStatus> {
-    let cli_output_format_set = args.output_format.is_some();
     let (cli, config_arguments) = args.partition(global_options)?;
-    let pyproject_config = resolve::resolve(&config_arguments, cli.stdin_filename.as_deref())?;
-    if cli_output_format_set && !pyproject_config.settings.formatter.preview.is_enabled() {
-        warn_user_once!(
-            "The --output-format flag for the formatter is unstable and requires preview mode to use."
-        );
-    }
+
     if is_stdin(&cli.files, cli.stdin_filename.as_deref()) {
-        commands::format_stdin::format_stdin(&cli, &config_arguments, &pyproject_config)
+        commands::format_stdin::format_stdin(&cli, &config_arguments)
     } else {
-        commands::format::format(cli, &config_arguments, &pyproject_config)
+        commands::format::format(cli, &config_arguments)
     }
 }
 

crates/ruff/src/main.rs

@@ -1,7 +1,6 @@
 use std::io::Write;
 use std::process::ExitCode;
 
-use anyhow::Context;
 use clap::Parser;
 use colored::Colorize;
 
@@ -40,46 +39,39 @@ pub fn main() -> ExitCode {
     }
 
     let args = wild::args_os();
-    let args = match argfile::expand_args_from(args, argfile::parse_fromfile, argfile::PREFIX)
-        .context("Failed to read CLI arguments from files")
-    {
-        Ok(args) => args,
-        Err(err) => return report_error(&err),
-    };
+    let args = argfile::expand_args_from(args, argfile::parse_fromfile, argfile::PREFIX).unwrap();
 
     let args = Args::parse_from(args);
 
     match run(args) {
         Ok(code) => code.into(),
-        Err(err) => report_error(&err),
-    }
-}
+        Err(err) => {
+            {
+                // Exit "gracefully" on broken pipe errors.
+                //
+                // See: https://github.com/BurntSushi/ripgrep/blob/bf63fe8f258afc09bae6caa48f0ae35eaf115005/crates/core/main.rs#L47C1-L61C14
+                for cause in err.chain() {
+                    if let Some(ioerr) = cause.downcast_ref::<std::io::Error>() {
+                        if ioerr.kind() == std::io::ErrorKind::BrokenPipe {
+                            return ExitCode::from(0);
+                        }
+                    }
+                }
 
-fn report_error(err: &anyhow::Error) -> ExitCode {
-    {
-        // Exit "gracefully" on broken pipe errors.
-        //
-        // See: https://github.com/BurntSushi/ripgrep/blob/bf63fe8f258afc09bae6caa48f0ae35eaf115005/crates/core/main.rs#L47C1-L61C14
-        for cause in err.chain() {
-            if let Some(ioerr) = cause.downcast_ref::<std::io::Error>() {
-                if ioerr.kind() == std::io::ErrorKind::BrokenPipe {
-                    return ExitCode::from(0);
+                // Use `writeln` instead of `eprintln` to avoid panicking when the stderr pipe is broken.
+                let mut stderr = std::io::stderr().lock();
+
+                // This communicates that this isn't a linter error but ruff itself hard-errored for
+                // some reason (e.g. failed to resolve the configuration)
+                writeln!(stderr, "{}", "ruff failed".red().bold()).ok();
+                // Currently we generally only see one error, but e.g. with io errors when resolving
+                // the configuration it is help to chain errors ("resolving configuration failed" ->
+                // "failed to read file: subdir/pyproject.toml")
+                for cause in err.chain() {
+                    writeln!(stderr, "  {} {cause}", "Cause:".bold()).ok();
                 }
             }
-        }
-
-        // Use `writeln` instead of `eprintln` to avoid panicking when the stderr pipe is broken.
-        let mut stderr = std::io::stderr().lock();
-
-        // This communicates that this isn't a linter error but ruff itself hard-errored for
-        // some reason (e.g. failed to resolve the configuration)
-        writeln!(stderr, "{}", "ruff failed".red().bold()).ok();
-        // Currently we generally only see one error, but e.g. with io errors when resolving
-        // the configuration it is help to chain errors ("resolving configuration failed" ->
-        // "failed to read file: subdir/pyproject.toml")
-        for cause in err.chain() {
-            writeln!(stderr, "  {} {cause}", "Cause:".bold()).ok();
+            ExitStatus::Error.into()
         }
     }
-    ExitStatus::Error.into()
 }

crates/ruff/src/printer.rs

@@ -10,11 +10,12 @@ use ruff_linter::linter::FixTable;
 use serde::Serialize;
 
 use ruff_db::diagnostic::{
-    Diagnostic, DiagnosticFormat, DisplayDiagnosticConfig, DisplayDiagnostics, SecondaryCode,
+    Diagnostic, DiagnosticFormat, DisplayDiagnosticConfig, DisplayDiagnostics,
+    DisplayGithubDiagnostics, GithubRenderer, SecondaryCode,
 };
 use ruff_linter::fs::relativize_path;
 use ruff_linter::logging::LogLevel;
-use ruff_linter::message::{EmitterContext, render_diagnostics};
+use ruff_linter::message::{Emitter, EmitterContext, GroupedEmitter, SarifEmitter, TextEmitter};
 use ruff_linter::notify_user;
 use ruff_linter::settings::flags::{self};
 use ruff_linter::settings::types::{OutputFormat, UnsafeFixes};
@@ -224,28 +225,86 @@ impl Printer {
         let context = EmitterContext::new(&diagnostics.notebook_indexes);
         let fixables = FixableStatistics::try_from(diagnostics, self.unsafe_fixes);
 
-        let config = DisplayDiagnosticConfig::default()
-            .preview(preview)
-            .hide_severity(true)
-            .color(!cfg!(test) && colored::control::SHOULD_COLORIZE.should_colorize())
-            .with_show_fix_status(show_fix_status(self.fix_mode, fixables.as_ref()))
-            .with_fix_applicability(self.unsafe_fixes.required_applicability())
-            .show_fix_diff(preview);
+        let config = DisplayDiagnosticConfig::default().preview(preview);
 
-        render_diagnostics(writer, self.format, config, &context, &diagnostics.inner)?;
-
-        if matches!(
-            self.format,
-            OutputFormat::Full | OutputFormat::Concise | OutputFormat::Grouped
-        ) {
-            if self.flags.intersects(Flags::SHOW_FIX_SUMMARY) {
-                if !diagnostics.fixed.is_empty() {
-                    writeln!(writer)?;
-                    print_fix_summary(writer, &diagnostics.fixed)?;
-                    writeln!(writer)?;
-                }
+        match self.format {
+            OutputFormat::Json => {
+                let config = config.format(DiagnosticFormat::Json);
+                let value = DisplayDiagnostics::new(&context, &config, &diagnostics.inner);
+                write!(writer, "{value}")?;
+            }
+            OutputFormat::Rdjson => {
+                let config = config.format(DiagnosticFormat::Rdjson);
+                let value = DisplayDiagnostics::new(&context, &config, &diagnostics.inner);
+                write!(writer, "{value}")?;
+            }
+            OutputFormat::JsonLines => {
+                let config = config.format(DiagnosticFormat::JsonLines);
+                let value = DisplayDiagnostics::new(&context, &config, &diagnostics.inner);
+                write!(writer, "{value}")?;
+            }
+            OutputFormat::Junit => {
+                let config = config.format(DiagnosticFormat::Junit);
+                let value = DisplayDiagnostics::new(&context, &config, &diagnostics.inner);
+                write!(writer, "{value}")?;
+            }
+            OutputFormat::Concise | OutputFormat::Full => {
+                TextEmitter::default()
+                    .with_show_fix_status(show_fix_status(self.fix_mode, fixables.as_ref()))
+                    .with_show_fix_diff(self.format == OutputFormat::Full && preview)
+                    .with_show_source(self.format == OutputFormat::Full)
+                    .with_fix_applicability(self.unsafe_fixes.required_applicability())
+                    .with_preview(preview)
+                    .emit(writer, &diagnostics.inner, &context)?;
+
+                if self.flags.intersects(Flags::SHOW_FIX_SUMMARY) {
+                    if !diagnostics.fixed.is_empty() {
+                        writeln!(writer)?;
+                        print_fix_summary(writer, &diagnostics.fixed)?;
+                        writeln!(writer)?;
+                    }
+                }
+
+                self.write_summary_text(writer, diagnostics)?;
+            }
+            OutputFormat::Grouped => {
+                GroupedEmitter::default()
+                    .with_show_fix_status(show_fix_status(self.fix_mode, fixables.as_ref()))
+                    .with_unsafe_fixes(self.unsafe_fixes)
+                    .emit(writer, &diagnostics.inner, &context)?;
+
+                if self.flags.intersects(Flags::SHOW_FIX_SUMMARY) {
+                    if !diagnostics.fixed.is_empty() {
+                        writeln!(writer)?;
+                        print_fix_summary(writer, &diagnostics.fixed)?;
+                        writeln!(writer)?;
+                    }
+                }
+                self.write_summary_text(writer, diagnostics)?;
+            }
+            OutputFormat::Github => {
+                let renderer = GithubRenderer::new(&context, "Ruff");
+                let value = DisplayGithubDiagnostics::new(&renderer, &diagnostics.inner);
+                write!(writer, "{value}")?;
+            }
+            OutputFormat::Gitlab => {
+                let config = config.format(DiagnosticFormat::Gitlab);
+                let value = DisplayDiagnostics::new(&context, &config, &diagnostics.inner);
+                write!(writer, "{value}")?;
+            }
+            OutputFormat::Pylint => {
+                let config = config.format(DiagnosticFormat::Pylint);
+                let value = DisplayDiagnostics::new(&context, &config, &diagnostics.inner);
+                write!(writer, "{value}")?;
+            }
+            OutputFormat::Azure => {
+                let config = config.format(DiagnosticFormat::Azure);
+                let value = DisplayDiagnostics::new(&context, &config, &diagnostics.inner);
+                write!(writer, "{value}")?;
+            }
+            OutputFormat::Sarif => {
+                SarifEmitter.emit(writer, &diagnostics.inner, &context)?;
             }
-            self.write_summary_text(writer, diagnostics)?;
         }
 
         writer.flush()?;
@@ -389,22 +448,11 @@ impl Printer {
             }
 
             let context = EmitterContext::new(&diagnostics.notebook_indexes);
-            let format = if preview {
-                DiagnosticFormat::Full
-            } else {
-                DiagnosticFormat::Concise
-            };
-            let config = DisplayDiagnosticConfig::default()
-                .hide_severity(true)
-                .color(!cfg!(test) && colored::control::SHOULD_COLORIZE.should_colorize())
+            TextEmitter::default()
                 .with_show_fix_status(show_fix_status(self.fix_mode, fixables.as_ref()))
-                .format(format)
-                .with_fix_applicability(self.unsafe_fixes.required_applicability());
-            write!(
-                writer,
-                "{}",
-                DisplayDiagnostics::new(&context, &config, &diagnostics.inner)
-            )?;
+                .with_show_source(preview)
+                .with_fix_applicability(self.unsafe_fixes.required_applicability())
+                .emit(writer, &diagnostics.inner, &context)?;
         }
         writer.flush()?;
 

crates/ruff/tests/format.rs

@@ -12,8 +12,8 @@ use tempfile::TempDir;
 
 const BIN_NAME: &str = "ruff";
 
-fn tempdir_filter(path: impl AsRef<Path>) -> String {
-    format!(r"{}\\?/?", escape(path.as_ref().to_str().unwrap()))
+fn tempdir_filter(tempdir: &TempDir) -> String {
+    format!(r"{}\\?/?", escape(tempdir.path().to_str().unwrap()))
 }
 
 #[test]
@@ -609,112 +609,6 @@ if __name__ == "__main__":
     Ok(())
 }
 
-#[test_case::test_case("concise")]
-#[test_case::test_case("full")]
-#[test_case::test_case("json")]
-#[test_case::test_case("json-lines")]
-#[test_case::test_case("junit")]
-#[test_case::test_case("grouped")]
-#[test_case::test_case("github")]
-#[test_case::test_case("gitlab")]
-#[test_case::test_case("pylint")]
-#[test_case::test_case("rdjson")]
-#[test_case::test_case("azure")]
-#[test_case::test_case("sarif")]
-fn output_format(output_format: &str) -> Result<()> {
-    const CONTENT: &str = r#"
-from test import say_hy
-
-if __name__ == "__main__":
-    say_hy("dear Ruff contributor")
-"#;
-
-    let tempdir = TempDir::new()?;
-    let input = tempdir.path().join("input.py");
-    fs::write(&input, CONTENT)?;
-
-    let snapshot = format!("output_format_{output_format}");
-
-    let project_dir = dunce::canonicalize(tempdir.path())?;
-
-    insta::with_settings!({
-        filters => vec![
-            (tempdir_filter(&project_dir).as_str(), "[TMP]/"),
-            (tempdir_filter(&tempdir).as_str(), "[TMP]/"),
-            (r#""[^"]+\\?/?input.py"#, r#""[TMP]/input.py"#),
-            (ruff_linter::VERSION, "[VERSION]"),
-        ]
-    }, {
-        assert_cmd_snapshot!(
-            snapshot,
-            Command::new(get_cargo_bin(BIN_NAME))
-                .args([
-                    "format",
-                    "--no-cache",
-                    "--output-format",
-                    output_format,
-                    "--preview",
-                    "--check",
-                    "input.py",
-                ])
-                .current_dir(&tempdir),
-        );
-    });
-
-    Ok(())
-}
-
-#[test]
-fn output_format_notebook() {
-    let args = ["format", "--no-cache", "--isolated", "--preview", "--check"];
-    let fixtures = Path::new("resources").join("test").join("fixtures");
-    let path = fixtures.join("unformatted.ipynb");
-    insta::with_settings!({filters => vec![
-        // Replace windows paths
-        (r"\\", "/"),
-    ]}, {
-        assert_cmd_snapshot!(
-            Command::new(get_cargo_bin(BIN_NAME)).args(args).arg(path),
-            @r"
-        success: false
-        exit_code: 1
-        ----- stdout -----
-        unformatted: File would be reformatted
-          --> resources/test/fixtures/unformatted.ipynb:cell 1:1:1
-         ::: cell 1
-        1 | import numpy
-          - maths = (numpy.arange(100)**2).sum()
-          - stats= numpy.asarray([1,2,3,4]).median()
-        2 + 
-        3 + maths = (numpy.arange(100) ** 2).sum()
-        4 + stats = numpy.asarray([1, 2, 3, 4]).median()
-         ::: cell 3
-        1 | # A cell with IPython escape command
-        2 | def some_function(foo, bar):
-        3 |     pass
-        4 + 
-        5 + 
-        6 | %matplotlib inline
-          ::: cell 4
-        1  | foo = %pwd
-           - def some_function(foo,bar,):
-        2  + 
-        3  + 
-        4  + def some_function(
-        5  +     foo,
-        6  +     bar,
-        7  + ):
-        8  |     # Another cell with IPython escape command
-        9  |     foo = %pwd
-        10 |     print(foo)
-
-        1 file would be reformatted
-
-        ----- stderr -----
-        ");
-    });
-}
-
 #[test]
 fn exit_non_zero_on_format() -> Result<()> {
     let tempdir = TempDir::new()?;
@@ -2461,21 +2355,3 @@ fn cookiecutter_globbing() -> Result<()> {
 
     Ok(())
 }
-
-#[test]
-fn stable_output_format_warning() {
-    assert_cmd_snapshot!(
-        Command::new(get_cargo_bin(BIN_NAME))
-            .args(["format", "--output-format=full", "-"])
-            .pass_stdin("1"),
-        @r"
-    success: true
-    exit_code: 0
-    ----- stdout -----
-    1
-
-    ----- stderr -----
-    warning: The --output-format flag for the formatter is unstable and requires preview mode to use.
-    ",
-    );
-}

crates/ruff/tests/integration_test.rs

@@ -1688,27 +1688,6 @@ fn check_input_from_argfile() -> Result<()> {
     Ok(())
 }
 
-#[test]
-// Regression test for https://github.com/astral-sh/ruff/issues/20655
-fn missing_argfile_reports_error() {
-    let mut cmd = RuffCheck::default().filename("@!.txt").build();
-    insta::with_settings!({filters => vec![
-        ("The system cannot find the file specified.", "No such file or directory")
-    ]}, {
-        assert_cmd_snapshot!(cmd, @r"
-    success: false
-    exit_code: 2
-    ----- stdout -----
-
-    ----- stderr -----
-    ruff failed
-      Cause: Failed to read CLI arguments from files
-      Cause: failed to open file `!.txt`
-      Cause: No such file or directory (os error 2)
-    ");
-    });
-}
-
 #[test]
 fn check_hints_hidden_unsafe_fixes() {
     let mut cmd = RuffCheck::default()

crates/ruff/tests/lint.rs

@@ -6199,36 +6199,6 @@ match 42:  # invalid-syntax
     Ok(())
 }
 
-#[test_case::test_case("concise"; "concise_show_fixes")]
-#[test_case::test_case("full"; "full_show_fixes")]
-#[test_case::test_case("grouped"; "grouped_show_fixes")]
-fn output_format_show_fixes(output_format: &str) -> Result<()> {
-    let tempdir = TempDir::new()?;
-    let input = tempdir.path().join("input.py");
-    fs::write(&input, "import os  # F401")?;
-
-    let snapshot = format!("output_format_show_fixes_{output_format}");
-
-    assert_cmd_snapshot!(
-        snapshot,
-        Command::new(get_cargo_bin(BIN_NAME))
-            .args([
-                "check",
-                "--no-cache",
-                "--output-format",
-                output_format,
-                "--select",
-                "F401",
-                "--fix",
-                "--show-fixes",
-                "input.py",
-            ])
-            .current_dir(&tempdir),
-    );
-
-    Ok(())
-}
-
 #[test]
 fn up045_nested_optional_flatten_all() {
     let contents = "\
@@ -6302,7 +6272,6 @@ fn rule_panic_mixed_results_concise() -> Result<()> {
         filters => vec![
             (tempdir_filter(&tempdir).as_str(), "[TMP]/"),
             (r"\\", r"/"),
-            (r"(Panicked at) [^:]+:\d+:\d+", "$1 <location>")
         ]
     }, {
     assert_cmd_snapshot!(
@@ -6319,7 +6288,7 @@ fn rule_panic_mixed_results_concise() -> Result<()> {
     [TMP]/normal.py:1:1: RUF903 Hey this is a stable test rule with a display only fix.
     [TMP]/normal.py:1:1: RUF911 Hey this is a preview test rule.
     [TMP]/normal.py:1:1: RUF950 Hey this is a test rule that was redirected from another.
-    [TMP]/panic.py: panic: Panicked at <location> when checking `[TMP]/panic.py`: `This is a fake panic for testing.`
+    [TMP]/panic.py: panic: Fatal error while linting: This is a fake panic for testing.
     Found 7 errors.
     [*] 1 fixable with the `--fix` option (1 hidden fix can be enabled with the `--unsafe-fixes` option).
 
@@ -6348,7 +6317,6 @@ fn rule_panic_mixed_results_full() -> Result<()> {
         filters => vec![
             (tempdir_filter(&tempdir).as_str(), "[TMP]/"),
             (r"\\", r"/"),
-            (r"(Panicked at) [^:]+:\d+:\d+", "$1 <location>"),
         ]
     }, {
     assert_cmd_snapshot!(
@@ -6379,11 +6347,12 @@ fn rule_panic_mixed_results_full() -> Result<()> {
     RUF950 Hey this is a test rule that was redirected from another.
     --> [TMP]/normal.py:1:1
 
-    panic: Panicked at <location> when checking `[TMP]/panic.py`: `This is a fake panic for testing.`
+    panic: Fatal error while linting: This is a fake panic for testing.
     --> [TMP]/panic.py:1:1
-    info: This indicates a bug in Ruff.
-    info: If you could open an issue at https://github.com/astral-sh/ruff/issues/new?title=%5Bpanic%5D, we'd be very appreciative!
-    info: run with `RUST_BACKTRACE=1` environment variable to show the full backtrace information
+    info: panicked at crates/ruff_linter/src/rules/ruff/rules/test_rules.rs:511:9:
+    This is a fake panic for testing.
+    run with `RUST_BACKTRACE=1` environment variable to display a backtrace
+
 
     Found 7 errors.
     [*] 1 fixable with the `--fix` option (1 hidden fix can be enabled with the `--unsafe-fixes` option).

crates/ruff/tests/snapshots/format__output_format_azure.snap

@@ -1,19 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - azure
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-##vso[task.logissue type=error;sourcepath=[TMP]/input.py;linenumber=1;columnnumber=1;code=unformatted;]File would be reformatted
-
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_concise.snap

@@ -1,20 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - concise
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-input.py:1:1: unformatted: File would be reformatted
-1 file would be reformatted
-
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_full.snap

@@ -1,26 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - full
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-unformatted: File would be reformatted
- --> input.py:1:1
-  - 
-1 | from test import say_hy
-2 | 
-3 | if __name__ == "__main__":
-
-1 file would be reformatted
-
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_github.snap

@@ -1,19 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - github
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-::error title=Ruff (unformatted),file=[TMP]/input.py,line=1,col=1,endLine=2,endColumn=1::input.py:1:1: unformatted: File would be reformatted
-
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_gitlab.snap

@@ -1,38 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - gitlab
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-[
-  {
-    "check_name": "unformatted",
-    "description": "unformatted: File would be reformatted",
-    "severity": "major",
-    "fingerprint": "d868d7da11a65fcf",
-    "location": {
-      "path": "input.py",
-      "positions": {
-        "begin": {
-          "line": 1,
-          "column": 1
-        },
-        "end": {
-          "line": 2,
-          "column": 1
-        }
-      }
-    }
-  }
-]
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_grouped.snap

@@ -1,21 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - grouped
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-input.py:
-  1:1 unformatted: File would be reformatted
-
-1 file would be reformatted
-
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_json-lines.snap

@@ -1,19 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - json-lines
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-{"cell":null,"code":"unformatted","end_location":{"column":1,"row":2},"filename":"[TMP]/input.py","fix":{"applicability":"safe","edits":[{"content":"","end_location":{"column":1,"row":2},"location":{"column":1,"row":1}}],"message":null},"location":{"column":1,"row":1},"message":"File would be reformatted","noqa_row":null,"url":null}
-
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_json.snap

@@ -1,52 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - json
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-[
-  {
-    "cell": null,
-    "code": "unformatted",
-    "end_location": {
-      "column": 1,
-      "row": 2
-    },
-    "filename": "[TMP]/input.py",
-    "fix": {
-      "applicability": "safe",
-      "edits": [
-        {
-          "content": "",
-          "end_location": {
-            "column": 1,
-            "row": 2
-          },
-          "location": {
-            "column": 1,
-            "row": 1
-          }
-        }
-      ],
-      "message": null
-    },
-    "location": {
-      "column": 1,
-      "row": 1
-    },
-    "message": "File would be reformatted",
-    "noqa_row": null,
-    "url": null
-  }
-]
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_junit.snap

@@ -1,26 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - junit
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-<?xml version="1.0" encoding="UTF-8"?>
-<testsuites name="ruff" tests="1" failures="1" errors="0">
-    <testsuite name="[TMP]/input.py" tests="1" disabled="0" errors="0" failures="1" package="org.ruff">
-        <testcase name="org.ruff.unformatted" classname="[TMP]/input" line="1" column="1">
-            <failure message="File would be reformatted">line 1, col 1, File would be reformatted</failure>
-        </testcase>
-    </testsuite>
-</testsuites>
-
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_pylint.snap

@@ -1,18 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - pylint
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-input.py:1: [unformatted] File would be reformatted
-
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_rdjson.snap

@@ -1,60 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - rdjson
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-{
-  "diagnostics": [
-    {
-      "code": {
-        "value": "unformatted"
-      },
-      "location": {
-        "path": "[TMP]/input.py",
-        "range": {
-          "end": {
-            "column": 1,
-            "line": 2
-          },
-          "start": {
-            "column": 1,
-            "line": 1
-          }
-        }
-      },
-      "message": "File would be reformatted",
-      "suggestions": [
-        {
-          "range": {
-            "end": {
-              "column": 1,
-              "line": 2
-            },
-            "start": {
-              "column": 1,
-              "line": 1
-            }
-          },
-          "text": ""
-        }
-      ]
-    }
-  ],
-  "severity": "WARNING",
-  "source": {
-    "name": "ruff",
-    "url": "https://docs.astral.sh/ruff"
-  }
-}
------ stderr -----

crates/ruff/tests/snapshots/format__output_format_sarif.snap

@@ -1,81 +0,0 @@
----
-source: crates/ruff/tests/format.rs
-info:
-  program: ruff
-  args:
-    - format
-    - "--no-cache"
-    - "--output-format"
-    - sarif
-    - "--preview"
-    - "--check"
-    - input.py
----
-success: false
-exit_code: 1
------ stdout -----
-{
-  "$schema": "https://json.schemastore.org/sarif-2.1.0.json",
-  "runs": [
-    {
-      "results": [
-        {
-          "fixes": [
-            {
-              "artifactChanges": [
-                {
-                  "artifactLocation": {
-                    "uri": "[TMP]/input.py"
-                  },
-                  "replacements": [
-                    {
-                      "deletedRegion": {
-                        "endColumn": 1,
-                        "endLine": 2,
-                        "startColumn": 1,
-                        "startLine": 1
-                      }
-                    }
-                  ]
-                }
-              ],
-              "description": {
-                "text": null
-              }
-            }
-          ],
-          "level": "error",
-          "locations": [
-            {
-              "physicalLocation": {
-                "artifactLocation": {
-                  "uri": "[TMP]/input.py"
-                },
-                "region": {
-                  "endColumn": 1,
-                  "endLine": 2,
-                  "startColumn": 1,
-                  "startLine": 1
-                }
-              }
-            }
-          ],
-          "message": {
-            "text": "File would be reformatted"
-          },
-          "ruleId": "unformatted"
-        }
-      ],
-      "tool": {
-        "driver": {
-          "informationUri": "https://github.com/astral-sh/ruff",
-          "name": "ruff",
-          "rules": [],
-          "version": "[VERSION]"
-        }
-      }
-    }
-  ],
-  "version": "2.1.0"
-}
------ stderr -----

crates/ruff/tests/snapshots/integration_test__rule_f401.snap

@@ -44,43 +44,6 @@ import some_module
 __all__ = ["some_module"]
 ```
 
-## Preview
-When [preview] is enabled (and certain simplifying assumptions
-are met), we analyze all import statements for a given module
-when determining whether an import is used, rather than simply
-the last of these statements. This can result in both different and
-more import statements being marked as unused.
-
-For example, if a module consists of
-
-```python
-import a
-import a.b
-```
-
-then both statements are marked as unused under [preview], whereas
-only the second is marked as unused under stable behavior.
-
-As another example, if a module consists of
-
-```python
-import a.b
-import a
-
-a.b.foo()
-```
-
-then a diagnostic will only be emitted for the first line under [preview],
-whereas a diagnostic would only be emitted for the second line under
-stable behavior.
-
-Note that this behavior is somewhat subjective and is designed
-to conform to the developer's intuition rather than Python's actual
-execution. To wit, the statement `import a.b` automatically executes
-`import a`, so in some sense `import a` is _always_ redundant
-in the presence of `import a.b`.
-
-
 ## Fix safety
 
 Fixes to remove unused imports are safe, except in `__init__.py` files.
@@ -133,6 +96,4 @@ else:
 - [Python documentation: `importlib.util.find_spec`](https://docs.python.org/3/library/importlib.html#importlib.util.find_spec)
 - [Typing documentation: interface conventions](https://typing.python.org/en/latest/spec/distributing.html#library-interface-public-and-private-symbols)
 
-[preview]: https://docs.astral.sh/ruff/preview/
-
 ----- stderr -----

crates/ruff/tests/snapshots/lint__output_format_sarif.snap

@@ -119,7 +119,7 @@ exit_code: 1
           "rules": [
             {
               "fullDescription": {
-                "text": "## What it does\nChecks for unused imports.\n\n## Why is this bad?\nUnused imports add a performance overhead at runtime, and risk creating\nimport cycles. They also increase the cognitive load of reading the code.\n\nIf an import statement is used to check for the availability or existence\nof a module, consider using `importlib.util.find_spec` instead.\n\nIf an import statement is used to re-export a symbol as part of a module's\npublic interface, consider using a \"redundant\" import alias, which\ninstructs Ruff (and other tools) to respect the re-export, and avoid\nmarking it as unused, as in:\n\n```python\nfrom module import member as member\n```\n\nAlternatively, you can use `__all__` to declare a symbol as part of the module's\ninterface, as in:\n\n```python\n# __init__.py\nimport some_module\n\n__all__ = [\"some_module\"]\n```\n\n## Preview\nWhen [preview] is enabled (and certain simplifying assumptions\nare met), we analyze all import statements for a given module\nwhen determining whether an import is used, rather than simply\nthe last of these statements. This can result in both different and\nmore import statements being marked as unused.\n\nFor example, if a module consists of\n\n```python\nimport a\nimport a.b\n```\n\nthen both statements are marked as unused under [preview], whereas\nonly the second is marked as unused under stable behavior.\n\nAs another example, if a module consists of\n\n```python\nimport a.b\nimport a\n\na.b.foo()\n```\n\nthen a diagnostic will only be emitted for the first line under [preview],\nwhereas a diagnostic would only be emitted for the second line under\nstable behavior.\n\nNote that this behavior is somewhat subjective and is designed\nto conform to the developer's intuition rather than Python's actual\nexecution. To wit, the statement `import a.b` automatically executes\n`import a`, so in some sense `import a` is _always_ redundant\nin the presence of `import a.b`.\n\n\n## Fix safety\n\nFixes to remove unused imports are safe, except in `__init__.py` files.\n\nApplying fixes to `__init__.py` files is currently in preview. The fix offered depends on the\ntype of the unused import. Ruff will suggest a safe fix to export first-party imports with\neither a redundant alias or, if already present in the file, an `__all__` entry. If multiple\n`__all__` declarations are present, Ruff will not offer a fix. Ruff will suggest an unsafe fix\nto remove third-party and standard library imports -- the fix is unsafe because the module's\ninterface changes.\n\nSee [this FAQ section](https://docs.astral.sh/ruff/faq/#how-does-ruff-determine-which-of-my-imports-are-first-party-third-party-etc)\nfor more details on how Ruff\ndetermines whether an import is first or third-party.\n\n## Example\n\n```python\nimport numpy as np  # unused import\n\n\ndef area(radius):\n    return 3.14 * radius**2\n```\n\nUse instead:\n\n```python\ndef area(radius):\n    return 3.14 * radius**2\n```\n\nTo check the availability of a module, use `importlib.util.find_spec`:\n\n```python\nfrom importlib.util import find_spec\n\nif find_spec(\"numpy\") is not None:\n    print(\"numpy is installed\")\nelse:\n    print(\"numpy is not installed\")\n```\n\n## Options\n- `lint.ignore-init-module-imports`\n- `lint.pyflakes.allowed-unused-imports`\n\n## References\n- [Python documentation: `import`](https://docs.python.org/3/reference/simple_stmts.html#the-import-statement)\n- [Python documentation: `importlib.util.find_spec`](https://docs.python.org/3/library/importlib.html#importlib.util.find_spec)\n- [Typing documentation: interface conventions](https://typing.python.org/en/latest/spec/distributing.html#library-interface-public-and-private-symbols)\n\n[preview]: https://docs.astral.sh/ruff/preview/\n"
+                "text": "## What it does\nChecks for unused imports.\n\n## Why is this bad?\nUnused imports add a performance overhead at runtime, and risk creating\nimport cycles. They also increase the cognitive load of reading the code.\n\nIf an import statement is used to check for the availability or existence\nof a module, consider using `importlib.util.find_spec` instead.\n\nIf an import statement is used to re-export a symbol as part of a module's\npublic interface, consider using a \"redundant\" import alias, which\ninstructs Ruff (and other tools) to respect the re-export, and avoid\nmarking it as unused, as in:\n\n```python\nfrom module import member as member\n```\n\nAlternatively, you can use `__all__` to declare a symbol as part of the module's\ninterface, as in:\n\n```python\n# __init__.py\nimport some_module\n\n__all__ = [\"some_module\"]\n```\n\n## Fix safety\n\nFixes to remove unused imports are safe, except in `__init__.py` files.\n\nApplying fixes to `__init__.py` files is currently in preview. The fix offered depends on the\ntype of the unused import. Ruff will suggest a safe fix to export first-party imports with\neither a redundant alias or, if already present in the file, an `__all__` entry. If multiple\n`__all__` declarations are present, Ruff will not offer a fix. Ruff will suggest an unsafe fix\nto remove third-party and standard library imports -- the fix is unsafe because the module's\ninterface changes.\n\nSee [this FAQ section](https://docs.astral.sh/ruff/faq/#how-does-ruff-determine-which-of-my-imports-are-first-party-third-party-etc)\nfor more details on how Ruff\ndetermines whether an import is first or third-party.\n\n## Example\n\n```python\nimport numpy as np  # unused import\n\n\ndef area(radius):\n    return 3.14 * radius**2\n```\n\nUse instead:\n\n```python\ndef area(radius):\n    return 3.14 * radius**2\n```\n\nTo check the availability of a module, use `importlib.util.find_spec`:\n\n```python\nfrom importlib.util import find_spec\n\nif find_spec(\"numpy\") is not None:\n    print(\"numpy is installed\")\nelse:\n    print(\"numpy is not installed\")\n```\n\n## Options\n- `lint.ignore-init-module-imports`\n- `lint.pyflakes.allowed-unused-imports`\n\n## References\n- [Python documentation: `import`](https://docs.python.org/3/reference/simple_stmts.html#the-import-statement)\n- [Python documentation: `importlib.util.find_spec`](https://docs.python.org/3/library/importlib.html#importlib.util.find_spec)\n- [Typing documentation: interface conventions](https://typing.python.org/en/latest/spec/distributing.html#library-interface-public-and-private-symbols)\n"
               },
               "help": {
                 "text": "`{name}` imported but unused; consider using `importlib.util.find_spec` to test for availability"

crates/ruff/tests/snapshots/lint__output_format_show_fixes_concise.snap

@@ -1,26 +0,0 @@
----
-source: crates/ruff/tests/lint.rs
-info:
-  program: ruff
-  args:
-    - check
-    - "--no-cache"
-    - "--output-format"
-    - concise
-    - "--select"
-    - F401
-    - "--fix"
-    - "--show-fixes"
-    - input.py
----
-success: true
-exit_code: 0
------ stdout -----
-
-Fixed 1 error:
-- input.py:
-    1 × F401 (unused-import)
-
-Found 1 error (1 fixed, 0 remaining).
-
------ stderr -----

crates/ruff/tests/snapshots/lint__output_format_show_fixes_full.snap

@@ -1,26 +0,0 @@
----
-source: crates/ruff/tests/lint.rs
-info:
-  program: ruff
-  args:
-    - check
-    - "--no-cache"
-    - "--output-format"
-    - full
-    - "--select"
-    - F401
-    - "--fix"
-    - "--show-fixes"
-    - input.py
----
-success: true
-exit_code: 0
------ stdout -----
-
-Fixed 1 error:
-- input.py:
-    1 × F401 (unused-import)
-
-Found 1 error (1 fixed, 0 remaining).
-
------ stderr -----

crates/ruff/tests/snapshots/lint__output_format_show_fixes_grouped.snap

@@ -1,26 +0,0 @@
----
-source: crates/ruff/tests/lint.rs
-info:
-  program: ruff
-  args:
-    - check
-    - "--no-cache"
-    - "--output-format"
-    - grouped
-    - "--select"
-    - F401
-    - "--fix"
-    - "--show-fixes"
-    - input.py
----
-success: true
-exit_code: 0
------ stdout -----
-
-Fixed 1 error:
-- input.py:
-    1 × F401 (unused-import)
-
-Found 1 error (1 fixed, 0 remaining).
-
------ stderr -----

crates/ruff_annotate_snippets/src/renderer/display_list.rs

@@ -56,7 +56,6 @@ pub(crate) struct DisplayList<'a> {
     pub(crate) stylesheet: &'a Stylesheet,
     pub(crate) anonymized_line_numbers: bool,
     pub(crate) cut_indicator: &'static str,
-    pub(crate) lineno_offset: usize,
 }
 
 impl PartialEq for DisplayList<'_> {
@@ -82,14 +81,13 @@ impl Display for DisplayList<'_> {
                 _ => max,
             })
         });
-        let lineno_width = self.lineno_offset
-            + if lineno_width == 0 {
-                lineno_width
-            } else if self.anonymized_line_numbers {
-                ANONYMIZED_LINE_NUM.len()
-            } else {
-                ((lineno_width as f64).log10().floor() as usize) + 1
-            };
+        let lineno_width = if lineno_width == 0 {
+            lineno_width
+        } else if self.anonymized_line_numbers {
+            ANONYMIZED_LINE_NUM.len()
+        } else {
+            ((lineno_width as f64).log10().floor() as usize) + 1
+        };
 
         let multiline_depth = self.body.iter().fold(0, |max, set| {
             set.display_lines.iter().fold(max, |max2, line| match line {
@@ -126,7 +124,6 @@ impl<'a> DisplayList<'a> {
         term_width: usize,
         cut_indicator: &'static str,
     ) -> DisplayList<'a> {
-        let lineno_offset = message.lineno_offset;
         let body = format_message(
             message,
             term_width,
@@ -140,7 +137,6 @@ impl<'a> DisplayList<'a> {
             stylesheet,
             anonymized_line_numbers,
             cut_indicator,
-            lineno_offset,
         }
     }
 
@@ -1092,7 +1088,6 @@ fn format_message<'m>(
         footer,
         snippets,
         is_fixable,
-        lineno_offset: _,
     } = message;
 
     let mut sets = vec![];

crates/ruff_annotate_snippets/src/snippet.rs

@@ -23,7 +23,6 @@ pub struct Message<'a> {
     pub(crate) snippets: Vec<Snippet<'a>>,
     pub(crate) footer: Vec<Message<'a>>,
     pub(crate) is_fixable: bool,
-    pub(crate) lineno_offset: usize,
 }
 
 impl<'a> Message<'a> {
@@ -60,16 +59,6 @@ impl<'a> Message<'a> {
         self.is_fixable = yes;
         self
     }
-
-    /// Add an offset used for aligning the header sigil (`-->`) with the line number separators.
-    ///
-    /// For normal diagnostics this is computed automatically based on the lines to be rendered.
-    /// This is intended only for use in the formatter, where we don't render a snippet directly but
-    /// still want the header to align with the diff.
-    pub fn lineno_offset(mut self, offset: usize) -> Self {
-        self.lineno_offset = offset;
-        self
-    }
 }
 
 /// Structure containing the slice of text to be annotated and
@@ -155,7 +144,7 @@ impl<'a> Annotation<'a> {
         self
     }
 
-    pub fn hide_snippet(mut self, yes: bool) -> Self {
+    pub fn is_file_level(mut self, yes: bool) -> Self {
         self.is_file_level = yes;
         self
     }
@@ -184,7 +173,6 @@ impl Level {
             snippets: vec![],
             footer: vec![],
             is_fixable: false,
-            lineno_offset: 0,
         }
     }
 

crates/ruff_benchmark/benches/ty.rs

@@ -444,7 +444,7 @@ fn benchmark_complex_constrained_attributes_2(criterion: &mut Criterion) {
     criterion.bench_function("ty_micro[complex_constrained_attributes_2]", |b| {
         b.iter_batched_ref(
             || {
-                // This is similar to the case above, but now the attributes are actually defined.
+                // This is is similar to the case above, but now the attributes are actually defined.
                 // https://github.com/astral-sh/ty/issues/711
                 setup_micro_case(
                     r#"

crates/ruff_benchmark/benches/ty_walltime.rs

@@ -117,7 +117,7 @@ static COLOUR_SCIENCE: std::sync::LazyLock<Benchmark<'static>> = std::sync::Lazy
             max_dep_date: "2025-06-17",
             python_version: PythonVersion::PY310,
         },
-        500,
+        477,
     )
 });
 

crates/ruff_db/src/diagnostic/mod.rs

@@ -69,7 +69,6 @@ impl Diagnostic {
             parent: None,
             noqa_offset: None,
             secondary_code: None,
-            header_offset: 0,
         });
         Diagnostic { inner }
     }
@@ -433,23 +432,14 @@ impl Diagnostic {
 
     /// Returns the URL for the rule documentation, if it exists.
     pub fn to_ruff_url(&self) -> Option<String> {
-        match self.id() {
-            DiagnosticId::Panic
-            | DiagnosticId::Io
-            | DiagnosticId::InvalidSyntax
-            | DiagnosticId::RevealedType
-            | DiagnosticId::UnknownRule
-            | DiagnosticId::InvalidGlob
-            | DiagnosticId::EmptyInclude
-            | DiagnosticId::UnnecessaryOverridesSection
-            | DiagnosticId::UselessOverridesSection
-            | DiagnosticId::DeprecatedSetting
-            | DiagnosticId::Unformatted
-            | DiagnosticId::InvalidCliOption
-            | DiagnosticId::InternalError => None,
-            DiagnosticId::Lint(lint_name) => {
-                Some(format!("{}/rules/{lint_name}", env!("CARGO_PKG_HOMEPAGE")))
-            }
+        if self.is_invalid_syntax() {
+            None
+        } else {
+            Some(format!(
+                "{}/rules/{}",
+                env!("CARGO_PKG_HOMEPAGE"),
+                self.name()
+            ))
         }
     }
 
@@ -522,11 +512,6 @@ impl Diagnostic {
 
         a.cmp(&b)
     }
-
-    /// Add an offset for aligning the header sigil with the line number separators in a diff.
-    pub fn set_header_offset(&mut self, offset: usize) {
-        Arc::make_mut(&mut self.inner).header_offset = offset;
-    }
 }
 
 #[derive(Debug, Clone, Eq, PartialEq, Hash, get_size2::GetSize)]
@@ -540,7 +525,6 @@ struct DiagnosticInner {
     parent: Option<TextSize>,
     noqa_offset: Option<TextSize>,
     secondary_code: Option<SecondaryCode>,
-    header_offset: usize,
 }
 
 struct RenderingSortKey<'a> {
@@ -758,11 +742,11 @@ pub struct Annotation {
     is_primary: bool,
     /// The diagnostic tags associated with this annotation.
     tags: Vec<DiagnosticTag>,
-    /// Whether the snippet for this annotation should be hidden.
+    /// Whether this annotation is a file-level or full-file annotation.
     ///
     /// When set, rendering will only include the file's name and (optional) range. Everything else
     /// is omitted, including any file snippet or message.
-    hide_snippet: bool,
+    is_file_level: bool,
 }
 
 impl Annotation {
@@ -781,7 +765,7 @@ impl Annotation {
             message: None,
             is_primary: true,
             tags: Vec::new(),
-            hide_snippet: false,
+            is_file_level: false,
         }
     }
 
@@ -798,7 +782,7 @@ impl Annotation {
             message: None,
             is_primary: false,
             tags: Vec::new(),
-            hide_snippet: false,
+            is_file_level: false,
         }
     }
 
@@ -865,20 +849,19 @@ impl Annotation {
         self.tags.push(tag);
     }
 
-    /// Set whether or not the snippet on this annotation should be suppressed when rendering.
+    /// Set whether or not this annotation is file-level.
     ///
-    /// Such annotations are only rendered with their file name and range, if available. This is
-    /// intended for backwards compatibility with Ruff diagnostics, which historically used
+    /// File-level annotations are only rendered with their file name and range, if available. This
+    /// is intended for backwards compatibility with Ruff diagnostics, which historically used
     /// `TextRange::default` to indicate a file-level diagnostic. In the new diagnostic model, a
     /// [`Span`] with a range of `None` should be used instead, as mentioned in the `Span`
     /// documentation.
     ///
     /// TODO(brent) update this usage in Ruff and remove `is_file_level` entirely. See
     /// <https://github.com/astral-sh/ruff/issues/19688>, especially my first comment, for more
-    /// details. As of 2025-09-26 we also use this to suppress snippet rendering for formatter
-    /// diagnostics, which also need to have a range, so we probably can't eliminate this entirely.
-    pub fn hide_snippet(&mut self, yes: bool) {
-        self.hide_snippet = yes;
+    /// details.
+    pub fn set_file_level(&mut self, yes: bool) {
+        self.is_file_level = yes;
     }
 }
 
@@ -1033,17 +1016,6 @@ pub enum DiagnosticId {
 
     /// Use of a deprecated setting.
     DeprecatedSetting,
-
-    /// The code needs to be formatted.
-    Unformatted,
-
-    /// Use of an invalid command-line option.
-    InvalidCliOption,
-
-    /// An internal assumption was violated.
-    ///
-    /// This indicates a bug in the program rather than a user error.
-    InternalError,
 }
 
 impl DiagnosticId {
@@ -1083,9 +1055,6 @@ impl DiagnosticId {
             DiagnosticId::UnnecessaryOverridesSection => "unnecessary-overrides-section",
             DiagnosticId::UselessOverridesSection => "useless-overrides-section",
             DiagnosticId::DeprecatedSetting => "deprecated-setting",
-            DiagnosticId::Unformatted => "unformatted",
-            DiagnosticId::InvalidCliOption => "invalid-cli-option",
-            DiagnosticId::InternalError => "internal-error",
         }
     }
 
@@ -1384,7 +1353,7 @@ impl DisplayDiagnosticConfig {
     }
 
     /// Whether to show a fix's availability or not.
-    pub fn with_show_fix_status(self, yes: bool) -> DisplayDiagnosticConfig {
+    pub fn show_fix_status(self, yes: bool) -> DisplayDiagnosticConfig {
         DisplayDiagnosticConfig {
             show_fix_status: yes,
             ..self
@@ -1405,20 +1374,12 @@ impl DisplayDiagnosticConfig {
     /// availability for unsafe or display-only fixes.
     ///
     /// Note that this option is currently ignored when `hide_severity` is false.
-    pub fn with_fix_applicability(self, applicability: Applicability) -> DisplayDiagnosticConfig {
+    pub fn fix_applicability(self, applicability: Applicability) -> DisplayDiagnosticConfig {
         DisplayDiagnosticConfig {
             fix_applicability: applicability,
             ..self
         }
     }
-
-    pub fn show_fix_status(&self) -> bool {
-        self.show_fix_status
-    }
-
-    pub fn fix_applicability(&self) -> Applicability {
-        self.fix_applicability
-    }
 }
 
 impl Default for DisplayDiagnosticConfig {

crates/ruff_db/src/diagnostic/render.rs

@@ -208,7 +208,6 @@ struct ResolvedDiagnostic<'a> {
     message: String,
     annotations: Vec<ResolvedAnnotation<'a>>,
     is_fixable: bool,
-    header_offset: usize,
 }
 
 impl<'a> ResolvedDiagnostic<'a> {
@@ -259,8 +258,7 @@ impl<'a> ResolvedDiagnostic<'a> {
             id,
             message: diag.inner.message.as_str().to_string(),
             annotations,
-            is_fixable: config.show_fix_status && diag.has_applicable_fix(config),
-            header_offset: diag.inner.header_offset,
+            is_fixable: diag.has_applicable_fix(config),
         }
     }
 
@@ -290,7 +288,6 @@ impl<'a> ResolvedDiagnostic<'a> {
             message: diag.inner.message.as_str().to_string(),
             annotations,
             is_fixable: false,
-            header_offset: 0,
         }
     }
 
@@ -388,7 +385,6 @@ impl<'a> ResolvedDiagnostic<'a> {
             message: &self.message,
             snippets_by_input,
             is_fixable: self.is_fixable,
-            header_offset: self.header_offset,
         }
     }
 }
@@ -408,7 +404,7 @@ struct ResolvedAnnotation<'a> {
     line_end: OneIndexed,
     message: Option<&'a str>,
     is_primary: bool,
-    hide_snippet: bool,
+    is_file_level: bool,
     notebook_index: Option<NotebookIndex>,
 }
 
@@ -456,7 +452,7 @@ impl<'a> ResolvedAnnotation<'a> {
             line_end,
             message: ann.get_message(),
             is_primary: ann.is_primary,
-            hide_snippet: ann.hide_snippet,
+            is_file_level: ann.is_file_level,
             notebook_index: resolver.notebook_index(&ann.span.file),
         })
     }
@@ -496,11 +492,6 @@ struct RenderableDiagnostic<'r> {
     ///
     /// This is rendered as a `[*]` indicator after the diagnostic ID.
     is_fixable: bool,
-    /// Offset to align the header sigil (`-->`) with the subsequent line number separators.
-    ///
-    /// This is only needed for formatter diagnostics where we don't render a snippet via
-    /// `annotate-snippets` and thus the alignment isn't computed automatically.
-    header_offset: usize,
 }
 
 impl RenderableDiagnostic<'_> {
@@ -513,11 +504,7 @@ impl RenderableDiagnostic<'_> {
                 .iter()
                 .map(|snippet| snippet.to_annotate(path))
         });
-        let mut message = self
-            .level
-            .title(self.message)
-            .is_fixable(self.is_fixable)
-            .lineno_offset(self.header_offset);
+        let mut message = self.level.title(self.message).is_fixable(self.is_fixable);
         if let Some(id) = self.id {
             message = message.id(id);
         }
@@ -722,8 +709,8 @@ struct RenderableAnnotation<'r> {
     message: Option<&'r str>,
     /// Whether this annotation is considered "primary" or not.
     is_primary: bool,
-    /// Whether the snippet for this annotation should be hidden instead of rendered.
-    hide_snippet: bool,
+    /// Whether this annotation applies to an entire file, rather than a snippet within it.
+    is_file_level: bool,
 }
 
 impl<'r> RenderableAnnotation<'r> {
@@ -745,7 +732,7 @@ impl<'r> RenderableAnnotation<'r> {
             range,
             message: ann.message,
             is_primary: ann.is_primary,
-            hide_snippet: ann.hide_snippet,
+            is_file_level: ann.is_file_level,
         }
     }
 
@@ -771,7 +758,7 @@ impl<'r> RenderableAnnotation<'r> {
         if let Some(message) = self.message {
             ann = ann.label(message);
         }
-        ann.hide_snippet(self.hide_snippet)
+        ann.is_file_level(self.is_file_level)
     }
 }
 
@@ -2631,7 +2618,7 @@ watermelon
         /// Show fix availability when rendering.
         pub(super) fn show_fix_status(&mut self, yes: bool) {
             let mut config = std::mem::take(&mut self.config);
-            config = config.with_show_fix_status(yes);
+            config = config.show_fix_status(yes);
             self.config = config;
         }
 
@@ -2645,7 +2632,7 @@ watermelon
         /// 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);
-            config = config.with_fix_applicability(applicability);
+            config = config.fix_applicability(applicability);
             self.config = config;
         }
 

crates/ruff_db/src/diagnostic/render/full.rs

@@ -366,7 +366,6 @@ mod tests {
     fn hide_severity_output() {
         let (mut env, diagnostics) = create_diagnostics(DiagnosticFormat::Full);
         env.hide_severity(true);
-        env.show_fix_status(true);
         env.fix_applicability(Applicability::DisplayOnly);
 
         insta::assert_snapshot!(env.render_diagnostics(&diagnostics), @r#"
@@ -573,7 +572,7 @@ print()
         let mut diagnostic = env.err().build();
         let span = env.path("example.py").with_range(TextRange::default());
         let mut annotation = Annotation::primary(span);
-        annotation.hide_snippet(true);
+        annotation.set_file_level(true);
         diagnostic.annotate(annotation);
 
         insta::assert_snapshot!(env.render(&diagnostic), @r"
@@ -585,8 +584,7 @@ print()
     /// Check that ranges in notebooks are remapped relative to the cells.
     #[test]
     fn notebook_output() {
-        let (mut env, diagnostics) = create_notebook_diagnostics(DiagnosticFormat::Full);
-        env.show_fix_status(true);
+        let (env, diagnostics) = create_notebook_diagnostics(DiagnosticFormat::Full);
         insta::assert_snapshot!(env.render_diagnostics(&diagnostics), @r"
         error[unused-import][*]: `os` imported but unused
          --> notebook.ipynb:cell 1:2:8
@@ -700,7 +698,6 @@ print()
     fn notebook_output_with_diff() {
         let (mut env, diagnostics) = create_notebook_diagnostics(DiagnosticFormat::Full);
         env.show_fix_diff(true);
-        env.show_fix_status(true);
         env.fix_applicability(Applicability::DisplayOnly);
 
         insta::assert_snapshot!(env.render_diagnostics(&diagnostics), @r"
@@ -755,7 +752,6 @@ print()
     fn notebook_output_with_diff_spanning_cells() {
         let (mut env, mut diagnostics) = create_notebook_diagnostics(DiagnosticFormat::Full);
         env.show_fix_diff(true);
-        env.show_fix_status(true);
         env.fix_applicability(Applicability::DisplayOnly);
 
         // Move all of the edits from the later diagnostics to the first diagnostic to simulate a
@@ -932,7 +928,6 @@ line 10
         env.add("example.py", contents);
         env.format(DiagnosticFormat::Full);
         env.show_fix_diff(true);
-        env.show_fix_status(true);
         env.fix_applicability(Applicability::DisplayOnly);
 
         let mut diagnostic = env.err().primary("example.py", "3", "3", "label").build();

crates/ruff_db/src/files/file_root.rs

@@ -95,10 +95,7 @@ impl FileRoots {
         self.by_path.insert(route.clone(), root).unwrap();
 
         // Insert a path that matches all subdirectories and files
-        if !route.ends_with("/") {
-            route.push('/');
-        }
-        route.push_str("{*filepath}");
+        route.push_str("/{*filepath}");
 
         self.by_path.insert(route, root).unwrap();
         self.roots.push(root);

crates/ruff_db/src/panic.rs

@@ -31,29 +31,6 @@ impl Payload {
     }
 }
 
-impl PanicError {
-    pub fn to_diagnostic_message(&self, path: Option<impl std::fmt::Display>) -> String {
-        use std::fmt::Write;
-
-        let mut message = String::new();
-        message.push_str("Panicked");
-
-        if let Some(location) = &self.location {
-            let _ = write!(&mut message, " at {location}");
-        }
-
-        if let Some(path) = path {
-            let _ = write!(&mut message, " when checking `{path}`");
-        }
-
-        if let Some(payload) = self.payload.as_str() {
-            let _ = write!(&mut message, ": `{payload}`");
-        }
-
-        message
-    }
-}
-
 impl std::fmt::Display for PanicError {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         write!(f, "panicked at")?;

crates/ruff_formatter/src/diagnostics.rs

@@ -38,12 +38,12 @@ impl std::fmt::Display for FormatError {
             ),
             FormatError::InvalidDocument(error) => std::write!(
                 fmt,
-                "Invalid document: {error}\n\n This is an internal Ruff error. Please report if necessary."
+                "Invalid document: {error}\n\n This is an internal Rome error. Please report if necessary."
             ),
             FormatError::PoorLayout => {
                 std::write!(
                     fmt,
-                    "Poor layout: The formatter wasn't able to pick a good layout for your document. This is an internal Ruff error. Please report if necessary."
+                    "Poor layout: The formatter wasn't able to pick a good layout for your document. This is an internal Rome error. Please report if necessary."
                 )
             }
         }

crates/ruff_linter/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "ruff_linter"
-version = "0.13.3"
+version = "0.13.2"
 publish = false
 authors = { workspace = true }
 edition = { workspace = true }

crates/ruff_linter/resources/test/fixtures/flake8_annotations/shadowed_builtins.py

@@ -1,21 +0,0 @@
-from collections import UserString as str
-from typing import override
-
-def foo():
-    return "!"
-
-def _foo():
-    return "!"
-
-class C:
-    @override
-    def __str__(self):
-        return "!"
-    
-    @staticmethod
-    def foo():
-        return "!"
-    
-    @classmethod
-    def bar(cls):
-        return "!"

crates/ruff_linter/resources/test/fixtures/flake8_future_annotations/no_future_import_uses_preview_generics.py

@@ -1,91 +0,0 @@
-import asyncio
-import collections
-import contextlib
-import dataclasses
-import functools
-import os
-import queue
-import re
-import shelve
-import types
-import weakref
-from collections.abc import (
-    AsyncGenerator,
-    AsyncIterable,
-    AsyncIterator,
-    Awaitable,
-    ByteString,
-    Callable,
-    Collection,
-    Container,
-    Coroutine,
-    Generator,
-    Iterable,
-    Iterator,
-    ItemsView,
-    KeysView,
-    Mapping,
-    MappingView,
-    MutableMapping,
-    MutableSequence,
-    MutableSet,
-    Reversible,
-    Sequence,
-    Set,
-    ValuesView,
-)
-
-
-def takes_preview_generics(
-    future: asyncio.Future[int],
-    task: asyncio.Task[str],
-    deque_object: collections.deque[int],
-    defaultdict_object: collections.defaultdict[str, int],
-    ordered_dict: collections.OrderedDict[str, int],
-    counter_obj: collections.Counter[str],
-    chain_map: collections.ChainMap[str, int],
-    context_manager: contextlib.AbstractContextManager[str],
-    async_context_manager: contextlib.AbstractAsyncContextManager[int],
-    dataclass_field: dataclasses.Field[int],
-    cached_prop: functools.cached_property[int],
-    partial_method: functools.partialmethod[int],
-    path_like: os.PathLike[str],
-    lifo_queue: queue.LifoQueue[int],
-    regular_queue: queue.Queue[int],
-    priority_queue: queue.PriorityQueue[int],
-    simple_queue: queue.SimpleQueue[int],
-    regex_pattern: re.Pattern[str],
-    regex_match: re.Match[str],
-    bsd_db_shelf: shelve.BsdDbShelf[str, int],
-    db_filename_shelf: shelve.DbfilenameShelf[str, int],
-    shelf_obj: shelve.Shelf[str, int],
-    mapping_proxy: types.MappingProxyType[str, int],
-    weak_key_dict: weakref.WeakKeyDictionary[object, int],
-    weak_method: weakref.WeakMethod[int],
-    weak_set: weakref.WeakSet[int],
-    weak_value_dict: weakref.WeakValueDictionary[object, int],
-    awaitable: Awaitable[int],
-    coroutine: Coroutine[int, None, str],
-    async_iterable: AsyncIterable[int],
-    async_iterator: AsyncIterator[int],
-    async_generator: AsyncGenerator[int, None],
-    iterable: Iterable[int],
-    iterator: Iterator[int],
-    generator: Generator[int, None, None],
-    reversible: Reversible[int],
-    container: Container[int],
-    collection: Collection[int],
-    callable_obj: Callable[[int], str],
-    set_obj: Set[int],
-    mutable_set: MutableSet[int],
-    mapping: Mapping[str, int],
-    mutable_mapping: MutableMapping[str, int],
-    sequence: Sequence[int],
-    mutable_sequence: MutableSequence[int],
-    byte_string: ByteString[int],
-    mapping_view: MappingView[str, int],
-    keys_view: KeysView[str],
-    items_view: ItemsView[str, int],
-    values_view: ValuesView[int],
-) -> None:
-    ...

crates/ruff_linter/resources/test/fixtures/flynt/FLY002.py

@@ -23,9 +23,3 @@ nok9 = '\n'.join([r"raw string", '<""">', "<'''>"])  # Not OK (both triple-quote
 # Regression test for: https://github.com/astral-sh/ruff/issues/7197
 def create_file_public_url(url, filename):
     return''.join([url, filename])
-
-# Regression test for: https://github.com/astral-sh/ruff/issues/19837
-nok10 = "".join((foo, '"'))
-nok11 = ''.join((foo, "'"))
-nok12 = ''.join([foo, "'", '"'])
-nok13 = "".join([foo, "'", '"'])

crates/ruff_linter/resources/test/fixtures/pyupgrade/UP029_1.py

@@ -1 +0,0 @@
-from builtins import str, int

crates/ruff_linter/resources/test/fixtures/ruff/RUF065.py

@@ -37,9 +37,3 @@ log(logging.INFO, "Hello %r", repr("World!"))
 def str(s): return f"str = {s}"
 # Don't flag this
 logging.info("Hello %s", str("World!"))
-
-logging.info("Debug info: %r", repr("test\nstring"))
-logging.warning("Value: %r", repr(42))
-logging.error("Error: %r", repr([1, 2, 3]))
-logging.info("Debug info: %s", repr("test\nstring"))
-logging.warning("Value: %s", repr(42))

crates/ruff_linter/src/checkers/ast/analyze/expression.rs

@@ -8,8 +8,7 @@ use ruff_text_size::Ranged;
 
 use crate::checkers::ast::Checker;
 use crate::preview::{
-    is_future_required_preview_generics_enabled, is_optional_as_none_in_union_enabled,
-    is_unnecessary_default_type_args_stubs_enabled,
+    is_optional_as_none_in_union_enabled, is_unnecessary_default_type_args_stubs_enabled,
 };
 use crate::registry::Rule;
 use crate::rules::{
@@ -70,11 +69,7 @@ pub(crate) fn expression(expr: &Expr, checker: &Checker) {
                     && checker.semantic.in_annotation()
                     && checker.semantic.in_runtime_evaluated_annotation()
                     && !checker.semantic.in_string_type_definition()
-                    && typing::is_pep585_generic(
-                        value,
-                        &checker.semantic,
-                        is_future_required_preview_generics_enabled(checker.settings()),
-                    )
+                    && typing::is_pep585_generic(value, &checker.semantic)
                 {
                     flake8_future_annotations::rules::future_required_type_annotation(
                         checker,

crates/ruff_linter/src/checkers/ast/mod.rs

@@ -2360,7 +2360,7 @@ impl<'a> Checker<'a> {
         }
     }
 
-    /// Visit a body of [`Stmt`] nodes within a type-checking block.
+    /// Visit an body of [`Stmt`] nodes within a type-checking block.
     fn visit_type_checking_block(&mut self, body: &'a [Stmt]) {
         let snapshot = self.semantic.flags;
         self.semantic.flags |= SemanticModelFlags::TYPE_CHECKING_BLOCK;

crates/ruff_linter/src/logging.rs

@@ -223,11 +223,6 @@ impl DisplayParseError {
     pub fn path(&self) -> Option<&Path> {
         self.path.as_deref()
     }
-
-    /// Return the underlying [`ParseError`].
-    pub fn error(&self) -> &ParseError {
-        &self.error
-    }
 }
 
 impl std::error::Error for DisplayParseError {}

crates/ruff_linter/src/message/grouped.rs

@@ -6,25 +6,17 @@ use std::num::NonZeroUsize;
 use colored::Colorize;
 
 use ruff_db::diagnostic::Diagnostic;
-use ruff_diagnostics::Applicability;
 use ruff_notebook::NotebookIndex;
 use ruff_source_file::{LineColumn, OneIndexed};
 
 use crate::fs::relativize_path;
 use crate::message::{Emitter, EmitterContext};
+use crate::settings::types::UnsafeFixes;
 
+#[derive(Default)]
 pub struct GroupedEmitter {
     show_fix_status: bool,
-    applicability: Applicability,
-}
-
-impl Default for GroupedEmitter {
-    fn default() -> Self {
-        Self {
-            show_fix_status: false,
-            applicability: Applicability::Safe,
-        }
-    }
+    unsafe_fixes: UnsafeFixes,
 }
 
 impl GroupedEmitter {
@@ -35,8 +27,8 @@ impl GroupedEmitter {
     }
 
     #[must_use]
-    pub fn with_applicability(mut self, applicability: Applicability) -> Self {
-        self.applicability = applicability;
+    pub fn with_unsafe_fixes(mut self, unsafe_fixes: UnsafeFixes) -> Self {
+        self.unsafe_fixes = unsafe_fixes;
         self
     }
 }
@@ -75,7 +67,7 @@ impl Emitter for GroupedEmitter {
                         notebook_index: context.notebook_index(&message.expect_ruff_filename()),
                         message,
                         show_fix_status: self.show_fix_status,
-                        applicability: self.applicability,
+                        unsafe_fixes: self.unsafe_fixes,
                         row_length,
                         column_length,
                     }
@@ -122,7 +114,7 @@ fn group_diagnostics_by_filename(
 struct DisplayGroupedMessage<'a> {
     message: MessageWithLocation<'a>,
     show_fix_status: bool,
-    applicability: Applicability,
+    unsafe_fixes: UnsafeFixes,
     row_length: NonZeroUsize,
     column_length: NonZeroUsize,
     notebook_index: Option<&'a NotebookIndex>,
@@ -170,7 +162,7 @@ impl Display for DisplayGroupedMessage<'_> {
             code_and_body = RuleCodeAndBody {
                 message,
                 show_fix_status: self.show_fix_status,
-                applicability: self.applicability
+                unsafe_fixes: self.unsafe_fixes
             },
         )?;
 
@@ -181,7 +173,7 @@ impl Display for DisplayGroupedMessage<'_> {
 pub(super) struct RuleCodeAndBody<'a> {
     pub(crate) message: &'a Diagnostic,
     pub(crate) show_fix_status: bool,
-    pub(crate) applicability: Applicability,
+    pub(crate) unsafe_fixes: UnsafeFixes,
 }
 
 impl Display for RuleCodeAndBody<'_> {
@@ -189,7 +181,7 @@ impl Display for RuleCodeAndBody<'_> {
         if self.show_fix_status {
             if let Some(fix) = self.message.fix() {
                 // Do not display an indicator for inapplicable fixes
-                if fix.applies(self.applicability) {
+                if fix.applies(self.unsafe_fixes.required_applicability()) {
                     if let Some(code) = self.message.secondary_code() {
                         write!(f, "{} ", code.red().bold())?;
                     }
@@ -225,12 +217,11 @@ impl Display for RuleCodeAndBody<'_> {
 mod tests {
     use insta::assert_snapshot;
 
-    use ruff_diagnostics::Applicability;
-
     use crate::message::GroupedEmitter;
     use crate::message::tests::{
         capture_emitter_output, create_diagnostics, create_syntax_error_diagnostics,
     };
+    use crate::settings::types::UnsafeFixes;
 
     #[test]
     fn default() {
@@ -260,7 +251,7 @@ mod tests {
     fn fix_status_unsafe() {
         let mut emitter = GroupedEmitter::default()
             .with_show_fix_status(true)
-            .with_applicability(Applicability::Unsafe);
+            .with_unsafe_fixes(UnsafeFixes::Enabled);
         let content = capture_emitter_output(&mut emitter, &create_diagnostics());
 
         assert_snapshot!(content);

crates/ruff_linter/src/message/mod.rs

@@ -1,30 +1,27 @@
-use std::backtrace::BacktraceStatus;
 use std::fmt::Display;
 use std::io::Write;
-use std::path::Path;
 
-use ruff_db::panic::PanicError;
 use rustc_hash::FxHashMap;
 
 use ruff_db::diagnostic::{
-    Annotation, Diagnostic, DiagnosticFormat, DiagnosticId, DisplayDiagnosticConfig,
-    DisplayDiagnostics, DisplayGithubDiagnostics, FileResolver, GithubRenderer, Input, LintName,
-    SecondaryCode, Severity, Span, SubDiagnostic, SubDiagnosticSeverity, UnifiedFile,
+    Annotation, Diagnostic, DiagnosticId, FileResolver, Input, LintName, SecondaryCode, Severity,
+    Span, UnifiedFile,
 };
 use ruff_db::files::File;
 
 pub use grouped::GroupedEmitter;
 use ruff_notebook::NotebookIndex;
-use ruff_source_file::{SourceFile, SourceFileBuilder};
+use ruff_source_file::SourceFile;
 use ruff_text_size::{Ranged, TextRange, TextSize};
 pub use sarif::SarifEmitter;
+pub use text::TextEmitter;
 
 use crate::Fix;
 use crate::registry::Rule;
-use crate::settings::types::{OutputFormat, RuffOutputFormat};
 
 mod grouped;
 mod sarif;
+mod text;
 
 /// Creates a `Diagnostic` from a syntax error, with the format expected by Ruff.
 ///
@@ -44,55 +41,6 @@ pub fn create_syntax_error_diagnostic(
     diag
 }
 
-/// Create a `Diagnostic` from a panic.
-pub fn create_panic_diagnostic(error: &PanicError, path: Option<&Path>) -> Diagnostic {
-    let mut diagnostic = Diagnostic::new(
-        DiagnosticId::Panic,
-        Severity::Fatal,
-        error.to_diagnostic_message(path.as_ref().map(|path| path.display())),
-    );
-
-    diagnostic.sub(SubDiagnostic::new(
-        SubDiagnosticSeverity::Info,
-        "This indicates a bug in Ruff.",
-    ));
-    let report_message = "If you could open an issue at \
-                            https://github.com/astral-sh/ruff/issues/new?title=%5Bpanic%5D, \
-                            we'd be very appreciative!";
-    diagnostic.sub(SubDiagnostic::new(
-        SubDiagnosticSeverity::Info,
-        report_message,
-    ));
-
-    if let Some(backtrace) = &error.backtrace {
-        match backtrace.status() {
-            BacktraceStatus::Disabled => {
-                diagnostic.sub(SubDiagnostic::new(
-                            SubDiagnosticSeverity::Info,
-                            "run with `RUST_BACKTRACE=1` environment variable to show the full backtrace information",
-                        ));
-            }
-            BacktraceStatus::Captured => {
-                diagnostic.sub(SubDiagnostic::new(
-                    SubDiagnosticSeverity::Info,
-                    format!("Backtrace:\n{backtrace}"),
-                ));
-            }
-            _ => {}
-        }
-    }
-
-    if let Some(path) = path {
-        let file = SourceFileBuilder::new(path.to_string_lossy(), "").finish();
-        let span = Span::from(file);
-        let mut annotation = Annotation::primary(span);
-        annotation.hide_snippet(true);
-        diagnostic.annotate(annotation);
-    }
-
-    diagnostic
-}
-
 #[expect(clippy::too_many_arguments)]
 pub fn create_lint_diagnostic<B, S>(
     body: B,
@@ -122,7 +70,7 @@ where
     // actually need it, but we need to be able to cache the new diagnostic model first. See
     // https://github.com/astral-sh/ruff/issues/19688.
     if range == TextRange::default() {
-        annotation.hide_snippet(true);
+        annotation.set_file_level(true);
     }
     diagnostic.annotate(annotation);
 
@@ -212,48 +160,14 @@ impl<'a> EmitterContext<'a> {
     }
 }
 
-pub fn render_diagnostics(
-    writer: &mut dyn Write,
-    format: OutputFormat,
-    config: DisplayDiagnosticConfig,
-    context: &EmitterContext<'_>,
-    diagnostics: &[Diagnostic],
-) -> std::io::Result<()> {
-    match DiagnosticFormat::try_from(format) {
-        Ok(format) => {
-            let config = config.format(format);
-            let value = DisplayDiagnostics::new(context, &config, diagnostics);
-            write!(writer, "{value}")?;
-        }
-        Err(RuffOutputFormat::Github) => {
-            let renderer = GithubRenderer::new(context, "Ruff");
-            let value = DisplayGithubDiagnostics::new(&renderer, diagnostics);
-            write!(writer, "{value}")?;
-        }
-        Err(RuffOutputFormat::Grouped) => {
-            GroupedEmitter::default()
-                .with_show_fix_status(config.show_fix_status())
-                .with_applicability(config.fix_applicability())
-                .emit(writer, diagnostics, context)
-                .map_err(std::io::Error::other)?;
-        }
-        Err(RuffOutputFormat::Sarif) => {
-            SarifEmitter
-                .emit(writer, diagnostics, context)
-                .map_err(std::io::Error::other)?;
-        }
-    }
-
-    Ok(())
-}
-
 #[cfg(test)]
 mod tests {
     use rustc_hash::FxHashMap;
 
     use ruff_db::diagnostic::Diagnostic;
+    use ruff_notebook::NotebookIndex;
     use ruff_python_parser::{Mode, ParseOptions, parse_unchecked};
-    use ruff_source_file::SourceFileBuilder;
+    use ruff_source_file::{OneIndexed, SourceFileBuilder};
     use ruff_text_size::{TextRange, TextSize};
 
     use crate::codes::Rule;
@@ -343,6 +257,104 @@ def fibonacci(n):
         vec![unused_import, unused_variable, undefined_name]
     }
 
+    pub(super) fn create_notebook_diagnostics()
+    -> (Vec<Diagnostic>, FxHashMap<String, NotebookIndex>) {
+        let notebook = r"# cell 1
+import os
+# cell 2
+import math
+
+print('hello world')
+# cell 3
+def foo():
+    print()
+    x = 1
+";
+
+        let notebook_source = SourceFileBuilder::new("notebook.ipynb", notebook).finish();
+
+        let unused_import_os_start = TextSize::from(16);
+        let unused_import_os = create_lint_diagnostic(
+            "`os` imported but unused",
+            Some("Remove unused import: `os`"),
+            TextRange::new(unused_import_os_start, TextSize::from(18)),
+            Some(Fix::safe_edit(Edit::range_deletion(TextRange::new(
+                TextSize::from(9),
+                TextSize::from(19),
+            )))),
+            None,
+            notebook_source.clone(),
+            Some(unused_import_os_start),
+            Rule::UnusedImport,
+        );
+
+        let unused_import_math_start = TextSize::from(35);
+        let unused_import_math = create_lint_diagnostic(
+            "`math` imported but unused",
+            Some("Remove unused import: `math`"),
+            TextRange::new(unused_import_math_start, TextSize::from(39)),
+            Some(Fix::safe_edit(Edit::range_deletion(TextRange::new(
+                TextSize::from(28),
+                TextSize::from(40),
+            )))),
+            None,
+            notebook_source.clone(),
+            Some(unused_import_math_start),
+            Rule::UnusedImport,
+        );
+
+        let unused_variable_start = TextSize::from(98);
+        let unused_variable = create_lint_diagnostic(
+            "Local variable `x` is assigned to but never used",
+            Some("Remove assignment to unused variable `x`"),
+            TextRange::new(unused_variable_start, TextSize::from(99)),
+            Some(Fix::unsafe_edit(Edit::deletion(
+                TextSize::from(94),
+                TextSize::from(104),
+            ))),
+            None,
+            notebook_source,
+            Some(unused_variable_start),
+            Rule::UnusedVariable,
+        );
+
+        let mut notebook_indexes = FxHashMap::default();
+        notebook_indexes.insert(
+            "notebook.ipynb".to_string(),
+            NotebookIndex::new(
+                vec![
+                    OneIndexed::from_zero_indexed(0),
+                    OneIndexed::from_zero_indexed(0),
+                    OneIndexed::from_zero_indexed(1),
+                    OneIndexed::from_zero_indexed(1),
+                    OneIndexed::from_zero_indexed(1),
+                    OneIndexed::from_zero_indexed(1),
+                    OneIndexed::from_zero_indexed(2),
+                    OneIndexed::from_zero_indexed(2),
+                    OneIndexed::from_zero_indexed(2),
+                    OneIndexed::from_zero_indexed(2),
+                ],
+                vec![
+                    OneIndexed::from_zero_indexed(0),
+                    OneIndexed::from_zero_indexed(1),
+                    OneIndexed::from_zero_indexed(0),
+                    OneIndexed::from_zero_indexed(1),
+                    OneIndexed::from_zero_indexed(2),
+                    OneIndexed::from_zero_indexed(3),
+                    OneIndexed::from_zero_indexed(0),
+                    OneIndexed::from_zero_indexed(1),
+                    OneIndexed::from_zero_indexed(2),
+                    OneIndexed::from_zero_indexed(3),
+                ],
+            ),
+        );
+
+        (
+            vec![unused_import_os, unused_import_math, unused_variable],
+            notebook_indexes,
+        )
+    }
+
     pub(super) fn capture_emitter_output(
         emitter: &mut dyn Emitter,
         diagnostics: &[Diagnostic],
@@ -354,4 +366,16 @@ def fibonacci(n):
 
         String::from_utf8(output).expect("Output to be valid UTF-8")
     }
+
+    pub(super) fn capture_emitter_notebook_output(
+        emitter: &mut dyn Emitter,
+        diagnostics: &[Diagnostic],
+        notebook_indexes: &FxHashMap<String, NotebookIndex>,
+    ) -> String {
+        let context = EmitterContext::new(notebook_indexes);
+        let mut output: Vec<u8> = Vec::new();
+        emitter.emit(&mut output, diagnostics, &context).unwrap();
+
+        String::from_utf8(output).expect("Output to be valid UTF-8")
+    }
 }

crates/ruff_linter/src/message/snapshots/ruff_linter__message__sarif__tests__results.snap

@@ -129,7 +129,7 @@ expression: value
           "rules": [
             {
               "fullDescription": {
-                "text": "## What it does\nChecks for unused imports.\n\n## Why is this bad?\nUnused imports add a performance overhead at runtime, and risk creating\nimport cycles. They also increase the cognitive load of reading the code.\n\nIf an import statement is used to check for the availability or existence\nof a module, consider using `importlib.util.find_spec` instead.\n\nIf an import statement is used to re-export a symbol as part of a module's\npublic interface, consider using a \"redundant\" import alias, which\ninstructs Ruff (and other tools) to respect the re-export, and avoid\nmarking it as unused, as in:\n\n```python\nfrom module import member as member\n```\n\nAlternatively, you can use `__all__` to declare a symbol as part of the module's\ninterface, as in:\n\n```python\n# __init__.py\nimport some_module\n\n__all__ = [\"some_module\"]\n```\n\n## Preview\nWhen [preview] is enabled (and certain simplifying assumptions\nare met), we analyze all import statements for a given module\nwhen determining whether an import is used, rather than simply\nthe last of these statements. This can result in both different and\nmore import statements being marked as unused.\n\nFor example, if a module consists of\n\n```python\nimport a\nimport a.b\n```\n\nthen both statements are marked as unused under [preview], whereas\nonly the second is marked as unused under stable behavior.\n\nAs another example, if a module consists of\n\n```python\nimport a.b\nimport a\n\na.b.foo()\n```\n\nthen a diagnostic will only be emitted for the first line under [preview],\nwhereas a diagnostic would only be emitted for the second line under\nstable behavior.\n\nNote that this behavior is somewhat subjective and is designed\nto conform to the developer's intuition rather than Python's actual\nexecution. To wit, the statement `import a.b` automatically executes\n`import a`, so in some sense `import a` is _always_ redundant\nin the presence of `import a.b`.\n\n\n## Fix safety\n\nFixes to remove unused imports are safe, except in `__init__.py` files.\n\nApplying fixes to `__init__.py` files is currently in preview. The fix offered depends on the\ntype of the unused import. Ruff will suggest a safe fix to export first-party imports with\neither a redundant alias or, if already present in the file, an `__all__` entry. If multiple\n`__all__` declarations are present, Ruff will not offer a fix. Ruff will suggest an unsafe fix\nto remove third-party and standard library imports -- the fix is unsafe because the module's\ninterface changes.\n\nSee [this FAQ section](https://docs.astral.sh/ruff/faq/#how-does-ruff-determine-which-of-my-imports-are-first-party-third-party-etc)\nfor more details on how Ruff\ndetermines whether an import is first or third-party.\n\n## Example\n\n```python\nimport numpy as np  # unused import\n\n\ndef area(radius):\n    return 3.14 * radius**2\n```\n\nUse instead:\n\n```python\ndef area(radius):\n    return 3.14 * radius**2\n```\n\nTo check the availability of a module, use `importlib.util.find_spec`:\n\n```python\nfrom importlib.util import find_spec\n\nif find_spec(\"numpy\") is not None:\n    print(\"numpy is installed\")\nelse:\n    print(\"numpy is not installed\")\n```\n\n## Options\n- `lint.ignore-init-module-imports`\n- `lint.pyflakes.allowed-unused-imports`\n\n## References\n- [Python documentation: `import`](https://docs.python.org/3/reference/simple_stmts.html#the-import-statement)\n- [Python documentation: `importlib.util.find_spec`](https://docs.python.org/3/library/importlib.html#importlib.util.find_spec)\n- [Typing documentation: interface conventions](https://typing.python.org/en/latest/spec/distributing.html#library-interface-public-and-private-symbols)\n\n[preview]: https://docs.astral.sh/ruff/preview/\n"
+                "text": "## What it does\nChecks for unused imports.\n\n## Why is this bad?\nUnused imports add a performance overhead at runtime, and risk creating\nimport cycles. They also increase the cognitive load of reading the code.\n\nIf an import statement is used to check for the availability or existence\nof a module, consider using `importlib.util.find_spec` instead.\n\nIf an import statement is used to re-export a symbol as part of a module's\npublic interface, consider using a \"redundant\" import alias, which\ninstructs Ruff (and other tools) to respect the re-export, and avoid\nmarking it as unused, as in:\n\n```python\nfrom module import member as member\n```\n\nAlternatively, you can use `__all__` to declare a symbol as part of the module's\ninterface, as in:\n\n```python\n# __init__.py\nimport some_module\n\n__all__ = [\"some_module\"]\n```\n\n## Fix safety\n\nFixes to remove unused imports are safe, except in `__init__.py` files.\n\nApplying fixes to `__init__.py` files is currently in preview. The fix offered depends on the\ntype of the unused import. Ruff will suggest a safe fix to export first-party imports with\neither a redundant alias or, if already present in the file, an `__all__` entry. If multiple\n`__all__` declarations are present, Ruff will not offer a fix. Ruff will suggest an unsafe fix\nto remove third-party and standard library imports -- the fix is unsafe because the module's\ninterface changes.\n\nSee [this FAQ section](https://docs.astral.sh/ruff/faq/#how-does-ruff-determine-which-of-my-imports-are-first-party-third-party-etc)\nfor more details on how Ruff\ndetermines whether an import is first or third-party.\n\n## Example\n\n```python\nimport numpy as np  # unused import\n\n\ndef area(radius):\n    return 3.14 * radius**2\n```\n\nUse instead:\n\n```python\ndef area(radius):\n    return 3.14 * radius**2\n```\n\nTo check the availability of a module, use `importlib.util.find_spec`:\n\n```python\nfrom importlib.util import find_spec\n\nif find_spec(\"numpy\") is not None:\n    print(\"numpy is installed\")\nelse:\n    print(\"numpy is not installed\")\n```\n\n## Options\n- `lint.ignore-init-module-imports`\n- `lint.pyflakes.allowed-unused-imports`\n\n## References\n- [Python documentation: `import`](https://docs.python.org/3/reference/simple_stmts.html#the-import-statement)\n- [Python documentation: `importlib.util.find_spec`](https://docs.python.org/3/library/importlib.html#importlib.util.find_spec)\n- [Typing documentation: interface conventions](https://typing.python.org/en/latest/spec/distributing.html#library-interface-public-and-private-symbols)\n"
               },
               "help": {
                 "text": "`{name}` imported but unused; consider using `importlib.util.find_spec` to test for availability"

crates/ruff_linter/src/message/snapshots/ruff_linter__message__text__tests__default.snap

@@ -0,0 +1,30 @@
+---
+source: crates/ruff_linter/src/message/text.rs
+expression: content
+---
+F401 `os` imported but unused
+ --> fib.py:1:8
+  |
+1 | import os
+  |        ^^
+  |
+help: Remove unused import: `os`
+
+F841 Local variable `x` is assigned to but never used
+ --> fib.py:6:5
+  |
+4 | def fibonacci(n):
+5 |     """Compute the nth number in the Fibonacci sequence."""
+6 |     x = 1
+  |     ^
+7 |     if n == 0:
+8 |         return 0
+  |
+help: Remove assignment to unused variable `x`
+
+F821 Undefined name `a`
+ --> undef.py:1:4
+  |
+1 | if a == 1: pass
+  |    ^
+  |

crates/ruff_linter/src/message/snapshots/ruff_linter__message__text__tests__fix_status.snap

@@ -0,0 +1,30 @@
+---
+source: crates/ruff_linter/src/message/text.rs
+expression: content
+---
+F401 `os` imported but unused
+ --> fib.py:1:8
+  |
+1 | import os
+  |        ^^
+  |
+help: Remove unused import: `os`
+
+F841 Local variable `x` is assigned to but never used
+ --> fib.py:6:5
+  |
+4 | def fibonacci(n):
+5 |     """Compute the nth number in the Fibonacci sequence."""
+6 |     x = 1
+  |     ^
+7 |     if n == 0:
+8 |         return 0
+  |
+help: Remove assignment to unused variable `x`
+
+F821 Undefined name `a`
+ --> undef.py:1:4
+  |
+1 | if a == 1: pass
+  |    ^
+  |

crates/ruff_linter/src/message/snapshots/ruff_linter__message__text__tests__fix_status_unsafe.snap

@@ -0,0 +1,30 @@
+---
+source: crates/ruff_linter/src/message/text.rs
+expression: content
+---
+F401 [*] `os` imported but unused
+ --> fib.py:1:8
+  |
+1 | import os
+  |        ^^
+  |
+help: Remove unused import: `os`
+
+F841 [*] Local variable `x` is assigned to but never used
+ --> fib.py:6:5
+  |
+4 | def fibonacci(n):
+5 |     """Compute the nth number in the Fibonacci sequence."""
+6 |     x = 1
+  |     ^
+7 |     if n == 0:
+8 |         return 0
+  |
+help: Remove assignment to unused variable `x`
+
+F821 Undefined name `a`
+ --> undef.py:1:4
+  |
+1 | if a == 1: pass
+  |    ^
+  |

crates/ruff_linter/src/message/snapshots/ruff_linter__message__text__tests__notebook_output.snap

@@ -0,0 +1,33 @@
+---
+source: crates/ruff_linter/src/message/text.rs
+expression: content
+---
+F401 [*] `os` imported but unused
+ --> notebook.ipynb:cell 1:2:8
+  |
+1 | # cell 1
+2 | import os
+  |        ^^
+  |
+help: Remove unused import: `os`
+
+F401 [*] `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`
+
+F841 [*] 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`

crates/ruff_linter/src/message/snapshots/ruff_linter__message__text__tests__syntax_errors.snap

@@ -0,0 +1,23 @@
+---
+source: crates/ruff_linter/src/message/text.rs
+expression: content
+---
+invalid-syntax: Expected one or more symbol names after import
+ --> syntax_errors.py:1:15
+  |
+1 | from os import
+  |               ^
+2 |
+3 | if call(foo
+  |
+
+invalid-syntax: Expected ')', found newline
+ --> syntax_errors.py:3:12
+  |
+1 | from os import
+2 |
+3 | if call(foo
+  |            ^
+4 |     def bar():
+5 |         pass
+  |

crates/ruff_linter/src/message/text.rs

@@ -0,0 +1,143 @@
+use std::io::Write;
+
+use ruff_db::diagnostic::{
+    Diagnostic, DiagnosticFormat, DisplayDiagnosticConfig, DisplayDiagnostics,
+};
+use ruff_diagnostics::Applicability;
+
+use crate::message::{Emitter, EmitterContext};
+
+pub struct TextEmitter {
+    config: DisplayDiagnosticConfig,
+}
+
+impl Default for TextEmitter {
+    fn default() -> Self {
+        Self {
+            config: DisplayDiagnosticConfig::default()
+                .format(DiagnosticFormat::Concise)
+                .hide_severity(true)
+                .color(!cfg!(test) && colored::control::SHOULD_COLORIZE.should_colorize()),
+        }
+    }
+}
+
+impl TextEmitter {
+    #[must_use]
+    pub fn with_show_fix_status(mut self, show_fix_status: bool) -> Self {
+        self.config = self.config.show_fix_status(show_fix_status);
+        self
+    }
+
+    #[must_use]
+    pub fn with_show_fix_diff(mut self, show_fix_diff: bool) -> Self {
+        self.config = self.config.show_fix_diff(show_fix_diff);
+        self
+    }
+
+    #[must_use]
+    pub fn with_show_source(mut self, show_source: bool) -> Self {
+        self.config = self.config.format(if show_source {
+            DiagnosticFormat::Full
+        } else {
+            DiagnosticFormat::Concise
+        });
+        self
+    }
+
+    #[must_use]
+    pub fn with_fix_applicability(mut self, applicability: Applicability) -> Self {
+        self.config = self.config.fix_applicability(applicability);
+        self
+    }
+
+    #[must_use]
+    pub fn with_preview(mut self, preview: bool) -> Self {
+        self.config = self.config.preview(preview);
+        self
+    }
+
+    #[must_use]
+    pub fn with_color(mut self, color: bool) -> Self {
+        self.config = self.config.color(color);
+        self
+    }
+}
+
+impl Emitter for TextEmitter {
+    fn emit(
+        &mut self,
+        writer: &mut dyn Write,
+        diagnostics: &[Diagnostic],
+        context: &EmitterContext,
+    ) -> anyhow::Result<()> {
+        write!(
+            writer,
+            "{}",
+            DisplayDiagnostics::new(context, &self.config, diagnostics)
+        )?;
+
+        Ok(())
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use insta::assert_snapshot;
+    use ruff_diagnostics::Applicability;
+
+    use crate::message::TextEmitter;
+    use crate::message::tests::{
+        capture_emitter_notebook_output, capture_emitter_output, create_diagnostics,
+        create_notebook_diagnostics, create_syntax_error_diagnostics,
+    };
+
+    #[test]
+    fn default() {
+        let mut emitter = TextEmitter::default().with_show_source(true);
+        let content = capture_emitter_output(&mut emitter, &create_diagnostics());
+
+        assert_snapshot!(content);
+    }
+
+    #[test]
+    fn fix_status() {
+        let mut emitter = TextEmitter::default()
+            .with_show_fix_status(true)
+            .with_show_source(true);
+        let content = capture_emitter_output(&mut emitter, &create_diagnostics());
+
+        assert_snapshot!(content);
+    }
+
+    #[test]
+    fn fix_status_unsafe() {
+        let mut emitter = TextEmitter::default()
+            .with_show_fix_status(true)
+            .with_show_source(true)
+            .with_fix_applicability(Applicability::Unsafe);
+        let content = capture_emitter_output(&mut emitter, &create_diagnostics());
+
+        assert_snapshot!(content);
+    }
+
+    #[test]
+    fn notebook_output() {
+        let mut emitter = TextEmitter::default()
+            .with_show_fix_status(true)
+            .with_show_source(true)
+            .with_fix_applicability(Applicability::Unsafe);
+        let (messages, notebook_indexes) = create_notebook_diagnostics();
+        let content = capture_emitter_notebook_output(&mut emitter, &messages, &notebook_indexes);
+
+        assert_snapshot!(content);
+    }
+
+    #[test]
+    fn syntax_errors() {
+        let mut emitter = TextEmitter::default().with_show_source(true);
+        let content = capture_emitter_output(&mut emitter, &create_syntax_error_diagnostics());
+
+        assert_snapshot!(content);
+    }
+}

crates/ruff_linter/src/preview.rs

@@ -200,11 +200,6 @@ pub(crate) const fn is_optional_as_none_in_union_enabled(settings: &LinterSettin
     settings.preview.is_enabled()
 }
 
-// https://github.com/astral-sh/ruff/pull/20659
-pub(crate) const fn is_future_required_preview_generics_enabled(settings: &LinterSettings) -> bool {
-    settings.preview.is_enabled()
-}
-
 // https://github.com/astral-sh/ruff/pull/18683
 pub(crate) const fn is_safe_super_call_with_parameters_fix_enabled(
     settings: &LinterSettings,
@@ -240,8 +235,3 @@ pub(crate) const fn is_a003_class_scope_shadowing_expansion_enabled(
 ) -> bool {
     settings.preview.is_enabled()
 }
-
-// https://github.com/astral-sh/ruff/pull/20200
-pub(crate) const fn is_refined_submodule_import_match_enabled(settings: &LinterSettings) -> bool {
-    settings.preview.is_enabled()
-}

crates/ruff_linter/src/rules/airflow/helpers.rs

@@ -17,13 +17,13 @@ use ruff_text_size::TextRange;
 pub(crate) enum Replacement {
     // There's no replacement or suggestion other than removal
     None,
-    // Additional information. Used when there's no direct maaping replacement.
-    Message(&'static str),
     // The attribute name of a class has been changed.
     AttrName(&'static str),
+    // Additional information. Used when there's replacement but they're not direct mapping.
+    Message(&'static str),
     // Symbols updated in Airflow 3 with replacement
     // e.g., `airflow.datasets.Dataset` to `airflow.sdk.Asset`
-    Rename {
+    AutoImport {
         module: &'static str,
         name: &'static str,
     },
@@ -37,7 +37,7 @@ pub(crate) enum Replacement {
 
 #[derive(Clone, Debug, Eq, PartialEq)]
 pub(crate) enum ProviderReplacement {
-    Rename {
+    AutoImport {
         module: &'static str,
         name: &'static str,
         provider: &'static str,

crates/ruff_linter/src/rules/airflow/rules/moved_to_provider_in_3.rs

@@ -50,7 +50,7 @@ impl Violation for Airflow3MovedToProvider<'_> {
             replacement,
         } = self;
         match replacement {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 name: _,
                 module: _,
                 provider,
@@ -70,7 +70,7 @@ impl Violation for Airflow3MovedToProvider<'_> {
     fn fix_title(&self) -> Option<String> {
         let Airflow3MovedToProvider { replacement, .. } = self;
         if let Some((module, name, provider, version)) = match &replacement {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module,
                 name,
                 provider,
@@ -125,18 +125,20 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             provider: "amazon",
             version: "1.0.0",
         },
-        ["airflow", "operators", "gcs_to_s3", "GCSToS3Operator"] => ProviderReplacement::Rename {
-            module: "airflow.providers.amazon.aws.transfers.gcs_to_s3",
-            name: "GCSToS3Operator",
-            provider: "amazon",
-            version: "1.0.0",
-        },
+        ["airflow", "operators", "gcs_to_s3", "GCSToS3Operator"] => {
+            ProviderReplacement::AutoImport {
+                module: "airflow.providers.amazon.aws.transfers.gcs_to_s3",
+                name: "GCSToS3Operator",
+                provider: "amazon",
+                version: "1.0.0",
+            }
+        }
         [
             "airflow",
             "operators",
             "google_api_to_s3_transfer",
             "GoogleApiToS3Operator" | "GoogleApiToS3Transfer",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.amazon.aws.transfers.google_api_to_s3",
             name: "GoogleApiToS3Operator",
             provider: "amazon",
@@ -147,7 +149,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "redshift_to_s3_operator",
             "RedshiftToS3Operator" | "RedshiftToS3Transfer",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.amazon.aws.transfers.redshift_to_s3",
             name: "RedshiftToS3Operator",
             provider: "amazon",
@@ -158,7 +160,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "s3_file_transform_operator",
             "S3FileTransformOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.amazon.aws.operators.s3",
             name: "S3FileTransformOperator",
             provider: "amazon",
@@ -169,13 +171,13 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "s3_to_redshift_operator",
             "S3ToRedshiftOperator" | "S3ToRedshiftTransfer",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.amazon.aws.transfers.s3_to_redshift",
             name: "S3ToRedshiftOperator",
             provider: "amazon",
             version: "1.0.0",
         },
-        ["airflow", "sensors", "s3_key_sensor", "S3KeySensor"] => ProviderReplacement::Rename {
+        ["airflow", "sensors", "s3_key_sensor", "S3KeySensor"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.amazon.aws.sensors.s3",
             name: "S3KeySensor",
             provider: "amazon",
@@ -188,20 +190,20 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "config_templates",
             "default_celery",
             "DEFAULT_CELERY_CONFIG",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.celery.executors.default_celery",
             name: "DEFAULT_CELERY_CONFIG",
             provider: "celery",
             version: "3.3.0",
         },
         ["airflow", "executors", "celery_executor", rest] => match *rest {
-            "app" => ProviderReplacement::Rename {
+            "app" => ProviderReplacement::AutoImport {
                 module: "airflow.providers.celery.executors.celery_executor_utils",
                 name: "app",
                 provider: "celery",
                 version: "3.3.0",
             },
-            "CeleryExecutor" => ProviderReplacement::Rename {
+            "CeleryExecutor" => ProviderReplacement::AutoImport {
                 module: "airflow.providers.celery.executors.celery_executor",
                 name: "CeleryExecutor",
                 provider: "celery",
@@ -214,7 +216,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "executors",
             "celery_kubernetes_executor",
             "CeleryKubernetesExecutor",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.celery.executors.celery_kubernetes_executor",
             name: "CeleryKubernetesExecutor",
             provider: "celery",
@@ -233,7 +235,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             provider: "common-sql",
             version: "1.0.0",
         },
-        ["airflow", "hooks", "dbapi_hook", "DbApiHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "dbapi_hook", "DbApiHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.common.sql.hooks.sql",
             name: "DbApiHook",
             provider: "common-sql",
@@ -250,7 +252,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "check_operator" | "druid_check_operator" | "presto_check_operator",
             "CheckOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.common.sql.operators.sql",
             name: "SQLCheckOperator",
             provider: "common-sql",
@@ -267,7 +269,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "presto_check_operator",
             "PrestoCheckOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.common.sql.operators.sql",
             name: "SQLCheckOperator",
             provider: "common-sql",
@@ -286,7 +288,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "IntervalCheckOperator",
         ]
         | ["airflow", "operators", "sql", "SQLIntervalCheckOperator"] => {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module: "airflow.providers.common.sql.operators.sql",
                 name: "SQLIntervalCheckOperator",
                 provider: "common-sql",
@@ -298,7 +300,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "presto_check_operator",
             "PrestoIntervalCheckOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.common.sql.operators.sql",
             name: "SQLIntervalCheckOperator",
             provider: "common-sql",
@@ -311,7 +313,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "SQLThresholdCheckOperator" | "ThresholdCheckOperator",
         ]
         | ["airflow", "operators", "sql", "SQLThresholdCheckOperator"] => {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module: "airflow.providers.common.sql.operators.sql",
                 name: "SQLThresholdCheckOperator",
                 provider: "common-sql",
@@ -330,18 +332,20 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "presto_check_operator",
             "ValueCheckOperator",
         ]
-        | ["airflow", "operators", "sql", "SQLValueCheckOperator"] => ProviderReplacement::Rename {
-            module: "airflow.providers.common.sql.operators.sql",
-            name: "SQLValueCheckOperator",
-            provider: "common-sql",
-            version: "1.1.0",
-        },
+        | ["airflow", "operators", "sql", "SQLValueCheckOperator"] => {
+            ProviderReplacement::AutoImport {
+                module: "airflow.providers.common.sql.operators.sql",
+                name: "SQLValueCheckOperator",
+                provider: "common-sql",
+                version: "1.1.0",
+            }
+        }
         [
             "airflow",
             "operators",
             "presto_check_operator",
             "PrestoValueCheckOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.common.sql.operators.sql",
             name: "SQLValueCheckOperator",
             provider: "common-sql",
@@ -366,12 +370,14 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             }
             _ => return,
         },
-        ["airflow", "sensors", "sql" | "sql_sensor", "SqlSensor"] => ProviderReplacement::Rename {
-            module: "airflow.providers.common.sql.sensors.sql",
-            name: "SqlSensor",
-            provider: "common-sql",
-            version: "1.0.0",
-        },
+        ["airflow", "sensors", "sql" | "sql_sensor", "SqlSensor"] => {
+            ProviderReplacement::AutoImport {
+                module: "airflow.providers.common.sql.sensors.sql",
+                name: "SqlSensor",
+                provider: "common-sql",
+                version: "1.0.0",
+            }
+        }
         ["airflow", "operators", "jdbc_operator", "JdbcOperator"]
         | ["airflow", "operators", "mssql_operator", "MsSqlOperator"]
         | ["airflow", "operators", "mysql_operator", "MySqlOperator"]
@@ -383,7 +389,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "PostgresOperator",
         ]
         | ["airflow", "operators", "sqlite_operator", "SqliteOperator"] => {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module: "airflow.providers.common.sql.operators.sql",
                 name: "SQLExecuteQueryOperator",
                 provider: "common-sql",
@@ -392,22 +398,24 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         }
 
         // apache-airflow-providers-daskexecutor
-        ["airflow", "executors", "dask_executor", "DaskExecutor"] => ProviderReplacement::Rename {
-            module: "airflow.providers.daskexecutor.executors.dask_executor",
-            name: "DaskExecutor",
-            provider: "daskexecutor",
-            version: "1.0.0",
-        },
+        ["airflow", "executors", "dask_executor", "DaskExecutor"] => {
+            ProviderReplacement::AutoImport {
+                module: "airflow.providers.daskexecutor.executors.dask_executor",
+                name: "DaskExecutor",
+                provider: "daskexecutor",
+                version: "1.0.0",
+            }
+        }
 
         // apache-airflow-providers-docker
-        ["airflow", "hooks", "docker_hook", "DockerHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "docker_hook", "DockerHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.docker.hooks.docker",
             name: "DockerHook",
             provider: "docker",
             version: "1.0.0",
         },
         ["airflow", "operators", "docker_operator", "DockerOperator"] => {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module: "airflow.providers.docker.operators.docker",
                 name: "DockerOperator",
                 provider: "docker",
@@ -432,7 +440,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "hive_to_druid",
             "HiveToDruidOperator" | "HiveToDruidTransfer",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.druid.transfers.hive_to_druid",
             name: "HiveToDruidOperator",
             provider: "apache-druid",
@@ -489,7 +497,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "fab",
             "fab_auth_manager",
             "FabAuthManager",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.fab.auth_manager.fab_auth_manager",
             name: "FabAuthManager",
             provider: "fab",
@@ -503,7 +511,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "security_manager",
             "override",
             "MAX_NUM_DATABASE_USER_SESSIONS",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.fab.auth_manager.security_manager.override",
             name: "MAX_NUM_DATABASE_USER_SESSIONS",
             provider: "fab",
@@ -523,7 +531,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "www",
             "security",
             "FabAirflowSecurityManagerOverride",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.fab.auth_manager.security_manager.override",
             name: "FabAirflowSecurityManagerOverride",
             provider: "fab",
@@ -531,18 +539,20 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-apache-hdfs
-        ["airflow", "hooks", "webhdfs_hook", "WebHDFSHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "webhdfs_hook", "WebHDFSHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hdfs.hooks.webhdfs",
             name: "WebHDFSHook",
             provider: "apache-hdfs",
             version: "1.0.0",
         },
-        ["airflow", "sensors", "web_hdfs_sensor", "WebHdfsSensor"] => ProviderReplacement::Rename {
-            module: "airflow.providers.apache.hdfs.sensors.web_hdfs",
-            name: "WebHdfsSensor",
-            provider: "apache-hdfs",
-            version: "1.0.0",
-        },
+        ["airflow", "sensors", "web_hdfs_sensor", "WebHdfsSensor"] => {
+            ProviderReplacement::AutoImport {
+                module: "airflow.providers.apache.hdfs.sensors.web_hdfs",
+                name: "WebHdfsSensor",
+                provider: "apache-hdfs",
+                version: "1.0.0",
+            }
+        }
 
         // apache-airflow-providers-apache-hive
         [
@@ -570,18 +580,20 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             provider: "apache-hive",
             version: "5.1.0",
         },
-        ["airflow", "operators", "hive_operator", "HiveOperator"] => ProviderReplacement::Rename {
-            module: "airflow.providers.apache.hive.operators.hive",
-            name: "HiveOperator",
-            provider: "apache-hive",
-            version: "1.0.0",
-        },
+        ["airflow", "operators", "hive_operator", "HiveOperator"] => {
+            ProviderReplacement::AutoImport {
+                module: "airflow.providers.apache.hive.operators.hive",
+                name: "HiveOperator",
+                provider: "apache-hive",
+                version: "1.0.0",
+            }
+        }
         [
             "airflow",
             "operators",
             "hive_stats_operator",
             "HiveStatsCollectionOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hive.operators.hive_stats",
             name: "HiveStatsCollectionOperator",
             provider: "apache-hive",
@@ -592,7 +604,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "hive_to_mysql",
             "HiveToMySqlOperator" | "HiveToMySqlTransfer",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hive.transfers.hive_to_mysql",
             name: "HiveToMySqlOperator",
             provider: "apache-hive",
@@ -603,7 +615,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "hive_to_samba_operator",
             "HiveToSambaOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hive.transfers.hive_to_samba",
             name: "HiveToSambaOperator",
             provider: "apache-hive",
@@ -614,7 +626,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "mssql_to_hive",
             "MsSqlToHiveOperator" | "MsSqlToHiveTransfer",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hive.transfers.mssql_to_hive",
             name: "MsSqlToHiveOperator",
             provider: "apache-hive",
@@ -625,7 +637,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "mysql_to_hive",
             "MySqlToHiveOperator" | "MySqlToHiveTransfer",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hive.transfers.mysql_to_hive",
             name: "MySqlToHiveOperator",
             provider: "apache-hive",
@@ -636,7 +648,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "s3_to_hive_operator",
             "S3ToHiveOperator" | "S3ToHiveTransfer",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hive.transfers.s3_to_hive",
             name: "S3ToHiveOperator",
             provider: "apache-hive",
@@ -647,7 +659,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "sensors",
             "hive_partition_sensor",
             "HivePartitionSensor",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hive.sensors.hive_partition",
             name: "HivePartitionSensor",
             provider: "apache-hive",
@@ -658,7 +670,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "sensors",
             "metastore_partition_sensor",
             "MetastorePartitionSensor",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hive.sensors.metastore_partition",
             name: "MetastorePartitionSensor",
             provider: "apache-hive",
@@ -669,7 +681,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "sensors",
             "named_hive_partition_sensor",
             "NamedHivePartitionSensor",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.hive.sensors.named_hive_partition",
             name: "NamedHivePartitionSensor",
             provider: "apache-hive",
@@ -677,7 +689,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-http
-        ["airflow", "hooks", "http_hook", "HttpHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "http_hook", "HttpHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.http.hooks.http",
             name: "HttpHook",
             provider: "http",
@@ -688,13 +700,13 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "http_operator",
             "SimpleHttpOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.http.operators.http",
             name: "HttpOperator",
             provider: "http",
             version: "5.0.0",
         },
-        ["airflow", "sensors", "http_sensor", "HttpSensor"] => ProviderReplacement::Rename {
+        ["airflow", "sensors", "http_sensor", "HttpSensor"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.http.sensors.http",
             name: "HttpSensor",
             provider: "http",
@@ -753,7 +765,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "kubernetes",
             "kubernetes_helper_functions",
             "add_pod_suffix",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.cncf.kubernetes.kubernetes_helper_functions",
             name: "add_unique_suffix",
             provider: "cncf-kubernetes",
@@ -764,7 +776,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "kubernetes",
             "kubernetes_helper_functions",
             "create_pod_id",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.cncf.kubernetes.kubernetes_helper_functions",
             name: "create_unique_id",
             provider: "cncf-kubernetes",
@@ -785,13 +797,13 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             version: "7.4.0",
         },
         ["airflow", "kubernetes", "pod", rest] => match *rest {
-            "Port" => ProviderReplacement::Rename {
+            "Port" => ProviderReplacement::AutoImport {
                 module: "kubernetes.client.models",
                 name: "V1ContainerPort",
                 provider: "cncf-kubernetes",
                 version: "7.4.0",
             },
-            "Resources" => ProviderReplacement::Rename {
+            "Resources" => ProviderReplacement::AutoImport {
                 module: "kubernetes.client.models",
                 name: "V1ResourceRequirements",
                 provider: "cncf-kubernetes",
@@ -811,19 +823,19 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
                 provider: "cncf-kubernetes",
                 version: "7.4.0",
             },
-            "PodDefaults" => ProviderReplacement::Rename {
+            "PodDefaults" => ProviderReplacement::AutoImport {
                 module: "airflow.providers.cncf.kubernetes.utils.xcom_sidecar",
                 name: "PodDefaults",
                 provider: "cncf-kubernetes",
                 version: "7.4.0",
             },
-            "PodGeneratorDeprecated" => ProviderReplacement::Rename {
+            "PodGeneratorDeprecated" => ProviderReplacement::AutoImport {
                 module: "airflow.providers.cncf.kubernetes.pod_generator",
                 name: "PodGenerator",
                 provider: "cncf-kubernetes",
                 version: "7.4.0",
             },
-            "add_pod_suffix" => ProviderReplacement::Rename {
+            "add_pod_suffix" => ProviderReplacement::AutoImport {
                 module: "airflow.providers.cncf.kubernetes.kubernetes_helper_functions",
                 name: "add_unique_suffix",
                 provider: "cncf-kubernetes",
@@ -853,7 +865,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "kubernetes",
             "pod_generator_deprecated" | "pod_launcher_deprecated",
             "PodDefaults",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.cncf.kubernetes.utils.xcom_sidecar",
             name: "PodDefaults",
             provider: "cncf-kubernetes",
@@ -864,7 +876,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "kubernetes",
             "pod_launcher_deprecated",
             "get_kube_client",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.cncf.kubernetes.kube_client",
             name: "get_kube_client",
             provider: "cncf-kubernetes",
@@ -875,7 +887,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "kubernetes",
             "pod_launcher" | "pod_launcher_deprecated",
             "PodLauncher",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.cncf.kubernetes.utils.pod_manager",
             name: "PodManager",
             provider: "cncf-kubernetes",
@@ -886,7 +898,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "kubernetes",
             "pod_launcher" | "pod_launcher_deprecated",
             "PodStatus",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: " airflow.providers.cncf.kubernetes.utils.pod_manager",
             name: "PodPhase",
             provider: "cncf-kubernetes",
@@ -897,20 +909,20 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "kubernetes",
             "pod_runtime_info_env",
             "PodRuntimeInfoEnv",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "kubernetes.client.models",
             name: "V1EnvVar",
             provider: "cncf-kubernetes",
             version: "7.4.0",
         },
         ["airflow", "kubernetes", "secret", rest] => match *rest {
-            "K8SModel" => ProviderReplacement::Rename {
+            "K8SModel" => ProviderReplacement::AutoImport {
                 module: "airflow.providers.cncf.kubernetes.k8s_model",
                 name: "K8SModel",
                 provider: "cncf-kubernetes",
                 version: "7.4.0",
             },
-            "Secret" => ProviderReplacement::Rename {
+            "Secret" => ProviderReplacement::AutoImport {
                 module: "airflow.providers.cncf.kubernetes.secret",
                 name: "Secret",
                 provider: "cncf-kubernetes",
@@ -918,21 +930,23 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             },
             _ => return,
         },
-        ["airflow", "kubernetes", "volume", "Volume"] => ProviderReplacement::Rename {
+        ["airflow", "kubernetes", "volume", "Volume"] => ProviderReplacement::AutoImport {
             module: "kubernetes.client.models",
             name: "V1Volume",
             provider: "cncf-kubernetes",
             version: "7.4.0",
         },
-        ["airflow", "kubernetes", "volume_mount", "VolumeMount"] => ProviderReplacement::Rename {
-            module: "kubernetes.client.models",
-            name: "V1VolumeMount",
-            provider: "cncf-kubernetes",
-            version: "7.4.0",
-        },
+        ["airflow", "kubernetes", "volume_mount", "VolumeMount"] => {
+            ProviderReplacement::AutoImport {
+                module: "kubernetes.client.models",
+                name: "V1VolumeMount",
+                provider: "cncf-kubernetes",
+                version: "7.4.0",
+            }
+        }
 
         // apache-airflow-providers-microsoft-mssql
-        ["airflow", "hooks", "mssql_hook", "MsSqlHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "mssql_hook", "MsSqlHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.microsoft.mssql.hooks.mssql",
             name: "MsSqlHook",
             provider: "microsoft-mssql",
@@ -940,7 +954,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-mysql
-        ["airflow", "hooks", "mysql_hook", "MySqlHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "mysql_hook", "MySqlHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.mysql.hooks.mysql",
             name: "MySqlHook",
             provider: "mysql",
@@ -951,7 +965,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "presto_to_mysql",
             "PrestoToMySqlOperator" | "PrestoToMySqlTransfer",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.mysql.transfers.presto_to_mysql",
             name: "PrestoToMySqlOperator",
             provider: "mysql",
@@ -959,7 +973,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-oracle
-        ["airflow", "hooks", "oracle_hook", "OracleHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "oracle_hook", "OracleHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.oracle.hooks.oracle",
             name: "OracleHook",
             provider: "oracle",
@@ -972,7 +986,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "papermill_operator",
             "PapermillOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.papermill.operators.papermill",
             name: "PapermillOperator",
             provider: "papermill",
@@ -980,21 +994,23 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-apache-pig
-        ["airflow", "hooks", "pig_hook", "PigCliHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "pig_hook", "PigCliHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.apache.pig.hooks.pig",
             name: "PigCliHook",
             provider: "apache-pig",
             version: "1.0.0",
         },
-        ["airflow", "operators", "pig_operator", "PigOperator"] => ProviderReplacement::Rename {
-            module: "airflow.providers.apache.pig.operators.pig",
-            name: "PigOperator",
-            provider: "apache-pig",
-            version: "1.0.0",
-        },
+        ["airflow", "operators", "pig_operator", "PigOperator"] => {
+            ProviderReplacement::AutoImport {
+                module: "airflow.providers.apache.pig.operators.pig",
+                name: "PigOperator",
+                provider: "apache-pig",
+                version: "1.0.0",
+            }
+        }
 
         // apache-airflow-providers-postgres
-        ["airflow", "hooks", "postgres_hook", "PostgresHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "postgres_hook", "PostgresHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.postgres.hooks.postgres",
             name: "PostgresHook",
             provider: "postgres",
@@ -1002,7 +1018,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-presto
-        ["airflow", "hooks", "presto_hook", "PrestoHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "presto_hook", "PrestoHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.presto.hooks.presto",
             name: "PrestoHook",
             provider: "presto",
@@ -1010,7 +1026,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-samba
-        ["airflow", "hooks", "samba_hook", "SambaHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "samba_hook", "SambaHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.samba.hooks.samba",
             name: "SambaHook",
             provider: "samba",
@@ -1018,7 +1034,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-slack
-        ["airflow", "hooks", "slack_hook", "SlackHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "slack_hook", "SlackHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.slack.hooks.slack",
             name: "SlackHook",
             provider: "slack",
@@ -1042,7 +1058,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "email_operator" | "email",
             "EmailOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.smtp.operators.smtp",
             name: "EmailOperator",
             provider: "smtp",
@@ -1050,7 +1066,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-sqlite
-        ["airflow", "hooks", "sqlite_hook", "SqliteHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "sqlite_hook", "SqliteHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.sqlite.hooks.sqlite",
             name: "SqliteHook",
             provider: "sqlite",
@@ -1058,7 +1074,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
         },
 
         // apache-airflow-providers-zendesk
-        ["airflow", "hooks", "zendesk_hook", "ZendeskHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "zendesk_hook", "ZendeskHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.zendesk.hooks.zendesk",
             name: "ZendeskHook",
             provider: "zendesk",
@@ -1077,12 +1093,14 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             provider: "standard",
             version: "0.0.3",
         },
-        ["airflow", "operators", "bash_operator", "BashOperator"] => ProviderReplacement::Rename {
-            module: "airflow.providers.standard.operators.bash",
-            name: "BashOperator",
-            provider: "standard",
-            version: "0.0.1",
-        },
+        ["airflow", "operators", "bash_operator", "BashOperator"] => {
+            ProviderReplacement::AutoImport {
+                module: "airflow.providers.standard.operators.bash",
+                name: "BashOperator",
+                provider: "standard",
+                version: "0.0.1",
+            }
+        }
         [
             "airflow",
             "operators",
@@ -1099,14 +1117,14 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "trigger_dagrun",
             "TriggerDagRunLink",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.operators.trigger_dagrun",
             name: "TriggerDagRunLink",
             provider: "standard",
             version: "0.0.2",
         },
         ["airflow", "operators", "datetime", "target_times_as_dates"] => {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module: "airflow.providers.standard.operators.datetime",
                 name: "target_times_as_dates",
                 provider: "standard",
@@ -1118,7 +1136,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "dummy" | "dummy_operator",
             "EmptyOperator" | "DummyOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.operators.empty",
             name: "EmptyOperator",
             provider: "standard",
@@ -1129,7 +1147,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "latest_only_operator",
             "LatestOnlyOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.operators.latest_only",
             name: "LatestOnlyOperator",
             provider: "standard",
@@ -1154,7 +1172,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "sensors",
             "external_task" | "external_task_sensor",
             "ExternalTaskSensorLink",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.sensors.external_task",
             name: "ExternalDagLink",
             provider: "standard",
@@ -1171,7 +1189,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             provider: "standard",
             version: "0.0.3",
         },
-        ["airflow", "sensors", "time_delta", "WaitSensor"] => ProviderReplacement::Rename {
+        ["airflow", "sensors", "time_delta", "WaitSensor"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.sensors.time_delta",
             name: "WaitSensor",
             provider: "standard",
@@ -1182,7 +1200,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
     };
 
     let (module, name) = match &replacement {
-        ProviderReplacement::Rename { module, name, .. } => (module, *name),
+        ProviderReplacement::AutoImport { module, name, .. } => (module, *name),
         ProviderReplacement::SourceModuleMovedToProvider { module, name, .. } => {
             (module, name.as_str())
         }

crates/ruff_linter/src/rules/airflow/rules/removal_in_3.rs

@@ -59,7 +59,7 @@ impl Violation for Airflow3Removal {
             Replacement::None
             | Replacement::AttrName(_)
             | Replacement::Message(_)
-            | Replacement::Rename { module: _, name: _ }
+            | Replacement::AutoImport { module: _, name: _ }
             | Replacement::SourceModuleMoved { module: _, name: _ } => {
                 format!("`{deprecated}` is removed in Airflow 3.0")
             }
@@ -72,7 +72,7 @@ impl Violation for Airflow3Removal {
             Replacement::None => None,
             Replacement::AttrName(name) => Some(format!("Use `{name}` instead")),
             Replacement::Message(message) => Some((*message).to_string()),
-            Replacement::Rename { module, name } => {
+            Replacement::AutoImport { module, name } => {
                 Some(format!("Use `{name}` from `{module}` instead."))
             }
             Replacement::SourceModuleMoved { module, name } => {
@@ -593,7 +593,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "api_connexion",
             "security",
             "requires_access_dataset",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.api_fastapi.core_api.security",
             name: "requires_access_asset",
         },
@@ -605,7 +605,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "managers",
             "base_auth_manager",
             "BaseAuthManager",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.api_fastapi.auth.managers.base_auth_manager",
             name: "BaseAuthManager",
         },
@@ -616,7 +616,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "models",
             "resource_details",
             "DatasetDetails",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.api_fastapi.auth.managers.models.resource_details",
             name: "AssetDetails",
         },
@@ -639,15 +639,15 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
 
         // airflow.datasets.manager
         ["airflow", "datasets", "manager", rest] => match *rest {
-            "DatasetManager" => Replacement::Rename {
+            "DatasetManager" => Replacement::AutoImport {
                 module: "airflow.assets.manager",
                 name: "AssetManager",
             },
-            "dataset_manager" => Replacement::Rename {
+            "dataset_manager" => Replacement::AutoImport {
                 module: "airflow.assets.manager",
                 name: "asset_manager",
             },
-            "resolve_dataset_manager" => Replacement::Rename {
+            "resolve_dataset_manager" => Replacement::AutoImport {
                 module: "airflow.assets.manager",
                 name: "resolve_asset_manager",
             },
@@ -657,24 +657,24 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
         ["airflow", "datasets", "DatasetAliasEvent"] => Replacement::None,
 
         // airflow.hooks
-        ["airflow", "hooks", "base_hook", "BaseHook"] => Replacement::Rename {
+        ["airflow", "hooks", "base_hook", "BaseHook"] => Replacement::AutoImport {
             module: "airflow.hooks.base",
             name: "BaseHook",
         },
 
         // airflow.lineage.hook
-        ["airflow", "lineage", "hook", "DatasetLineageInfo"] => Replacement::Rename {
+        ["airflow", "lineage", "hook", "DatasetLineageInfo"] => Replacement::AutoImport {
             module: "airflow.lineage.hook",
             name: "AssetLineageInfo",
         },
 
         // airflow.listeners.spec
         ["airflow", "listeners", "spec", "dataset", rest] => match *rest {
-            "on_dataset_created" => Replacement::Rename {
+            "on_dataset_created" => Replacement::AutoImport {
                 module: "airflow.listeners.spec.asset",
                 name: "on_asset_created",
             },
-            "on_dataset_changed" => Replacement::Rename {
+            "on_dataset_changed" => Replacement::AutoImport {
                 module: "airflow.listeners.spec.asset",
                 name: "on_asset_changed",
             },
@@ -683,11 +683,11 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
 
         // airflow.metrics.validators
         ["airflow", "metrics", "validators", rest] => match *rest {
-            "AllowListValidator" => Replacement::Rename {
+            "AllowListValidator" => Replacement::AutoImport {
                 module: "airflow.metrics.validators",
                 name: "PatternAllowListValidator",
             },
-            "BlockListValidator" => Replacement::Rename {
+            "BlockListValidator" => Replacement::AutoImport {
                 module: "airflow.metrics.validators",
                 name: "PatternBlockListValidator",
             },
@@ -695,7 +695,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
         },
 
         // airflow.notifications
-        ["airflow", "notifications", "basenotifier", "BaseNotifier"] => Replacement::Rename {
+        ["airflow", "notifications", "basenotifier", "BaseNotifier"] => Replacement::AutoImport {
             module: "airflow.sdk.bases.notifier",
             name: "BaseNotifier",
         },
@@ -705,23 +705,23 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             Replacement::Message("The whole `airflow.subdag` module has been removed.")
         }
         ["airflow", "operators", "postgres_operator", "Mapping"] => Replacement::None,
-        ["airflow", "operators", "python", "get_current_context"] => Replacement::Rename {
+        ["airflow", "operators", "python", "get_current_context"] => Replacement::AutoImport {
             module: "airflow.sdk",
             name: "get_current_context",
         },
 
         // airflow.secrets
-        ["airflow", "secrets", "cache", "SecretCache"] => Replacement::Rename {
+        ["airflow", "secrets", "cache", "SecretCache"] => Replacement::AutoImport {
             module: "airflow.sdk",
             name: "SecretCache",
         },
-        ["airflow", "secrets", "local_filesystem", "load_connections"] => Replacement::Rename {
+        ["airflow", "secrets", "local_filesystem", "load_connections"] => Replacement::AutoImport {
             module: "airflow.secrets.local_filesystem",
             name: "load_connections_dict",
         },
 
         // airflow.security
-        ["airflow", "security", "permissions", "RESOURCE_DATASET"] => Replacement::Rename {
+        ["airflow", "security", "permissions", "RESOURCE_DATASET"] => Replacement::AutoImport {
             module: "airflow.security.permissions",
             name: "RESOURCE_ASSET",
         },
@@ -732,7 +732,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "sensors",
             "base_sensor_operator",
             "BaseSensorOperator",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.sdk.bases.sensor",
             name: "BaseSensorOperator",
         },
@@ -743,7 +743,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "timetables",
             "simple",
             "DatasetTriggeredTimetable",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.timetables.simple",
             name: "AssetTriggeredTimetable",
         },
@@ -775,25 +775,25 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             ] => Replacement::None,
 
             // airflow.utils.file
-            ["file", "TemporaryDirectory"] => Replacement::Rename {
+            ["file", "TemporaryDirectory"] => Replacement::AutoImport {
                 module: "tempfile",
                 name: "TemporaryDirectory",
             },
             ["file", "mkdirs"] => Replacement::Message("Use `pathlib.Path({path}).mkdir` instead"),
 
             // airflow.utils.helpers
-            ["helpers", "chain"] => Replacement::Rename {
+            ["helpers", "chain"] => Replacement::AutoImport {
                 module: "airflow.sdk",
                 name: "chain",
             },
-            ["helpers", "cross_downstream"] => Replacement::Rename {
+            ["helpers", "cross_downstream"] => Replacement::AutoImport {
                 module: "airflow.sdk",
                 name: "cross_downstream",
             },
 
             // TODO: update it as SourceModuleMoved
             // airflow.utils.log.secrets_masker
-            ["log", "secrets_masker"] => Replacement::Rename {
+            ["log", "secrets_masker"] => Replacement::AutoImport {
                 module: "airflow.sdk.execution_time",
                 name: "secrets_masker",
             },
@@ -834,15 +834,15 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "s3",
             rest,
         ] => match *rest {
-            "create_dataset" => Replacement::Rename {
+            "create_dataset" => Replacement::AutoImport {
                 module: "airflow.providers.amazon.aws.assets.s3",
                 name: "create_asset",
             },
-            "convert_dataset_to_openlineage" => Replacement::Rename {
+            "convert_dataset_to_openlineage" => Replacement::AutoImport {
                 module: "airflow.providers.amazon.aws.assets.s3",
                 name: "convert_asset_to_openlineage",
             },
-            "sanitize_uri" => Replacement::Rename {
+            "sanitize_uri" => Replacement::AutoImport {
                 module: "airflow.providers.amazon.aws.assets.s3",
                 name: "sanitize_uri",
             },
@@ -858,7 +858,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "entities",
             "AvpEntities",
             "DATASET",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.providers.amazon.aws.auth_manager.avp.entities",
             name: "AvpEntities.ASSET",
         },
@@ -874,15 +874,15 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "file",
             rest,
         ] => match *rest {
-            "create_dataset" => Replacement::Rename {
+            "create_dataset" => Replacement::AutoImport {
                 module: "airflow.providers.common.io.assets.file",
                 name: "create_asset",
             },
-            "convert_dataset_to_openlineage" => Replacement::Rename {
+            "convert_dataset_to_openlineage" => Replacement::AutoImport {
                 module: "airflow.providers.common.io.assets.file",
                 name: "convert_asset_to_openlineage",
             },
-            "sanitize_uri" => Replacement::Rename {
+            "sanitize_uri" => Replacement::AutoImport {
                 module: "airflow.providers.common.io.assets.file",
                 name: "sanitize_uri",
             },
@@ -892,19 +892,19 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
         // airflow.providers.google
         // airflow.providers.google.datasets
         ["airflow", "providers", "google", "datasets", rest @ ..] => match &rest {
-            ["bigquery", "create_dataset"] => Replacement::Rename {
+            ["bigquery", "create_dataset"] => Replacement::AutoImport {
                 module: "airflow.providers.google.assets.bigquery",
                 name: "create_asset",
             },
-            ["gcs", "create_dataset"] => Replacement::Rename {
+            ["gcs", "create_dataset"] => Replacement::AutoImport {
                 module: "airflow.providers.google.assets.gcs",
                 name: "create_asset",
             },
-            ["gcs", "convert_dataset_to_openlineage"] => Replacement::Rename {
+            ["gcs", "convert_dataset_to_openlineage"] => Replacement::AutoImport {
                 module: "airflow.providers.google.assets.gcs",
                 name: "convert_asset_to_openlineage",
             },
-            ["gcs", "sanitize_uri"] => Replacement::Rename {
+            ["gcs", "sanitize_uri"] => Replacement::AutoImport {
                 module: "airflow.providers.google.assets.gcs",
                 name: "sanitize_uri",
             },
@@ -920,7 +920,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "datasets",
             "mysql",
             "sanitize_uri",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.providers.mysql.assets.mysql",
             name: "sanitize_uri",
         },
@@ -933,7 +933,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "datasets",
             "postgres",
             "sanitize_uri",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.providers.postgres.assets.postgres",
             name: "sanitize_uri",
         },
@@ -948,12 +948,12 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "utils",
             rest,
         ] => match *rest {
-            "DatasetInfo" => Replacement::Rename {
+            "DatasetInfo" => Replacement::AutoImport {
                 module: "airflow.providers.openlineage.utils.utils",
                 name: "AssetInfo",
             },
 
-            "translate_airflow_dataset" => Replacement::Rename {
+            "translate_airflow_dataset" => Replacement::AutoImport {
                 module: "airflow.providers.openlineage.utils.utils",
                 name: "translate_airflow_asset",
             },
@@ -968,7 +968,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "datasets",
             "trino",
             "sanitize_uri",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.providers.trino.assets.trino",
             name: "sanitize_uri",
         },
@@ -977,7 +977,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
     };
 
     let (module, name) = match &replacement {
-        Replacement::Rename { module, name } => (module, *name),
+        Replacement::AutoImport { module, name } => (module, *name),
         Replacement::SourceModuleMoved { module, name } => (module, name.as_str()),
         _ => {
             checker.report_diagnostic(

crates/ruff_linter/src/rules/airflow/rules/suggested_to_move_to_provider_in_3.rs

@@ -65,7 +65,7 @@ impl Violation for Airflow3SuggestedToMoveToProvider<'_> {
             replacement,
         } = self;
         match replacement {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 name: _,
                 module: _,
                 provider,
@@ -88,7 +88,7 @@ impl Violation for Airflow3SuggestedToMoveToProvider<'_> {
     fn fix_title(&self) -> Option<String> {
         let Airflow3SuggestedToMoveToProvider { replacement, .. } = self;
         match replacement {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module,
                 name,
                 provider,
@@ -130,32 +130,34 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
 
     let replacement = match qualified_name.segments() {
         // apache-airflow-providers-standard
-        ["airflow", "hooks", "filesystem", "FSHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "filesystem", "FSHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.hooks.filesystem",
             name: "FSHook",
             provider: "standard",
             version: "0.0.1",
         },
-        ["airflow", "hooks", "package_index", "PackageIndexHook"] => ProviderReplacement::Rename {
-            module: "airflow.providers.standard.hooks.package_index",
-            name: "PackageIndexHook",
-            provider: "standard",
-            version: "0.0.1",
-        },
-        ["airflow", "hooks", "subprocess", "SubprocessHook"] => ProviderReplacement::Rename {
+        ["airflow", "hooks", "package_index", "PackageIndexHook"] => {
+            ProviderReplacement::AutoImport {
+                module: "airflow.providers.standard.hooks.package_index",
+                name: "PackageIndexHook",
+                provider: "standard",
+                version: "0.0.1",
+            }
+        }
+        ["airflow", "hooks", "subprocess", "SubprocessHook"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.hooks.subprocess",
             name: "SubprocessHook",
             provider: "standard",
             version: "0.0.3",
         },
-        ["airflow", "operators", "bash", "BashOperator"] => ProviderReplacement::Rename {
+        ["airflow", "operators", "bash", "BashOperator"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.operators.bash",
             name: "BashOperator",
             provider: "standard",
             version: "0.0.1",
         },
         ["airflow", "operators", "datetime", "BranchDateTimeOperator"] => {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module: "airflow.providers.standard.operators.datetime",
                 name: "BranchDateTimeOperator",
                 provider: "standard",
@@ -167,20 +169,20 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             "operators",
             "trigger_dagrun",
             "TriggerDagRunOperator",
-        ] => ProviderReplacement::Rename {
+        ] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.operators.trigger_dagrun",
             name: "TriggerDagRunOperator",
             provider: "standard",
             version: "0.0.2",
         },
-        ["airflow", "operators", "empty", "EmptyOperator"] => ProviderReplacement::Rename {
+        ["airflow", "operators", "empty", "EmptyOperator"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.operators.empty",
             name: "EmptyOperator",
             provider: "standard",
             version: "0.0.2",
         },
         ["airflow", "operators", "latest_only", "LatestOnlyOperator"] => {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module: "airflow.providers.standard.operators.latest_only",
                 name: "LatestOnlyOperator",
                 provider: "standard",
@@ -202,14 +204,14 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             version: "0.0.1",
         },
         ["airflow", "operators", "weekday", "BranchDayOfWeekOperator"] => {
-            ProviderReplacement::Rename {
+            ProviderReplacement::AutoImport {
                 module: "airflow.providers.standard.operators.weekday",
                 name: "BranchDayOfWeekOperator",
                 provider: "standard",
                 version: "0.0.1",
             }
         }
-        ["airflow", "sensors", "bash", "BashSensor"] => ProviderReplacement::Rename {
+        ["airflow", "sensors", "bash", "BashSensor"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.sensor.bash",
             name: "BashSensor",
             provider: "standard",
@@ -237,13 +239,13 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             provider: "standard",
             version: "0.0.3",
         },
-        ["airflow", "sensors", "filesystem", "FileSensor"] => ProviderReplacement::Rename {
+        ["airflow", "sensors", "filesystem", "FileSensor"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.sensors.filesystem",
             name: "FileSensor",
             provider: "standard",
             version: "0.0.2",
         },
-        ["airflow", "sensors", "python", "PythonSensor"] => ProviderReplacement::Rename {
+        ["airflow", "sensors", "python", "PythonSensor"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.sensors.python",
             name: "PythonSensor",
             provider: "standard",
@@ -271,7 +273,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             provider: "standard",
             version: "0.0.1",
         },
-        ["airflow", "sensors", "weekday", "DayOfWeekSensor"] => ProviderReplacement::Rename {
+        ["airflow", "sensors", "weekday", "DayOfWeekSensor"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.sensors.weekday",
             name: "DayOfWeekSensor",
             provider: "standard",
@@ -288,7 +290,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
             provider: "standard",
             version: "0.0.3",
         },
-        ["airflow", "triggers", "file", "FileTrigger"] => ProviderReplacement::Rename {
+        ["airflow", "triggers", "file", "FileTrigger"] => ProviderReplacement::AutoImport {
             module: "airflow.providers.standard.triggers.file",
             name: "FileTrigger",
             provider: "standard",
@@ -309,7 +311,7 @@ fn check_names_moved_to_provider(checker: &Checker, expr: &Expr, ranged: TextRan
     };
 
     let (module, name) = match &replacement {
-        ProviderReplacement::Rename { module, name, .. } => (module, *name),
+        ProviderReplacement::AutoImport { module, name, .. } => (module, *name),
         ProviderReplacement::SourceModuleMovedToProvider { module, name, .. } => {
             (module, name.as_str())
         }

crates/ruff_linter/src/rules/airflow/rules/suggested_to_update_3_0.rs

@@ -55,7 +55,7 @@ impl Violation for Airflow3SuggestedUpdate {
             Replacement::None
             | Replacement::AttrName(_)
             | Replacement::Message(_)
-            | Replacement::Rename { module: _, name: _ }
+            | Replacement::AutoImport { module: _, name: _ }
             | Replacement::SourceModuleMoved { module: _, name: _ } => {
                 format!(
                     "`{deprecated}` is removed in Airflow 3.0; \
@@ -71,7 +71,7 @@ impl Violation for Airflow3SuggestedUpdate {
             Replacement::None => None,
             Replacement::AttrName(name) => Some(format!("Use `{name}` instead")),
             Replacement::Message(message) => Some((*message).to_string()),
-            Replacement::Rename { module, name } => {
+            Replacement::AutoImport { module, name } => {
                 Some(format!("Use `{name}` from `{module}` instead."))
             }
             Replacement::SourceModuleMoved { module, name } => {
@@ -191,30 +191,30 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
 
     let replacement = match qualified_name.segments() {
         // airflow.datasets.metadata
-        ["airflow", "datasets", "metadata", "Metadata"] => Replacement::Rename {
+        ["airflow", "datasets", "metadata", "Metadata"] => Replacement::AutoImport {
             module: "airflow.sdk",
             name: "Metadata",
         },
         // airflow.datasets
-        ["airflow", "Dataset"] | ["airflow", "datasets", "Dataset"] => Replacement::Rename {
+        ["airflow", "Dataset"] | ["airflow", "datasets", "Dataset"] => Replacement::AutoImport {
             module: "airflow.sdk",
             name: "Asset",
         },
         ["airflow", "datasets", rest] => match *rest {
             "DatasetAliasEvent" => Replacement::None,
-            "DatasetAlias" => Replacement::Rename {
+            "DatasetAlias" => Replacement::AutoImport {
                 module: "airflow.sdk",
                 name: "AssetAlias",
             },
-            "DatasetAll" => Replacement::Rename {
+            "DatasetAll" => Replacement::AutoImport {
                 module: "airflow.sdk",
                 name: "AssetAll",
             },
-            "DatasetAny" => Replacement::Rename {
+            "DatasetAny" => Replacement::AutoImport {
                 module: "airflow.sdk",
                 name: "AssetAny",
             },
-            "expand_alias_to_datasets" => Replacement::Rename {
+            "expand_alias_to_datasets" => Replacement::AutoImport {
                 module: "airflow.models.asset",
                 name: "expand_alias_to_assets",
             },
@@ -261,7 +261,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
                 name: (*rest).to_string(),
             }
         }
-        ["airflow", "models", "Param"] => Replacement::Rename {
+        ["airflow", "models", "Param"] => Replacement::AutoImport {
             module: "airflow.sdk.definitions.param",
             name: "Param",
         },
@@ -276,7 +276,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             module: "airflow.sdk",
             name: (*rest).to_string(),
         },
-        ["airflow", "models", "baseoperatorlink", "BaseOperatorLink"] => Replacement::Rename {
+        ["airflow", "models", "baseoperatorlink", "BaseOperatorLink"] => Replacement::AutoImport {
             module: "airflow.sdk",
             name: "BaseOperatorLink",
         },
@@ -299,7 +299,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
         },
 
         // airflow.timetables
-        ["airflow", "timetables", "datasets", "DatasetOrTimeSchedule"] => Replacement::Rename {
+        ["airflow", "timetables", "datasets", "DatasetOrTimeSchedule"] => Replacement::AutoImport {
             module: "airflow.timetables.assets",
             name: "AssetOrTimeSchedule",
         },
@@ -310,7 +310,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
             "utils",
             "dag_parsing_context",
             "get_parsing_context",
-        ] => Replacement::Rename {
+        ] => Replacement::AutoImport {
             module: "airflow.sdk",
             name: "get_parsing_context",
         },
@@ -319,7 +319,7 @@ fn check_name(checker: &Checker, expr: &Expr, range: TextRange) {
     };
 
     let (module, name) = match &replacement {
-        Replacement::Rename { module, name } => (module, *name),
+        Replacement::AutoImport { module, name } => (module, *name),
         Replacement::SourceModuleMoved { module, name } => (module, name.as_str()),
         _ => {
             checker.report_diagnostic(

crates/ruff_linter/src/rules/flake8_annotations/helpers.rs

@@ -142,25 +142,23 @@ impl AutoPythonType {
                 });
                 Some((expr, vec![no_return_edit]))
             }
-            AutoPythonType::Atom(python_type) => type_expr(python_type, checker, at),
+            AutoPythonType::Atom(python_type) => {
+                let expr = type_expr(python_type)?;
+                Some((expr, vec![]))
+            }
             AutoPythonType::Union(python_types) => {
                 if target_version >= PythonVersion::PY310 {
                     // Aggregate all the individual types (e.g., `int`, `float`).
-                    let mut all_edits = Vec::new();
                     let names = python_types
                         .iter()
                         .sorted_unstable()
-                        .map(|python_type| {
-                            let (expr, mut edits) = type_expr(*python_type, checker, at)?;
-                            all_edits.append(&mut edits);
-                            Some(expr)
-                        })
+                        .map(|python_type| type_expr(*python_type))
                         .collect::<Option<Vec<_>>>()?;
 
                     // Wrap in a bitwise union (e.g., `int | float`).
                     let expr = pep_604_union(&names);
 
-                    Some((expr, all_edits))
+                    Some((expr, vec![]))
                 } else {
                     let python_types = python_types
                         .into_iter()
@@ -169,7 +167,7 @@ impl AutoPythonType {
 
                     match python_types.as_slice() {
                         [python_type, PythonType::None] | [PythonType::None, python_type] => {
-                            let (element, mut edits) = type_expr(*python_type, checker, at)?;
+                            let element = type_expr(*python_type)?;
 
                             // Ex) `Optional[int]`
                             let (optional_edit, binding) = checker
@@ -177,18 +175,12 @@ impl AutoPythonType {
                                 .import(at)
                                 .ok()?;
                             let expr = typing_optional(element, Name::from(binding));
-                            edits.push(optional_edit);
-                            Some((expr, edits))
+                            Some((expr, vec![optional_edit]))
                         }
                         _ => {
-                            let mut all_edits = Vec::new();
                             let elements = python_types
                                 .into_iter()
-                                .map(|python_type| {
-                                    let (expr, mut edits) = type_expr(python_type, checker, at)?;
-                                    all_edits.append(&mut edits);
-                                    Some(expr)
-                                })
+                                .map(type_expr)
                                 .collect::<Option<Vec<_>>>()?;
 
                             // Ex) `Union[int, str]`
@@ -197,8 +189,7 @@ impl AutoPythonType {
                                 .import(at)
                                 .ok()?;
                             let expr = typing_union(&elements, Name::from(binding));
-                            all_edits.push(union_edit);
-                            Some((expr, all_edits))
+                            Some((expr, vec![union_edit]))
                         }
                     }
                 }
@@ -208,44 +199,26 @@ impl AutoPythonType {
 }
 
 /// Given a [`PythonType`], return an [`Expr`] that resolves to that type.
-///
-/// If the [`Expr`] relies on importing any external symbols, those imports will be returned as
-/// additional edits.
-pub(crate) fn type_expr(
-    python_type: PythonType,
-    checker: &Checker,
-    at: TextSize,
-) -> Option<(Expr, Vec<Edit>)> {
-    fn name(name: &str, checker: &Checker, at: TextSize) -> Option<(Expr, Vec<Edit>)> {
-        let (edit, binding) = checker
-            .importer()
-            .get_or_import_builtin_symbol(name, at, checker.semantic())
-            .ok()?;
-        let expr = Expr::Name(ast::ExprName {
-            id: binding.into(),
+fn type_expr(python_type: PythonType) -> Option<Expr> {
+    fn name(name: &str) -> Expr {
+        Expr::Name(ast::ExprName {
+            id: name.into(),
             range: TextRange::default(),
             node_index: ruff_python_ast::AtomicNodeIndex::NONE,
             ctx: ExprContext::Load,
-        });
-        Some((expr, edit.map_or_else(Vec::new, |edit| vec![edit])))
+        })
     }
 
     match python_type {
-        PythonType::String => name("str", checker, at),
-        PythonType::Bytes => name("bytes", checker, at),
-        PythonType::Number(number) => {
-            let symbol = match number {
-                NumberLike::Integer => "int",
-                NumberLike::Float => "float",
-                NumberLike::Complex => "complex",
-                NumberLike::Bool => "bool",
-            };
-            name(symbol, checker, at)
-        }
-        PythonType::None => {
-            let expr = Expr::NoneLiteral(ast::ExprNoneLiteral::default());
-            Some((expr, vec![]))
-        }
+        PythonType::String => Some(name("str")),
+        PythonType::Bytes => Some(name("bytes")),
+        PythonType::Number(number) => match number {
+            NumberLike::Integer => Some(name("int")),
+            NumberLike::Float => Some(name("float")),
+            NumberLike::Complex => Some(name("complex")),
+            NumberLike::Bool => Some(name("bool")),
+        },
+        PythonType::None => Some(name("None")),
         PythonType::Ellipsis => None,
         PythonType::Dict => None,
         PythonType::List => None,

crates/ruff_linter/src/rules/flake8_annotations/mod.rs

@@ -229,20 +229,4 @@ mod tests {
         assert_diagnostics!(diagnostics);
         Ok(())
     }
-
-    #[test]
-    fn shadowed_builtins() -> Result<()> {
-        let diagnostics = test_path(
-            Path::new("flake8_annotations/shadowed_builtins.py"),
-            &LinterSettings::for_rules(vec![
-                Rule::MissingReturnTypeUndocumentedPublicFunction,
-                Rule::MissingReturnTypePrivateFunction,
-                Rule::MissingReturnTypeSpecialMethod,
-                Rule::MissingReturnTypeStaticMethod,
-                Rule::MissingReturnTypeClassMethod,
-            ]),
-        )?;
-        assert_diagnostics!(diagnostics);
-        Ok(())
-    }
 }

crates/ruff_linter/src/rules/flake8_annotations/rules/definition.rs

@@ -4,14 +4,13 @@ use ruff_python_ast::identifier::Identifier;
 use ruff_python_ast::visitor::Visitor;
 use ruff_python_ast::{self as ast, Expr, Stmt};
 use ruff_python_semantic::Definition;
-use ruff_python_semantic::analyze::type_inference::{NumberLike, PythonType};
 use ruff_python_semantic::analyze::visibility;
 use ruff_python_stdlib::typing::simple_magic_return_type;
 use ruff_text_size::Ranged;
 
 use crate::checkers::ast::{Checker, DiagnosticGuard};
 use crate::registry::Rule;
-use crate::rules::flake8_annotations::helpers::{auto_return_type, type_expr};
+use crate::rules::flake8_annotations::helpers::auto_return_type;
 use crate::rules::ruff::typing::type_hint_resolves_to_any;
 use crate::{Edit, Fix, FixAvailability, Violation};
 
@@ -826,31 +825,11 @@ pub(crate) fn definition(
                     },
                     function.identifier(),
                 );
-                if let Some(return_type_str) = return_type {
-                    // Convert the simple return type to a proper expression that handles shadowed builtins
-                    let python_type = match return_type_str {
-                        "str" => PythonType::String,
-                        "bytes" => PythonType::Bytes,
-                        "int" => PythonType::Number(NumberLike::Integer),
-                        "float" => PythonType::Number(NumberLike::Float),
-                        "complex" => PythonType::Number(NumberLike::Complex),
-                        "bool" => PythonType::Number(NumberLike::Bool),
-                        "None" => PythonType::None,
-                        _ => return, // Unknown type, skip
-                    };
-
-                    if let Some((expr, edits)) =
-                        type_expr(python_type, checker, function.parameters.start())
-                    {
-                        let return_type_expr = checker.generator().expr(&expr);
-                        diagnostic.set_fix(Fix::unsafe_edits(
-                            Edit::insertion(
-                                format!(" -> {return_type_expr}"),
-                                function.parameters.end(),
-                            ),
-                            edits,
-                        ));
-                    }
+                if let Some(return_type) = return_type {
+                    diagnostic.set_fix(Fix::unsafe_edit(Edit::insertion(
+                        format!(" -> {return_type}"),
+                        function.parameters.end(),
+                    )));
                 }
                 diagnostics.push(diagnostic);
             }

crates/ruff_linter/src/rules/flake8_annotations/snapshots/ruff_linter__rules__flake8_annotations__tests__shadowed_builtins.snap

@@ -1,124 +0,0 @@
----
-source: crates/ruff_linter/src/rules/flake8_annotations/mod.rs
----
-ANN201 [*] Missing return type annotation for public function `foo`
- --> shadowed_builtins.py:4:5
-  |
-2 | from typing import override
-3 |
-4 | def foo():
-  |     ^^^
-5 |     return "!"
-  |
-help: Add return type annotation: `builtins.str`
-1 | from collections import UserString as str
-2 | from typing import override
-3 + import builtins
-4 | 
-  - def foo():
-5 + def foo() -> builtins.str:
-6 |     return "!"
-7 | 
-8 | def _foo():
-note: This is an unsafe fix and may change runtime behavior
-
-ANN202 [*] Missing return type annotation for private function `_foo`
- --> shadowed_builtins.py:7:5
-  |
-5 |     return "!"
-6 |
-7 | def _foo():
-  |     ^^^^
-8 |     return "!"
-  |
-help: Add return type annotation: `builtins.str`
-1  | from collections import UserString as str
-2  | from typing import override
-3  + import builtins
-4  | 
-5  | def foo():
-6  |     return "!"
-7  | 
-   - def _foo():
-8  + def _foo() -> builtins.str:
-9  |     return "!"
-10 | 
-11 | class C:
-note: This is an unsafe fix and may change runtime behavior
-
-ANN204 [*] Missing return type annotation for special method `__str__`
-  --> shadowed_builtins.py:12:9
-   |
-10 | class C:
-11 |     @override
-12 |     def __str__(self):
-   |         ^^^^^^^
-13 |         return "!"
-   |
-help: Add return type annotation: `str`
-1  | from collections import UserString as str
-2  | from typing import override
-3  + import builtins
-4  | 
-5  | def foo():
-6  |     return "!"
---------------------------------------------------------------------------------
-10 | 
-11 | class C:
-12 |     @override
-   -     def __str__(self):
-13 +     def __str__(self) -> builtins.str:
-14 |         return "!"
-15 |     
-16 |     @staticmethod
-note: This is an unsafe fix and may change runtime behavior
-
-ANN205 [*] Missing return type annotation for staticmethod `foo`
-  --> shadowed_builtins.py:16:9
-   |
-15 |     @staticmethod
-16 |     def foo():
-   |         ^^^
-17 |         return "!"
-   |
-help: Add return type annotation: `builtins.str`
-1  | from collections import UserString as str
-2  | from typing import override
-3  + import builtins
-4  | 
-5  | def foo():
-6  |     return "!"
---------------------------------------------------------------------------------
-14 |         return "!"
-15 |     
-16 |     @staticmethod
-   -     def foo():
-17 +     def foo() -> builtins.str:
-18 |         return "!"
-19 |     
-20 |     @classmethod
-note: This is an unsafe fix and may change runtime behavior
-
-ANN206 [*] Missing return type annotation for classmethod `bar`
-  --> shadowed_builtins.py:20:9
-   |
-19 |     @classmethod
-20 |     def bar(cls):
-   |         ^^^
-21 |         return "!"
-   |
-help: Add return type annotation: `builtins.str`
-1  | from collections import UserString as str
-2  | from typing import override
-3  + import builtins
-4  | 
-5  | def foo():
-6  |     return "!"
---------------------------------------------------------------------------------
-18 |         return "!"
-19 |     
-20 |     @classmethod
-   -     def bar(cls):
-21 +     def bar(cls) -> builtins.str:
-22 |         return "!"
-note: This is an unsafe fix and may change runtime behavior

crates/ruff_linter/src/rules/flake8_future_annotations/mod.rs

@@ -9,7 +9,6 @@ mod tests {
     use test_case::test_case;
 
     use crate::registry::Rule;
-    use crate::settings::types::PreviewMode;
     use crate::test::test_path;
     use crate::{assert_diagnostics, settings};
     use ruff_python_ast::PythonVersion;
@@ -40,7 +39,6 @@ mod tests {
     }
 
     #[test_case(Path::new("no_future_import_uses_lowercase.py"))]
-    #[test_case(Path::new("no_future_import_uses_preview_generics.py"))]
     #[test_case(Path::new("no_future_import_uses_union.py"))]
     #[test_case(Path::new("no_future_import_uses_union_inner.py"))]
     #[test_case(Path::new("ok_no_types.py"))]
@@ -58,19 +56,4 @@ mod tests {
         assert_diagnostics!(snapshot, diagnostics);
         Ok(())
     }
-
-    #[test_case(Path::new("no_future_import_uses_preview_generics.py"))]
-    fn fa102_preview(path: &Path) -> Result<()> {
-        let snapshot = format!("fa102_preview_{}", path.to_string_lossy());
-        let diagnostics = test_path(
-            Path::new("flake8_future_annotations").join(path).as_path(),
-            &settings::LinterSettings {
-                unresolved_target_version: PythonVersion::PY37.into(),
-                preview: PreviewMode::Enabled,
-                ..settings::LinterSettings::for_rule(Rule::FutureRequiredTypeAnnotation)
-            },
-        )?;
-        assert_diagnostics!(snapshot, diagnostics);
-        Ok(())
-    }
 }

crates/ruff_linter/src/rules/flake8_future_annotations/snapshots/ruff_linter__rules__flake8_future_annotations__tests__fa102_no_future_import_uses_preview_generics.py.snap

@@ -1,36 +0,0 @@
----
-source: crates/ruff_linter/src/rules/flake8_future_annotations/mod.rs
----
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:42:19
-   |
-40 |     future: asyncio.Future[int],
-41 |     task: asyncio.Task[str],
-42 |     deque_object: collections.deque[int],
-   |                   ^^^^^^^^^^^^^^^^^^^^^^
-43 |     defaultdict_object: collections.defaultdict[str, int],
-44 |     ordered_dict: collections.OrderedDict[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:43:25
-   |
-41 |     task: asyncio.Task[str],
-42 |     deque_object: collections.deque[int],
-43 |     defaultdict_object: collections.defaultdict[str, int],
-   |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-44 |     ordered_dict: collections.OrderedDict[str, int],
-45 |     counter_obj: collections.Counter[str],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior

crates/ruff_linter/src/rules/flake8_future_annotations/snapshots/ruff_linter__rules__flake8_future_annotations__tests__fa102_preview_no_future_import_uses_preview_generics.py.snap

@@ -1,851 +0,0 @@
----
-source: crates/ruff_linter/src/rules/flake8_future_annotations/mod.rs
----
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:40:13
-   |
-39 | def takes_preview_generics(
-40 |     future: asyncio.Future[int],
-   |             ^^^^^^^^^^^^^^^^^^^
-41 |     task: asyncio.Task[str],
-42 |     deque_object: collections.deque[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:41:11
-   |
-39 | def takes_preview_generics(
-40 |     future: asyncio.Future[int],
-41 |     task: asyncio.Task[str],
-   |           ^^^^^^^^^^^^^^^^^
-42 |     deque_object: collections.deque[int],
-43 |     defaultdict_object: collections.defaultdict[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:42:19
-   |
-40 |     future: asyncio.Future[int],
-41 |     task: asyncio.Task[str],
-42 |     deque_object: collections.deque[int],
-   |                   ^^^^^^^^^^^^^^^^^^^^^^
-43 |     defaultdict_object: collections.defaultdict[str, int],
-44 |     ordered_dict: collections.OrderedDict[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:43:25
-   |
-41 |     task: asyncio.Task[str],
-42 |     deque_object: collections.deque[int],
-43 |     defaultdict_object: collections.defaultdict[str, int],
-   |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-44 |     ordered_dict: collections.OrderedDict[str, int],
-45 |     counter_obj: collections.Counter[str],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:44:19
-   |
-42 |     deque_object: collections.deque[int],
-43 |     defaultdict_object: collections.defaultdict[str, int],
-44 |     ordered_dict: collections.OrderedDict[str, int],
-   |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-45 |     counter_obj: collections.Counter[str],
-46 |     chain_map: collections.ChainMap[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:45:18
-   |
-43 |     defaultdict_object: collections.defaultdict[str, int],
-44 |     ordered_dict: collections.OrderedDict[str, int],
-45 |     counter_obj: collections.Counter[str],
-   |                  ^^^^^^^^^^^^^^^^^^^^^^^^
-46 |     chain_map: collections.ChainMap[str, int],
-47 |     context_manager: contextlib.AbstractContextManager[str],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:46:16
-   |
-44 |     ordered_dict: collections.OrderedDict[str, int],
-45 |     counter_obj: collections.Counter[str],
-46 |     chain_map: collections.ChainMap[str, int],
-   |                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-47 |     context_manager: contextlib.AbstractContextManager[str],
-48 |     async_context_manager: contextlib.AbstractAsyncContextManager[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:47:22
-   |
-45 |     counter_obj: collections.Counter[str],
-46 |     chain_map: collections.ChainMap[str, int],
-47 |     context_manager: contextlib.AbstractContextManager[str],
-   |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-48 |     async_context_manager: contextlib.AbstractAsyncContextManager[int],
-49 |     dataclass_field: dataclasses.Field[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:48:28
-   |
-46 |     chain_map: collections.ChainMap[str, int],
-47 |     context_manager: contextlib.AbstractContextManager[str],
-48 |     async_context_manager: contextlib.AbstractAsyncContextManager[int],
-   |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-49 |     dataclass_field: dataclasses.Field[int],
-50 |     cached_prop: functools.cached_property[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:49:22
-   |
-47 |     context_manager: contextlib.AbstractContextManager[str],
-48 |     async_context_manager: contextlib.AbstractAsyncContextManager[int],
-49 |     dataclass_field: dataclasses.Field[int],
-   |                      ^^^^^^^^^^^^^^^^^^^^^^
-50 |     cached_prop: functools.cached_property[int],
-51 |     partial_method: functools.partialmethod[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:50:18
-   |
-48 |     async_context_manager: contextlib.AbstractAsyncContextManager[int],
-49 |     dataclass_field: dataclasses.Field[int],
-50 |     cached_prop: functools.cached_property[int],
-   |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-51 |     partial_method: functools.partialmethod[int],
-52 |     path_like: os.PathLike[str],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:51:21
-   |
-49 |     dataclass_field: dataclasses.Field[int],
-50 |     cached_prop: functools.cached_property[int],
-51 |     partial_method: functools.partialmethod[int],
-   |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-52 |     path_like: os.PathLike[str],
-53 |     lifo_queue: queue.LifoQueue[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:52:16
-   |
-50 |     cached_prop: functools.cached_property[int],
-51 |     partial_method: functools.partialmethod[int],
-52 |     path_like: os.PathLike[str],
-   |                ^^^^^^^^^^^^^^^^
-53 |     lifo_queue: queue.LifoQueue[int],
-54 |     regular_queue: queue.Queue[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:53:17
-   |
-51 |     partial_method: functools.partialmethod[int],
-52 |     path_like: os.PathLike[str],
-53 |     lifo_queue: queue.LifoQueue[int],
-   |                 ^^^^^^^^^^^^^^^^^^^^
-54 |     regular_queue: queue.Queue[int],
-55 |     priority_queue: queue.PriorityQueue[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:54:20
-   |
-52 |     path_like: os.PathLike[str],
-53 |     lifo_queue: queue.LifoQueue[int],
-54 |     regular_queue: queue.Queue[int],
-   |                    ^^^^^^^^^^^^^^^^
-55 |     priority_queue: queue.PriorityQueue[int],
-56 |     simple_queue: queue.SimpleQueue[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:55:21
-   |
-53 |     lifo_queue: queue.LifoQueue[int],
-54 |     regular_queue: queue.Queue[int],
-55 |     priority_queue: queue.PriorityQueue[int],
-   |                     ^^^^^^^^^^^^^^^^^^^^^^^^
-56 |     simple_queue: queue.SimpleQueue[int],
-57 |     regex_pattern: re.Pattern[str],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:56:19
-   |
-54 |     regular_queue: queue.Queue[int],
-55 |     priority_queue: queue.PriorityQueue[int],
-56 |     simple_queue: queue.SimpleQueue[int],
-   |                   ^^^^^^^^^^^^^^^^^^^^^^
-57 |     regex_pattern: re.Pattern[str],
-58 |     regex_match: re.Match[str],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:57:20
-   |
-55 |     priority_queue: queue.PriorityQueue[int],
-56 |     simple_queue: queue.SimpleQueue[int],
-57 |     regex_pattern: re.Pattern[str],
-   |                    ^^^^^^^^^^^^^^^
-58 |     regex_match: re.Match[str],
-59 |     bsd_db_shelf: shelve.BsdDbShelf[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:58:18
-   |
-56 |     simple_queue: queue.SimpleQueue[int],
-57 |     regex_pattern: re.Pattern[str],
-58 |     regex_match: re.Match[str],
-   |                  ^^^^^^^^^^^^^
-59 |     bsd_db_shelf: shelve.BsdDbShelf[str, int],
-60 |     db_filename_shelf: shelve.DbfilenameShelf[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:59:19
-   |
-57 |     regex_pattern: re.Pattern[str],
-58 |     regex_match: re.Match[str],
-59 |     bsd_db_shelf: shelve.BsdDbShelf[str, int],
-   |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^
-60 |     db_filename_shelf: shelve.DbfilenameShelf[str, int],
-61 |     shelf_obj: shelve.Shelf[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:60:24
-   |
-58 |     regex_match: re.Match[str],
-59 |     bsd_db_shelf: shelve.BsdDbShelf[str, int],
-60 |     db_filename_shelf: shelve.DbfilenameShelf[str, int],
-   |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-61 |     shelf_obj: shelve.Shelf[str, int],
-62 |     mapping_proxy: types.MappingProxyType[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:61:16
-   |
-59 |     bsd_db_shelf: shelve.BsdDbShelf[str, int],
-60 |     db_filename_shelf: shelve.DbfilenameShelf[str, int],
-61 |     shelf_obj: shelve.Shelf[str, int],
-   |                ^^^^^^^^^^^^^^^^^^^^^^
-62 |     mapping_proxy: types.MappingProxyType[str, int],
-63 |     weak_key_dict: weakref.WeakKeyDictionary[object, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:62:20
-   |
-60 |     db_filename_shelf: shelve.DbfilenameShelf[str, int],
-61 |     shelf_obj: shelve.Shelf[str, int],
-62 |     mapping_proxy: types.MappingProxyType[str, int],
-   |                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-63 |     weak_key_dict: weakref.WeakKeyDictionary[object, int],
-64 |     weak_method: weakref.WeakMethod[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:63:20
-   |
-61 |     shelf_obj: shelve.Shelf[str, int],
-62 |     mapping_proxy: types.MappingProxyType[str, int],
-63 |     weak_key_dict: weakref.WeakKeyDictionary[object, int],
-   |                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-64 |     weak_method: weakref.WeakMethod[int],
-65 |     weak_set: weakref.WeakSet[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:64:18
-   |
-62 |     mapping_proxy: types.MappingProxyType[str, int],
-63 |     weak_key_dict: weakref.WeakKeyDictionary[object, int],
-64 |     weak_method: weakref.WeakMethod[int],
-   |                  ^^^^^^^^^^^^^^^^^^^^^^^
-65 |     weak_set: weakref.WeakSet[int],
-66 |     weak_value_dict: weakref.WeakValueDictionary[object, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:65:15
-   |
-63 |     weak_key_dict: weakref.WeakKeyDictionary[object, int],
-64 |     weak_method: weakref.WeakMethod[int],
-65 |     weak_set: weakref.WeakSet[int],
-   |               ^^^^^^^^^^^^^^^^^^^^
-66 |     weak_value_dict: weakref.WeakValueDictionary[object, int],
-67 |     awaitable: Awaitable[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:66:22
-   |
-64 |     weak_method: weakref.WeakMethod[int],
-65 |     weak_set: weakref.WeakSet[int],
-66 |     weak_value_dict: weakref.WeakValueDictionary[object, int],
-   |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-67 |     awaitable: Awaitable[int],
-68 |     coroutine: Coroutine[int, None, str],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:67:16
-   |
-65 |     weak_set: weakref.WeakSet[int],
-66 |     weak_value_dict: weakref.WeakValueDictionary[object, int],
-67 |     awaitable: Awaitable[int],
-   |                ^^^^^^^^^^^^^^
-68 |     coroutine: Coroutine[int, None, str],
-69 |     async_iterable: AsyncIterable[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:68:16
-   |
-66 |     weak_value_dict: weakref.WeakValueDictionary[object, int],
-67 |     awaitable: Awaitable[int],
-68 |     coroutine: Coroutine[int, None, str],
-   |                ^^^^^^^^^^^^^^^^^^^^^^^^^
-69 |     async_iterable: AsyncIterable[int],
-70 |     async_iterator: AsyncIterator[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:69:21
-   |
-67 |     awaitable: Awaitable[int],
-68 |     coroutine: Coroutine[int, None, str],
-69 |     async_iterable: AsyncIterable[int],
-   |                     ^^^^^^^^^^^^^^^^^^
-70 |     async_iterator: AsyncIterator[int],
-71 |     async_generator: AsyncGenerator[int, None],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:70:21
-   |
-68 |     coroutine: Coroutine[int, None, str],
-69 |     async_iterable: AsyncIterable[int],
-70 |     async_iterator: AsyncIterator[int],
-   |                     ^^^^^^^^^^^^^^^^^^
-71 |     async_generator: AsyncGenerator[int, None],
-72 |     iterable: Iterable[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:71:22
-   |
-69 |     async_iterable: AsyncIterable[int],
-70 |     async_iterator: AsyncIterator[int],
-71 |     async_generator: AsyncGenerator[int, None],
-   |                      ^^^^^^^^^^^^^^^^^^^^^^^^^
-72 |     iterable: Iterable[int],
-73 |     iterator: Iterator[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:72:15
-   |
-70 |     async_iterator: AsyncIterator[int],
-71 |     async_generator: AsyncGenerator[int, None],
-72 |     iterable: Iterable[int],
-   |               ^^^^^^^^^^^^^
-73 |     iterator: Iterator[int],
-74 |     generator: Generator[int, None, None],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:73:15
-   |
-71 |     async_generator: AsyncGenerator[int, None],
-72 |     iterable: Iterable[int],
-73 |     iterator: Iterator[int],
-   |               ^^^^^^^^^^^^^
-74 |     generator: Generator[int, None, None],
-75 |     reversible: Reversible[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:74:16
-   |
-72 |     iterable: Iterable[int],
-73 |     iterator: Iterator[int],
-74 |     generator: Generator[int, None, None],
-   |                ^^^^^^^^^^^^^^^^^^^^^^^^^^
-75 |     reversible: Reversible[int],
-76 |     container: Container[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:75:17
-   |
-73 |     iterator: Iterator[int],
-74 |     generator: Generator[int, None, None],
-75 |     reversible: Reversible[int],
-   |                 ^^^^^^^^^^^^^^^
-76 |     container: Container[int],
-77 |     collection: Collection[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:76:16
-   |
-74 |     generator: Generator[int, None, None],
-75 |     reversible: Reversible[int],
-76 |     container: Container[int],
-   |                ^^^^^^^^^^^^^^
-77 |     collection: Collection[int],
-78 |     callable_obj: Callable[[int], str],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:77:17
-   |
-75 |     reversible: Reversible[int],
-76 |     container: Container[int],
-77 |     collection: Collection[int],
-   |                 ^^^^^^^^^^^^^^^
-78 |     callable_obj: Callable[[int], str],
-79 |     set_obj: Set[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:78:19
-   |
-76 |     container: Container[int],
-77 |     collection: Collection[int],
-78 |     callable_obj: Callable[[int], str],
-   |                   ^^^^^^^^^^^^^^^^^^^^
-79 |     set_obj: Set[int],
-80 |     mutable_set: MutableSet[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:79:14
-   |
-77 |     collection: Collection[int],
-78 |     callable_obj: Callable[[int], str],
-79 |     set_obj: Set[int],
-   |              ^^^^^^^^
-80 |     mutable_set: MutableSet[int],
-81 |     mapping: Mapping[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:80:18
-   |
-78 |     callable_obj: Callable[[int], str],
-79 |     set_obj: Set[int],
-80 |     mutable_set: MutableSet[int],
-   |                  ^^^^^^^^^^^^^^^
-81 |     mapping: Mapping[str, int],
-82 |     mutable_mapping: MutableMapping[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:81:14
-   |
-79 |     set_obj: Set[int],
-80 |     mutable_set: MutableSet[int],
-81 |     mapping: Mapping[str, int],
-   |              ^^^^^^^^^^^^^^^^^
-82 |     mutable_mapping: MutableMapping[str, int],
-83 |     sequence: Sequence[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:82:22
-   |
-80 |     mutable_set: MutableSet[int],
-81 |     mapping: Mapping[str, int],
-82 |     mutable_mapping: MutableMapping[str, int],
-   |                      ^^^^^^^^^^^^^^^^^^^^^^^^
-83 |     sequence: Sequence[int],
-84 |     mutable_sequence: MutableSequence[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:83:15
-   |
-81 |     mapping: Mapping[str, int],
-82 |     mutable_mapping: MutableMapping[str, int],
-83 |     sequence: Sequence[int],
-   |               ^^^^^^^^^^^^^
-84 |     mutable_sequence: MutableSequence[int],
-85 |     byte_string: ByteString[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:84:23
-   |
-82 |     mutable_mapping: MutableMapping[str, int],
-83 |     sequence: Sequence[int],
-84 |     mutable_sequence: MutableSequence[int],
-   |                       ^^^^^^^^^^^^^^^^^^^^
-85 |     byte_string: ByteString[int],
-86 |     mapping_view: MappingView[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:85:18
-   |
-83 |     sequence: Sequence[int],
-84 |     mutable_sequence: MutableSequence[int],
-85 |     byte_string: ByteString[int],
-   |                  ^^^^^^^^^^^^^^^
-86 |     mapping_view: MappingView[str, int],
-87 |     keys_view: KeysView[str],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:86:19
-   |
-84 |     mutable_sequence: MutableSequence[int],
-85 |     byte_string: ByteString[int],
-86 |     mapping_view: MappingView[str, int],
-   |                   ^^^^^^^^^^^^^^^^^^^^^
-87 |     keys_view: KeysView[str],
-88 |     items_view: ItemsView[str, int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:87:16
-   |
-85 |     byte_string: ByteString[int],
-86 |     mapping_view: MappingView[str, int],
-87 |     keys_view: KeysView[str],
-   |                ^^^^^^^^^^^^^
-88 |     items_view: ItemsView[str, int],
-89 |     values_view: ValuesView[int],
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:88:17
-   |
-86 |     mapping_view: MappingView[str, int],
-87 |     keys_view: KeysView[str],
-88 |     items_view: ItemsView[str, int],
-   |                 ^^^^^^^^^^^^^^^^^^^
-89 |     values_view: ValuesView[int],
-90 | ) -> None:
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior
-
-FA102 [*] Missing `from __future__ import annotations`, but uses PEP 585 collection
-  --> no_future_import_uses_preview_generics.py:89:18
-   |
-87 |     keys_view: KeysView[str],
-88 |     items_view: ItemsView[str, int],
-89 |     values_view: ValuesView[int],
-   |                  ^^^^^^^^^^^^^^^
-90 | ) -> None:
-91 |     ...
-   |
-help: Add `from __future__ import annotations`
-1 + from __future__ import annotations
-2 | import asyncio
-3 | import collections
-4 | import contextlib
-note: This is an unsafe fix and may change runtime behavior

crates/ruff_linter/src/rules/flynt/rules/static_join_to_fstring.rs

@@ -2,7 +2,7 @@ use ast::FStringFlags;
 use itertools::Itertools;
 
 use ruff_macros::{ViolationMetadata, derive_message_formats};
-use ruff_python_ast::{self as ast, Arguments, Expr, StringFlags, str::Quote};
+use ruff_python_ast::{self as ast, Arguments, Expr, StringFlags};
 use ruff_text_size::{Ranged, TextRange};
 
 use crate::checkers::ast::Checker;
@@ -115,8 +115,6 @@ fn build_fstring(joiner: &str, joinees: &[Expr], flags: FStringFlags) -> Option<
     }
 
     let mut f_string_elements = Vec::with_capacity(joinees.len() * 2);
-    let mut has_single_quote = joiner.contains('\'');
-    let mut has_double_quote = joiner.contains('"');
     let mut first = true;
 
     for expr in joinees {
@@ -128,31 +126,14 @@ fn build_fstring(joiner: &str, joinees: &[Expr], flags: FStringFlags) -> Option<
         if !std::mem::take(&mut first) {
             f_string_elements.push(helpers::to_interpolated_string_literal_element(joiner));
         }
-        let element = helpers::to_interpolated_string_element(expr)?;
-        if let ast::InterpolatedStringElement::Literal(ast::InterpolatedStringLiteralElement {
-            value,
-            ..
-        }) = &element
-        {
-            has_single_quote |= value.contains('\'');
-            has_double_quote |= value.contains('"');
-        }
-        f_string_elements.push(element);
+        f_string_elements.push(helpers::to_interpolated_string_element(expr)?);
     }
 
-    let quote = flags.quote_style();
-    let adjusted_quote = match quote {
-        Quote::Single if has_single_quote && !has_double_quote => quote.opposite(),
-        Quote::Double if has_double_quote && !has_single_quote => quote.opposite(),
-        _ if has_double_quote && has_single_quote => return None,
-        _ => quote,
-    };
-
     let node = ast::FString {
         elements: f_string_elements.into(),
         range: TextRange::default(),
         node_index: ruff_python_ast::AtomicNodeIndex::NONE,
-        flags: flags.with_quote_style(adjusted_quote),
+        flags,
     };
     Some(node.into())
 }

crates/ruff_linter/src/rules/flynt/snapshots/ruff_linter__rules__flynt__tests__FLY002_FLY002.py.snap

@@ -153,8 +153,6 @@ FLY002 [*] Consider `f"{url}{filename}"` instead of string join
 24 | def create_file_public_url(url, filename):
 25 |     return''.join([url, filename])
    |           ^^^^^^^^^^^^^^^^^^^^^^^^
-26 |
-27 | # Regression test for: https://github.com/astral-sh/ruff/issues/19837
    |
 help: Replace with `f"{url}{filename}"`
 22 | 
@@ -162,47 +160,4 @@ help: Replace with `f"{url}{filename}"`
 24 | def create_file_public_url(url, filename):
    -     return''.join([url, filename])
 25 +     return f"{url}{filename}"
-26 | 
-27 | # Regression test for: https://github.com/astral-sh/ruff/issues/19837
-28 | nok10 = "".join((foo, '"'))
-note: This is an unsafe fix and may change runtime behavior
-
-FLY002 [*] Consider `f'{foo}"'` instead of string join
-  --> FLY002.py:28:9
-   |
-27 | # Regression test for: https://github.com/astral-sh/ruff/issues/19837
-28 | nok10 = "".join((foo, '"'))
-   |         ^^^^^^^^^^^^^^^^^^^
-29 | nok11 = ''.join((foo, "'"))
-30 | nok12 = ''.join([foo, "'", '"'])
-   |
-help: Replace with `f'{foo}"'`
-25 |     return''.join([url, filename])
-26 | 
-27 | # Regression test for: https://github.com/astral-sh/ruff/issues/19837
-   - nok10 = "".join((foo, '"'))
-28 + nok10 = f'{foo}"'
-29 | nok11 = ''.join((foo, "'"))
-30 | nok12 = ''.join([foo, "'", '"'])
-31 | nok13 = "".join([foo, "'", '"'])
-note: This is an unsafe fix and may change runtime behavior
-
-FLY002 [*] Consider `f"{foo}'"` instead of string join
-  --> FLY002.py:29:9
-   |
-27 | # Regression test for: https://github.com/astral-sh/ruff/issues/19837
-28 | nok10 = "".join((foo, '"'))
-29 | nok11 = ''.join((foo, "'"))
-   |         ^^^^^^^^^^^^^^^^^^^
-30 | nok12 = ''.join([foo, "'", '"'])
-31 | nok13 = "".join([foo, "'", '"'])
-   |
-help: Replace with `f"{foo}'"`
-26 | 
-27 | # Regression test for: https://github.com/astral-sh/ruff/issues/19837
-28 | nok10 = "".join((foo, '"'))
-   - nok11 = ''.join((foo, "'"))
-29 + nok11 = f"{foo}'"
-30 | nok12 = ''.join([foo, "'", '"'])
-31 | nok13 = "".join([foo, "'", '"'])
 note: This is an unsafe fix and may change runtime behavior

crates/ruff_linter/src/rules/pyflakes/mod.rs

@@ -29,7 +29,7 @@ mod tests {
     use crate::settings::{LinterSettings, flags};
     use crate::source_kind::SourceKind;
     use crate::test::{test_contents, test_path, test_snippet};
-    use crate::{Locator, assert_diagnostics, assert_diagnostics_diff, directives};
+    use crate::{Locator, assert_diagnostics, directives};
 
     #[test_case(Rule::UnusedImport, Path::new("F401_0.py"))]
     #[test_case(Rule::UnusedImport, Path::new("F401_1.py"))]
@@ -392,154 +392,6 @@ mod tests {
         Ok(())
     }
 
-    #[test_case(Rule::UnusedImport, Path::new("F401_0.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_1.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_2.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_3.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_4.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_5.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_6.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_7.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_8.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_9.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_10.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_11.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_12.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_13.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_14.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_15.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_16.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_17.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_18.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_19.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_20.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_21.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_22.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_23.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_32.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_34.py"))]
-    #[test_case(Rule::UnusedImport, Path::new("F401_35.py"))]
-    fn f401_preview_refined_submodule_handling_diffs(rule_code: Rule, path: &Path) -> Result<()> {
-        let snapshot = format!("preview_diff__{}", path.to_string_lossy());
-        assert_diagnostics_diff!(
-            snapshot,
-            Path::new("pyflakes").join(path).as_path(),
-            &LinterSettings::for_rule(rule_code),
-            &LinterSettings {
-                preview: PreviewMode::Enabled,
-                ..LinterSettings::for_rule(rule_code)
-            }
-        );
-        Ok(())
-    }
-
-    #[test_case(
-        r"
-        import a
-        import a.b
-        import a.c",
-        "f401_multiple_unused_submodules"
-    )]
-    #[test_case(
-        r"
-        import a
-        import a.b
-        a.foo()",
-        "f401_use_top_member"
-    )]
-    #[test_case(
-        r"
-        import a
-        import a.b
-        a.foo()
-        a.bar()",
-        "f401_use_top_member_twice"
-    )]
-    #[test_case(
-        r"
-        # reverts to stable behavior - used between imports
-        import a
-        a.foo()
-        import a.b",
-        "f401_use_top_member_before_second_import"
-    )]
-    #[test_case(
-        r"
-        # reverts to stable behavior - used between imports
-        import a
-        a.foo()
-        a = 1
-        import a.b",
-        "f401_use_top_member_and_redefine_before_second_import"
-    )]
-    #[test_case(
-        r"
-        # reverts to stable behavior - used between imports
-        import a
-        a.foo()
-        import a.b
-        a = 1",
-        "f401_use_top_member_then_import_then_redefine"
-    )]
-    #[test_case(
-        r#"
-        import a
-        import a.b
-        __all__ = ["a"]"#,
-        "f401_use_in_dunder_all"
-    )]
-    #[test_case(
-        r"
-        import a.c
-        import a.b
-        a.foo()",
-        "f401_import_submodules_but_use_top_level"
-    )]
-    #[test_case(
-        r"
-        import a.c
-        import a.b.d
-        a.foo()",
-        "f401_import_submodules_different_lengths_but_use_top_level"
-    )]
-    #[test_case(
-        r"
-        # refined logic only applied _within_ scope
-        import a
-        def foo():
-            import a.b
-            a.foo()",
-        "f401_import_submodules_in_function_scope"
-    )]
-    #[test_case(
-        r"
-        # reverts to stable behavior - used between bindings
-        import a
-        a.b
-        import a.b",
-        "f401_use_in_between_imports"
-    )]
-    #[test_case(
-        r"
-        # reverts to stable behavior - used between bindings
-        import a.b
-        a
-        import a",
-        "f401_use_in_between_imports"
-    )]
-    fn f401_preview_refined_submodule_handling(contents: &str, snapshot: &str) {
-        let diagnostics = test_contents(
-            &SourceKind::Python(dedent(contents).to_string()),
-            Path::new("f401_preview_submodule.py"),
-            &LinterSettings {
-                preview: PreviewMode::Enabled,
-                ..LinterSettings::for_rule(Rule::UnusedImport)
-            },
-        )
-        .0;
-        assert_diagnostics!(snapshot, diagnostics);
-    }
-
     #[test]
     fn f841_dummy_variable_rgx() -> Result<()> {
         let diagnostics = test_path(

crates/ruff_linter/src/rules/pyflakes/rules/unused_import.rs

@@ -5,22 +5,19 @@ use anyhow::{Result, anyhow, bail};
 use std::collections::BTreeMap;
 
 use ruff_macros::{ViolationMetadata, derive_message_formats};
-use ruff_python_ast::name::{QualifiedName, QualifiedNameBuilder};
+use ruff_python_ast::name::QualifiedName;
 use ruff_python_ast::{self as ast, Stmt};
 use ruff_python_semantic::{
-    AnyImport, Binding, BindingFlags, BindingId, BindingKind, Exceptions, Imported, NodeId, Scope,
-    ScopeId, SemanticModel, SubmoduleImport,
+    AnyImport, BindingKind, Exceptions, Imported, NodeId, Scope, ScopeId, SemanticModel,
+    SubmoduleImport,
 };
 use ruff_text_size::{Ranged, TextRange};
 
 use crate::checkers::ast::Checker;
 use crate::fix;
-use crate::preview::{
-    is_dunder_init_fix_unused_import_enabled, is_refined_submodule_import_match_enabled,
-};
+use crate::preview::is_dunder_init_fix_unused_import_enabled;
 use crate::registry::Rule;
 use crate::rules::{isort, isort::ImportSection, isort::ImportType};
-use crate::settings::LinterSettings;
 use crate::{Applicability, Fix, FixAvailability, Violation};
 
 /// ## What it does
@@ -52,43 +49,6 @@ use crate::{Applicability, Fix, FixAvailability, Violation};
 /// __all__ = ["some_module"]
 /// ```
 ///
-/// ## Preview
-/// When [preview] is enabled (and certain simplifying assumptions
-/// are met), we analyze all import statements for a given module
-/// when determining whether an import is used, rather than simply
-/// the last of these statements. This can result in both different and
-/// more import statements being marked as unused.
-///
-/// For example, if a module consists of
-///
-/// ```python
-/// import a
-/// import a.b
-/// ```
-///
-/// then both statements are marked as unused under [preview], whereas
-/// only the second is marked as unused under stable behavior.
-///
-/// As another example, if a module consists of
-///
-/// ```python
-/// import a.b
-/// import a
-///
-/// a.b.foo()
-/// ```
-///
-/// then a diagnostic will only be emitted for the first line under [preview],
-/// whereas a diagnostic would only be emitted for the second line under
-/// stable behavior.
-///
-/// Note that this behavior is somewhat subjective and is designed
-/// to conform to the developer's intuition rather than Python's actual
-/// execution. To wit, the statement `import a.b` automatically executes
-/// `import a`, so in some sense `import a` is _always_ redundant
-/// in the presence of `import a.b`.
-///
-///
 /// ## Fix safety
 ///
 /// Fixes to remove unused imports are safe, except in `__init__.py` files.
@@ -140,8 +100,6 @@ use crate::{Applicability, Fix, FixAvailability, Violation};
 /// - [Python documentation: `import`](https://docs.python.org/3/reference/simple_stmts.html#the-import-statement)
 /// - [Python documentation: `importlib.util.find_spec`](https://docs.python.org/3/library/importlib.html#importlib.util.find_spec)
 /// - [Typing documentation: interface conventions](https://typing.python.org/en/latest/spec/distributing.html#library-interface-public-and-private-symbols)
-///
-/// [preview]: https://docs.astral.sh/ruff/preview/
 #[derive(ViolationMetadata)]
 pub(crate) struct UnusedImport {
     /// Qualified name of the import
@@ -326,7 +284,17 @@ pub(crate) fn unused_import(checker: &Checker, scope: &Scope) {
     let mut unused: BTreeMap<(NodeId, Exceptions), Vec<ImportBinding>> = BTreeMap::default();
     let mut ignored: BTreeMap<(NodeId, Exceptions), Vec<ImportBinding>> = BTreeMap::default();
 
-    for binding in unused_imports_in_scope(checker.semantic(), scope, checker.settings()) {
+    for binding_id in scope.binding_ids() {
+        let binding = checker.semantic().binding(binding_id);
+
+        if binding.is_used()
+            || binding.is_explicit_export()
+            || binding.is_nonlocal()
+            || binding.is_global()
+        {
+            continue;
+        }
+
         let Some(import) = binding.as_any_import() else {
             continue;
         };
@@ -618,302 +586,3 @@ fn fix_by_reexporting<'a>(
     let isolation = Checker::isolation(checker.semantic().parent_statement_id(node_id));
     Ok(Fix::safe_edits(head, tail).isolate(isolation))
 }
-
-/// Returns an iterator over bindings to import statements that appear unused.
-///
-/// The stable behavior is to return those bindings to imports
-/// satisfying the following properties:
-///
-/// - they are not shadowed
-/// - they are not `global`, not `nonlocal`, and not explicit exports (i.e. `import foo as foo`)
-/// - they have no references, according to the semantic model
-///
-/// Under preview, there is a more refined analysis performed
-/// in the case where all bindings shadowed by a given import
-/// binding (including the binding itself) are of a simple form:
-/// they are required to be un-aliased imports or submodule imports.
-///
-/// This alternative analysis is described in the documentation for
-/// [`unused_imports_from_binding`].
-fn unused_imports_in_scope<'a, 'b>(
-    semantic: &'a SemanticModel<'b>,
-    scope: &'a Scope,
-    settings: &'a LinterSettings,
-) -> impl Iterator<Item = &'a Binding<'b>> {
-    scope
-        .binding_ids()
-        .map(|id| (id, semantic.binding(id)))
-        .filter(|(_, bdg)| {
-            matches!(
-                bdg.kind,
-                BindingKind::Import(_)
-                    | BindingKind::FromImport(_)
-                    | BindingKind::SubmoduleImport(_)
-            )
-        })
-        .filter(|(_, bdg)| !bdg.is_global() && !bdg.is_nonlocal() && !bdg.is_explicit_export())
-        .flat_map(|(id, bdg)| {
-            if is_refined_submodule_import_match_enabled(settings)
-                // No need to apply refined logic if there is only a single binding
-                && scope.shadowed_bindings(id).nth(1).is_some()
-                // Only apply the new logic in certain situations to avoid
-                // complexity, false positives, and intersection with
-                // `redefined-while-unused` (`F811`).
-                && has_simple_shadowed_bindings(scope, id, semantic)
-            {
-                unused_imports_from_binding(semantic, id, scope)
-            } else if bdg.is_used() {
-                vec![]
-            } else {
-                vec![bdg]
-            }
-        })
-}
-
-/// Returns a `Vec` of bindings to unused import statements that
-/// are shadowed by a given binding.
-///
-/// This is best explained by example. So suppose we have:
-///
-/// ```python
-/// import a
-/// import a.b
-/// import a.b.c
-///
-/// __all__ = ["a"]
-///
-/// a.b.foo()
-/// ```
-///
-/// As of 2025-09-25, Ruff's semantic model, upon visiting
-/// the whole module, will have a single live binding for
-/// the symbol `a` that points to the line `import a.b.c`,
-/// and the remaining two import bindings are considered shadowed
-/// by the last.
-///
-/// This function expects to receive the `id`
-/// for the live binding and will begin by collecting
-/// all bindings shadowed by the given one - i.e. all
-/// the different import statements binding the symbol `a`.
-/// We iterate over references to this
-/// module and decide (somewhat subjectively) which
-/// import statement the user "intends" to reference. To that end,
-/// to each reference we attempt to build a [`QualifiedName`]
-/// corresponding to an iterated attribute access (e.g. `a.b.foo`).
-/// We then determine the closest matching import statement to that
-/// qualified name, and mark it as used.
-///
-/// In the present example, the qualified name associated to the
-/// reference from the dunder all export is `"a"` and the qualified
-/// name associated to the reference in the last line is `"a.b.foo"`.
-/// The closest matches are `import a` and `import a.b`, respectively,
-/// leaving `import a.b.c` unused.
-///
-/// For a precise definition of "closest match" see [`best_match`]
-/// and [`rank_matches`].
-///
-/// Note: if any reference comes from something other than
-/// a `Name` or a dunder all expression, then we return just
-/// the original binding, thus reverting the stable behavior.
-fn unused_imports_from_binding<'a, 'b>(
-    semantic: &'a SemanticModel<'b>,
-    id: BindingId,
-    scope: &'a Scope,
-) -> Vec<&'a Binding<'b>> {
-    let mut marked = MarkedBindings::from_binding_id(semantic, id, scope);
-
-    let binding = semantic.binding(id);
-
-    // ensure we only do this once
-    let mut marked_dunder_all = false;
-
-    for ref_id in binding.references() {
-        let resolved_reference = semantic.reference(ref_id);
-        if !marked_dunder_all && resolved_reference.in_dunder_all_definition() {
-            let first = *binding
-                                .as_any_import()
-                                .expect("binding to be import binding since current function called after restricting to these in `unused_imports_in_scope`")
-                                .qualified_name()
-                                .segments().first().expect("import binding to have nonempty qualified name");
-            mark_uses_of_qualified_name(&mut marked, &QualifiedName::user_defined(first));
-            marked_dunder_all = true;
-            continue;
-        }
-        let Some(expr_id) = resolved_reference.expression_id() else {
-            // If there is some other kind of reference, abandon
-            // the refined approach for the usual one
-            return vec![binding];
-        };
-        let Some(prototype) = expand_to_qualified_name_attribute(semantic, expr_id) else {
-            return vec![binding];
-        };
-
-        mark_uses_of_qualified_name(&mut marked, &prototype);
-    }
-
-    marked.into_unused()
-}
-
-#[derive(Debug)]
-struct MarkedBindings<'a, 'b> {
-    bindings: Vec<&'a Binding<'b>>,
-    used: Vec<bool>,
-}
-
-impl<'a, 'b> MarkedBindings<'a, 'b> {
-    fn from_binding_id(semantic: &'a SemanticModel<'b>, id: BindingId, scope: &'a Scope) -> Self {
-        let bindings: Vec<_> = scope
-            .shadowed_bindings(id)
-            .map(|id| semantic.binding(id))
-            .collect();
-
-        Self {
-            used: vec![false; bindings.len()],
-            bindings,
-        }
-    }
-
-    fn into_unused(self) -> Vec<&'a Binding<'b>> {
-        self.bindings
-            .into_iter()
-            .zip(self.used)
-            .filter_map(|(bdg, is_used)| (!is_used).then_some(bdg))
-            .collect()
-    }
-
-    fn iter_mut(&mut self) -> impl Iterator<Item = (&'a Binding<'b>, &mut bool)> {
-        self.bindings.iter().copied().zip(self.used.iter_mut())
-    }
-}
-
-/// Returns `Some` [`QualifiedName`] delineating the path for the
-/// maximal [`ExprName`] or [`ExprAttribute`] containing the expression
-/// associated to the given [`NodeId`], or `None` otherwise.
-///
-/// For example, if the `expr_id` points to `a` in `a.b.c.foo()`
-/// then the qualified name would have segments [`a`, `b`, `c`, `foo`].
-fn expand_to_qualified_name_attribute<'b>(
-    semantic: &SemanticModel<'b>,
-    expr_id: NodeId,
-) -> Option<QualifiedName<'b>> {
-    let mut builder = QualifiedNameBuilder::with_capacity(16);
-
-    let mut expr_id = expr_id;
-
-    let expr = semantic.expression(expr_id)?;
-
-    let name = expr.as_name_expr()?;
-
-    builder.push(&name.id);
-
-    while let Some(node_id) = semantic.parent_expression_id(expr_id) {
-        let Some(expr) = semantic.expression(node_id) else {
-            break;
-        };
-        let Some(expr_attr) = expr.as_attribute_expr() else {
-            break;
-        };
-        builder.push(expr_attr.attr.as_str());
-        expr_id = node_id;
-    }
-    Some(builder.build())
-}
-
-fn mark_uses_of_qualified_name(marked: &mut MarkedBindings, prototype: &QualifiedName) {
-    let Some(best) = best_match(&marked.bindings, prototype) else {
-        return;
-    };
-
-    let Some(best_import) = best.as_any_import() else {
-        return;
-    };
-
-    let best_name = best_import.qualified_name();
-
-    // We loop through all bindings in case there are repeated instances
-    // of the `best_name`. For example, if we have
-    //
-    // ```python
-    // import a
-    // import a
-    //
-    // a.foo()
-    // ```
-    //
-    // then we want to mark both import statements as used. It
-    // is the job of `redefined-while-unused` (`F811`) to catch
-    // the repeated binding in this case.
-    for (binding, is_used) in marked.iter_mut() {
-        if *is_used {
-            continue;
-        }
-
-        if binding
-            .as_any_import()
-            .is_some_and(|imp| imp.qualified_name() == best_name)
-        {
-            *is_used = true;
-        }
-    }
-}
-
-/// Returns a pair with first component the length of the largest
-/// shared prefix between the qualified name of the import binding
-/// and the `prototype` and second component the length of the
-/// qualified name of the import binding (i.e. the number of path
-/// segments). Moreover, we regard the second component as ordered
-/// in reverse.
-///
-/// For example, if the binding corresponds to `import a.b.c`
-/// and the prototype to `a.b.foo()`, then the function returns
-/// `(2,std::cmp::Reverse(3))`.
-fn rank_matches(binding: &Binding, prototype: &QualifiedName) -> (usize, std::cmp::Reverse<usize>) {
-    let Some(import) = binding.as_any_import() else {
-        unreachable!()
-    };
-    let qname = import.qualified_name();
-    let left = qname
-        .segments()
-        .iter()
-        .zip(prototype.segments())
-        .take_while(|(x, y)| x == y)
-        .count();
-    (left, std::cmp::Reverse(qname.segments().len()))
-}
-
-/// Returns the import binding that shares the longest prefix
-/// with the `prototype` and is of minimal length amongst these.
-///
-/// See also [`rank_matches`].
-fn best_match<'a, 'b>(
-    bindings: &Vec<&'a Binding<'b>>,
-    prototype: &QualifiedName,
-) -> Option<&'a Binding<'b>> {
-    bindings
-        .iter()
-        .copied()
-        .max_by_key(|binding| rank_matches(binding, prototype))
-}
-
-#[inline]
-fn has_simple_shadowed_bindings(scope: &Scope, id: BindingId, semantic: &SemanticModel) -> bool {
-    scope.shadowed_bindings(id).enumerate().all(|(i, shadow)| {
-        let shadowed_binding = semantic.binding(shadow);
-        // Bail if one of the shadowed bindings is
-        // used before the last live binding. This is
-        // to avoid situations like this:
-        //
-        // ```
-        // import a
-        // a.b
-        // import a.b
-        // ```
-        if i > 0 && shadowed_binding.is_used() {
-            return false;
-        }
-        matches!(
-            shadowed_binding.kind,
-            BindingKind::Import(_) | BindingKind::SubmoduleImport(_)
-        ) && !shadowed_binding.flags.contains(BindingFlags::ALIAS)
-    })
-}

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_import_submodules_but_use_top_level.snap

@@ -1,16 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-F401 [*] `a.b` imported but unused
- --> f401_preview_submodule.py:3:8
-  |
-2 | import a.c
-3 | import a.b
-  |        ^^^
-4 | a.foo()
-  |
-help: Remove unused import: `a.b`
-1 | 
-2 | import a.c
-  - import a.b
-3 | a.foo()

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_import_submodules_different_lengths_but_use_top_level.snap

@@ -1,16 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-F401 [*] `a.b.d` imported but unused
- --> f401_preview_submodule.py:3:8
-  |
-2 | import a.c
-3 | import a.b.d
-  |        ^^^^^
-4 | a.foo()
-  |
-help: Remove unused import: `a.b.d`
-1 | 
-2 | import a.c
-  - import a.b.d
-3 | a.foo()

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_import_submodules_in_function_scope.snap

@@ -1,19 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-F401 [*] `a` imported but unused
- --> f401_preview_submodule.py:3:8
-  |
-2 | # refined logic only applied _within_ scope
-3 | import a
-  |        ^
-4 | def foo():
-5 |     import a.b
-  |
-help: Remove unused import: `a`
-1 | 
-2 | # refined logic only applied _within_ scope
-  - import a
-3 | def foo():
-4 |     import a.b
-5 |     a.foo()

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_multiple_unused_submodules.snap

@@ -1,44 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-F401 [*] `a` imported but unused
- --> f401_preview_submodule.py:2:8
-  |
-2 | import a
-  |        ^
-3 | import a.b
-4 | import a.c
-  |
-help: Remove unused import: `a`
-1 | 
-  - import a
-2 | import a.b
-3 | import a.c
-
-F401 [*] `a.b` imported but unused
- --> f401_preview_submodule.py:3:8
-  |
-2 | import a
-3 | import a.b
-  |        ^^^
-4 | import a.c
-  |
-help: Remove unused import: `a.b`
-1 | 
-2 | import a
-  - import a.b
-3 | import a.c
-
-F401 [*] `a.c` imported but unused
- --> f401_preview_submodule.py:4:8
-  |
-2 | import a
-3 | import a.b
-4 | import a.c
-  |        ^^^
-  |
-help: Remove unused import: `a.c`
-1 | 
-2 | import a
-3 | import a.b
-  - import a.c

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_use_in_between_imports.snap

@@ -1,4 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_use_in_dunder_all.snap

@@ -1,16 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-F401 [*] `a.b` imported but unused
- --> f401_preview_submodule.py:3:8
-  |
-2 | import a
-3 | import a.b
-  |        ^^^
-4 | __all__ = ["a"]
-  |
-help: Remove unused import: `a.b`
-1 | 
-2 | import a
-  - import a.b
-3 | __all__ = ["a"]

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_use_top_member.snap

@@ -1,16 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-F401 [*] `a.b` imported but unused
- --> f401_preview_submodule.py:3:8
-  |
-2 | import a
-3 | import a.b
-  |        ^^^
-4 | a.foo()
-  |
-help: Remove unused import: `a.b`
-1 | 
-2 | import a
-  - import a.b
-3 | a.foo()

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_use_top_member_and_redefine_before_second_import.snap

@@ -1,4 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_use_top_member_before_second_import.snap

@@ -1,4 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_use_top_member_then_import_then_redefine.snap

@@ -1,4 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__f401_use_top_member_twice.snap

@@ -1,18 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
-F401 [*] `a.b` imported but unused
- --> f401_preview_submodule.py:3:8
-  |
-2 | import a
-3 | import a.b
-  |        ^^^
-4 | a.foo()
-5 | a.bar()
-  |
-help: Remove unused import: `a.b`
-1 | 
-2 | import a
-  - import a.b
-3 | a.foo()
-4 | a.bar()

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__preview_diff__F401_0.py.snap

@@ -1,50 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
---- Linter settings ---
--linter.preview = disabled
-+linter.preview = enabled
-
---- Summary ---
-Removed: 0
-Added: 2
-
---- Added ---
-F401 [*] `multiprocessing.process` imported but unused
-  --> F401_0.py:10:8
-   |
- 8 | )
- 9 | import multiprocessing.pool
-10 | import multiprocessing.process
-   |        ^^^^^^^^^^^^^^^^^^^^^^^
-11 | import logging.config
-12 | import logging.handlers
-   |
-help: Remove unused import: `multiprocessing.process`
-7  |     namedtuple,
-8  | )
-9  | import multiprocessing.pool
-   - import multiprocessing.process
-10 | import logging.config
-11 | import logging.handlers
-12 | from typing import (
-
-
-F401 [*] `logging.config` imported but unused
-  --> F401_0.py:11:8
-   |
- 9 | import multiprocessing.pool
-10 | import multiprocessing.process
-11 | import logging.config
-   |        ^^^^^^^^^^^^^^
-12 | import logging.handlers
-13 | from typing import (
-   |
-help: Remove unused import: `logging.config`
-8  | )
-9  | import multiprocessing.pool
-10 | import multiprocessing.process
-   - import logging.config
-11 | import logging.handlers
-12 | from typing import (
-13 |     TYPE_CHECKING,

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__preview_diff__F401_1.py.snap

@@ -1,10 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
---- Linter settings ---
--linter.preview = disabled
-+linter.preview = enabled
-
---- Summary ---
-Removed: 0
-Added: 0

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__preview_diff__F401_10.py.snap

@@ -1,10 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
---- Linter settings ---
--linter.preview = disabled
-+linter.preview = enabled
-
---- Summary ---
-Removed: 0
-Added: 0

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__preview_diff__F401_11.py.snap

@@ -1,10 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
---- Linter settings ---
--linter.preview = disabled
-+linter.preview = enabled
-
---- Summary ---
-Removed: 0
-Added: 0

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__preview_diff__F401_12.py.snap

@@ -1,10 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
---- Linter settings ---
--linter.preview = disabled
-+linter.preview = enabled
-
---- Summary ---
-Removed: 0
-Added: 0

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__preview_diff__F401_13.py.snap

@@ -1,10 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
---- Linter settings ---
--linter.preview = disabled
-+linter.preview = enabled
-
---- Summary ---
-Removed: 0
-Added: 0

crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__preview_diff__F401_14.py.snap

@@ -1,10 +0,0 @@
----
-source: crates/ruff_linter/src/rules/pyflakes/mod.rs
----
---- Linter settings ---
--linter.preview = disabled
-+linter.preview = enabled
-
---- Summary ---
-Removed: 0
-Added: 0