Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

sep-2640.yaml is extracted at a pre-Final revision of the SEP

オープン
#513 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 7 日以内に返信

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
68/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
typescript
領域
testing

調査の方向性

Start with src/seps/sep-2640.yaml and compare its rows with the Final SEP revision, then inspect src/traceability/index.ts:93 and the traceability entry point in src/index.ts. Run the provided drift check and traceability build to verify the quoted text, requirement coverage, and header counts. Done means the YAML reflects the Final SEP and the reported counts match the traceability output.

索引モデルが issue の本文から書いたものです。

説明

bug

Describe the bug

src/seps/sep-2640.yaml is extracted against SEP commit a3e147ca2710, taken on 2026-08-25 while the CM vote was still open. The file's own provenance header says another pass is expected: "Because the SEP is mid-vote, expect one more pass at the CM-stamped head."

SEP-2640 was marked Accepted on 2026-09-03 (d6b31a03) and Final on 2026-09-11 (8075db4a). Four commits changed the SEP text after the extraction point, and the yaml has not moved since #330 merged. Three things are wrong as a result:

  1. Twelve rows quote sentences the SEP no longer contains. A copy-editing pass (582d814a) replaced em-dashes with commas, colons and parentheses throughout. No requirement changed meaning, but the quoted text: no longer appears in the published document.
  2. One normative requirement has no row. 8079d781 added a paragraph to Skill Format: "In the event that the Agent Skills specification changes in a backwards incompatible way, clients MUST honor any backwards compatibility mechanisms provided by the Agent Skills specification and SHOULD continue to support the Agent Skills specification as it existed prior to any incompatible change." Neither half is declared.
  3. Two capability rows quote superseded wording. fd72d3f1 reworded Capability Declaration: the declaration is now located in "the extensions field of their capabilities" rather than "their initialize response", and the resources-capability requirement is stated there outright instead of being derived from Dependencies. sep-2640-capability-declaration-inline and sep-2640-capability-requires-resources both still quote the old text.

Separately, and true before any of the above, the header's own coverage counts do not match the file.

To Reproduce

  1. Fetch both revisions of the SEP:

    gh api repos/modelcontextprotocol/modelcontextprotocol/contents/seps/2640-skills-extension.md?ref=a3e147ca2710 --jq .content | base64 -d > old.md
    gh api repos/modelcontextprotocol/modelcontextprotocol/contents/seps/2640-skills-extension.md?ref=main --jq .content | base64 -d > new.md
    
  2. Count rows whose text: no longer appears in the Final document:

    python3 - <<'PY'
    import yaml, re
    norm = lambda s: re.sub(r'\s+', ' ', s).strip()
    spec = norm(open('new.md').read())
    old  = norm(open('old.md').read())
    rows = yaml.safe_load(open('src/seps/sep-2640.yaml'))['requirements']
    drift = [r for r in rows if r.get('text')
             and norm(r['text']) not in spec and norm(r['text']) in old]
    print(len(drift), "rows quote the 08-25 text but not the Final text")
    for r in drift: print(" ", r.get('check') or '[excluded]')
    PY
    
  3. Confirm the missing requirement: grep -c "backwards compatibility mechanisms" src/seps/sep-2640.yaml returns 0, while the same grep against new.md returns 1.

  4. Compare the header's declared counts against a real run:

    npm run build
    # run the three server scenarios against any SDK, then:
    npx tsx src/index.ts traceability --results <results-dir>
    

Expected behavior

  • Every row's text: quotes the SEP as published, so a reader can find the sentence a check is derived from. Step 2 should print 0.
  • Every RFC 2119 requirement in the SEP has either a declared row or an excluded: row saying why a harness cannot observe it.
  • The header's coverage counts match what traceability reports.

Logs

Step 2, against the yaml as it stands:

12 rows quote the 08-25 text but not the Final text
   sep-2640-nested-skills-permitted
   sep-2640-skills-list-entry-atomic
   sep-2640-enumeration-scheme-uniform
   sep-2640-host-disambiguate-listing-collision
   sep-2640-entry-resources-required
   sep-2640-resources-complete
   sep-2640-resources-size-required
   sep-2640-skills-get-entry-shape
   sep-2640-skills-get-unknown-uri-invalid-params
   sep-2640-host-no-prefetch
   sep-2640-directory-read-invalid-params
   [excluded]  (stale-entry recovery presentation)

Header counts against a traceability run:

header claims actual
declared checks 89 90
excluded rows 7 8
emitted by the three server scenarios 40 42
untested 49 48

Additional context

The four commits between the extraction point and Final are d6b31a03 (Accepted), 8075db4a (Final), fd72d3f1 (capability wording), 8079d781 (backward-compatibility requirement), plus 582d814a (copy-editing) and b6182df0 / f56f204f, which moved the design-history links from experimental-ext-skills to ext-skills/docs/archive.

One judgment call worth a maintainer's opinion: the two backward-compatibility halves both condition on a breaking revision of the Agent Skills specification that does not yet exist. There is nothing for a harness to present and nothing for a client to honor, so excluded: seems right, but untested checks are defensible if you would rather they stayed visible.

I have this written already as #511, which I opened before reading CONTRIBUTING.md's "open an issue first" — filing this after the fact rather than pretending the order was different. Happy to close either one if you would rather discuss before reviewing a diff.

On what this was and was not checked against. The change cannot move any implementation's score. Matching is r.check against the emitted check-ID set (src/traceability/index.ts:93); text: is carried into the manifest for display only; and the two added rows are excluded:, so they carry no check ID at all. No scenario file is touched.

So what I ran is a regression check, not the cross-implementation validation #330 needed: the three server scenarios against mcpkit examples/skills report 45 pass / 0 fail / 0 warn, unchanged before and after, confirming the yaml still parses and the scenarios still load. I also broke the fixture deliberately — a SKILL.md whose name no longer matches its URI's final segment — and watched the suite go red, then green on restore, which confirms it still grades rather than passing vacuously.

Happy to run it against the TypeScript, Go and C# SDKs as #330 was if you would rather see that, but I do not think it would tell you anything: with no check ID moving and no scenario changed, three more green runs are guaranteed by construction rather than earned. Say the word if you disagree — you know better than I do whether the suite has surprised you this way before.

One downstream effect. The two added excluded: rows change the row counts, so src/seps/traceability.json goes stale and the refresh workflow will open its usual follow-up PR. Nothing fails, but it is easier to expect than to explain.

主要言語
TypeScript
スター
130
フォーク
107
平均マージ
8日 19時間
マージ済み PR(30日)
2

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

modelcontextprotocol/conformance のほかの issue

modelcontextprotocol/conformance の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。