Skip to content

Fix push-to-pull-request-branch summary to link commit URL instead of branch URL#6031

Merged
pelikhan merged 6 commits into
mainfrom
copilot/add-context-to-comments
Dec 10, 2025
Merged

Fix push-to-pull-request-branch summary to link commit URL instead of branch URL#6031
pelikhan merged 6 commits into
mainfrom
copilot/add-context-to-comments

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 10, 2025

The workflow run summary for push-to-pull-request-branch was linking the "Commit" label to the branch tree view (/tree/{branchName}) instead of the commit view (/commit/{commitSha}).

Changes

Changed the summary template in pkg/workflow/js/push_to_pull_request_branch.cjs line 424:

-- **Commit**: [${commitSha.substring(0, 7)}](${pushUrl})
+- **Commit**: [${commitSha.substring(0, 7)}](${commitUrl})

The commitUrl variable was already correctly defined and used for activation comment updates. This aligns the summary output with that behavior.

Original prompt

The conclusion JavaScript that adds context to the comment should add the commit url instead of the branch name. For push-to-pull-request-brancb agentic workflow

See this output: #5999 (comment)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.


Changeset

  • Type: patch
  • Description: Fix push-to-pull-request-branch summary to link commit URL instead of branch URL

Ahoy! This treasure was crafted by 🏴‍☠️ Changeset Generator

Copilot AI and others added 3 commits December 10, 2025 14:16
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add commit URL to comment context Fix push-to-pull-request-branch summary to link commit URL instead of branch URL Dec 10, 2025
Copilot AI requested a review from pelikhan December 10, 2025 14:29
@pelikhan pelikhan marked this pull request as ready for review December 10, 2025 14:36
@pelikhan pelikhan added the smoke label Dec 10, 2025
@github-actions
Copy link
Copy Markdown
Contributor

🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request...

@github-actions
Copy link
Copy Markdown
Contributor

📰 BREAKING: Smoke Copilot Playwright is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Copy Markdown
Contributor

💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...]

@github-actions
Copy link
Copy Markdown
Contributor

📰🚀🔍👀📡🕵️ https://github.com/githubnext/gh-aw/actions/runs/20102297015

@github-actions
Copy link
Copy Markdown
Contributor

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 10, 2025

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

https://github.com/githubnext/gh-aw/tree/copilot/add-context-to-comments

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP Testing
  • ✅ File Writing Testing
  • ✅ Bash Tool Testing
  • ✅ GitHub MCP Default Toolset Testing (get_me correctly unavailable)
  • ✅ Cache Memory Testing

Overall Status: PASS

📰 BREAKING: Report filed by Smoke Copilot fer issue #6031 🗺️

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

Tests:

  • ❌ GitHub MCP (API connection error)
  • ✅ File Writing
  • ✅ Bash Tool
  • ⏭️ Serena MCP (skipped - GitHub MCP prerequisite failed)
  • ⏭️ Safe Input gh (skipped - GitHub MCP prerequisite failed)

Status: FAIL (GitHub API connectivity issue)

📰🔥📋 https://github.com/githubnext/gh-aw/actions/runs/20102297015 fer issue #6031 🗺️

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test: Claude Engine - Run 20102297001

Last 2 Merged PRs:

Test Results:
✅ GitHub MCP - Retrieved 2 recent merged PRs
✅ File Writing - Created /tmp/gh-aw/agent/smoke-test-claude-20102297001.txt
✅ Bash Tool - Verified file creation successful
✅ Playwright MCP - Navigated to GitHub, page title confirmed
✅ Cache Memory - Created /tmp/gh-aw/cache-memory/smoke-test-20102297001.txt
❌ Safe Input gh Tool - Tool not available in current environment

Overall Status: PASS (5/6 tests passed)

💥 [THE END] — Illustrated by Smoke Claude fer issue #6031 🗺️

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

Playwright MCP: Page title contains "GitHub"
Cache Memory: File created and verified
Safe Inputs gh Tool: HTTP/HTTPS protocol mismatch

Overall Status: FAIL (1/3 tests failed)

📰 BREAKING: Report filed by Smoke Copilot Playwright fer issue #6031 🗺️

@pelikhan
Copy link
Copy Markdown
Collaborator

@copilot also update the code that updates the add-comment comment to display commit url

See #6031 (comment)

@github-actions
Copy link
Copy Markdown
Contributor

Latest merged PRs: Add bots field for workflow trigger authorization; Remove COPILOT_CLI_TOKEN environment variable support
GitHub MCP: ✅
File write: ✅
Cache memory: ✅
Playwright: ✅
safeinputs-gh: ❌ (command not found in env)
Overall: FAIL

🔮 The oracle has spoken through Smoke Codex fer issue #6031 🗺️

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan merged commit d89041b into main Dec 10, 2025
5 checks passed
@pelikhan pelikhan deleted the copilot/add-context-to-comments branch December 10, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants