Skip to content

fix: link default footers to workflow run URL instead of agentic_workflow subpage#31611

Merged
pelikhan merged 1 commit into
mainfrom
copilot/update-footer-generation-url
May 12, 2026
Merged

fix: link default footers to workflow run URL instead of agentic_workflow subpage#31611
pelikhan merged 1 commit into
mainfrom
copilot/update-footer-generation-url

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 12, 2026

Bug Fix

What was the bug?

Default footer links pointed to {runUrl}/agentic_workflow — the agentic workflow tab within a run. This subpage doesn't surface the run summary and isn't detected by the bile app, which keys on the standard actions/runs/{id} URL shape.

How did you fix it?

Replaced {agentic_workflow_url} with {run_url} in all five default footer template strings in messages_footer.cjs:

  • getFooterMessage
  • getFooterWorkflowRecompileMessage
  • getFootowrkflowRecompileCommentMessage
  • getFooterAgentFailureIssueMessage
  • getFooterAgentFailureCommentMessage

Before:

> Generated by [My Workflow](https://github.com/org/repo/actions/runs/123/agentic_workflow)

After:

> Generated by [My Workflow](https://github.com/org/repo/actions/runs/123)

The {agentic_workflow_url} placeholder is still computed and injected into the template context, so custom footers configured via GH_AW_SAFE_OUTPUT_MESSAGES can still opt into it.

…ult footers

Change the default footer link target from the `/agentic_workflow` subpage
(runUrl + "/agentic_workflow") to the plain workflow run URL, so that the
run summary is visible and the app can detect the standard actions/runs URL
format. The {agentic_workflow_url} placeholder is still available for custom
footer templates.

Co-authored-by: pelikhan <[email protected]>
Copilot AI changed the title fix: use workflow run URL instead of agentic workflow subpage in default footers fix: link default footers to workflow run URL instead of agentic_workflow subpage May 12, 2026
Copilot AI requested a review from pelikhan May 12, 2026 01:37
@pelikhan pelikhan marked this pull request as ready for review May 12, 2026 01:40
Copilot AI review requested due to automatic review settings May 12, 2026 01:40
@pelikhan pelikhan merged commit d130cf2 into main May 12, 2026
@pelikhan pelikhan deleted the copilot/update-footer-generation-url branch May 12, 2026 01:40
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes default footer links to point to the standard GitHub Actions run URL (.../actions/runs/{id}) instead of the .../agentic_workflow subpage, while keeping {agentic_workflow_url} available for custom templates.

Changes:

  • Updated all default footer templates in messages_footer.cjs to use {run_url} rather than {agentic_workflow_url}.
  • Updated unit tests to assert the new default footer link shape and to confirm {agentic_workflow_url} still works for custom footers.
Show a summary per file
File Description
actions/setup/js/messages_footer.cjs Switches default footer templates to link to {run_url} while preserving computed {agentic_workflow_url} for custom templates.
actions/setup/js/messages.test.cjs Updates expectations for default footers and adds/adjusts coverage for custom templates using {agentic_workflow_url}.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants