Skip to content

fix: Bluesky link card repeats the post caption as the embed description#1096

Open
Alexia-Soare wants to merge 1 commit into
developmentfrom
fix/bluesky-card
Open

fix: Bluesky link card repeats the post caption as the embed description#1096
Alexia-Soare wants to merge 1 commit into
developmentfrom
fix/bluesky-card

Conversation

@Alexia-Soare

@Alexia-Soare Alexia-Soare commented Jul 15, 2026

Copy link
Copy Markdown

Fixes Codeinwp/tweet-old-post-pro#676

Changes

  • class-rop-bluesky-api.php — the external embed (app.bsky.embed.external) description is now empty instead of duplicating $post['content'], which is already sent as the post text. Bluesky renders the card as title + domain, like any page without an OG description. Covers both call sites: link posts and image posts with a URL. The description key stays present (empty string) since the AT Protocol lexicon requires it.
  • Excerpts were deliberately not used: WordPress auto-generates missing excerpts from the post content, which would reintroduce the duplication for most posts.

Tests

New bluesky-card PHPUnit suite captures the outbound createRecord request (pre_http_request, no Bluesky account needed): fails on the old code, passes with the fix.

@Alexia-Soare Alexia-Soare added the pr-checklist-skip Allow this Pull Request to skip checklist. label Jul 15, 2026
@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Jul 15, 2026
@pirate-bot

pirate-bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Plugin build for 47ddc01 is ready 🛎️!

Leave the external embed description empty instead of copying the post
content already sent as the record text. Applies to both link posts and
image posts that carry a URL.

Fixes Codeinwp/tweet-old-post-pro#676

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants