-
Notifications
You must be signed in to change notification settings - Fork 430
Add MCP Server example #2299
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
base: main
Are you sure you want to change the base?
Add MCP Server example #2299
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2299 +/- ##
=======================================
Coverage 85.34% 85.34%
=======================================
Files 26 26
Lines 2613 2613
Branches 481 481
=======================================
Hits 2230 2230
Misses 377 377
Partials 6 6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
examples/mcp/.gitignore
Outdated
@@ -0,0 +1,41 @@ | |||
# See https://help.github.com/articles/ignoring-files/ for more about ignoring files. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we review to see what's actually needed here? Most of this is already covered in the root .gitignore
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cleaned up in 22d9446
|
||
## Auth0 Tenant Setup | ||
|
||
For detailed instructions on setting up your Auth0 tenant for MCP server integration, please refer to the [Auth0 Tenant Setup guide](https://github.com/auth0/auth0-auth-js/blob/main/examples/example-fastmcp-mcp/README.md#auth0-tenant-setup). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This link 404s
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This link should be available once auth0/auth0-auth-js#45 merges in
examples/mcp/README.md
Outdated
With the configuration in place, the example can be started by running: | ||
|
||
```bash | ||
npm run start |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why npm
since we're using pnpm
for build? Also why start
? Probably need to either:
- Include a
run build
step sostart
works (fails currently w/out it) - Or use
run dev
instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addressed in 5882a9f
lockfileVersion: '9.0' | ||
|
||
settings: | ||
autoInstallPeers: true | ||
excludeLinksFromLockfile: false | ||
|
||
importers: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we just have the one root pnpm-lock.yaml
file? Why is this here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am following the pattern used in other examples: https://github.com/search?q=repo%3Aauth0%2Fnextjs-auth0%20pnpm-lock.yaml&type=code and
Line 13 in fe1f5e7
"install:examples": "pnpm install --filter ./examples/with-next-intl --shamefully-hoist && pnpm install --filter ./examples/with-shadcn --shamefully-hoist", |
Not too sure on the reason here though. I will check in with the sdk team to confirm
📋 Changes
Adds MCP Server example with Auth0 integration
📎 References
🎯 Testing
Use a MCP client like MCP inspector to test your MCP server