Skip to content

feat: add script and workflow for contributing registry #233

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

Merged
merged 1 commit into from
Aug 15, 2025
Merged

feat: add script and workflow for contributing registry #233

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
81 changes: 81 additions & 0 deletions .github/workflows/generate-manifest.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,81 @@
name: (TESTING) Generate MCP Manifest

on:
workflow_dispatch:
inputs:
repo_url:
description: 'Repository URL to generate manifest for'
required: true
type: string

jobs:
generate-manifest:
runs-on: ubuntu-latest
permissions:
contents: write
pull-requests: write

steps:
- name: Checkout repository
uses: actions/checkout@v4
with:
token: ${{ secrets.GITHUB_TOKEN }}

- name: Set up Python
uses: actions/setup-python@v4
with:
python-version: '3.11'
Comment on lines +25 to +27
Copy link

@coderabbitai coderabbitai bot Aug 15, 2025

Choose a reason for hiding this comment

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

⚠️ Potential issue

Update setup-python action to v5 (Node16 deprecation fix)

actionlint flagged v4 as too old; v5 is required on modern runners.

Apply this diff:

-      - name: Set up Python
-        uses: actions/setup-python@v4
+      - name: Set up Python
+        uses: actions/setup-python@v5
         with:
           python-version: '3.11'
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
uses: actions/setup-python@v4
with:
python-version: '3.11'
- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: '3.11'
🧰 Tools
🪛 actionlint (1.7.7)

25-25: the runner of "actions/setup-python@v4" action is too old to run on GitHub Actions. update the action's version to fix this issue

(action)

🤖 Prompt for AI Agents 
In .github/workflows/generate-manifest.yml around lines 25 to 27, update the
setup-python GitHub Action version from v4 to v5 to address actionlint/runner
deprecation; change the uses line to use actions/setup-python@v5 and keep the
python-version input unchanged so the workflow runs on the modern runner without
Node16-related warnings.


- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install requests

- name: Generate manifest
env:
ANYON_API_KEY: ${{ secrets.ANYON_API_KEY }}
run: |
python scripts/get_manifest.py "${{ github.event.inputs.repo_url }}"

- name: Extract repo name for branch
id: repo-info
run: |
REPO_URL="${{ github.event.inputs.repo_url }}"
REPO_NAME=$(echo "$REPO_URL" | sed 's/.*github\.com[:/]//' | sed 's/\.git$//' | tr '/' '-')
echo "repo_name=$REPO_NAME" >> $GITHUB_OUTPUT
echo "branch_name=add-manifest-$REPO_NAME" >> $GITHUB_OUTPUT

- name: Create Pull Request
uses: peter-evans/create-pull-request@v5
Copy link

@coderabbitai coderabbitai bot Aug 15, 2025

Choose a reason for hiding this comment

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

⚠️ Potential issue

Bump create-pull-request action to v6 (Node16 deprecation fix)

actionlint flagged v5 as too old; v6 uses the supported runtime.

Apply this diff:

-      - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v5
+      - name: Create Pull Request
+        uses: peter-evans/create-pull-request@v6
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
uses: peter-evans/create-pull-request@v5
- name: Create Pull Request
uses: peter-evans/create-pull-request@v6
🧰 Tools
🪛 actionlint (1.7.7)

49-49: the runner of "peter-evans/create-pull-request@v5" action is too old to run on GitHub Actions. update the action's version to fix this issue

(action)

🤖 Prompt for AI Agents 
In .github/workflows/generate-manifest.yml at line 49, the workflow uses
peter-evans/create-pull-request@v5 which is flagged as outdated; update the
action reference to peter-evans/create-pull-request@v6 to use the supported
Node16 runtime and resolve the actionlint warning, then commit the change to the
workflow file.

with:
token: ${{ secrets.GITHUB_TOKEN }}
commit-message: |
feat: add manifest for ${{ steps.repo-info.outputs.repo_name }}

Generated manifest JSON for repository: ${{ github.event.inputs.repo_url }}

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
title: "feat: Add MCP manifest for ${{ steps.repo-info.outputs.repo_name }}"
Comment on lines +52 to +60
Copy link

@coderabbitai coderabbitai bot Aug 15, 2025

Choose a reason for hiding this comment

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

⚠️ Potential issue

Remove trailing spaces in commit-message block (yamllint errors)

YAMLlint reports trailing spaces. Remove them to unblock CI.

Apply this diff:

           commit-message: |
             feat: add manifest for ${{ steps.repo-info.outputs.repo_name }}
-            
+
             Generated manifest JSON for repository: ${{ github.event.inputs.repo_url }}
