Skip to content

Conversation

Copy link

Copilot AI commented Jan 16, 2026

The --list-enabled flag was silently ignored when combined with enable/disable subcommands due to conditional logic that only checked the flag when no subcommands were present.

Changes

  • Added yargs conflicts declarations to make --list-enabled mutually exclusive with enable/disable subcommands
  • Fixed missing spread operator in disablePlugin that would overwrite entire config instead of preserving fields
  • Added defensive null check for enabledPlugins array

Behavior

# Now produces clear error instead of silently ignoring flag
$ construct plugin enable foo --list-enabled
Arguments pluginName and list-enabled are mutually exclusive

# List command works independently
$ construct plugin --list-enabled
No plugins enabled.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits January 16, 2026 01:42
Co-authored-by: scaryrawr <661373+scaryrawr@users.noreply.github.com>
Co-authored-by: scaryrawr <661373+scaryrawr@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on adding plugin/marketplace commands Make --list-enabled mutually exclusive with enable/disable subcommands Jan 16, 2026
Copilot AI requested a review from scaryrawr January 16, 2026 01:49
@scaryrawr scaryrawr closed this Jan 16, 2026
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