Skip to content

Conversation

@njbrake
Copy link
Contributor

@njbrake njbrake commented Oct 17, 2025

Description

Portkey docs show that it supports returning:

https://portkey.ai/docs/product/ai-gateway/multimodal-capabilities/thinking-mode#understanding-response-format

I need to make some changes to support and test this but I'm waiting for our portkey account to support a model that outputs thinking, that way I can edit the portkey response as needed.

PR Type

🆕 New Feature 🐛 Bug Fix 💅 Refactor 📚 Documentation 🚦 Infrastructure ## Relevant issues

Checklist

  • I have added unit tests that prove my fix/feature works
  • New and existing tests pass locally
  • Documentation was updated where necessary
  • I have read and followed the contribution guidelines```

@njbrake njbrake linked an issue Oct 17, 2025 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Oct 17, 2025

Codecov Report

❌ Patch coverage is 80.82192% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/any_llm/providers/portkey/portkey.py 82.60% 6 Missing and 2 partials ⚠️
src/any_llm/providers/portkey/utils.py 77.77% 1 Missing and 5 partials ⚠️
Files with missing lines Coverage Δ
src/any_llm/providers/portkey/utils.py 77.77% <77.77%> (ø)
src/any_llm/providers/portkey/portkey.py 86.20% <82.60%> (-13.80%) ⬇️

... and 34 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@njbrake njbrake marked this pull request as ready for review October 20, 2025 12:07
@njbrake
Copy link
Contributor Author

njbrake commented Oct 20, 2025

@njbrake njbrake merged commit 9aadb2d into main Oct 20, 2025
18 of 19 checks passed
@njbrake njbrake deleted the 292-portkey-completion-api-reasoning-support branch October 20, 2025 12:16
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.

Portkey Completion API Reasoning Support

2 participants