-            
+
             🤖 Generated with [Claude Code](https://claude.ai/code)
-            
+
             Co-Authored-By: Claude <[email protected]>
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
commit-message: |
feat: add manifest for ${{ steps.repo-info.outputs.repo_name }}
Generated manifest JSON for repository: ${{ github.event.inputs.repo_url }}
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>
title: "feat: Add MCP manifest for ${{ steps.repo-info.outputs.repo_name }}"
commit-message: |
feat: add manifest for ${{ steps.repo-info.outputs.repo_name }}
Generated manifest JSON for repository: ${{ github.event.inputs.repo_url }}
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>
title: "feat: Add MCP manifest for ${{ steps.repo-info.outputs.repo_name }}"
🧰 Tools
🪛 YAMLlint (1.37.1)

[error] 54-54: trailing spaces

(trailing-spaces)

[error] 56-56: trailing spaces

(trailing-spaces)

[error] 58-58: trailing spaces

(trailing-spaces)

🤖 Prompt for AI Agents 
.github/workflows/generate-manifest.yml around lines 52 to 60: the
commit-message block contains trailing spaces at the ends of lines causing
yamllint errors; remove all trailing whitespace characters from each line in
that multi-line scalar (including the blank line after the second line and any
spaces at the end of the emoji/Co-Authored lines), save the file, and rerun the
linter/CI to confirm the YAML no longer reports trailing-space issues.

body: |
## Summary

This PR adds a new MCP server manifest generated from the repository: ${{ github.event.inputs.repo_url }}

## Changes

- Added new manifest JSON file in `mcp-registry/servers/`
- Manifest was automatically generated using the chatxiv.org API

## Test plan

- [ ] Verify the generated JSON is valid
- [ ] Check that all required fields are present
- [ ] Validate installation instructions work correctly

---

🤖 Generated with [Claude Code](https://claude.ai/code)
branch: ${{ steps.repo-info.outputs.branch_name }}
Comment on lines +61 to +80
Copy link

@coderabbitai coderabbitai bot Aug 15, 2025

Choose a reason for hiding this comment

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

⚠️ Potential issue

Remove trailing spaces in PR body (yamllint errors)

Clean up blank lines with trailing whitespace.

Apply this diff:

           body: |
             ## Summary
-            
+
             This PR adds a new MCP server manifest generated from the repository: ${{ github.event.inputs.repo_url }}
-            
+
             ## Changes
-            
+
             - Added new manifest JSON file in `mcp-registry/servers/`
             - Manifest was automatically generated using the chatxiv.org API
-            
+
             ## Test plan
-            
+
             - [ ] Verify the generated JSON is valid
             - [ ] Check that all required fields are present
             - [ ] Validate installation instructions work correctly
-            
+
             ---
-            
+
             🤖 Generated with [Claude Code](https://claude.ai/code)
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
body: |
## Summary
This PR adds a new MCP server manifest generated from the repository: ${{ github.event.inputs.repo_url }}
## Changes
- Added new manifest JSON file in `mcp-registry/servers/`
- Manifest was automatically generated using the chatxiv.org API
## Test plan
- [ ] Verify the generated JSON is valid
- [ ] Check that all required fields are present
- [ ] Validate installation instructions work correctly
---
🤖 Generated with [Claude Code](https://claude.ai/code)
branch: ${{ steps.repo-info.outputs.branch_name }}
body: |
## Summary
This PR adds a new MCP server manifest generated from the repository: ${{ github.event.inputs.repo_url }}
## Changes
- Added new manifest JSON file in `mcp-registry/servers/`
- Manifest was automatically generated using the chatxiv.org API
## Test plan
- [ ] Verify the generated JSON is valid
- [ ] Check that all required fields are present
- [ ] Validate installation instructions work correctly
---
🤖 Generated with [Claude Code](https://claude.ai/code)
branch: ${{ steps.repo-info.outputs.branch_name }}
🧰 Tools
🪛 YAMLlint (1.37.1)

[error] 63-63: trailing spaces

(trailing-spaces)

[error] 65-65: trailing spaces

(trailing-spaces)

[error] 67-67: trailing spaces

(trailing-spaces)

[error] 70-70: trailing spaces

(trailing-spaces)

[error] 72-72: trailing spaces

(trailing-spaces)

[error] 76-76: trailing spaces

(trailing-spaces)

[error] 78-78: trailing spaces

(trailing-spaces)

🤖 Prompt for AI Agents 
.github/workflows/generate-manifest.yml around lines 61 to 80: the PR body
contains blank lines and lines with trailing whitespace causing yamllint errors;
remove any trailing spaces at the ends of lines and clean up unnecessary blank
lines in the multiline body block (preserve content and indentation, but ensure
no lines end with spaces and there are no extra blank lines).

delete-branch: true
Loading
Loading