openrewrite-migrate-python 0.17.0.dev20261022214418__tar.gz → 0.17.0.dev20261024133624__tar.gz

{openrewrite_migrate_python-0.17.0.dev20261022214418 → openrewrite_migrate_python-0.17.0.dev20261024133624}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openrewrite-migrate-python
-Version: 0.17.0.dev20261022214418
+Version: 0.17.0.dev20261024133624
 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.17.0.dev20261022214418 → openrewrite_migrate_python-0.17.0.dev20261024133624}/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.17.0.dev20261022214418"  # Updated dynamically during release
+version = "0.17.0.dev20261024133624"  # Updated dynamically during release
 authors = [{ name = "Moderne Inc.", email = "support@moderne.io" }]
 license = { text = "Moderne Proprietary" }
 readme = "README.md"

{openrewrite_migrate_python-0.17.0.dev20261022214418 → openrewrite_migrate_python-0.17.0.dev20261024133624}/src/openrewrite_migrate_python/migrate/upgrade_to_python313.py

@@ -47,9 +47,9 @@ class UpgradeToPython313(Recipe):
         return (
             "Migrate deprecated and removed APIs for Python 3.13 compatibility. "
             "This includes detecting usage of modules removed in PEP 594 "
-            "('dead batteries'), other API changes between Python 3.12 and 3.13, and bumping "
-            "third-party dependencies (`ruff`, `avro-python3`) whose pinned releases do not "
-            "install cleanly on Python 3.13."
+            "('dead batteries'), other API changes between Python 3.12 and 3.13, bumping `ruff` "
+            "past the pinned releases that do not install cleanly on Python 3.13, and replacing "
+            "the deprecated `avro-python3` with Apache `avro`."
         )
 
     @property
@@ -67,6 +67,7 @@ class UpgradeToPython313(Recipe):
             FindSndhdrModule,
             FindSunauModule,
         )
+        from .avro_migrations import MigrateAvroPython3ToAvro
         from .cgi_migrations import FindCgiModule, FindCgitbModule
         from .locale_deprecations import ReplaceLocaleResetlocale
         from .array_deprecations import FindArrayUnicodeTypecode
@@ -112,11 +113,11 @@ class UpgradeToPython313(Recipe):
             FindXdrlibModule(),
             # 'u' array typecode deprecated in 3.13, removed in 3.16
             FindArrayUnicodeTypecode(),
-            # Bump third-party pins that don't install cleanly on 3.13 (avro-python3 off its fragile 1.8.2 sdist).
+            # Bump third-party pins that don't install cleanly on 3.13.
             RpcRecipe("org.openrewrite.python.UpgradeDependencyVersion",
                       packageName="ruff", newVersion=">=0.15.21"),
-            RpcRecipe("org.openrewrite.python.UpgradeDependencyVersion",
-                      packageName="avro-python3", newVersion=">=1.10.2"),
+            # avro-python3's fragile 1.8.2 sdist doesn't install on 3.13; avro is its maintained successor.
+            MigrateAvroPython3ToAvro(),
             # Target version in project metadata, packaging, Dockerfiles, CI, docs and infrastructure files
             RpcRecipe("org.openrewrite.python.migrate.UpgradePythonVersionTo313"),
         ]

{openrewrite_migrate_python-0.17.0.dev20261022214418 → openrewrite_migrate_python-0.17.0.dev20261024133624}/src/openrewrite_migrate_python.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openrewrite-migrate-python
-Version: 0.17.0.dev20261022214418
+Version: 0.17.0.dev20261024133624
 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.17.0.dev20261022214418 → openrewrite_migrate_python-0.17.0.dev20261024133624}/src/openrewrite_migrate_python.egg-info/SOURCES.txt

@@ -199,6 +199,7 @@ tests/test_pydantic_deprecations.py
 tests/test_pydantic_fields.py
 tests/test_pydantic_model_fields.py
 tests/test_re_deprecations.py
