cloudsmith-cli 1.9.2__tar.gz → 1.9.4__tar.gz

{cloudsmith_cli-1.9.2/cloudsmith_cli.egg-info → cloudsmith_cli-1.9.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cloudsmith-cli
-Version: 1.9.2
+Version: 1.9.4
 Summary: Cloudsmith Command-Line Interface (CLI)
 Home-page: https://github.com/cloudsmith-io/cloudsmith-cli
 Author: Cloudsmith Ltd

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/commands/dependencies.py

@@ -42,7 +42,7 @@ def list_dependencies(ctx, opts, owner_repo_package):
     for the package format yet.
 
     - OWNER/REPO/PACKAGE: Specify the OWNER namespace (i.e. user or org), the
-    REPO name where the package is stored, and the PACKAGE name (identifier) of the
+    REPO name where the package is stored, and the PACKAGE name (identifier/slug) of the
     package itself. All separated by a slash.
 
     Example: 'your-org/awesome-repo/better-pkg'.

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/commands/list_.py

@@ -7,6 +7,7 @@ import click
 
 from ...core.api.distros import list_distros
 from ...core.api.packages import get_package_format_names_with_distros, list_packages
+from ...core.pagination import paginate_results
 from .. import command, decorators, utils, validators
 from ..exceptions import handle_api_exceptions
 from ..utils import maybe_spinner
@@ -156,7 +157,7 @@ def entitlements_(*args, **kwargs):  # pylint: disable=missing-docstring
     help=("Sort packages by field. Prefix with '-' for descending order."),
 )
 @click.pass_context
-def packages(ctx, opts, owner_repo, page, page_size, query, sort):
+def packages(ctx, opts, owner_repo, page, page_size, query, sort, show_all):
     """
     List packages for a repository.
 
@@ -217,11 +218,13 @@ def packages(ctx, opts, owner_repo, page, page_size, query, sort):
     context_msg = "Failed to get list of packages!"
     with handle_api_exceptions(ctx, opts=opts, context_msg=context_msg):
         with maybe_spinner(opts):
-            packages_, page_info = list_packages(
-                owner=owner,
-                repo=repo,
+            packages_, page_info = paginate_results(
+                list_packages,
+                show_all=show_all,
                 page=page,
                 page_size=page_size,
+                owner=owner,
+                repo=repo,
                 query=query,
                 sort=sort,
             )
@@ -256,9 +259,12 @@ def packages(ctx, opts, owner_repo, page, page_size, query, sort):
     click.echo()
 
     num_results = len(packages_)
-    list_suffix = "package%s visible" % ("s" if num_results != 1 else "")
+    list_suffix = "package%s" % ("s" if num_results != 1 else "")
     utils.pretty_print_list_info(
-        num_results=num_results, page_info=page_info, suffix=list_suffix
+        num_results=num_results,
+        page_info=None if show_all else page_info,
+        suffix=f"{list_suffix} retrieved" if show_all else f"{list_suffix} visible",
+        show_all=show_all,
     )
 
 
@@ -276,7 +282,7 @@ def packages(ctx, opts, owner_repo, page, page_size, query, sort):
     required=False,
 )
 @click.pass_context
-def repos(ctx, opts, owner_repo, page, page_size):
+def repos(ctx, opts, owner_repo, page, page_size, show_all):
     """
     List repositories for a namespace (owner).
 

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/commands/policy/license.py

@@ -5,6 +5,7 @@ import json
 import click
 
 from ....core.api import orgs as api
