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

Merged
coilyco-ops merged 1 commit from issue-1388 into main 2026-07-15 12:52:48 +00:00
Owner

Fix PR log hook identifier resolution.

This keeps the PR status object from advertising Forgejo UI placeholders as executable log endpoints. The status snapshot now distinguishes the display run index from the API run id and marks placeholder hooks unavailable instead of letting ward agent pr logs chase a 404.

  • Preserve the existing PR wait/status behavior.
  • Add regression coverage for the #1387/#1388 shape with mismatched run ids and placeholder job ids.
  • Update the PR status/log docs to describe the executable hook contract.

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

Fix PR log hook identifier resolution. This keeps the PR status object from advertising Forgejo UI placeholders as executable log endpoints. The status snapshot now distinguishes the display run index from the API run id and marks placeholder hooks unavailable instead of letting `ward agent pr logs` chase a 404. - Preserve the existing PR wait/status behavior. - Add regression coverage for the #1387/#1388 shape with mismatched run ids and placeholder job ids. - Update the PR status/log docs to describe the executable hook contract. closes #1388 ward.workflow: pull-request-and-merge
Fix PR log hook identifier resolution
All checks were successful
test / test (pull_request) Successful in 1m0s
4f682a6f1d
closes #1388

ward.workflow: pull-request-and-merge
Commenting is not possible because the repository is archived.
No reviewers
No milestone
No project
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!1439
No description provided.