+tests/test_recipe_reachability.py
 tests/test_removed_modules_312.py
 tests/test_shutil_deprecations.py
 tests/test_socket_deprecations.py

openrewrite_migrate_python-0.17.0.dev20261024133624/tests/test_recipe_reachability.py

@@ -0,0 +1,125 @@
+"""Every recipe this repository publishes is run by some composite, unless it is meant to stand alone.
+
+The composites live in Python, but their leaves span both languages: the per-version
+``UpgradePythonVersionTo3XX`` recipes are declared in YAML and reached only through ``RpcRecipe``,
+and they reach the per-file recipes through ``UpgradePythonVersion.getRecipeList()``. So the graph
+is built from the marketplace the entry points install, the declarative recipes, and the Java
+recipe classes together.
+"""
+
+import re
+from pathlib import Path
+
+import openrewrite_code_quality_python
+import openrewrite_migrate_python
+from rewrite import Recipe, RecipeMarketplace
+from rewrite.rpc import RpcRecipe
+
+_ROOT = Path(__file__).resolve().parents[2]
+_JAVA_SOURCES = _ROOT / "src" / "main" / "java"
+_DECLARATIVE_SOURCES = _ROOT / "src" / "main" / "resources" / "META-INF" / "rewrite"
+_JVM_PREFIX = "org.openrewrite.python.migrate."
+
+_STANDALONE = {
+    # Entry points
+    "org.openrewrite.python.cleanup.PythonBestPractices",
+    "org.openrewrite.python.migrate.UpgradeToPython315",
+    "org.openrewrite.python.migrate.MigrateBackoffToTenacity",
+    "org.openrewrite.python.migrate.langchain.UpgradeToLangChain1",
+    "org.openrewrite.python.migrate.pandas.UpgradeToPandas20",
+    "org.openrewrite.python.migrate.pydantic.UpgradeToPydantic214",
+    "org.openrewrite.python.migrate.MigrateToPyprojectToml",
+    # Search recipes
+    "org.openrewrite.python.codequality.FindSuspiciousChainedComparison",
+    "org.openrewrite.python.migrate.FindBackoffDecoratorsNotMigrated",
+    "org.openrewrite.python.migrate.FindImplicitTextEncoding",
+    "org.openrewrite.python.migrate.DependencyInsight",
+    "org.openrewrite.python.migrate.FindFutureImports",
+    "org.openrewrite.python.migrate.FindMethods",
+    "org.openrewrite.python.migrate.FindTypes",
+}
+
+
+def _python_recipes() -> dict[str, Recipe]:
+    marketplace = RecipeMarketplace()
+    openrewrite_migrate_python.activate(marketplace)
+    openrewrite_code_quality_python.activate(marketplace)
+    recipes = {}
+    for descriptor in marketplace.all_recipes():
+        found = marketplace.find_recipe(descriptor.name)
+        assert found is not None and found[1] is not None
+        recipes[descriptor.name] = found[1]()
+    return recipes
+
+
+def _declarative_recipes() -> dict[str, set[str]]:
+    recipes = {}
+    for path in _DECLARATIVE_SOURCES.glob("*.yml"):
+        for document in re.split(r"^---$", path.read_text(), flags=re.MULTILINE):
+            if not re.search(r"^type: specs\.openrewrite\.org/v1beta/recipe$", document, re.MULTILINE):
+                continue
+            name = re.search(r"^name: (\S+)$", document, re.MULTILINE)
+            assert name is not None, f"a recipe in {path.name} has no name"
+            recipes[name.group(1)] = set(re.findall(r"^\s*- (org\.openrewrite\.[\w.]+):?$", document, re.MULTILINE))
+    return recipes
+
+
+def _java_recipes() -> dict[str, set[str]]:
+    recipes = {}
+    for path in _JAVA_SOURCES.rglob("*.java"):
+        source = path.read_text()
+        package = re.search(r"^package ([\w.]+);", source, re.MULTILINE)
+        imports = {fqn.rsplit(".", 1)[1]: fqn for fqn in re.findall(r"^import ([\w.]+);", source, re.MULTILINE)}
+        for cls in re.findall(r"^public (?:final )?class (\w+) extends (?:Scanning)?Recipe\b", source, re.MULTILINE):
+            assert package is not None
+            recipe_list = re.search(r"getRecipeList\(\) \{(.*?)^    }", source, re.MULTILINE | re.DOTALL)
+            children = re.findall(r"\bnew (\w+)\(", recipe_list.group(1)) if recipe_list else []
+            recipes[f"{package.group(1)}.{cls}"] = {imports.get(c, f"{package.group(1)}.{c}") for c in children}
+    return recipes
+
+
+def _referenced(recipe: Recipe) -> tuple[set[str], set[str]]:
+    """The names a composite runs at any depth, and the subset it reaches over RPC."""
+    names, rpc = set(), set()
+    for sub in recipe.recipe_list():
+        names.add(sub.name)
+        if isinstance(sub, RpcRecipe):
+            rpc.add(sub.name)
+        else:
+            nested, nested_rpc = _referenced(sub)
+            names |= nested
+            rpc |= nested_rpc
+    return names, rpc
+
+
+class TestRecipeReachability:
+
+    def setup_method(self):
+        self.python = _python_recipes()
+        self.declarative = _declarative_recipes()
+        self.java = _java_recipes()
+        self.referenced: set[str] = set()
+        self.rpc: set[str] = set()
+        for recipe in self.python.values():
+            names, rpc = _referenced(recipe)
+            self.referenced |= names
+            self.rpc |= rpc
+        for children in [*self.declarative.values(), *self.java.values()]:
+            self.referenced |= children
+
+    def test_every_recipe_is_reachable_from_a_composite(self):
+        published = self.python.keys() | self.declarative.keys() | self.java.keys()
+        unreachable = published - self.referenced
+        assert unreachable - _STANDALONE == set(), (
+            "Add these to a composite, or to _STANDALONE if they are meant to be run on their own"
+        )
+        assert _STANDALONE - unreachable == set(), (
+            "These are now run by a composite; remove them from _STANDALONE"
+        )
+
+    def test_every_jvm_recipe_a_composite_names_exists(self):
+        jvm = self.declarative.keys() | self.java.keys()
+        named = {n for n in self.rpc if n.startswith(_JVM_PREFIX)}
+        for children in [*self.declarative.values(), *self.java.values()]:
+            named |= {n for n in children if n.startswith(_JVM_PREFIX)}
+        assert named - jvm == set()

