Feature: Extract Social microservice from monolith (MSA step 6) - #47
Merged
Conversation
…locks Own social graph data in a dedicated service with internal access APIs and its own Postgres schema.
Route user detach and related social checks through ISocialClient instead of in-process friendship and block domains.
…h social-service Move friendship and block checks off IMonolithAccessClient onto ISocialClient so callers no longer depend on Api for social graph data.
Delete friendships, friend requests, and user-blocks from the monolith and drop legacy public tables without copying rows.
…Api.Tests Port friendship and block integration and unit tests onto the social-service harness, and fake social access in Api.Tests.
Include Social and Social.Tests in the solution, default test runs, and gateway routing, and document the social-service boundary.
Stop reading the JWT caller for friendship and block checks so invitee and background paths work, run detach in a transaction, and map detach failures to domain exceptions in Api.
Move the former Api.Tests friendship, request, block, and internal-access coverage onto the social-service factory and Testcontainers harness.
Run dependent service tests when Social changes, and list Social in publish and full-suite docs.
Order posts and user comments newest-first with Id tie-breakers, keep post comment trees chronological, and cover the behavior in integration tests.
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
socialPostgres schema) with public/api/friendships*and/api/users/blocksroutes and internal validate/detach APIs.ISocialClient/ social-service, and have the monolith delegate user-deletion detach to the new service.Social.Testsand wire Social into Compose, nginx, CI, and MSA docs.Test plan
./scripts/ci/docker-test.sh(or CIsocial-tests/ full ladder) passesdocker compose down -v && docker compose up --buildboots social-service and routes/api/friendships*and/api/users/blocksvia nginx