Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrations in MCP #13

Draft
wants to merge 15 commits into
base: migrations-tools
Choose a base branch
from
Draft

Migrations in MCP #13

wants to merge 15 commits into from

Conversation

davidgomes
Copy link
Contributor

No description provided.

davidgomes and others added 6 commits December 25, 2024 18:19
- Add Neon logo
- Add link to related blog
## Description

This PR aims to setup Neon's MCP server evaluation.

**FEATURES**

- Refactor MCP Client to serve as the testing client
- Add the 1st evaluation to the `prepare_database_migration` MCP Server
tool
- Adapt the `prepare_database_migration` tool to be way more
deterministic
- Add command to run evaluation locally

## Test Plan

In 172 test runs, with 9 different prompts, we had a 96% factuality
check rate for the `prepare_database_migration` prompt, meaning that the
tool behaved very much as expected for most of the time.

Experiments:
https://www.braintrust.dev/app/Neon%20Inc./p/prepare_database_migration/experiments/mcp-evaluation-1735815763?c=mcp-evaluation-1735810092&tc=985c1e51-a5d6-49dd-93cf-a05eef322064&s=51591878-c460-46d7-8852-e65893592d22


![image](https://github.com/user-attachments/assets/51127b66-3321-4dfd-949d-e732ba050211)
@davidgomes davidgomes self-assigned this Jan 14, 2025
Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@davidgomes davidgomes marked this pull request as draft January 14, 2025 17:39
extended from #14

- As we use bun during build stage, `node:alpine` docker image is not
enough
- Update the MCP command

---------

Co-authored-by: Henry Mao <[email protected]>
Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 2   low 0   info 0 View in Orca
🔑 The following Secrets have been detected in your pull request across all commits

⚠️ Please take action to mitigate the risk of the identified secrets by revoking them, and if already in use, updating all dependent systems

NAME FILE LINE NUM COMMIT
medium .env File ...-client/.env.example 1 View in code
medium .env File .env.example 1 View in code

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

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.

4 participants