Approval-gated action interrupted before its result is saved can never be retried
Maintainer thường phản hồi trong vòng 2 ngày
@DerekChia đang làm issue này rồi.
Từ ngày 10/10/2026.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 68/100
Hướng nghiên cứu
Start with the action receipt and the claimAction/finishAction flow around the mcp_actions table and the POST /api/conversations/thread/connection-actions endpoint. Also trace the approval action card’s polling and retry controls, using the 60-second ConnectionService timeout as the stale-run threshold. Done means an interrupted result-less run can be reclaimed only by its original approval, fresh claims still receive 409, saved results remain protected, and the UI exposes a successful retry.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
If the server stops between claiming an approval-gated MCP action (claimAction) and saving its result (finishAction), that (threadId, toolCallId) is bricked: every retry returns 409 This action is already running. forever, because mcp_actions rows have no expiry and there is no code path that clears a result-less running row.
Worse, the owner has no way to retry from the UI: the action card polls the receipt every 2s, sees status: running, shows "Running / This action is still running on the server.", and hides the Approve buttons while running is true. The card sits on "Running" forever with no affordance.
Reproduction
- Start OpenDots (local
npm run dev, Node v22.23.1) with a Dot that has an MCP connection with a gated (non-read-only) tool. - Have the Dot call the gated tool so the owner gets an approval card.
- Approve it, and kill the server process after the claim INSERT but before the result UPDATE lands (equivalently:
INSERT INTO mcp_actions(threadId,toolCallId,connectionId,tool,status,result,createdAt,approvalId) VALUES('thread','tc1','c','tool','running',NULL,<now>,'<approvalId>');). - Restart the server and open the conversation.
- The card shows "Running" forever; approving/refreshing never completes.
POST /api/conversations/thread/connection-actionswith the same approval returns409 {"error":"This action is already running."}on every attempt, until the approval itself expires an hour later.
Expected: after the in-flight run is demonstrably gone, the receipt should surface the interruption and the same approval should be able to run the stored request again (the request is already bound to its exact connection/tool/arguments, and only that approval may reclaim it).
Actual: misleading "already running" 409 with no self-healing and no owner retry path.
Notes
- MCP tool calls are bounded to 60s (
ConnectionServicetimeout), so a result-less claim older than a few minutes cannot belong to a live run. - The fix must preserve the double-claim guard: a fresh
runningrow must still 409 (double click / retried request must not execute twice), a saved result must still be returned only to its own approval, and a different approval must never steal the claim.
- Ngôn ngữ chính
- TypeScript
- Star
- 4.8k
- Fork
- 690
- Merge trung bình
- 2 ngày 5 giờ
- Pull request đã merge (30 ngày)
- 53
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của CopilotKit/OpenDots
-
Page conversation panel overflows horizontallyCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
CopilotKit/OpenDots#69 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Long page titles are clipped in the page editorCó thể đã có người làm @charan-rathore đã nhận 12 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
CopilotKit/OpenDots#68 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Add a way to delete specialist DotsCó thể đã có người làm @XENONCYBER đã nhận 1 ngày trước. Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
CopilotKit/OpenDots#151 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 8/100
CopilotKit/OpenDots#149 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Adopt Knip with staged cleanup and blocking CI checksCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
CopilotKit/OpenDots#147 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của CopilotKit/OpenDots
Issue tương tự
-
bnv-manager-v2: accept_invite creates the Mailcow domain admin before the commitCó thể đã có người làm @myOmikron đã nhận hôm nay. Đang mởbug ready-for-agent
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
Windows: mio 1.2.3 named-pipe use-after-free panics the native IPC broker and hangs every toolĐang mởplatform:windows tool triaged wontfix
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
nicolettas-muggelbude/RechnungsFee#448 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Android bug feature/error-tracking react-native team/error-tracking
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
PostHog/posthog-js#5310 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày