Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

AGENTS.md: the dependency lockstep bullet and the npm test annotation are stale

Geschlossen Anfängerfreundlich
  • Stimmt etwas nicht?
#4,924 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
1/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
92/100
Issue-Typ
Dokumentation
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
typescript
Bereich
documentation

Rechercherichtung

Beginne mit den Abschnitten Dependencies und Development setup in AGENTS.md und vergleiche sie anschließend mit dem Guard verify:dep-lockstep sowie den Workspace-Test- und Coverage-Skripten. Aktualisiere den Dependency-Aufzählungspunkt so, dass er die Regel „ein Range überall“ nennt und den Guard benennt, und korrigiere die npm test-Anmerkung, ohne den nicht dazugehörigen Absatz Before pushing zu ändern. Prüfe, ob der PR zu #4871 bereits eine der beiden Zeilen geändert hat.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

agent guidance documentation v2

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.

Vorherrschende Sprache
TypeScript
Sterne
91k
Forks
11.8k
Ø Merge
6 Std. 33 Min.
Gemergte PRs (30 T.)
79

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus modelcontextprotocol/servers

Alle Issues in modelcontextprotocol/servers

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.