AGENTS.md: the dependency lockstep bullet and the npm test annotation are stale
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 1/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 92/100
- Issue 类型
- 文档
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- typescript
调研方向
先从 AGENTS.md 中的 Dependencies 和 Development setup 部分开始，然后将其与 verify:dep-lockstep guard 以及 workspace 的 test 和 coverage 脚本进行比较。更新 dependency 条目，说明“各处使用同一个 range”的规则并写明 guard，同时更正 npm test 注释，但不要修改无关的 Before pushing 段落。检查 #4871 的 PR 是否已经改动过其中任一行。
由索引模型根据 Issue 内容生成。
描述
Found while writing the knowledge skills (#4872, PR #4921). Two statements in AGENTS.md no longer match the repository, and AGENTS.md says of itself that every rule in it is true of the repository as it stands.
1. Dependencies: the shared-devDependency bullet contradicts the guard
AGENTS.md → Dependencies says:
When you bump a devDependency that several TypeScript workspaces declare (
typescript,vitest,@vitest/coverage-v8,@types/node, …), bump it in every workspace that declares it. Their ranges already differ; do not widen the skew.
Since #4864, npm run verify:dep-lockstep (in validate:guards, so in CI) fails when two declarations of a package in SHARED_DEV_DEPENDENCIES (typescript, vitest, @vitest/coverage-v8, prettier, @types/node) carry different ranges. The ranges no longer differ, and "do not widen the skew" understates a rule that is now "one range everywhere it is declared". The local-dev skill describes the guard, so an agent gets a different instruction depending on whether the skill loaded. Copilot raised this on PR #4921 (round 3).
Expected: the bullet states the lockstep rule the guard enforces and names the guard.
2. Development setup: npm test is annotated "with coverage"
The command block says:
npm test -w src/<server> # one workspace (vitest, with coverage)
Each workspace's test script is vitest run. Coverage is the separate coverage script (vitest run --coverage).
Expected: the annotation drops "with coverage", or the block lists npm run coverage -w src/<server> on its own line.
Not included
The Before pushing paragraph saying CI does not run ruff check is also stale (python.yml runs scripts/validate-py.mjs, which does), but #4871's PR rewrites that section, so it is left to that change.
Branch
Targets v2/main. If #4871's PR is still open when this is picked up, check whether it already touched either line.
- 主要语言
- TypeScript
- 星标
- 91.4k
- 派生
- 11.8k
- 平均合并
- 7 小时 51 分钟
- 30 天内合并 PR
- 121
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
modelcontextprotocol/servers 的其他 Issue
-
documentation v2
难度 2/5 1-3 小时 新手友好度 78/100
modelcontextprotocol/servers#5170 ·
维护者通常 1 天内回复
-
documentation v2
难度 2/5 1-3 小时 新手友好度 72/100
modelcontextprotocol/servers#5164 ·
维护者通常 1 天内回复
-
memory: persistence EISDIR test fails on Node 26.7 (the path suffix depends on the Node minor, not the major)可能已有人在做 @ace-ify 今天认领。 未关闭bug server-memory v2
难度 2/5 1-3 小时 新手友好度 83/100
modelcontextprotocol/servers#5162 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
modelcontextprotocol/servers#5156 ·
维护者通常 1 天内回复
-
memory: create_entities, create_relations and add_observations could declare idempotentHint: true可能已有人在做 @syedmustafan 于 2 天前认领。 未关闭enhancement v2
难度 1/5 1 小时以内 新手友好度 78/100
modelcontextprotocol/servers#5153 · 2 条评论 ·
维护者通常 1 天内回复
查看 modelcontextprotocol/servers 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 76/100
ccch1mneyyy/dsh-TUI#1631 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
MiniMax-AI/minimax-code#507 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
-
Transient OAuth login failure persists in getLoginStatus and masks active account in dashboard可能重新可做 关联的 PR 已关闭且未合并。 未关闭account-pool bug gui
难度 2/5 1-3 小时 新手友好度 67/100
lidge-jun/opencodex#7378 · 2 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复