Improve validations for explicit topologies

[PascalSenn]

No description provided.

[Copilot]

Pull request overview

This PR strengthens “explicit topology/bind” behavior in Mocha by adding centralized validation for unbound inbound routes, improving exception diagnostics, and refining transport routing/topology discovery (notably for RabbitMQ) to better honor explicit binding semantics.

Changes:

• Add a build-time validator that aggregates and formats all unbound inbound routes into a single actionable InvalidOperationException (with transport/endpoint hints), plus snapshot coverage.
• Ensure reply inbound routes are bound to the reply receive endpoint during endpoint discovery.
• Refine RabbitMQ topology discovery to avoid declaring convention publish/send exchange chains when endpoints are in explicit bind mode; improve endpoint bind-mode inheritance behavior.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/Mocha/test/Mocha.Transport.RabbitMQ.Tests/Descriptors/RabbitMQHandlerBindingTests.cs	Tighten explicit-bind failure assertions to validate the new aggregated error message content.
src/Mocha/test/Mocha.Transport.Postgres.Tests/Descriptors/PostgresHandlerBindingTests.cs	Same as RabbitMQ: validate explicit-bind failures include the expected diagnostics.
src/Mocha/test/Mocha.Tests/MessageTypes/MessageBusConfigurationValidationTests.cs	New test coverage for aggregated validation messaging + saga reply route binding behavior (with snapshots).
src/Mocha/test/Mocha.Tests/MessageTypes/snapshots/MessageBusConfigurationValidationTests.Build_Should_ReportNoTransportHint_When_HandlerRegisteredWithoutTransport.snap	Snapshot for “no transport configured” validation hint.
src/Mocha/test/Mocha.Tests/MessageTypes/snapshots/MessageBusConfigurationValidationTests.Build_Should_ReportAlsoBoundElsewhere_When_SameMessageHasAnotherBoundRoute.snap	Snapshot for “also bound elsewhere” hint.
src/Mocha/test/Mocha.Tests/MessageTypes/snapshots/MessageBusConfigurationValidationTests.Build_Should_ReportAllUnboundInboundRoutes_When_ExplicitBindLeavesHandlersUnbound.snap	Snapshot for multiple unbound routes under explicit bind mode.
src/Mocha/test/Mocha.Tests/MessageTypes/snapshots/MessageBusConfigurationValidationTests.Build_Should_NotUseReplyRouteAsDuplicate_When_NormalRouteWithSameMessageIsUnbound.snap	Snapshot ensuring reply routes don’t count as duplicates for validation hints.
src/Mocha/src/Mocha/Transport/RoutingStrategy.cs	Bind all existing reply inbound routes to the reply receive endpoint when available.
src/Mocha/src/Mocha/ThrowHelper.cs	Improve route-not-connected exceptions with richer context; remove the RabbitMQ-specific auto-bind helper.
src/Mocha/src/Mocha/MessageTypes/OutboundRoute.cs	Use the new contextual ThrowHelper.RouteEndpointNotConnected(OutboundRoute) overload.
src/Mocha/src/Mocha/MessageTypes/InboundRoute.cs	Use the new contextual ThrowHelper.RouteEndpointNotConnected(InboundRoute) overload.
src/Mocha/src/Mocha/Builder/MessageBusBuilder.Validation.cs	New centralized validator + formatted diagnostics for unbound inbound routes.
src/Mocha/src/Mocha/Builder/MessageBusBuilder.cs	Invoke the new validator during build prior to completing inbound routes.
src/Mocha/src/Mocha.Transport.RabbitMQ/Topology/RabbitMQRoutingStrategy.cs	Adjust convention topology discovery to better respect explicit bind mode; update dispatch provisioning behavior.
src/Mocha/src/Mocha.Transport.RabbitMQ/Descriptors/RabbitMQReceiveEndpointDescriptor.cs	Stop forcing receive endpoint bind mode to implicit (allow inheritance).
src/Mocha/src/Mocha.Transport.RabbitMQ/Descriptors/RabbitMQMessagingTransportDescriptor.cs	Stop defaulting queue bind mode to implicit; allow null to inherit from transport bind mode.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[PascalSenn]

LGTM