fix(cli): address policy get json clippy

Author: johntmyers

crates/openshell-cli/src/run.rs

@@ -5656,32 +5656,46 @@ pub async fn sandbox_policy_get(
     output: &str,
     tls: &TlsOptions,
 ) -> Result<()> {
-    let mut stdout = std::io::stdout().lock();
-    let mut stderr = std::io::stderr().lock();
+    let mut stdout = Vec::new();
+    let mut stderr = Vec::new();
     sandbox_policy_get_to_writer(
         server,
         name,
         version,
         full,
         output,
         tls,
-        &mut stdout,
-        &mut stderr,
+        (&mut stdout, &mut stderr),
     )
-    .await
+    .await?;
+
+    {
+        let mut terminal_stdout = std::io::stdout().lock();
+        terminal_stdout.write_all(&stdout).into_diagnostic()?;
+    }
+    {
+        let mut terminal_stderr = std::io::stderr().lock();
+        terminal_stderr.write_all(&stderr).into_diagnostic()?;
+    }
+
+    Ok(())
 }
 
 #[doc(hidden)]
-pub async fn sandbox_policy_get_to_writer<W: Write, E: Write>(
+pub async fn sandbox_policy_get_to_writer<W, E>(
     server: &str,
     name: &str,
     version: u32,
     full: bool,
     output: &str,
     tls: &TlsOptions,
-    stdout: &mut W,
-    stderr: &mut E,
-) -> Result<()> {
+    writers: (&mut W, &mut E),
+) -> Result<()>
+where
+    W: Write + Send,
+    E: Write + Send,
+{
+    let (stdout, stderr) = writers;
     let mut client = grpc_client(server, tls).await?;
 
     let status_resp = client

crates/openshell-cli/tests/sandbox_name_fallback_integration.rs

@@ -323,7 +323,7 @@ impl OpenShell for TestOpenShell {
 
         let policy = SandboxPolicy {
             version: 7,
-            network_policies: [(
+            network_policies: std::iter::once((
                 "api".to_string(),
                 NetworkPolicyRule {
                     name: "api".to_string(),
@@ -337,8 +337,7 @@ impl OpenShell for TestOpenShell {
                     }],
                     ..Default::default()
                 },
-            )]
-            .into_iter()
+            ))
             .collect(),
             ..Default::default()
         };
@@ -610,8 +609,7 @@ async fn policy_get_full_json_cli_prints_policy_payload() {
         true,
         "json",
         &ts.tls,
-        &mut stdout,
-        &mut stderr,
+        (&mut stdout, &mut stderr),
     )
     .await
     .expect("policy get should succeed");