Conversation
🦋 Changeset detectedLatest commit: 6752cad The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
📝 WalkthroughWalkthroughThis pull request introduces a new profile lexicon for Hypercert account profiles, including changeset documentation, schema definitions, and a new lexicon file. The profile supports displayName, description, pronouns, website, avatar, banner, and createdAt fields. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@lexicons/app/certified/profile.json`:
- Around line 11-46: Add an optional "labels" property to the profile schema's
"properties" object: define "labels" as a union with a ref to
"com.atproto.label.defs#selfLabels" and include a brief description (e.g., "User
self-assigned labels"), ensuring it's optional (no required entry) and follows
the same schema style as existing "avatar"/"banner" union fields so tools
consuming profile.json will recognize self-labels.
Summary
Add profile lexicon for Hypercert account profiles with support for display name, description, pronouns, website, avatar, banner, and self-labels.
Note
This profile lexicon is adapted from Bluesky's actor.profile with some fields removed that aren't relevant to hypercerts profiles.
Edit by Sharfy: The removed fields are
joinedViaStarterPackandpinnedPostSummary by CodeRabbit