Add generated asset links to conclusion job comment updates - #5940
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- Modified notify_comment.go to pass safe output job information - Added buildSafeOutputJobsEnvVars helper to generate env vars for URLs - Updated notify_comment_error.cjs to collect and display asset URLs - Added collectGeneratedAssets function to process job outputs - Added comprehensive tests for both Go and JavaScript changes - All tests passing Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
| // Collect generated asset URLs from safe output jobs | ||
| const generatedAssets = collectGeneratedAssets(); | ||
| if (generatedAssets.length > 0) { | ||
| message += "\n\n### Generated Assets\n\n"; |
There was a problem hiding this comment.
Done in 29f37d2. Removed the "### Generated Assets" heading - URLs are now displayed directly.
| if (generatedAssets.length > 0) { | ||
| message += "\n\n### Generated Assets\n\n"; | ||
| generatedAssets.forEach(asset => { | ||
| message += `- ${asset.label}: ${asset.url}\n`; |
There was a problem hiding this comment.
@copilot just put the link and github will pretty render it. Remove custom label code
There was a problem hiding this comment.
Done in 29f37d2. Removed all custom labels - now displaying just the raw URLs so GitHub can render them with rich previews.
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Remove "Generated Assets" heading and custom labels per PR feedback. Now displays just the URLs so GitHub can render them with rich previews. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
📰 BREAKING: Smoke Copilot Playwright is now investigating this pull request. Sources say the story is developing... |
|
💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...] |
|
🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request... |
Copilot Smoke Test ResultsPR Titles: ❌ (GitHub MCP connection error) Overall Status: ❌ FAIL Issues: GitHub API connection failures preventing MCP and gh tool testing. 📰🔥📋 https://github.com/githubnext/gh-aw/actions/runs/20064257991 fer issue #5940 🗺️ |
|
Smoke Test Results for Copilot Engine
Status: PASS
|
Smoke Test Results - Run 20064257950✅ Playwright MCP: Successfully navigated to https://github.com - page title verified as "GitHub · Change is constant. GitHub keeps you ahead. · GitHub" Overall Status: FAIL (1/3 tests passed)
|
Smoke Test Results - Claude EngineLast 2 Merged PRs:
Test Results:
Status: PASS
|
|
@copilot add support for commit for push to pull request branch |
|
Recent merged PRs:
|
Add push_to_pull_request_branch to safe output jobs that display URLs. The job outputs push_url which links to the branch tree view. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request... |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...] |
|
📰 BREAKING: Smoke Copilot Playwright is now investigating this pull request. Sources say the story is developing... |
Smoke Test Results (Claude - Run 20064608018)Recent PRs:
Test Results: Status: PASS (5/6 tests passed - gh CLI not available in this context)
|
Smoke Test Results (Run 20064608017)✅ Playwright: GitHub page loaded, title verified Overall: FAIL (1/3 tests failed)
|
Smoke Test ResultsPR #5940 Test Results:
Overall Status: FAIL Note: safeinputs-gh tool experiencing connectivity issues with GitHub API. 📰🔥📋 https://github.com/githubnext/gh-aw/actions/runs/20064608016 fer issue #5940 🗺️ |
Smoke Test Results - Run 20064608010✅ GitHub MCP - PR #5943: [WIP] Format message footer to use emoji as link text Tools Available: bash, view, edit, create, glob, grep, github-* (repos/issues/PRs/search), safeoutputs-* (create_issue/add_comment/add_labels/noop/missing_tool) Overall Status: PASS
|
|
Recent merged PRs: [WIP] Format message footer to use emoji as link text; [docs] Update documentation for features from December 8-9, 2025
|
The conclusion job now includes direct links to created assets (issues, PRs, comments, branches, etc.) in the workflow completion comment, eliminating the need to search through logs.
Changes
Backend (
pkg/workflow/notify_comment.go)buildSafeOutputJobsEnvVars()to map safe output jobs to their URL outputs and generate corresponding environment variablesbuildConclusionJob()to pass job metadata and URL expressions as env vars to the conclusion scriptcreate_issue,add_comment,create_pull_request,create_discussion,create_pr_review_comment,close_issue,close_pull_request,close_discussion,create_agent_task,push_to_pull_request_branchFrontend (
pkg/workflow/js/notify_comment_error.cjs)collectGeneratedAssets()to extract URLs from environment variables using the job metadata mappingExample
Given a workflow that creates an issue, a PR, and pushes to a branch, the conclusion comment now displays:
GitHub automatically renders these URLs with rich preview cards showing the issue/PR title, status, labels, branch details, and other metadata.
The implementation passes URL outputs through GitHub Actions expressions (
${{ needs.job.outputs.url }}), maintaining the existing job dependency model without additional API calls.Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.
Changeset
Changeset