fix: drain stdout pipe to prevent project listing deadlock by andrew-kline · Pull Request #106 · googleworkspace/cli

andrew-kline · 2026-03-05T05:20:33Z

Description

The list_gcloud_projects() function piped stdout from gcloud projects list but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout.

Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion.

Projects in a massive organization now list correctly.

Fixes#96

Checklist:

My code follows the AGENTS.md guidelines (no generated google-* crates).
I have run cargo fmt --all to format the code perfectly.
I have run cargo clippy -- -D warnings and resolved all warnings.
I have added tests that prove my fix is effective or that my feature works.
I have provided a Changeset file (e.g. via pnpx changeset) to document my changes.

gemini-code-assist · 2026-03-05T05:20:37Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

changeset-bot · 2026-03-05T05:20:40Z

🦋 Changeset detected

Latest commit: 7018988

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@googleworkspace/cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

google-cla · 2026-03-05T05:20:43Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

jpoehnelt · 2026-03-05T05:22:37Z

Thanks, just need the CLA and a changeset.

codecov · 2026-03-05T05:23:49Z

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.96%. Comparing base (99a6284) to head (7018988).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/setup.rs	0.00%	7 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## main #106 +/- ##
=======================================
Coverage 54.95% 54.96% =======================================
Files 37 37 Lines 12218 12216 -2 =======================================
Hits 6715 6715 + Misses 5503 5501 -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:30:41Z

Thanks, just need the CLA and a changeset.

done, thanks!

…kspace#106) The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:20:33Z

Description

The list_gcloud_projects() function piped stdout from gcloud projects list but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout.

Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion.

Projects in a massive organization now list correctly.

Fixes#96

Checklist:

My code follows the AGENTS.md guidelines (no generated google-* crates).
I have run cargo fmt --all to format the code perfectly.
I have run cargo clippy -- -D warnings and resolved all warnings.
I have added tests that prove my fix is effective or that my feature works.
I have provided a Changeset file (e.g. via pnpx changeset) to document my changes.

gemini-code-assist · 2026-03-05T05:20:37Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

changeset-bot · 2026-03-05T05:20:40Z

🦋 Changeset detected

Latest commit: 7018988

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@googleworkspace/cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

google-cla · 2026-03-05T05:20:43Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

jpoehnelt · 2026-03-05T05:22:37Z

Thanks, just need the CLA and a changeset.

codecov · 2026-03-05T05:23:49Z

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.96%. Comparing base (99a6284) to head (7018988).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/setup.rs	0.00%	7 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## main #106 +/- ##
=======================================
Coverage 54.95% 54.96% =======================================
Files 37 37 Lines 12218 12216 -2 =======================================
Hits 6715 6715 + Misses 5503 5501 -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:30:41Z

Thanks, just need the CLA and a changeset.

done, thanks!

…kspace#106) The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:20:33Z

Description

The list_gcloud_projects() function piped stdout from gcloud projects list but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout.

Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion.

Projects in a massive organization now list correctly.

Fixes#96

Checklist:

My code follows the AGENTS.md guidelines (no generated google-* crates).
I have run cargo fmt --all to format the code perfectly.
I have run cargo clippy -- -D warnings and resolved all warnings.
I have added tests that prove my fix is effective or that my feature works.
I have provided a Changeset file (e.g. via pnpx changeset) to document my changes.

gemini-code-assist · 2026-03-05T05:20:37Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

changeset-bot · 2026-03-05T05:20:40Z

🦋 Changeset detected

Latest commit: 7018988

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@googleworkspace/cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

google-cla · 2026-03-05T05:20:43Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

jpoehnelt · 2026-03-05T05:22:37Z

Thanks, just need the CLA and a changeset.

codecov · 2026-03-05T05:23:49Z

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.96%. Comparing base (99a6284) to head (7018988).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/setup.rs	0.00%	7 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## main #106 +/- ##
=======================================
Coverage 54.95% 54.96% =======================================
Files 37 37 Lines 12218 12216 -2 =======================================
Hits 6715 6715 + Misses 5503 5501 -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:30:41Z

