Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

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

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#4,924 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
1/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
92/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
documentation

Hướng nghiên cứu

Bắt đầu với các phần Dependencies và Development setup trong AGENTS.md, sau đó đối chiếu chúng với guard verify:dep-lockstep và các script test và coverage của workspace. Cập nhật mục dependency để nêu quy tắc «một range ở mọi nơi» và ghi tên guard, đồng thời sửa chú thích npm test mà không thay đổi đoạn Before pushing không liên quan. Kiểm tra xem PR của #4871 đã chạm vào một trong hai dòng đó chưa.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
TypeScript
Star
91k
Fork
11.8k
Merge trung bình
6 giờ 47 phút
Pull request đã merge (30 ngày)
74

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của modelcontextprotocol/servers

Tất cả issue của modelcontextprotocol/servers

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.