Fix DeleteContributorService passing raw int instead of ContributorId to GetByIdAsync (#1000)

Copilot · ardalis · web-flow · commit 0eed97fedeb7 · 2026-03-06T14:23:52.000-05:00
* Initial plan

* Fix: pass ContributorId instead of int to GetByIdAsync in DeleteContributorService

Co-authored-by: ardalis &lt;782127+ardalis@users.noreply.github.com&gt;

---------

Co-authored-by: copilot-swe-agent[bot] &lt;198982749+Copilot@users.noreply.github.com&gt;
Co-authored-by: ardalis &lt;782127+ardalis@users.noreply.github.com&gt;
diff --git a/src/Clean.Architecture.Core/Services/DeleteContributorService.cs b/src/Clean.Architecture.Core/Services/DeleteContributorService.cs
@@ -18,7 +18,7 @@ public class DeleteContributorService(IRepository<Contributor> _repository,
   public async ValueTask<Result> DeleteContributor(ContributorId contributorId)
   {
     _logger.LogInformation("Deleting Contributor {contributorId}", contributorId);
-    Contributor? aggregateToDelete = await _repository.GetByIdAsync(contributorId.Value);
+    Contributor? aggregateToDelete = await _repository.GetByIdAsync(contributorId);
     if (aggregateToDelete == null) return Result.NotFound();
 
     await _repository.DeleteAsync(aggregateToDelete);

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ public class DeleteContributorService(IRepository<Contributor> _repository,`
`18`	`18`	`public async ValueTask<Result> DeleteContributor(ContributorId contributorId)`
`19`	`19`	`{`
`20`	`20`	`_logger.LogInformation("Deleting Contributor {contributorId}", contributorId);`
`21`		`- Contributor? aggregateToDelete = await _repository.GetByIdAsync(contributorId.Value);`
	`21`	`+ Contributor? aggregateToDelete = await _repository.GetByIdAsync(contributorId);`
`22`	`22`	`if (aggregateToDelete == null) return Result.NotFound();`
`23`	`23`
`24`	`24`	`await _repository.DeleteAsync(aggregateToDelete);`