Repository navigation
MeshWeaver.Social leaves core — the SocialMedia repo is the master and the registry serves the bundle - #2508
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Removes the MeshWeaver.Social module (and its associated tests/solution wiring) from the core repo now that Systemorph/MeshWeaver.SocialMedia is the source of truth and the module bundle is served via the registry.
Changes:
- Deleted the
src/MeshWeaver.Socialproject and the dedicatedtest/MeshWeaver.Social.Testproject. - Removed Monolith/Orleans social integration tests and dropped
MeshWeaver.Socialproject references from remaining test projects. - Updated solution + build/shard configuration to stop including/building the removed social projects.
Reviewed changes
Copilot reviewed 43 out of 43 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| test/MeshWeaver.Social.Test/ScheduledPublishIdentityTest.cs | Removed (Social test project relocation cleanup). |
| test/MeshWeaver.Social.Test/NodeContentTypeRoundTripTest.cs | Removed (Social test project relocation cleanup). |
| test/MeshWeaver.Social.Test/MeshWeaver.Social.Test.csproj | Removed (Social test project relocation cleanup). |
| test/MeshWeaver.Social.Test/LinkedInPublisherEngagementTest.cs | Removed (Social test project relocation cleanup). |
| test/MeshWeaver.Social.Test/LinkedInPostsApiTest.cs | Removed (Social test project relocation cleanup). |
| test/MeshWeaver.Social.Test/LinkedInIdentitySyncTest.cs | Removed (Social test project relocation cleanup). |
| test/MeshWeaver.Social.Test/LinkedInConnectScopeTest.cs | Removed (Social test project relocation cleanup). |
| test/MeshWeaver.Social.Test/LinkedInAnalyticsTest.cs | Removed (Social test project relocation cleanup). |
| test/MeshWeaver.Hosting.Orleans.Test/OrleansScheduledPostTest.cs | Removed (Orleans lane social scheduling test moved out with module). |
| test/MeshWeaver.Hosting.Orleans.Test/MeshWeaver.Hosting.Orleans.Test.csproj | Dropped MeshWeaver.Social project reference. |
| test/MeshWeaver.Hosting.Monolith.Test/SocialTestNodeTypes.cs | Removed (Monolith social test fixture wiring moved out with module). |
| test/MeshWeaver.Hosting.Monolith.Test/SocialModuleContributionTest.cs | Removed (module attribute/contribution tests moved out with module). |
| test/MeshWeaver.Hosting.Monolith.Test/SocialHistoryMatchTest.cs | Removed (history matching tests moved out with module). |
| test/MeshWeaver.Hosting.Monolith.Test/ScheduledPublishIdentityTest.cs | Removed (Monolith timed-publish identity tests moved out with module). |
| test/MeshWeaver.Hosting.Monolith.Test/ScheduledPostWatcherTest.cs | Removed (Monolith scheduling watcher integration tests moved out with module). |
| test/MeshWeaver.Hosting.Monolith.Test/MeshWeaver.Hosting.Monolith.Test.csproj | Dropped MeshWeaver.Social project reference. |
| test/MeshWeaver.Hosting.Monolith.Test/LinkedInPullActionsTest.cs | Removed (compile-guard/in-mesh social code test moved out with module). |
| test/MeshWeaver.Hosting.Monolith.Test/LinkedInPublishServiceTest.cs | Removed (publish integration tests moved out with module). |
| test/MeshWeaver.Hosting.Monolith.Test/LinkedInPublishAccessTest.cs | Removed (publish access-control tests moved out with module). |
| src/MeshWeaver.Social/SocialPostMenuProvider.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/SocialModuleAttribute.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/SocialHistoryMatch.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/ScheduledSocialPublishHandler.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/README.md | Removed (module relocation note no longer applies in core repo). |
| src/MeshWeaver.Social/PlatformCredential.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/NodeContentJson.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/MeshWeaver.Social.csproj | Removed (module project removed from core). |
| src/MeshWeaver.Social/LinkedInPublishEndpoints.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/LinkedInPostsApi.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/LinkedInIdentitySync.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/LinkedInCredentialMenuProvider.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/LinkedInAnalytics.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/IPlatformPublisher.cs | Removed (module sources relocated out of core). |
| src/MeshWeaver.Social/ApiCredentialNodeType.cs | Removed (module sources relocated out of core). |
| MeshWeaver.slnx | Removed Social project + Social test project from the solution. |
| memex/MeshModulesPublish.targets | Removed Social closure-lane entry and updated commentary about relocation. |
| .github/scripts/shard-assign.sh | Removed Social test project weight entry from shard assignment. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Contributor
Test Results (shard 2)1 084 tests 892 ✅ 5m 49s ⏱️ Results for commit e080fcd. ♻️ This comment has been updated with latest results. |
Contributor
Test Results (shard 0)1 308 tests 1 308 ✅ 6m 27s ⏱️ Results for commit e080fcd. ♻️ This comment has been updated with latest results. |
Contributor
Test Results (shard 3)1 080 tests 1 080 ✅ 12m 38s ⏱️ Results for commit e080fcd. ♻️ This comment has been updated with latest results. |
Contributor
Test Results (shard 4) 8 files 8 suites 10m 17s ⏱️ Results for commit e080fcd. ♻️ This comment has been updated with latest results. |
Contributor
Test Results (shard 5)1 583 tests 1 582 ✅ 8m 46s ⏱️ Results for commit e080fcd. ♻️ This comment has been updated with latest results. |
Contributor
Test Results (shard 1)740 tests 736 ✅ 6m 51s ⏱️ Results for commit e080fcd. ♻️ This comment has been updated with latest results. |
Contributor
Test Results 44 files 44 suites 50m 50s ⏱️ Results for commit e080fcd. ♻️ This comment has been updated with latest results. |
…d the registry serves the bundle The relocation that src/MeshWeaver.Social/README.md tracked completes: the module's sources, its own suite and the Monolith-hosted social tests (LinkedIn publish/pull/access, the scheduler watcher, publish identity, history matching, the module contribution, the Orleans scheduler test) all live in Systemorph/MeshWeaver.SocialMedia now, its CI packs and hands the bundle to the registry, and the portal hosts in MeshWeaver.Plugins declare MeshWeaver.Social under Modules:Required — a rollout that lost the module stalls on readiness instead of shipping without it. So the project, its tests and the closure-lane row go; the lane comment says where it went. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01X8RtDNQGywnbbY2T5j5T28
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01X8RtDNQGywnbbY2T5j5T28
rbuergi
force-pushed
the
feat/social-leaves-core
branch
from
August 27, 2026 14:42
7de8805 to
e080fcd
Compare
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.
What
The relocation
src/MeshWeaver.Social/README.mdtracked completes ("projects removed from core"):src/MeshWeaver.Social,test/MeshWeaver.Social.Test, the seven Monolith-hosted social tests +SocialTestNodeTypes,OrleansScheduledPostTest, the closure-lane row inmemex/MeshModulesPublish.targets, slnx + shard entries.SocialMedia@1.1.2published and confirmed served), and the portal hosts declareMeshWeaver.SocialunderModules:Requiredwith the SocialMedia repo as a Store package source (Plugins#751, incl. theSocialRequiredCutoverTestcontract).Order (all landed)
SocialMedia #86 → #87 (registry lane;
REGISTRY_PUBLISH_TOKENprovisioned from the memex-cloud vault value) → #88 (tests) → Plugins#751 (source + Required) → this.Verification
MW_ALLOW_SOLUTION_BUILD=1 dotnet build MeshWeaver.slnx -c Release -p:CIRun=true -warnaserrorafter the rebase: 0 errors.MeshWeaver.Documentation.Test: green. Three-dot diff: 43 files, +4/−7172, deletion only.No What's New entry: the module keeps shipping (now from the registry); no user-visible change.
🤖 Generated with Claude Code
https://claude.ai/code/session_01X8RtDNQGywnbbY2T5j5T28