Development record
When the auto-plan's start digest cannot copy the checkout's index (a symlinked index, or one past the copy bound), the lane still runs both plan reviews before the end digest differs and hands it back (internal/core/implement/loop/autoplan.go, the start digest at about line 166), spending two reviews on a result already decided.
issueresolvedminoriss-2610312044551391
When the auto-plan's start digest cannot copy the checkout's index (a symlinked index, or one past the copy bound), the lane still runs both plan reviews before the end digest differs and hands it back (internal/core/implement/loop/autoplan.go, the start digest at about line 166), spending two reviews on a result already decided.
Grounds
- pursued: TestAStartThatCannotCopyTheIndexHandsBackBeforeAnyReview fails before the change (the first step hands the design review out) and passes after; shown wrong if an uncopied start index still reaches a plan review, or the hand-back reason still says the checkout changed.
Frontmatter
| id | iss-2610312044551391 |
|---|---|
| type | issue |
| lifecycle | resolved |
| severity | minor |
.abcd/work/issues/resolved/iss-2610312044551391-when-the-auto-plan-s-start-digest-cannot.md
open on GitHub ↗ · commit history ↗
mentions1
- dcn-26110120490564912026-11-01 — The drain's user-visible hand-back of iss-2610312044551391 is accepted as written: when the auto-plan's start digest cannot copy the index, the lane is handed back before the first plan review, with a reason naming the index it cannot copy. The issue is built as an ordinary issue, its new reason being the wording the person accepted.