Run the linter

Author: ctrueden

src/jgo/cli/commands/search.py

@@ -10,8 +10,10 @@
 
 import rich_click as click
 
+from ...parse.coordinate import Coordinate
 from ...util import is_info_enabled
 from ..console import console_print
+from ..rich.formatters import format_coordinate
 
 if TYPE_CHECKING:
     from ..parser import ParsedArgs
@@ -217,10 +219,8 @@ def _display_results(results: list[dict]) -> None:
         version = result["latest_version"]
 
         # Basic format: coordinate and latest version
-        from ...parse.coordinate import Coordinate
-
         coord = Coordinate(group_id, artifact_id, version)
-        console_print(f"{i}. {coord.rich()}")
+        console_print(f"{i}. {format_coordinate(coord)}")
 
         if verbose:
             # Show additional details in verbose mode

src/jgo/cli/commands/versions.py

@@ -8,6 +8,7 @@
 import rich_click as click
 
 from ..console import console_print
+from ..rich.formatters import format_coordinate
 
 if TYPE_CHECKING:
     from ..parser import ParsedArgs
@@ -71,14 +72,14 @@ def execute(args: ParsedArgs, config: dict) -> int:
         # Get available versions
         metadata = project.metadata
         if not metadata or not metadata.versions:
-            console_print(f"No versions found for {coord.rich()}")
+            console_print(f"No versions found for {format_coordinate(coord)}")
             return 0
 
         # Use project's smart release/latest resolution
         release_version = project.release
         latest_version = project.latest
 
-        console_print(f"Available versions for {coord.rich()}:")
+        console_print(f"Available versions for {format_coordinate(coord)}:")
         for version in metadata.versions:
             marker = ""
             if release_version and version == release_version:

src/jgo/cli/rich/formatters.py

@@ -11,10 +11,53 @@
 
 from rich.tree import Tree
 
+from ...parse.coordinate import coord2str
 from .widgets import NoWrapTree
 
 if TYPE_CHECKING:
     from ...maven.core import DependencyNode
+    from ...parse.coordinate import Coordinate
+
+
+def format_coordinate(coord: Coordinate) -> str:
+    """
+    Format a Maven coordinate with Rich markup for semantic coloring.
+
+    Uses Rich markup to colorize components based on their semantic meaning:
+    - groupId: bold cyan
+    - artifactId: bold
+    - version: green
+    - packaging: default color
+    - classifier: default color
+    - scope: dim
+    - colons: dim
+
+    The markup is automatically stripped by Rich when --color=plain is used.
+
+    Args:
+        coord: The Coordinate to format
+
+    Returns:
+        Formatted string with Rich markup
+
+    Examples:
+        >>> from jgo.parse.coordinate import Coordinate
+        >>> coord = Coordinate("sc.fiji", "fiji", "2.17.0")
+        >>> format_coordinate(coord)
+        '[bold cyan]sc.fiji[/][dim]:[/][bold]fiji[/][dim]:[/][green]2.17.0[/]'
+    """
+    return coord2str(
+        coord.groupId,
+        coord.artifactId,
+        coord.version,
+        coord.classifier,
+        coord.packaging,
+        coord.scope,
+        coord.optional,
+        coord.raw,
+        coord.placement,
+        rich=True,
+    )
 
 
 def format_dependency_list(

src/jgo/parse/coordinate.py

@@ -76,42 +76,6 @@ def __str__(self) -> str:
             self.placement,
         )
 
-    def rich(self) -> str:
-        """
-        Return Rich-formatted string representation with semantic colors.
-
-        Uses Rich markup to colorize components based on their semantic meaning:
-        - groupId: bold cyan
-        - artifactId: bold
-        - version: green
-        - packaging: default color
-        - classifier: default color
-        - scope: dim
-        - colons: dim
-
-        The markup is automatically stripped by Rich when --color=plain is used.
-
-        Returns:
-            Formatted string with Rich markup
-
-        Examples:
-            >>> coord = Coordinate("sc.fiji", "fiji", "2.17.0")
-            >>> coord.rich()
-            '[bold cyan]sc.fiji[/][dim]:[/][bold]fiji[/][dim]:[/][green]2.17.0[/]'
-        """
-        return coord2str(
-            self.groupId,
-            self.artifactId,
-            self.version,
-            self.classifier,
-            self.packaging,
-            self.scope,
-            self.optional,
-            self.raw,
-            self.placement,
-            rich=True,
-        )
-
     @classmethod
     def parse(cls, coordinate: "Coordinate" | str) -> "Coordinate":
         """

tests/conftest.py

@@ -5,9 +5,9 @@
 from pathlib import Path
 
 import pytest
+from tests.fixtures.thicket import DEFAULT_SEED, generate_thicket
 
 from jgo.util.maven import ensure_maven_available
-from tests.fixtures.thicket import DEFAULT_SEED, generate_thicket
 
 
 @pytest.fixture(scope="session")
@@ -61,7 +61,13 @@ def m2_repo(tmp_path_factory):
     # If cache is already populated, these will be quick no-ops
     for goal in ["dependency:list", "dependency:tree"]:
         subprocess.run(
-            [maven_cmd, "-f", str(bootstrap_pom), f"-Dmaven.repo.local={cache_dir}", goal],
+            [
+                maven_cmd,
+                "-f",
+                str(bootstrap_pom),
+                f"-Dmaven.repo.local={cache_dir}",
+                goal,
+            ],
             check=True,
             capture_output=True,
         )

tests/test_resolution.py

@@ -201,7 +201,6 @@ def mvn_resolver(maven_command):
     ids=lambda case: case.endpoint,
 )
 def test_resolution_modes(m2_repo, scenario, mvn_resolver):
-
     python_resolver = PythonResolver()
     context = MavenContext(repo_cache=m2_repo, resolver=python_resolver)
     components = components_from_endpoint(context, scenario.endpoint)