fix(desktop): remove Buzz entity link previews - #6512
Merged
Merged
Conversation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Category: improvement
User Impact: Buzz-native project, repository, issue, and pull request links now appear once as compact inline chips, with their details available on hover.
Problem: Buzz-native entity links rendered both an inline chip and a standalone preview card, repeating the same metadata and adding visual noise to conversations. Solution: Exclude Buzz-native links from the shared standalone-preview extractor while leaving entity parsing intact for chip tooltips and preserving external web previews and attachment cards.
File changes
desktop/src/shared/lib/linkPreview.ts
Stops Buzz-native preview candidates after parsing, including same-relay git clone URLs that normalize to repository entities, while allowing external URLs through the existing snapshot path.
desktop/src/shared/lib/linkPreview.test.mjs
Covers project, repository, issue, pull request, markdown-labeled, same-relay clone, and mixed external-link extraction behavior.
desktop/src/shared/ui/markdown/useMessageLinkPreviews.test.mjs
Confirms sent messages no longer merge a standalone Buzz entity card while external sender snapshots still render.
Reproduction steps
buzz://project,buzz://repo,buzz://issue, orbuzz://prlink./git/<owner>/<repo>clone URL and confirm it uses the repository chip without a duplicate card.Screenshots
After — rich metadata stays available on hover
Verification
At commit
3fa74cdd342ac1f6721b7d56a7f111af31e0e6e9:cd desktop && pnpm exec tsc --noEmit— passedgit diff --check origin/main...HEAD— passed