Skip to content

Commit d024b8b

Browse files
panyampanyam
committed
docs: update example + plan references to point at fork conformance
The bulk of the v2 tasks + MRTR conformance lives in the upstream-bound fork now (panyam/mcpconformance, branch feat/tasks-mrtr-extension; upstream Draft PR modelcontextprotocol/conformance#262). Updates the README/WALKTHROUGH/walkthrough.go references in examples/tasks-v2 + examples/mrtr to point at the fork, the migration guide (docs/TASKS_V2_MIGRATION.md) likewise, and the matching Go test skip (server/mrtr_test.go) to point at the new conformance scenario path. No runtime changes.
1 parent 9e71417 commit d024b8b

8 files changed

Lines changed: 14 additions & 13 deletions

File tree

‎docs/TASKS_V2_MIGRATION.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,4 +210,4 @@ The expected flow for a deployment migrating from v1 to v2 without downtime:
210210
- SEP-2243 (Mcp-Name HTTP header): adopted from spec discussion
211211
- Implementation plan + open questions: [`PLAN.md`](../PLAN.md)
212212
- v2 example walkthrough: [`examples/tasks-v2/WALKTHROUGH.md`](../examples/tasks-v2/WALKTHROUGH.md)
213-
- v2 conformance suite: [`conformance/tasks-v2/scenarios.test.ts`](../conformance/tasks-v2/scenarios.test.ts) (33 scenarios)
213+
- v2 conformance suite: [panyam/mcpconformance — `src/scenarios/server/tasks/`](https://github.com/panyam/mcpconformance/tree/feat/tasks-mrtr-extension/src/scenarios/server/tasks) (8 ClientScenario classes / ~33 internal checks; upstream Draft PR modelcontextprotocol/conformance#262). Local sentinel for mcpkit-stricter scenarios: [`conformance/tasks-v2/`](../conformance/tasks-v2/).

‎examples/mrtr/README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,5 +77,5 @@ testconf-mrtr` from the repo root.
7777
| Tool handler API | [`core/handler_context.go`](../../core/handler_context.go) — `ctx.RequestInput` sentinel + `InputResponse(key)` / `HasInputResponses()` / `RequestState()` accessors |
7878
| Client auto-loop | [`client/mrtr.go`](../../client/mrtr.go) — `CallToolWithInputs` + `DefaultInputHandler` |
7979
| Single dispatcher | [`client.HandleServerRequestWithContext`](../../client/client.go) — same switch handles real server-initiated requests AND MRTR-synthesized ones |
80-
| Conformance | [`conformance/mrtr/scenarios.test.ts`](../../conformance/mrtr/scenarios.test.ts) |
80+
| Conformance | [panyam/mcpconformance — `src/scenarios/server/mrtr/`](https://github.com/panyam/mcpconformance/tree/feat/tasks-mrtr-extension/src/scenarios/server/mrtr) (upstream Draft PR modelcontextprotocol/conformance#262); local sentinel: [`conformance/mrtr/`](../../conformance/mrtr/) |
8181
| SEP | [SEP-2322 spec PR](https://github.com/modelcontextprotocol/specification/pull/2322) |

‎examples/mrtr/WALKTHROUGH.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ The wire only ships the LATEST round's `inputResponses`. Dispatch decodes prior
8787
- Tool handler API: `ctx.RequestInput(reqs)` sentinel + `ctx.InputResponse(key)` / `HasInputResponses()` / `RequestState()` accessors — core/handler_context.go
8888
- Client auto-loop: `client.CallToolWithInputs` + `DefaultInputHandler` — client/mrtr.go
8989
- Client dispatch unification: `client.HandleServerRequestWithContext` — single switch for both real server-initiated requests AND MRTR-synthesized ones — client/client.go
90-
- Conformance: `conformance/mrtr/scenarios.test.ts` (7 scenarios + 1 skipped composition; `make testconf-mrtr`)
90+
- Conformance: panyam/mcpconformance fork (`src/scenarios/server/mrtr/`, 7 checks + 1 SKIPPED composition; upstream Draft PR modelcontextprotocol/conformance#262; `make testconf-mrtr` runs it)
9191
- SEP-2322 spec: https://github.com/modelcontextprotocol/specification/pull/2322
9292

9393
## Run it

‎examples/mrtr/walkthrough.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ func runDemo() {
156156
"- Tool handler API: `ctx.RequestInput(reqs)` sentinel + `ctx.InputResponse(key)` / `HasInputResponses()` / `RequestState()` accessors — core/handler_context.go",
157157
"- Client auto-loop: `client.CallToolWithInputs` + `DefaultInputHandler` — client/mrtr.go",
158158
"- Client dispatch unification: `client.HandleServerRequestWithContext` — single switch for both real server-initiated requests AND MRTR-synthesized ones — client/client.go",
159-
"- Conformance: `conformance/mrtr/scenarios.test.ts` (7 scenarios + 1 skipped composition; `make testconf-mrtr`)",
159+
"- Conformance: panyam/mcpconformance fork (`src/scenarios/server/mrtr/`, 7 checks + 1 SKIPPED composition; upstream Draft PR modelcontextprotocol/conformance#262; `make testconf-mrtr` runs it)",
160160
"- SEP-2322 spec: https://github.com/modelcontextprotocol/specification/pull/2322",
161161
)
162162

‎examples/tasks-v2/README.md‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,15 +57,15 @@ See [WALKTHROUGH.md](WALKTHROUGH.md) for the full step-by-step description and s
5757

5858
## Conformance
5959

60-
- 21 scenarios in `conformance/tasks-v2/scenarios.test.ts` (Node + tsx)
61-
- Implementation plan: [`PLAN.md`](../../PLAN.md) (SEP-2663, marked COMPLETE)
60+
- Scenarios live in the [`panyam/mcpconformance`](https://github.com/panyam/mcpconformance) fork (branch `feat/tasks-mrtr-extension`, upstream Draft PR modelcontextprotocol/conformance#262). Run via `make testconf-tasks-v2` from the repo root — points the fork's vitest run at this binary.
6261
- Migration guide: [`docs/TASKS_V2_MIGRATION.md`](../../docs/TASKS_V2_MIGRATION.md)
63-
- Spec: [SEP-2557](https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2557)
62+
- Spec: [SEP-2663](https://github.com/modelcontextprotocol/specification/pull/2663)
6463

6564
## Where to look in the code
6665

6766
- v1 example: [`examples/tasks/`](../tasks/)
6867
- Server library: [`server/tasks_v2.go`](../../server/tasks_v2.go)
6968
- Wire types: [`core/task_v2.go`](../../core/task_v2.go)
7069
- Client helpers: [`client/tasks.go`](../../client/tasks.go) (`ToolCall`, `GetTask`, `WaitForTask`, `UpdateTask`, `CancelTask`)
71-
- Conformance: [`conformance/tasks-v2/scenarios.test.ts`](../../conformance/tasks-v2/scenarios.test.ts)
70+
- Conformance scenarios: [panyam/mcpconformance — `src/scenarios/server/tasks/`](https://github.com/panyam/mcpconformance/tree/feat/tasks-mrtr-extension/src/scenarios/server/tasks)
71+
- Local sentinel for mcpkit-stricter scenarios: [`conformance/tasks-v2/`](../../conformance/tasks-v2/)

‎examples/tasks-v2/WALKTHROUGH.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,8 @@ Same cooperative cancellation as v1 (server cancels the goroutine context; tools
120120
- v2 server library: `server/tasks_v2.go` (RegisterTasks, gating, MRTR runtime)
121121
- v2 wire types (`CreateTaskResult`, `DetailedTask`, `TaskInfoV2`, `UpdateTaskRequest`, `ResultTypeTask`): `core/task_v2.go` (SEP-2663)
122122
- v2 client helpers (`ToolCall`, `GetTask`, `UpdateTask`, `WaitForTask`, `CancelTask`): `client/tasks.go`
123-
- Conformance tests: `conformance/tasks-v2/scenarios.test.ts`
124-
- Implementation plan + open questions: `PLAN.md`
123+
- Conformance scenarios: panyam/mcpconformance fork (branch feat/tasks-mrtr-extension, upstream Draft PR modelcontextprotocol/conformance#262)
124+
- Local sentinel for mcpkit-stricter tests: `conformance/tasks-v2/`
125125

126126
## Run it
127127

‎examples/tasks-v2/walkthrough.go‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -313,8 +313,8 @@ func runDemo() {
313313
"- v2 server library: `server/tasks_v2.go` (RegisterTasks, gating, MRTR runtime)",
314314
"- v2 wire types (`CreateTaskResult`, `DetailedTask`, `TaskInfoV2`, `UpdateTaskRequest`, `ResultTypeTask`): `core/task_v2.go` (SEP-2663)",
315315
"- v2 client helpers (`ToolCall`, `GetTask`, `UpdateTask`, `WaitForTask`, `CancelTask`): `client/tasks.go`",
316-
"- Conformance tests: `conformance/tasks-v2/scenarios.test.ts`",
317-
"- Implementation plan + open questions: `PLAN.md`",
316+
"- Conformance scenarios: panyam/mcpconformance fork (branch feat/tasks-mrtr-extension, upstream Draft PR modelcontextprotocol/conformance#262)",
317+
"- Local sentinel for mcpkit-stricter tests: `conformance/tasks-v2/`",
318318
)
319319

320320
common.SetupRenderer(demo)

‎server/mrtr_test.go‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,8 @@ func TestMRTR_MultiRoundAccumulatesAnswers(t *testing.T) {
366366
// Resolving this is a real design choice (always-sync handler vs. handler-
367367
// signalled async) tracked as mcpkit issue 347. Re-enable by deleting the
368368
// t.Skip once that lands. A matching scenario lives skipped in the
369-
// conformance suite (conformance/mrtr/scenarios.test.ts:mrtr-08).
369+
// conformance suite (panyam/mcpconformance, branch feat/tasks-mrtr-extension,
370+
// src/scenarios/server/mrtr/ephemeral-flow.ts:mrtr-tasks-composition).
370371
func TestMRTR_TaskComposition_Skipped(t *testing.T) {
371372
t.Skip("MRTR→Tasks composition deferred — tracking: mcpkit issue 347")
372373

0 commit comments

Comments
 (0)