Skip to content

feat(ai-gateway): allow Responses API for OpenCode Go - #5044

Merged
chrarnoldus merged 1 commit into
mainfrom
scarlet-minute
Aug 5, 2026
Merged

feat(ai-gateway): allow Responses API for OpenCode Go#5044
chrarnoldus merged 1 commit into
mainfrom
scarlet-minute

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary\n- allow the Responses API for the OpenCode Go direct BYOK provider\n- add a regression test for the provider capability\n\n## Verification\n- pnpm --filter web test -- src/lib/ai-gateway/providers/direct-byok/opencode-go.test.ts --runInBand\n- pnpm --filter web typecheck\n- pnpm exec oxlint --config .oxlintrc.json apps/web/src/lib/ai-gateway/providers/direct-byok/opencode-go.ts apps/web/src/lib/ai-gateway/providers/direct-byok/opencode-go.test.ts

@chrarnoldus chrarnoldus self-assigned this Aug 5, 2026
@kilo-code-bot

kilo-code-bot Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Adds 'responses' to the OpenCode Go direct-BYOK provider's supported_chat_apis with a regression test; the change matches the established pattern used by other providers (martian, orcarouter), auth for the Responses path is covered by the standard Authorization: Bearer header set in upstreamRequest, and the Anthropic-only x-api-key transform correctly remains scoped to messages requests.

Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/providers/direct-byok/opencode-go.ts
  • apps/web/src/lib/ai-gateway/providers/direct-byok/opencode-go.test.ts

Reviewed by kimi-k3 · Input: 54.7K · Output: 5.8K · Cached: 532.5K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus enabled auto-merge (squash) August 5, 2026 16:05
@chrarnoldus
chrarnoldus merged commit 20d40c3 into main Aug 5, 2026
16 checks passed
@chrarnoldus
chrarnoldus deleted the scarlet-minute branch August 5, 2026 16:07
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.

2 participants