Skip to content

Conversation

georgevanderson
Copy link
Collaborator

@georgevanderson georgevanderson commented Oct 10, 2025

Note

Adds an MCP tool to fetch source file paths for infra components and introduces Metadata.source across proto and Rust models.

  • MCP Server/Tools:
    • New tool get_source: returns source file path for components; added handler and registration in mcp/server.rs and mcp/tools/mod.rs.
    • Implements search across topics, tables, and api_endpoints with flexible matching in mcp/tools/get_source.rs.
  • Schema/Models:
    • Proto: add SourceLocation and Metadata.source in packages/protobuf/infrastructure_map.proto.
    • Rust infra structs: add optional source to Metadata (with SourceLocation { file }) and map to/from proto in:
      • framework/core/infrastructure/table.rs
      • framework/core/infrastructure/topic.rs
      • framework/core/infrastructure/api_endpoint.rs
      • framework/core/infrastructure/function_process.rs.

Written by Cursor Bugbot for commit 2d0dd49. This will update automatically on new commits. Configure here.

Copy link

vercel bot commented Oct 10, 2025

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

Project Deployment Preview Comments Updated (UTC)
framework-docs Ready Ready Preview Comment Oct 10, 2025 0:03am

});
}
}
}
Copy link

Choose a reason for hiding this comment

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

Bug: Query Filter Ignores Components Without Metadata

The get_source tool's search logic silently filters out components that match a query but lack source location metadata (e.g., metadata.source.file). This can lead to "Component not found" for existing components, as the SourceLocationNotAvailable error variant is never used.

Additional Locations (1)

Fix in Cursor Fix in Web

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.

1 participant