Make deplist command's --direct flag actually work

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

Author: ctrueden

docs/changelog.md

@@ -9,6 +9,7 @@
 
 ### Bug fixes
 
+- **Make `deplist --direct` actually work** -- The `--direct` flag was being ignored for the `deplist` command. Now it's passed through to the interior logic to handle it properly.
 - **Sort `info versions` output** -- The reported versions list is now ordered according to Maven's versioning semantics.
 - **Add missing type hints** -- The CLI layer's click-related functions now have full type hints.
 

src/jgo/cli/_commands/info.py

@@ -228,7 +228,7 @@ def deptree(ctx: click.Context, endpoint: str | None) -> None:
 @click.pass_context
 def deplist(ctx: click.Context, endpoint: str | None, direct: bool) -> None:
     """Show a flat list of all dependencies for the given endpoint."""
-    _print_deps(ctx, endpoint, list_mode=True)
+    _print_deps(ctx, endpoint, list_mode=True, direct=direct)
 
 
 @click.command(help="Show Java version requirements.")
@@ -614,7 +614,9 @@ def _parse_coord_or_die(ctx: click.Context, coord_str: str) -> Coordinate:
         ctx.exit(1)
 
 
-def _print_deps(ctx: click.Context, endpoint: str | None, list_mode: bool) -> None:
+def _print_deps(
+    ctx: click.Context, endpoint: str | None, list_mode: bool, direct: bool = False
+) -> None:
     """Common logic for deptree and deplist."""
 
     opts = ctx.obj
@@ -639,6 +641,7 @@ def _print_deps(ctx: click.Context, endpoint: str | None, list_mode: bool) -> No
                 pom_path,
                 context,
                 list_mode=list_mode,
+                direct_only=direct,
                 optional_depth=args.get_effective_optional_depth(),
             )
         except (RuntimeError, ValueError) as e:
@@ -669,5 +672,5 @@ def _print_deps(ctx: click.Context, endpoint: str | None, list_mode: bool) -> No
             ctx.exit(1)
         dependencies = builder._coordinates_to_dependencies(parsed.coordinates)
 
-    print_dependencies(dependencies, context, list_mode=list_mode)
+    print_dependencies(dependencies, context, list_mode=list_mode, direct_only=direct)
     ctx.exit(0)

tests/cli/info.t

@@ -136,16 +136,26 @@ Test info deplist with endpoint.
      com.google.j2objc:j2objc-annotations:2.8
      org.checkerframework:checker-qual:3.41.0
 
-Test info deplist --direct flag.
-
-  $ jgo info deplist --direct com.google.guava:guava:33.0.0-jre
-  com.google.guava:guava:33.0.0-jre
-     com.google.code.findbugs:jsr305:3.0.2
-     com.google.errorprone:error_prone_annotations:2.23.0
-     com.google.guava:failureaccess:1.0.2
-     com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava
-     com.google.j2objc:j2objc-annotations:2.8
-     org.checkerframework:checker-qual:3.41.0
+Test info deplist --direct flag. Use an artifact whose direct dependencies are a
+strict subset of its transitive closure, so the flag's effect is observable
+(guava's direct deps happen to equal its transitive set, which would not catch a
+regression where --direct is ignored).
+
+  $ jgo info deplist org.junit.jupiter:junit-jupiter:5.10.0
+  org.junit.jupiter:junit-jupiter:5.10.0
+     org.apiguardian:apiguardian-api:1.1.2
+     org.junit.jupiter:junit-jupiter-api:5.10.0
+     org.junit.jupiter:junit-jupiter-engine:5.10.0:runtime
+     org.junit.jupiter:junit-jupiter-params:5.10.0
+     org.junit.platform:junit-platform-commons:1.10.0
+     org.junit.platform:junit-platform-engine:1.10.0:runtime
+     org.opentest4j:opentest4j:1.3.0
+
+  $ jgo info deplist --direct org.junit.jupiter:junit-jupiter:5.10.0
+  org.junit.jupiter:junit-jupiter:5.10.0
+     org.junit.jupiter:junit-jupiter-api:5.10.0
+     org.junit.jupiter:junit-jupiter-engine:5.10.0:runtime
+     org.junit.jupiter:junit-jupiter-params:5.10.0
 
 Test info javainfo with no endpoint.