Skip to content

fix: make sure there are no duplicates when setting stream_options.include_usage - #2747

Open
nacx wants to merge 2 commits into
mainfrom
dupliace-stream-opts
Open

nacx wants to merge 2 commits into
mainfrom
dupliace-stream-opts

Conversation

@nacx

@nacx nacx commented Sep 26, 2026

Copy link
Copy Markdown
Member

Description

Make sure we don't leave duplicate keys in the JSON payload after injecting the stream_options.include_usage.

Related Issues/PRs (if applicable)

N/A

Special notes for reviewers (if applicable)

N/A

…clude_usage

Signed-off-by: Ignasi Barrera <ignasi@tetrate.io>
@nacx
nacx requested a review from a team as a code owner September 26, 2026 17:40
@netlify

netlify Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for theagentrouter ready!

Name Link
🔨 Latest commit 7f59e6c
🔍 Latest deploy log https://app.netlify.com/projects/theagentrouter/deploys/6ab814c698c196000803b52e
😎 Deploy Preview https://deploy-preview-2747--theagentrouter.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov

codecov Bot commented Sep 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/endpointspec/endpointspec.go 85.71% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

Signed-off-by: Ignasi Barrera <ignasi@tetrate.io>
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.

1 participant