Skip to content

Add verified badge to PR timeline#8834

Open
alexr00 wants to merge 2 commits into
mainfrom
alexr00/curly-canid
Open

Add verified badge to PR timeline#8834
alexr00 wants to merge 2 commits into
mainfrom
alexr00/curly-canid

Conversation

@alexr00

@alexr00 alexr00 commented Jul 9, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 9, 2026 10:52
@alexr00 alexr00 enabled auto-merge (squash) July 9, 2026 10:52
@alexr00 alexr00 self-assigned this Jul 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a commit signature verification badge (with details popover) to the PR timeline in the editor webview, plumbing verification data from the GitHub GraphQL API through the extension’s timeline event model to the React UI.

Changes:

  • Fetch commit signature data via GraphQL and map it into a new CommitEvent.verification field.
  • Render a new CommitVerificationBadge in the timeline commit row.
  • Add CSS for the verified/unverified pill and popover layout.
Show a summary per file
File Description
webviews/editorWebview/index.css Styles for the verification pill and details popover.
webviews/components/timeline.tsx Wires the new badge into commit timeline entries.
webviews/components/commitVerification.tsx New UI component for verified/unverified pill + popover.
src/github/utils.ts Maps GraphQL commit signature data into CommitEvent.verification.
src/github/queriesShared.gql Extends commit fragment to request signature fields.
src/github/graphql.ts Adds signature typing to the commit GraphQL model.
src/common/timelineEvent.ts Adds CommitVerification and optional CommitEvent.verification.

Review details

  • Files reviewed: 7/7 changed files
  • Comments generated: 3
  • Review effort level: Low

Comment thread webviews/components/commitVerification.tsx
Comment thread webviews/components/commitVerification.tsx
Comment thread src/github/utils.ts
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