openrewrite-migrate-python 0.14.0.dev20260830170445__tar.gz → 0.14.0.dev20260901133827__tar.gz

{openrewrite_migrate_python-0.14.0.dev20260830170445 → openrewrite_migrate_python-0.14.0.dev20260901133827}/PKG-INFO

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

{openrewrite_migrate_python-0.14.0.dev20260830170445 → openrewrite_migrate_python-0.14.0.dev20260901133827}/src/openrewrite_migrate_python/migrate/string_formatting.py

@@ -72,6 +72,16 @@ def _get_fstring_delimiter(value_source: str) -> Optional[str]:
     return None
 
 
+def _interpolates_string_literal(expr: Any) -> bool:
+    """Whether substituting ``expr`` would nest a string literal inside the f-string.
+
+    Reusing the outer quote inside a replacement field is PEP 701, so the result needs
+    3.12 -- newer than the 3.8 target these recipes first run for.
+    """
+    value = expr.value if isinstance(expr, NamedArgument) else expr
+    return _is_string_literal(value)
+
+
 def _is_side_effect_free(expr: Any) -> bool:
     """Check if an expression is safe to duplicate (no side effects)."""
     return isinstance(expr, (Identifier, Literal, FieldAccess))
@@ -124,6 +134,9 @@ def _convert_format_to_fstring(method: MethodInvocation) -> Optional[FormattedSt
         if isinstance(arg, Star):
             return None
 
+    if any(_interpolates_string_literal(arg) for arg in raw_args):
+        return None
+
     # Parse format string
     try:
         parsed = list(_string_module.Formatter().parse(format_string))
@@ -410,6 +423,9 @@ def _convert_percent_to_fstring(binary: Binary) -> Optional[FormattedString]:
     if len(args) != spec_count:
         return None
 
+    if any(_interpolates_string_literal(arg) for arg in args):
+        return None
+
     # Build f-string parts
     parts: List[Expression] = []
     arg_index = 0

{openrewrite_migrate_python-0.14.0.dev20260830170445 → openrewrite_migrate_python-0.14.0.dev20260901133827}/src/openrewrite_migrate_python.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openrewrite-migrate-python
-Version: 0.14.0.dev20260830170445
+Version: 0.14.0.dev20260901133827
 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.14.0.dev20260830170445 → openrewrite_migrate_python-0.14.0.dev20260901133827}/tests/test_string_formatting.py

@@ -185,6 +185,24 @@ class TestReplaceStrFormatWithFString:
             )
         )
 
+    def test_no_change_string_literal_argument(self):
+        """'{}'.format("a") — a literal argument would nest quotes, no change."""
+        spec = RecipeSpec(recipe=ReplaceStrFormatWithFString())
+        spec.rewrite_run(
+            python(
+                """
+                url = "http://{}:{}".format("host", "80")
+                """
+            )
+        )
+        spec.rewrite_run(
+            python(
+                """
+                label = "{} = {}".format(name, "literal")
+                """
+            )
+        )
+
     def test_no_change_repeated_side_effect(self):
         """'{0} {0}'.format(func()) — repeated + side effect, no change."""
         spec = RecipeSpec(recipe=ReplaceStrFormatWithFString())
@@ -322,6 +340,17 @@ class TestReplacePercentFormatWithFString:
             )
         )
 
+    def test_no_change_string_literal_argument(self):
+        """'%s' % ("a",) — a literal argument would nest quotes, no change."""
+        spec = RecipeSpec(recipe=ReplacePercentFormatWithFString())
+        spec.rewrite_run(
+            python(
+                """
+                pair = "%s=%s" % ("key", "value")
+                """
+            )
+        )
+
     def test_no_change_unsupported_specifier(self):
         """'%d' % x — unsupported specifier, no change."""
         spec = RecipeSpec(recipe=ReplacePercentFormatWithFString())