feat: adds auth to social api#28919
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 5c5c16e. Configure here.
…k-mobile into TSA-395-social-api-auth
…k-mobile into TSA-395-social-api-auth
…k-mobile into TSA-395-social-api-auth
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: Performance Test Selection: |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
|
✅ E2E Fixture Validation — Schema is up to date |




Description
Adds
AuthenticationControllerto thesocial-service-messengerto be used by the core social package.Changelog
CHANGELOG entry: no-changelog
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Medium Risk
Adds delegation for
AuthenticationController:getBearerTokeninto the SocialService messenger and upgrades@metamask/social-controllersto^1.0.0, which could affect how auth tokens are fetched and used by social API calls.Overview
Enables SocialService code to request authentication by delegating
AuthenticationController:getBearerTokenthroughgetSocialServiceMessenger, so the social controllers can fetch a bearer token via the root messenger.Upgrades
@metamask/social-controllersfrom^0.2.0to^1.0.0(and updatesyarn.lockaccordingly), pulling in new transitive dependencies (notably@metamask/profile-sync-controller).Reviewed by Cursor Bugbot for commit a42cf50. Bugbot is set up for automated code reviews on this repo. Configure here.