Skip to content

chore: Update broken links and folder structure in mcp-registry/README.md #193

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 30, 2025
Merged

chore: Update broken links and folder structure in mcp-registry/README.md #193

merged 2 commits into from
Jun 30, 2025

Conversation

@HareeshBahuleyan
Copy link
Contributor

@HareeshBahuleyan HareeshBahuleyan commented Jun 30, 2025 •
edited by qodo-merge-pro bot
Loading

User description

Some information in mcp-registry/README.md were found to be outdated. This PR fixes broken links and updates the README.

PR Type

Documentation

Description

  • Fix broken GitHub repository links in README

  • Update folder structure documentation to reflect current layout

  • Correct schema file references and paths

Changes diagram

flowchart LR
  A["Old GitHub Links"] --> B["Updated Repository URLs"]
  C["Outdated Folder Structure"] --> D["Current JSON File Structure"]
  E["Incorrect Schema References"] --> F["Corrected Schema Paths"]
Loading

Changes walkthrough 📝

Relevant files
Documentation
README.md
Fix links and update documentation structure                         

mcp-registry/README.md

  • Updated GitHub repository URL from pathintegral-institute/mcp-registry
    to pathintegral-institute/mcpm.sh
  • Changed folder structure from individual server directories to single
    JSON files
  • Updated schema reference from manifest-schema.json to
    server-schema.json
  • Fixed LICENSE file path reference
    		• +8/-12   
    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
    •

    Copilot AI review requested due to automatic review settings June 30, 2025 09:02
    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 30, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Broken Reference

    The schema reference points to schema/server-schema.json but it's unclear if this file actually exists in the repository. This could lead to a 404 error when users try to access the schema.

     
    2. Create a JSON file in `mcp-registry/servers/` named `[your-server-name].json`. The JSON should follow our [schema](schema/server-schema.json)
3. Submit a pull request
    Path Inconsistency

    The LICENSE file path is referenced as ../LICENSE which assumes the README is in a subdirectory, but this may not match the actual repository structure. This could result in a broken link.

     
    This registry is licensed under the MIT License - see the [LICENSE](../LICENSE) file for details.

    This comment was marked as outdated.

    Sign in to view

    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 30, 2025 •
    edited
    Loading

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @HareeshBahuleyan HareeshBahuleyan requested a review from Copilot June 30, 2025 09:06
    Copilot AI reviewed Jun 30, 2025
    View reviewed changes
    Copy link
    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 updates the mcp-registry README to fix broken GitHub repository links and adjust the documentation to reflect the new folder structure. Key changes include updating the issue URL to the new repository, modifying the contribution instructions to use JSON files instead of directories with manifest files, and correcting schema and LICENSE file references.

    GabrielDrapor
    GabrielDrapor approved these changes Jun 30, 2025
    View reviewed changes
    Copy link
    Contributor

    @GabrielDrapor GabrielDrapor left a comment

    Choose a reason for hiding this comment

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

    Thanks for contributing❤️ @HareeshBahuleyan

    @HareeshBahuleyan
    Copy link
    Contributor Author

    HareeshBahuleyan commented Jun 30, 2025

    Thanks @GabrielDrapor
    The CI test has been running for a while now. When I checked the error seems to be

    ERROR: Missing environment variable: `OPENAI_API_KEY`

    @HareeshBahuleyan HareeshBahuleyan changed the title Update broken links and folder structure in mcp-registry/README.md chore: Update broken links and folder structure in mcp-registry/README.md Jun 30, 2025
    @GabrielDrapor
    Copy link
    Contributor

    GabrielDrapor commented Jun 30, 2025

    @HareeshBahuleyan let's ignore the codex review for now. (somethings's probably wrong with the config, we'll check it later

    @GabrielDrapor GabrielDrapor merged commit be80fa7 into pathintegral-institute:main Jun 30, 2025
    3 checks passed
    @mcpm-semantic-release
    Copy link

    mcpm-semantic-release bot commented Jun 30, 2025

    🎉 This PR is included in version 1.14.1 🎉

    The release is available on GitHub release

    Your semantic-release bot 📦🚀

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Reviewers

    Copilot code review Copilot Copilot left review comments

    @GabrielDrapor GabrielDrapor GabrielDrapor approved these changes

    Assignees

    No one assigned

    Labels

    Review effort 2/5

    Projects

    None yet

    Milestone

    No milestone

    Development

    Successfully merging this pull request may close these issues.

    2 participants

    @HareeshBahuleyan @GabrielDrapor