Skip to content

Run proxy servers based on bundle definitions #660

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

Merged
merged 1 commit into from
Apr 19, 2025
Merged

Conversation

rhernandez35
Copy link
Contributor

@rhernandez35 rhernandez35 commented Apr 19, 2025

This commit has the following high-level changes:

  • mix the PreRequest format directly into the service bundle instead of applying it at runtime in the MCP server layer
  • rewrite ConfigProvider into a client plugin

Things to follow up on:

  • stop mixing the PreRequest into the bundle, or at least put it into a separate "generic arguments" section so we can reuse it in other places like the CLI
  • strip out the PreRequest fields from the request – right now we send the whole Document to the backend service
  • nest the actual request inside the PreRequest as a document instead of mixing the fields in
  • give auth schemes and identity resolvers access to the full, unredacted request context

@rhernandez35 rhernandez35 requested a review from adwsingh April 19, 2025 05:49
@rhernandez35 rhernandez35 enabled auto-merge (rebase) April 19, 2025 05:53
@rhernandez35 rhernandez35 merged commit d215464 into main Apr 19, 2025
2 checks passed
@rhernandez35 rhernandez35 deleted the bundle-apply branch April 19, 2025 17:14
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.

2 participants