Skip to content

Add check run logs#8506

Merged
alexr00 merged 2 commits intomainfrom
alexr00/outrageous-snail
Feb 11, 2026
Merged

Add check run logs#8506
alexr00 merged 2 commits intomainfrom
alexr00/outrageous-snail

Conversation

@alexr00
Copy link
Copy Markdown
Member

@alexr00 alexr00 commented Feb 11, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 11, 2026 09:13
@alexr00 alexr00 enabled auto-merge (squash) February 11, 2026 09:13
@alexr00 alexr00 self-assigned this Feb 11, 2026
@vs-code-engineering vs-code-engineering Bot added this to the February 2026 milestone Feb 11, 2026
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

Adds “view check run logs” support to the PR experience by wiring check run identifiers through GraphQL/models, exposing a new checkrunlog: document, and adding a webview button to open logs for failed checks.

Changes:

  • Add a “View Logs” icon button for failed check runs in the PR status checks webview and route the request to the extension host.
  • Introduce a checkrunlog: text document content provider and a corresponding URI scheme to display fetched logs in an editor.
  • Plumb databaseId/isCheckRun through GraphQL and status check models, and extend the LM tool to include logs for failed checks.

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
webviews/editorWebview/index.css Layout and hover styling for the new status check log action UI.
webviews/components/merge.tsx Adds “View Logs” action for failed check runs and calls into the webview context.
webviews/components/icon.tsx Exposes a new outputIcon for the log button.
webviews/common/context.tsx Adds viewCheckLogs message plumbing from webview to extension.
src/view/checkRunLogContentProvider.ts New content provider to resolve checkrunlog: URIs into log text.
src/lm/tools/activePullRequestTool.ts Extends LM tool output to include failed check run logs.
src/github/queriesShared.gql Adds databaseId to CheckRun selections for checks queries.
src/github/pullRequestOverview.ts Handles the new pr.view-check-logs message and opens the checkrunlog: document.
src/github/interface.ts Extends PullRequestCheckStatus with databaseId and isCheckRun.
src/github/graphql.ts Adds databaseId to the generated CheckRun type.
src/github/githubRepository.ts Implements getCheckRunLogs and maps check run fields into status check models.
src/extension.ts Registers the new Schemes.CheckRunLog text document content provider.
src/common/uri.ts Adds Schemes.CheckRunLog to the scheme enum.
resources/icons/codicons/output.svg New icon asset for the “View Logs” button.

Comment thread src/github/pullRequestOverview.ts
Comment thread webviews/common/context.tsx Outdated
Comment thread webviews/editorWebview/index.css Outdated
Comment thread src/github/githubRepository.ts Outdated
Comment thread src/github/githubRepository.ts
Comment thread src/lm/tools/activePullRequestTool.ts
Comment thread src/github/pullRequestOverview.ts
Comment thread src/view/checkRunLogContentProvider.ts
@alexr00 alexr00 merged commit 148c313 into main Feb 11, 2026
6 checks passed
@alexr00 alexr00 deleted the alexr00/outrageous-snail branch February 11, 2026 09:49
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