Skip to content

refactor(di): split module service composition - #241

Open
gbaudrit wants to merge 7 commits into
mainfrom
refactor/issue-240-di-composition
Open

refactor(di): split module service composition#241
gbaudrit wants to merge 7 commits into
mainfrom
refactor/issue-240-di-composition

Conversation

@gbaudrit

@gbaudrit gbaudrit commented Sep 10, 2026

Copy link
Copy Markdown
Owner

Summary

Closes #240.

Split the platform, Web Console, Web host, and Workplace host dependency-injection composition into focused, module-owned extension methods while preserving the public AddAgentstration compatibility overload.

Changes

  • added a cohesive AgentstrationServiceRegistrationOptions entry point and ten focused platform composition extensions
  • extracted Web Console, Web host, and Workplace host registrations from broad roots into owned extensions
  • removed the duplicate IRuntimeRunExecutionScope descriptor and made fallback/replacement semantics explicit with TryAdd and Replace
  • added descriptor-cardinality, provider-selection, ValidateOnBuild, host lifecycle, and architecture guard tests
  • documented the composition ownership map and registration policy
  • added CAR-0240 for the ai-defect analysis and prevention record

Validation

  • git diff --check
  • GitHub Actions CI run 507: formatting, full build, MSTest suite, Source Registry smoke test, PostgreSQL migration/restart and concurrency validation, container build, and Windows host lifecycle — passed
  • Dependency Review run 381 — passed
  • Documentation run 337 — passed
  • CodeQL run 503 — passed

Breaking changes

None. The existing AddAgentstration overload remains available and delegates to the new options-based façade.

@gbaudrit gbaudrit added the ai-defect Structural, architectural, or code-quality defect introduced by AI. label Sep 10, 2026 — with ChatGPT Codex Connector
@gbaudrit
gbaudrit marked this pull request as ready for review September 10, 2026 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-defect Structural, architectural, or code-quality defect introduced by AI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Split service registration into module-owned composition extensions

1 participant