{openrewrite_migrate_python-0.17.0.dev20261022214418 → openrewrite_migrate_python-0.17.0.dev20261024133624}/tests/test_upgrade_recipes.py

@@ -121,16 +121,15 @@ class TestUpgradeRecipeChaining:
                 expected = f"org.openrewrite.python.migrate.{name}"
                 assert expected in recipe_names, f"{recipe_cls.__name__} must run {name}"
 
-    def test_upgrade_to_python313_bumps_incompatible_dependencies(self):
-        """UpgradeToPython313 inlines the 3.13-incompatible dependency bumps over RPC."""
+    def test_upgrade_to_python313_replaces_incompatible_dependencies(self):
         recipe_list = UpgradeToPython313().recipe_list()
         dep_bumps = {
             (r.delegates_to_options.get("packageName"), r.delegates_to_options.get("newVersion"))
             for r in recipe_list
             if isinstance(r, RpcRecipe) and r.name == "org.openrewrite.python.UpgradeDependencyVersion"
         }
-        assert ("ruff", ">=0.15.21") in dep_bumps
-        assert ("avro-python3", ">=1.10.2") in dep_bumps
+        assert dep_bumps == {("ruff", ">=0.15.21")}
+        assert "org.openrewrite.python.migrate.MigrateAvroPython3ToAvro" in {r.name for r in recipe_list}
 
 
 class _NativeUpgradeToPython313(Recipe):