Skip to content

Use neutral badges for automated PR comments - #15882

Merged
mtsgrd merged 1 commit into
masterfrom
codex/neutral-agent-comments
Sep 9, 2026
Merged

mtsgrd merged 1 commit into
masterfrom
codex/neutral-agent-comments

Conversation

@mtsgrd

@mtsgrd mtsgrd commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Replace purple Agent badges with the existing light-gray variant and remove the purple card background and border tint. Automated comments remain identifiable by a single Agent badge, with circular avatars and standard card styling.

Validation: Lite typecheck, oxlint, Knip, formatting checks, and inspection of React Compiler output for the author component. The PR comment feed is not covered by the screenshot catalogue; screenshots have not been captured.

@mtsgrd
mtsgrd marked this pull request as ready for review September 9, 2026 22:30
Copilot AI lite review requested due to automatic review settings September 9, 2026 22:30
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

This pull request changes Lite's UI, so it is labelled screenshots needed. Before/after screenshots make it reviewable without checking the branch out.

Attach them however you like — drag images straight into a comment, or have an agent capture them for you (the lite-screenshots skill in this repository does it against seeded fixtures).

Swap the label for screenshots once they are posted, or remove it if this change is not visual. Either sticks — this is asked once per pull request, so later pushes will not put it back.

cc @PavelLaptev — visual change.

@github-actions github-actions Bot added the screenshots needed Before/after UI screenshots are wanted on this PR label Sep 9, 2026
@mtsgrd

mtsgrd commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Ad hoc before/after captures of the running Lite app, showing the same Copilot review on PR #15874. Captured with this branch unapplied/applied. This state is not covered by the screenshot catalogue.

Before After
Purple Agent badge and tinted card Neutral Agent badge and standard card

mtsgrd added a commit that referenced this pull request Sep 9, 2026
@mtsgrd mtsgrd added screenshots Before/after UI screenshots are attached and removed screenshots needed Before/after UI screenshots are wanted on this PR labels Sep 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.

🟢 Approval recommended

No unresolved issues were identified that would block approval.

Pull request overview

Updates Lite PR comment styling so automated comments use neutral badges and standard card appearance.

Changes:

  • Uses the existing lightGray Agent badge consistently.
  • Removes purple agent-specific card styling.
  • Preserves circular avatars and compact thread layout.
File summaries
File Description
apps/lite/ui/src/routes/project/$id/workspace/PullRequestComments.tsx Updated as part of this pull request.
apps/lite/ui/src/routes/project/$id/workspace/PullRequestComments.module.css Updated as part of this pull request.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mtsgrd
mtsgrd merged commit d1ccd21 into master Sep 9, 2026
45 checks passed
@mtsgrd
mtsgrd deleted the codex/neutral-agent-comments branch September 9, 2026 22:33

This branch was previously deployed

1 inactive deployment
Preview — 8a00ee7e Deployed Sep 9, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@gitbutler/lite screenshots Before/after UI screenshots are attached

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants