Use console_print, not print, for data output

Author: ctrueden

src/jgo/cli/commands/config.py

@@ -210,23 +210,23 @@ def _get_jgorc(config_file: Path, section: str, key: str, args: ParsedArgs) -> i
     # Get value based on section
     if section == "settings":
         if key == "cache_dir":
-            print(settings.cache_dir)
+            console_print(settings.cache_dir)
         elif key == "repo_cache":
-            print(settings.repo_cache)
+            console_print(settings.repo_cache)
         elif key == "links":
-            print(settings.links)
+            console_print(settings.links)
         else:
             _log.error(f"Unknown setting: {key}")
             return 1
     elif section == "repositories":
         if key in settings.repositories:
-            print(settings.repositories[key])
+            console_print(settings.repositories[key])
         else:
             _log.error(f"Repository '{key}' not found")
             return 1
     elif section == "shortcuts":
         if key in settings.shortcuts:
-            print(settings.shortcuts[key])
+            console_print(settings.shortcuts[key])
         else:
             _log.error(f"Shortcut '{key}' not found")
             return 1
@@ -254,7 +254,7 @@ def _get_toml(config_file: Path, section: str, key: str, args: ParsedArgs) -> in
         return 1
 
     value = data[section][key]
-    print(value)
+    console_print(value)
     return 0
 
 

src/jgo/cli/commands/info.py

@@ -8,7 +8,7 @@
 import rich_click as click
 
 from ...parse.coordinate import Coordinate
-from ..console import console_print, get_color_mode
+from ..console import console_print
 
 _log = logging.getLogger(__name__)
 
@@ -80,7 +80,7 @@ def envdir(ctx, endpoint):
             ctx.exit(1)
         environment = builder.from_endpoint(endpoint, update=args.update)
 
-    print(environment.path)
+    console_print(environment.path)
     ctx.exit(0)
 
 
@@ -327,7 +327,7 @@ def manifest(ctx, endpoint, raw):
             if manifest_content is None:
                 _log.error(f"No MANIFEST.MF found in {jar_path}")
                 ctx.exit(1)
-            print(manifest_content, end="")
+            console_print(manifest_content, end="")
         else:
             manifest_dict = parse_manifest(jar_path)
             if manifest_dict is None:
@@ -336,7 +336,7 @@ def manifest(ctx, endpoint, raw):
 
             # Display parsed manifest
             for key, value in manifest_dict.items():
-                print(f"{key}: {value}")
+                console_print(f"{key}: {value}")
 
     except SystemExit:
         raise
@@ -401,12 +401,7 @@ def pom(ctx, endpoint):
             # If pretty-printing fails, use raw content
             xml_output = pom_content
 
-        # Disable highlighting for plain mode
-        # (Rich automatically disables syntax highlighting when piping in auto mode)
-        use_highlighting = get_color_mode() != "plain"
-
-        # console_print auto-sets soft_wrap based on wrap mode
-        console_print(xml_output, highlight=use_highlighting)
+        console_print(xml_output)
 
     except SystemExit:
         raise

src/jgo/cli/commands/versions.py

@@ -71,23 +71,21 @@ def execute(args: ParsedArgs, config: dict) -> int:
         # Get available versions
         metadata = project.metadata
         if not metadata or not metadata.versions:
-            # Use rich() for styled output
             console_print(f"No versions found for {coord.rich()}")
             return 0
 
         # Use project's smart release/latest resolution
         release_version = project.release
         latest_version = project.latest
 
-        # Data output - use str() for plain text (no Rich markup)
-        print(f"Available versions for {coord}:")
+        console_print(f"Available versions for {coord.rich()}:")
         for version in metadata.versions:
             marker = ""
             if release_version and version == release_version:
                 marker = " (release)"
             elif latest_version and version == latest_version:
                 marker = " (latest)"
-            print(f"  {version}{marker}")
+            console_print(f"  {version}[dim]{marker}[/]")
 
     except Exception as e:
         _log.error(f"Error fetching versions: {e}")

src/jgo/cli/output.py

@@ -68,9 +68,9 @@ def print_classpath(environment: Environment) -> None:
         )
         return
 
-    # Print one classpath element per line (raw output, no Rich formatting)
+    # Print one classpath element per line
     for jar_path in class_path_jars:
-        print(jar_path)
+        console_print(jar_path)
 
 
 def print_modulepath(environment: Environment) -> None:
@@ -89,9 +89,9 @@ def print_modulepath(environment: Environment) -> None:
         )
         return
 
-    # Print one module-path element per line (raw output, no Rich formatting)
+    # Print one module-path element per line
     for jar_path in module_jars:
-        print(jar_path)
+        console_print(jar_path)
 
 
 def print_jars(environment: Environment) -> None:
@@ -112,7 +112,7 @@ def print_jars(environment: Environment) -> None:
     if cp_jars:
         console_print("[bold cyan]Classpath:[/]")
         for jar_path in cp_jars:
-            print(jar_path)
+            console_print(jar_path)
     else:
         console_print("[yellow]No classpath JARs[/]")
 
@@ -123,7 +123,7 @@ def print_jars(environment: Environment) -> None:
         else:
             console_print("[bold cyan]Module-path:[/]")
         for jar_path in mp_jars:
-            print(jar_path)
+            console_print(jar_path)
     else:
         console_print("[yellow]No module-path JARs[/]")
 

tests/cli/info.t

@@ -180,49 +180,49 @@ Test info manifest with endpoint.
 Test info manifest --raw flag.
 
   $ jgo info manifest --raw com.google.guava:guava:33.0.0-jre
