-
Notifications
You must be signed in to change notification settings - Fork 214
Add skill.md and .well-known/skills/* reverse proxy configuration #2790
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
Conversation
Co-Authored-By: mintlify[bot] <109931778+mintlify[bot]@users.noreply.github.com>
Co-Authored-By: mintlify[bot] <109931778+mintlify[bot]@users.noreply.github.com>
Co-Authored-By: mintlify[bot] <109931778+mintlify[bot]@users.noreply.github.com>
Co-authored-by: tembo[bot] <208362400+tembo[bot]@users.noreply.github.com>
Co-authored-by: tembo[bot] <208362400+tembo[bot]@users.noreply.github.com>
Co-authored-by: tembo[bot] <208362400+tembo[bot]@users.noreply.github.com>
ethanpalm
left a comment
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.
lgtm
| title: "Reverse proxy" | ||
| description: "Configure a custom reverse proxy to serve your documentation." | ||
| keywords: ["reverse proxy configuration", "nginx", "proxy routing", "header forwarding"] | ||
| keywords: ["reverse proxy configuration","nginx","proxy routing","header forwarding"] |
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.
Minor formatting nit: can we keep spaces after commas in keywords for readability/consistency?
| keywords: ["reverse proxy configuration","nginx","proxy routing","header forwarding"] | |
| keywords: ["reverse proxy configuration", "nginx", "proxy routing", "header forwarding"] |
| proxy_set_header X-Forwarded-Proto $scheme; | ||
| proxy_set_header User-Agent $http_user_agent; | ||
| # Disable caching for verification paths |
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.
Looks like a copy/paste comment; this block is for skills, not verification.
| # Disable caching for verification paths | |
| # Disable caching for skills paths |
deploy/reverse-proxy.mdx
Outdated
| **Cause**: Incorrect caching configuration. | ||
|
|
||
| **Solution**: Enable caching only for `/mintlify-assets/_next/static/*` paths. | ||
| **Solution**: Enable caching only for `/mintlify-assets/_next/static/*` paths. |
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 file seems to be missing a trailing newline (shows up as \ No newline at end of file in the diff). Worth restoring to avoid noisy diffs/tooling warnings.
Added routing configuration for
/skill.mdand/.well-known/skills/*paths to the reverse proxy documentation. Updated both the routing table and nginx example to include these skill-related endpoints.Files changed:
Created by Mintlify agent
Note
Introduces documentation for
skill.mdand updates reverse proxy guidance to support AI skills endpoints.ai/skillmd.mdx: explains theskill.mdcapability file, usage with agents/CLI, and notes reverse proxy forwarding for/skill.mdand/.well-known/skills/*deploy/reverse-proxy.mdx: adds routing table entries and nginxlocationblocks for/.well-known/skills/*and/skill.mdwith caching disabled; tidies frontmatterkeywordsWritten by Cursor Bugbot for commit 21eb3ae. This will update automatically on new commits. Configure here.