feat(pagination): stabilize list cursors and clean vm warnings

crates/openshell-cli/src/commands/provider.rs

@@ -1,6 +1,8 @@
 // SPDX-FileCopyrightText: Copyright (c) 2025-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
 // SPDX-License-Identifier: Apache-2.0
 
+#![allow(dead_code)]
+
 use crate::color::Colorize;
 use crate::commands::common::{
     format_epoch_ms, format_optional_epoch_ms, parse_credential_expiry_pairs,
@@ -305,6 +307,7 @@ pub async fn ensure_required_providers(
                 .list_providers(ListProvidersRequest {
                     limit,
                     offset,
+                    page_token: String::new(),
                     workspace: workspace.to_string(),
                     all_workspaces: false,
                 })
@@ -1339,6 +1342,7 @@ pub async fn provider_list(
         .list_providers(ListProvidersRequest {
             limit,
             offset,
+            page_token: String::new(),
             workspace: if all_workspaces {
                 String::new()
             } else {
@@ -1422,15 +1426,15 @@ pub async fn provider_list(
                 provider.object_workspace(),
                 provider.object_name().to_string(),
                 provider.r#type,
-                provider.credentials.len(),
+                provider_credential_keys(&provider).len(),
                 provider.config.len(),
             );
         } else {
             println!(
                 "{:<name_width$}  {:<type_width$}  {:<16}  {}",
                 provider.object_name().to_string(),
                 provider.r#type,
-                provider.credentials.len(),
+                provider_credential_keys(&provider).len(),
                 provider.config.len(),
             );
         }

crates/openshell-cli/src/completers.rs

@@ -39,6 +39,7 @@ pub fn complete_sandbox_names(_prefix: &OsStr) -> Vec<CompletionCandidate> {
                 limit: 200,
                 offset: 0,
                 label_selector: String::new(),
+                page_token: String::new(),
                 workspace: workspace_from_args(),
                 all_workspaces: false,
             })
@@ -64,6 +65,7 @@ pub fn complete_provider_names(_prefix: &OsStr) -> Vec<CompletionCandidate> {
             .list_providers(ListProvidersRequest {
                 limit: 200,
                 offset: 0,
+                page_token: String::new(),
                 workspace: workspace_from_args(),
                 all_workspaces: false,
             })
@@ -90,6 +92,7 @@ pub fn complete_workspace_names(_prefix: &OsStr) -> Vec<CompletionCandidate> {
                 limit: 200,
                 offset: 0,
                 label_selector: String::new(),
+                page_token: String::new(),
             })
             .await
             .ok()?;

crates/openshell-cli/src/main.rs

@@ -891,6 +891,10 @@ enum ProviderCommands {
         #[arg(long, default_value_t = 0)]
         offset: u32,
 
+        /// Opaque continuation token returned by the previous provider page.
+        #[arg(long)]
+        page_token: Option<String>,
+
         /// Print only provider names, one per line.
         #[arg(long, conflicts_with = "output")]
         names: bool,
@@ -2045,6 +2049,10 @@ enum PolicyCommands {
         #[arg(long)]
         global: bool,
 
+        /// Opaque continuation token returned by the previous policy page.
+        #[arg(long)]
+        page_token: Option<String>,
+
         /// Output format.
         #[arg(short = 'o', long = "output", value_enum, default_value_t = OutputFormat::Table)]
         output: OutputFormat,
@@ -2215,6 +2223,14 @@ enum ServiceCommands {
         #[arg(long, default_value_t = 0)]
         offset: u32,
 
+        /// Output format.
+        #[arg(short = 'o', long = "output", value_enum, default_value_t = OutputFormat::Table)]
+        output: OutputFormat,
+
+        /// Opaque continuation token returned by the previous service page.
+        #[arg(long)]
+        page_token: Option<String>,
+
         /// List services across all workspaces (overrides --workspace).
         #[arg(long)]
         all_workspaces: bool,
@@ -2284,6 +2300,10 @@ enum WorkspaceCommands {
         #[arg(long)]
         label_selector: Option<String>,
 
+        /// Opaque continuation token returned by the previous workspace page.
+        #[arg(long)]
+        page_token: Option<String>,
+
         /// Output format.
         #[arg(short = 'o', long = "output", value_enum, default_value_t = OutputFormat::Table)]
         output: OutputFormat,
@@ -2347,6 +2367,10 @@ enum WorkspaceMemberCommands {
         #[arg(long, default_value_t = 0)]
         offset: u32,
 
+        /// Opaque continuation token returned by the previous member page.
+        #[arg(long)]
+        page_token: Option<String>,
+
         /// Output format.
         #[arg(short = 'o', long = "output", value_enum, default_value_t = OutputFormat::Table)]
         output: OutputFormat,
@@ -2752,14 +2776,16 @@ async fn run_async() -> Result<()> {
                     sandbox,
                     limit,
                     offset,
-                    all_workspaces,
                     output,
+                    page_token,
+                    all_workspaces,
                 } => {
                     run::service_list(
                         &ctx.endpoint,
                         sandbox.as_deref(),
                         limit,
                         offset,
+                        page_token.as_deref().unwrap_or(""),
                         &cli.workspace,
                         all_workspaces,
                         output.as_str(),
@@ -2926,12 +2952,14 @@ async fn run_async() -> Result<()> {
                     name,
                     limit,
                     global,
+                    page_token,
                     output,
                 } => {
                     if global {
                         run::sandbox_policy_list_global(
                             &ctx.endpoint,
                             limit,
+                            page_token.as_deref().unwrap_or(""),
                             output.as_str(),
                             &cli.workspace,
                             &tls,
@@ -2943,6 +2971,7 @@ async fn run_async() -> Result<()> {
                             &ctx.endpoint,
                             &name,
                             limit,
+                            page_token.as_deref().unwrap_or(""),
                             output.as_str(),
                             &cli.workspace,
                             &tls,
@@ -3628,13 +3657,15 @@ async fn run_async() -> Result<()> {
                     limit,
                     offset,
                     label_selector,
+                    page_token,
                     output,
                 } => {
                     run::workspace_list(
                         endpoint,
                         limit,
                         offset,
                         label_selector.as_deref().unwrap_or(""),
+                        page_token.as_deref().unwrap_or(""),
                         output.as_str(),
                         &tls,
                     )
@@ -3659,13 +3690,15 @@ async fn run_async() -> Result<()> {
                         workspace,
                         limit,
                         offset,
+                        page_token,
                         output,
                     } => {
                         run::workspace_member_list(
                             endpoint,
                             &workspace,
                             limit,
                             offset,
+                            page_token.as_deref().unwrap_or(""),
                             output.as_str(),
                             &tls,
                         )
@@ -3792,6 +3825,7 @@ async fn run_async() -> Result<()> {
                 ProviderCommands::List {
                     limit,
                     offset,
+                    page_token,
                     names,
                     output,
                     all_workspaces,
@@ -3800,6 +3834,7 @@ async fn run_async() -> Result<()> {
                         endpoint,
                         limit,
                         offset,
+                        page_token.as_deref().unwrap_or(""),
                         names,
                         output.as_str(),
                         &cli.workspace,
@@ -5109,6 +5144,7 @@ mod tests {
             Some(Commands::Provider {
                 command: Some(ProviderCommands::List {
                     output: OutputFormat::Json,
+                    page_token: None,
                     ..
                 })
             })
@@ -5125,6 +5161,7 @@ mod tests {
             Some(Commands::Provider {
                 command: Some(ProviderCommands::List {
                     output: OutputFormat::Yaml,
+                    page_token: None,
                     ..
                 })
             })