Add recipe delete endpoint

[dgee2]

Summary

• DELETE /api/recipe/{recipeId}, owner-only: 404 if missing, 403 if the caller isn't the owner, otherwise deletes via a single ExecuteDeleteAsync statement.
• RecipeIngredient/RecipeStep rows cascade via the FK constraints already configured with DeleteBehavior.Cascade in Epic [Epic] Core recipe model changes (DB schema) #1098 — no RecipeShare/RecipePublication/etc. tables exist yet, consistent with epic [Epic] Recipe authoring API expansion #1099's own scope note that those are Epic 6 work.

Closes #1117
Part of #1099

Test plan

• dotnet test MenuApi.Tests (unit)
• dotnet test MenuApi.Integration.Tests — owner delete (verifying both ingredient and step rows are actually gone, not just that the recipe 404s afterward), non-owner 403, missing-recipe 404

[Copilot]

Pull request overview

Adds an owner-authorized recipe deletion endpoint to MenuApi and aligns API error handling with RFC7807 Problem Details (including new 409/403 handlers). The PR also expands recipe create/update behavior to persist steps alongside ingredients, and updates unit/integration test coverage accordingly.

Changes:

• Add DELETE /api/recipe/{recipeId} (owner-only) plus service/repository support using ExecuteDeleteAsync.
• Add reusable ProblemDetailsExceptionHandler<T> and new Conflict (409) / Forbidden (403) exception handlers; update recipe duplicate-title behavior to return 409.
• Update recipe create/update flow to upsert steps; adjust validation + unit/integration tests.

Reviewed changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
backend/MenuApi/Validation/UpsertRecipeValidator.cs	Allows empty ingredients while still rejecting null ingredients/steps.
backend/MenuApi/Services/RecipeService.cs	Adds owner checks; upserts steps on create/update; adds delete orchestration.
backend/MenuApi/Services/IRecipeService.cs	Updates update signature (returns bool, takes callerId) and adds delete.
backend/MenuApi/Repositories/RecipeRepository.cs	Adds EF-translatable projection expression; uses 409 on unique conflicts; adds ExecuteDeleteAsync delete.
backend/MenuApi/Repositories/IRecipeRepository.cs	Adds repository delete contract.
backend/MenuApi/Recipes/RecipeApi.cs	Wires PUT authorization/404 handling and adds DELETE endpoint + response contracts.
backend/MenuApi/Program.cs	Registers new exception handlers (409/403).
backend/MenuApi/Exceptions/ProblemDetailsExceptionHandler.cs	New shared RFC7807 exception handler base class.
backend/MenuApi/Exceptions/ForbiddenAccessExceptionHandler.cs	New 403 Problem Details exception handler.
backend/MenuApi/Exceptions/ForbiddenAccessException.cs	New exception type for forbidden recipe access.
backend/MenuApi/Exceptions/ConflictExceptionHandler.cs	New 409 Problem Details exception handler.
backend/MenuApi/Exceptions/ConflictException.cs	New exception type for conflicts (e.g., duplicate recipe title).
backend/MenuApi/Exceptions/BusinessValidationExceptionHandler.cs	Refactors 422 handler to use the shared base.
backend/MenuApi.Tests/Validation/UpsertRecipeValidatorTests.cs	Updates validator expectations: empty ingredients now pass, null fails.
backend/MenuApi.Tests/Services/RecipeServiceTests.cs	Adds owner/not-found cases; adds delete tests; asserts steps upsert calls.
backend/MenuApi.Tests/Controllers/RecipeApiTests.cs	Updates PUT tests for auth + result shapes; adds DELETE tests.
backend/MenuApi.Integration.Tests/RecipeIntegrationTests.cs	Updates duplicate-title expectations from 422 to 409.
backend/MenuApi.Integration.Tests/RecipeDeleteIntegrationTests.cs	New end-to-end tests for owner delete cascade, non-owner 403, missing 404.
backend/MenuApi.Integration.Tests/RecipeCreateUpdateIntegrationTests.cs	New end-to-end tests covering steps on create/update and owner/not-found cases.
backend/MenuApi.Integration.Tests/Factory/TestDatabaseSeeder.cs	Adds helper to count steps for cascade verification.

---

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud