Purge unnecessary f-strings

Author: ctrueden

docs/output-subsystem.md

@@ -556,7 +556,7 @@ def print_config_section():
 def print_error_info():
     """Print diagnostic info to stderr."""
     console_print("Diagnostic information:", stderr=True)
-    console_print(f"  Error code: 42", stderr=True)
+    console_print("  Error code: 42", stderr=True)
 
 # === Progress Reporting Pattern ===
 import logging

src/jgo/cli/commands/remove.py

@@ -21,7 +21,7 @@
     nargs=-1,
     required=True,
     cls=click.RichArgument,
-    help=f"One or more Maven coordinates in format "
+    help="One or more Maven coordinates in format "
     f"{styled('g:a', g='groupId', a='artifactId')} (version optional)",
 )
 @click.option(

src/jgo/cli/commands/search.py

@@ -23,11 +23,11 @@
 
 
 @click.command(
-    help=f"Search for artifacts in [magenta]Maven repositories[/]. "
+    help="Search for artifacts in [magenta]Maven repositories[/]. "
     f"Supports plain text, coordinates ({COORD_HELP_SHORT}), or SOLR syntax ([cyan]g: a:[/]).",
-    epilog=f"[dim]TIP: Try [yellow]g:groupId a:artifactId[/] for SOLR syntax, "
+    epilog="[dim]TIP: Try [yellow]g:groupId a:artifactId[/] for SOLR syntax, "
     f"[yellow]{COORD_HELP_FULL}[/] for coordinates, or plain text. "
-    f"Use [yellow]*[/] for wildcards and [yellow]~[/] for fuzzy search.[/]",
+    "Use [yellow]*[/] for wildcards and [yellow]~[/] for fuzzy search.[/]",
 )
 @click.option(
     "--limit",

src/jgo/exec/java_source.py

@@ -109,7 +109,7 @@ def _locate_system_java(self, required_version: int | None = None) -> Path:
             if actual_version < required_version:
                 raise RuntimeError(
                     f"Java {required_version} or higher required, but system Java is version {actual_version}. "
-                    f"Please upgrade Java or use auto mode for automatic Java management."
+                    "Please upgrade Java or use auto mode for automatic Java management."
                 )
             else:
                 self._maybe_log(f"Using system Java {actual_version} at {java_path}")

src/jgo/maven/model.py

@@ -435,8 +435,8 @@ def _import_boms(self, candidates: dict[GACT, Dependency]) -> None:
             if dep.version == "MANAGED":
                 raise ValueError(
                     f"{self.gav}: BOM {dep.groupId}:{dep.artifactId} has unresolved "
-                    f"MANAGED version. MANAGED versions must be resolved before "
-                    f"creating the wrapper POM."
+                    "MANAGED version. MANAGED versions must be resolved before "
+                    "creating the wrapper POM."
                 )
 
             bom_gav = f"{dep.groupId}:{dep.artifactId}:{dep.version}"
@@ -599,9 +599,9 @@ def _interpolate_deps(self, deps: dict[GACT, Dependency]) -> dict[GACT, Dependen
             # Check for collisions (rare, but possible)
             if new_gact in new_deps and new_gact != old_gact:
                 _log.warning(
-                    f"Interpolation caused GACT key collision: "
+                    "Interpolation caused GACT key collision: "
                     f"{old_gact} -> {new_gact}. "
-                    f"Keeping first occurrence (nearest wins)."
+                    "Keeping first occurrence (nearest wins)."
                 )
                 continue
 

src/jgo/parse/endpoint.py

@@ -139,7 +139,7 @@ def parse(cls, endpoint: "Endpoint" | str) -> "Endpoint":
             warnings.warn(
                 f"Deprecated main class syntax: '{endpoint}'. "
                 f"Use '{normalized}' instead. "
-                f"The '@MainClass' should be at the end of the endpoint.",
+                "The '@MainClass' should be at the end of the endpoint.",
                 DeprecationWarning,
                 stacklevel=2,
             )

tests/test_resolution.py

@@ -332,6 +332,6 @@ def test_managed_version_resolution(m2_repo):
     assert len(optional_nodes) == 1, "Expected minimaven as root child"
     # The version should be resolved from dependency management (2.2.1)
     assert optional_nodes[0].dep.version == "2.2.1", (
-        f"Expected minimaven version 2.2.1 from dependency management, "
+        "Expected minimaven version 2.2.1 from dependency management, "
         f"got {optional_nodes[0].dep.version}"
     )

tests/test_thicket.py

@@ -204,7 +204,7 @@ def test_bom_imports(self, thicket_pom, thicket_context):
         # The model should have more managed dependencies than just the direct imports,
         # because BOMs import other BOMs transitively
         assert len(model.dep_mgmt) > direct_imports, (
-            f"Expected transitive BOM imports to increase managed dependencies. "
+            "Expected transitive BOM imports to increase managed dependencies. "
             f"Direct imports: {direct_imports}, total managed: {len(model.dep_mgmt)}"
         )