AGENTS.md: the dependency lockstep bullet and the npm test annotation are stale
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 92/100
- Tipo di issue
- Documentazione
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- documentation
Direzione di ricerca
Inizia dalle sezioni Dependencies e Development setup in AGENTS.md, quindi confrontale con il guard verify:dep-lockstep e gli script di workspace per test e coverage. Aggiorna il punto elenco della dipendenza in modo che indichi la regola «un range ovunque» e nomini il guard, quindi correggi l’annotazione di npm test senza modificare il paragrafo non correlato Before pushing. Verifica se il PR di #4871 ha già modificato una delle due righe.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- TypeScript
- Stelle
- 91k
- Fork
- 11.8k
- Merge medio
- 6h 47m
- PR unite (30g)
- 74
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di modelcontextprotocol/servers
-
git: git_create_branch is marked non-destructive but silently resets an existing packed branchAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
modelcontextprotocol/servers#5059 ·
I maintainer di solito rispondono entro 1 giorno
-
mcp-server-fetch: `fetch` prompt returns JSON-RPC error code 0 with the raw exception text for an invalid URLForse già presa @DawnofGenX l’ha presa 7 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
modelcontextprotocol/servers#4914 ·
I maintainer di solito rispondono entro 1 giorno
-
CLAUDE.md: tool-naming rule (kebab-case) disagrees with filesystem and memory serversForse già presa @liang0417 l’ha presa 8 giorni fa. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
modelcontextprotocol/servers#4892 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Filesystem README recommends deprecated MCP Roots protocol for restricting directory accessForse già presa @its-amann l’ha presa 13 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
modelcontextprotocol/servers#4844 ·
I maintainer di solito rispondono entro 1 giorno
-
Docs: `fetch` installs npm packages during a tool call, which is worth stating for deploymentsForse già presa @teddiesloco l’ha presa 17 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
modelcontextprotocol/servers#4830 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di modelcontextprotocol/servers
Issue simili
-
refactor
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
tomnewport/memprot-topo#55 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
WalletConnect/walletconnect-monorepo#7368 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
BU-Spark/se-chem-apll#47 ·
-
embed: handleTurboSignMessage header comment says the signing page posts to '*' (it never does)Apertadocumentation
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 Mezza giornata Idoneità per principianti 70/100
udistrital/paginaweb_root#23 ·