Skip to content

Conversation

@dean-zavad
Copy link
Contributor

@dean-zavad dean-zavad commented Nov 3, 2025

Guardrail Image Support

Relevant issues

Pre-Submission checklist

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

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • I have added a screenshot of my new test passing locally
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

Type

🆕 New Feature
🐛 Bug Fix

Changes

  • Migrating to noma v2 API for additional support in guardrailing images.
  • Fixing partial output schema when integration with external logger

Notes

We removed apply_guardrails hook from noma code from several reasons:

  • It broke datadog integration - this seems to be a bug beyond the scope of the noma guardrail.
  • It broke guardrail features - guardrail "anonymize_input" and "monitor_mode" have stopped working.
  • Streaming did not worked properly.

Once the above issues will be resolved, we'll be happy to opt in

Screenshot Of Passing Tests

image

@vercel
Copy link

vercel bot commented Nov 3, 2025

@dean-zavad is attempting to deploy a commit to the CLERKIEAI Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Nov 3, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

Choose a reason for hiding this comment

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

@dean-zavad can you add apply_guardrail support in this PR - it would allow noma to work across all LLM endpoints (/v1/responses, /v1/messages, /audio/transcription, etc.)

Copy link

@itaimodi itaimodi Nov 4, 2025

Choose a reason for hiding this comment

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

we will implement that in another PR. apply_guardrail broke the current implementation. @ishaan-jaff

@krrishdholakia krrishdholakia merged commit f19356d into BerriAI:main Nov 5, 2025
3 of 6 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.

4 participants