Skip to content

Feat: Messenger & Friends & Relationships & Blocking#8

Merged
Diddyy merged 8 commits intonitrodevco:devfrom
Dippys:feature/messenger
Feb 10, 2026
Merged

Feat: Messenger & Friends & Relationships & Blocking#8
Diddyy merged 8 commits intonitrodevco:devfrom
Dippys:feature/messenger

Conversation

@Dippys
Copy link
Contributor

@Dippys Dippys commented Feb 8, 2026

Everything related to friendship & messenger & Relationship & Blocking implemented here with migrations.

Dippys and others added 6 commits February 8, 2026 18:44
MessengerGrain:
- Add ILogger, replace 10 bare catch blocks with logged exceptions
- Replace .Ignore() with LogAndForget helper (logs on failure)
- Parallel online status checks on activation, search, and accept
- Timer-based delivered flag batching (RegisterGrainTimer + flush on deactivate)
- Batch RemoveFriendsAsync into single DB query and single composer call
- Hoist mySummary before AcceptFriendRequestsAsync loop
- Tracked delete for atomic request removal + friendship creation
- Bound session message history per conversation
- Remove hardcoded limits: searchLimit, pageSize, maxIgnoreCapacity
  now passed from handlers via IConfiguration

Handlers:
- Add missing PlayerId guard clause to GetHeightMapMessageHandler
- Add IConfiguration to HabboSearchMessageHandler, GetMessengerHistoryMessageHandler,
  IgnoreUserMessageHandler for configurable limits
- Delete duplicate no-op GetCreditsInfoMessageHandler

Primitives:
- Remove unused MESSENGER_STATE constant from OrleansStateNames
- Add searchLimit, pageSize, maxIgnoreCapacity params to IMessengerGrain

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Diddyy
Copy link
Collaborator

Diddyy commented Feb 9, 2026

Done a review, fixed a few things. LGTM now.

@Diddyy Diddyy changed the base branch from main to dev February 10, 2026 14:31
@Diddyy Diddyy merged commit 0282c90 into nitrodevco:dev Feb 10, 2026
3 checks passed
@Diddyy Diddyy deleted the feature/messenger branch February 10, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants