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

MCP: one error model and one serializer for every tool (validation errors are returned as success)

Đang mở
  • Có gì không ổn?
#92 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
35/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
csharp
Lĩnh vực
api, backend, cli

Hướng nghiên cứu

Start with McpInvocationFilter, McpStdioToolCallTests.McpToolCall_WhenTheApiFails_IsReportedAsAToolError, and OutputHelper.WriteApiError to trace the existing error contract. Then inspect Features/Mcp/Tools, Program.cs, and McpHostCommand for duplicated handling and registration. Done means validation, write-service, serialization, logging, and registration behavior are unified without unreviewed golden or wire-contract changes.

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

Mô tả

Priority: P1 Type: Refactor

Why

Sharing was started but the MCP side of the contract was never finished, so every other slice of #38 inherits the problem.

  • Validation and auth failures are returned as successful tool results (return """{"error": "..."}"""), about 20 sites across TimesheetMcpTools, LeaveMcpTools and AccountingMcpTools. API failures throw, so the SDK marks them isError. A client checking isError sees the first class as success.
  • A third class escapes as a protocol error: DateOnly.ParseExact in the tools throws FormatException for bad input, even though TimesheetCreateService.ParseDate already turns the same input into a validation exception.
  • TimesheetWriteResult.Success == false is an error on the CLI (UpdateCommand, AcceptCommand, CreateCommand) but serialised as ordinary content by the MCP tools.
  • McpInvocationFilter only records failure on a throw, so the local command log shows returned errors as successes.
  • Serialiser policy is split: OutputHelper.JsonOptions omits nulls, AccountingMcpTools copies that, the timesheet/leave/lookup tool classes write nulls, and CreateTimesheet alone uses OutputHelper.SerializeJson. This is why parity rows need PermittedDifferences like $.timesheet.invoiceId.
  • Adapter boilerplate is copied: three ResolveEmpId helpers with two different semantics, three "not logged in" strings, and the DI registration list duplicated between Program.cs and McpHostCommand (a new service must be registered twice).

Migrate

  • One base validation exception (or error record) that TimesheetValidationException, the leave validation exceptions and the import exceptions derive from; carry message, detail, recovery hint and mayHaveBeenApplied.
  • Map it once, in McpInvocationFilter, to a CallToolResult with IsError = true and the same {"error":{code,message,detail}} body the CLI --json envelope uses. Record the failure in the command log.
  • Remove every catch … return "{\"error\"…}" and every JsonOpts field from Features/Mcp/Tools; serialise through OutputHelper.SerializeJson.
  • Parse dates in tools through one invariant helper that throws the validation exception.
  • Treat Success == false from a write service the same way on both surfaces.
  • One AddTimeProServices() extension used by Program.cs and McpHostCommand; one shared tool helper for tenant/emp-id resolution.

How

Do this first; the other sharing issues assume it. McpStdioToolCallTests.ToolCall_WhenTheApiFails_IsReportedAsAToolError already proves the SDK maps a thrown exception to isError, so the filter can own the mapping and tools stop catching anything. OutputHelper.WriteApiError is the CLI half to mirror for detail and requestId.

Contract

Error payloads and null omission change MCP output. The code can land in a patch behind the existing goldens only if the old shapes are preserved; otherwise hold the wire change for 0.4.0 and regenerate goldens once (#71 first, so the regenerate is reviewable).

Part of #38.

Ngôn ngữ chính
C#
Star
3
Fork
0
Merge trung bình
6 phút
Pull request đã merge (30 ngày)
1

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

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

  • Không có Dockerfile hay tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

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 SSWConsulting/TimePro.Tools

Tất cả issue của SSWConsulting/TimePro.Tools

Issue tương tự

Thêm issue về C#

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.