When adding two steering message the first gets queued which messes up order.
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Shell
- Star
- 11.2k
- Fork
- 1.9k
- Merge trung bình
- 14 giờ 16 phút
- Pull request đã merge (30 ngày)
- 6
Mô tả
Describe the bug
When steering, and then you have another idea and steer again, the order of the steering messages is flipped and the first message is queued. Leading to the wrong order of execution.
This happens a lot when you think of relevant context after firing off the steering message.
Affected version
GitHub Copilot CLI 1.0.78. Run 'copilot update' to check for updates.
Steps to reproduce the behavior
- Start a task in copilot CLI
- Give it a steering message
- Before it's finished processing steer again, e.g. adding more context
This leads to the first steering message being queued, and the second steering message being used now, which would even be confusion for a human.
Expected behavior
Steering happens in the order we steer, and is not queued without asking to queue.
Additional context
- OS: MacOS
- Shell: Nushell
- Terminal: Ghostty (seems to have with normal Mac terminal as well though)
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.
Hướng nghiên cứu
Bắt đầu bằng cách tái hiện hành vi trong Copilot CLI 1.0.78: gửi một steering message, sau đó gửi một steering message khác trước khi quá trình xử lý hoàn tất, theo các bước trong issue. Theo dõi việc xử lý steering message và điểm vào của hàng đợi, sau đó xác minh rằng các steering message được thực thi theo thứ tự đã gửi và không được xếp hàng nếu không có yêu cầu rõ ràng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- shell
- Lĩnh vực
- cli
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 55/100