Skip to content

[WIP] Fix issues with MCPMethodInfo.Arguments removal and optimization - #17

Draft
Daigrin with Copilot wants to merge 4 commits into
mainfrom
copilot/fix-mcpmethodinfo-arguments
Draft

Daigrin with Copilot wants to merge 4 commits into
mainfrom
copilot/fix-mcpmethodinfo-arguments

Conversation

Copilot AI commented Sep 7, 2026 •

Copy link
Copy Markdown
  • Inspect PR perf(http): avoid full argument map allocation in MCP parse middleware #9 state, review feedback, and current local branch context
  • Audit MCPMethodInfo producers, consumers, and tests for Arguments/RawArguments behavior
  • Implement a minimal compatibility/performance fix for MCP parsing
  • Add focused regression tests and any practical benchmark coverage
  • Run formatting and targeted validation (script/lint, focused tests, broader checks as practical)
  • Summarize concrete fixes, results, and remaining risks for the PR context

Copilot AI and others added 3 commits September 7, 2026 04:44
Co-authored-by: Daigrin <198701947+Daigrin@users.noreply.github.com>
Co-authored-by: Daigrin <198701947+Daigrin@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Daigrin/github-mcp-server/sessions/71dde06c-a762-4e60-b24e-16708114b0b2

Co-authored-by: Daigrin <198701947+Daigrin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants