Report created tasks when create_tasks hits a transport error

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

Maintainer thường phản hồi trong vòng 1 ngày

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

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
68/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
csharp
Lĩnh vực
api, backend

Hướng nghiên cứu

Read assistant/src/TaskFlow.Mcp/Tools/TaskTools.cs around line 392 and compare create_tasks with the existing transport error handling in update_tasks. Then inspect assistant/src/TaskFlow.Mcp/Tools/JournalTools.cs around lines 194–216 for set_journal_todos. Done when both tools stop after a transport failure and report created items plus remaining items as “not attempted.”

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

Mô tả

bug

Follow-up from #413 (chore: move taskflow-assistant into assistant/), raised during review.

Problem

create_tasks catches only McpException and TaskFlowApiException per item. TaskFlowApiClient does not wrap transport errors, so a timeout (TaskCanceledException) or HttpRequestException on item N escapes the loop. ToolGuard then returns a generic "cannot reach / did not respond" error and the report of the items already created is lost.

Scenario: a 25-item batch times out at item 10. Tasks 1-9 exist in the API, but the tool result lists none of them. The model retries the whole batch and creates duplicates. update_tasks already handles this case with "not attempted" lines, so the two batch tools behave differently. set_journal_todos (JournalTools.cs, around lines 194-216) only catches TaskFlowApiException and has the same gap.

Severity: 🟠 Important. Pre-existing in the imported assistant code, not introduced by #413.

Location

assistant/src/TaskFlow.Mcp/Tools/TaskTools.cs:392 at 3c3e6b4 (permalink)

Suggested approach

Handle transport failures the way update_tasks does: on a transport error, stop the loop, report the items already created, and list the remaining items as "not attempted". Apply the same to set_journal_todos. Deferred from #413 because the assistant code was imported unchanged and feature work there waits for #400.

Created by @nevridge with Claude Code during review of #413

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

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

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 nevridge/TaskFlow

Tất cả issue của nevridge/TaskFlow

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.