Report created tasks when create_tasks hits a transport error
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 68/100
調査の方向性
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.”
索引モデルが issue の本文から書いたものです。
説明
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
- 主要言語
- C#
- スター
- 1
- フォーク
- 0
- 平均マージ
- 2時間 2分
- マージ済み PR（30日）
- 98
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nevridge/TaskFlow のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
tech-debt
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
nevridge/TaskFlow の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
juandresrodca/AdminForge#54 ·
-
要トリアージ 辞書
難易度 1/5 1〜3時間 初心者へのやさしさ 86/100
yksr-melt/Meltype#1625 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Docs typoオープン
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
MonoGame/docs.monogame.github.io#328 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
space-wizards/RobustToolbox#7197 ·
メンテナーはふだん 1 日以内に返信
-
documentation high priority
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
lachydotmcg/capri-desktop#201 ·
メンテナーはふだん 1 日以内に返信