Skip to content

feat: Add query parameter support to ls and acl ls commands#35

Merged
aviatco merged 37 commits intomicrosoft:mainfrom
aviatco:dev/aviatcohen/support-quering-ls-command
Dec 1, 2025
Merged

feat: Add query parameter support to ls and acl ls commands#35
aviatco merged 37 commits intomicrosoft:mainfrom
aviatco:dev/aviatcohen/support-quering-ls-command

Conversation

@aviatco
Copy link
Copy Markdown
Collaborator

@aviatco aviatco commented Oct 21, 2025

📥 Pull Request

✨ Description of new changes

Added a new -q/--query parameter to the ls and acl ls commands that allows users to specify which columns they want to display in the output. This enhancement provides more control over the output format and makes it easier to focus on specific properties.

This feature works across all resource types (workspace, item, OneLake etc...).
Use the -l flag to filter by fields that aren’t included in the default ls output.

Key Changes

  • Added -q/--query parameter to ls and acls ls command parsers
  • Updated ls and acls ls implementation to filter output columns based on query parameter and override -l flag when multiple fields are specified
  • Added comprehensive test coverage for query functionality
  • Updated documentation to reflect new capability

Examples

Basic usage with single field:

fab acl ls workspace1.workspace -q identity
fab fs ls -q name

Multiple fields:

fab acl ls workspace1.workspace -q identity name
fab fs ls -q name id

Testing

Added new test cases covering:

  • Single field queries
  • Multiple field queries
  • Invalid field handling
  • Query parameter interaction with different resource types (workspace, connection, gateway)
  • Documentation
  • Updated command reference in docs/commands/acls/index.md
  • Added example usage in docs/examples/acl_examples.md
  • Backwards Compatibility
  • This change is fully backwards compatible. The -q/--query parameter is optional, and existing behavior is preserved when not using the parameter.

@aviatco aviatco requested a review from a team as a code owner October 21, 2025 12:55
may-hartov
may-hartov previously approved these changes Nov 30, 2025
@aviatco aviatco merged commit db965a5 into microsoft:main Dec 1, 2025
9 checks passed
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.

3 participants