Skip to content

feat(zoo-gateway): auth callback and multi-profile token sync#347

Open
JamesRobert20 wants to merge 2 commits into
feat/zoo-gateway-settings-uifrom
feat/zoo-gateway-auth-sync
Open

feat(zoo-gateway): auth callback and multi-profile token sync#347
JamesRobert20 wants to merge 2 commits into
feat/zoo-gateway-settings-uifrom
feat/zoo-gateway-auth-sync

Conversation

@JamesRobert20
Copy link
Copy Markdown
Contributor

Summary

  • Propagates OAuth callback token to all ClineProvider instances (sequential writes to avoid profile-store races)
  • Seeds/syncs zoo-gateway profiles, including non-active profiles for model fetch
  • Clears in-memory active profile on sign-out even when disk is already clean

Part 3 of the Zoo Gateway stack. Depends on PR2 (stacked on #344).

Test plan

  • handleUri multi-instance + serialization tests
  • webviewMessageHandler separate-profile lookup test
  • Manual: sign in/out across multiple profiles

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 27, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: df25ba45-f31a-4eda-acd0-78d36fb91161

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/zoo-gateway-auth-sync

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@JamesRobert20 JamesRobert20 mentioned this pull request May 27, 2026
1 task
@JamesRobert20 JamesRobert20 force-pushed the feat/zoo-gateway-settings-ui branch from adedf0a to 8634510 Compare May 27, 2026 14:25
@JamesRobert20 JamesRobert20 force-pushed the feat/zoo-gateway-auth-sync branch from 654117d to 1326a97 Compare May 27, 2026 14:29
@JamesRobert20 JamesRobert20 force-pushed the feat/zoo-gateway-settings-ui branch from 8634510 to 7bc8c0c Compare May 27, 2026 14:42
@JamesRobert20 JamesRobert20 force-pushed the feat/zoo-gateway-auth-sync branch 2 times, most recently from 4584d81 to f245208 Compare May 27, 2026 14:46
Co-authored-by: Cursor <cursoragent@cursor.com>
@JamesRobert20 JamesRobert20 force-pushed the feat/zoo-gateway-auth-sync branch from f245208 to 078bce0 Compare May 27, 2026 15:10
@codecov
Copy link
Copy Markdown

codecov Bot commented May 27, 2026

Codecov Report

❌ Patch coverage is 85.82090% with 19 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/core/webview/ClineProvider.ts 83.75% 13 Missing ⚠️
src/core/webview/webviewMessageHandler.ts 88.23% 6 Missing ⚠️

📢 Thoughts on this report? Let us know!

Add ClineProvider tests for handleZooCodeCallback, ensureZooGatewayProfileSeeded, and webviewMessageHandler zooCodeSignOut to satisfy codecov patch on PR #347.

Co-authored-by: Cursor <cursoragent@cursor.com>
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