Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

dev.on_timeout salvage-if-done: a timed-out dev session with a finished, gate-green product is retried and then deferred like an empty one

Đang mở
  • Có gì không ổn?
#776 1 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ó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
55/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
tooling

Hướng nghiên cứu

Bắt đầu với decide_dev trong escalation.py và reconcile sau kill được mô tả trong policy.py, sau đó so sánh cách xử lý review.on_timeout hiện có được tham chiếu trong #271. Sử dụng các lệnh [verify] hiện có trong worktree để kiểm tra các trường hợp timeout, stalled và over-budget; done có nghĩa là salvage-if-done chỉ chuyển sang review đối với công việc đã finalized và gate-green, còn retry và defer vẫn giữ nguyên hành vi đã nêu.

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

Mô tả

area:engine enhancement needs-design P3

Summary

A dev session that hits session_timeout_min with a finished, gate-green product in the worktree is routed like one that produced nothing: decide_dev (escalation.py) returns RETRY while max_dev_attempts lasts, then defers. The review side distinguishes these cases through review.on_timeout = "salvage-if-done"; the dev side has no equivalent.

Observed (0.11.1)

Setup: an unattended nightly sweep run with [limits] session_timeout_min = 180, max_dev_attempts = 2 and [scm] isolation = "worktree"; the dev adapter is Claude Code.

One story, two dev attempts, both session-end status timeout after 10800 s.

  • Attempt 1 was killed re-running the deterministic gates after a successful review-fix round. Its worktree held a 486-line diff; the snapshot at refs/attempt-preserve-dirty/<run>-<baseline>-1 built and its changed test files passed.
  • Attempt 2 was killed in the same phase with the deterministic gates green.
  • Journal: dev-decision action: retry, reason: dev session timeout, then action: defer.
  • The kept unit branch merged to main by hand the next morning after every [verify] command passed in the worktree.

Proposal

A [dev] on_timeout policy mirroring the review-side modes:

  • retry (default, current behaviour).
  • salvage-if-done: on timeout / stalled / over_budget, if the spec frontmatter shows the product finalized (done, or in-review reset forward, as review.on_timeout already tests) and the [verify] commands pass against the worktree, proceed to review instead of rolling back; otherwise retry.
  • defer: give up on the first timeout-like verdict.

The post-kill reconcile already rescues a finished session whose artifact is on disk (policy.py comment on dev_stall_nudges); this extends it to a session killed during verification before the artifact was written. With salvage-if-done a timeout costs one verify pass instead of the attempt plus a from-scratch retry (#777). Related: #271 (the review-side salvage this mirrors), #764.

Ngôn ngữ chính
Python
Star
165
Fork
69
Merge trung bình
1 ngày 2 giờ
Pull request đã merge (30 ngày)
78

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 bmad-code-org/bmad-loop

Tất cả issue của bmad-code-org/bmad-loop

Issue tương tự

Thêm issue về Python

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.