Skip to content

[FA-2] Adding login func to mcp server#1

Merged
scott-clare1 merged 5 commits intomainfrom
FA-2-allowing-login
Apr 1, 2025
Merged

[FA-2] Adding login func to mcp server#1
scott-clare1 merged 5 commits intomainfrom
FA-2-allowing-login

Conversation

@scott-clare1
Copy link
Contributor

This PR adds two MCP endpoints for the server:

  1. A login endpoint accepting an API key
  2. An endpoint for retrieving account information.

A demo for interacting with server with Claude Desktop:

Screen.Recording.2025-04-01.at.10.04.54.mov

Checklist

Please ensure you have done the following:

  • I have read the CONTRIBUTING guide.
  • I have updated the documentation if required.
  • I have added tests which cover my changes.

Type of change

Tick all those that apply:

  • Bug Fix (non-breaking change, fixing an issue)
  • New feature (non-breaking change to add functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Other (add details above)

MacOS tests

To trigger the CI to run on a macOS backed workflow, add the macos-ci-test label to the pull request (PR).

Our advice is to only run this workflow when testing the compatability between operating systems for a change that you've made, e.g., adding a new dependency to the virtual environment.

Note: This can take up to 5 minutes to run. This workflow costs x10 more than a Linux-based workflow, use at discretion.

@scott-clare1 scott-clare1 self-assigned this Apr 1, 2025
@scott-clare1 scott-clare1 added the enhancement New feature or request label Apr 1, 2025
@scott-clare1 scott-clare1 requested a review from samgwd April 1, 2025 09:50
@scott-clare1 scott-clare1 marked this pull request as ready for review April 1, 2025 09:50
Copy link
Collaborator

@samgwd samgwd left a comment

Choose a reason for hiding this comment

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

LGTM

@scott-clare1 scott-clare1 merged commit e63f38e into main Apr 1, 2025
1 check passed
@scott-clare1 scott-clare1 deleted the FA-2-allowing-login branch April 1, 2025 11:42
@scott-clare1 scott-clare1 restored the FA-2-allowing-login branch April 1, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants