Skip to content

Polish message author labels#1045

Merged
SuuBro merged 1 commit into
masterfrom
fix/message-author-display
Jul 23, 2026
Merged

Polish message author labels#1045
SuuBro merged 1 commit into
masterfrom
fix/message-author-display

Conversation

@SuuBro

@SuuBro SuuBro commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • align prompt-author badges with the approved mock, including neutral initials and centered accessory-bearing agent sprites
  • make resolvable agent labels link to their originating session
  • remove duplicate restart-prompt attribution
  • pin badge sizing, clipping, mobile geometry, sprite alignment, and navigation coverage

Tests

  • npm run check
  • npm run test:unit — 968 files passed, 4 skipped; 8,438 tests passed, 26 skipped
  • npm run test:browser — 672 passed, 9 skipped; browser budget passed
  • npm run test:e2e — all four phases passed; Docker unavailable, so the Docker-gated sandbox path self-skipped

🤖 Generated with Bobbit

@greptile-apps

greptile-apps Bot commented Jul 23, 2026

Copy link
Copy Markdown

Greptile Summary

Polish message-author presentation and attribution behavior.

  • Add neutral initials and revised badge geometry for user and system prompts.
  • Center static agent sprites and link resolvable agent labels to their originating sessions.
  • Remove duplicate system attribution from restart continuation prompts.
  • Expand DOM, browser, sprite, and prompt-lifecycle coverage.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failures remain.

Important Files Changed

Filename Overview
src/ui/components/Messages.ts Renders revised author badges and links agent badges when an originating session is resolvable.
src/ui/bobbit-render.ts Adds opt-in centering for static author sprites while preserving sidebar alignment behavior.
src/ui/app.css Updates author badge sizing, clipping, alignment, initials, and interactive link states.
src/server/agent/session-manager.ts Removes redundant inline system attribution from restart continuation text.

Reviews (4): Last reviewed commit: "Polish message author labels" | Re-trigger Greptile

@SuuBro
SuuBro force-pushed the fix/message-author-display branch 2 times, most recently from d7f374c to 9dc9939 Compare July 23, 2026 09:05
Co-authored-by: bobbit-ai <bobbit@bobbit.ai>
Co-Authored-By: Bobbit (Gpt 5.6 Sol) <bobbit@bobbit.ai>
@SuuBro
SuuBro force-pushed the fix/message-author-display branch from 9dc9939 to 972e374 Compare July 23, 2026 09:14
@SuuBro

SuuBro commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator Author

@greptileai

@SuuBro
SuuBro merged commit 2409688 into master Jul 23, 2026
6 checks passed
@SuuBro
SuuBro deleted the fix/message-author-display branch July 23, 2026 10:01
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.

1 participant