Skip to content

Update fastah-ip-geo-tools plugin with latest changes from Fastah repo 🤖🤖🤖#1194

Open
punit-fastah wants to merge 1 commit intogithub:stagedfrom
punit-fastah:update-fastah-ip-geo-tools-plugin
Open

Update fastah-ip-geo-tools plugin with latest changes from Fastah repo 🤖🤖🤖#1194
punit-fastah wants to merge 1 commit intogithub:stagedfrom
punit-fastah:update-fastah-ip-geo-tools-plugin

Conversation

@punit-fastah
Copy link
Copy Markdown
Contributor

@punit-fastah punit-fastah commented Mar 27, 2026

This pull request adds a new file, .mcp.json, for the fastah-ip-geo-tools plugin, specifying an MCP server endpoint, as described in the Visual Studio Code documentation: https://code.visualstudio.com/docs/copilot/customization/agent-plugins#_mcp-configuration-format

Configuration update:

  • Added a .mcp.json file to the plugins/fastah-ip-geo-tools directory, defining the fastahIpGeofeed MCP server with its type and URL.

Copilot AI review requested due to automatic review settings March 27, 2026 06:29
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds an MCP server configuration to the fastah-ip-geo-tools plugin so the plugin can connect to Fastah’s IP Geofeed MCP endpoint.

Changes:

  • Added plugins/fastah-ip-geo-tools/.mcp.json defining an HTTP MCP server pointing at https://mcp.fastah.ai/mcp.

@@ -0,0 +1,8 @@
{
"mcpServers": {
"fastahIpGeofeed": {
Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

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

The MCP server key is named fastahIpGeofeed, but the geofeed-tuner skill documentation/examples reference the server as fastah-ip-geofeed. If agents/skills are written to target the documented server name, this config won’t be picked up and MCP tool calls will fail. Align the server name here with the name used by the skill (or update the skill docs/usage to match this name) so the configuration is discoverable.

Suggested change
"fastahIpGeofeed": {
"fastah-ip-geofeed": {

Copilot uses AI. Check for mistakes.
@aaronpowell
Copy link
Copy Markdown
Contributor

I can't find any docs about the MCP server and I'm feeling like this is getting very close to the line on our policy regarding paid services.

@sid6mathur
Copy link
Copy Markdown

I can't find any docs about the MCP server and I'm feeling like this is getting very close to the line on our policy regarding paid services.

Hello @aaronpowell , the MCP server key in the .mcp.json being out of sync with the key in the SKILL.md is an inadvertent error and will be fixed.

The MCP server this is a free, non-commercial MCP service for geocoding RFC 8805 style place names - it's usage is explained in the SKILL.md. How do we document it so it's aligned in letter and spirit with the policy? Thank you in advance!
-Sid

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.

5 participants