openrewrite-migrate-python 0.2.0.dev20260304105122__tar.gz → 0.3.0__tar.gz

{openrewrite_migrate_python-0.2.0.dev20260304105122 → openrewrite_migrate_python-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openrewrite-migrate-python
-Version: 0.2.0.dev20260304105122
+Version: 0.3.0
 Summary: OpenRewrite recipes for migrating Python codebases to newer Python versions.
 Author-email: "Moderne Inc." <support@moderne.io>
 License: Moderne Proprietary

{openrewrite_migrate_python-0.2.0.dev20260304105122 → openrewrite_migrate_python-0.3.0}/pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "setuptools.build_meta"
 [project]
 name = "openrewrite-migrate-python"
 description = "OpenRewrite recipes for migrating Python codebases to newer Python versions."
-version = "0.2.0.dev20260304105122"  # Updated dynamically during release
+version = "0.3.0"  # Updated dynamically during release
 authors = [{ name = "Moderne Inc.", email = "support@moderne.io" }]
 license = { text = "Moderne Proprietary" }
 readme = "README.md"

{openrewrite_migrate_python-0.2.0.dev20260304105122 → openrewrite_migrate_python-0.3.0}/src/openrewrite_migrate_python/migrate/__init__.py

@@ -32,7 +32,7 @@ from .asyncio_coroutine_to_async import MigrateAsyncioCoroutine
 from .asyncio_deprecations import FindAsyncioCoroutineDecorator
 from .calendar_deprecations import ReplaceCalendarConstants
 from .cgi_migrations import FindCgiModule, FindCgitbModule
-from .cgi_parse_deprecations import FindCgiParseQs, FindCgiParseQsl
+from .cgi_parse_deprecations import ReplaceCgiParseQs, ReplaceCgiParseQsl
 from .collections_abc_migrations import ReplaceCollectionsAbcImports
 from .configparser_deprecations import (
     ReplaceConfigparserReadfp,
@@ -42,8 +42,9 @@ from .datetime_utc import ReplaceDatetimeUtcFromTimestamp, ReplaceDatetimeUtcNow
 from .distutils_deprecations import FindDistutilsUsage
 from .distutils_migrations import ReplaceDistutilsVersion
 from .functools_deprecations import FindFunctoolsCmpToKey
+from .future_imports import RemoveFutureImports
 from .gettext_deprecations import ReplaceGettextDeprecations
-from .html_parser_deprecations import FindHtmlParserUnescape
+from .html_parser_deprecations import ReplaceHtmlParserUnescape
 from .imp_migrations import FindImpUsage
 from .locale_deprecations import ReplaceLocaleResetlocale
 from .locale_getdefaultlocale_deprecation import FindLocaleGetdefaultlocale
@@ -52,7 +53,7 @@ from .mailcap_migrations import FindMailcapModule
 from .nntplib_migrations import FindNntplibModule
 from .os_deprecations import FindOsPopen, FindOsSpawn
 from .pathlib_deprecations import FindPathlibLinkTo
-from .platform_deprecations import FindPlatformPopen
+from .platform_deprecations import ReplacePlatformPopen
 from .pep594_system_migrations import (
     FindCryptModule,
     FindMsilibModule,
@@ -62,15 +63,15 @@ from .pep594_system_migrations import (
 )
 from .pipes_migrations import FindPipesModule
 from .pkgutil_deprecations import ReplacePkgutilFindLoader, ReplacePkgutilGetLoader
-from .re_deprecations import FindReTemplate
+from .re_deprecations import ReplaceReTemplate
 from .removed_modules_312 import FindRemovedModules312
 from .shutil_deprecations import FindShutilRmtreeOnerror
 from .socket_deprecations import FindSocketGetFQDN
 from .ssl_deprecations import FindSslMatchHostname
 from .string_formatting import ReplacePercentFormatWithFString, ReplaceStrFormatWithFString
 from .sys_deprecations import FindSysCoroutineWrapper
-from .sys_last_deprecations import FindSysLastExcInfo
-from .tarfile_deprecations import FindTarfileFilemode
+from .sys_last_deprecations import ReplaceSysLastExcInfo
+from .tarfile_deprecations import ReplaceTarfileFilemode
 from .telnetlib_migrations import FindTelnetlibModule
 from .tempfile_deprecations import FindTempfileMktemp
 from .threading_is_alive_deprecation import ReplaceThreadIsAlive
@@ -100,7 +101,7 @@ from .upgrade_to_python314 import UpgradeToPython314
 from .urllib_deprecations import FindUrllibParseSplitFunctions, FindUrllibParseToBytes
 from .uu_migrations import FindUuModule
 from .xdrlib_migrations import FindXdrlibModule
-from .xml_deprecations import FindElementGetchildren, ReplaceElementGetiterator
+from .xml_deprecations import ReplaceElementGetchildren, ReplaceElementGetiterator
 
 __all__ = [
     # String formatting auto-fix recipes
@@ -110,15 +111,16 @@ __all__ = [
     "FindAsyncioCoroutineDecorator",
     "FindAudioopModule",
     "FindCgiModule",
-    "FindCgiParseQs",
-    "FindCgiParseQsl",
+    "ReplaceCgiParseQs",
+    "ReplaceCgiParseQsl",
     "FindCgitbModule",
     "FindChunkModule",
     "FindCryptModule",
     "FindDistutilsUsage",
-    "FindElementGetchildren",
+    "ReplaceElementGetchildren",
     "FindFunctoolsCmpToKey",
-    "FindHtmlParserUnescape",
+    "RemoveFutureImports",
+    "ReplaceHtmlParserUnescape",
     "FindImghdrModule",
     "FindImpUsage",
     "FindMacpathModule",
@@ -132,18 +134,18 @@ __all__ = [
     "FindOssaudiodevModule",
     "FindPathlibLinkTo",
     "FindPipesModule",
-    "FindPlatformPopen",
-    "FindReTemplate",
+    "ReplacePlatformPopen",
+    "ReplaceReTemplate",
     "FindRemovedModules312",
     "FindShutilRmtreeOnerror",
     "FindSndhdrModule",
     "FindSocketGetFQDN",
     "FindSpwdModule",
     "FindSslMatchHostname",
-    "FindSysLastExcInfo",
+    "ReplaceSysLastExcInfo",
     "FindSunauModule",
     "FindSysCoroutineWrapper",
-    "FindTarfileFilemode",
+    "ReplaceTarfileFilemode",
     "FindTelnetlibModule",
     "FindTempfileMktemp",
     "ReplaceThreadGetName",

{openrewrite_migrate_python-0.2.0.dev20260304105122 → openrewrite_migrate_python-0.3.0}/src/openrewrite_migrate_python/migrate/cgi_parse_deprecations.py

@@ -1,5 +1,6 @@
 """
-Recipes for migrating deprecated cgi module parse functions.
+Recipes for replacing deprecated cgi.parse_qs/parse_qsl with
+urllib.parse.parse_qs/parse_qsl.
 
 The following functions were deprecated in Python 3.2 and removed in Python 3.8:
 - cgi.parse_qs() -> urllib.parse.parse_qs()
@@ -16,9 +17,11 @@ from rewrite import ExecutionContext, Recipe, TreeVisitor
 from rewrite.category import CategoryDescriptor
 from rewrite.decorators import categorize
 from rewrite.marketplace import Python
+from rewrite.markers import Markers
 from rewrite.python.visitor import PythonVisitor
-from rewrite.java.tree import Identifier, MethodInvocation
-from ._markers import mark_deprecated
+from rewrite.java.support_types import JLeftPadded, Space
+from rewrite.java.tree import FieldAccess, Identifier, MethodInvocation
+from rewrite.utils import random_id
 
 # Define category path: Python > Migrate > Python 3.8
 _Python38 = [
@@ -28,10 +31,51 @@ _Python38 = [
 ]
 
 
+def _create_urllib_parse_select(original_select: Identifier) -> FieldAccess:
+    """Create a FieldAccess node representing 'urllib.parse'.
+
+    Preserves the prefix/whitespace from the original select.
+    """
+    parse_identifier = Identifier(
+        _id=random_id(),
+        _prefix=Space.EMPTY,
+        _markers=Markers.EMPTY,
+        _annotations=[],
+        _simple_name="parse",
+        _type=None,
+        _field_type=None,
+    )
+
+    padded_parse = JLeftPadded(
+        _before=Space.EMPTY,
+        _element=parse_identifier,
+        _markers=Markers.EMPTY,
+    )
+
+    urllib_identifier = Identifier(
+        _id=random_id(),
+        _prefix=original_select.prefix,
+        _markers=Markers.EMPTY,
+        _annotations=[],
+        _simple_name="urllib",
+        _type=None,
+        _field_type=None,
+    )
+
+    return FieldAccess(
+        _id=random_id(),
+        _prefix=Space.EMPTY,
+        _markers=Markers.EMPTY,
+        _target=urllib_identifier,
+        _name=padded_parse,
+        _type=None,
+    )
+
+
 @categorize(_Python38)
-class FindCgiParseQs(Recipe):
+class ReplaceCgiParseQs(Recipe):
     """
-    Find usages of removed `cgi.parse_qs()`.
+    Replace `cgi.parse_qs()` with `urllib.parse.parse_qs()`.
 
     `cgi.parse_qs()` was deprecated in Python 3.2 and removed in Python 3.8.
     Use `urllib.parse.parse_qs()` instead.
@@ -43,24 +87,24 @@ class FindCgiParseQs(Recipe):
 
         After:
             from urllib.parse import parse_qs
-            params = parse_qs(query_string)
-
-    Note: This is a search recipe because the import change is non-trivial.
+            params = urllib.parse.parse_qs(query_string)
     """
 
     @property
     def name(self) -> str:
-        return "org.openrewrite.python.migrate.FindCgiParseQs"
+        return "org.openrewrite.python.migrate.ReplaceCgiParseQs"
 
     @property
     def display_name(self) -> str:
-        return "Find removed `cgi.parse_qs()` usage"
+        return "Replace `cgi.parse_qs()` with `urllib.parse.parse_qs()`"
 
     @property
     def description(self) -> str:
         return (
             "`cgi.parse_qs()` was removed in Python 3.8. "
-            "Use `urllib.parse.parse_qs()` instead."
+            "Use `urllib.parse.parse_qs()` instead. "
+            "Note: this rewrites call sites but does not manage imports. "
+            "Use with `ChangeImport` in a composite recipe to update `from` imports."
         )
 
     @property
@@ -85,19 +129,19 @@ class FindCgiParseQs(Recipe):
                 if select.simple_name != "cgi":
                     return method
 
-                return mark_deprecated(
-                    method,
-                    "cgi.parse_qs() was removed in Python 3.8. "
-                    "Use urllib.parse.parse_qs() instead."
-                )
+                # Replace cgi with urllib.parse
+                new_select = _create_urllib_parse_select(select)
+                old_padded = method.padding.select
+                new_padded = old_padded.replace(_element=new_select)
+                return method.padding.replace(_select=new_padded)
 
         return Visitor()
 
 
 @categorize(_Python38)
-class FindCgiParseQsl(Recipe):
+class ReplaceCgiParseQsl(Recipe):
     """
-    Find usages of removed `cgi.parse_qsl()`.
+    Replace `cgi.parse_qsl()` with `urllib.parse.parse_qsl()`.
 
     `cgi.parse_qsl()` was deprecated in Python 3.2 and removed in Python 3.8.
     Use `urllib.parse.parse_qsl()` instead.
@@ -109,24 +153,24 @@ class FindCgiParseQsl(Recipe):
 
         After:
             from urllib.parse import parse_qsl
-            params = parse_qsl(query_string)
-
-    Note: This is a search recipe because the import change is non-trivial.
+            params = urllib.parse.parse_qsl(query_string)
     """
 
     @property
     def name(self) -> str:
-        return "org.openrewrite.python.migrate.FindCgiParseQsl"
+        return "org.openrewrite.python.migrate.ReplaceCgiParseQsl"
 
     @property
     def display_name(self) -> str:
-        return "Find removed `cgi.parse_qsl()` usage"
+        return "Replace `cgi.parse_qsl()` with `urllib.parse.parse_qsl()`"
 
     @property
     def description(self) -> str:
         return (
             "`cgi.parse_qsl()` was removed in Python 3.8. "
-            "Use `urllib.parse.parse_qsl()` instead."
+            "Use `urllib.parse.parse_qsl()` instead. "
+            "Note: this rewrites call sites but does not manage imports. "
+            "Use with `ChangeImport` in a composite recipe to update `from` imports."
         )
 
     @property
@@ -151,10 +195,10 @@ class FindCgiParseQsl(Recipe):
                 if select.simple_name != "cgi":
                     return method
 
-                return mark_deprecated(
-                    method,
-                    "cgi.parse_qsl() was removed in Python 3.8. "
-                    "Use urllib.parse.parse_qsl() instead."
-                )
+                # Replace cgi with urllib.parse
+                new_select = _create_urllib_parse_select(select)
+                old_padded = method.padding.select
+                new_padded = old_padded.replace(_element=new_select)
+                return method.padding.replace(_select=new_padded)
 
         return Visitor()

{openrewrite_migrate_python-0.2.0.dev20260304105122 → openrewrite_migrate_python-0.3.0}/src/openrewrite_migrate_python/migrate/configparser_deprecations.py

@@ -73,37 +73,20 @@ def _replace_configparser_attribute(field_access: FieldAccess, new_name: str) ->
 
 def _is_configparser_method(method: MethodInvocation, method_name: str) -> bool:
     """
-    Check if this is a method call on a ConfigParser instance.
-
-    Since we can't always determine the type of the receiver, we check:
-    1. Method name matches
-    2. Type attribution if available
-    3. Common variable names like 'config', 'parser', 'cfg', 'cp'
+    Check if this is a method call on a ConfigParser instance using type attribution.
     """
     if not isinstance(method.name, Identifier):
         return False
     if method.name.simple_name != method_name:
         return False
 
-    # Check if type info confirms this is a configparser method
-    if method.method_type and method.method_type.declaring_type:
-        dt = method.method_type.declaring_type
-        if hasattr(dt, '_fully_qualified_name'):
-            fqn = str(dt._fully_qualified_name)
-            if 'configparser' in fqn.lower() or 'ConfigParser' in fqn:
-                return True
-
-    # Fallback: Check for common ConfigParser variable names
-    select = method.select
-    if select is None:
+    if not method.method_type or not method.method_type.declaring_type:
         return False
-    if isinstance(select, Identifier):
-        name = select.simple_name.lower()
-        # Common names for ConfigParser instances
-        if name in ('config', 'parser', 'cfg', 'cp', 'conf', 'configparser', 'safe_parser'):
-            return True
-
-    return False
+    dt = method.method_type.declaring_type
+    if not hasattr(dt, '_fully_qualified_name'):
+        return False
+    fqn = str(dt._fully_qualified_name)
+    return 'configparser' in fqn.lower() or 'ConfigParser' in fqn
 
 
 def _rename_method(method: MethodInvocation, new_name: str) -> MethodInvocation:

openrewrite_migrate_python-0.3.0/src/openrewrite_migrate_python/migrate/future_imports.py

@@ -0,0 +1,153 @@
+"""
+Recipe for removing obsolete __future__ imports.
+
+In Python 3, many __future__ imports are no longer needed because their
+features are enabled by default:
+- print_function (default since Python 3.0)
+- division (default since Python 3.0)
+- unicode_literals (default since Python 3.0)
+- absolute_import (default since Python 3.0)
+- generators (default since Python 2.3)
+- nested_scopes (default since Python 2.2)
+- with_statement (default since Python 2.6)
+
+The `annotations` import (PEP 563) is kept because it still has meaning
+in Python 3.10+ (postponed evaluation of annotations).
+"""
+
+from typing import Any, List, Optional
+
+from rewrite import ExecutionContext, Recipe, TreeVisitor
+from rewrite.category import CategoryDescriptor
+from rewrite.decorators import categorize
+from rewrite.marketplace import Python
+from rewrite.python.visitor import PythonVisitor
+from rewrite.python.tree import MultiImport
+from rewrite.java.tree import FieldAccess, Identifier
+
+# Define category path: Python > Migrate
+_Migrate = [
+    *Python,
+    CategoryDescriptor(display_name="Migrate"),
+]
+
+# __future__ imports that are obsolete in Python 3 and can be safely removed
+OBSOLETE_FUTURE_IMPORTS = {
+    "print_function",
+    "division",
+    "unicode_literals",
+    "absolute_import",
+    "generators",
+    "nested_scopes",
+    "with_statement",
+}
+
+
+def _get_import_name(import_node: Any) -> Optional[str]:
+    """Extract the imported name from an import node's qualid."""
+    qualid = import_node.qualid
+    if isinstance(qualid, FieldAccess):
+        if isinstance(qualid.name, Identifier):
+            return qualid.name.simple_name
+    elif isinstance(qualid, Identifier):
+        return qualid.simple_name
+    return None
+
+
+@categorize(_Migrate)
+class RemoveFutureImports(Recipe):
+    """
+    Remove obsolete `from __future__ import ...` statements.
+
+    In Python 3, many `__future__` imports are enabled by default and
+    no longer needed. This recipe removes imports that are obsolete:
+    - `print_function` (default since 3.0)
+    - `division` (default since 3.0)
+    - `unicode_literals` (default since 3.0)
+    - `absolute_import` (default since 3.0)
+    - `generators` (default since 2.3)
+    - `nested_scopes` (default since 2.2)
+    - `with_statement` (default since 2.6)
+
+    The `annotations` import (PEP 563) is preserved.
+
+    Example:
+        Before:
+            from __future__ import print_function, division, annotations
+
+        After:
+            from __future__ import annotations
+    """
+
+    @property
+    def name(self) -> str:
+        return "org.openrewrite.python.migrate.RemoveFutureImports"
+
+    @property
+    def display_name(self) -> str:
+        return "Remove obsolete `__future__` imports"
+
+    @property
+    def description(self) -> str:
+        return (
+            "Remove `from __future__ import ...` statements for features that "
+            "are enabled by default in Python 3."
+        )
+
+    @property
+    def tags(self) -> List[str]:
+        return ["python", "migration", "future"]
+
+    def editor(self) -> TreeVisitor[Any, ExecutionContext]:
+        class Visitor(PythonVisitor[ExecutionContext]):
+            def visit_multi_import(
+                self, multi: MultiImport, p: ExecutionContext
+            ) -> Optional[MultiImport]:
+                multi = super().visit_multi_import(multi, p)
+
+                # Check if this is a 'from __future__ import ...' statement
+                from_part = multi.from_
+                if from_part is None:
+                    return multi
+                if not isinstance(from_part, Identifier):
+                    return multi
+                if from_part.simple_name != "__future__":
+                    return multi
+
+                # Collect imported names and categorize them
+                names = multi.names
+                keep_indices = []
+                for i, import_node in enumerate(names):
+                    imported_name = _get_import_name(import_node)
+                    if imported_name is not None and imported_name not in OBSOLETE_FUTURE_IMPORTS:
+                        keep_indices.append(i)
+
+                # If all names should be kept, no change
+                if len(keep_indices) == len(names):
+                    return multi
+
+                # If all names should be removed, remove the entire statement
+                if len(keep_indices) == 0:
+                    # noinspection PyTypeChecker
+                    return None
+
+                # Partial removal: keep only non-obsolete names
+                old_container = multi.padding.names
+                old_elements = list(old_container.padding.elements)
+                new_elements = [old_elements[i] for i in keep_indices]
+
+                # Fix spacing: if the first remaining element had extra prefix
+                # from being after a comma, use the prefix of the original first element
+                if new_elements and keep_indices[0] != 0:
+                    original_first = old_elements[0]
+                    first = new_elements[0]
+                    # Copy the prefix from original first element
+                    new_first_elem = first.element.replace(
+                        _prefix=original_first.element.prefix
+                    )
+                    new_elements[0] = first.replace(_element=new_first_elem)
+
+                new_container = old_container.replace(_elements=new_elements)
+                return multi.replace(_names=new_container)
+
+        return Visitor()

{openrewrite_migrate_python-0.2.0.dev20260304105122 → openrewrite_migrate_python-0.3.0}/src/openrewrite_migrate_python/migrate/gettext_deprecations.py

@@ -49,34 +49,20 @@ def _rename_method(method: MethodInvocation, new_name: str) -> MethodInvocation:
 
 def _is_gettext_method(method: MethodInvocation, method_name: str) -> bool:
     """
-    Check if this is a gettext.method_name() call.
-
-    Uses multiple strategies to identify the method:
-    1. Check method name matches
-    2. Check type attribution if available
-    3. Check simple name on select is 'gettext'
+    Check if this is a gettext.method_name() call using type attribution.
     """
     if not isinstance(method.name, Identifier):
         return False
     if method.name.simple_name != method_name:
         return False
 
-    # Check if type info confirms this is a gettext method
-    if method.method_type and method.method_type.declaring_type:
-        dt = method.method_type.declaring_type
-        if hasattr(dt, '_fully_qualified_name'):
-            fqn = str(dt._fully_qualified_name)
-            if 'gettext' in fqn:
-                return True
-
-    # Fallback: Check simple name on select
-    select = method.select
-    if select is None:
+    if not method.method_type or not method.method_type.declaring_type:
         return False
-    if isinstance(select, Identifier) and select.simple_name == "gettext":
-        return True
-
-    return False
+    dt = method.method_type.declaring_type
+    if not hasattr(dt, '_fully_qualified_name'):
+        return False
+    fqn = str(dt._fully_qualified_name)
+    return 'gettext' in fqn
 
 
 @categorize(_Python311)

{openrewrite_migrate_python-0.2.0.dev20260304105122 → openrewrite_migrate_python-0.3.0}/src/openrewrite_migrate_python/migrate/html_parser_deprecations.py

@@ -1,5 +1,5 @@
 """
-Recipe for migrating deprecated HTMLParser.unescape() method.
+Recipe for replacing deprecated HTMLParser.unescape() with html.unescape().
 
 HTMLParser.unescape() was deprecated in Python 3.4 and removed in Python 3.9.
 Use html.unescape() instead.
@@ -15,7 +15,6 @@ from rewrite.decorators import categorize
 from rewrite.marketplace import Python
 from rewrite.python.visitor import PythonVisitor
 from rewrite.java.tree import Identifier, MethodInvocation
-from ._markers import mark_deprecated
 
 # Define category path: Python > Migrate > Python 3.9
 _Python39 = [
@@ -26,9 +25,9 @@ _Python39 = [
 
 
 @categorize(_Python39)
-class FindHtmlParserUnescape(Recipe):
+class ReplaceHtmlParserUnescape(Recipe):
     """
-    Find usages of removed `HTMLParser.unescape()`.
+    Replace `HTMLParser.unescape()` with `html.unescape()`.
 
     `HTMLParser.unescape()` was deprecated in Python 3.4 and removed in Python 3.9.
     Use `html.unescape()` instead.
@@ -42,18 +41,15 @@ class FindHtmlParserUnescape(Recipe):
         After:
             import html
             text = html.unescape(html_text)
-
-    Note: This is a search recipe because the transformation requires
-    changing from an instance method to a module function.
     """
 
     @property
     def name(self) -> str:
-        return "org.openrewrite.python.migrate.FindHtmlParserUnescape"
+        return "org.openrewrite.python.migrate.ReplaceHtmlParserUnescape"
 
     @property
     def display_name(self) -> str:
-        return "Find removed `HTMLParser.unescape()` usage"
+        return "Replace `HTMLParser.unescape()` with `html.unescape()`"
 
     @property
     def description(self) -> str:
@@ -78,23 +74,32 @@ class FindHtmlParserUnescape(Recipe):
                 if method.name.simple_name != "unescape":
                     return method
 
-                # We can't easily distinguish HTMLParser.unescape from other
-                # .unescape() calls without type information, so we mark all
-                # .unescape() calls on non-module selects as potential matches
-                # when the select is not a known module name like "html"
+                # Already using html.unescape(), no change needed
                 select = method.select
                 if isinstance(select, Identifier) and select.simple_name == "html":
-                    # Already using html.unescape(), no change needed
                     return method
 
                 # Skip if no select (bare function call)
-                if method.select is None:
+                if select is None:
+                    return method
+
+                # Require type info to confirm HTMLParser
+                if not method.method_type or not method.method_type.declaring_type:
                     return method
+                dt = method.method_type.declaring_type
+                if not hasattr(dt, '_fully_qualified_name'):
+                    return method
+                fqn = str(dt._fully_qualified_name)
+                if 'html.parser' not in fqn:
+                    return method
+
+                # Replace select with html identifier
+                if isinstance(select, Identifier):
+                    new_select = select.replace(_simple_name="html")
+                    old_padded = method.padding.select
+                    new_padded = old_padded.replace(_element=new_select)
+                    return method.padding.replace(_select=new_padded)
 
-                return mark_deprecated(
-                    method,
-                    "HTMLParser.unescape() was removed in Python 3.9. "
-                    "Use html.unescape() instead."
-                )
+                return method
 
         return Visitor()

{openrewrite_migrate_python-0.2.0.dev20260304105122 → openrewrite_migrate_python-0.3.0}/src/openrewrite_migrate_python/migrate/locale_deprecations.py

@@ -29,35 +29,20 @@ _Python311 = [
 
 def _is_locale_method(method: MethodInvocation, method_name: str) -> bool:
     """
-    Check if this is a locale.method_name() call.
-
-    Uses multiple strategies to identify the method:
-    1. Check method name matches
-    2. Check type attribution if available (FQN contains 'locale')
-    3. Check simple name on select is 'locale'
+    Check if this is a locale.method_name() call using type attribution.
     """
-    # Check method name first
     if not isinstance(method.name, Identifier):
         return False
     if method.name.simple_name != method_name:
         return False
 
-    # Check if type info confirms this is a locale method
-    if method.method_type and method.method_type.declaring_type:
-        dt = method.method_type.declaring_type
-        if hasattr(dt, '_fully_qualified_name'):
-            fqn = str(dt._fully_qualified_name)
-            if 'locale' in fqn:
-                return True
-
-    # Fallback: Check simple name on select
-    select = method.select
-    if select is None:
+    if not method.method_type or not method.method_type.declaring_type:
         return False
-    if isinstance(select, Identifier) and select.simple_name == "locale":
-        return True
-
-    return False
+    dt = method.method_type.declaring_type
+    if not hasattr(dt, '_fully_qualified_name'):
+        return False
+    fqn = str(dt._fully_qualified_name)
+    return 'locale' in fqn
 
 
 def _rename_method(method: MethodInvocation, new_name: str) -> MethodInvocation: