Skip to content

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

Merged
anisaoshafi merged 1 commit into
mainfrom
devx-1002-lstk-terraform-help-should-work-correctly-even-if-terraform
Jul 20, 2026
Merged

Proxy commands --help/-h fails without a running emulator#392
anisaoshafi merged 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
@anisaoshafi
anisaoshafi merged commit 5810508 into main Jul 20, 2026
25 of 42 checks passed
@anisaoshafi
anisaoshafi deleted the devx-1002-lstk-terraform-help-should-work-correctly-even-if-terraform branch July 20, 2026 13:39
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.

2 participants