feat(ui): filter tags by name and description on the Tag Management page by galovics · Pull Request #42949 · BerriAI/litellm

galovics · 2026-09-24T13:31:43Z

TLDR

Problem this solves:

The Tag Management table has no way to narrow the list
Finding one tag among many means reading every row

How it solves it:

Adds Tag Name and Description filter inputs above the table
Both are case-insensitive "contains" matches and combine as AND
A filter that matches nothing shows "No matching tags"

User Flow

Before: an admin with many tags cannot narrow the Tag Management table, so finding one tag means reading down the list

The admin opens https://litellm-domain/ui/?page=tag-management and sees every tag in one table
They want the tags whose name contains "prod" or whose description mentions "billing"
There is no search box or filter anywhere on the page, so they read the rows by eye

After: the same admin narrows the list by typing

The admin opens https://litellm-domain/ui/?page=tag-management and sees every tag in one table
They type "prod" in the Tag Name filter and only tags whose name contains "prod" stay, whatever the letter case
They type "billing" in the Description filter and only tags matching both filters stay
A filter that matches nothing shows "No matching tags" instead of the "No tags yet" message

Relevant issues

Fixes #42946

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

I have added meaningful tests
The handful of test files covering my change pass locally, e.g. uv run pytest tests/test_litellm/<your_test_file>.py -v. Leave the suites (make test-unit-*, make test-unit) to CI: it finishes in ~15 minutes where a laptop takes an hour or more
My PR passes all required CI/CD checks (e.g., lint, schema.d.ts sync check, etc.)
My PR's scope is as isolated as possible; it only solves 1 specific problem
I have received a Greptile Confidence Score of at least 4/5 before requesting a maintainer review (Greptile reviews automatically once the PR is opened; only comment @greptileai to re-request a review after pushing changes)

Delays in PR merge?

If you're seeing a delay in your PR being merged, ping the LiteLLM Team on Slack (#pr-review).

Screenshots / Proof of Fix

No screenshots or live UI run are attached to this description. The Admin UI needs an interactive password login that was not available while preparing the PR, so the behavior is covered by component tests only. The 7 new cases in TagTable.test.tsx type into the two filters and check the rows the user sees: name filter ignoring case, match in the middle of the name, description filter, both filters together, clearing restores every row, a no-match filter shows "No matching tags" and not "No tags yet", and a tag without a description does not break the filter. npx vitest run "src/app/(dashboard)/tag-management" passes 4 files and 32 tests at 7ce5dde

Type

🆕 New Feature

Caveats (if any)

Medium

No before and after screenshots yet, the UI was verified through component tests only

Low

The filter inputs also show when there are no tags at all
Leading and trailing spaces are not trimmed, same as the shared table search
Filtering is client-side over the already loaded list, with no debounce

Final Attestation

The tests check the right things, including the edge cases, and regressions in the respective real-world customer use-cases are not possible after this PR

greptile-apps · 2026-09-24T13:35:15Z

Confidence Score: 5/5

The PR appears safe to merge; no actionable regression was identified.

Summary

Adds case-insensitive name and description filters to the Tag Management table, combining them as AND conditions. It also distinguishes an empty tag list from a filtered list with no matches and adds component tests for the filter behavior.

_{Reviews (1) · Last reviewed commit: "feat(ui): filter tags by name and descri..."}

galovics · 2026-09-24T15:25:15Z

CI is green and Greptile scored 5/5. @yuneng-berri @ryan-crabbe-berri could one of you review? Docs PR: BerriAI/litellm-docs#1684

galovics · 2026-09-29T17:57:16Z

Friendly bump: CI is green and Greptile scored 5/5. @yuneng-berri @ryan-crabbe-berri could one of you review when you have a moment?

ryan-crabbe-berri

LGTM; thanks!

