Skip to content

fix(nvidia): remover stream_options do NIM - #195

Merged
actus7 merged 1 commit into
mainfrom
fix/nvidia-nim-stream-options
Aug 2, 2026
Merged

fix(nvidia): remover stream_options do NIM#195
actus7 merged 1 commit into
mainfrom
fix/nvidia-nim-stream-options

Conversation

@actus7

@actus7 actus7 commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Resumo

  • remove stream_options das chamadas NVIDIA NIM
  • adiciona teste para o body transform do provider

Verificação

  • pnpm test -- server/tests/openai-compatible.test.ts
  • pnpm typecheck
  • pnpm lint (1 warning preexistente em .remember/tmp/last-ndc.ts)

Copilot AI review requested due to automatic review settings August 2, 2026 00:07
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@vercel

vercel Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
modelhub Ready Ready Preview Aug 2, 2026 12:09am

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Este PR ajusta o provider NVIDIA NIM para não enviar stream_options nas requisições OpenAI-compatible, evitando incompatibilidade com o endpoint da NVIDIA, e adiciona um teste unitário garantindo esse comportamento.

Changes:

  • Introduz buildNvidiaNimBody() para montar o body OpenAI-compatible e remover stream_options antes do envio ao NIM.
  • Conecta o novo bodyTransform ao provider nvidianim.
  • Adiciona teste cobrindo que stream_options não é enviado para chat via NVIDIA NIM.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
server/providers/nvidianim.ts Adiciona bodyTransform específico para NVIDIA NIM removendo stream_options do body gerado.
server/tests/openai-compatible.test.ts Inclui teste para garantir que o body transform do NVIDIA NIM não contém stream_options.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@actus7
actus7 merged commit f8c1269 into main Aug 2, 2026
13 checks passed
@actus7
actus7 deleted the fix/nvidia-nim-stream-options branch August 2, 2026 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Backend changes tests Test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants