F2d: already-fixed check and opening the task's PR - #84

Merged

mchwang merged 66 commits into

mainfrom

feat/f2d-already-fixed-pr

Oct 1, 2026

Commits on Sep 29, 2026

F2d: check whether the issue is already fixed, then open the task's PR

mchwang
and
claude
committed
cba0a9b
Address review round 1 on F2d: ownership, lost openings, stale responses

mchwang
and
claude
committed
7e3830b
Keep a needs-human task in needs human when it reuses a ready PR; avoid spreading backtick runs

mchwang
and
claude
committed
1dcd29e
Pass the D-Bus session bus to gh, so Linux keyring sign-in works

mchwang
and
claude
committed
7f3e5fe
AGENTS.md: rules from the F2d review rounds

mchwang
and
claude
committed
ea96836
Address Copilot round 2 on F2d: pre-push re-read, disconnects, deadline, refresh record

mchwang
and
claude
committed
0f750f1
Address Copilot round 3 on F2d: review version, one publish per task, abort, deadline

mchwang
and
claude
committed
89621e9
Share the one-publish-per-task guard across publishers over the same Store

mchwang
and
claude
committed
6b26e65
Close handed-off review findings on F2d: references, adoption, ProjectV2, order

mchwang
and
claude
committed
9980445
Address Copilot round 4 on F2d: stdin transport, exact problem cut, settle both reads

mchwang
and
claude
committed
fc5887b
Escalate to SIGKILL when gh ignores SIGTERM, so a timeout or abort always settles

mchwang
and
claude
committed
b32a330
Close high-effort review findings on F2d: settle, head read-back, drafts

mchwang
and
claude
committed
07c6630
Draft the earlier PR before recording a matching check, so a failure stays retryable

mchwang
and
claude
committed
dcc2b0c

Commits on Sep 30, 2026

Address Copilot round 5 on F2d: verified draft state, re-read before drafting, title mentions, deadline budget

mchwang
and
claude
committed
0be7147
Close the xhigh review findings on F2d: own merged PR, drafts unsupported, refresh ordering

mchwang
and
claude
committed
f5859f4
Address Copilot round 6 on F2d: branch lookup always, draft first, docs, v6→v7 test

mchwang
and
claude
committed
a0b5206
Merge main into F2d: keep F1c/F1d beside the v7 PR records

mchwang
and
claude
committed
fa037c9
Address Copilot round 7 on F2d: marker line only, draft flag repair, recovery draft reconcile

mchwang
and
claude
committed
46bbeba
Address Copilot round 8 on F2d: duplicate commits, stale recovery, verified draft opening

mchwang
and
claude
committed
f094853
Address Copilot round 9 on F2d: one deadline per PR operation, adopt on sight

mchwang
and
claude
committed
8a4e5ea
Refresh only an opened PR: drop beginRefresh's unused adopt path

mchwang
and
claude
committed
af56100
Address Copilot round 10 on F2d: own the review version too

mchwang
and
claude
committed
6c15c41
Close the independent review's findings on F2d: guard after push, head mismatch, no-changes, settle

mchwang
and
claude
committed
94fa810
Close independent review round 2 on F2d: no-changes guards, leftReady, mismatch order

mchwang
and
claude
committed
0022b18
Close independent review round 3 on F2d: recovery drafts, no-changes re-read, settle errors

mchwang
and
claude
committed
3c8d31b
Close independent review round 4 on F2d: busy tasks, early guard, wrong-repo updates

mchwang
and
claude
committed
3c20a0c
Close independent review round 5 on F2d: re-check before ready, adopt in recovery

mchwang
and
claude
committed
e36bcbb
Close independent review round 6 on F2d: shutdown guard, owed drafts

mchwang
and
claude
committed
3399abf
Close independent review round 7 on F2d: adopt late PRs after a stop

mchwang
and
claude
committed
63aaadc
Test the draft step's late-adoption record and abort after a lookup