…ject_key_prefix * upstream/main: (62 commits) fix(guardrails): scan Responses API input in Azure Prompt Shield (BerriAI#43786) feat(lens): investigate sampled traces and retain batch results (BerriAI#43942) fix(proxy): restore pre-config-wins handling of pass-through endpoints (BerriAI#43962) fix(cost-map): raise baseten DeepSeek-V4.1-Flash max output to 262144 (BerriAI#43916) chore(cost-map): add deprecation date for anthropic claude-sonnet-4-5 (BerriAI#43898) chore(cost-map): add fireworks inkling priority prices from the prices api (BerriAI#43949) feat(guardrails): honor litellm_params.timeout in every HTTP guardrail (BerriAI#43134) test(e2e): typed per-test metadata for the e2e suite (BerriAI#42044) fix(caching): write the response-cache SET to Redis at once instead of on the post-call batch (BerriAI#43973) feat(ui): filter tags by name and description on the Tag Management page (BerriAI#42949) feat(providers): add Cortecs as an OpenAI-compatible provider (BerriAI#43872) feat(e2e): record each e2e test's steps, starting with ProxyClient (BerriAI#42393) test(ci): repair stale tests and move retired OpenAI text-completion fixtures (BerriAI#43958) feat(proxy): record in spend logs whether a request used a client-forwarded Anthropic OAuth token (BerriAI#43063) fix(azure_storage): keep the DataLakeServiceClient alive until its TTL elapses (BerriAI#43082) chore(deps): bump gitpython and tornado, extend diskcache osv ignore to Nov 1 (BerriAI#43961) fix(guardrails): treat an unknown straiker api_version as unset instead of skipping the guardrail (BerriAI#43956) fix(azure_storage): name Data Lake objects without base64 padding or slashes (BerriAI#43914) fix(grayswan): send request conversation and tool calls to post-call monitor (BerriAI#43770) chore(cost-map): sync openrouter prices from the models API (BerriAI#43950) ...

#42949 added 'Filter by tag name' and 'Filter by description' inputs to the Tag Management page, so page-wide getByLabel('Tag Name') and getByLabel('Description') match two elements and Playwright's strict mode fails the create step

* test(ci): add used_client_oauth_token to the GCS pub/sub spend-log golden #43063 stamps used_client_oauth_token into spend-log metadata, so test_async_gcs_pub_sub_v1 failed on main with an extra metadata key * test(ui): give the auto-router threshold save wait room for the availability debounce #42625 keeps Save disabled while a 300ms-debounced availability check runs. This test waits for Save right after the change, so the whole debounce lands inside waitFor's 1s default and it times out under CI load. It is the recurring UI Unit Tests failure on main since #42625 landed * test(e2e): expect no pricing tier on bills for streamed calls OpenAI served at default #42870 added both the rule that a served default or standard tier bills at base pricing and records no service_tier, and streamed tests expecting the row to record 'default'. They have failed on every scheduled litellm-e2e run since. The tests now map the served tier to the pricing basis the bill must record and check input is billed at that basis's rate; the messages case registers custom rates so the rate check has something to compare against * test(e2e-ui): wait for the call-id search before hovering the logs row The row the spec hovers is already on the unfiltered first page, so it was found before the search request returned. The search response then re-rendered the table under the mouse, and the Base UI tooltip never opened. Reproduced with Playwright against a local proxy: hovering right after the fill never shows the tooltip, hovering after the search response shows the call id every time * test(e2e): run the Together structured-output case on the hybrid Qwen with reasoning off The case picked the cheapest Together row flagged supports_response_schema. DeepSeek-V4-Flash-0731 hit its cost-map deprecation date on 2026-09-29, so the pick moved to GLM-5.3-Flash, a reasoning-only model that spends the 1024-token budget thinking and returns content=None. Qwen3.5-9B is the pinned hybrid model the reasoning_effort=none case already exercises, and Together lists it with structured output support * test(integration): read the agent 365 guardrail status by its own name in spend logs The MCP shard runs under xdist against one database, and a sibling file creates a default_on pre_mcp_call content filter there. The owned proxy reloads DB guardrails, so that filter's 'success' entry could land first in guardrail_information and the test read it instead of the agent 365 verdict * test(unit): ignore asyncio's leaked-task records in the budget limiter push-failure log check gc.collect() inside the caplog window can collect a pending task an earlier test left on a closed loop, and asyncio logs 'Task was destroyed but it is pending' into this test's records. The check still counts every LiteLLM logger, and unretrieved task exceptions on this loop still go through the asserted exception handler * test(e2e-ui): fill the create-tag fields inside the dialog #42949 added 'Filter by tag name' and 'Filter by description' inputs to the Tag Management page, so page-wide getByLabel('Tag Name') and getByLabel('Description') match two elements and Playwright's strict mode fails the create step * test(integration): run integration proxies with the CI license Multi-worker proxies start each uvicorn worker in a fresh process, so every worker reads the license from its environment. Forward LITELLM_LICENSE into the proxy and test runner environments * ci: save GitHub Actions caches only from main and bump codecov-action to 5.5.5 Every pull request saved its own uv, maturin, Rust and Prisma caches, about 4.5 GB per PR, so the repository's 10 GB cache budget evicted main's entries within minutes. Pull request jobs then missed every cache, downloaded all dependencies from PyPI and hit the install step timeouts. Pull requests now restore only, and main keeps the caches warm for them. test-linting and check-ui-api-types run only on pull requests and keep saving codecov-action 5.5.4 imports its signing key from the deleted codecovsecurity keybase account, so every upload failed signature verification. 5.5.5 reads it from codecovsecops; the key ID matches the one signing the current CLI * test(unit): join the session-minting thread before collecting the handler asyncio.to_thread resumes the test as soon as the worker sets its result, while the pool thread can still hold the work item and through it the handler. gc.collect() then cannot finalize the handler and the session stays open. A pool that shuts down before the test continues drops that reference * test(integration): relaunch owned proxies that lose their port, expire idle gateway connections early owned_proxy_process released its reserved port and the proxy bound it only after full startup, so another xdist worker or an outgoing connection could take it first and the proxy exited with 'address already in use'. The launch now retries on a fresh port when that happens and stops every failed attempt. uvicorn closes idle keep-alive connections after 5 seconds and httpx expired them at the same 5 seconds, so a request sent right at that mark could reuse a socket the server was closing and get 'Connection reset by peer'. Gateway clients now drop idle connections after 2 seconds * ci(circleci): give the base SDK wheel build the same 30 minute no-output window as the Windows build The release profile builds with fat LTO and one codegen unit, so the final link of litellm-cache-s3 runs silently for minutes. Successful builds take 711 to 749 seconds, right at the default 10 minute no-output limit, and about 30% of recent runs were killed there * test(integration): model the budget-reset database outage as 10 seconds instead of 5 refused connections The proxy retries the database about every 30 seconds and each retry opens roughly one connection, so a 5-connection outage took 3 to 4 retries to clear and recovery landed between 60 and 90 seconds, straddling the test's 80 second reset window. A fixed 10 second outage still refuses the immediate reconnect and recovers on the next retry * ci: move the unit-test uv cache split into a composite action check_workflow_startup_safety sums every setup step's timeout, so the save and restore variants each counted 5 minutes although only one runs. One composite step keeps the setup ceiling at 35 minutes * test(unit): point tiktoken at the bundled cache for every unit test The rust_bridge tokenizer tests loaded o200k_base before any test in their xdist worker had imported default_encoding, so tiktoken fell back to the temp cache and tried to download under pytest-socket. Move the session fixture from litellm_core_utils/conftest.py to the root unit conftest. * test(integration): answer model discovery probes in the hosted_vllm wire tests The router's periodic upstream model info refresh sends GET /v1/models to hosted_vllm deployments, so a wire server that is live during a refresh sees an extra request. Answer the probe with an empty model list and leave it out of the provider-call assertions, matching the responses bridge tests.

feat(ui): filter tags by name and description on the Tag Management page

7ce5dde

galovics requested a review from a team September 24, 2026 13:31

galovics requested review from ryan-crabbe-berri and yuneng-berri as code owners September 24, 2026 13:31

galovics mentioned this pull request Sep 24, 2026

docs(tags): document the Tag Management table filters BerriAI/litellm-docs#1684

Open

galovics mentioned this pull request Sep 24, 2026

[Feature]: Contains filters for tag name and description on the Tag Management page #42946

Closed

1 task done

ryan-crabbe-berri approved these changes Oct 1, 2026

View reviewed changes

ryan-crabbe-berri merged commit f67caac into BerriAI:main Oct 1, 2026
95 checks passed

yuneng-berri mentioned this pull request Oct 1, 2026

test(ci): repair stale tests and flaky CI infrastructure #43983

Merged

1 of 23 tasks

Navigation Menu

Repository navigation

feat(ui): filter tags by name and description on the Tag Management page - #42949

feat(ui): filter tags by name and description on the Tag Management page#42949
ryan-crabbe-berri merged 1 commit into
BerriAI:mainfrom
galovics:tag-table-filters

galovics commented Sep 24, 2026 •

edited

Loading

Uh oh!

greptile-apps Bot commented Sep 24, 2026

Uh oh!

galovics commented Sep 24, 2026

Uh oh!

galovics commented Sep 29, 2026

Uh oh!

ryan-crabbe-berri left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

galovics commented Sep 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

TLDR

User Flow

Relevant issues

Pre-Submission checklist

Delays in PR merge?

Screenshots / Proof of Fix

Type

Caveats (if any)

Medium

Low

Final Attestation

Uh oh!

greptile-apps Bot commented Sep 24, 2026

Confidence Score: 5/5

Summary

Uh oh!

galovics commented Sep 24, 2026

Uh oh!

galovics commented Sep 29, 2026

Uh oh!

ryan-crabbe-berri left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

galovics commented Sep 24, 2026 •

edited

Loading