-  Manifest-Version: 1.0\r (esc)
-  Automatic-Module-Name: com.google.common\r (esc)
-  Build-Jdk-Spec: 11\r (esc)
-  Bundle-Description: Guava is a suite of core and expanded libraries th\r (esc)
-   at include    utility classes, Google's collections, I/O classes, and\r (esc)
-       much more.\r (esc)
-  Bundle-DocURL: https://github.com/google/guava/\r (esc)
-  Bundle-License: http://www.apache.org/licenses/LICENSE-2.0.txt\r (esc)
-  Bundle-ManifestVersion: 2\r (esc)
-  Bundle-Name: Guava: Google Core Libraries for Java\r (esc)
-  Bundle-SymbolicName: com.google.guava\r (esc)
-  Bundle-Version: 33.0.0.jre\r (esc)
-  Created-By: Apache Maven Bundle Plugin 5.1.8\r (esc)
-  Export-Package: com.google.common.annotations;version="33.0.0",com.goo\r (esc)
-   gle.common.base;version="33.0.0";uses:="javax.annotation",com.google.\r (esc)
-   common.cache;version="33.0.0";uses:="com.google.common.base,com.googl\r (esc)
-   e.common.collect,com.google.common.util.concurrent,javax.annotation",\r (esc)
-   com.google.common.collect;version="33.0.0";uses:="com.google.common.b\r (esc)
-   ase,javax.annotation",com.google.common.escape;version="33.0.0";uses:\r (esc)
-   ="com.google.common.base,javax.annotation",com.google.common.eventbus\r (esc)
-   ;version="33.0.0",com.google.common.graph;version="33.0.0";uses:="com\r (esc)
-   .google.common.collect,javax.annotation",com.google.common.hash;versi\r (esc)
-   on="33.0.0";uses:="com.google.common.base,javax.annotation",com.googl\r (esc)
-   e.common.html;version="33.0.0";uses:="com.google.common.escape",com.g\r (esc)
-   oogle.common.io;version="33.0.0";uses:="com.google.common.base,com.go\r (esc)
-   ogle.common.collect,com.google.common.graph,com.google.common.hash,ja\r (esc)
-   vax.annotation",com.google.common.math;version="33.0.0";uses:="javax.\r (esc)
-   annotation",com.google.common.net;version="33.0.0";uses:="com.google.\r (esc)
-   common.base,com.google.common.collect,com.google.common.escape,javax.\r (esc)
-   annotation",com.google.common.primitives;version="33.0.0";uses:="com.\r (esc)
-   google.common.base,javax.annotation",com.google.common.reflect;versio\r (esc)
-   n="33.0.0";uses:="com.google.common.collect,com.google.common.io,java\r (esc)
-   x.annotation",com.google.common.util.concurrent;version="33.0.0";uses\r (esc)
-   :="com.google.common.base,com.google.common.collect,com.google.common\r (esc)
-   .util.concurrent.internal,javax.annotation",com.google.common.xml;ver\r (esc)
-   sion="33.0.0";uses:="com.google.common.escape"\r (esc)
-  Import-Package: com.google.common.util.concurrent.internal;version="[1\r (esc)
-   .0,2)",javax.annotation;resolution:=optional;version="[3.0,4)",javax.\r (esc)
-   crypto;resolution:=optional,javax.crypto.spec;resolution:=optional,su\r (esc)
-   n.misc;resolution:=optional\r (esc)
-  Require-Capability: osgi.ee;filter:="(&(osgi.ee=JavaSE)(version=1.8))"\r (esc)
-  Tool: Bnd-6.3.1.202206071316\r (esc)
-  \r (esc)
+  Manifest-Version: 1.0
+  Automatic-Module-Name: com.google.common
+  Build-Jdk-Spec: 11
+  Bundle-Description: Guava is a suite of core and expanded libraries th
+   at include    utility classes, Google's collections, I/O classes, and
+       much more.
+  Bundle-DocURL: https://github.com/google/guava/
+  Bundle-License: http://www.apache.org/licenses/LICENSE-2.0.txt
+  Bundle-ManifestVersion: 2
+  Bundle-Name: Guava: Google Core Libraries for Java
+  Bundle-SymbolicName: com.google.guava
+  Bundle-Version: 33.0.0.jre
+  Created-By: Apache Maven Bundle Plugin 5.1.8
+  Export-Package: com.google.common.annotations;version="33.0.0",com.goo
+   gle.common.base;version="33.0.0";uses:="javax.annotation",com.google.
+   common.cache;version="33.0.0";uses:="com.google.common.base,com.googl
+   e.common.collect,com.google.common.util.concurrent,javax.annotation",
+   com.google.common.collect;version="33.0.0";uses:="com.google.common.b
+   ase,javax.annotation",com.google.common.escape;version="33.0.0";uses:
+   ="com.google.common.base,javax.annotation",com.google.common.eventbus
+   ;version="33.0.0",com.google.common.graph;version="33.0.0";uses:="com
+   .google.common.collect,javax.annotation",com.google.common.hash;versi
+   on="33.0.0";uses:="com.google.common.base,javax.annotation",com.googl
+   e.common.html;version="33.0.0";uses:="com.google.common.escape",com.g
+   oogle.common.io;version="33.0.0";uses:="com.google.common.base,com.go
+   ogle.common.collect,com.google.common.graph,com.google.common.hash,ja
+   vax.annotation",com.google.common.math;version="33.0.0";uses:="javax.
+   annotation",com.google.common.net;version="33.0.0";uses:="com.google.
+   common.base,com.google.common.collect,com.google.common.escape,javax.
+   annotation",com.google.common.primitives;version="33.0.0";uses:="com.
+   google.common.base,javax.annotation",com.google.common.reflect;versio
+   n="33.0.0";uses:="com.google.common.collect,com.google.common.io,java
+   x.annotation",com.google.common.util.concurrent;version="33.0.0";uses
+   :="com.google.common.base,com.google.common.collect,com.google.common
+   .util.concurrent.internal,javax.annotation",com.google.common.xml;ver
+   sion="33.0.0";uses:="com.google.common.escape"
+  Import-Package: com.google.common.util.concurrent.internal;version="[1
+   .0,2)",javax.annotation;resolution:=optional;version="[3.0,4)",javax.
+   crypto;resolution:=optional,javax.crypto.spec;resolution:=optional,su
+   n.misc;resolution:=optional
+  Require-Capability: osgi.ee;filter:="(&(osgi.ee=JavaSE)(version=1.8))"
+  Tool: Bnd-6.3.1.202206071316
+  
 
 Test info pom requires endpoint.