feat: add --dry-run flag to toggle-on, toggle-off, and archive commands

[nieblara]

Adds --dry-run to the three hand-rolled flag commands (toggle-on, toggle-off, archive). When set, the API validates the patch without persisting it and returns a preview of the flag after the change would be applied.

The LaunchDarkly API already supports dryRun=true as a query parameter on PATCH /api/v2/flags/{projectKey}/{featureFlagKey}. The auto-generated ldcli flags update --dry-run true already works, but the hand-rolled commands passed nil for query params and had no way to send it.

Changes

• cmd/cliflags/flags.go — add DryRunFlag constant and description
• cmd/flags/toggle.go — register --dry-run boolean flag, build url.Values{"dryRun": ["true"]} when set, pass to MakeRequest
• cmd/flags/archive.go — same pattern
• internal/resources/mock_client.go — add Query url.Values field so tests can verify query params are passed
• cmd/flags/toggle_test.go — two new tests: --dry-run passes dryRun=true, omitting it passes nothing
• cmd/flags/archive_test.go — same two tests

Not in scope

• Segments — no hand-rolled segment commands; ldcli segments update already exposes --dry-run via codegen
• Workflows — in maintenance mode / planned for deprecation
• Output formatting — API returns the same flag preview shape for dry-run, so existing rendering works as-is

Made with Cursor

---

Note

Low Risk
Low risk: small, opt-in CLI flag that only changes request query parameters when explicitly set, plus test/mocking updates.

Overview
Adds a new --dry-run CLI flag for flags archive, flags toggle-on, and flags toggle-off that, when set, sends dryRun=true as a query parameter on the underlying PATCH /api/v2/flags/... request.

Updates the resources MockClient to record request query params and adds coverage to ensure dryRun is passed only when --dry-run is provided.

^{Reviewed by Cursor Bugbot for commit 5eb2a6f. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 5eb2a6f. Configure here.}

[cursor]

Dry-run plaintext output misleadingly says "Successfully updated"

Medium Severity

When --dry-run is set and plaintext output is used, CmdOutput is called with the hardcoded action "update", which produces the prefix "Successfully updated". Nothing was actually persisted — the API only validated and returned a preview. A user seeing "Successfully updated" after explicitly requesting --dry-run may believe the mutation was applied, undermining the purpose of the flag.

Additional Locations (1)

• cmd/flags/archive.go#L57-L61

 

^{Reviewed by Cursor Bugbot for commit 5eb2a6f. Configure here.}