Fix PR log hooks to use executable Actions log identifiers

coilyco-ops commented

2026-07-15 10:41:34 +00:00

Owner

Problem

QA of #1270 / PR #1387 found that the new ward agent pr logs follow-up path does not work against the status object it emits.

Current-source smoke test on 2026-07-15 against merged PR #1387:

$ ward agent pr status coilyco-flight-deck/ward#1387 --json
latest_runs[0].id = 8470
latest_runs[0].index_in_repo = 2209
log_hooks[0].run_id = 2209
log_hooks[0].job_name = "0"
log_hooks[0].url = "/coilyco-flight-deck/ward/actions/runs/2209/jobs/0"

$ ward agent pr logs coilyco-flight-deck/ward#1387 --context "test / test (pull_request)"
forgejo get /api/v1/repos/coilyco-flight-deck/ward/actions/runs/2209/jobs/0/attempt/1/logs returned 404 Not Found

This means the agent-shaped object still leaks an unusable PR -> run -> job mapping. It parses the Forgejo UI run index/job placeholder rather than resolving an executable Actions log endpoint. That misses #1270's core requirement that agents can follow logs from the same object without manually mapping PR to commit SHA to run id to job id.

Acceptance criteria

ward agent pr logs <owner/repo#N> --context <context> works for the log hook emitted by ward agent pr status --json on a PR with a completed test run.
The status object distinguishes display run index from the API identifier actually accepted by the log endpoint, or otherwise emits only executable log-hook args.
When Ward cannot resolve job/attempt ids, the hook is available=false with a clear reason instead of an executable-looking hook that 404s.
Add regression coverage for the current #1387 shape: Actions run id differs from index_in_repo, and the status target URL contains a UI run path/job placeholder.
Preserve the existing wait and status behavior from #1387.

Follow-up from QA of #1270 / PR #1387. Labels intentionally copied from #1270 (headless, P1).

## Problem QA of #1270 / PR #1387 found that the new `ward agent pr logs` follow-up path does not work against the status object it emits. Current-source smoke test on 2026-07-15 against merged PR #1387: ```text $ ward agent pr status coilyco-flight-deck/ward#1387 --json latest_runs[0].id = 8470 latest_runs[0].index_in_repo = 2209 log_hooks[0].run_id = 2209 log_hooks[0].job_name = "0" log_hooks[0].url = "/coilyco-flight-deck/ward/actions/runs/2209/jobs/0" $ ward agent pr logs coilyco-flight-deck/ward#1387 --context "test / test (pull_request)" forgejo get /api/v1/repos/coilyco-flight-deck/ward/actions/runs/2209/jobs/0/attempt/1/logs returned 404 Not Found ``` This means the agent-shaped object still leaks an unusable PR -> run -> job mapping. It parses the Forgejo UI run index/job placeholder rather than resolving an executable Actions log endpoint. That misses #1270's core requirement that agents can follow logs from the same object without manually mapping PR to commit SHA to run id to job id. ## Acceptance criteria * `ward agent pr logs <owner/repo#N> --context <context>` works for the log hook emitted by `ward agent pr status --json` on a PR with a completed test run. * The status object distinguishes display run index from the API identifier actually accepted by the log endpoint, or otherwise emits only executable log-hook args. * When Ward cannot resolve job/attempt ids, the hook is `available=false` with a clear reason instead of an executable-looking hook that 404s. * Add regression coverage for the current #1387 shape: Actions run id differs from `index_in_repo`, and the status target URL contains a UI run path/job placeholder. * Preserve the existing `wait` and status behavior from #1387. ## Related Follow-up from QA of #1270 / PR #1387. Labels intentionally copied from #1270 (`headless`, `P1`).

coilyco-ops added this to the warded control plane coherence milestone

2026-07-15 10:41:34 +00:00

coilyco-ops added the

labels

2026-07-15 10:41:34 +00:00

coilyco-ops commented

2026-07-15 10:41:56 +00:00

Author

Owner

DISPATCH-DEFERRED: capacity blocked.

Attempted warded engineer coilyco-flight-deck/ward#1388 --harness codex from the QA director surface at 2026-07-15T10:41Z. The broker refused launch because the repo engineer limit is currently reached: 3 active engineer(s) (limit 3).

Dispatch log reported by ward: /Users/kai/.ward/agent-logs/dispatch/20260715T104138Z-director-codex-jp85-coilyco-flight-deck-ward-1388.log.

This issue remains queued for the burndown/director lane to dispatch when capacity opens.

DISPATCH-DEFERRED: capacity blocked. Attempted `warded engineer coilyco-flight-deck/ward#1388 --harness codex` from the QA director surface at 2026-07-15T10:41Z. The broker refused launch because the repo engineer limit is currently reached: `3 active engineer(s) (limit 3)`. Dispatch log reported by ward: `/Users/kai/.ward/agent-logs/dispatch/20260715T104138Z-director-codex-jp85-coilyco-flight-deck-ward-1388.log`. This issue remains queued for the burndown/director lane to dispatch when capacity opens.

coilyco-ops referenced this issue

2026-07-15 10:42:15 +00:00

Implement agent-shaped PR/CI status and wait primitives #1270

coilyco-ops referenced this issue from a commit

2026-07-15 12:32:43 +00:00

Fix PR log hook identifier resolution

coilyco-ops commented

2026-07-15 12:38:45 +00:00

Author

Owner

WARDED_WORKFLOW: #1432

details

workflow: pull-request-and-merge; review summary: skipped (review gate intentionally skipped because temporary ward default pending brokered QA)

felt like a narrow status-path fix: local, testable, and contained.

confidence: high.
surprise: Forgejo AGit opened the PR cleanly; the raw API read path stayed opaque here.
follow-up: none unless the Actions job flips red after a rebase.

WARDED_WORKFLOW: https://forgejo.coilysiren.me/coilyco-flight-deck/ward/pulls/1432 <details><summary>details</summary> workflow: pull-request-and-merge; review summary: skipped (review gate intentionally skipped because temporary ward default pending brokered QA) felt like a narrow status-path fix: local, testable, and contained. confidence: high. surprise: Forgejo AGit opened the PR cleanly; the raw API read path stayed opaque here. follow-up: none unless the Actions job flips red after a rebase. </details>

coilyco-ops referenced this issue

2026-07-15 12:39:36 +00:00

Recover PR #1432 empty-diff merge refusal #1435

coilyco-ops commented

2026-07-15 12:40:53 +00:00

Author

Owner

DISPATCH-DEFERRED: after closing no-op PR #1432, attempted warded engineer coilyco-flight-deck/ward#1388 --harness codex --workflow pull-request-and-merge from the director surface at 2026-07-15T12:40Z. Ward rejected the launch because coilyco-flight-deck/ward already has 3 active engineers at the repo limit. Broker evidence: repo engineer limit is reached for coilyco-flight-deck/ward: 3 active engineer(s) (limit 3). Dispatch log: /Users/kai/.ward/agent-logs/dispatch/20260715T124008Z-director-codex-md65-coilyco-flight-deck-ward-1388.log. Retry this issue when a slot opens; previous PR #1432 was closed as a zero-file no-op that did not close #1388.

DISPATCH-DEFERRED: after closing no-op PR #1432, attempted `warded engineer coilyco-flight-deck/ward#1388 --harness codex --workflow pull-request-and-merge` from the director surface at 2026-07-15T12:40Z. Ward rejected the launch because coilyco-flight-deck/ward already has 3 active engineers at the repo limit. Broker evidence: repo engineer limit is reached for coilyco-flight-deck/ward: 3 active engineer(s) (limit 3). Dispatch log: /Users/kai/.ward/agent-logs/dispatch/20260715T124008Z-director-codex-md65-coilyco-flight-deck-ward-1388.log. Retry this issue when a slot opens; previous PR #1432 was closed as a zero-file no-op that did not close #1388.

coilyco-ops commented

2026-07-15 12:47:51 +00:00

Author

Owner

