fix(helpers): respect --format flag in sheets +append, +read, and docs +write

.changeset/helpers-respect-format-flag.md

@@ -0,0 +1,12 @@
+---
+"@googleworkspace/cli": patch
+---
+
+Fix `--format` flag being silently ignored in sheets and docs helpers
+
+`sheets +append`, `sheets +read`, and `docs +write` all hard-coded
+`OutputFormat::default()` (JSON) when calling `executor::execute_method`,
+meaning `--format table`, `--format yaml`, and `--format csv` had no effect.
+The handlers now read the global `--format` flag from `ArgMatches` and pass
+it through to the executor, consistent with how `calendar +agenda` and
+`gmail +triage` already behave.

crates/google-workspace-cli/src/helpers/docs.rs

@@ -68,6 +68,10 @@ TIPS:
         Box::pin(async move {
             if let Some(matches) = matches.subcommand_matches("+write") {
                 let (params_str, body_str, scopes) = build_write_request(matches, doc)?;
+                let output_format = matches
+                    .get_one::<String>("format")
+                    .map(|s| crate::formatter::OutputFormat::from_str(s))
+                    .unwrap_or_default();
 
                 let scope_strs: Vec<&str> = scopes.iter().map(|s| s.as_str()).collect();
                 let (token, auth_method) = match auth::get_token(&scope_strs).await {
@@ -104,7 +108,7 @@ TIPS:
                     &pagination,
                     None,
                     &crate::helpers::modelarmor::SanitizeMode::Warn,
-                    &crate::formatter::OutputFormat::default(),
+                    &output_format,
                     false,
                 )
                 .await?;

crates/google-workspace-cli/src/helpers/sheets.rs

@@ -112,6 +112,10 @@ TIPS:
             if let Some(matches) = matches.subcommand_matches("+append") {
                 let config = parse_append_args(matches);
                 let (params_str, body_str, scopes) = build_append_request(&config, doc)?;
+                let output_format = matches
+                    .get_one::<String>("format")
+                    .map(|s| crate::formatter::OutputFormat::from_str(s))
+                    .unwrap_or_default();
 
                 let scope_strs: Vec<&str> = scopes.iter().map(|s| s.as_str()).collect();
                 let (token, auth_method) = match auth::get_token(&scope_strs).await {
@@ -149,7 +153,7 @@ TIPS:
                     &pagination,
                     None,
                     &crate::helpers::modelarmor::SanitizeMode::Warn,
-                    &crate::formatter::OutputFormat::default(),
+                    &output_format,
                     false,
                 )
                 .await?;
@@ -160,6 +164,10 @@ TIPS:
             if let Some(matches) = matches.subcommand_matches("+read") {
                 let config = parse_read_args(matches);
                 let (params_str, scopes) = build_read_request(&config, doc)?;
+                let output_format = matches
+                    .get_one::<String>("format")
+                    .map(|s| crate::formatter::OutputFormat::from_str(s))
+                    .unwrap_or_default();
 
                 // Re-find method
                 let spreadsheets_res = doc.resources.get("spreadsheets").ok_or_else(|| {
@@ -192,7 +200,7 @@ TIPS:
                     &executor::PaginationConfig::default(),
                     None,
                     &crate::helpers::modelarmor::SanitizeMode::Warn,
-                    &crate::formatter::OutputFormat::default(),
+                    &output_format,
                     false,
                 )
                 .await?;