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

Skip the second `[verify]` pass when the review is skipped and the tree is unchanged

Đang mở
  • Có gì không ổn?
#1,002 0 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
49/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
git, python
Lĩnh vực
devtools

Hướng nghiên cứu

Start by tracing _skip_review_and_commit through _review_verify_gate, _verify_review, verify_review_stories, and _verify_review_commands, then locate the post-dev verify-command-result journal record. Check how verification settings and Git tree state are represented. Done means unchanged trees can reuse a prior result only when enabled, while changes still run verification and the reuse is recorded in the journal.

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

Mô tả

Describe your idea

When the review session is skipped (review-not-recommended → review-skipped), _skip_review_and_commit runs the review verify gate, which executes [verify] commands again on a working tree that has not changed since the post-dev verify that just passed. Allow that second pass to reuse the first result when nothing changed.

Why is this needed?

The two passes exist for a good reason: on the review path, a review session can change code between them, and the gate proves what goes into the commit. On the skip-review path no session runs in between, so the second pass re-proves the same tree with the same commands.

With a slow suite this is the dominant cost of a story. In our project [verify] commands = ["npm test"] takes 11–14 minutes on Windows, so every story whose review is skipped spends that time twice. In one run (0.12.0), the post-dev verify-command-result passed at 15:15:32 and the review gate's npm test started in the same second on the identical tree. Checked against main (v0.13.2): _skip_review_and_commit → _review_verify_gate → _verify_review → verify_review_stories → _verify_review_commands still runs the commands unconditionally.

How should it work?

A deterministic check, no LLM involved:

  1. After the post-dev verify passes, record a fingerprint of what it verified: HEAD plus a hash of the working tree, tracked and untracked but not ignored (for example git add -A into a temporary index, GIT_INDEX_FILE, followed by git write-tree), together with the resolved [verify] commands.
  2. At the review gate on the skip-review path, compute the same fingerprint. If it matches, journal a verify-command-result that points to the earlier pass (for example reused_from_sequence: N) instead of running the commands again. The spec-done gates still run as today.
  3. Any difference (a fix session, a review session, an operator edit, a policy change) runs the commands as now.

Because suites can depend on state outside git (services, clocks, caches), it could be opt-in, for example [verify] reuse_unchanged = true, defaulting to today's behaviour.

PR

Not working on it. Happy to test a build on Windows.

Additional context

  • Platform: Windows 11, psmux 3.3.8, Claude Code adapter, stories mode (bmad-loop run --spec), [gates] mode = "none".
  • The journal already has what step 2 needs (verify-command-result with verification_stage and verification_sequence), so reuse stays auditable.
Ngôn ngữ chính
Python
Star
165
Fork
69
Merge trung bình
1 ngày 1 giờ
Pull request đã merge (30 ngày)
79

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.