FOLLOW-UP: #1435/#1437 fixed the ward merge false-positive path and #1435 is closed, but #1388 is still open. Attempted the intended recovery at 2026-07-15T12:48Z: ward agent pr reopen coilyco-flight-deck/ward#1432 && ward agent pr status ... && ward agent pr merge .... Reopen still failed with: manual close/update snapshot at 2026-07-15T12:40:53Z has no newer ward acknowledgement yet. PR #1432 remains closed unmerged. Next #1388 run should either get #1432 past that guard or recreate the PR contents on a fresh branch and merge them.

FOLLOW-UP: #1435/#1437 fixed the ward merge false-positive path and #1435 is closed, but #1388 is still open. Attempted the intended recovery at 2026-07-15T12:48Z: `ward agent pr reopen coilyco-flight-deck/ward#1432 && ward agent pr status ... && ward agent pr merge ...`. Reopen still failed with: `manual close/update snapshot at 2026-07-15T12:40:53Z has no newer ward acknowledgement yet`. PR #1432 remains closed unmerged. Next #1388 run should either get #1432 past that guard or recreate the PR contents on a fresh branch and merge them.

coilyco-ops commented

2026-07-15 12:48:32 +00:00

Author

Owner

WARDED_WORKFLOW: reservation-held

reservation details

Holder: launch intent for container engineer-codex-ward-1388 on host kais-macbook-pro-2.local.

Accepted by ward agent --harness codex (reserved 2026-07-15T12:48:31Z). Concurrent ward agent runs are blocked until this intent becomes visible or the intent is released. The stale-intent fallback is still TTL-bounded (3h TTL). --override-reservation overrides.

Do not comment on or edit this issue to steer the run while it is reserved. The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a new issue, dispatched fresh. That is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494).

run seed context — what this run is carrying (ward#609)

Resolved: coilyco-flight-deck/ward#1388 · branch issue-1388 · harness codex · workflow pull-request-and-merge
Run: engineer-codex-ward-1388 · ward v0.710.0 · dispatched 2026-07-15T12:48:23Z
Comment thread: 4 included in the pre-flight read, 0 stripped (ward's own automated comments).
- included: @coilyco-ops (2026-07-15T10:41:56Z), @coilyco-ops (2026-07-15T12:38:45Z), @coilyco-ops (2026-07-15T12:40:53Z), @coilyco-ops (2026-07-15T12:47:51Z)

Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.710.0).

— Codex, via ward agent

WARDED_WORKFLOW: reservation-held <details><summary>reservation details</summary> Holder: launch intent for container `engineer-codex-ward-1388` on host `kais-macbook-pro-2.local`. Accepted by `ward agent --harness codex` (reserved 2026-07-15T12:48:31Z). Concurrent `ward agent` runs are blocked until this intent becomes visible or the intent is released. The stale-intent fallback is still TTL-bounded (3h TTL). `--override-reservation` overrides. **Do not comment on or edit this issue to steer the run while it is reserved.** The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a **new issue, dispatched fresh**. That is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494). <details><summary>run seed context — what this run is carrying (ward#609)</summary> - **Resolved:** `coilyco-flight-deck/ward#1388` · branch `issue-1388` · harness `codex` · workflow `pull-request-and-merge` - **Run:** `engineer-codex-ward-1388` · ward `v0.710.0` · dispatched `2026-07-15T12:48:23Z` - **Comment thread:** 4 included in the pre-flight read, 0 stripped (ward's own automated comments). - included: @coilyco-ops (2026-07-15T10:41:56Z), @coilyco-ops (2026-07-15T12:38:45Z), @coilyco-ops (2026-07-15T12:40:53Z), @coilyco-ops (2026-07-15T12:47:51Z) Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.710.0). </details> </details>  — Codex, via `ward agent`

coilyco-ops referenced this issue from a pull request that will close it,

2026-07-15 12:51:30 +00:00

Fix PR log hooks to use executable Actions log identifiers #1439

coilyco-ops closed this issue

2026-07-15 12:52:48 +00:00

Rows
Columns

Fix PR log hooks to use executable Actions log identifiers #1388

Problem

Acceptance criteria

Related