mchwang
and
claude
committed
153050b
Close independent review round 8 on F2d: tests for untested guards

mchwang
and
claude
committed
e0f7976
Close independent review round 9 on F2d: both sides of a manual link

mchwang
and
claude
committed
f6a90ec
Close independent review round 10 on F2d: no read:project field

mchwang
and
claude
committed
4f964fd
Close independent review round 11 on F2d: definite 422, retargeted PRs

mchwang
and
claude
committed
eed8477
Count only closing cross-references in the already-fixed check

mchwang
and
claude
committed
9ad2d16
Close independent review round 12 on F2d: branch lookup across bases

mchwang
and
claude
committed
8ef6bed
Count same-repo PRs into a non-default base in the already-fixed check

mchwang
and
claude
committed
dc2af66
Close independent review round 13 on F2d: configured base everywhere

mchwang
and
claude
committed
ba6f8f6
Close independent review round 14 on F2d: draft a stopped task's PR in any base

mchwang
and
claude
committed
c8119c6
Close independent review round 15 on F2d: observe in any base, write in one

mchwang
and
claude
committed
e2f7bad
Close independent review round 16 on F2d: in review only from the configured base

mchwang
and
claude
committed
b286a29
Close independent review round 17 on F2d: draft misplaced PRs before refusing

mchwang
and
claude
committed
eb46d94
Close independent review round 18 on F2d: look up every misplaced PR

mchwang
and
claude
committed
8089152
Close independent review round 19 on F2d: one path for misplaced PRs

mchwang
and
claude
committed
9dbccdb
Close independent review round 20 on F2d: no early ends in recovery

mchwang
and
claude
committed
09ee8b2
Close independent review round 21 on F2d: doc follows round 20

mchwang
and
claude
committed
1465606
Close independent review round 22 on F2d: tests for untested guards

mchwang
and
claude
committed
188b022
Close independent review round 23 on F2d: tests for the remaining guards

mchwang
and
claude
committed
d82cc23

Commits on Oct 1, 2026

Close independent review round 24 on F2d: control characters, blocking test

mchwang
and
claude
committed
553135c
Close independent review round 25 on F2d: non-JSON bodies are not read

mchwang
and
claude
committed
b84afe7
Close independent review round 26 on F2d: list lag, CRLF markers, surrogates

mchwang
and
claude
committed
f83c260
Close independent review round 27 on F2d: tell list lag from a moved PR

mchwang
and
claude
committed
13bdfad
Close independent review round 28 on F2d: draft a moved PR before refusing

mchwang
and
claude
committed
61c0a9f
Close independent review round 29 on F2d: moved-PR draft record

mchwang
and
claude
committed
6a56491
Close independent review round 30 on F2d: own PR without its marker

mchwang
and
claude
committed
719224f
Close independent review round 31 on F2d: unmarked PRs, faithfully

mchwang
and
claude
committed
e22ec2a
Fix the unmarked-PR publisher test pushed failing in e22ec2a

mchwang
and
claude
committed
08a8402
Close independent review round 32 on F2d: a more faithful fake GitHub

mchwang
and
claude
committed
d49d713
Close independent review round 33 on F2d: abort tests for two lookups

mchwang
and
claude
committed
2ab52a0
Close independent review round 34 on F2d: three tests, one scope note

mchwang
and
claude
committed
d97da73
Address Copilot round 11 on F2d: full pages, malformed entries, approvals

mchwang
and
claude
committed
7c5ca43
Follow the code review of 7c5ca43 on F2d

mchwang
and
claude
committed
3c91d8b
Address Copilot round 12 on F2d: fail closed on ambiguous markers

mchwang
and
claude
committed
683f538
Follow the code review of 683f538 on F2d: settle updates by number

mchwang
and
claude
committed
4cc74ca
Address Copilot round 13 on F2d: no issue tokens inside URL paths

mchwang
and
claude
committed
2aaa4fd
Follow the code review of 2aaa4fd on F2d

mchwang
and
claude
committed
bf8d283