fix(ui): exclude vector_store_ids from model edit JSON textarea

[Jwrede]

Summary

• Exclude vector_store_ids, guardrails, and tags from the litellm_extra_params JSON textarea in the model edit form, matching the existing litellm_credential_name exclusion
• Also delete these keys from the parsed JSON on submit, so manually injecting them into the textarea cannot override the dedicated form fields

These fields all have dedicated form inputs that handle their own submission logic. Including them in the catch-all JSON textarea creates a duplication path where the JSON values can override the form field logic, silently injecting parameters like vector_store_ids: [] into litellm_params for non-OpenAI models.

Fixes #27117

Test plan

• Existing test should not include vector_store_ids in update payload when model has none passes
• New test should not leak vector_store_ids from JSON textarea when model already has them -- verifies the JSON textarea does not contain vector_store_ids even when the model has them
• Full model_info_view test suite passes (36/36)

[greptile-apps]

Greptile Summary

This PR fixes a duplication bug in the model edit form where vector_store_ids, guardrails, and tags were leaking into the catch-all JSON textarea, allowing textarea values to silently override the dedicated form fields for those inputs on save.

• Textarea exclusion (model_info_view.tsx line 657): The three fields are now filtered out when building the textarea's initial JSON value, matching the pre-existing litellm_credential_name exclusion.
• Submit-time stripping (model_info_view.tsx lines 238–240): Even if a user manually injects these keys into the textarea, they are deleted from parsedExtraParams before it is merged into updatedLitellmParams, so the dedicated form-field logic at lines 274–283 always wins.
• New test (model_info_view.test.tsx): Verifies the textarea does not contain vector_store_ids when the model already has them; no existing tests are altered.

Confidence Score: 5/5

Safe to merge — the change is narrowly scoped to excluding three fields from a JSON textarea, with the fix covered by a new test and no existing tests modified.

The fix is minimal and additive: it extends an already-proven pattern (litellm_credential_name exclusion) to three additional fields, each of which has its own dedicated form handler that correctly owns the submit logic. The textarea-strip on submit provides defence-in-depth against manual injection. A new test directly validates the regression path, and the existing 36-test suite continues to pass.

No files require special attention.

Important Files Changed

Filename	Overview
ui/litellm-dashboard/src/components/model_info_view.tsx	Excludes vector_store_ids, guardrails, and tags from the JSON textarea initial value and strips them on submit — mirrors existing litellm_credential_name exclusion. Dedicated form-field handlers at lines 274–283 remain the authoritative source for those values.
ui/litellm-dashboard/src/components/model_info_view.test.tsx	Adds a new test verifying that vector_store_ids does not appear in the JSON textarea even when the model already has them; no existing tests are modified or weakened.

_{Reviews (2): Last reviewed commit: "fix(ui): exclude vector_store_ids from m..." | Re-trigger Greptile}

[Jwrede]

@greptileai review

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.