Skip to content

Conversation

@tmeckel
Copy link
Owner

@tmeckel tmeckel commented Oct 19, 2025

Closes: #78

…oduces FindGroupsByDisplayName on the extension client to return hydrated graph groups with case-insensitive matching. This unlocks the security group commands to resolve descriptors without reimplementing query logic.
…ndGroupsByDisplayName stub so tests can configure the extensions client when resolving security groups. Without this extension, the updated commands would panic when the mock is injected and the helper is called.
…tion\n\nCreates reusable parsing and descriptor utilities so all security group commands share consistent validation. This common layer eliminates duplicated parsing code and ensures descriptors resolve through the extensions client.
…tches scope parsing to the shared helpers and runs graph mutations with the command context. This keeps create aligned with the new resolution flow and reduces ad-hoc string handling.
…places inline scope parsing with the shared utilities and routes deletions through the extensions client lookup. The command now provides consistent progress feedback and descriptor handling, preventing accidental deletions when multiple groups share a name.
… command to the security group parent so users can discover it from the CLI. This wires the implementation into the command tree and exposes the functionality.
…ves organization and descriptor derivation to the shared helpers and starts the progress indicator earlier. This keeps the listing command consistent with the new architecture and reduces repeated code.
…s\n\nUses the shared target parsing helpers and descriptor resolution so membership listing aligns with other commands. This change ensures the command can correctly disambiguate groups across projects.
…he show subcommand with JSON and table output, leveraging the shared group finder. Users can now inspect individual security groups without manual API queries.
…ommand synopsis to the aggregated help reference so users can discover invocation details. The update mirrors the CLI behaviour introduced in this iteration.
… the group command overview with a link to the show documentation. This ensures navigation reflects the newly available command.
…e dedicated help topic describing arguments, examples, and formatting options for the show command. The page aligns with other generated command docs.
@tmeckel tmeckel merged commit 08ccf33 into master Oct 19, 2025
1 check passed
@tmeckel tmeckel deleted the tmeckel/issue78 branch October 19, 2025 16:03
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.

feat: Implement azdo security group show command

1 participant