+from ....core.pagination import paginate_results
 from ... import command, decorators, utils, validators
 from ...exceptions import handle_api_exceptions
 from ...utils import (
@@ -19,7 +20,6 @@ from .command import policy
 
 def print_license_policies(policies):
     """Print license policies as a table or output in another format."""
-
     headers = [
         "Name",
         "Description",
@@ -57,10 +57,6 @@ def print_license_policies(policies):
     utils.pretty_print_table(headers, rows)
     click.echo()
 
-    num_results = len(rows)
-    list_suffix = "license polic%s" % ("y" if num_results == 1 else "ies")
-    utils.pretty_print_list_info(num_results=num_results, suffix=list_suffix)
-
 
 @policy.group(cls=command.AliasGroup, name="license", aliases=[])
 @decorators.common_cli_config_options
@@ -86,7 +82,7 @@ def licence(*args, **kwargs):
     "owner", metavar="OWNER", callback=validators.validate_owner, required=True
 )
 @click.pass_context
-def ls(ctx, opts, owner, page, page_size):
+def ls(ctx, opts, owner, page, page_size, show_all):
     """
     List license policies.
 
@@ -111,8 +107,8 @@ def ls(ctx, opts, owner, page, page_size):
     context_msg = "Failed to get license policies!"
     with handle_api_exceptions(ctx, opts=opts, context_msg=context_msg):
         with maybe_spinner(opts):
-            policies, page_info = api.list_license_policies(
-                owner=owner, page=page, page_size=page_size
+            policies, page_info = paginate_results(
+                api.list_license_policies, show_all, page, page_size, owner=owner
             )
 
     click.secho("OK", fg="green", err=use_stderr)
@@ -122,6 +118,17 @@ def ls(ctx, opts, owner, page, page_size):
 
     print_license_policies(policies)
 
+    click.echo()
+
+    num_results = len(policies)
+    list_suffix = "license polic%s" % ("y" if num_results == 1 else "ies")
+    utils.pretty_print_list_info(
+        num_results=num_results,
+        page_info=None if show_all else page_info,
+        suffix=list_suffix,
+        show_all=show_all,
+    )
+
 
 @licence.command(aliases=["new"])
 @decorators.common_cli_config_options

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/commands/policy/vulnerability.py

@@ -5,6 +5,7 @@ import json
 import click
 
 from ....core.api import orgs as api
+from ....core.pagination import paginate_results
 from ... import command, decorators, utils, validators
 from ...exceptions import handle_api_exceptions
 from ...utils import fmt_bool, fmt_datetime, maybe_spinner, maybe_truncate_string
@@ -46,10 +47,6 @@ def print_vulnerability_policies(policies):
     utils.pretty_print_table(headers, rows, title="Vulnerability Policies")
     click.echo()
 
-    num_results = len(rows)
-    list_suffix = "vulnerability polic%s" % ("y" if num_results == 1 else "ies")
-    utils.pretty_print_list_info(num_results=num_results, suffix=list_suffix)
-
 
 @policy.group(cls=command.AliasGroup, name="vulnerability", aliases=[])
 @decorators.common_cli_config_options
@@ -75,7 +72,7 @@ def vulnerability(*args, **kwargs):
     "owner", metavar="OWNER", callback=validators.validate_owner, required=True
 )
 @click.pass_context
-def ls(ctx, opts, owner, page, page_size):
+def ls(ctx, opts, owner, page, page_size, show_all):
     """
     List vulnerability policies.
 
@@ -100,8 +97,8 @@ def ls(ctx, opts, owner, page, page_size):
     context_msg = "Failed to get package vulnerability policies!"
     with handle_api_exceptions(ctx, opts=opts, context_msg=context_msg):
         with maybe_spinner(opts):
-            policies, page_info = api.list_vulnerability_policies(
-                owner=owner, page=page, page_size=page_size
+            policies, page_info = paginate_results(
+                api.list_vulnerability_policies, show_all, page, page_size, owner=owner
             )
 
     click.secho("OK", fg="green", err=use_stderr)
@@ -111,6 +108,17 @@ def ls(ctx, opts, owner, page, page_size):
 
     print_vulnerability_policies(policies)
 
+    click.echo()
+
+    num_results = len(policies)
+    list_suffix = "vulnerability polic%s" % ("y" if num_results == 1 else "ies")
+    utils.pretty_print_list_info(
+        num_results=num_results,
+        page_info=None if show_all else page_info,
+        suffix=list_suffix,
+        show_all=show_all,
+    )
+
 
 @vulnerability.command(aliases=["new"])
 @decorators.common_cli_config_options

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/commands/repos.py

@@ -6,13 +6,14 @@ from operator import itemgetter
 import click
 
 from ...core.api import repos as api
+from ...core.pagination import paginate_results
 from .. import command, decorators, utils, validators
 from ..exceptions import handle_api_exceptions
 from ..utils import maybe_spinner
 from .main import main
 
 
-def print_repositories(opts, data, page_info=None, show_list_info=True):
+def print_repositories(opts, data, page_info=None, show_list_info=True, show_all=False):
     """Print repositories as a table or output in another format."""
     headers = [
         "Name",
@@ -48,10 +49,16 @@ def print_repositories(opts, data, page_info=None, show_list_info=True):
 
     click.echo()
 
+    if not show_list_info:
+        return
+
     num_results = len(data)
-    list_suffix = "repositor%s visible" % ("ies" if num_results != 1 else "y")
+    list_suffix = "repositor%s" % ("ies" if num_results != 1 else "y")
     utils.pretty_print_list_info(
-        num_results=num_results, page_info=page_info, suffix=list_suffix
+        num_results=num_results,
+        page_info=None if show_all else page_info,
+        suffix=f"{list_suffix} retrieved" if show_all else f"{list_suffix} visible",
+        show_all=show_all,
     )
 
 
@@ -83,7 +90,7 @@ def repositories(ctx, opts):  # pylink: disable=unused-argument
     required=False,
 )
 @click.pass_context
-def get(ctx, opts, owner_repo, page, page_size):
+def get(ctx, opts, owner_repo, page, page_size, show_all):
     """
     List repositories for a namespace (owner).
 
@@ -96,30 +103,34 @@ def get(ctx, opts, owner_repo, page, page_size):
     If OWNER isn't specified it'll default to the currently authenticated user
     (if any). If you're unauthenticated, no results will be returned.
     """
-    # Use stderr for messages if the output is something else (e.g.  # JSON)
+    # Use stderr for messages if the output is something else (e.g. JSON)
     use_stderr = opts.output != "pretty"
 
-    click.echo("Getting list of repositories ... ", nl=False, err=use_stderr)
-
     if isinstance(owner_repo, list):
         if len(owner_repo) == 1:
             owner = owner_repo[0]
             repo = None
         else:
             owner, repo = owner_repo
-    if isinstance(owner_repo, str):
+    elif isinstance(owner_repo, str):
+        repo = None
+        owner = owner_repo or None
+    else:
+        owner = None
         repo = None
 
-        if owner_repo:
-            owner = owner_repo
-        else:
-            owner = None
+    if show_all and repo:
+        raise click.UsageError(
+            "The --show-all option cannot be used when specifying a single repository (OWNER/REPO). Omit the repository slug or remove --show-all."
+        )
+
+    click.echo("Getting list of repositories ... ", nl=False, err=use_stderr)
 
     context_msg = "Failed to get list of repositories!"
     with handle_api_exceptions(ctx, opts=opts, context_msg=context_msg):
         with maybe_spinner(opts):
-            repos_, page_info = api.list_repos(
-                owner=owner, repo=repo, page=page, page_size=page_size
+            repos_, page_info = paginate_results(
+                api.list_repos, show_all, page, page_size, owner=owner, repo=repo
             )
 
     click.secho("OK", fg="green", err=use_stderr)
@@ -128,7 +139,11 @@ def get(ctx, opts, owner_repo, page, page_size):
         return
 
     print_repositories(
-        opts=opts, data=repos_, show_list_info=False, page_info=page_info
+        opts=opts,
+        data=repos_,
+        show_list_info=True,
+        page_info=page_info,
+        show_all=show_all,
     )
 
 
@@ -192,7 +207,10 @@ def create(ctx, opts, owner, repo_config_file):
 
     click.secho("OK", fg="green", err=use_stderr)
 
-    print_repositories(opts=opts, data=[repository], show_list_info=False)
+    if utils.maybe_print_as_json(opts, [repository]):
+        return
+
+    print_repositories(opts=opts, data=[repository], show_list_info=True)
 
 
 @repositories.command()
@@ -252,7 +270,10 @@ def update(ctx, opts, owner_repo, repo_config_file):
 
     click.secho("OK", fg="green", err=use_stderr)
 
-    print_repositories(opts=opts, data=[repository], show_list_info=False)
+    if utils.maybe_print_as_json(opts, [repository]):
+        return
+
+    print_repositories(opts=opts, data=[repository], show_list_info=True)
 
 
 @repositories.command(aliases=["rm"])

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/commands/upstream.py

@@ -5,6 +5,7 @@ import json
 import click
 
 from ...core.api import upstreams as api
+from ...core.pagination import paginate_results
 from .. import command, decorators, utils, validators
 from ..exceptions import handle_api_exceptions
 from ..utils import (
@@ -37,7 +38,7 @@ UPSTREAM_FORMATS = [
 ]
 
 
-def print_upstreams(upstreams, upstream_fmt):
+def print_upstreams(upstreams, upstream_fmt, page_info=None, show_all=False):
     """Print upstreams as a table or output in another format."""
 
     def build_row(u):
@@ -115,9 +116,14 @@ def print_upstreams(upstreams, upstream_fmt):
     utils.pretty_print_table(headers, rows)
     click.echo()
 
-    num_results = len(rows)
+    num_results = len(upstreams)
     list_suffix = "upstream%s" % ("" if num_results == 1 else "s")
-    utils.pretty_print_list_info(num_results=num_results, suffix=list_suffix)
+    utils.pretty_print_list_info(
+        num_results=num_results,
+        page_info=None if show_all else page_info,
+        suffix=list_suffix,
+        show_all=show_all,
+    )
 
 
 @main.group(cls=command.AliasGroup, name="upstream", aliases=[])
@@ -164,7 +170,7 @@ def build_upstream_list_command(upstream_fmt):
         "owner_repo", metavar="OWNER/REPO", callback=validators.validate_owner_repo
     )
     @click.pass_context
-    def func(ctx, opts, owner_repo, page, page_size):
+    def func(ctx, opts, owner_repo, page, page_size, show_all):
         owner, repo = owner_repo
 
         # Use stderr for messages if the output is something else (e.g.  # JSON)
@@ -176,12 +182,14 @@ def build_upstream_list_command(upstream_fmt):
         context_msg = "Failed to get upstreams!"
         with handle_api_exceptions(ctx, opts=opts, context_msg=context_msg):
             with maybe_spinner(opts):
-                upstreams, page_info = api.list_upstreams(
+                upstreams, page_info = paginate_results(
+                    api.list_upstreams,
+                    show_all,
+                    page,
+                    page_size,
                     owner=owner,
                     repo=repo,
                     upstream_format=upstream_fmt,
-                    page=page,
-                    page_size=page_size,
                 )
 
         if not use_stderr:
@@ -190,7 +198,7 @@ def build_upstream_list_command(upstream_fmt):
         if utils.maybe_print_as_json(opts, upstreams, page_info):
             return
 
-        print_upstreams(upstreams, upstream_fmt)
+        print_upstreams(upstreams, upstream_fmt, page_info, show_all)
 
     func.__doc__ = f"""
         List {upstream_fmt} upstreams for a repository.

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/decorators.py

@@ -4,8 +4,10 @@ import functools
 
 import click
 
+from cloudsmith_cli.cli import validators
+
 from ..core.api.init import initialise_api as _initialise_api
-from . import config, utils, validators
+from . import config, utils
 
 
 def report_retry(seconds, context=None):
@@ -144,6 +146,14 @@ def common_cli_output_options(f):
 def common_cli_list_options(f):
     """Add common list options to commands."""
 
+    @click.option(
+        "-l",
+        "--page-size",
+        default=30,
+        type=int,
+        help="The amount of items to view per page for lists.",
+        callback=validators.validate_page_size,
+    )
     @click.option(
         "-p",
         "--page",
@@ -153,18 +163,20 @@ def common_cli_list_options(f):
         callback=validators.validate_page,
     )
     @click.option(
-        "-l",
-        "--page-size",
-        default=30,
-        type=int,
-        help="The amount of items to view per page for lists.",
-        callback=validators.validate_page_size,
-    )
+        "--show-all",
+        default=False,
+        is_flag=True,
+        help="Show all results. Cannot be used with --page (-p) or --page-size (-l).",
+    )  # Validation performed post-parse for order-independence.
     @click.pass_context
     @functools.wraps(f)
     def wrapper(ctx, *args, **kwargs):
         # pylint: disable=missing-docstring
         opts = config.get_or_create_options(ctx)
+        # Order-independent validation: perform after all options parsed.
+        show_all = kwargs.get("show_all")
+        if show_all:
+            validators.enforce_show_all_exclusive(ctx)
         kwargs["opts"] = opts
         return ctx.invoke(f, *args, **kwargs)
 

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/tests/commands/policy/test_licence.py

@@ -125,6 +125,22 @@ def test_license_policy_commands(runner, organization, tmp_path):
     assert "Getting license policies ... OK" in result.output
     assert_output_matches_policy_config(result.output, policy_config_file_path)
 
+    # Minimal show-all success
+    result = runner.invoke(
+        ls, args=[organization, "--show-all"], catch_exceptions=False
+    )
+    assert result.exit_code == 0
+    assert "Getting license policies" in result.output
+    assert "Invalid value for '--show-all'" not in result.output
+
+    # Conflict: show-all with explicit page
+    conflict = runner.invoke(
+        ls, args=[organization, "--page", "1", "--show-all"], catch_exceptions=False
+    )
+    assert conflict.exit_code != 0
+    assert "Invalid value for '--show-all'" in conflict.output
+    assert "Cannot be used with --page (-p) or --page-size (-l)." in conflict.output
+
     # Change the values in the config file
     policy_config_file_path = create_license_policy_config_file(
         directory=tmp_path,

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/tests/commands/policy/test_vulnerability.py

@@ -127,6 +127,24 @@ def test_vulnerability_policy_commands(runner, organization, tmp_path):
     assert "Getting vulnerability policies ... OK" in result.output
     assert_output_matches_policy_config(result.output, policy_config_file_path)
 
+    # Minimal show-all success (no pagination args besides flag)
+    result = runner.invoke(
+        ls, args=[organization, "--show-all"], catch_exceptions=False
+    )
+    assert result.exit_code == 0
+    assert "Getting vulnerability policies" in result.output
+    assert "Invalid value for '--show-all'" not in result.output
+
+    # Conflict: show-all with explicit page
+    conflict = runner.invoke(
+        ls,
+        args=[organization, "--show-all", "--page-size", "2"],
+        catch_exceptions=False,
+    )
+    assert conflict.exit_code != 0
+    assert "Invalid value for '--show-all'" in conflict.output
+    assert "Cannot be used with --page (-p) or --page-size (-l)." in conflict.output
+
     # Change the values in the config file
     policy_config_file_path = create_vulnerability_policy_config_file(
         directory=tmp_path,

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/tests/commands/test_package_commands.py

@@ -50,6 +50,13 @@ def test_push_and_delete_raw_package(
     small_file_data = data[0]
     assert small_file_data["filename"] == pkg_file.name
 
+    # List packages with --show-all flag
+    result = runner.invoke(
+        list_, args=["pkgs", org_repo, "--show-all"], catch_exceptions=False
+    )
+    assert "Getting list of packages ... OK" in result.output
+    assert "Results: 1 package retrieved" in result.output
+
     # Wait for the package to sync.
     org_repo_package = f"{org_repo}/{small_file_data['slug']}"
     for _ in range(10):

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/tests/commands/test_repos.py

@@ -2,6 +2,7 @@ import json
 
 import pytest
 
+from ...commands.list_ import repos as list_repos
 from ...commands.repos import create, delete, get, update
 from ..utils import random_str
 
@@ -119,6 +120,21 @@ def test_repos_commands(runner, organization, tmp_path):
         result.output, organization, repo_config_file_path
     )
 
+    # Demonstrate list repos with --show-all succeeds (no pagination args).
+    result = runner.invoke(
+        list_repos, [organization, "--show-all"], catch_exceptions=False
+    )
+    assert result.exit_code == 0
+    assert "Getting list of repositories ... OK" in result.output
+
+    # Show that --show-all with an explicit page conflicts.
+    conflict = runner.invoke(
+        list_repos, [organization, "--show-all", "--page", "2"], catch_exceptions=False
+    )
+    assert conflict.exit_code != 0
+    assert "Invalid value for '--show-all'" in conflict.output
+    assert "Cannot be used with --page (-p) or --page-size (-l)." in conflict.output
+
     # Change the repository description in the repo config file.
     repository_description = random_str()
     repo_config_file_path = create_repo_config_file(
@@ -134,6 +150,14 @@ def test_repos_commands(runner, organization, tmp_path):
         update, [owner_slash_repo, str(repo_config_file_path)], catch_exceptions=False
     )
     assert result.exit_code == 0
+    assert (
+        "Updating "
+        + repository_slug
+        + " repository in the "
+        + organization
+        + " namespace ...OK"
+        in result.output
+    )
     assert "Results: 1 repository visible" in result.output
     assert_output_is_equal_to_repo_config(
         result.output, organization, repo_config_file_path

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/tests/commands/test_upstream.py

@@ -71,6 +71,36 @@ def test_upstream_commands(
     assert result_data["name"] == upstream_config["name"]
     assert result_data["upstream_url"] == upstream_config["upstream_url"]
 
+    # Minimal show-all success (no pagination args besides flag)
+    show_all = runner.invoke(
+        upstream,
+        args=[upstream_format, "ls", org_repo, "--show-all", "-F", "json"],
+        catch_exceptions=False,
+    )
+    assert show_all.exit_code == 0
+    show_all_data = json.loads(show_all.output)["data"]
+    assert len(show_all_data) == 1  # Should return the same single upstream
+    assert "Invalid value for '--show-all'" not in show_all.output
+
+    # Conflict: show-all with explicit page number
+    conflict = runner.invoke(
+        upstream,
+        args=[
+            upstream_format,
+            "ls",
+            org_repo,
+            "--show-all",
+            "--page",
+            "1",
+            "-F",
+            "json",
+        ],
+        catch_exceptions=False,
+    )
+    assert conflict.exit_code != 0
+    assert "Invalid value for '--show-all'" in conflict.output
+    assert "Cannot be used with --page (-p) or --page-size (-l)." in conflict.output
+
     slug_perm = result_data["slug_perm"]
     assert slug_perm
     org_repo_slug_perm = f"{org_repo}/{slug_perm}"

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/utils.py

@@ -19,32 +19,38 @@ def make_user_agent(prefix=None):
     return f"cloudsmith-cli/{prefix} cli:{get_cli_version()} api:{get_api_version()}"
 
 
-def pretty_print_list_info(num_results, page_info=None, suffix=None):
-    """Pretty print list info, with pagination, for user display."""
-    num_results_fg = "green" if num_results else "red"
-    num_results_text = click.style(str(num_results), fg=num_results_fg)
-
-    if page_info and page_info.is_valid:
-        page_range = page_info.calculate_range(num_results)
-        page_info_text = f"page: {click.style(str(page_info.page), bold=True)}/{click.style(str(page_info.page_total), bold=True)}, page size: {click.style(str(page_info.page_size), bold=True)}"
-        range_results_text = "%(from)s-%(to)s (%(num_results)s) of %(total)s" % {
-            "num_results": num_results_text,
-            "from": click.style(str(page_range[0]), fg=num_results_fg),
-            "to": click.style(str(page_range[1]), fg=num_results_fg),
-            "total": click.style(str(page_info.count), fg=num_results_fg),
-        }
+def pretty_print_list_info(num_results, page_info=None, suffix="", show_all=False):
+    """Print information about list results."""
+    if show_all:
+        click.echo(
+            "Results: %(num_results)d %(suffix)s"
+            % {
+                "num_results": num_results,
+                "suffix": suffix,
+            }
+        )
+    elif page_info and page_info.page is not None and page_info.page_size is not None:
+        start = (page_info.page - 1) * page_info.page_size + 1
+        end = min(start + num_results - 1, page_info.count or 0)
+        click.echo(
+            "Results: %(start)d-%(end)d (%(count)d) of %(total)d %(suffix)s "
+            "(page: %(page)d/%(pages)d, page size: %(page_size)d)"
+            % {
+                "start": start,
+                "end": end,
+                "count": num_results,
+                "total": page_info.count or 0,
+                "suffix": suffix,
+                "page": page_info.page,
+                "pages": page_info.page_total or 1,
+                "page_size": page_info.page_size,
+            }
+        )
     else:
-        page_info_text = ""
-        range_results_text = num_results_text
-
-    click.secho(
-        "Results: %(range_results)s %(suffix)s%(page_info)s"
-        % {
-            "range_results": range_results_text,
-            "page_info": " (%s)" % page_info_text if page_info_text else "",
-            "suffix": suffix or "item(s)",
-        }
-    )
+        click.echo(
+            "Results: %(num_results)d %(suffix)s"
+            % {"num_results": num_results, "suffix": suffix}
+        )
 
 
 def fmt_datetime(value):

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/cli/validators.py

@@ -4,6 +4,7 @@ import base64
 from datetime import datetime
 
 import click
+from click.core import ParameterSource
 
 from .types import ExpandPath
 
@@ -157,6 +158,40 @@ def validate_page_size(ctx, param, value):
     return value
 
 
+def enforce_show_all_exclusive(ctx):
+    """Order-independent mutual exclusivity check for pagination options.
+
+    Raises click.BadParameter bound to the --show-all option if it was used
+    together with explicit --page or --page-size. "Explicit" means supplied
+    via command line, environment variable, or prompt (Click ParameterSource).
+    """
+    show_all = ctx.params.get("show_all")
+    if not show_all:
+        return
+
+    explicit_sources = {
+        src
+        for src in (
+            ParameterSource.COMMANDLINE,
+            ParameterSource.ENVIRONMENT,
+            getattr(ParameterSource, "PROMPT", None),
+        )
+        if src is not None
+    }
+
+    page_explicit = ctx.get_parameter_source("page") in explicit_sources
+    size_explicit = ctx.get_parameter_source("page_size") in explicit_sources
+
+    if page_explicit or size_explicit:
+        show_all_param = next(
+            (p for p in ctx.command.params if p.name == "show_all"), None
+        )
+        raise click.BadParameter(
+            "Cannot be used with --page (-p) or --page-size (-l).",
+            param=show_all_param,
+        )
+
+
 def validate_optional_timestamp(ctx, param, value):
     """Ensure that a valid value for a timestamp is used."""
 

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4}/cloudsmith_cli/core/pagination.py

@@ -1,5 +1,9 @@
 """Core pagination utilities."""
 
+from typing import Any, Callable, List, Optional, Sequence, Tuple
+
+MAX_PAGE_SIZE = 1000
+
 
 class PageInfo:
     """Data for pagination results."""
@@ -73,3 +77,54 @@ class PageInfo:
             info.page_total = int(headers["X-Pagination-PageTotal"])
 
         return info
+
+
+def paginate_results(
+    api_function: Callable[..., Tuple[Sequence[Any], PageInfo]],
+    show_all: bool,
+    page: int,
+    page_size: int = MAX_PAGE_SIZE,
+    **kwargs: Any,
+) -> Tuple[List[Any], PageInfo]:
+    """Retrieve paginated results.
+
+    Behaviour:
+    - If ``show_all`` is False: perform a single paged request and return the
+      results plus the (possibly invalid) ``PageInfo``. Single-resource API
+      endpoints frequently omit pagination headers; we tolerate that here.
+    - If ``show_all`` is True: iterate all pages requesting ``MAX_PAGE_SIZE``.
+      Missing pagination headers during aggregation are treated as a user
+      misuse (e.g. attempting ``--show-all`` against a single-resource
+      endpoint) and raise a ``click.ClickException`` for consistent UX.
+
+    Raises:
+        click.ClickException: If pagination headers are absent while trying to
+            aggregate multiple pages with ``show_all``.
+    """
+    if not show_all:
+        results, page_info = api_function(page=page, page_size=page_size, **kwargs)
+        # For single resource endpoints (e.g. repos_read) pagination headers may be absent.
+        # In that case we return the results with potentially invalid page_info (empty when serialized)
+        # rather than raising. Downstream pretty printers handle an invalid page_info gracefully.
+        return list(results), page_info
+
+    all_results: List[Any] = []
+    current_page = 1
+    last_page_info: Optional[PageInfo] = None
+    while True:
+        page_results, last_page_info = api_function(
+            page=current_page, page_size=MAX_PAGE_SIZE, **kwargs
+        )
+        if not last_page_info.is_valid:
+            # No pagination headers (single-resource endpoint). Treat as single page.
+            # Return accumulated results without raising; command-level validators
+            # handle misuse of --show-all with single-resource endpoints.
+            all_results.extend(page_results)
+            return all_results, last_page_info
+        all_results.extend(page_results)
+
+        if current_page >= last_page_info.page_total:
+            break
+        current_page += 1
+
+    return all_results, last_page_info

cloudsmith_cli-1.9.4/cloudsmith_cli/data/VERSION

@@ -0,0 +1 @@
+1.9.4

{cloudsmith_cli-1.9.2 → cloudsmith_cli-1.9.4/cloudsmith_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cloudsmith-cli
-Version: 1.9.2
+Version: 1.9.4
 Summary: Cloudsmith Command-Line Interface (CLI)
 Home-page: https://github.com/cloudsmith-io/cloudsmith-cli
 Author: Cloudsmith Ltd

cloudsmith_cli-1.9.2/cloudsmith_cli/data/VERSION

@@ -1 +0,0 @@
-1.9.2