Skip to content

[Improve] Show Zoo Code in outbound provider activity logs#219

Draft
roomote[bot] wants to merge 2 commits into
mainfrom
improve/outbound-identity-2tafayi62nl92
Draft

[Improve] Show Zoo Code in outbound provider activity logs#219
roomote[bot] wants to merge 2 commits into
mainfrom
improve/outbound-identity-2tafayi62nl92

Conversation

@roomote
Copy link
Copy Markdown
Contributor

@roomote roomote Bot commented May 20, 2026

Opened on behalf of Elliott de Launay. View the task or mention @roomote for follow-up asks.

What problem this solves

Fixes an issue where some outbound requests from Zoo Code still identified themselves as Roo Code in provider-side activity and auth flows. The remaining gaps were in direct request paths such as OpenAI Codex, Unbound, Bedrock, the code-index embedders, image generation requests, and the cloud auth/share package.

Why this change was made

These paths bypassed the shared header helper or carried their own app metadata, so renaming the common defaults was not enough. This finishes the remaining identity updates, includes the review-requested Codex and Unbound fixes, and adds focused regression coverage for those custom request paths.

User impact

Provider logs and related integrations now consistently attribute these requests to Zoo Code instead of Roo Code. There is no UI change, but the outbound identity seen by OpenAI Codex, Unbound, Bedrock, and the related supporting services is now consistent.

Related PRs

  • #218 OAuth callback charset fix

@roomote roomote Bot added the roomote:auto-resolve-conflicts Allow Roomote to auto-resolve merge conflicts for this PR label May 20, 2026
@roomote
Copy link
Copy Markdown
Contributor Author

roomote Bot commented May 20, 2026

No actionable issues found. See task

  • src/api/providers/openai-codex.ts now sends originator and User-Agent as zoo-code for the SDK path, fetch fallback, and prompt completion path.
  • src/api/providers/unbound.ts now sends zoo-code in both X-Unbound-Metadata and unbound_metadata.originApp.
  • platform-unit-test (windows-latest) passed on commit 23fbb96.

Comment thread src/integrations/openai-codex/oauth.ts
Comment thread src/api/providers/constants.ts
@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

❌ Patch coverage is 96.29630% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/integrations/openai-codex/oauth.ts 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@roomote
Copy link
Copy Markdown
Contributor Author

roomote Bot commented May 20, 2026

Rebased onto main and fixed in bbc7d19. The remaining Codex and Unbound outbound identity gaps are addressed, the PR description is refreshed, and the matching review threads are resolved. See task

@roomote roomote Bot force-pushed the improve/outbound-identity-2tafayi62nl92 branch from 5849da5 to bbc7d19 Compare May 20, 2026 13:43
@edelauna edelauna force-pushed the improve/outbound-identity-2tafayi62nl92 branch from bbc7d19 to 23fbb96 Compare May 20, 2026 23:02
@edelauna
Copy link
Copy Markdown
Contributor

edelauna commented May 20, 2026

Wait for #123 to be merged, as it integrates the final Roo and Zoo renaming changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

roomote:auto-resolve-conflicts Allow Roomote to auto-resolve merge conflicts for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants