feat: align ActivityPub article previews for Mastodon #226
No reviewers
Labels
No labels
activitypub
admin
api
architecture
articles
auth
bug
cleanup
cli
comments
compatibility
config
contacts
content
database
deployment
design
dev-env
diagnostics
docs
documentation
email
enhancement
feature
federation
feed
health
homepage
implementation
integration
logging
mastodon
media
navigation
npm-packages
openapi
opengraph
packages
passkeys
preview
priority:high
priority:low
priority:medium
production
proof
proxy
public-routes
public-ui
release
safety
slug-api
slug-auth
slug-core
slug-federation
slug-media
slug-template
social
sources
status:active
status:canceled
status:done
status:inprogress
status:waiting
syndication
tailwind
template
test
web
webauthn
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
erik/slugkit!226
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "feat/task-e11e34ff-mastodon-article-previews"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
contentMap.envia FedifyLanguageString.name/summaryand image attachments in ActivityPub objects so Mastodon renders a link preview card instead of duplicated status text/media gallery.contentMap, URL-only content, and absence of article image attachment/name/summary.Verification
Task: #task-e11e34ff
Review summary
Approved.
Acceptance criteria
og:title,og:description,og:url,og:type, andog:imagewhen available. Evidence: PR #225 already merged this template support; existing public metadata tests continue passing in this PR.og:imageandtwitter:imageare absolute reachable URLs. Evidence: PR #225 route tests assert absolute/mediaimage URLs; unchanged and passing here.twitter:cardusessummary_large_imagewith image andsummaryotherwise. Evidence: PR #225 route tests cover both cases; unchanged and passing here.buildActivityPubPostContent()now returns only the post URL anchor for articles; federation route test asserts URL-only content.contentMap.en. Evidence:LanguageString(content, "en")is emitted viacontents; test assertscontentMap.enfor article and note objects.attachment,name, orsummaryon the article object.contentMap, and absence of article image attachment. Evidence: public metadata tests from PR #225 remain passing; this PR adds federation assertions forcontentMapand absent article attachment.Blocking issues
Warnings
Verification reviewed
./scripts/pre-pr.shpassed locally.git diff --check main...HEAD.Final verdict
Approved. Ready for human merge approval.