Thanks, just need the CLA and a changeset.

done, thanks!

…kspace#106) The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:20:33Z

Description

The list_gcloud_projects() function piped stdout from gcloud projects list but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout.

Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion.

Projects in a massive organization now list correctly.

Fixes#96

Checklist:

My code follows the AGENTS.md guidelines (no generated google-* crates).
I have run cargo fmt --all to format the code perfectly.
I have run cargo clippy -- -D warnings and resolved all warnings.
I have added tests that prove my fix is effective or that my feature works.
I have provided a Changeset file (e.g. via pnpx changeset) to document my changes.

gemini-code-assist · 2026-03-05T05:20:37Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

changeset-bot · 2026-03-05T05:20:40Z

🦋 Changeset detected

Latest commit: 7018988

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@googleworkspace/cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

google-cla · 2026-03-05T05:20:43Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

jpoehnelt · 2026-03-05T05:22:37Z

Thanks, just need the CLA and a changeset.

codecov · 2026-03-05T05:23:49Z

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.96%. Comparing base (99a6284) to head (7018988).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/setup.rs	0.00%	7 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## main #106 +/- ##
=======================================
Coverage 54.95% 54.96% =======================================
Files 37 37 Lines 12218 12216 -2 =======================================
Hits 6715 6715 + Misses 5503 5501 -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:30:41Z

Thanks, just need the CLA and a changeset.

done, thanks!

…kspace#106) The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:20:33Z

Description

The list_gcloud_projects() function piped stdout from gcloud projects list but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout.

Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion.

Projects in a massive organization now list correctly.

Fixes#96

Checklist:

My code follows the AGENTS.md guidelines (no generated google-* crates).
I have run cargo fmt --all to format the code perfectly.
I have run cargo clippy -- -D warnings and resolved all warnings.
I have added tests that prove my fix is effective or that my feature works.
I have provided a Changeset file (e.g. via pnpx changeset) to document my changes.

gemini-code-assist · 2026-03-05T05:20:37Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

changeset-bot · 2026-03-05T05:20:40Z

🦋 Changeset detected

Latest commit: 7018988

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@googleworkspace/cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

google-cla · 2026-03-05T05:20:43Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

jpoehnelt · 2026-03-05T05:22:37Z

Thanks, just need the CLA and a changeset.

codecov · 2026-03-05T05:23:49Z

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.96%. Comparing base (99a6284) to head (7018988).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/setup.rs	0.00%	7 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## main #106 +/- ##
=======================================
Coverage 54.95% 54.96% =======================================
Files 37 37 Lines 12218 12216 -2 =======================================
Hits 6715 6715 + Misses 5503 5501 -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:30:41Z

Thanks, just need the CLA and a changeset.

done, thanks!

…kspace#106) The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:20:33Z

Description

The list_gcloud_projects() function piped stdout from gcloud projects list but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout.

Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion.

Projects in a massive organization now list correctly.

Fixes#96

Checklist:

My code follows the AGENTS.md guidelines (no generated google-* crates).
I have run cargo fmt --all to format the code perfectly.
I have run cargo clippy -- -D warnings and resolved all warnings.
I have added tests that prove my fix is effective or that my feature works.
I have provided a Changeset file (e.g. via pnpx changeset) to document my changes.

gemini-code-assist · 2026-03-05T05:20:37Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

changeset-bot · 2026-03-05T05:20:40Z

🦋 Changeset detected

Latest commit: 7018988

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@googleworkspace/cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

google-cla · 2026-03-05T05:20:43Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

jpoehnelt · 2026-03-05T05:22:37Z

Thanks, just need the CLA and a changeset.

codecov · 2026-03-05T05:23:49Z

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.96%. Comparing base (99a6284) to head (7018988).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/setup.rs	0.00%	7 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## main #106 +/- ##
=======================================
Coverage 54.95% 54.96% =======================================
Files 37 37 Lines 12218 12216 -2 =======================================
Hits 6715 6715 + Misses 5503 5501 -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:30:41Z

Thanks, just need the CLA and a changeset.

done, thanks!

…kspace#106) The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:20:33Z

Description

The list_gcloud_projects() function piped stdout from gcloud projects list but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout.

Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion.

Projects in a massive organization now list correctly.

Fixes#96

Checklist:

My code follows the AGENTS.md guidelines (no generated google-* crates).
I have run cargo fmt --all to format the code perfectly.
I have run cargo clippy -- -D warnings and resolved all warnings.
I have added tests that prove my fix is effective or that my feature works.
I have provided a Changeset file (e.g. via pnpx changeset) to document my changes.

gemini-code-assist · 2026-03-05T05:20:37Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

changeset-bot · 2026-03-05T05:20:40Z

🦋 Changeset detected

Latest commit: 7018988

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@googleworkspace/cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

google-cla · 2026-03-05T05:20:43Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

jpoehnelt · 2026-03-05T05:22:37Z

Thanks, just need the CLA and a changeset.

codecov · 2026-03-05T05:23:49Z

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.96%. Comparing base (99a6284) to head (7018988).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/setup.rs	0.00%	7 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## main #106 +/- ##
=======================================
Coverage 54.95% 54.96% =======================================
Files 37 37 Lines 12218 12216 -2 =======================================
Hits 6715 6715 + Misses 5503 5501 -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:30:41Z

Thanks, just need the CLA and a changeset.

done, thanks!

…kspace#106) The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:20:33Z

Description

The list_gcloud_projects() function piped stdout from gcloud projects list but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout.

Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion.

Projects in a massive organization now list correctly.

Fixes#96

Checklist:

My code follows the AGENTS.md guidelines (no generated google-* crates).
I have run cargo fmt --all to format the code perfectly.
I have run cargo clippy -- -D warnings and resolved all warnings.
I have added tests that prove my fix is effective or that my feature works.
I have provided a Changeset file (e.g. via pnpx changeset) to document my changes.

gemini-code-assist · 2026-03-05T05:20:37Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

changeset-bot · 2026-03-05T05:20:40Z

🦋 Changeset detected

Latest commit: 7018988

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@googleworkspace/cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

google-cla · 2026-03-05T05:20:43Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

jpoehnelt · 2026-03-05T05:22:37Z

Thanks, just need the CLA and a changeset.

codecov · 2026-03-05T05:23:49Z

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.96%. Comparing base (99a6284) to head (7018988).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/setup.rs	0.00%	7 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## main #106 +/- ##
=======================================
Coverage 54.95% 54.96% =======================================
Files 37 37 Lines 12218 12216 -2 =======================================
Hits 6715 6715 + Misses 5503 5501 -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline · 2026-03-05T05:30:41Z

Thanks, just need the CLA and a changeset.

done, thanks!

…kspace#106) The list_gcloud_projects() function piped stdout from `gcloud projects list` but only read it after the child process exited. When a user has enough GCP projects that the output exceeds the OS pipe buffer (~64KB), gcloud blocks on write, the parent blocks waiting for exit, and neither side makes progress — hitting the 10s timeout. Spawn a thread to drain stdout concurrently so the pipe buffer never fills up while the main thread polls for process completion. Fixesgoogleworkspace#96

andrew-kline force-pushed the fix-drain-stdout-pipe-to-prevent branch from b3e9ccd to 7018988Compare March 5, 2026 05:29

jpoehnelt approved these changes Mar 5, 2026
View reviewed changes

jpoehnelt merged commit a1be14f into googleworkspace:mainMar 5, 2026
22 checks passed

andrew-kline deleted the fix-drain-stdout-pipe-to-prevent branch April 11, 2026 00:37