Prompt security litellm #16365
Prompt security litellm #16365
Conversation
|
@lior-ps is attempting to deploy a commit to the CLERKIEAI Team on Vercel. A member of the Team first needs to authorize it. |
|
|
|
@lior-ps can you fix the conflicts? I'm unable to merge this |
…ame-named pip package
…bedding call type
The GitHub Actions workflow was installing litellm-enterprise to system Python using 'python -m pip install -e .', but tests run in Poetry's virtualenv using 'poetry run pytest'. This caused ImportError for enterprise package types. Changed to 'poetry run pip install -e .' so the package is available in the same virtualenv where pytest executes. Fixes enterprise test collection errors in GitHub Actions CI.
@krrishdholakia fixed the tests related to our change. the other failures are not related to our code. can you please have a look? |
Per reviewer feedback, move test_prompt_security_guardrails.py from tests/guardrails_tests/ to tests/test_litellm/proxy/guardrails/ so it will be executed by GitHub Actions workflow test-litellm.yml. This ensures the Prompt Security integration tests run in CI.
|
Hello @krrishdholakia , |
|
@krrishdholakia how can we get this PR approved? Thanks |
5 participants
Title
Add Prompt Security Guardrail
Relevant issues
Add Prompt Security Guardrail
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/litellm/directory, Adding at least 1 test is a hard requirement - see detailsmake test-unitType
🆕 New Feature
📖 Documentation
✅ Test
Changes