Skip to content

chore(mcp): glama.json 移到仓库根目录并声明 maintainers,解锁 Glama 认领 - #511

Merged
modusensus merged 1 commit into
mainfrom
chore/glama-root-claim
Oct 23, 2026
Merged

modusensus merged 1 commit into
mainfrom
chore/glama-root-claim

Conversation

@modusensus

@modusensus modusensus commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

外部清单收录 PR(punkpeye/awesome-mcp-servers#14974)卡在最后一步:仓库主人 punkpeye 要求先在 Glama 完成两件事——认领服务器、生成质量评分——才合并。而 Glama 对组织名下服务器的认领流程(认领弹层 FAQ 原文)要求:

在仓库根目录放一份 glama.json,其中 maintainers 声明 GitHub 用户名,然后用该账号 Login with GitHub 认领。

现状问题

问题 说明
位置不对 glama.json 一直躺在 mcp/ 子包里(当时随 Glama 提交一起放进包内),认领流程只读仓库根目录,读不到
缺字段 只有 author,没有 maintainers —— 组织仓库不声明 maintainers,所有权验证无从谈起,Glama 页面始终显示未认领

改动

  • git mv mcp/glama.json glama.json(git 按 rename 识别,其余元数据一字未动);
  • 开头新增两个字段:
"$schema": "https://glama.ai/mcp/schemas/server.json",
"maintainers": ["modusensus"],

影响面

  • 不碰 src/,无需 npm run sync;纯元数据搬移,无行为变更,不配新测试;
  • npm 发布不受影响:mcp/package.json 没有 files 清单锁,glama.json 本也不属于 npm 包语义——Glama 只从 GitHub 仓库根目录读它;
  • 仓库内无代码引用 mcp/glama.json 这条路径(仅 CHANGELOG 提过一次 glama 提交);
  • 无对应 Issue:一文件元数据修复,动机即上述外部收录 PR。

Summary by CodeRabbit

  • 维护
    • 更新项目元数据,补充 Glama server schema 声明,并将 modusensus 列为维护者。
    • 此次更新未改变面向用户的功能。

Copilot AI lite review requested due to automatic review settings October 23, 2026 23:13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
📥 Commits

Reviewing files that changed from the base of the PR and between cecd2b5 and 9431534.

📒 Files selected for processing: 1
  • glama.json

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 0 remain after this review.

📝 Walkthrough

glama.json 新增 Glama server schema 声明,并将 modusensus 列为维护者。

Changes: Glama 元数据

Layer / File(s) Summary
添加 Glama 元数据
glama.json
新增 Glama server schema 地址,并列出维护者 modusensus。

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 94315

The Glama metadata update presents no merge-blocking risk.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed 标题准确概括了主要变更:将 glama.json 移至仓库根目录并声明维护者,以支持 Glama 认领流程。
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@codecov

codecov Bot commented Oct 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@modusensus
modusensus merged commit 3f6696f into main Oct 23, 2026
11 checks passed
@modusensus
modusensus deleted the chore/glama-root-claim branch October 23, 2026 23:22
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