Skip to content

Conversation

@qstearns
Copy link
Contributor

Add resources to functions manifest for any resources that are defined on the mcp sdk.

Additionally, add handleResources export to the mcp server template, so calls will be proxied for functions defined in the SDK

@qstearns qstearns requested a review from a team as a code owner October 24, 2025 17:24
@changeset-bot
Copy link

changeset-bot bot commented Oct 24, 2025

🦋 Changeset detected

Latest commit: 04d1443

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@gram-ai/functions Minor
@gram-ai/create-function Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Oct 24, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
gram Ready Ready Preview Comment Oct 27, 2025 4:41pm
gram-docs-redirect Ready Ready Preview Comment Oct 27, 2025 4:41pm
gram-landing-redirect Ready Ready Preview Comment Oct 27, 2025 4:41pm

}>;
resources?: Array<{
name: string;
description?: string | undefined;
Copy link
Contributor

Choose a reason for hiding this comment

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

I think something we need to figure out. Although descriptions seemingly are optional at the MCP SDK level for tools and resources our Gram Functions system requires them. Not totally sure what to do about this

@qstearns qstearns merged commit 3fc0efc into main Oct 27, 2025
16 checks passed
@qstearns qstearns deleted the quinn/add-resources-and-meta-to-mcp-manifests branch October 27, 2025 16:47
@github-actions github-actions bot locked and limited conversation to collaborators Oct 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants