Make `TaskOrchestrationContext.LoggerFactory` public

[BalassaMarton]

// TaskOrchestrationContext.cs
protected abstract ILoggerFactory LoggerFactory { get; }

This is a problem in scenarios where the context is wrapped by another context implementation (concrete use case for me is implementing a cancellable orchestrator which wraps the original context). My current workaround is to override the methods that actually use this property (namely the CreateReplaySafeLogger overloads), but that's a dependency on implementation details. The proper solution in this case would be to just return the LoggerFactory of the wrapped context, if it was made public. LoggerFactory is the only member of TaskOrchestrationContext that is not public.

[github-actions]

Issue Triage

Classification: triage/needs-human-verification

Reasoning:
The request to change TaskOrchestrationContext.LoggerFactory from protected abstract to public abstract is technically a source-breaking change. In C#, all override implementations must match the base member accessibility, so any derived class with protected override ILoggerFactory LoggerFactory would fail to compile after this change. While the fix itself is trivial (~3 lines), it requires a maintainer decision on whether this API surface change is acceptable given the breaking change implications.

Automated fix candidate: No

• Requires maintainer approval for a public API / breaking change decision
• While the code change is small, the impact on downstream consumers who derive from TaskOrchestrationContext needs human evaluation

[BalassaMarton]

If the breaking change is not possible, a new interface could be added that makes the logger factory accessible (like IInfrastructure<T> in other libraries).