Skip to content

Enable complete Tasks extension conformance coverage - #1741

Merged
PranavSenthilnathan merged 2 commits into
modelcontextprotocol:mainfrom
PranavSenthilnathan:pranavsenthilnathan-complete-task-conformance
Jul 27, 2026
Merged

Enable complete Tasks extension conformance coverage#1741
PranavSenthilnathan merged 2 commits into
modelcontextprotocol:mainfrom
PranavSenthilnathan:pranavsenthilnathan-complete-task-conformance

Conversation

@PranavSenthilnathan

Copy link
Copy Markdown
Contributor

Summary

  • add Tasks-owned per-call execution modes for synchronous, optional, and required task execution
  • allow WithTasks callers to select execution behavior from the tool request without introducing Tasks dependencies in Core or ASP.NET Core
  • enable all ten SEP-2663 Tasks extension conformance scenarios
  • cover mode selection, capability enforcement, legacy protocol behavior, and invalid configuration

Validation

  • dotnet build (0 warnings, 0 errors)
  • all ten Tasks conformance scenarios on .NET 8, .NET 9, and .NET 10 using @modelcontextprotocol/conformance@0.2.0-alpha.9
  • full Core tests on .NET 8, .NET 9, .NET 10, and .NET Framework 4.7.2
  • full ASP.NET Core tests on .NET 8, .NET 9, and .NET 10
  • analyzer and experimental API regression tests

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7c99928c-a943-49f4-9a63-8b04a0e4d18b
@PranavSenthilnathan
PranavSenthilnathan marked this pull request as ready for review July 25, 2026 02:55
Comment thread tests/ModelContextProtocol.Tests/Server/McpTaskStoreTests.cs
jeffhandley
jeffhandley previously approved these changes Jul 27, 2026

@jeffhandley jeffhandley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one minor question/suggestion.

tarekgh
tarekgh previously approved these changes Jul 27, 2026

@tarekgh tarekgh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Modulo @jeffhandley comment, LGTM!

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7c99928c-a943-49f4-9a63-8b04a0e4d18b
@PranavSenthilnathan
PranavSenthilnathan dismissed stale reviews from tarekgh and jeffhandley via 0993bc9 July 27, 2026 19:11
@PranavSenthilnathan
PranavSenthilnathan enabled auto-merge (squash) July 27, 2026 19:13
@PranavSenthilnathan
PranavSenthilnathan merged commit 7838900 into modelcontextprotocol:main Jul 27, 2026
10 checks passed
@PranavSenthilnathan
PranavSenthilnathan deleted the pranavsenthilnathan-complete-task-conformance branch July 27, 2026 19:51
@jeffhandley jeffhandley mentioned this pull request Jul 28, 2026
@jeffhandley jeffhandley mentioned this pull request Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants