Repository navigation

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

Merged

F2d: already-fixed check and opening the task's PR#84
mchwang merged 66 commits into
mainfrom
feat/f2d-already-fixed-pr

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

Commits

Commits on Sep 29, 2026

Commits on Sep 30, 2026

Commits on Oct 1, 2026