Skip to content

Conversation

@ChiragAgg5k
Copy link
Member

@ChiragAgg5k ChiragAgg5k commented Sep 28, 2025

Summary by CodeRabbit

  • Chores

    • Consolidated publishing: MCP Registry publishing now runs within the main release workflow; legacy separate workflow removed.
    • Version bumped to 0.2.3.
  • Metadata

    • Added JSON schema reference and updated package/version metadata.
    • Replaced homepage with website URL.
    • Simplified repository info to use URL and source.
    • Removed deprecated fields (author, license, homepage) and legacy blocks (capabilities, command, environment, keywords) from server metadata.

@coderabbitai
Copy link

coderabbitai bot commented Sep 28, 2025

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

📥 Commits

Reviewing files that changed from the base of the PR and between 64cddeb and 566dfa9.

📒 Files selected for processing (4)
  • .github/workflows/publish-mcp.yml (0 hunks)
  • .github/workflows/publish.yml (1 hunks)
  • pyproject.toml (1 hunks)
  • server.json (1 hunks)
 _______________________________
< Copilot: Off. CodeRabbit: On. >
 -------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-mcp-registry-file

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ChiragAgg5k ChiragAgg5k requested a review from Copilot September 28, 2025 02:39
Copy link

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

Fixes the MCP registry file configuration by updating it to conform to the new schema and simplifying the structure. The PR also consolidates CI/CD workflows by moving MCP publishing into the main publish workflow.

  • Updated server.json to use the new MCP schema and simplified structure
  • Version bump from 0.2.2 to 0.2.3 across project files
  • Consolidated MCP publishing workflow into the main publish workflow

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
server.json Updated schema, simplified structure, and removed deprecated fields
pyproject.toml Version bump to 0.2.3
.github/workflows/publish.yml Added MCP registry publishing job
.github/workflows/publish-mcp.yml Removed standalone MCP publishing workflow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

"url": "https://github.com/appwrite/mcp-for-api.git",
"source": "https://github.com/appwrite/mcp-for-api"
"url": "https://github.com/appwrite/mcp-for-api",
"source": "github"
Copy link

Copilot AI Sep 28, 2025

Choose a reason for hiding this comment

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

[nitpick] The source value 'github' is ambiguous. Consider using the full repository URL or a more descriptive identifier for clarity.

Suggested change
"source": "github"
"source": "https://github.com/appwrite/mcp-for-api"

Copilot uses AI. Check for mistakes.
@ChiragAgg5k ChiragAgg5k merged commit 88a98c3 into main Sep 28, 2025
1 check was pending
@ChiragAgg5k ChiragAgg5k deleted the fix-mcp-registry-file branch September 28, 2025 02:44
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