Incorrect RelatedEntityCollector Registration in Elsa Docs

[McahitKutsal]

In the Elsa documentation about Retention, the usage example for the RelatedEntityCollector is shown as follows:

Services.AddScoped<IRelatedEntityCollector<WorkflowInstanceData>, WorkflowInstanceDataRecordCollector>();

However, it should actually be registered like this:

Services.AddScoped<IRelatedEntityCollector, WorkflowInstanceDataRecordCollector>();

[sfmskywalker]

Thanks, this looks like a real docs inconsistency.

From the current retention code:

• the cleanup job resolves both IRelatedEntityCollector and IRelatedEntityCollector<>
• the generic interface IRelatedEntityCollector<TEntity> extends the non-generic IRelatedEntityCollector
• the built-in registrations in RetentionFeature use the non-generic service type

Relevant source:

• elsa-extensions/src/modules/retention/Elsa.Retention/Jobs/CleanupJob.cs
• elsa-extensions/src/modules/retention/Elsa.Retention/Contracts/IRelatedEntityCollector.cs
• elsa-extensions/src/modules/retention/Elsa.Retention/Feature/RetentionFeature.cs
• elsa-gitbook/optimize/retention.md

So the guide example in elsa-gitbook/optimize/retention.md does look inconsistent with the module’s own registration pattern. Your suggested registration is the one that matches the built-in feature registrations.

The best next step here is probably a docs PR against the guide.