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

Add a director queue view for stale reservations and redispatch candidates #984

Closed
coilyco-ops wants to merge 0 commits from issue-977 into main
pull from: issue-977
merge into: coilyco-flight-deck:main
Conversation 2 Commits - Files changed - +593 -2
coilyco-ops commented 2026-07-10 09:16:37 +00:00
Owner
Copy link

closes #977
ward.workflow: pull-requests-and-merge

Add a director queue/status view for stale reservations, redispatch candidates, submitted PRs, and stale-open done issues.

closes #977 ward.workflow: pull-requests-and-merge Add a director queue/status view for stale reservations, redispatch candidates, submitted PRs, and stale-open done issues.
feat: add director queue status view
All checks were successful
test / test (pull_request) Successful in 28s
Details
50a13c9e3d
coilyco-ops commented 2026-07-10 09:24:10 +00:00
Author
Owner
Copy link

verdict: block
confidence: high
summary: The queue command wires the requested states, but one PR classification branch compares a display prefix to the internal pull-request kind. That makes done PR carries look like closeable issues.
findings:

  • severity: high
    file: cmd/ward/agent_director_queue.go
    line: 231
    problem: classifyDirectorQueueOutcome checks item.Kind == backlogKindPullRequest, but classifyDirectorQueuePR stores Kind: backlogEntryKindPrefix(backlogKindPullRequest), which is the display string PR. The comparison can never be true for PR rows.
    impact: A pull request with WARD-OUTCOME: done is classified with action close stale-open done issue instead of wait, so the queue view can instruct the operator to close a PR carry as if it were a stale-open issue.
    conclusion: Review blocks PR #984. The queue surface is useful, but PR rows need to keep an internal kind or compare through backlogKindOf before the done-state action can be trusted.
verdict: block confidence: high summary: The queue command wires the requested states, but one PR classification branch compares a display prefix to the internal pull-request kind. That makes done PR carries look like closeable issues. findings: - severity: high file: cmd/ward/agent_director_queue.go line: 231 problem: `classifyDirectorQueueOutcome` checks `item.Kind == backlogKindPullRequest`, but `classifyDirectorQueuePR` stores `Kind: backlogEntryKindPrefix(backlogKindPullRequest)`, which is the display string `PR`. The comparison can never be true for PR rows. impact: A pull request with `WARD-OUTCOME: done` is classified with action `close stale-open done issue` instead of `wait`, so the queue view can instruct the operator to close a PR carry as if it were a stale-open issue. conclusion: Review blocks PR #984. The queue surface is useful, but PR rows need to keep an internal kind or compare through `backlogKindOf` before the done-state action can be trusted.
coilyco-ops commented 2026-07-10 17:15:28 +00:00
Author
Owner
Copy link

Superseded by PR #1043 (issue #995): it merges this branch and repairs the review-blocking classification bug - classifyDirectorQueueOutcome compared item.Kind against backlogKindPullRequest while the queue items stored the display prefix from backlogEntryKindPrefix, so done PR carries were mis-actioned as closeable stale-open done issues. Closing this PR in favor of #1043.

Superseded by PR #1043 (issue #995): it merges this branch and repairs the review-blocking classification bug - `classifyDirectorQueueOutcome` compared `item.Kind` against `backlogKindPullRequest` while the queue items stored the display prefix from `backlogEntryKindPrefix`, so done PR carries were mis-actioned as closeable stale-open done issues. Closing this PR in favor of #1043.
coilyco-ops closed this pull request 2026-07-10 17:15:28 +00:00
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!984
No description provided.