coilyco-flight-deck/ward
Archived
Watch
2
0
Code Issues Pull requests Projects Releases 820 Packages 1 Wiki Activity Actions

Fix closed-unmerged launched PR reaping #1292

Merged
coilyco-ops merged 2 commits from issue-1007 into main 2026-07-14 19:22:22 +00:00
Conversation 1 Commits 2 Files changed 2 +107
coilyco-ops commented 2026-07-14 19:06:43 +00:00
Owner
Copy link

This patch teaches the container reaper to notice a launched run whose submitted PR later closes without merging, then comment failure and release the reservation instead of leaving the issue silently stuck.

Why:

  • It closes the gap where a closed-unmerged PR path left the carried issue reserved.
  • It keeps the fix on consumer code only.

Checks:

  • Targeted go test on the reaper paths passed before the local scratch exhaustion hit broader linting.
  • ward exec test could not complete in this container because the overlay ran out of temp space.

closes #1007
ward.workflow: pull-request-and-merge

This patch teaches the container reaper to notice a launched run whose submitted PR later closes without merging, then comment failure and release the reservation instead of leaving the issue silently stuck. Why: - It closes the gap where a closed-unmerged PR path left the carried issue reserved. - It keeps the fix on consumer code only. Checks: - Targeted `go test` on the reaper paths passed before the local scratch exhaustion hit broader linting. - `ward exec test` could not complete in this container because the overlay ran out of temp space. closes #1007 ward.workflow: pull-request-and-merge
Fix closed-unmerged launched PR reaping
Some checks failed
test / test (pull_request) Failing after 1m2s
Details
c7bd5c7d73
coilyco-ops commented 2026-07-14 19:13:17 +00:00
Author
Owner
Copy link

PR #1292 failed in Forgejo Actions at golangci-lint after go vet and go test passed. The same scratch-space limit also showed up locally while reproducing the lint stage. Please rerun after the runner scratch cache has room again.

PR #1292 failed in Forgejo Actions at `golangci-lint` after `go vet` and `go test` passed. The same scratch-space limit also showed up locally while reproducing the lint stage. Please rerun after the runner scratch cache has room again.
Refactor closed-unmerged PR reap check
All checks were successful
test / test (pull_request) Successful in 37s
Details
f9508a36e7
Commenting is not possible because the repository is archived.
Reviewers
No reviewers
Labels
Clear labels   
burndown-2026-06
Backlog burndown June 2026

  
pressure-test
Cold-read release pressure-test findings and coordination

  
sunday-sprint
Burn-down by Sunday 2026-06-07

  
autonomy
async-consult
 A human needs to consult on the issue to upgrade it to headless

  
autonomy
epic
 This issue has many units of sub work - its size makes it meaningfully exclusive with other autonomy types

  
autonomy
headless
 The agent can perform the work on its own

  
autonomy
live-collab
 The agent and the human need to work together in realtime

  
coherence-core
Core review set for the warded control plane coherence milestone. These issues form the release spine; adjacent milestone issues are stretch or supporting work.

  
priority
P0
 priority tier

  
priority
P1
 priority tier

  
priority
P2
 priority tier

  
priority
P3
 priority tier

  
priority
P4
 priority tier

  
qa-fixture
Disposable issue admitted to the bounded Ward QA verification lane.

  
role/advocate
requires work from the Developer Advocate seat

  
role/director
requires work from the Portfolio Director seat

  
role/exec
requires work from the exec role

  
role/frontend
requires work from the Frontend Engineer seat

  
role/gamedev
requires work from the Game Developer seat

  
role/human
requires a person, and specifically not an agent seat

  
role/platform
requires work from the Platform Engineer seat

  
role/qa
requires work from the QA role

  
role/science
requires work from the Applied Scientist seat

  
role/sysadmin
requires work from the Systems Administrator seat

  
state
ambient
 ambient and ephemeral work, held as a maintained document rather than a queue

No labels
burndown-2026-06
pressure-test
sunday-sprint
autonomy
async-consult
autonomy
epic
autonomy
headless
autonomy
live-collab
coherence-core
priority
P0
priority
P1
priority
P2
priority
P3
priority
P4
qa-fixture
role/advocate
role/director
role/exec
role/frontend
role/gamedev
role/human
role/platform
role/qa
role/science
role/sysadmin
state
ambient
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
coilyco-flight-deck/ward!1292
No description provided.