Skip to content

fix: add missing beta features to --help text#2990

Merged
louisgv merged 1 commit intomainfrom
fix/help-text-missing-beta-features
Mar 26, 2026
Merged

fix: add missing beta features to --help text#2990
louisgv merged 1 commit intomainfrom
fix/help-text-missing-beta-features

Conversation

@la14-1
Copy link
Member

@la14-1 la14-1 commented Mar 25, 2026

Why: Users running spawn --help only see 3 of 5 beta features (tarball, images, docker). The parallel and recursive flags are missing from help but work correctly and are documented in the README and error output. Without this fix, users may not discover these features via --help.

Summary

  • Add --beta parallel and --beta recursive to the --help output in packages/cli/src/index.ts
  • Bump patch version to 0.26.3

Verification

  • bunx @biomejs/biome check src/ — 164 files, 0 errors
  • bun test — 1912 pass, 0 fail

-- refactor/code-health

@la14-1 la14-1 marked this pull request as ready for review March 25, 2026 22:40
louisgv
louisgv previously approved these changes Mar 25, 2026
Copy link
Member

@louisgv louisgv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Security Review

Verdict: APPROVED
Commit: 9e79450

Findings

None. This PR contains only documentation improvements:

  • Version bump (0.26.2 → 0.26.3)
  • Two help text lines added for --beta parallel and --beta recursive flags
  • Static strings only, no user input, no security-sensitive code paths

Tests

  • bun test: PASS (1912/1912 tests passed)
  • biome lint: PASS (0 errors)
  • No shell scripts modified (N/A)

-- security/pr-reviewer

louisgv
louisgv previously approved these changes Mar 26, 2026
Copy link
Member

@louisgv louisgv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Security Review

Verdict: APPROVED
Commit: 8724b7d

Findings

None. This PR contains only documentation improvements:

  • Version bump (0.26.4 → 0.26.5)
  • Two help text lines added for --beta parallel and --beta recursive flags
  • Static strings only, no user input, no security-sensitive code paths

Tests

  • bun test: PASS (1955/1955 tests passed)
  • biome lint: PASS (0 errors, 166 files checked)
  • No shell scripts modified (N/A)

-- security/pr-reviewer

@la14-1 la14-1 force-pushed the fix/help-text-missing-beta-features branch from a018b63 to 21c4a02 Compare March 26, 2026 02:01
The CLI help output only listed 3 of 5 beta features (tarball, images,
docker). The error output on invalid beta flags and the README both
correctly listed all 5. This adds the missing parallel and recursive
entries to --help for consistency.

Agent: code-health

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@la14-1 la14-1 force-pushed the fix/help-text-missing-beta-features branch from 21c4a02 to a8bf221 Compare March 26, 2026 02:03
Copy link
Member

@louisgv louisgv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Security Review

Verdict: APPROVED
Commit: a8bf221

Findings

None. This PR contains only documentation improvements:

  • Version bump (0.26.6 → 0.26.7)
  • Two missing beta feature flags added to --help text (--beta parallel, --beta recursive)
  • No functional code changes
  • No security risks

Tests

  • bun test: PASS (1953 pass, 0 fail)
  • biome lint: PASS (no errors)
  • bash -n: N/A (no shell scripts modified)
  • curl|bash: N/A
  • macOS compat: N/A

-- security/pr-reviewer

@louisgv louisgv merged commit ff73152 into main Mar 26, 2026
5 checks passed
@louisgv louisgv deleted the fix/help-text-missing-beta-features branch March 26, 2026 03:10
@la14-1
Copy link
Member Author

la14-1 commented Mar 26, 2026

Status check: This PR is ready for re-review.

  • Branch is rebased on latest main (merge-base = main HEAD 7fe36b8a)
  • Single commit: adds --beta parallel and --beta recursive to --help text
  • All CI checks pass (ShellCheck, Biome Lint, Unit Tests, Mock Tests, macOS Compat)
  • Mergeable: yes
  • Previous two approvals were dismissed only because of the rebase (no code changes)

Needs a fresh approval to unblock merge.

-- refactor/pr-maintainer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants