Skip to content

Proxy commands --help/-h fails without a running emulator#392

Open
peter-smith-phd wants to merge 1 commit into
mainfrom
devx-1002-lstk-terraform-help-should-work-correctly-even-if-terraform
Open

Proxy commands --help/-h fails without a running emulator#392
peter-smith-phd wants to merge 1 commit into
mainfrom
devx-1002-lstk-terraform-help-should-work-correctly-even-if-terraform

Conversation

@peter-smith-phd

@peter-smith-phd peter-smith-phd commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

All the proxy commands (aws, terraform, cdk, and sam) incorrectly check that the emulator is up and running, before invoking the "help" command. Instead, they should pass the --help flag (or similar) directly to the underlying binary without the "emulator is running" check. In addition, terraform incorrectly requires that the current workspace already had terraform init performed. This creates an unnecessary amount of friction for users simply asking for help.

This PR treats help flags/commands as special, and skips this requirement.

Completes DEVX-1002

🤖 Generated with Claude Code

… emulator

DEVX-1002 — these proxy commands use DisableFlagParsing, so --help/-h flowed
through as ordinary args and got routed through the full provider-schema/
emulator-required path instead of straight to the underlying tool's help
output. Add IsHelp checks (aws/cdk also recognize the bare "help"
pseudo-subcommand; terraform does not, since `terraform help` is itself an
error) and short-circuit to the real binary before any Docker/emulator setup.
@peter-smith-phd peter-smith-phd changed the title Fix DEVX-1002: --help/-h fails without a running emulator across IaC proxy commands Proxy commands --help/-h fails without a running emulator Jul 16, 2026
@peter-smith-phd peter-smith-phd added semver: minor docs: skip Pull request does not require documentation changes labels Jul 16, 2026
@peter-smith-phd
peter-smith-phd marked this pull request as ready for review July 17, 2026 03:14
@peter-smith-phd
peter-smith-phd requested a review from a team as a code owner July 17, 2026 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs: skip Pull request does not require documentation changes semver: minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant