fix(ui): add null guard for models in API keys table

[veeceey]

Summary

• Fix crash in the Virtual Keys table when a key's models field is null or undefined
• The className expression on the TableCell tried to access .length on a potentially null value, throwing a TypeError that broke the entire keys page
• Added Array.isArray() guard before accessing .length on the models value

Root Cause

In VirtualKeysTable.tsx line 730, the expression:

cell.column.id === "models" && (cell.getValue() as string[]).length > 3

crashes when cell.getValue() returns null or undefined (which happens when a key has no models set). JavaScript short-circuit evaluation evaluates the second operand when the first is truthy, causing null.length to throw.

Fix

cell.column.id === "models" && Array.isArray(cell.getValue()) && (cell.getValue() as string[]).length > 3

Test plan

• Added unit tests for keys with null models
• Added unit tests for keys with undefined models
• Existing tests continue to pass

Fixes #20611

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
litellm	Ready	Preview, Comment	Feb 7, 2026 7:27pm

[greptile-apps]

Greptile Overview

Greptile Summary

Fixed a runtime crash in the Virtual Keys table caused by null/undefined models field. The className expression on line 730 attempted to access .length on a potentially null value without a guard, throwing a TypeError that broke the entire page.

• Added Array.isArray() check before accessing .length on models value in the conditional className
• The fix is consistent with existing null handling in the models column cell renderer (line 349)
• Added comprehensive test coverage for both null and undefined cases
• Simple, targeted fix that directly addresses the root cause

Confidence Score: 5/5

• Safe to merge - straightforward null guard fix with proper test coverage
• The fix is minimal, correct, and well-tested. It adds a defensive check that prevents a crash without changing any logic or introducing side effects. The solution matches the existing pattern used elsewhere in the same file.
• No files require special attention

Important Files Changed

Filename	Overview
ui/litellm-dashboard/src/components/VirtualKeysPage/VirtualKeysTable.tsx	Added Array.isArray() guard on line 730 to prevent crash when accessing .length on null/undefined models value in className expression
ui/litellm-dashboard/src/components/VirtualKeysPage/VirtualKeysTable.test.tsx	Added two test cases verifying table renders without crashing when models field is null or undefined

Sequence Diagram

sequenceDiagram
    participant User
    participant VirtualKeysTable
    participant TableCell
    participant CellRenderer
    
    User->>VirtualKeysTable: View API Keys page
    VirtualKeysTable->>VirtualKeysTable: Fetch keys data
    Note over VirtualKeysTable: Key with models=null
    
    VirtualKeysTable->>TableCell: Render table row
    TableCell->>TableCell: Compute className
    
    alt Before fix
        TableCell->>TableCell: Check: cell.column.id === "models"
        TableCell->>TableCell: Access: cell.getValue().length
        Note over TableCell: TypeError: Cannot read<br/>property 'length' of null
        TableCell-->>User: Page crashes
    end
    
    alt After fix
        TableCell->>TableCell: Check: cell.column.id === "models"
        TableCell->>TableCell: Check: Array.isArray(cell.getValue())
        Note over TableCell: Returns false for null
        TableCell->>TableCell: Skip .length check
        TableCell->>CellRenderer: Render cell content
        CellRenderer->>CellRenderer: Array.isArray(models) check
        CellRenderer-->>User: Display empty cell (null case)
    end

Loading

[greptile-apps]

_{2 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}