Skip to content

Feature/integrated auth and dashboard#42

Merged
madtank merged 4 commits intoax-platform:mainfrom
nikhilkulkarni1755:feature/integrated_auth_and_dashboard
Dec 26, 2025
Merged

Feature/integrated auth and dashboard#42
madtank merged 4 commits intoax-platform:mainfrom
nikhilkulkarni1755:feature/integrated_auth_and_dashboard

Conversation

@nikhilkulkarni1755
Copy link
Contributor

🔐 Integrate OAuth Authentication into Dashboard Workflow

Overview

Streamlines agent setup by eliminating the confusing two-step authentication process. Users now authenticate directly through the dashboard UI instead of manually running npx mcp-remote.
What Changed

Before:
Step 1: Manual auth (confusing for beginners)
npx mcp-remote https://mcp.paxai.app/mcp/agents/twitter_agent --oauth-server https://api.paxai.app

Step 2: Start dashboard
python3 scripts/start_dashboard.py

After:
Single command - auth happens in UI
python3 scripts/start_dashboard.py

Key Features

✅ One-click authentication - Auth modal appears when needed
✅ Visual feedback - Real-time progress during OAuth flow
✅ Auto-retry - Monitor starts automatically after successful auth
✅ Version-agnostic - Finds tokens across all mcp-remote versions (0.1.29, 0.1.35, 0.1.36+)
✅ Universal - Works identically for all agents (twitter, message_draft, nano_banana_pro, etc.)

Technical Improvements

New TokenValidator class searches all mcp-remote directories
OAuthManager handles async OAuth flow with proper timeout/error handling
Fixed stdin JSON parsing errors in mcp-remote subprocess
Added pre-flight auth checks before starting monitors
Robust process cleanup prevents success/failure state corruption

Files Added/Modified

src/ax_agent_studio/auth/ - New authentication modules
src/ax_agent_studio/dashboard/backend/main.py - Auth status API endpoints
src/ax_agent_studio/dashboard/frontend/ - Auth modal and flow handling

Fixes the big user onboarding pain point and makes the setup process significantly more beginner-friendly.

@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

Copy link
Member

@madtank madtank left a comment

Choose a reason for hiding this comment

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

Thanks for this PR - nice improvement for first-time user onboarding!

A couple of small fixes before we merge:

  1. Type hint fix: Dict[str, any] should be Dict[str, Any] (capital A) in:

    • src/ax_agent_studio/auth/token_validator.py:74
    • src/ax_agent_studio/auth/oauth_manager.py:33

    You'll also need to add Any to the typing import.

  2. Redundant import: In src/ax_agent_studio/dashboard/backend/main.py:647, there's an inline from ax_agent_studio.auth import OAuthManager but it's already imported at the top of the file (line 21). Can remove the duplicate.

Once those are fixed, we'll merge it. 👍

@madtank
Copy link
Member

madtank commented Dec 26, 2025

Also, the CLA check is failing. You'll need to sign the Contributor License Agreement:

  1. Look for the CLA bot comment on this PR (or check the failed check)
  2. Post a comment on this PR with the text: I have read the CLA Document and I hereby sign the CLA
  3. The CLA check should pass after that

Let us know if you have any questions!

@nikhilkulkarni1755
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@madtank madtank merged commit 69bb79d into ax-platform:main Dec 26, 2025
2 of 3 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 26, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants