Skip to content

feat: unify group elevation into root command - #22

Merged
aaearon merged 1 commit into
mainfrom
feat/unified-groups
Feb 19, 2026
Merged

feat: unify group elevation into root command#22
aaearon merged 1 commit into
mainfrom
feat/unified-groups

Conversation

@aaearon

@aaearon aaearon commented Feb 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Merges group membership elevation into the root command — grant groups subcommand removed
  • Interactive selector now shows both cloud roles and Entra ID groups in a single unified list
  • grant favorites add interactive mode also shows the unified list, saving group selections as groups-type favorites
  • Adds --groups flag (filter to groups only) and --group/-g flag (direct group elevation)
  • Removes groupSelector interface and dead code in favor of unifiedSelector

Test plan

  • make test — all unit tests pass (16 files changed, new tests for unified selection in root + favorites)
  • make lint — clean
  • Existing favorites tests still pass (cloud selection, flag-based, persistence)
  • New test verifies group selection in favorites add creates groups-type favorite
  • Root command tests cover unified interactive, --groups filter, --group direct, group favorites

Merge group membership elevation into the root command instead of a
separate `grant groups` subcommand. The interactive selector now shows
both cloud roles and Entra ID groups in a single unified list.

- Add --groups flag to filter interactive selector to groups only
- Add --group/-g flag for direct group elevation
- Replace targetSelector + groupSelector with unifiedSelector in
  favorites add, showing both cloud and group items
- Remove grant groups subcommand (absorbed into root)
- Remove groupSelector interface and related dead code
- Add selectionItem tagged union and unified selection helpers
@aaearon
aaearon merged commit 6e0f7a5 into main Feb 19, 2026
1 check passed
@aaearon
aaearon deleted the feat/unified-groups branch February 19, 2026 17:05
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.

1 participant