Composio follow-up: bound startup cancellation and forward Compose API key
Maintainer thường phản hồi trong vòng 2 ngà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
- 25/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- docker-compose, typescript
Hướng nghiên cứu
Start in src/server/dot-agent.ts, where Composio session creation and tool discovery are awaited before the inner agent is created, and trace how the 90-second turn-timeout abort reaches the observable. The payload names a regression probe that mocks composioSession with a never-settling promise and advances fake timers by 91 seconds, so reproduce that first. Done means the completion callback fires on timeout, cancellation stops the inner agent from starting, and compose.yml forwards an optional COMPOSIO_API_KEY. Pull request #156 is already open against this issue, so check it before starting.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Follow-up to #150 and the review findings, verified against 268772f8a395ad92f98ff33698f90713f1aca756.
1. Finish turns when Composio startup is cancelled or times out
src/server/dot-agent.ts awaits Composio session creation and tool discovery before creating the inner agent. These requests receive no abort signal. If either stalls, the 90-second turn timeout aborts the controller but does not complete the observable; the turn stays open until the request settles.
Reproduction: mock composioSession with a never-settling promise, enable global and per-Dot app permissions, subscribe to DotAgent.run, and advance fake timers by 91 seconds. The completion callback is not called. This regression probe failed against the reviewed head.
Acceptance criteria:
- Pass the turn abort signal through session creation/reuse and tool discovery using the SDK request options.
- Ensure a timeout emits the appropriate terminal error and completes the stream even while startup is pending.
- Ensure cancellation during startup prevents the inner agent from starting afterward.
- Add regression coverage for stalled startup, timeout, and cancellation.
2. Forward the Composio API key in Docker Compose
The server reads COMPOSIO_API_KEY, but compose.yml does not forward it to the app container. Setting it in .env as documented leaves connected apps disabled and the controls hidden for Compose deployments.
Acceptance criteria:
- Add optional
COMPOSIO_API_KEYforwarding to the app service environment. - Verify a configured key reaches the container and an omitted key keeps the integration disabled.
Validation context
Typecheck and all 307 existing tests passed at the reviewed head. The targeted startup-timeout probe failed. These are bounded follow-ups for the optional integration; they do not regress existing functionality when connected apps are disabled.
- Ngôn ngữ chính
- TypeScript
- Star
- 5.5k
- Fork
- 796
- 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 13 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 2 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ự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
Maintainer thường phản hồi trong vòng 1 ngày
-
auth: agent-bearer --manager-call is told to use the MCP instance argument on an ambiguous managerĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Lyric and ApostrophesĐang mởbug module: playback
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
chthollyphile/folia-major#739 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
agilepathway/label-checker#773 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày