Rename autofix::helpers to autofix::actions (#3866)

crates/ruff/src/autofix/actions.rs

@@ -1,3 +1,4 @@
+//! Interface for generating autofix edits from higher-level actions (e.g., "remove an argument").
 use anyhow::{bail, Result};
 use itertools::Itertools;
 use libcst_native::{
@@ -531,7 +532,7 @@ mod tests {
 
     use ruff_python_ast::source_code::Locator;
 
-    use crate::autofix::helpers::{next_stmt_break, trailing_semicolon};
+    use crate::autofix::actions::{next_stmt_break, trailing_semicolon};
 
     #[test]
     fn find_semicolon() -> Result<()> {

crates/ruff/src/autofix/mod.rs

@@ -11,7 +11,7 @@ use ruff_python_ast::types::Range;
 use crate::linter::FixTable;
 use crate::registry::{AsRule, Rule};
 
-pub mod helpers;
+pub mod actions;
 
 /// Auto-fix errors in a file, and write the fixed source code to disk.
 pub fn fix_file(diagnostics: &[Diagnostic], locator: &Locator) -> Option<(String, FixTable)> {

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

@@ -5066,7 +5066,7 @@ impl<'a> Checker<'a> {
 
                     let fix = if !in_init && !in_except_handler && self.patch(Rule::UnusedImport) {
                         let deleted: Vec<&Stmt> = self.deletions.iter().map(Into::into).collect();
-                        match autofix::helpers::remove_unused_imports(
+                        match autofix::actions::remove_unused_imports(
                             unused_imports.iter().map(|(full_name, _)| *full_name),
                             child,
                             parent,

crates/ruff/src/rules/flake8_pie/rules.rs

@@ -16,7 +16,7 @@ use ruff_python_ast::helpers::{any_over_expr, create_expr, match_trailing_commen
 use ruff_python_ast::types::{Range, RefEquality};
 use ruff_python_stdlib::identifiers::is_identifier;
 
-use crate::autofix::helpers::delete_stmt;
+use crate::autofix::actions::delete_stmt;
 use crate::checkers::ast::Checker;
 use crate::message::Location;
 use crate::registry::AsRule;

crates/ruff/src/rules/flake8_pyi/rules/pass_in_class_body.rs

@@ -1,4 +1,4 @@
-use crate::autofix::helpers::delete_stmt;
+use crate::autofix::actions::delete_stmt;
 use log::error;
 use ruff_diagnostics::{AlwaysAutofixableViolation, Diagnostic};
 use ruff_macros::{derive_message_formats, violation};

crates/ruff/src/rules/flake8_pytest_style/rules/fixture.rs

@@ -11,7 +11,7 @@ use ruff_python_ast::types::Range;
 use ruff_python_ast::visitor;
 use ruff_python_ast::visitor::Visitor;
 
-use crate::autofix::helpers::remove_argument;
+use crate::autofix::actions::remove_argument;
 use crate::checkers::ast::Checker;
 use crate::registry::{AsRule, Rule};
 

crates/ruff/src/rules/flake8_type_checking/rules/empty_type_checking_block.rs

@@ -5,7 +5,7 @@ use ruff_diagnostics::{AlwaysAutofixableViolation, Diagnostic};
 use ruff_macros::{derive_message_formats, violation};
 use ruff_python_ast::types::{Range, RefEquality};
 
-use crate::autofix::helpers::delete_stmt;
+use crate::autofix::actions::delete_stmt;
 use crate::checkers::ast::Checker;
 use crate::registry::AsRule;
 

crates/ruff/src/rules/pandas_vet/fixes.rs

@@ -3,7 +3,7 @@ use rustpython_parser::ast::{Expr, ExprKind, Keyword, Location};
 use ruff_diagnostics::{Edit, Fix};
 use ruff_python_ast::source_code::Locator;
 
-use crate::autofix::helpers::remove_argument;
+use crate::autofix::actions::remove_argument;
 
 fn match_name(expr: &Expr) -> Option<&str> {
     if let ExprKind::Call { func, .. } = &expr.node {

crates/ruff/src/rules/pyflakes/rules/unused_variable.rs

@@ -10,7 +10,7 @@ use ruff_python_ast::scope::{ScopeId, ScopeKind};
 use ruff_python_ast::source_code::Locator;
 use ruff_python_ast::types::{Range, RefEquality};
 
-use crate::autofix::helpers::delete_stmt;
+use crate::autofix::actions::delete_stmt;
 use crate::checkers::ast::Checker;
 use crate::registry::AsRule;
 

crates/ruff/src/rules/pylint/rules/sys_exit_alias.rs

@@ -4,7 +4,7 @@ use ruff_diagnostics::{AutofixKind, Diagnostic, Edit, Fix, Violation};
 use ruff_macros::{derive_message_formats, violation};
 use ruff_python_ast::types::Range;
 
-use crate::autofix::helpers::get_or_import_symbol;
+use crate::autofix::actions::get_or_import_symbol;
 use crate::checkers::ast::Checker;
 use crate::registry::AsRule;
 

crates/ruff/src/rules/pylint/rules/useless_return.rs

@@ -7,7 +7,7 @@ use ruff_python_ast::helpers::{is_const_none, ReturnStatementVisitor};
 use ruff_python_ast::types::{Range, RefEquality};
 use ruff_python_ast::visitor::Visitor;
 
-use crate::autofix::helpers::delete_stmt;
+use crate::autofix::actions::delete_stmt;
 use crate::checkers::ast::Checker;
 use crate::registry::AsRule;
 

crates/ruff/src/rules/pyupgrade/rules/lru_cache_with_maxsize_none.rs

@@ -4,7 +4,7 @@ use ruff_diagnostics::{AlwaysAutofixableViolation, Diagnostic, Edit, Fix};
 use ruff_macros::{derive_message_formats, violation};
 use ruff_python_ast::types::Range;
 
-use crate::autofix::helpers::get_or_import_symbol;
+use crate::autofix::actions::get_or_import_symbol;
 use crate::checkers::ast::Checker;
 use crate::registry::AsRule;
 

crates/ruff/src/rules/pyupgrade/rules/outdated_version_block.rs

@@ -11,7 +11,7 @@ use ruff_python_ast::source_code::Locator;
 use ruff_python_ast::types::{Range, RefEquality};
 use ruff_python_ast::whitespace::indentation;
 
-use crate::autofix::helpers::delete_stmt;
+use crate::autofix::actions::delete_stmt;
 use crate::checkers::ast::Checker;
 use crate::registry::AsRule;
 use crate::rules::pyupgrade::fixes::adjust_indentation;

crates/ruff/src/rules/pyupgrade/rules/replace_stdout_stderr.rs

@@ -7,7 +7,7 @@ use ruff_python_ast::helpers::find_keyword;
 use ruff_python_ast::source_code::Locator;
 use ruff_python_ast::types::Range;
 
-use crate::autofix::helpers::remove_argument;
+use crate::autofix::actions::remove_argument;
 use crate::checkers::ast::Checker;
 use crate::registry::AsRule;
 

crates/ruff/src/rules/pyupgrade/rules/unnecessary_builtin_import.rs

@@ -112,7 +112,7 @@ pub fn unnecessary_builtin_import(
             .iter()
             .map(|alias| format!("{module}.{}", alias.node.name))
             .collect();
-        match autofix::helpers::remove_unused_imports(
+        match autofix::actions::remove_unused_imports(
             unused_imports.iter().map(String::as_str),
             defined_by.into(),
             defined_in.map(Into::into),

crates/ruff/src/rules/pyupgrade/rules/unnecessary_encode_utf8.rs

@@ -6,7 +6,7 @@ use ruff_macros::{derive_message_formats, violation};
 use ruff_python_ast::source_code::Locator;
 use ruff_python_ast::types::Range;
 
-use crate::autofix::helpers::remove_argument;
+use crate::autofix::actions::remove_argument;
 use crate::checkers::ast::Checker;
 use crate::registry::Rule;
 

crates/ruff/src/rules/pyupgrade/rules/unnecessary_future_import.rs

@@ -92,7 +92,7 @@ pub fn unnecessary_future_import(checker: &mut Checker, stmt: &Stmt, names: &[Lo
             .iter()
             .map(|alias| format!("__future__.{}", alias.node.name))
             .collect();
-        match autofix::helpers::remove_unused_imports(
+        match autofix::actions::remove_unused_imports(
             unused_imports.iter().map(String::as_str),
             defined_by.into(),
             defined_in.map(Into::into),

crates/ruff/src/rules/pyupgrade/rules/useless_metaclass_type.rs

@@ -5,7 +5,7 @@ use ruff_diagnostics::{AlwaysAutofixableViolation, Diagnostic};
 use ruff_macros::{derive_message_formats, violation};
 use ruff_python_ast::types::Range;
 
-use crate::autofix::helpers;
+use crate::autofix::actions;
 use crate::checkers::ast::Checker;
 use crate::registry::AsRule;
 
@@ -52,7 +52,7 @@ pub fn useless_metaclass_type(checker: &mut Checker, stmt: &Stmt, value: &Expr,
         let deleted: Vec<&Stmt> = checker.deletions.iter().map(Into::into).collect();
         let defined_by = checker.ctx.current_stmt();
         let defined_in = checker.ctx.current_stmt_parent();
-        match helpers::delete_stmt(
+        match actions::delete_stmt(
             defined_by.into(),
             defined_in.map(Into::into),
             &deleted,

crates/ruff/src/rules/pyupgrade/rules/useless_object_inheritance.rs

@@ -6,7 +6,7 @@ use ruff_python_ast::binding::{Binding, BindingKind, Bindings};
 use ruff_python_ast::scope::Scope;
 use ruff_python_ast::types::Range;
 
-use crate::autofix::helpers::remove_argument;
+use crate::autofix::actions::remove_argument;
 use crate::checkers::ast::Checker;
 use crate::registry::AsRule;