Skip to content

Goal: Close the fallback-route landing-evidence gap in `internal/loopdriver/run. - #345

Merged
linhdmn merged 1 commit into
mainfrom
devagent/TASK-mtxu01sd-1ted
Sep 12, 2026
Merged

linhdmn merged 1 commit into
mainfrom
devagent/TASK-mtxu01sd-1ted

Conversation

@linhdmn

@linhdmn linhdmn commented Sep 12, 2026

Copy link
Copy Markdown
Member

Automated task via devagent task.

Prompt

Goal: Close the fallback-route landing-evidence gap in internal/loopdriver/run.go. Off the tracker path pick.mergePR is 0, so PO/queued goals like Goal: Land PR #344 whose worker merge lands mid-run record no-pr (loops 270/271/274/280). Before runTaskPhase, when pick.mergePR == 0, derive the merge target from the goal text via the existing mergePickRe, gated by the pick-time prState == OPEN guard (run.go:249-253) so stale mentions self-disqualify. A derived PR merged by record time flows through the existing evidencePR/landedArtifactsVerified wiring and records ok, never no-pr. Retire the queue claim on the no-pr early return (goalRejectedDetail precedent). Regression tests on the loop-280 shape: OPEN-at-dispatch PR merged by the worker → ok; already-merged mention → no-pr, claim retired.

Repo policy: every PR keeps docs/PRD.md current — update the sections this change affects (status claims, architecture notes, roadmap/completion notes) and bump the Last updated footer in the same PR. A PR that changes repo state without reflecting it in docs/PRD.md is incomplete.

PRD state update (repo policy)

  • docs/PRD.md sections touched by this change are updated to the post-PR state
  • the Last updated footer reflects this change

@linhdmn

linhdmn commented Sep 12, 2026

Copy link
Copy Markdown
Member Author

DevAgent zombie-PR sweep: auto-closing this PR.
Base branch main was merged or deleted, so this branch can no longer integrate.
Reopen with a re-targeted base if the work is still needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant