Skip to content

fix(zsh): filter provider list to show only llm providers#2098

Merged
tusharmath merged 7 commits intomainfrom
fix/zsh-provider-list
Dec 9, 2025
Merged

fix(zsh): filter provider list to show only llm providers#2098
tusharmath merged 7 commits intomainfrom
fix/zsh-provider-list

Conversation

@amitksingh1490
Copy link
Copy Markdown
Contributor

@amitksingh1490 amitksingh1490 commented Dec 9, 2025

:provider

  • before
image
  • after
image

:login

  • before
image
  • after
image

@github-actions github-actions Bot added the type: fix Iterations on existing features or infrastructure. label Dec 9, 2025
@amitksingh1490 amitksingh1490 marked this pull request as ready for review December 9, 2025 06:36
Comment thread shell-plugin/lib/actions/provider.zsh Outdated
# Preserve the header line and filter the rest by TYPE column
local header=$(echo "$output" | head -n 1)
local filtered=$(echo "$output" | tail -n +2 | awk -v type="$filter_type" '$4 == type')
if [[ -z "$filtered" ]]; then
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

filter can be optional, if not specified, we can list everything.

@tusharmath tusharmath merged commit 9ba2435 into main Dec 9, 2025
7 checks passed
@tusharmath tusharmath deleted the fix/zsh-provider-list branch December 9, 2025 08:09
tusharmath added a commit that referenced this pull request Dec 9, 2025
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
laststylebender14 pushed a commit that referenced this pull request Dec 16, 2025
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: fix Iterations on existing features or infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants